[U-Boot] [PATCH] usb: storage: Show number of storage devices detected for DM_USB

2016-12-21 Thread Michal Simek
By enabling DM_USB information about number of storage devices
was lost.
Get this information back simply by printing number of devices detected
via BLK uclass.

For example:
scanning bus 0 for devices... 7 USB Device(s) found
   scanning usb for storage devices... 3 Storage Device(s) found
   scanning usb for ethernet devices... 0 Ethernet Device(s) found

Signed-off-by: Michal Simek 
---

 cmd/usb.c| 2 +-
 common/usb_storage.c | 8 
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/cmd/usb.c b/cmd/usb.c
index 455127c844b9..4fa456e31834 100644
--- a/cmd/usb.c
+++ b/cmd/usb.c
@@ -571,11 +571,11 @@ static void do_usb_start(void)
return;
 
/* Driver model will probe the devices as they are found */
-#ifndef CONFIG_DM_USB
 # ifdef CONFIG_USB_STORAGE
/* try to recognize storage devices immediately */
usb_stor_curr_dev = usb_stor_scan(1);
 # endif
+#ifndef CONFIG_DM_USB
 # ifdef CONFIG_USB_KEYBOARD
drv_usb_kbd_init();
 # endif
diff --git a/common/usb_storage.c b/common/usb_storage.c
index 0345aa22eff5..b524a15e2bf9 100644
--- a/common/usb_storage.c
+++ b/common/usb_storage.c
@@ -303,7 +303,6 @@ void usb_stor_reset(void)
usb_max_devs = 0;
 }
 
-#ifndef CONFIG_DM_USB
 
/***
  * scan the usb and reports device info
  * to the user if mode = 1
@@ -311,11 +310,12 @@ void usb_stor_reset(void)
  */
 int usb_stor_scan(int mode)
 {
-   unsigned char i;
-
if (mode == 1)
printf("   scanning usb for storage devices... ");
 
+#ifndef CONFIG_DM_USB
+   unsigned char i;
+
usb_disable_asynch(1); /* asynch transfer not allowed */
 
usb_stor_reset();
@@ -329,12 +329,12 @@ int usb_stor_scan(int mode)
} /* for */
 
usb_disable_asynch(0); /* asynch transfer allowed */
+#endif
printf("%d Storage Device(s) found\n", usb_max_devs);
if (usb_max_devs > 0)
return 0;
return -1;
 }
-#endif
 
 static int usb_stor_irq(struct usb_device *dev)
 {
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 1/3] usb: ehci-mx6: implement ofdata_to_platdata

2016-12-21 Thread Peng Fan
Implement ofdata_to_platdata to set the type to host or device.
 - Check "dr-mode" property.
 - If there is no "dr-mode", check phy_ctrl for i.MX6
   and phy_status for i.MX7

Signed-off-by: Peng Fan 
Cc: Marek Vasut 
Cc: Simon Glass 
Cc: Stefano Babic 
---
 drivers/usb/host/ehci-mx6.c | 66 +
 1 file changed, 66 insertions(+)

diff --git a/drivers/usb/host/ehci-mx6.c b/drivers/usb/host/ehci-mx6.c
index 48889c1..91413c9 100644
--- a/drivers/usb/host/ehci-mx6.c
+++ b/drivers/usb/host/ehci-mx6.c
@@ -15,10 +15,13 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 
 #include "ehci.h"
 
+DECLARE_GLOBAL_DATA_PTR;
+
 #define USB_OTGREGS_OFFSET 0x000
 #define USB_H1REGS_OFFSET  0x200
 #define USB_H2REGS_OFFSET  0x400
@@ -48,6 +51,7 @@
 #define ANADIG_USB2_PLL_480_CTRL_EN_USB_CLKS   0x0040
 
 #define USBNC_OFFSET   0x200
+#define USBNC_PHY_STATUS_OFFSET0x23C
 #define USBNC_PHYSTATUS_ID_DIG (1 << 4) /* otg_id status */
 #define USBNC_PHYCFG2_ACAENB   (1 << 4) /* otg_id detection enable */
 #define UCTRL_PWR_POL  (1 << 9) /* OTG Polarity of Power Pin */
@@ -417,6 +421,67 @@ static const struct ehci_ops mx6_ehci_ops = {
.init_after_reset = mx6_init_after_reset
 };
 
+static int ehci_usb_ofdata_to_platdata(struct udevice *dev)
+{
+   struct usb_platdata *plat = dev_get_platdata(dev);
+   void *__iomem addr = (void *__iomem)dev_get_addr(dev);
+   void *__iomem phy_ctrl, *__iomem phy_status;
+   const void *blob = gd->fdt_blob;
+   int offset = dev->of_offset, phy_off;
+   const char *mode;
+   u32 val;
+
+   mode = fdt_getprop(blob, offset, "dr_mode", NULL);
+   if (mode) {
+   if (strcmp(mode, "peripheral") == 0)
+   plat->init_type = USB_INIT_DEVICE;
+   else if (strcmp(mode, "host") == 0)
+   plat->init_type = USB_INIT_HOST;
+   else if (strcmp(mode, "otg") == 0)
+   plat->init_type = USB_INIT_HOST;
+   else
+   return -EINVAL;
+   } else {
+   /*
+* About fsl,usbphy, Refer to
+* Documentation/devicetree/bindings/usb/ci-hdrc-usb2.txt.
+*/
+   if (is_mx6()) {
+   phy_off = fdtdec_lookup_phandle(blob,
+   offset,
+   "fsl,usbphy");
+   if (phy_off < 0)
+   return -EINVAL;
+
+   addr = (void __iomem *)fdtdec_get_addr(blob, phy_off,
+  "reg");
+   if ((fdt_addr_t)addr == FDT_ADDR_T_NONE)
+   return -EINVAL;
+
+   phy_ctrl = (void __iomem *)(addr + USBPHY_CTRL);
+   val = readl(phy_ctrl);
+
+   if (val & USBPHY_CTRL_OTG_ID)
+   plat->init_type = USB_INIT_DEVICE;
+   else
+   plat->init_type = USB_INIT_HOST;
+   } else if (is_mx7()) {
+   phy_status = (void __iomem *)(addr +
+ USBNC_PHY_STATUS_OFFSET);
+   val = readl(phy_status);
+
+   if (val & USBNC_PHYSTATUS_ID_DIG)
+   plat->init_type = USB_INIT_DEVICE;
+   else
+   plat->init_type = USB_INIT_HOST;
+   } else {
+   return -EINVAL;
+   }
+   }
+
+   return 0;
+}
+
 static int ehci_usb_probe(struct udevice *dev)
 {
struct usb_platdata *plat = dev_get_platdata(dev);
@@ -460,6 +525,7 @@ U_BOOT_DRIVER(usb_mx6) = {
.name   = "ehci_mx6",
.id = UCLASS_USB,
.of_match = mx6_usb_ids,
+   .ofdata_to_platdata = ehci_usb_ofdata_to_platdata,
.probe  = ehci_usb_probe,
.remove = ehci_deregister,
.ops= &ehci_usb_ops,
-- 
2.6.2

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2] arm64: mvebu: Fix A8K memory mapping and add documentation

2016-12-21 Thread Stefan Roese

On 19.12.2016 16:04, kos...@marvell.com wrote:

From: Konstantin Porotchkin 

Fix the MMU mapping for A8K device family:
 - Separate A7K and A8K memory mappings
 - Fix memory regions by including IO mapping for all
   3 PCIe interfaces existing on each connected CP110 controller
Add A8K memory mapping documentation with all regions
configured by Marvell ATF.

Change-Id: I9c930569b1853900f5fba2d5db319b092cc7a2a6
Signed-off-by: Konstantin Porotchkin 
Signed-off-by: Stefan Roese 
Cc: Stefan Roese 
Cc: Nadav Haklai 
Cc: Neta Zur Hershkovits 
Cc: Omri Itach 
Cc: Igal Liberman 
Cc: Haim Boot 
Cc: Hanna Hawa 


Applied to u-boot-marvell/master.

Thanks,
Stefan
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] Please pull u-boot-marvell/master

2016-12-21 Thread Stefan Roese
Hi Tom,

please pull the following A7k/8k fix.

Thanks,
Stefan

The following changes since commit 0bd1f96aa2f18d29e8321e519b0152853e500d4d:

  Merge git://git.denx.de/u-boot-mpc85xx (2016-12-20 12:20:12 -0500)

are available in the git repository at:

  git://www.denx.de/git/u-boot-marvell.git 

for you to fetch changes up to 0d92f2141ac5ef5c80d13e9501890f914525d43c:

  arm64: mvebu: Fix A8K memory mapping and add documentation (2016-12-21 
09:52:35 +0100)


Konstantin Porotchkin (1):
  arm64: mvebu: Fix A8K memory mapping and add documentation

 arch/arm/mach-mvebu/armada8k/cpu.c | 58 +-
 doc/mvebu/armada-8k-memory.txt | 56 
 2 files changed, 101 insertions(+), 13 deletions(-)
 create mode 100644 doc/mvebu/armada-8k-memory.txt
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 2/3] usb: ehci-mx6: handle vbus-supply

2016-12-21 Thread Peng Fan
Drop board_ehci_power when dm usb used and switch to use
regulator api to handle vbus.

Signed-off-by: Peng Fan 
Cc: Marek Vasut 
Cc: Simon Glass 
Cc: Stefano Babic 
---
 drivers/usb/host/ehci-mx6.c | 30 +++---
 1 file changed, 27 insertions(+), 3 deletions(-)

diff --git a/drivers/usb/host/ehci-mx6.c b/drivers/usb/host/ehci-mx6.c
index 91413c9..123befe 100644
--- a/drivers/usb/host/ehci-mx6.c
+++ b/drivers/usb/host/ehci-mx6.c
@@ -17,6 +17,7 @@
 #include 
 #include 
 #include 
+#include 
 
 #include "ehci.h"
 
@@ -388,6 +389,7 @@ int ehci_hcd_stop(int index)
 struct ehci_mx6_priv_data {
struct ehci_ctrl ctrl;
struct usb_ehci *ehci;
+   struct udevice *vbus_supply;
enum usb_init_type init_type;
int portnr;
 };
@@ -403,7 +405,15 @@ static int mx6_init_after_reset(struct ehci_ctrl *dev)
if (ret)
return ret;
 
-   board_ehci_power(priv->portnr, (type == USB_INIT_DEVICE) ? 0 : 1);
+   if (priv->vbus_supply) {
+   ret = regulator_set_enable(priv->vbus_supply,
+  (type == USB_INIT_DEVICE) ?
+  false : true);
+   if (ret) {
+   puts("Error enabling VBUS supply\n");
+   return ret;
+   }
+   }
 
if (type == USB_INIT_DEVICE)
return 0;
@@ -487,19 +497,33 @@ static int ehci_usb_probe(struct udevice *dev)
struct usb_platdata *plat = dev_get_platdata(dev);
struct usb_ehci *ehci = (struct usb_ehci *)dev_get_addr(dev);
struct ehci_mx6_priv_data *priv = dev_get_priv(dev);
+   enum usb_init_type type = plat->init_type;
struct ehci_hccr *hccr;
struct ehci_hcor *hcor;
int ret;
 
priv->ehci = ehci;
priv->portnr = dev->seq;
-   priv->init_type = plat->init_type;
+   priv->init_type = type;
+
+   ret = device_get_supply_regulator(dev, "vbus-supply",
+ &priv->vbus_supply);
+   if (ret)
+   debug("%s: No vbus supply\n", dev->name);
 
ret = ehci_mx6_common_init(ehci, priv->portnr);
if (ret)
return ret;
 
-   board_ehci_power(priv->portnr, (priv->init_type == USB_INIT_DEVICE) ? 0 
: 1);
+   if (priv->vbus_supply) {
+   ret = regulator_set_enable(priv->vbus_supply,
+  (type == USB_INIT_DEVICE) ?
+  false : true);
+   if (ret) {
+   puts("Error enabling VBUS supply\n");
+   return ret;
+   }
+   }
 
if (priv->init_type == USB_INIT_HOST) {
setbits_le32(&ehci->usbmode, CM_HOST);
-- 
2.6.2

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 00/10] i.MX6: engicam: Add nandboot env and FIT support

2016-12-21 Thread Jagan Teki
From: Jagan Teki 

This patchset, add support for
- legacy image boot, with bootm
- NAND boot env support, with UBIFS as rootfs
- FIT image boot support

on i.MX6 based Engicam Quad/DualLite/Dual/Solo module kits.

Jagan Teki (10):
  configs: engicam: Increase nand kernel partition size
  imx6: engicam: Use bootm instead of bootz
  configs: engicam: Rename nand with gpmi-name in mtdparts
  defconfigs: engicam: Enable MMC commands in nand
  defconfigs: engicam: Enable UBI commands
  imx6: engicam: Add nandboot env support
  defconfigs: imx6: engicam: Enable FIT
  configs: engicam: Enable CONFIG_IMAGE_FORMAT_LEGACY
  configs: engicam: Cleanup on mmcboot env
  configs: engicam: Add fitboot env support

 configs/imx6dl_icore_mmc_defconfig |  4 +-
 configs/imx6dl_icore_nand_defconfig| 10 -
 configs/imx6dl_icore_rqs_mmc_defconfig |  4 +-
 configs/imx6q_icore_mmc_defconfig  |  4 +-
 configs/imx6q_icore_nand_defconfig | 10 -
 configs/imx6q_icore_rqs_mmc_defconfig  |  4 +-
 configs/imx6ul_geam_mmc_defconfig  |  4 +-
 configs/imx6ul_geam_nand_defconfig | 10 -
 include/configs/imx6qdl_icore.h| 77 +-
 include/configs/imx6qdl_icore_rqs.h| 43 +--
 include/configs/imx6ul_geam.h  | 76 -
 11 files changed, 187 insertions(+), 59 deletions(-)

-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 01/10] configs: engicam: Increase nand kernel partition size

2016-12-21 Thread Jagan Teki
From: Jagan Teki 

Increase the nand kernel partition size, for supporting
large uImage files, maximum 8MiB.

Cc: Stefano Babic 
Cc: Matteo Lisi 
Cc: Michael Trimarchi 
Signed-off-by: Jagan Teki 
---
 include/configs/imx6qdl_icore.h | 2 +-
 include/configs/imx6ul_geam.h   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/configs/imx6qdl_icore.h b/include/configs/imx6qdl_icore.h
index 204e96e..0125385 100644
--- a/include/configs/imx6qdl_icore.h
+++ b/include/configs/imx6qdl_icore.h
@@ -131,7 +131,7 @@
 # define CONFIG_MTD_PARTITIONS
 # define MTDIDS_DEFAULT"nand0=nand"
 # define MTDPARTS_DEFAULT  "mtdparts=nand:2m(spl),2m(uboot)," \
-   "1m(env),4m(kernel),1m(dtb),-(rootfs)"
+   "1m(env),8m(kernel),1m(dtb),-(rootfs)"
 
 # define CONFIG_APBH_DMA
 # define CONFIG_APBH_DMA_BURST
diff --git a/include/configs/imx6ul_geam.h b/include/configs/imx6ul_geam.h
index 48b1120..787da08 100644
--- a/include/configs/imx6ul_geam.h
+++ b/include/configs/imx6ul_geam.h
@@ -130,7 +130,7 @@
 # define CONFIG_MTD_PARTITIONS
 # define MTDIDS_DEFAULT"nand0=nand"
 # define MTDPARTS_DEFAULT  "mtdparts=nand:2m(spl),2m(uboot)," \
-   "1m(env),4m(kernel),1m(dtb),-(rootfs)"
+   "1m(env),8m(kernel),1m(dtb),-(rootfs)"
 
 # define CONFIG_APBH_DMA
 # define CONFIG_APBH_DMA_BURST
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 02/10] imx6: engicam: Use bootm instead of bootz

2016-12-21 Thread Jagan Teki
From: Jagan Teki 

Boot Linux with uImage instead of zImage, so update
bootz with bootm.

Cc: Stefano Babic 
Cc: Matteo Lisi 
Cc: Michael Trimarchi 
Signed-off-by: Jagan Teki 
---
 configs/imx6dl_icore_mmc_defconfig | 1 -
 configs/imx6dl_icore_nand_defconfig| 1 -
 configs/imx6dl_icore_rqs_mmc_defconfig | 1 -
 configs/imx6q_icore_mmc_defconfig  | 1 -
 configs/imx6q_icore_nand_defconfig | 1 -
 configs/imx6q_icore_rqs_mmc_defconfig  | 1 -
 configs/imx6ul_geam_mmc_defconfig  | 1 -
 configs/imx6ul_geam_nand_defconfig | 1 -
 include/configs/imx6qdl_icore.h| 8 
 include/configs/imx6qdl_icore_rqs.h| 8 
 include/configs/imx6ul_geam.h  | 8 
 11 files changed, 12 insertions(+), 20 deletions(-)

diff --git a/configs/imx6dl_icore_mmc_defconfig 
b/configs/imx6dl_icore_mmc_defconfig
index 5f09425..2ac0e66 100644
--- a/configs/imx6dl_icore_mmc_defconfig
+++ b/configs/imx6dl_icore_mmc_defconfig
@@ -15,7 +15,6 @@ CONFIG_SYS_MAXARGS=32
 # CONFIG_CMD_IMLS is not set
 # CONFIG_BLK is not set
 # CONFIG_DM_MMC_OPS is not set
-CONFIG_CMD_BOOTZ=y
 CONFIG_CMD_GPIO=y
 CONFIG_CMD_I2C=y
 CONFIG_CMD_MII=y
diff --git a/configs/imx6dl_icore_nand_defconfig 
b/configs/imx6dl_icore_nand_defconfig
index af3a9f8..4e81a41 100644
--- a/configs/imx6dl_icore_nand_defconfig
+++ b/configs/imx6dl_icore_nand_defconfig
@@ -13,7 +13,6 @@ CONFIG_HUSH_PARSER=y
 CONFIG_AUTO_COMPLETE=y
 CONFIG_SYS_MAXARGS=32
 # CONFIG_CMD_IMLS is not set
-CONFIG_CMD_BOOTZ=y
 CONFIG_CMD_GPIO=y
 CONFIG_CMD_I2C=y
 CONFIG_CMD_MII=y
diff --git a/configs/imx6dl_icore_rqs_mmc_defconfig 
b/configs/imx6dl_icore_rqs_mmc_defconfig
index 230cd20..0757371 100644
--- a/configs/imx6dl_icore_rqs_mmc_defconfig
+++ b/configs/imx6dl_icore_rqs_mmc_defconfig
@@ -15,7 +15,6 @@ CONFIG_SYS_MAXARGS=32
 # CONFIG_CMD_IMLS is not set
 # CONFIG_BLK is not set
 # CONFIG_DM_MMC_OPS is not set
-CONFIG_CMD_BOOTZ=y
 CONFIG_CMD_GPIO=y
 CONFIG_CMD_MEMTEST=y
 CONFIG_CMD_MII=y
diff --git a/configs/imx6q_icore_mmc_defconfig 
b/configs/imx6q_icore_mmc_defconfig
index 8f812b4..551c688 100644
--- a/configs/imx6q_icore_mmc_defconfig
+++ b/configs/imx6q_icore_mmc_defconfig
@@ -15,7 +15,6 @@ CONFIG_SYS_MAXARGS=32
 # CONFIG_CMD_IMLS is not set
 # CONFIG_BLK is not set
 # CONFIG_DM_MMC_OPS is not set
-CONFIG_CMD_BOOTZ=y
 CONFIG_CMD_GPIO=y
 CONFIG_CMD_I2C=y
 CONFIG_CMD_MII=y
diff --git a/configs/imx6q_icore_nand_defconfig 
b/configs/imx6q_icore_nand_defconfig
index 4735be8..274e62b 100644
--- a/configs/imx6q_icore_nand_defconfig
+++ b/configs/imx6q_icore_nand_defconfig
@@ -13,7 +13,6 @@ CONFIG_HUSH_PARSER=y
 CONFIG_AUTO_COMPLETE=y
 CONFIG_SYS_MAXARGS=32
 # CONFIG_CMD_IMLS is not set
-CONFIG_CMD_BOOTZ=y
 CONFIG_CMD_GPIO=y
 CONFIG_CMD_I2C=y
 CONFIG_CMD_MII=y
diff --git a/configs/imx6q_icore_rqs_mmc_defconfig 
b/configs/imx6q_icore_rqs_mmc_defconfig
index f7c25c4..e29afa0 100644
--- a/configs/imx6q_icore_rqs_mmc_defconfig
+++ b/configs/imx6q_icore_rqs_mmc_defconfig
@@ -15,7 +15,6 @@ CONFIG_SYS_MAXARGS=32
 # CONFIG_CMD_IMLS is not set
 # CONFIG_BLK is not set
 # CONFIG_DM_MMC_OPS is not set
-CONFIG_CMD_BOOTZ=y
 CONFIG_CMD_GPIO=y
 CONFIG_CMD_MEMTEST=y
 CONFIG_CMD_MII=y
diff --git a/configs/imx6ul_geam_mmc_defconfig 
b/configs/imx6ul_geam_mmc_defconfig
index d5be076..4aaf93d 100644
--- a/configs/imx6ul_geam_mmc_defconfig
+++ b/configs/imx6ul_geam_mmc_defconfig
@@ -15,7 +15,6 @@ CONFIG_SYS_MAXARGS=32
 # CONFIG_CMD_IMLS is not set
 # CONFIG_BLK is not set
 # CONFIG_DM_MMC_OPS is not set
-CONFIG_CMD_BOOTZ=y
 CONFIG_CMD_GPIO=y
 CONFIG_CMD_MEMTEST=y
 CONFIG_CMD_MII=y
diff --git a/configs/imx6ul_geam_nand_defconfig 
b/configs/imx6ul_geam_nand_defconfig
index cc45602..a21c53a 100644
--- a/configs/imx6ul_geam_nand_defconfig
+++ b/configs/imx6ul_geam_nand_defconfig
@@ -15,7 +15,6 @@ CONFIG_SYS_MAXARGS=32
 # CONFIG_CMD_IMLS is not set
 # CONFIG_BLK is not set
 # CONFIG_DM_MMC_OPS is not set
-CONFIG_CMD_BOOTZ=y
 CONFIG_CMD_GPIO=y
 CONFIG_CMD_MEMTEST=y
 CONFIG_CMD_MII=y
diff --git a/include/configs/imx6qdl_icore.h b/include/configs/imx6qdl_icore.h
index 0125385..d57c73e 100644
--- a/include/configs/imx6qdl_icore.h
+++ b/include/configs/imx6qdl_icore.h
@@ -38,7 +38,7 @@
 #define CONFIG_EXTRA_ENV_SETTINGS \
"script=boot.scr\0" \
"splashpos=m,m\0" \
-   "image=zImage\0" \
+   "image=uImage\0" \
"console=ttymxc3\0" \
"fdt_high=0x\0" \
"fdt_file=" CONFIG_DEFAULT_FDT_FILE "\0" \
@@ -60,16 +60,16 @@
"run mmcargs; " \
"if test ${boot_fdt} = yes || test ${boot_fdt} = try; then " \
"if run loadfdt; then " \
-   "bootz ${loadaddr} - ${fdt_addr}; " \
+   "bootm ${loadaddr} - ${fdt_addr}; " \
"else " \
"if test ${boot_fdt} = try; then " \
-   "bootz; " \
+   "bootm; " \
   

[U-Boot] [PATCH 05/10] defconfigs: engicam: Enable UBI commands

2016-12-21 Thread Jagan Teki
From: Jagan Teki 

Create ubifs.img:
$ mkfs.ubifs -q -r /rootfs -m 4096 -e 253952 -c 7936 -o ubifs.img

Write ubifs.img:
---
icorem6qdl> nand erase.part rootfs
icorem6qdl> ubi part rootfs
icorem6qdl> ubi create rootfs

icorem6qdl> ext4load mmc 0:2 ${loadaddr} ubifs.img
166592512 bytes read in 8091 ms (19.6 MiB/s)
icorem6qdl> ubi write ${loadaddr} rootfs ${filesize}
166592512 bytes written to volume rootfs
icorem6qdl> ubifsmount ubi0:rootfs

Cc: Stefano Babic 
Cc: Matteo Lisi 
Cc: Michael Trimarchi 
Signed-off-by: Jagan Teki 
---
 configs/imx6dl_icore_nand_defconfig | 1 +
 configs/imx6q_icore_nand_defconfig  | 1 +
 configs/imx6ul_geam_nand_defconfig  | 1 +
 include/configs/imx6qdl_icore.h | 5 +
 include/configs/imx6ul_geam.h   | 5 +
 5 files changed, 13 insertions(+)

diff --git a/configs/imx6dl_icore_nand_defconfig 
b/configs/imx6dl_icore_nand_defconfig
index 7f45744..67397c3 100644
--- a/configs/imx6dl_icore_nand_defconfig
+++ b/configs/imx6dl_icore_nand_defconfig
@@ -20,6 +20,7 @@ CONFIG_CMD_PING=y
 CONFIG_CMD_MEMTEST=y
 CONFIG_CMD_MMC=y
 CONFIG_CMD_NAND=y
+CONFIG_CMD_UBI=y
 CONFIG_CMD_CACHE=y
 CONFIG_CMD_EXT4=y
 CONFIG_CMD_EXT4_WRITE=y
diff --git a/configs/imx6q_icore_nand_defconfig 
b/configs/imx6q_icore_nand_defconfig
index 572e33a..350dc0d 100644
--- a/configs/imx6q_icore_nand_defconfig
+++ b/configs/imx6q_icore_nand_defconfig
@@ -20,6 +20,7 @@ CONFIG_CMD_PING=y
 CONFIG_CMD_MEMTEST=y
 CONFIG_CMD_MMC=y
 CONFIG_CMD_NAND=y
+CONFIG_CMD_UBI=y
 CONFIG_CMD_CACHE=y
 CONFIG_CMD_EXT4=y
 CONFIG_CMD_EXT4_WRITE=y
diff --git a/configs/imx6ul_geam_nand_defconfig 
b/configs/imx6ul_geam_nand_defconfig
index 55fa003..6d8336d 100644
--- a/configs/imx6ul_geam_nand_defconfig
+++ b/configs/imx6ul_geam_nand_defconfig
@@ -21,6 +21,7 @@ CONFIG_CMD_MMC=y
 CONFIG_CMD_MII=y
 CONFIG_CMD_PING=y
 CONFIG_CMD_NAND=y
+CONFIG_CMD_UBI=y
 CONFIG_CMD_CACHE=y
 CONFIG_CMD_EXT4=y
 CONFIG_CMD_EXT4_WRITE=y
diff --git a/include/configs/imx6qdl_icore.h b/include/configs/imx6qdl_icore.h
index 3848a6f..55eb100 100644
--- a/include/configs/imx6qdl_icore.h
+++ b/include/configs/imx6qdl_icore.h
@@ -133,6 +133,11 @@
 # define MTDPARTS_DEFAULT  "mtdparts=gpmi-nand:2m(spl),2m(uboot)," 
\
"1m(env),8m(kernel),1m(dtb),-(rootfs)"
 
+/* UBI */
+# define CONFIG_CMD_UBIFS
+# define CONFIG_RBTREE
+# define CONFIG_LZO
+
 # define CONFIG_APBH_DMA
 # define CONFIG_APBH_DMA_BURST
 # define CONFIG_APBH_DMA_BURST8
diff --git a/include/configs/imx6ul_geam.h b/include/configs/imx6ul_geam.h
index e925c8f..3e26a81 100644
--- a/include/configs/imx6ul_geam.h
+++ b/include/configs/imx6ul_geam.h
@@ -132,6 +132,11 @@
 # define MTDPARTS_DEFAULT  "mtdparts=gpmi-nand:2m(spl),2m(uboot)," 
\
"1m(env),8m(kernel),1m(dtb),-(rootfs)"
 
+/* UBI */
+# define CONFIG_CMD_UBIFS
+# define CONFIG_RBTREE
+# define CONFIG_LZO
+
 # define CONFIG_APBH_DMA
 # define CONFIG_APBH_DMA_BURST
 # define CONFIG_APBH_DMA_BURST8
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 03/10] configs: engicam: Rename nand with gpmi-name in mtdparts

2016-12-21 Thread Jagan Teki
From: Jagan Teki 

gpmi-nand is the proper name used in nand driver from Linux for all
imx related nand boards, so rename mtdparts name as gpmi-nand instead
of nand, this will eventually reflects all nand info to Linux from
u-boot like mtdparts.

Cc: Stefano Babic 
Cc: Matteo Lisi 
Cc: Michael Trimarchi 
Signed-off-by: Jagan Teki 
---
 include/configs/imx6qdl_icore.h | 4 ++--
 include/configs/imx6ul_geam.h   | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/include/configs/imx6qdl_icore.h b/include/configs/imx6qdl_icore.h
index d57c73e..3848a6f 100644
--- a/include/configs/imx6qdl_icore.h
+++ b/include/configs/imx6qdl_icore.h
@@ -129,8 +129,8 @@
 # define CONFIG_MTD_DEVICE
 # define CONFIG_CMD_MTDPARTS
 # define CONFIG_MTD_PARTITIONS
-# define MTDIDS_DEFAULT"nand0=nand"
-# define MTDPARTS_DEFAULT  "mtdparts=nand:2m(spl),2m(uboot)," \
+# define MTDIDS_DEFAULT"nand0=gpmi-nand"
+# define MTDPARTS_DEFAULT  "mtdparts=gpmi-nand:2m(spl),2m(uboot)," 
\
"1m(env),8m(kernel),1m(dtb),-(rootfs)"
 
 # define CONFIG_APBH_DMA
diff --git a/include/configs/imx6ul_geam.h b/include/configs/imx6ul_geam.h
index 7c8a420..e925c8f 100644
--- a/include/configs/imx6ul_geam.h
+++ b/include/configs/imx6ul_geam.h
@@ -128,8 +128,8 @@
 # define CONFIG_MTD_DEVICE
 # define CONFIG_CMD_MTDPARTS
 # define CONFIG_MTD_PARTITIONS
-# define MTDIDS_DEFAULT"nand0=nand"
-# define MTDPARTS_DEFAULT  "mtdparts=nand:2m(spl),2m(uboot)," \
+# define MTDIDS_DEFAULT"nand0=gpmi-nand"
+# define MTDPARTS_DEFAULT  "mtdparts=gpmi-nand:2m(spl),2m(uboot)," 
\
"1m(env),8m(kernel),1m(dtb),-(rootfs)"
 
 # define CONFIG_APBH_DMA
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 04/10] defconfigs: engicam: Enable MMC commands in nand

2016-12-21 Thread Jagan Teki
From: Jagan Teki 

For writing Linux or rootfs on to NAND, the best suitable way
is to use MMC commands since MMC driver by default enabled by
mx6_common.h, hence enabled MMC commands in nand defconfigs.

Cc: Stefano Babic 
Cc: Matteo Lisi 
Cc: Michael Trimarchi 
Signed-off-by: Jagan Teki 
---
 configs/imx6dl_icore_nand_defconfig | 5 +
 configs/imx6q_icore_nand_defconfig  | 5 +
 configs/imx6ul_geam_nand_defconfig  | 5 +
 3 files changed, 15 insertions(+)

diff --git a/configs/imx6dl_icore_nand_defconfig 
b/configs/imx6dl_icore_nand_defconfig
index 4e81a41..7f45744 100644
--- a/configs/imx6dl_icore_nand_defconfig
+++ b/configs/imx6dl_icore_nand_defconfig
@@ -18,8 +18,13 @@ CONFIG_CMD_I2C=y
 CONFIG_CMD_MII=y
 CONFIG_CMD_PING=y
 CONFIG_CMD_MEMTEST=y
+CONFIG_CMD_MMC=y
 CONFIG_CMD_NAND=y
 CONFIG_CMD_CACHE=y
+CONFIG_CMD_EXT4=y
+CONFIG_CMD_EXT4_WRITE=y
+CONFIG_CMD_FAT=y
+CONFIG_CMD_FS_GENERIC=y
 CONFIG_OF_LIBFDT=y
 CONFIG_FEC_MXC=y
 CONFIG_MXC_UART=y
diff --git a/configs/imx6q_icore_nand_defconfig 
b/configs/imx6q_icore_nand_defconfig
index 274e62b..572e33a 100644
--- a/configs/imx6q_icore_nand_defconfig
+++ b/configs/imx6q_icore_nand_defconfig
@@ -18,8 +18,13 @@ CONFIG_CMD_I2C=y
 CONFIG_CMD_MII=y
 CONFIG_CMD_PING=y
 CONFIG_CMD_MEMTEST=y
+CONFIG_CMD_MMC=y
 CONFIG_CMD_NAND=y
 CONFIG_CMD_CACHE=y
+CONFIG_CMD_EXT4=y
+CONFIG_CMD_EXT4_WRITE=y
+CONFIG_CMD_FAT=y
+CONFIG_CMD_FS_GENERIC=y
 CONFIG_OF_LIBFDT=y
 CONFIG_FEC_MXC=y
 CONFIG_MXC_UART=y
diff --git a/configs/imx6ul_geam_nand_defconfig 
b/configs/imx6ul_geam_nand_defconfig
index a21c53a..55fa003 100644
--- a/configs/imx6ul_geam_nand_defconfig
+++ b/configs/imx6ul_geam_nand_defconfig
@@ -17,10 +17,15 @@ CONFIG_SYS_MAXARGS=32
 # CONFIG_DM_MMC_OPS is not set
 CONFIG_CMD_GPIO=y
 CONFIG_CMD_MEMTEST=y
+CONFIG_CMD_MMC=y
 CONFIG_CMD_MII=y
 CONFIG_CMD_PING=y
 CONFIG_CMD_NAND=y
 CONFIG_CMD_CACHE=y
+CONFIG_CMD_EXT4=y
+CONFIG_CMD_EXT4_WRITE=y
+CONFIG_CMD_FAT=y
+CONFIG_CMD_FS_GENERIC=y
 CONFIG_OF_LIBFDT=y
 CONFIG_FEC_MXC=y
 CONFIG_MXC_UART=y
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 06/10] imx6: engicam: Add nandboot env support

2016-12-21 Thread Jagan Teki
From: Jagan Teki 

Add config options for booting Linux from NAND in UBI format.

Cc: Stefano Babic 
Cc: Matteo Lisi 
Cc: Michael Trimarchi 
Signed-off-by: Jagan Teki 
---
 include/configs/imx6qdl_icore.h | 21 +++--
 include/configs/imx6ul_geam.h   | 21 +++--
 2 files changed, 38 insertions(+), 4 deletions(-)

diff --git a/include/configs/imx6qdl_icore.h b/include/configs/imx6qdl_icore.h
index 55eb100..9d5951f 100644
--- a/include/configs/imx6qdl_icore.h
+++ b/include/configs/imx6qdl_icore.h
@@ -47,9 +47,12 @@
"mmcdev=0\0" \
"mmcpart=1\0" \
"mmcroot=/dev/mmcblk0p2 rootwait rw\0" \
+   "nandroot=ubi0:rootfs rootfstype=ubifs\0" \
"mmcautodetect=yes\0" \
"mmcargs=setenv bootargs console=${console},${baudrate} " \
"root=${mmcroot}\0" \
+   "ubiargs=setenv bootargs console=${console},${baudrate} " \
+   "ubi.mtd=5 root=${nandroot} ${mtdparts}\0" \
"loadbootscript=" \
"fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${script};\0" \
"bootscript=echo Running bootscript from mmc ...; " \
@@ -70,9 +73,22 @@
"fi; " \
"else " \
"bootm; " \
-   "fi\0"
+   "fi\0" \
+   "nandboot=echo Booting from nand ...; " \
+   "if mtdparts; then " \
+   "echo Starting nand boot ...; " \
+   "else " \
+   "mtdparts default; " \
+   "fi; " \
+   "run ubiargs; " \
+   "nand read ${loadaddr} kernel 0x80; " \
+   "nand read ${fdt_addr} dtb 0x10; " \
+   "bootm ${loadaddr} - ${fdt_addr}\0"
 
-#define CONFIG_BOOTCOMMAND \
+#ifdef CONFIG_NAND_MXS
+# define CONFIG_BOOTCOMMAND"run nandboot"
+#else
+# define CONFIG_BOOTCOMMAND \
   "mmc dev ${mmcdev};" \
   "mmc dev ${mmcdev}; if mmc rescan; then " \
   "if run loadbootscript; then " \
@@ -83,6 +99,7 @@
"fi; " \
   "fi; " \
   "fi"
+#endif
 
 /* Miscellaneous configurable options */
 #define CONFIG_SYS_MEMTEST_START   0x8000
diff --git a/include/configs/imx6ul_geam.h b/include/configs/imx6ul_geam.h
index 3e26a81..507e743 100644
--- a/include/configs/imx6ul_geam.h
+++ b/include/configs/imx6ul_geam.h
@@ -46,9 +46,12 @@
"mmcdev=0\0" \
"mmcpart=1\0" \
"mmcroot=/dev/mmcblk0p2 rootwait rw\0" \
+   "nandroot=ubi0:rootfs rootfstype=ubifs\0" \
"mmcautodetect=yes\0" \
"mmcargs=setenv bootargs console=${console},${baudrate} " \
"root=${mmcroot}\0" \
+   "ubiargs=setenv bootargs console=${console},${baudrate} " \
+   "ubi.mtd=5 root=${nandroot} ${mtdparts}\0" \
"loadbootscript=" \
"fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${script};\0" \
"bootscript=echo Running bootscript from mmc ...; " \
@@ -69,9 +72,22 @@
"fi; " \
"else " \
"bootm; " \
-   "fi\0"
+   "fi\0" \
+   "nandboot=echo Booting from nand ...; " \
+   "if mtdparts; then " \
+   "echo Starting nand boot ...; " \
+   "else " \
+   "mtdparts default; " \
+   "fi; " \
+   "run ubiargs; " \
+   "nand read ${loadaddr} kernel 0x80; " \
+   "nand read ${fdt_addr} dtb 0x10; " \
+   "bootm ${loadaddr} - ${fdt_addr}\0"
 
-#define CONFIG_BOOTCOMMAND \
+#ifdef CONFIG_NAND_MXS
+# define CONFIG_BOOTCOMMAND"run nandboot"
+#else
+# define CONFIG_BOOTCOMMAND \
   "mmc dev ${mmcdev};" \
   "mmc dev ${mmcdev}; if mmc rescan; then " \
   "if run loadbootscript; then " \
@@ -82,6 +98,7 @@
"fi; " \
   "fi; " \
   "fi"
+#endif
 
 /* Miscellaneous configurable options */
 #define CONFIG_SYS_MEMTEST_START   0x8000
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 09/10] configs: engicam: Cleanup on mmcboot env

2016-12-21 Thread Jagan Teki
From: Jagan Teki 

- Add tab space
- remove exctra 'mmc dev ${mmcdev}'

Cc: Stefano Babic 
Cc: Matteo Lisi 
Cc: Michael Trimarchi 
Signed-off-by: Jagan Teki 
---
 include/configs/imx6qdl_icore.h | 14 +++---
 include/configs/imx6qdl_icore_rqs.h | 14 +++---
 include/configs/imx6ul_geam.h   | 13 ++---
 3 files changed, 20 insertions(+), 21 deletions(-)

diff --git a/include/configs/imx6qdl_icore.h b/include/configs/imx6qdl_icore.h
index f25f364..2c255e2 100644
--- a/include/configs/imx6qdl_icore.h
+++ b/include/configs/imx6qdl_icore.h
@@ -89,16 +89,16 @@
 # define CONFIG_BOOTCOMMAND"run nandboot"
 #else
 # define CONFIG_BOOTCOMMAND \
-  "mmc dev ${mmcdev};" \
-  "mmc dev ${mmcdev}; if mmc rescan; then " \
-  "if run loadbootscript; then " \
-  "run bootscript; " \
-  "else " \
+   "mmc dev ${mmcdev};" \
+   "if mmc rescan; then " \
+   "if run loadbootscript; then " \
+   "run bootscript; " \
+   "else " \
"if run loadimage; then " \
"run mmcboot; " \
"fi; " \
-  "fi; " \
-  "fi"
+   "fi; " \
+   "fi"
 #endif
 
 /* Miscellaneous configurable options */
diff --git a/include/configs/imx6qdl_icore_rqs.h 
b/include/configs/imx6qdl_icore_rqs.h
index ee25882..6f297c1 100644
--- a/include/configs/imx6qdl_icore_rqs.h
+++ b/include/configs/imx6qdl_icore_rqs.h
@@ -68,16 +68,16 @@
"fi\0"
 
 #define CONFIG_BOOTCOMMAND \
-  "mmc dev ${mmcdev};" \
-  "mmc dev ${mmcdev}; if mmc rescan; then " \
-  "if run loadbootscript; then " \
-  "run bootscript; " \
-  "else " \
+   "mmc dev ${mmcdev};" \
+   "if mmc rescan; then " \
+   "if run loadbootscript; then " \
+   "run bootscript; " \
+   "else " \
"if run loadimage; then " \
"run mmcboot; " \
"fi; " \
-  "fi; " \
-  "fi"
+   "fi; " \
+   "fi"
 
 /* Miscellaneous configurable options */
 #define CONFIG_SYS_MEMTEST_START   0x8000
diff --git a/include/configs/imx6ul_geam.h b/include/configs/imx6ul_geam.h
index a8ee401..8d91fed 100644
--- a/include/configs/imx6ul_geam.h
+++ b/include/configs/imx6ul_geam.h
@@ -88,16 +88,15 @@
 # define CONFIG_BOOTCOMMAND"run nandboot"
 #else
 # define CONFIG_BOOTCOMMAND \
-  "mmc dev ${mmcdev};" \
-  "mmc dev ${mmcdev}; if mmc rescan; then " \
-  "if run loadbootscript; then " \
-  "run bootscript; " \
-  "else " \
+   "if mmc rescan; then " \
+   "if run loadbootscript; then " \
+   "run bootscript; " \
+   "else " \
"if run loadimage; then " \
"run mmcboot; " \
"fi; " \
-  "fi; " \
-  "fi"
+   "fi; " \
+   "fi"
 #endif
 
 /* Miscellaneous configurable options */
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 08/10] configs: engicam: Enable CONFIG_IMAGE_FORMAT_LEGACY

2016-12-21 Thread Jagan Teki
From: Jagan Teki 

Enabling FIT along with Signature will make bootm to
not-understanding u-boot legacy image formats like uImage, etc.
So this patch enabling legacy image format for backward compatibility.

Cc: Stefano Babic 
Cc: Matteo Lisi 
Cc: Michael Trimarchi 
Signed-off-by: Jagan Teki 
---
 include/configs/imx6qdl_icore.h | 1 +
 include/configs/imx6qdl_icore_rqs.h | 1 +
 include/configs/imx6ul_geam.h   | 1 +
 3 files changed, 3 insertions(+)

diff --git a/include/configs/imx6qdl_icore.h b/include/configs/imx6qdl_icore.h
index 2c07a9b..f25f364 100644
--- a/include/configs/imx6qdl_icore.h
+++ b/include/configs/imx6qdl_icore.h
@@ -126,6 +126,7 @@
 # define CONFIG_HASH_VERIFY
 # define CONFIG_SHA1
 # define CONFIG_SHA256
+# define CONFIG_IMAGE_FORMAT_LEGACY
 #endif
 
 /* UART */
diff --git a/include/configs/imx6qdl_icore_rqs.h 
b/include/configs/imx6qdl_icore_rqs.h
index c36359b..ee25882 100644
--- a/include/configs/imx6qdl_icore_rqs.h
+++ b/include/configs/imx6qdl_icore_rqs.h
@@ -104,6 +104,7 @@
 # define CONFIG_HASH_VERIFY
 # define CONFIG_SHA1
 # define CONFIG_SHA256
+# define CONFIG_IMAGE_FORMAT_LEGACY
 #endif
 
 /* UART */
diff --git a/include/configs/imx6ul_geam.h b/include/configs/imx6ul_geam.h
index 5a4f291..a8ee401 100644
--- a/include/configs/imx6ul_geam.h
+++ b/include/configs/imx6ul_geam.h
@@ -125,6 +125,7 @@
 # define CONFIG_HASH_VERIFY
 # define CONFIG_SHA1
 # define CONFIG_SHA256
+# define CONFIG_IMAGE_FORMAT_LEGACY
 #endif
 
 /* UART */
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] imx6ul: geam6ul: Add MAINTAINERS for nand_defconfig

2016-12-21 Thread Jagan Teki
From: Jagan Teki 

Add Jagan as MAINTAINERS of configs/imx6ul_geam_nand_defconfig

Cc: Stefano Babic 
Cc: Matteo Lisi 
Cc: Michael Trimarchi 
Signed-off-by: Jagan Teki 
---
 board/engicam/geam6ul/MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/board/engicam/geam6ul/MAINTAINERS 
b/board/engicam/geam6ul/MAINTAINERS
index 6691450..079370c 100644
--- a/board/engicam/geam6ul/MAINTAINERS
+++ b/board/engicam/geam6ul/MAINTAINERS
@@ -4,3 +4,4 @@ S:  Maintained
 F: board/engicam/geam6ul
 F: include/configs/imx6ul_geam.h
 F: configs/imx6ul_geam_mmc_defconfig
+F: configs/imx6ul_geam_nand_defconfig
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 10/10] configs: engicam: Add fitboot env support

2016-12-21 Thread Jagan Teki
From: Jagan Teki 

Add FIT image booting from MMC device, during MMC bootcmd
u-boot env script look for bootscript, else fit image or else
finally look for legacy image uImage.

Cc: Stefano Babic 
Cc: Matteo Lisi 
Cc: Michael Trimarchi 
Signed-off-by: Jagan Teki 
---
 include/configs/imx6qdl_icore.h | 13 +++--
 include/configs/imx6qdl_icore_rqs.h | 13 +++--
 include/configs/imx6ul_geam.h   | 13 +++--
 3 files changed, 33 insertions(+), 6 deletions(-)

diff --git a/include/configs/imx6qdl_icore.h b/include/configs/imx6qdl_icore.h
index 2c255e2..eb83d23 100644
--- a/include/configs/imx6qdl_icore.h
+++ b/include/configs/imx6qdl_icore.h
@@ -39,6 +39,7 @@
"script=boot.scr\0" \
"splashpos=m,m\0" \
"image=uImage\0" \
+   "fit_image=fit.itb\0" \
"console=ttymxc3\0" \
"fdt_high=0x\0" \
"fdt_file=" CONFIG_DEFAULT_FDT_FILE "\0" \
@@ -59,6 +60,10 @@
"source\0" \
"loadimage=fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${image}\0" \
"loadfdt=fatload mmc ${mmcdev}:${mmcpart} ${fdt_addr} ${fdt_file}\0" \
+   "loadfit=fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${fit_image}\0" \
+   "fitboot=echo Booting FIT image from mmc ...; " \
+   "run mmcargs; " \
+   "bootm ${loadaddr}\0" \
"mmcboot=echo Booting from mmc ...; " \
"run mmcargs; " \
"if test ${boot_fdt} = yes || test ${boot_fdt} = try; then " \
@@ -94,8 +99,12 @@
"if run loadbootscript; then " \
"run bootscript; " \
"else " \
-   "if run loadimage; then " \
-   "run mmcboot; " \
+   "if run loadfit; then " \
+   "run fitboot; " \
+   "else " \
+   "if run loadimage; then " \
+   "run mmcboot; " \
+   "fi; " \
"fi; " \
"fi; " \
"fi"
diff --git a/include/configs/imx6qdl_icore_rqs.h 
b/include/configs/imx6qdl_icore_rqs.h
index 6f297c1..6f7195d 100644
--- a/include/configs/imx6qdl_icore_rqs.h
+++ b/include/configs/imx6qdl_icore_rqs.h
@@ -34,6 +34,7 @@
 #define CONFIG_EXTRA_ENV_SETTINGS \
"script=boot.scr\0" \
"image=uImage\0" \
+   "fit_image=fit.itb\0" \
"console=ttymxc3\0" \
"fdt_high=0x\0" \
"fdt_file=" CONFIG_DEFAULT_FDT_FILE "\0" \
@@ -51,6 +52,10 @@
"source\0" \
"loadimage=fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${image}\0" \
"loadfdt=fatload mmc ${mmcdev}:${mmcpart} ${fdt_addr} ${fdt_file}\0" \
+   "loadfit=fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${fit_image}\0" \
+   "fitboot=echo Booting FIT image from mmc ...; " \
+   "run mmcargs; " \
+   "bootm ${loadaddr}\0" \
"mmcboot=echo Booting from mmc ...; " \
"run mmcargs; " \
"if test ${boot_fdt} = yes || test ${boot_fdt} = try; then " \
@@ -73,8 +78,12 @@
"if run loadbootscript; then " \
"run bootscript; " \
"else " \
-   "if run loadimage; then " \
-   "run mmcboot; " \
+   "if run loadfit; then " \
+   "run fitboot; " \
+   "else " \
+   "if run loadimage; then " \
+   "run mmcboot; " \
+   "fi; " \
"fi; " \
"fi; " \
"fi"
diff --git a/include/configs/imx6ul_geam.h b/include/configs/imx6ul_geam.h
index 8d91fed..23fa3ee 100644
--- a/include/configs/imx6ul_geam.h
+++ b/include/configs/imx6ul_geam.h
@@ -38,6 +38,7 @@
 #define CONFIG_EXTRA_ENV_SETTINGS \
"script=boot.scr\0" \
"image=uImage\0" \
+   "fit_image=fit.itb\0" \
"console=ttymxc0\0" \
"fdt_high=0x\0" \
"fdt_file=" CONFIG_DEFAULT_FDT_FILE "\0" \
@@ -58,6 +59,10 @@
"source\0" \
"loadimage=fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${image}\0" \
"loadfdt=fatload mmc ${mmcdev}:${mmcpart} ${fdt_addr} ${fdt_file}\0" \
+   "loadfit=fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${fit_image}\0" \
+   "fitboot=echo Booting FIT image from mmc ...; " \
+   "run mmcargs; " \
+   "bootm ${loadaddr}\0" \
"mmcboot=echo Booting from mmc ...; " \
"run mmcargs; " \
"if test ${boot_fdt} = yes || test ${boot_fdt} = try; then " \
@@ -92,8 +97,12 @@
"if run loadbootscript; then " \
"run bootscript; " \
"else " \
-   "if run loadimage; then " \
-   "run mmcboot

[U-Boot] [PATCH 07/10] defconfigs: imx6: engicam: Enable FIT

2016-12-21 Thread Jagan Teki
From: Jagan Teki 

Enable Flattened Image Tree support for all Engicam boards.

Cc: Stefano Babic 
Cc: Matteo Lisi 
Cc: Michael Trimarchi 
Signed-off-by: Jagan Teki 
---
 configs/imx6dl_icore_mmc_defconfig | 3 +++
 configs/imx6dl_icore_nand_defconfig| 3 +++
 configs/imx6dl_icore_rqs_mmc_defconfig | 3 +++
 configs/imx6q_icore_mmc_defconfig  | 3 +++
 configs/imx6q_icore_nand_defconfig | 3 +++
 configs/imx6q_icore_rqs_mmc_defconfig  | 3 +++
 configs/imx6ul_geam_mmc_defconfig  | 3 +++
 configs/imx6ul_geam_nand_defconfig | 3 +++
 include/configs/imx6qdl_icore.h| 7 +++
 include/configs/imx6qdl_icore_rqs.h| 7 +++
 include/configs/imx6ul_geam.h  | 7 +++
 11 files changed, 45 insertions(+)

diff --git a/configs/imx6dl_icore_mmc_defconfig 
b/configs/imx6dl_icore_mmc_defconfig
index 2ac0e66..9a50349 100644
--- a/configs/imx6dl_icore_mmc_defconfig
+++ b/configs/imx6dl_icore_mmc_defconfig
@@ -12,6 +12,9 @@ CONFIG_DISPLAY_CPUINFO=y
 CONFIG_HUSH_PARSER=y
 CONFIG_AUTO_COMPLETE=y
 CONFIG_SYS_MAXARGS=32
+CONFIG_FIT=y
+CONFIG_FIT_VERBOSE=y
+CONFIG_FIT_SIGNATURE=y
 # CONFIG_CMD_IMLS is not set
 # CONFIG_BLK is not set
 # CONFIG_DM_MMC_OPS is not set
diff --git a/configs/imx6dl_icore_nand_defconfig 
b/configs/imx6dl_icore_nand_defconfig
index 67397c3..957bb7d 100644
--- a/configs/imx6dl_icore_nand_defconfig
+++ b/configs/imx6dl_icore_nand_defconfig
@@ -12,6 +12,9 @@ CONFIG_DISPLAY_CPUINFO=y
 CONFIG_HUSH_PARSER=y
 CONFIG_AUTO_COMPLETE=y
 CONFIG_SYS_MAXARGS=32
+CONFIG_FIT=y
+CONFIG_FIT_VERBOSE=y
+CONFIG_FIT_SIGNATURE=y
 # CONFIG_CMD_IMLS is not set
 CONFIG_CMD_GPIO=y
 CONFIG_CMD_I2C=y
diff --git a/configs/imx6dl_icore_rqs_mmc_defconfig 
b/configs/imx6dl_icore_rqs_mmc_defconfig
index 0757371..9d91837 100644
--- a/configs/imx6dl_icore_rqs_mmc_defconfig
+++ b/configs/imx6dl_icore_rqs_mmc_defconfig
@@ -12,6 +12,9 @@ CONFIG_DISPLAY_CPUINFO=y
 CONFIG_HUSH_PARSER=y
 CONFIG_AUTO_COMPLETE=y
 CONFIG_SYS_MAXARGS=32
+CONFIG_FIT=y
+CONFIG_FIT_VERBOSE=y
+CONFIG_FIT_SIGNATURE=y
 # CONFIG_CMD_IMLS is not set
 # CONFIG_BLK is not set
 # CONFIG_DM_MMC_OPS is not set
diff --git a/configs/imx6q_icore_mmc_defconfig 
b/configs/imx6q_icore_mmc_defconfig
index 551c688..d3d210f 100644
--- a/configs/imx6q_icore_mmc_defconfig
+++ b/configs/imx6q_icore_mmc_defconfig
@@ -12,6 +12,9 @@ CONFIG_DISPLAY_CPUINFO=y
 CONFIG_HUSH_PARSER=y
 CONFIG_AUTO_COMPLETE=y
 CONFIG_SYS_MAXARGS=32
+CONFIG_FIT=y
+CONFIG_FIT_VERBOSE=y
+CONFIG_FIT_SIGNATURE=y
 # CONFIG_CMD_IMLS is not set
 # CONFIG_BLK is not set
 # CONFIG_DM_MMC_OPS is not set
diff --git a/configs/imx6q_icore_nand_defconfig 
b/configs/imx6q_icore_nand_defconfig
index 350dc0d..50ad76c 100644
--- a/configs/imx6q_icore_nand_defconfig
+++ b/configs/imx6q_icore_nand_defconfig
@@ -12,6 +12,9 @@ CONFIG_DISPLAY_CPUINFO=y
 CONFIG_HUSH_PARSER=y
 CONFIG_AUTO_COMPLETE=y
 CONFIG_SYS_MAXARGS=32
+CONFIG_FIT=y
+CONFIG_FIT_VERBOSE=y
+CONFIG_FIT_SIGNATURE=y
 # CONFIG_CMD_IMLS is not set
 CONFIG_CMD_GPIO=y
 CONFIG_CMD_I2C=y
diff --git a/configs/imx6q_icore_rqs_mmc_defconfig 
b/configs/imx6q_icore_rqs_mmc_defconfig
index e29afa0..1dcb232 100644
--- a/configs/imx6q_icore_rqs_mmc_defconfig
+++ b/configs/imx6q_icore_rqs_mmc_defconfig
@@ -12,6 +12,9 @@ CONFIG_DISPLAY_CPUINFO=y
 CONFIG_HUSH_PARSER=y
 CONFIG_AUTO_COMPLETE=y
 CONFIG_SYS_MAXARGS=32
+CONFIG_FIT=y
+CONFIG_FIT_VERBOSE=y
+CONFIG_FIT_SIGNATURE=y
 # CONFIG_CMD_IMLS is not set
 # CONFIG_BLK is not set
 # CONFIG_DM_MMC_OPS is not set
diff --git a/configs/imx6ul_geam_mmc_defconfig 
b/configs/imx6ul_geam_mmc_defconfig
index 4aaf93d..ca59fa5 100644
--- a/configs/imx6ul_geam_mmc_defconfig
+++ b/configs/imx6ul_geam_mmc_defconfig
@@ -12,6 +12,9 @@ CONFIG_DISPLAY_CPUINFO=y
 CONFIG_HUSH_PARSER=y
 CONFIG_AUTO_COMPLETE=y
 CONFIG_SYS_MAXARGS=32
+CONFIG_FIT=y
+CONFIG_FIT_VERBOSE=y
+CONFIG_FIT_SIGNATURE=y
 # CONFIG_CMD_IMLS is not set
 # CONFIG_BLK is not set
 # CONFIG_DM_MMC_OPS is not set
diff --git a/configs/imx6ul_geam_nand_defconfig 
b/configs/imx6ul_geam_nand_defconfig
index 6d8336d..71ef06a 100644
--- a/configs/imx6ul_geam_nand_defconfig
+++ b/configs/imx6ul_geam_nand_defconfig
@@ -12,6 +12,9 @@ CONFIG_DISPLAY_CPUINFO=y
 CONFIG_HUSH_PARSER=y
 CONFIG_AUTO_COMPLETE=y
 CONFIG_SYS_MAXARGS=32
+CONFIG_FIT=y
+CONFIG_FIT_VERBOSE=y
+CONFIG_FIT_SIGNATURE=y
 # CONFIG_CMD_IMLS is not set
 # CONFIG_BLK is not set
 # CONFIG_DM_MMC_OPS is not set
diff --git a/include/configs/imx6qdl_icore.h b/include/configs/imx6qdl_icore.h
index 9d5951f..2c07a9b 100644
--- a/include/configs/imx6qdl_icore.h
+++ b/include/configs/imx6qdl_icore.h
@@ -121,6 +121,13 @@
 #define CONFIG_SYS_INIT_SP_ADDR(CONFIG_SYS_INIT_RAM_ADDR + \
CONFIG_SYS_INIT_SP_OFFSET)
 
+/* FIT */
+#ifdef CONFIG_FIT
+# define CONFIG_HASH_VERIFY
+# define CONFIG_SHA1
+# define CONFIG_SHA256
+#endif
+
 /* UART */
 #ifdef CONFIG_MXC_UART
 # define CONFIG_MXC_UART_BASE  UART4_BASE
diff --git a/include/configs/imx6qdl_icore_rqs.h 
b/inclu

Re: [U-Boot] [PATCH] sf: ids: Correct Micron n25q128 flags

2016-12-21 Thread Jagan Teki
On Fri, Dec 16, 2016 at 11:38 AM, Phil Edworthy
 wrote:
> The n25q128 devices support 4K erase.

It's OK, but go with 64K erase.

thanks!
-- 
Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] sf: ids: Correct Micron n25q128 flags

2016-12-21 Thread Phil Edworthy
Hi Jagan,

On 21 December 2016 11:10, Jagan Teki wrote:
> On Fri, Dec 16, 2016 at 11:38 AM, Phil Edworthy
>  wrote:
> > The n25q128 devices support 4K erase.
> 
> It's OK, but go with 64K erase.
Sorry, I don’t understand your comment.
Do you mean we should stick with 64K erase?
If so, then what about all the other Micron devices that have SECT_4K?

Thanks
Phil
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] spi: Zap armada100_spi.c and env

2016-12-21 Thread Jagan Teki
On Fri, Nov 25, 2016 at 6:06 AM, Ajay Bhargav
 wrote:
> I will update driver and submit. Thanks for notification.

Any further update? I'm going with this and submit the dm driver
whenever possible.

thanks!
-- 
Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] sf: ids: Correct Micron n25q128 flags

2016-12-21 Thread Jagan Teki
On Wed, Dec 21, 2016 at 12:15 PM, Phil Edworthy
 wrote:
> Hi Jagan,
>
> On 21 December 2016 11:10, Jagan Teki wrote:
>> On Fri, Dec 16, 2016 at 11:38 AM, Phil Edworthy
>>  wrote:
>> > The n25q128 devices support 4K erase.
>>
>> It's OK, but go with 64K erase.
> Sorry, I don’t understand your comment.
> Do you mean we should stick with 64K erase?
> If so, then what about all the other Micron devices that have SECT_4K?

It's because we have tested 64K on these parts and it's fine why can't
we move 4K because 64K look faster, did you find any issue with 64K on
this specific part?

thanks!
-- 
Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] zynq: spi: Honour the activation / deactivation delay

2016-12-21 Thread Jagan Teki
On Sun, Dec 11, 2016 at 9:27 PM, Simon Glass  wrote:
> On 8 December 2016 at 15:11, Moritz Fischer  wrote:
>> This is not currently implemented. Add support for this so that the
>> Chrome OS EC can be used reliably.
>>
>> Signed-off-by: Moritz Fischer 
>> Cc: Jagan Teki 
>> Cc: Simon Glass 
>> Cc: u-boot@lists.denx.de
>> ---
>>  drivers/spi/zynq_spi.c | 24 
>>  1 file changed, 24 insertions(+)
>
> Acked-by: Simon Glass 

Applied to u-boot-spi/next

thanks!
-- 
Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] sf: ids: Correct Micron n25q128 flags

2016-12-21 Thread Phil Edworthy
Hi Jagan,

On 21 December 2016 11:21, Jagan Teki wrote:
> On Wed, Dec 21, 2016 at 12:15 PM, Phil Edworthy
>  wrote:
> > On 21 December 2016 11:10, Jagan Teki wrote:
> >> On Fri, Dec 16, 2016 at 11:38 AM, Phil Edworthy
> >>  wrote:
> >> > The n25q128 devices support 4K erase.
> >>
> >> It's OK, but go with 64K erase.
> > Sorry, I don’t understand your comment.
> > Do you mean we should stick with 64K erase?
> > If so, then what about all the other Micron devices that have SECT_4K?
> 
> It's because we have tested 64K on these parts and it's fine why can't
> we move 4K because 64K look faster, did you find any issue with 64K on
> this specific part?
No, I didn’t find any problems with it at 64K erase.
As you say, erasing 64K is much faster than 16 x 4K erase. So should we
move the other Micron parts to 64K?

Thanks
Phil
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] Update Maintainer and Auther's email address

2016-12-21 Thread Ajay Bhargav
I am not longer using my old email address
"ajay.bhar...@einfochips.com". For U-Boot development email address is
now updated to cont...@8051projects.net

Signed-off-by: Ajay Bhargav 
---
 arch/arm/include/asm/arch-armada100/gpio.h   | 2 +-
 arch/arm/include/asm/arch-armada100/spi.h| 2 +-
 arch/arm/include/asm/arch-armada100/utmi-armada100.h | 2 +-
 board/Marvell/gplugd/MAINTAINERS | 2 +-
 board/Marvell/gplugd/Makefile| 2 +-
 board/Marvell/gplugd/gplugd.c| 2 +-
 drivers/gpio/mvgpio.c| 2 +-
 drivers/gpio/mvgpio.h| 2 +-
 drivers/net/armada100_fec.c  | 2 +-
 drivers/net/armada100_fec.h  | 2 +-
 drivers/spi/armada100_spi.c  | 2 +-
 drivers/usb/host/ehci-armada100.c| 2 +-
 drivers/usb/host/utmi-armada100.c| 2 +-
 include/configs/gplugd.h | 2 +-
 14 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/arch/arm/include/asm/arch-armada100/gpio.h 
b/arch/arm/include/asm/arch-armada100/gpio.h
index 4927abea9c..54e6ccf0c2 100644
--- a/arch/arm/include/asm/arch-armada100/gpio.h
+++ b/arch/arm/include/asm/arch-armada100/gpio.h
@@ -1,7 +1,7 @@
 /*
  * (C) Copyright 2011
  * eInfochips Ltd. 
- * Written-by: Ajay Bhargav 
+ * Written-by: Ajay Bhargav 
  *
  * (C) Copyright 2010
  * Marvell Semiconductor 
diff --git a/arch/arm/include/asm/arch-armada100/spi.h 
b/arch/arm/include/asm/arch-armada100/spi.h
index 9efa1bf1e7..631acf3656 100644
--- a/arch/arm/include/asm/arch-armada100/spi.h
+++ b/arch/arm/include/asm/arch-armada100/spi.h
@@ -1,7 +1,7 @@
 /*
  * (C) Copyright 2011
  * eInfochips Ltd. 
- * Written-by: Ajay Bhargav 
+ * Written-by: Ajay Bhargav 
  *
  * (C) Copyright 2010
  * Marvell Semiconductor 
diff --git a/arch/arm/include/asm/arch-armada100/utmi-armada100.h 
b/arch/arm/include/asm/arch-armada100/utmi-armada100.h
index 953dd44138..fca18276d5 100644
--- a/arch/arm/include/asm/arch-armada100/utmi-armada100.h
+++ b/arch/arm/include/asm/arch-armada100/utmi-armada100.h
@@ -1,7 +1,7 @@
 /*
  * (C) Copyright 2012
  * eInfochips Ltd. 
- * Written-by: Ajay Bhargav 
+ * Written-by: Ajay Bhargav 
  *
  * (C) Copyright 2009
  * Marvell Semiconductor 
diff --git a/board/Marvell/gplugd/MAINTAINERS b/board/Marvell/gplugd/MAINTAINERS
index 320bc09de5..197c6a01dd 100644
--- a/board/Marvell/gplugd/MAINTAINERS
+++ b/board/Marvell/gplugd/MAINTAINERS
@@ -1,5 +1,5 @@
 GPLUGD BOARD
-M: Ajay Bhargav 
+M: Ajay Bhargav 
 S: Maintained
 F: board/Marvell/gplugd/
 F: include/configs/gplugd.h
diff --git a/board/Marvell/gplugd/Makefile b/board/Marvell/gplugd/Makefile
index b38457845d..8929da5be2 100644
--- a/board/Marvell/gplugd/Makefile
+++ b/board/Marvell/gplugd/Makefile
@@ -1,7 +1,7 @@
 #
 # (C) Copyright 2011
 # eInfochips Ltd. 
-# Written-by: Ajay Bhargav 
+# Written-by: Ajay Bhargav 
 #
 # Based on Aspenite:
 # (C) Copyright 2010
diff --git a/board/Marvell/gplugd/gplugd.c b/board/Marvell/gplugd/gplugd.c
index 0e8ebcc70d..c8c4ad2a98 100644
--- a/board/Marvell/gplugd/gplugd.c
+++ b/board/Marvell/gplugd/gplugd.c
@@ -1,7 +1,7 @@
 /*
  * (C) Copyright 2011
  * eInfochips Ltd. 
- * Written-by: Ajay Bhargav 
+ * Written-by: Ajay Bhargav 
  *
  * Based on Aspenite:
  * (C) Copyright 2010
diff --git a/drivers/gpio/mvgpio.c b/drivers/gpio/mvgpio.c
index 8bfbc3a585..407385bdf0 100644
--- a/drivers/gpio/mvgpio.c
+++ b/drivers/gpio/mvgpio.c
@@ -1,7 +1,7 @@
 /*
  * (C) Copyright 2011
  * eInfochips Ltd. 
- * Written-by: Ajay Bhargav 
+ * Written-by: Ajay Bhargav 
  *
  * (C) Copyright 2010
  * Marvell Semiconductor 
diff --git a/drivers/gpio/mvgpio.h b/drivers/gpio/mvgpio.h
index 1de739568a..1eb7acd451 100644
--- a/drivers/gpio/mvgpio.h
+++ b/drivers/gpio/mvgpio.h
@@ -1,7 +1,7 @@
 /*
  * (C) Copyright 2011
  * eInfochips Ltd. 
- * Written-by: Ajay Bhargav 
+ * Written-by: Ajay Bhargav 
  *
  * (C) Copyright 2010
  * Marvell Semiconductor 
diff --git a/drivers/net/armada100_fec.c b/drivers/net/armada100_fec.c
index ba2cb1ad6d..caca8fde9f 100644
--- a/drivers/net/armada100_fec.c
+++ b/drivers/net/armada100_fec.c
@@ -1,7 +1,7 @@
 /*
  * (C) Copyright 2011
  * eInfochips Ltd. 
- * Written-by: Ajay Bhargav 
+ * Written-by: Ajay Bhargav 
  *
  * (C) Copyright 2010
  * Marvell Semiconductor 
diff --git a/drivers/net/armada100_fec.h b/drivers/net/armada100_fec.h
index 5a0a3d982d..826048ea1c 100644
--- a/drivers/net/armada100_fec.h
+++ b/drivers/net/armada100_fec.h
@@ -1,7 +1,7 @@
 /*
  * (C) Copyright 2011
  * eInfochips Ltd. 
- * Written-by: Ajay Bhargav 
+ * Written-by: Ajay Bhargav 
  *
  * (C) Copyright 2010
  * Marvell Semiconductor 
diff --git a/drivers/spi/armada100_spi.c b/drivers/spi/armada100_spi.c
index 53aaf95340..acd65b1c64 100644
--- a/drivers/spi/armada100_spi.c
+++ b/drivers/spi/armada100_spi.c
@@ -1,7 +1,7 @@
 /*
  * (C) C

[U-Boot] driver micrel ksz8873 in U-BOOT

2016-12-21 Thread Дмитрий Мелоневич
I am a student, and I have the task to collect the U-boot driver micrel ksz8873 
ethernet switch, but I do not get it, you can describe what you need to do.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 1/2] Convert CONFIG_SPL_RAM_DEVICE to defconfig

2016-12-21 Thread Tom Rini
On Tue, Dec 20, 2016 at 05:54:58PM +0100, Stefan Agner wrote:
> From: Stefan Agner 
> 
> This converts the following to Kconfig:
>   CONFIG_SPL_RAM_DEVICE
> 
> Signed-off-by: Stefan Agner 
> ---
> 
> Changes in v2: None
> 
>  common/spl/Kconfig | 8 
>  configs/apalis_t30_defconfig   | 1 +
>  configs/beaver_defconfig   | 1 +
>  configs/cardhu_defconfig   | 1 +
>  configs/cei-tk1-som_defconfig  | 1 +
>  configs/colibri_t20_defconfig  | 1 +
>  configs/colibri_t30_defconfig  | 1 +
>  configs/dalmore_defconfig  | 1 +
>  configs/harmony_defconfig  | 1 +
>  configs/jetson-tk1_defconfig   | 1 +
>  configs/medcom-wide_defconfig  | 1 +
>  configs/nyan-big_defconfig | 1 +
>  configs/paz00_defconfig| 1 +
>  configs/plutux_defconfig   | 1 +
>  configs/seaboard_defconfig | 1 +
>  configs/socfpga_arria5_defconfig   | 1 +
>  configs/socfpga_cyclone5_defconfig | 1 +
>  configs/socfpga_de0_nano_soc_defconfig | 1 +
>  configs/socfpga_de1_soc_defconfig  | 1 +
>  configs/socfpga_is1_defconfig  | 1 +
>  configs/socfpga_mcvevk_defconfig   | 1 +
>  configs/socfpga_sockit_defconfig   | 1 +
>  configs/socfpga_socrates_defconfig | 1 +
>  configs/socfpga_sr1500_defconfig   | 1 +
>  configs/socfpga_vining_fpga_defconfig  | 1 +
>  configs/tec-ng_defconfig   | 1 +
>  configs/tec_defconfig  | 1 +
>  configs/topic_miami_defconfig  | 1 +
>  configs/topic_miamiplus_defconfig  | 1 +
>  configs/trimslice_defconfig| 1 +
>  configs/venice2_defconfig  | 1 +
>  configs/ventana_defconfig  | 1 +
>  configs/whistler_defconfig | 1 +
>  configs/zynq_microzed_defconfig| 1 +
>  configs/zynq_picozed_defconfig | 1 +
>  configs/zynq_zc702_defconfig   | 1 +
>  configs/zynq_zc706_defconfig   | 1 +
>  configs/zynq_zc770_xm010_defconfig | 1 +
>  configs/zynq_zc770_xm011_defconfig | 1 +
>  configs/zynq_zc770_xm012_defconfig | 1 +
>  configs/zynq_zc770_xm013_defconfig | 1 +
>  configs/zynq_zed_defconfig | 1 +
>  configs/zynq_zybo_defconfig| 1 +
>  include/configs/microblaze-generic.h   | 1 -
>  include/configs/socfpga_common.h   | 1 -
>  include/configs/tegra-common.h | 1 -
>  include/configs/xilinx_zynqmp.h| 1 -
>  include/configs/zynq-common.h  | 1 -
>  48 files changed, 50 insertions(+), 5 deletions(-)
> 
> diff --git a/common/spl/Kconfig b/common/spl/Kconfig
> index cba51f5..8f779e6 100644
> --- a/common/spl/Kconfig
> +++ b/common/spl/Kconfig
> @@ -449,6 +449,14 @@ config SPL_POWER_SUPPORT
> in drivers/power, drivers/power/pmic and drivers/power/regulator
> as part of an SPL build.
>  
> +config SPL_RAM_DEVICE
> + bool "Support booting from preloaded image in RAM"
> + depends on SPL
> + help
> +   Enable booting of an image already loaded in RAM. The image has to
> +   be already in memory when SPL takes over, e.g. loaded by the boot
> +   ROM.

We should do this as a 'default y if ...' for
microblaze/socfpga/tegra/xilinx, until we can import the 'imply' keyword
from the kernel and use that instead.  Thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 2/2] spl: move RAM boot support in separate file

2016-12-21 Thread Tom Rini
On Tue, Dec 20, 2016 at 05:54:59PM +0100, Stefan Agner wrote:
> From: Stefan Agner 
> 
> Add a new top-level config option so support booting an image stored
> in RAM. This allows to move the RAM boot support into a sparate file
> and having a single condition to compile that file.
> 
> Signed-off-by: Stefan Agner 
> ---
> The series has been build tested using buildman.
> 
> $ ./tools/buildman/buildman
> boards.cfg is up to date. Nothing to do.
> Building current source for 1230 boards (8 threads, 1 job per thread)
>  10190  211 /1230   0:00:03  : P1022DS_NAND
> 
> --
> Stefan
> 
> Changes in v2:
> - Added missing new file spl_ram.c
> - Add new config also to defconfig file
> 
>  common/spl/Kconfig | 11 -
>  common/spl/Makefile|  1 +
>  common/spl/spl.c   | 58 ---
>  common/spl/spl_ram.c   | 73 
> ++
>  configs/apalis_t30_defconfig   |  1 +
>  configs/beaver_defconfig   |  1 +
>  configs/cardhu_defconfig   |  1 +
>  configs/cei-tk1-som_defconfig  |  1 +
>  configs/colibri_t20_defconfig  |  1 +
>  configs/colibri_t30_defconfig  |  1 +
>  configs/dalmore_defconfig  |  1 +
>  configs/harmony_defconfig  |  1 +
>  configs/jetson-tk1_defconfig   |  1 +
>  configs/medcom-wide_defconfig  |  1 +
>  configs/nyan-big_defconfig |  1 +
>  configs/paz00_defconfig|  1 +
>  configs/plutux_defconfig   |  1 +
>  configs/seaboard_defconfig |  1 +
>  configs/socfpga_arria5_defconfig   |  1 +
>  configs/socfpga_cyclone5_defconfig |  2 +
>  configs/socfpga_de0_nano_soc_defconfig |  1 +
>  configs/socfpga_de1_soc_defconfig  |  1 +
>  configs/socfpga_is1_defconfig  |  1 +
>  configs/socfpga_mcvevk_defconfig   |  1 +
>  configs/socfpga_sockit_defconfig   |  1 +
>  configs/socfpga_socrates_defconfig |  1 +
>  configs/socfpga_sr1500_defconfig   |  1 +
>  configs/socfpga_vining_fpga_defconfig  |  1 +
>  configs/tec-ng_defconfig   |  1 +
>  configs/tec_defconfig  |  1 +
>  configs/topic_miami_defconfig  |  1 +
>  configs/topic_miamiplus_defconfig  |  1 +
>  configs/trimslice_defconfig|  1 +
>  configs/venice2_defconfig  |  1 +
>  configs/ventana_defconfig  |  1 +
>  configs/whistler_defconfig |  1 +
>  configs/zynq_microzed_defconfig|  1 +
>  configs/zynq_picozed_defconfig |  1 +
>  configs/zynq_zc702_defconfig   |  1 +
>  configs/zynq_zc706_defconfig   |  1 +
>  configs/zynq_zc770_xm010_defconfig |  1 +
>  configs/zynq_zc770_xm011_defconfig |  1 +
>  configs/zynq_zc770_xm012_defconfig |  1 +
>  configs/zynq_zc770_xm013_defconfig |  1 +
>  configs/zynq_zed_defconfig |  1 +
>  configs/zynq_zybo_defconfig|  1 +
>  46 files changed, 126 insertions(+), 60 deletions(-)
>  create mode 100644 common/spl/spl_ram.c
> 
> diff --git a/common/spl/Kconfig b/common/spl/Kconfig
> index 8f779e6..802779b 100644
> --- a/common/spl/Kconfig
> +++ b/common/spl/Kconfig
> @@ -449,9 +449,16 @@ config SPL_POWER_SUPPORT
> in drivers/power, drivers/power/pmic and drivers/power/regulator
> as part of an SPL build.
>  
> +config SPL_RAM_SUPPORT
> + bool "Support booting from RAM"
> + depends on SPL
> + help
> +   Enable booting of an image in RAM. The image can be preloaded or
> +   it can be loaded by SPL directly into RAM (e.g. using USB).
> +
>  config SPL_RAM_DEVICE
>   bool "Support booting from preloaded image in RAM"
> - depends on SPL
> + depends on SPL_RAM_SUPPORT

default y if SPL_RAM_SUPPORT or if ... list of targets from
SPL_RAM_SUPPORT, I'm not quite sure which makes more sense but I suspect
the latter.  Thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Please pull u-boot-marvell/master

2016-12-21 Thread Tom Rini
On Wed, Dec 21, 2016 at 10:56:58AM +0100, Stefan Roese wrote:

> Hi Tom,
> 
> please pull the following A7k/8k fix.
> 
> Thanks,
> Stefan
> 
> The following changes since commit 0bd1f96aa2f18d29e8321e519b0152853e500d4d:
> 
>   Merge git://git.denx.de/u-boot-mpc85xx (2016-12-20 12:20:12 -0500)
> 
> are available in the git repository at:
> 
>   git://www.denx.de/git/u-boot-marvell.git 
> 
> for you to fetch changes up to 0d92f2141ac5ef5c80d13e9501890f914525d43c:
> 
>   arm64: mvebu: Fix A8K memory mapping and add documentation (2016-12-21 
> 09:52:35 +0100)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] driver micrel ksz8873 in U-BOOT

2016-12-21 Thread Fabio Estevam
On Wed, Dec 21, 2016 at 9:42 AM, Дмитрий Мелоневич
 wrote:
> I am a student, and I have the task to collect the U-boot driver micrel 
> ksz8873 ethernet switch, but I do not get it, you can describe what you need 
> to do.

The micrel drivers are available at drivers/net/phy/micrel.c inside
U-Boot, and as you can see there is no support for ksz8873 there yet.

You can look at the kernel source code (drivers/net/phy/micrel.c),
which supports ksz8873 since commit 93272e07d8539 ("net: add micrel
KSZ8873MLL switch support"). This can help you to port the driver to
U-Boot.

Good luck!
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] drivers: usb: Add USB_XHCI_ZYNQMP to Kconfig

2016-12-21 Thread Michal Simek
Move symbol to Kconfig to cleanup configuration file.

Signed-off-by: Michal Simek 
---

 configs/xilinx_zynqmp_ep_defconfig   | 1 +
 configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig | 1 +
 configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig | 1 +
 configs/xilinx_zynqmp_zcu102_defconfig   | 1 +
 configs/xilinx_zynqmp_zcu102_revB_defconfig  | 1 +
 drivers/usb/host/Kconfig | 6 ++
 include/configs/xilinx_zynqmp.h  | 1 -
 scripts/config_whitelist.txt | 1 -
 8 files changed, 11 insertions(+), 2 deletions(-)

diff --git a/configs/xilinx_zynqmp_ep_defconfig 
b/configs/xilinx_zynqmp_ep_defconfig
index b223e110c86a..68475ac8e6cb 100644
--- a/configs/xilinx_zynqmp_ep_defconfig
+++ b/configs/xilinx_zynqmp_ep_defconfig
@@ -65,6 +65,7 @@ CONFIG_DEBUG_UART_ANNOUNCE=y
 CONFIG_USB=y
 CONFIG_USB_XHCI_HCD=y
 CONFIG_USB_XHCI_DWC3=y
+CONFIG_USB_XHCI_ZYNQMP=y
 CONFIG_USB_DWC3=y
 CONFIG_USB_DWC3_GADGET=y
 CONFIG_USB_STORAGE=y
diff --git a/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig 
b/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig
index e3cfcee05971..4dc2197c83a0 100644
--- a/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig
+++ b/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig
@@ -56,6 +56,7 @@ CONFIG_DEBUG_UART_ANNOUNCE=y
 CONFIG_USB=y
 CONFIG_USB_XHCI_HCD=y
 CONFIG_USB_XHCI_DWC3=y
+CONFIG_USB_XHCI_ZYNQMP=y
 CONFIG_USB_DWC3=y
 CONFIG_USB_DWC3_GADGET=y
 CONFIG_USB_ULPI_VIEWPORT=y
diff --git a/configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig 
b/configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig
index f8f596f9df7a..2c46727f7410 100644
--- a/configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig
+++ b/configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig
@@ -57,6 +57,7 @@ CONFIG_DEBUG_UART_ANNOUNCE=y
 CONFIG_USB=y
 CONFIG_USB_XHCI_HCD=y
 CONFIG_USB_XHCI_DWC3=y
+CONFIG_USB_XHCI_ZYNQMP=y
 CONFIG_USB_DWC3=y
 CONFIG_USB_DWC3_GADGET=y
 CONFIG_USB_ULPI_VIEWPORT=y
diff --git a/configs/xilinx_zynqmp_zcu102_defconfig 
b/configs/xilinx_zynqmp_zcu102_defconfig
index 9f2df491ec54..e8cf047e7aed 100644
--- a/configs/xilinx_zynqmp_zcu102_defconfig
+++ b/configs/xilinx_zynqmp_zcu102_defconfig
@@ -57,6 +57,7 @@ CONFIG_DEBUG_UART_ANNOUNCE=y
 CONFIG_USB=y
 CONFIG_USB_XHCI_HCD=y
 CONFIG_USB_XHCI_DWC3=y
+CONFIG_USB_XHCI_ZYNQMP=y
 CONFIG_USB_DWC3=y
 CONFIG_USB_DWC3_GADGET=y
 CONFIG_USB_ULPI_VIEWPORT=y
diff --git a/configs/xilinx_zynqmp_zcu102_revB_defconfig 
b/configs/xilinx_zynqmp_zcu102_revB_defconfig
index b2b1720a03d1..53bac1267e28 100644
--- a/configs/xilinx_zynqmp_zcu102_revB_defconfig
+++ b/configs/xilinx_zynqmp_zcu102_revB_defconfig
@@ -57,6 +57,7 @@ CONFIG_DEBUG_UART_ANNOUNCE=y
 CONFIG_USB=y
 CONFIG_USB_XHCI_HCD=y
 CONFIG_USB_XHCI_DWC3=y
+CONFIG_USB_XHCI_ZYNQMP=y
 CONFIG_USB_DWC3=y
 CONFIG_USB_DWC3_GADGET=y
 CONFIG_USB_ULPI_VIEWPORT=y
diff --git a/drivers/usb/host/Kconfig b/drivers/usb/host/Kconfig
index b9c5fbe381fb..5129a573042c 100644
--- a/drivers/usb/host/Kconfig
+++ b/drivers/usb/host/Kconfig
@@ -37,6 +37,12 @@ config USB_XHCI_ROCKCHIP
help
  Enables support for the on-chip xHCI controller on Rockchip SoCs.
 
+config USB_XHCI_ZYNQMP
+   bool "Support for Xilinx ZynqMP on-chip xHCI USB controller"
+   depends on ARCH_ZYNQMP
+   help
+ Enables support for the on-chip xHCI controller on Xilinx ZynqMP SoCs.
+
 endif # USB_XHCI_HCD
 
 config USB_EHCI_HCD
diff --git a/include/configs/xilinx_zynqmp.h b/include/configs/xilinx_zynqmp.h
index ce95aefbd936..b095d1f97882 100644
--- a/include/configs/xilinx_zynqmp.h
+++ b/include/configs/xilinx_zynqmp.h
@@ -111,7 +111,6 @@
 
 #if defined(CONFIG_ZYNQMP_USB)
 #define CONFIG_SYS_USB_XHCI_MAX_ROOT_PORTS  2
-#define CONFIG_USB_XHCI_ZYNQMP
 
 #define CONFIG_SYS_DFU_DATA_BUF_SIZE   0x180
 #define DFU_DEFAULT_POLL_TIMEOUT   300
diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt
index 8814841e1f44..6b0acd280c74 100644
--- a/scripts/config_whitelist.txt
+++ b/scripts/config_whitelist.txt
@@ -8110,7 +8110,6 @@ CONFIG_USB_XHCI_FSL
 CONFIG_USB_XHCI_KEYSTONE
 CONFIG_USB_XHCI_OMAP
 CONFIG_USB_XHCI_PCI
-CONFIG_USB_XHCI_ZYNQMP
 CONFIG_USER_LOWLEVEL_INIT
 CONFIG_USE_ARCH_MEMCPY
 CONFIG_USE_ARCH_MEMSET
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 3/3] imx: mx6sllevk: add usb support

2016-12-21 Thread Peng Fan
Add usb support for mx6sllevk board.

Signed-off-by: Peng Fan 
Cc: Stefano Babic 
---
 board/freescale/mx6sllevk/mx6sllevk.c | 18 ++
 configs/mx6sllevk_defconfig   |  5 +
 configs/mx6sllevk_plugin_defconfig|  5 +
 include/configs/mx6sllevk.h   |  9 +
 4 files changed, 37 insertions(+)

diff --git a/board/freescale/mx6sllevk/mx6sllevk.c 
b/board/freescale/mx6sllevk/mx6sllevk.c
index 74a27a3..e6679fd 100644
--- a/board/freescale/mx6sllevk/mx6sllevk.c
+++ b/board/freescale/mx6sllevk/mx6sllevk.c
@@ -129,3 +129,21 @@ int mmc_map_to_kernel_blk(int devno)
 {
return devno;
 }
+
+#define USB_OTHERREGS_OFFSET   0x800
+#define UCTRL_PWR_POL  (1 << 9)
+
+int board_ehci_hcd_init(int port)
+{
+   u32 *usbnc_usb_ctrl;
+
+   if (port > 1)
+   return -EINVAL;
+
+   usbnc_usb_ctrl = (u32 *)(USB_BASE_ADDR + USB_OTHERREGS_OFFSET +
+port * 4);
+
+   /* Set Power polarity */
+   setbits_le32(usbnc_usb_ctrl, UCTRL_PWR_POL);
+   return 0;
+}
diff --git a/configs/mx6sllevk_defconfig b/configs/mx6sllevk_defconfig
index 8ae049e..267cdc6 100644
--- a/configs/mx6sllevk_defconfig
+++ b/configs/mx6sllevk_defconfig
@@ -10,6 +10,7 @@ CONFIG_CMD_BOOTZ=y
 CONFIG_CMD_MEMTEST=y
 CONFIG_CMD_MMC=y
 CONFIG_CMD_I2C=y
+CONFIG_CMD_USB=y
 CONFIG_CMD_GPIO=y
 CONFIG_CMD_DHCP=y
 CONFIG_CMD_PING=y
@@ -34,3 +35,7 @@ CONFIG_DM_REGULATOR=y
 CONFIG_DM_REGULATOR_PFUZE100=y
 CONFIG_DM_REGULATOR_FIXED=y
 CONFIG_DM_REGULATOR_GPIO=y
+CONFIG_USB=y
+CONFIG_DM_USB=y
+CONFIG_USB_EHCI_HCD=y
+CONFIG_USB_STORAGE=y
diff --git a/configs/mx6sllevk_plugin_defconfig 
b/configs/mx6sllevk_plugin_defconfig
index e6be979..63d5bbc 100644
--- a/configs/mx6sllevk_plugin_defconfig
+++ b/configs/mx6sllevk_plugin_defconfig
@@ -11,6 +11,7 @@ CONFIG_CMD_BOOTZ=y
 CONFIG_CMD_MEMTEST=y
 CONFIG_CMD_MMC=y
 CONFIG_CMD_I2C=y
+CONFIG_CMD_USB=y
 CONFIG_CMD_GPIO=y
 CONFIG_CMD_DHCP=y
 CONFIG_CMD_PING=y
@@ -35,3 +36,7 @@ CONFIG_DM_REGULATOR=y
 CONFIG_DM_REGULATOR_PFUZE100=y
 CONFIG_DM_REGULATOR_FIXED=y
 CONFIG_DM_REGULATOR_GPIO=y
+CONFIG_USB=y
+CONFIG_DM_USB=y
+CONFIG_USB_EHCI_HCD=y
+CONFIG_USB_STORAGE=y
diff --git a/include/configs/mx6sllevk.h b/include/configs/mx6sllevk.h
index b9f25cf..3f665a3 100644
--- a/include/configs/mx6sllevk.h
+++ b/include/configs/mx6sllevk.h
@@ -149,4 +149,13 @@
 
 #define CONFIG_IOMUX_LPSR
 
+/* USB Configs */
+#ifdef CONFIG_CMD_USB
+#define CONFIG_USB_HOST_ETHER
+#define CONFIG_USB_ETHER_ASIX
+#define CONFIG_USB_ETHER_RTL8152
+#define CONFIG_MXC_USB_PORTSC  (PORT_PTS_UTMI | PORT_PTS_PTW)
+#define CONFIG_USB_MAX_CONTROLLER_COUNT2
+#endif
+
 #endif /* __CONFIG_H */
-- 
2.6.2

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] drivers: usb: Add USB_XHCI_ZYNQMP to Kconfig

2016-12-21 Thread Marek Vasut
On 12/21/2016 02:18 PM, Michal Simek wrote:
> Move symbol to Kconfig to cleanup configuration file.
> 
> Signed-off-by: Michal Simek 

Reviewed-by: Marek Vasut 

> ---
> 
>  configs/xilinx_zynqmp_ep_defconfig   | 1 +
>  configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig | 1 +
>  configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig | 1 +
>  configs/xilinx_zynqmp_zcu102_defconfig   | 1 +
>  configs/xilinx_zynqmp_zcu102_revB_defconfig  | 1 +
>  drivers/usb/host/Kconfig | 6 ++
>  include/configs/xilinx_zynqmp.h  | 1 -
>  scripts/config_whitelist.txt | 1 -
>  8 files changed, 11 insertions(+), 2 deletions(-)
> 
> diff --git a/configs/xilinx_zynqmp_ep_defconfig 
> b/configs/xilinx_zynqmp_ep_defconfig
> index b223e110c86a..68475ac8e6cb 100644
> --- a/configs/xilinx_zynqmp_ep_defconfig
> +++ b/configs/xilinx_zynqmp_ep_defconfig
> @@ -65,6 +65,7 @@ CONFIG_DEBUG_UART_ANNOUNCE=y
>  CONFIG_USB=y
>  CONFIG_USB_XHCI_HCD=y
>  CONFIG_USB_XHCI_DWC3=y
> +CONFIG_USB_XHCI_ZYNQMP=y
>  CONFIG_USB_DWC3=y
>  CONFIG_USB_DWC3_GADGET=y
>  CONFIG_USB_STORAGE=y
> diff --git a/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig 
> b/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig
> index e3cfcee05971..4dc2197c83a0 100644
> --- a/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig
> +++ b/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig
> @@ -56,6 +56,7 @@ CONFIG_DEBUG_UART_ANNOUNCE=y
>  CONFIG_USB=y
>  CONFIG_USB_XHCI_HCD=y
>  CONFIG_USB_XHCI_DWC3=y
> +CONFIG_USB_XHCI_ZYNQMP=y
>  CONFIG_USB_DWC3=y
>  CONFIG_USB_DWC3_GADGET=y
>  CONFIG_USB_ULPI_VIEWPORT=y
> diff --git a/configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig 
> b/configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig
> index f8f596f9df7a..2c46727f7410 100644
> --- a/configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig
> +++ b/configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig
> @@ -57,6 +57,7 @@ CONFIG_DEBUG_UART_ANNOUNCE=y
>  CONFIG_USB=y
>  CONFIG_USB_XHCI_HCD=y
>  CONFIG_USB_XHCI_DWC3=y
> +CONFIG_USB_XHCI_ZYNQMP=y
>  CONFIG_USB_DWC3=y
>  CONFIG_USB_DWC3_GADGET=y
>  CONFIG_USB_ULPI_VIEWPORT=y
> diff --git a/configs/xilinx_zynqmp_zcu102_defconfig 
> b/configs/xilinx_zynqmp_zcu102_defconfig
> index 9f2df491ec54..e8cf047e7aed 100644
> --- a/configs/xilinx_zynqmp_zcu102_defconfig
> +++ b/configs/xilinx_zynqmp_zcu102_defconfig
> @@ -57,6 +57,7 @@ CONFIG_DEBUG_UART_ANNOUNCE=y
>  CONFIG_USB=y
>  CONFIG_USB_XHCI_HCD=y
>  CONFIG_USB_XHCI_DWC3=y
> +CONFIG_USB_XHCI_ZYNQMP=y
>  CONFIG_USB_DWC3=y
>  CONFIG_USB_DWC3_GADGET=y
>  CONFIG_USB_ULPI_VIEWPORT=y
> diff --git a/configs/xilinx_zynqmp_zcu102_revB_defconfig 
> b/configs/xilinx_zynqmp_zcu102_revB_defconfig
> index b2b1720a03d1..53bac1267e28 100644
> --- a/configs/xilinx_zynqmp_zcu102_revB_defconfig
> +++ b/configs/xilinx_zynqmp_zcu102_revB_defconfig
> @@ -57,6 +57,7 @@ CONFIG_DEBUG_UART_ANNOUNCE=y
>  CONFIG_USB=y
>  CONFIG_USB_XHCI_HCD=y
>  CONFIG_USB_XHCI_DWC3=y
> +CONFIG_USB_XHCI_ZYNQMP=y
>  CONFIG_USB_DWC3=y
>  CONFIG_USB_DWC3_GADGET=y
>  CONFIG_USB_ULPI_VIEWPORT=y
> diff --git a/drivers/usb/host/Kconfig b/drivers/usb/host/Kconfig
> index b9c5fbe381fb..5129a573042c 100644
> --- a/drivers/usb/host/Kconfig
> +++ b/drivers/usb/host/Kconfig
> @@ -37,6 +37,12 @@ config USB_XHCI_ROCKCHIP
>   help
> Enables support for the on-chip xHCI controller on Rockchip SoCs.
>  
> +config USB_XHCI_ZYNQMP
> + bool "Support for Xilinx ZynqMP on-chip xHCI USB controller"
> + depends on ARCH_ZYNQMP
> + help
> +   Enables support for the on-chip xHCI controller on Xilinx ZynqMP SoCs.
> +
>  endif # USB_XHCI_HCD
>  
>  config USB_EHCI_HCD
> diff --git a/include/configs/xilinx_zynqmp.h b/include/configs/xilinx_zynqmp.h
> index ce95aefbd936..b095d1f97882 100644
> --- a/include/configs/xilinx_zynqmp.h
> +++ b/include/configs/xilinx_zynqmp.h
> @@ -111,7 +111,6 @@
>  
>  #if defined(CONFIG_ZYNQMP_USB)
>  #define CONFIG_SYS_USB_XHCI_MAX_ROOT_PORTS  2
> -#define CONFIG_USB_XHCI_ZYNQMP
>  
>  #define CONFIG_SYS_DFU_DATA_BUF_SIZE 0x180
>  #define DFU_DEFAULT_POLL_TIMEOUT 300
> diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt
> index 8814841e1f44..6b0acd280c74 100644
> --- a/scripts/config_whitelist.txt
> +++ b/scripts/config_whitelist.txt
> @@ -8110,7 +8110,6 @@ CONFIG_USB_XHCI_FSL
>  CONFIG_USB_XHCI_KEYSTONE
>  CONFIG_USB_XHCI_OMAP
>  CONFIG_USB_XHCI_PCI
> -CONFIG_USB_XHCI_ZYNQMP
>  CONFIG_USER_LOWLEVEL_INIT
>  CONFIG_USE_ARCH_MEMCPY
>  CONFIG_USE_ARCH_MEMSET
> 


-- 
Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 3/3] imx: mx6sllevk: add usb support

2016-12-21 Thread Jagan Teki
On Wed, Dec 21, 2016 at 9:14 AM, Peng Fan  wrote:
> Add usb support for mx6sllevk board.
>
> Signed-off-by: Peng Fan 
> Cc: Stefano Babic 
> ---
>  board/freescale/mx6sllevk/mx6sllevk.c | 18 ++
>  configs/mx6sllevk_defconfig   |  5 +
>  configs/mx6sllevk_plugin_defconfig|  5 +
>  include/configs/mx6sllevk.h   |  9 +
>  4 files changed, 37 insertions(+)
>
> diff --git a/board/freescale/mx6sllevk/mx6sllevk.c 
> b/board/freescale/mx6sllevk/mx6sllevk.c
> index 74a27a3..e6679fd 100644
> --- a/board/freescale/mx6sllevk/mx6sllevk.c
> +++ b/board/freescale/mx6sllevk/mx6sllevk.c
> @@ -129,3 +129,21 @@ int mmc_map_to_kernel_blk(int devno)
>  {
> return devno;
>  }
> +
> +#define USB_OTHERREGS_OFFSET   0x800
> +#define UCTRL_PWR_POL  (1 << 9)
> +
> +int board_ehci_hcd_init(int port)

Can't we do this from driver itself, .probe or somewhere?

> +{
> +   u32 *usbnc_usb_ctrl;
> +
> +   if (port > 1)
> +   return -EINVAL;
> +
> +   usbnc_usb_ctrl = (u32 *)(USB_BASE_ADDR + USB_OTHERREGS_OFFSET +
> +port * 4);
> +
> +   /* Set Power polarity */
> +   setbits_le32(usbnc_usb_ctrl, UCTRL_PWR_POL);
> +   return 0;
> +}
> diff --git a/configs/mx6sllevk_defconfig b/configs/mx6sllevk_defconfig
> index 8ae049e..267cdc6 100644
> --- a/configs/mx6sllevk_defconfig
> +++ b/configs/mx6sllevk_defconfig
> @@ -10,6 +10,7 @@ CONFIG_CMD_BOOTZ=y
>  CONFIG_CMD_MEMTEST=y
>  CONFIG_CMD_MMC=y
>  CONFIG_CMD_I2C=y
> +CONFIG_CMD_USB=y
>  CONFIG_CMD_GPIO=y
>  CONFIG_CMD_DHCP=y
>  CONFIG_CMD_PING=y
> @@ -34,3 +35,7 @@ CONFIG_DM_REGULATOR=y
>  CONFIG_DM_REGULATOR_PFUZE100=y
>  CONFIG_DM_REGULATOR_FIXED=y
>  CONFIG_DM_REGULATOR_GPIO=y
> +CONFIG_USB=y
> +CONFIG_DM_USB=y
> +CONFIG_USB_EHCI_HCD=y
> +CONFIG_USB_STORAGE=y
> diff --git a/configs/mx6sllevk_plugin_defconfig 
> b/configs/mx6sllevk_plugin_defconfig
> index e6be979..63d5bbc 100644
> --- a/configs/mx6sllevk_plugin_defconfig
> +++ b/configs/mx6sllevk_plugin_defconfig
> @@ -11,6 +11,7 @@ CONFIG_CMD_BOOTZ=y
>  CONFIG_CMD_MEMTEST=y
>  CONFIG_CMD_MMC=y
>  CONFIG_CMD_I2C=y
> +CONFIG_CMD_USB=y
>  CONFIG_CMD_GPIO=y
>  CONFIG_CMD_DHCP=y
>  CONFIG_CMD_PING=y
> @@ -35,3 +36,7 @@ CONFIG_DM_REGULATOR=y
>  CONFIG_DM_REGULATOR_PFUZE100=y
>  CONFIG_DM_REGULATOR_FIXED=y
>  CONFIG_DM_REGULATOR_GPIO=y
> +CONFIG_USB=y
> +CONFIG_DM_USB=y
> +CONFIG_USB_EHCI_HCD=y
> +CONFIG_USB_STORAGE=y
> diff --git a/include/configs/mx6sllevk.h b/include/configs/mx6sllevk.h
> index b9f25cf..3f665a3 100644
> --- a/include/configs/mx6sllevk.h
> +++ b/include/configs/mx6sllevk.h
> @@ -149,4 +149,13 @@
>
>  #define CONFIG_IOMUX_LPSR
>
> +/* USB Configs */
> +#ifdef CONFIG_CMD_USB
> +#define CONFIG_USB_HOST_ETHER
> +#define CONFIG_USB_ETHER_ASIX
> +#define CONFIG_USB_ETHER_RTL8152
> +#define CONFIG_MXC_USB_PORTSC  (PORT_PTS_UTMI | PORT_PTS_PTW)
> +#define CONFIG_USB_MAX_CONTROLLER_COUNT2

I think this is even should managed from driver isn't it?

thanks!
-- 
Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 1/3] usb: ehci-mx6: implement ofdata_to_platdata

2016-12-21 Thread Marek Vasut
On 12/21/2016 09:14 AM, Peng Fan wrote:
> Implement ofdata_to_platdata to set the type to host or device.
>  - Check "dr-mode" property.
>  - If there is no "dr-mode", check phy_ctrl for i.MX6
>and phy_status for i.MX7
> 
> Signed-off-by: Peng Fan 
> Cc: Marek Vasut 
> Cc: Simon Glass 
> Cc: Stefano Babic 
> ---
>  drivers/usb/host/ehci-mx6.c | 66 
> +
>  1 file changed, 66 insertions(+)
> 
> diff --git a/drivers/usb/host/ehci-mx6.c b/drivers/usb/host/ehci-mx6.c
> index 48889c1..91413c9 100644
> --- a/drivers/usb/host/ehci-mx6.c
> +++ b/drivers/usb/host/ehci-mx6.c
> @@ -15,10 +15,13 @@
>  #include 
>  #include 
>  #include 
> +#include 
>  #include 
>  
>  #include "ehci.h"
>  
> +DECLARE_GLOBAL_DATA_PTR;
> +
>  #define USB_OTGREGS_OFFSET   0x000
>  #define USB_H1REGS_OFFSET0x200
>  #define USB_H2REGS_OFFSET0x400
> @@ -48,6 +51,7 @@
>  #define ANADIG_USB2_PLL_480_CTRL_EN_USB_CLKS 0x0040
>  
>  #define USBNC_OFFSET 0x200
> +#define USBNC_PHY_STATUS_OFFSET  0x23C
>  #define USBNC_PHYSTATUS_ID_DIG   (1 << 4) /* otg_id status */
>  #define USBNC_PHYCFG2_ACAENB (1 << 4) /* otg_id detection enable */
>  #define UCTRL_PWR_POL(1 << 9) /* OTG Polarity of Power Pin */
> @@ -417,6 +421,67 @@ static const struct ehci_ops mx6_ehci_ops = {
>   .init_after_reset = mx6_init_after_reset
>  };
>  
> +static int ehci_usb_ofdata_to_platdata(struct udevice *dev)
> +{
> + struct usb_platdata *plat = dev_get_platdata(dev);
> + void *__iomem addr = (void *__iomem)dev_get_addr(dev);
> + void *__iomem phy_ctrl, *__iomem phy_status;
> + const void *blob = gd->fdt_blob;
> + int offset = dev->of_offset, phy_off;
> + const char *mode;
> + u32 val;
> +
> + mode = fdt_getprop(blob, offset, "dr_mode", NULL);
> + if (mode) {
> + if (strcmp(mode, "peripheral") == 0)
> + plat->init_type = USB_INIT_DEVICE;
> + else if (strcmp(mode, "host") == 0)
> + plat->init_type = USB_INIT_HOST;
> + else if (strcmp(mode, "otg") == 0)
> + plat->init_type = USB_INIT_HOST;

Shouldn't this case check the phy status register ?

> + else
> + return -EINVAL;
> + } else {

You can probably do return 0 in the if (mode) branch and then indent
this whole else branch to the left.

> + /*
> +  * About fsl,usbphy, Refer to
> +  * Documentation/devicetree/bindings/usb/ci-hdrc-usb2.txt.
> +  */
> + if (is_mx6()) {
> + phy_off = fdtdec_lookup_phandle(blob,
> + offset,
> + "fsl,usbphy");
> + if (phy_off < 0)
> + return -EINVAL;
> +
> + addr = (void __iomem *)fdtdec_get_addr(blob, phy_off,
> +"reg");
> + if ((fdt_addr_t)addr == FDT_ADDR_T_NONE)
> + return -EINVAL;
> +
> + phy_ctrl = (void __iomem *)(addr + USBPHY_CTRL);
> + val = readl(phy_ctrl);
> +
> + if (val & USBPHY_CTRL_OTG_ID)
> + plat->init_type = USB_INIT_DEVICE;
> + else
> + plat->init_type = USB_INIT_HOST;
> + } else if (is_mx7()) {
> + phy_status = (void __iomem *)(addr +
> +   USBNC_PHY_STATUS_OFFSET);
> + val = readl(phy_status);
> +
> + if (val & USBNC_PHYSTATUS_ID_DIG)
> + plat->init_type = USB_INIT_DEVICE;
> + else
> + plat->init_type = USB_INIT_HOST;
> + } else {
> + return -EINVAL;
> + }
> + }
> +
> + return 0;
> +}
> +
>  static int ehci_usb_probe(struct udevice *dev)
>  {
>   struct usb_platdata *plat = dev_get_platdata(dev);
> @@ -460,6 +525,7 @@ U_BOOT_DRIVER(usb_mx6) = {
>   .name   = "ehci_mx6",
>   .id = UCLASS_USB,
>   .of_match = mx6_usb_ids,
> + .ofdata_to_platdata = ehci_usb_ofdata_to_platdata,
>   .probe  = ehci_usb_probe,
>   .remove = ehci_deregister,
>   .ops= &ehci_usb_ops,
> 


-- 
Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 3/3] imx: mx6sllevk: add usb support

2016-12-21 Thread Marek Vasut
On 12/21/2016 09:14 AM, Peng Fan wrote:
> Add usb support for mx6sllevk board.
> 
> Signed-off-by: Peng Fan 
> Cc: Stefano Babic 
> ---
>  board/freescale/mx6sllevk/mx6sllevk.c | 18 ++
>  configs/mx6sllevk_defconfig   |  5 +
>  configs/mx6sllevk_plugin_defconfig|  5 +
>  include/configs/mx6sllevk.h   |  9 +
>  4 files changed, 37 insertions(+)
> 
> diff --git a/board/freescale/mx6sllevk/mx6sllevk.c 
> b/board/freescale/mx6sllevk/mx6sllevk.c
> index 74a27a3..e6679fd 100644
> --- a/board/freescale/mx6sllevk/mx6sllevk.c
> +++ b/board/freescale/mx6sllevk/mx6sllevk.c
> @@ -129,3 +129,21 @@ int mmc_map_to_kernel_blk(int devno)
>  {
>   return devno;
>  }
> +
> +#define USB_OTHERREGS_OFFSET   0x800
> +#define UCTRL_PWR_POL  (1 << 9)
> +
> +int board_ehci_hcd_init(int port)
> +{
> + u32 *usbnc_usb_ctrl;
> +
> + if (port > 1)
> + return -EINVAL;
> +
> + usbnc_usb_ctrl = (u32 *)(USB_BASE_ADDR + USB_OTHERREGS_OFFSET +
> +  port * 4);
> +
> + /* Set Power polarity */
> + setbits_le32(usbnc_usb_ctrl, UCTRL_PWR_POL);
> + return 0;
> +}

Is this function similar to what usb_oc_config() does ?

-- 
Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] usb: storage: Show number of storage devices detected for DM_USB

2016-12-21 Thread Marek Vasut
On 12/21/2016 09:35 AM, Michal Simek wrote:
> By enabling DM_USB information about number of storage devices
> was lost.
> Get this information back simply by printing number of devices detected
> via BLK uclass.
> 
> For example:
> scanning bus 0 for devices... 7 USB Device(s) found
>scanning usb for storage devices... 3 Storage Device(s) found
>scanning usb for ethernet devices... 0 Ethernet Device(s) found
> 
> Signed-off-by: Michal Simek 
> ---
> 
>  cmd/usb.c| 2 +-
>  common/usb_storage.c | 8 
>  2 files changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/cmd/usb.c b/cmd/usb.c
> index 455127c844b9..4fa456e31834 100644
> --- a/cmd/usb.c
> +++ b/cmd/usb.c
> @@ -571,11 +571,11 @@ static void do_usb_start(void)
>   return;
>  
>   /* Driver model will probe the devices as they are found */
> -#ifndef CONFIG_DM_USB
>  # ifdef CONFIG_USB_STORAGE
>   /* try to recognize storage devices immediately */
>   usb_stor_curr_dev = usb_stor_scan(1);
>  # endif
> +#ifndef CONFIG_DM_USB
>  # ifdef CONFIG_USB_KEYBOARD
>   drv_usb_kbd_init();
>  # endif
> diff --git a/common/usb_storage.c b/common/usb_storage.c
> index 0345aa22eff5..b524a15e2bf9 100644
> --- a/common/usb_storage.c
> +++ b/common/usb_storage.c
> @@ -303,7 +303,6 @@ void usb_stor_reset(void)
>   usb_max_devs = 0;
>  }
>  
> -#ifndef CONFIG_DM_USB
>  
> /***
>   * scan the usb and reports device info
>   * to the user if mode = 1
> @@ -311,11 +310,12 @@ void usb_stor_reset(void)
>   */
>  int usb_stor_scan(int mode)
>  {
> - unsigned char i;
> -
>   if (mode == 1)
>   printf("   scanning usb for storage devices... ");
>  
> +#ifndef CONFIG_DM_USB
> + unsigned char i;

Won't this complain about mixing variables and code ? I think it will.
You can use __maybe_unused if you want to avoid excess ifdeffery.

>   usb_disable_asynch(1); /* asynch transfer not allowed */
>  
>   usb_stor_reset();
> @@ -329,12 +329,12 @@ int usb_stor_scan(int mode)
>   } /* for */
>  
>   usb_disable_asynch(0); /* asynch transfer allowed */
> +#endif
>   printf("%d Storage Device(s) found\n", usb_max_devs);
>   if (usb_max_devs > 0)
>   return 0;
>   return -1;
>  }
> -#endif
>  
>  static int usb_stor_irq(struct usb_device *dev)
>  {
> 


-- 
Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] usb: storage: Show number of storage devices detected for DM_USB

2016-12-21 Thread Michal Simek
On 21.12.2016 15:11, Marek Vasut wrote:
> On 12/21/2016 09:35 AM, Michal Simek wrote:
>> By enabling DM_USB information about number of storage devices
>> was lost.
>> Get this information back simply by printing number of devices detected
>> via BLK uclass.
>>
>> For example:
>> scanning bus 0 for devices... 7 USB Device(s) found
>>scanning usb for storage devices... 3 Storage Device(s) found
>>scanning usb for ethernet devices... 0 Ethernet Device(s) found
>>
>> Signed-off-by: Michal Simek 
>> ---
>>
>>  cmd/usb.c| 2 +-
>>  common/usb_storage.c | 8 
>>  2 files changed, 5 insertions(+), 5 deletions(-)
>>
>> diff --git a/cmd/usb.c b/cmd/usb.c
>> index 455127c844b9..4fa456e31834 100644
>> --- a/cmd/usb.c
>> +++ b/cmd/usb.c
>> @@ -571,11 +571,11 @@ static void do_usb_start(void)
>>  return;
>>  
>>  /* Driver model will probe the devices as they are found */
>> -#ifndef CONFIG_DM_USB
>>  # ifdef CONFIG_USB_STORAGE
>>  /* try to recognize storage devices immediately */
>>  usb_stor_curr_dev = usb_stor_scan(1);
>>  # endif
>> +#ifndef CONFIG_DM_USB
>>  # ifdef CONFIG_USB_KEYBOARD
>>  drv_usb_kbd_init();
>>  # endif
>> diff --git a/common/usb_storage.c b/common/usb_storage.c
>> index 0345aa22eff5..b524a15e2bf9 100644
>> --- a/common/usb_storage.c
>> +++ b/common/usb_storage.c
>> @@ -303,7 +303,6 @@ void usb_stor_reset(void)
>>  usb_max_devs = 0;
>>  }
>>  
>> -#ifndef CONFIG_DM_USB
>>  
>> /***
>>   * scan the usb and reports device info
>>   * to the user if mode = 1
>> @@ -311,11 +310,12 @@ void usb_stor_reset(void)
>>   */
>>  int usb_stor_scan(int mode)
>>  {
>> -unsigned char i;
>> -
>>  if (mode == 1)
>>  printf("   scanning usb for storage devices... ");
>>  
>> +#ifndef CONFIG_DM_USB
>> +unsigned char i;
> 
> Won't this complain about mixing variables and code ? I think it will.
> You can use __maybe_unused if you want to avoid excess ifdeffery.

I didn't see this issue on my PC. But I will use travis to validate this
https://travis-ci.org/michalsimek-test/u-boot/builds/185787335

Thanks,
Michal

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 0/3] dm: goni: support the DM PMIC

2016-12-21 Thread Minkyu Kang
Hi,

On 15 December 2016 at 18:21, Jaehoon Chung  wrote:

> This patchest is for supporting pmic driver-model on Goni board.
> (Goni board is S5PC110.)
>
> For using that, add the new file as max8998.c for only driver-model.
> pmic_max8998.c is remained for legacy.
> In future, it might be removed. Instead, max8998 should be used.
>
> *dm tree
>  serial  [ + ]|-- serial@e2900800
>  i2c [   ]`-- i2c-pmic
>  pmic[   ]`-- pmic@66
>
> *dm uclass
> uclass 20: i2c
> - * i2c-pmic @ 47f6c6d0, seq 3, (req 3)
>
> uclass 22: i2c_generic
> uclass 42: pmic
> - * pmic@66 @ 47f6c748, seq 0, (req -1)
>
> * After using pmic command.
> Goni # pmic list
> | Name| Parent name | Parent uclass @
> seq
> | pmic@66 | i2c-pmic| i2c @ 3
>
> Node, it needs to implement the regulator driver for using regulator
> framework.
>
> Jaehoon Chung (3):
>   power: pmic: add the max8998 controller for DM
>   arm: dts: s5pc1xx-goni: add the pmic node for using DM
>   configs: enable the DM_PMIC and DM_I2C_GPIO for max8998 pmic
>
>  arch/arm/dts/s5pc1xx-goni.dts | 165 ++
> 
>  configs/s5p_goni_defconfig|   3 +
>  drivers/power/pmic/Kconfig|   7 ++
>  drivers/power/pmic/Makefile   |   1 +
>  drivers/power/pmic/max8998.c  |  61 
>  5 files changed, 237 insertions(+)
>  create mode 100644 drivers/power/pmic/max8998.c
>
> --
> 2.10.2
>
> ___
> U-Boot mailing list
> U-Boot@lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>


This patchset seems to ready to submit.
Who will pick this up? power? DM? or samsang?


Thanks,
-- 
from. prom.
www.promsoft.net
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [linux-sunxi] Problems to Allwinner H3's eFUSE/SID

2016-12-21 Thread Bernhard Nortmann

Side note:

Am 19.12.2016 um 16:22 schrieb Icenowy Zheng:

[...]

According to some facts:
- The register based access to SID is weird: it needs ~5 register
   operations per word of SID.
[...]



My experiments seem to indicate that Allwinner's implementation might be
overly complicated. I have used an alternative approach and did not notice
any issues so far.

They start by reading SID_PRCTL and use bit mask operations to get in the
key index value (PG_INDEX), then use a second write to set OP_LOCK and
READ_START. As far as I can tell, it's sufficient to construct (and write)
a single value from these three components. Allwinner finally clears out
the used bits by another mask operation, simply writing zero seems to get
that job done equally well.

My implementation can be seen in
https://github.com/n1tehawk/sunxi-tools/blob/20161220_sid-fix/uart0-helloworld-sdboot.c#L242-L256
or as assembler code in
https://github.com/n1tehawk/sunxi-tools/blob/20161220_sid-fix/sid_read_root.S#L43-L65

A corresponding PR is at 
https://github.com/linux-sunxi/sunxi-tools/pull/91 ,

or you may clone my branch directly
("git clone https://github.com/n1tehawk/sunxi-tools.git -b 
20161220_sid-fix").


Testers would be very welcome.

Regards, B. Nortmann

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 1/2] sunxi: add proper device tree for Lichee Pi One

2016-12-21 Thread Icenowy Zheng
Signed-off-by: Icenowy Zheng 
---
 arch/arm/dts/Makefile   |   1 +
 arch/arm/dts/sun5i-a13-licheepi-one.dts | 224 
 2 files changed, 225 insertions(+)
 create mode 100644 arch/arm/dts/sun5i-a13-licheepi-one.dts

diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
index 3ee608b5b4..c1ecf72c3d 100644
--- a/arch/arm/dts/Makefile
+++ b/arch/arm/dts/Makefile
@@ -203,6 +203,7 @@ dtb-$(CONFIG_MACH_SUN5I) += \
sun5i-a13-hsg-h702.dtb \
sun5i-a13-inet-86vs.dtb \
sun5i-a13-inet-98v-rev2.dtb \
+   sun5i-a13-licheepi-one.dtb \
sun5i-a13-olinuxino.dtb \
sun5i-a13-olinuxino-micro.dtb \
sun5i-a13-q8-tablet.dtb \
diff --git a/arch/arm/dts/sun5i-a13-licheepi-one.dts 
b/arch/arm/dts/sun5i-a13-licheepi-one.dts
new file mode 100644
index 00..54d0fec858
--- /dev/null
+++ b/arch/arm/dts/sun5i-a13-licheepi-one.dts
@@ -0,0 +1,224 @@
+/*
+ * Copyright 2016 Icenowy Zheng 
+ *
+ * Based on sun5i-a13-olinuxino.dts, which is
+ *   Copyright 2012 Maxime Ripard 
+ *   Copyright 2013 Hans de Goede 
+ *
+ * This file is dual-licensed: you can use it either under the terms
+ * of the GPL or the X11 license, at your option. Note that this dual
+ * licensing only applies to this file, and not this project as a
+ * whole.
+ *
+ *  a) This file is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of the
+ * License, or (at your option) any later version.
+ *
+ * This file is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * Or, alternatively,
+ *
+ *  b) Permission is hereby granted, free of charge, to any person
+ * obtaining a copy of this software and associated documentation
+ * files (the "Software"), to deal in the Software without
+ * restriction, including without limitation the rights to use,
+ * copy, modify, merge, publish, distribute, sublicense, and/or
+ * sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following
+ * conditions:
+ *
+ * The above copyright notice and this permission notice shall be
+ * included in all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+ * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
+ * OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+ * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
+ * HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
+ * WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
+ * OTHER DEALINGS IN THE SOFTWARE.
+ */
+
+/dts-v1/;
+#include "sun5i-a13.dtsi"
+#include "sunxi-common-regulators.dtsi"
+
+#include 
+#include 
+#include 
+
+/ {
+   model = "Lichee Pi One";
+   compatible = "licheepi,licheepi-one", "allwinner,sun5i-a13";
+
+   aliases {
+   serial0 = &uart1;
+   };
+
+   chosen {
+   stdout-path = "serial0:115200n8";
+   };
+
+   leds {
+   compatible = "gpio-leds";
+
+   red {
+   label ="licheepi:red:usr";
+   gpios = <&pio 2 5 GPIO_ACTIVE_LOW>;
+   };
+
+   green {
+   label ="licheepi:green:usr";
+   gpios = <&pio 2 19 GPIO_ACTIVE_LOW>;
+   default-state = "on";
+   };
+
+   blue {
+   label ="licheepi:blue:usr";
+   gpios = <&pio 2 4 GPIO_ACTIVE_LOW>;
+   };
+
+   };
+};
+
+&cpu0 {
+   cpu-supply = <®_dcdc2>;
+};
+
+&ehci0 {
+   status = "okay";
+};
+
+&i2c0 {
+   pinctrl-names = "default";
+   pinctrl-0 = <&i2c0_pins_a>;
+   status = "okay";
+
+   axp209: pmic@34 {
+   compatible = "x-powers,axp209";
+   reg = <0x34>;
+   interrupts = <0>;
+
+   interrupt-controller;
+   #interrupt-cells = <1>;
+   };
+};
+
+&i2c1 {
+   pinctrl-names = "default";
+   pinctrl-0 = <&i2c1_pins_a>;
+   status = "disabled";
+};
+
+&i2c2 {
+   pinctrl-names = "default";
+   pinctrl-0 = <&i2c2_pins_a>;
+   status = "disabled";
+};
+
+&lradc {
+   vref-supply = <®_ldo2>;
+   status = "okay";
+
+   button@984 {
+   label = "Home";
+   linux,code = ;
+   channel = <0>;
+   voltage = <984126>;
+   };
+};
+
+&mmc0 {
+   pinctrl-names = "default";
+   pinctrl-0 = <&mmc0_

[U-Boot] [PATCH] imx6ul: geam6ul: Enable I2C support

2016-12-21 Thread Jagan Teki
From: Jagan Teki 

Enable I2C support for Engicam GEAM6UL NAND module.

Cc: Stefano Babic 
Cc: Matteo Lisi 
Cc: Michael Trimarchi 
Signed-off-by: Jagan Teki 
---
 configs/imx6ul_geam_nand_defconfig | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/configs/imx6ul_geam_nand_defconfig 
b/configs/imx6ul_geam_nand_defconfig
index 71ef06a..66274e5 100644
--- a/configs/imx6ul_geam_nand_defconfig
+++ b/configs/imx6ul_geam_nand_defconfig
@@ -26,6 +26,7 @@ CONFIG_CMD_PING=y
 CONFIG_CMD_NAND=y
 CONFIG_CMD_UBI=y
 CONFIG_CMD_CACHE=y
+CONFIG_CMD_I2C=y
 CONFIG_CMD_EXT4=y
 CONFIG_CMD_EXT4_WRITE=y
 CONFIG_CMD_FAT=y
@@ -37,6 +38,7 @@ CONFIG_NAND_MXS=y
 CONFIG_IMX_THERMAL=y
 CONFIG_PINCTRL=y
 CONFIG_PINCTRL_IMX6=y
+CONFIG_SYS_I2C_MXC=y
 CONFIG_SPL_LIBCOMMON_SUPPORT=y
 CONFIG_SPL_LIBGENERIC_SUPPORT=y
 CONFIG_SPL_SERIAL_SUPPORT=y
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 0/3] dm: goni: support the DM PMIC

2016-12-21 Thread Jaehoon Chung
Hi,

On 12/21/2016 11:54 PM, Minkyu Kang wrote:
> Hi,
> 
> On 15 December 2016 at 18:21, Jaehoon Chung  wrote:
> 
>> This patchest is for supporting pmic driver-model on Goni board.
>> (Goni board is S5PC110.)
>>
>> For using that, add the new file as max8998.c for only driver-model.
>> pmic_max8998.c is remained for legacy.
>> In future, it might be removed. Instead, max8998 should be used.
>>
>> *dm tree
>>  serial  [ + ]|-- serial@e2900800
>>  i2c [   ]`-- i2c-pmic
>>  pmic[   ]`-- pmic@66
>>
>> *dm uclass
>> uclass 20: i2c
>> - * i2c-pmic @ 47f6c6d0, seq 3, (req 3)
>>
>> uclass 22: i2c_generic
>> uclass 42: pmic
>> - * pmic@66 @ 47f6c748, seq 0, (req -1)
>>
>> * After using pmic command.
>> Goni # pmic list
>> | Name| Parent name | Parent uclass @
>> seq
>> | pmic@66 | i2c-pmic| i2c @ 3
>>
>> Node, it needs to implement the regulator driver for using regulator
>> framework.
>>
>> Jaehoon Chung (3):
>>   power: pmic: add the max8998 controller for DM
>>   arm: dts: s5pc1xx-goni: add the pmic node for using DM
>>   configs: enable the DM_PMIC and DM_I2C_GPIO for max8998 pmic
>>
>>  arch/arm/dts/s5pc1xx-goni.dts | 165 ++
>> 
>>  configs/s5p_goni_defconfig|   3 +
>>  drivers/power/pmic/Kconfig|   7 ++
>>  drivers/power/pmic/Makefile   |   1 +
>>  drivers/power/pmic/max8998.c  |  61 
>>  5 files changed, 237 insertions(+)
>>  create mode 100644 drivers/power/pmic/max8998.c
>>
>> --
>> 2.10.2
>>
>> ___
>> U-Boot mailing list
>> U-Boot@lists.denx.de
>> http://lists.denx.de/mailman/listinfo/u-boot
>>
> 
> 
> This patchset seems to ready to submit.
> Who will pick this up? power? DM? or samsang?

Delegated to you (Minkyu), Could you apply on u-boot-samsung?
There is no Power maintainer, now. 
I sent the patch for updating Power maintainer, but i didn't reply anything.

Best Regards,
Jaehoon Chung

> 
> 
> Thanks,
> 

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [GIT PULL] Please pull u-boot-mmc master

2016-12-21 Thread Jaehoon Chung
Dear Tom,

Could you pull these patches on your master branch?

The following changes since commit 0d92f2141ac5ef5c80d13e9501890f914525d43c:

  arm64: mvebu: Fix A8K memory mapping and add documentation (2016-12-21 
09:52:35 +0100)

are available in the git repository at:

  http://git.denx.de/u-boot-mmc.git master

for you to fetch changes up to 9c4132b5267b8d58712077c1c231717c76c4d52f:

  mmc: Extend dependencies for zynq sdhci (2016-12-22 07:08:52 +0900)


Jaehoon Chung (1):
  mmc: spear: remove the entire spear_sdhci.c file

Michal Simek (1):
  mmc: Extend dependencies for zynq sdhci

 drivers/mmc/Kconfig   |  2 +-
 drivers/mmc/Makefile  |  1 -
 drivers/mmc/spear_sdhci.c | 27 ---
 3 files changed, 1 insertion(+), 29 deletions(-)
 delete mode 100644 drivers/mmc/spear_sdhci.c

Best Regards,
Jaehoon Chung
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 1/3] usb: ehci-mx6: implement ofdata_to_platdata

2016-12-21 Thread Peng Fan


> -Original Message-
> From: Marek Vasut [mailto:ma...@denx.de]
> Sent: Wednesday, December 21, 2016 10:08 PM
> To: Peng Fan ; sba...@denx.de
> Cc: u-boot@lists.denx.de; van.free...@gmail.com; Simon Glass
> 
> Subject: Re: [PATCH 1/3] usb: ehci-mx6: implement ofdata_to_platdata
> 
> On 12/21/2016 09:14 AM, Peng Fan wrote:
> > Implement ofdata_to_platdata to set the type to host or device.
> >  - Check "dr-mode" property.
> >  - If there is no "dr-mode", check phy_ctrl for i.MX6
> >and phy_status for i.MX7
> >
> > Signed-off-by: Peng Fan 
> > Cc: Marek Vasut 
> > Cc: Simon Glass 
> > Cc: Stefano Babic 
> > ---
> >  drivers/usb/host/ehci-mx6.c | 66
> > +
> >  1 file changed, 66 insertions(+)
> >
> > diff --git a/drivers/usb/host/ehci-mx6.c b/drivers/usb/host/ehci-mx6.c
> > index 48889c1..91413c9 100644
> > --- a/drivers/usb/host/ehci-mx6.c
> > +++ b/drivers/usb/host/ehci-mx6.c
> > @@ -15,10 +15,13 @@
> >  #include 
> >  #include 
> >  #include 
> > +#include 
> >  #include 
> >
> >  #include "ehci.h"
> >
> > +DECLARE_GLOBAL_DATA_PTR;
> > +
> >  #define USB_OTGREGS_OFFSET 0x000
> >  #define USB_H1REGS_OFFSET  0x200
> >  #define USB_H2REGS_OFFSET  0x400
> > @@ -48,6 +51,7 @@
> >  #define ANADIG_USB2_PLL_480_CTRL_EN_USB_CLKS   0x0040
> >
> >  #define USBNC_OFFSET   0x200
> > +#define USBNC_PHY_STATUS_OFFSET0x23C
> >  #define USBNC_PHYSTATUS_ID_DIG (1 << 4) /* otg_id status */
> >  #define USBNC_PHYCFG2_ACAENB   (1 << 4) /* otg_id detection enable */
> >  #define UCTRL_PWR_POL  (1 << 9) /* OTG Polarity of Power Pin */
> > @@ -417,6 +421,67 @@ static const struct ehci_ops mx6_ehci_ops = {
> > .init_after_reset = mx6_init_after_reset  };
> >
> > +static int ehci_usb_ofdata_to_platdata(struct udevice *dev) {
> > +   struct usb_platdata *plat = dev_get_platdata(dev);
> > +   void *__iomem addr = (void *__iomem)dev_get_addr(dev);
> > +   void *__iomem phy_ctrl, *__iomem phy_status;
> > +   const void *blob = gd->fdt_blob;
> > +   int offset = dev->of_offset, phy_off;
> > +   const char *mode;
> > +   u32 val;
> > +
> > +   mode = fdt_getprop(blob, offset, "dr_mode", NULL);
> > +   if (mode) {
> > +   if (strcmp(mode, "peripheral") == 0)
> > +   plat->init_type = USB_INIT_DEVICE;
> > +   else if (strcmp(mode, "host") == 0)
> > +   plat->init_type = USB_INIT_HOST;
> > +   else if (strcmp(mode, "otg") == 0)
> > +   plat->init_type = USB_INIT_HOST;
> 
> Shouldn't this case check the phy status register ?

Yeah. Thanks for pointing this out. Will fix it in V2.

> 
> > +   else
> > +   return -EINVAL;
> > +   } else {
> 
> You can probably do return 0 in the if (mode) branch and then indent this
> whole else branch to the left.

Will fix this in V2.

Thanks,
Peng.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 3/3] imx: mx6sllevk: add usb support

2016-12-21 Thread Peng Fan


> -Original Message-
> From: Marek Vasut [mailto:ma...@denx.de]
> Sent: Wednesday, December 21, 2016 10:10 PM
> To: Peng Fan ; sba...@denx.de
> Cc: u-boot@lists.denx.de; van.free...@gmail.com
> Subject: Re: [PATCH 3/3] imx: mx6sllevk: add usb support
> 
> On 12/21/2016 09:14 AM, Peng Fan wrote:
> > Add usb support for mx6sllevk board.
> >
> > Signed-off-by: Peng Fan 
> > Cc: Stefano Babic 
> > ---
[..]

> > +
> > +#define USB_OTHERREGS_OFFSET   0x800
> > +#define UCTRL_PWR_POL  (1 << 9)
> > +
> > +int board_ehci_hcd_init(int port)
> > +{
> > +   u32 *usbnc_usb_ctrl;
> > +
> > +   if (port > 1)
> > +   return -EINVAL;
> > +
> > +   usbnc_usb_ctrl = (u32 *)(USB_BASE_ADDR + USB_OTHERREGS_OFFSET
> +
> > +port * 4);
> > +
> > +   /* Set Power polarity */
> > +   setbits_le32(usbnc_usb_ctrl, UCTRL_PWR_POL);
> > +   return 0;
> > +}
> 
> Is this function similar to what usb_oc_config() does ?

No, this bit is not for overcurrent. According to RM, this is OTG Power Polarity
This bit should be set according to power switch's enable polarity.
1 Power switch has an active-high enable input
0 Power switch has an active-low enable input

This is board specific.

Regards,
Peng.

> 
> --
> Best regards,
> Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 3/3] imx: mx6sllevk: add usb support

2016-12-21 Thread Peng Fan


> -Original Message-
> From: Jagan Teki [mailto:ja...@openedev.com]
> Sent: Wednesday, December 21, 2016 10:02 PM
> To: Peng Fan 
> Cc: Stefano Babic ; Marek Vasut ; u-
> b...@lists.denx.de
> Subject: Re: [U-Boot] [PATCH 3/3] imx: mx6sllevk: add usb support
> 
> On Wed, Dec 21, 2016 at 9:14 AM, Peng Fan  wrote:
> > Add usb support for mx6sllevk board.
> >
[.]

> > +#define UCTRL_PWR_POL  (1 << 9)
> > +
> > +int board_ehci_hcd_init(int port)
> 
> Can't we do this from driver itself, .probe or somewhere?

This is to handle OTG power polarity, which is board specific.
There is no dts property now, so I add this in board file just like other fsl 
boards.
When there is an dts property upstreamed in Linux side, we could use that and 
move this code into driver part.

> 
> > +{
> > +   u32 *usbnc_usb_ctrl;
> > +
> > +   if (port > 1)
> > +   return -EINVAL;
> > +
> > +   usbnc_usb_ctrl = (u32 *)(USB_BASE_ADDR + USB_OTHERREGS_OFFSET
> +
> > +port * 4);
> > +
> > +   /* Set Power polarity */
> > +   setbits_le32(usbnc_usb_ctrl, UCTRL_PWR_POL);
> > +   return 0;
> > +}
> > diff --git a/configs/mx6sllevk_defconfig b/configs/mx6sllevk_defconfig
> > index 8ae049e..267cdc6 100644
> > --- a/configs/mx6sllevk_defconfig
> > +++ b/configs/mx6sllevk_defconfig
> > @@ -10,6 +10,7 @@ CONFIG_CMD_BOOTZ=y
> >  CONFIG_CMD_MEMTEST=y
> >  CONFIG_CMD_MMC=y
> >  CONFIG_CMD_I2C=y
> > +CONFIG_CMD_USB=y
> >  CONFIG_CMD_GPIO=y
> >  CONFIG_CMD_DHCP=y
> >  CONFIG_CMD_PING=y
> > @@ -34,3 +35,7 @@ CONFIG_DM_REGULATOR=y
> > CONFIG_DM_REGULATOR_PFUZE100=y  CONFIG_DM_REGULATOR_FIXED=y
> > CONFIG_DM_REGULATOR_GPIO=y
> > +CONFIG_USB=y
> > +CONFIG_DM_USB=y
> > +CONFIG_USB_EHCI_HCD=y
> > +CONFIG_USB_STORAGE=y
> > diff --git a/configs/mx6sllevk_plugin_defconfig
> > b/configs/mx6sllevk_plugin_defconfig
> > index e6be979..63d5bbc 100644
> > --- a/configs/mx6sllevk_plugin_defconfig
> > +++ b/configs/mx6sllevk_plugin_defconfig
> > @@ -11,6 +11,7 @@ CONFIG_CMD_BOOTZ=y
> >  CONFIG_CMD_MEMTEST=y
> >  CONFIG_CMD_MMC=y
> >  CONFIG_CMD_I2C=y
> > +CONFIG_CMD_USB=y
> >  CONFIG_CMD_GPIO=y
> >  CONFIG_CMD_DHCP=y
> >  CONFIG_CMD_PING=y
> > @@ -35,3 +36,7 @@ CONFIG_DM_REGULATOR=y
> > CONFIG_DM_REGULATOR_PFUZE100=y  CONFIG_DM_REGULATOR_FIXED=y
> > CONFIG_DM_REGULATOR_GPIO=y
> > +CONFIG_USB=y
> > +CONFIG_DM_USB=y
> > +CONFIG_USB_EHCI_HCD=y
> > +CONFIG_USB_STORAGE=y
> > diff --git a/include/configs/mx6sllevk.h b/include/configs/mx6sllevk.h
> > index b9f25cf..3f665a3 100644
> > --- a/include/configs/mx6sllevk.h
> > +++ b/include/configs/mx6sllevk.h
> > @@ -149,4 +149,13 @@
> >
> >  #define CONFIG_IOMUX_LPSR
> >
> > +/* USB Configs */
> > +#ifdef CONFIG_CMD_USB
> > +#define CONFIG_USB_HOST_ETHER
> > +#define CONFIG_USB_ETHER_ASIX
> > +#define CONFIG_USB_ETHER_RTL8152
> > +#define CONFIG_MXC_USB_PORTSC  (PORT_PTS_UTMI |
> PORT_PTS_PTW)
> > +#define CONFIG_USB_MAX_CONTROLLER_COUNT2
> 
> I think this is even should managed from driver isn't it?

Thanks for pointing this out. This is not needed by DM USB, I'll remove it in 
V2.

Thanks,
Peng.

> 
> thanks!
> --
> Jagan Teki
> Free Software Engineer | www.openedev.com U-Boot, Linux | Upstream
> Maintainer Hyderabad, India.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Please pull u-boot-mpc85xx master

2016-12-21 Thread Tom Rini
On Tue, Dec 20, 2016 at 05:15:57PM +, york sun wrote:

> Tom,
> 
> The following changes since commit 0b4bc1b3ab1850fccbade3e6103f2036f6bdb364:
> 
>Merge branch 'master' of git://git.denx.de/u-boot-spi (2016-12-16 
> 18:32:43 -0500)
> 
> are available in the git repository at:
> 
>git://git.denx.de/u-boot-mpc85xx.git
> 
> for you to fetch changes up to 01b25d42c157e9132f2bb7366e63d28deccb554e:
> 
>powerpc: Retain compatible property for L2 cache (2016-12-20 09:13:19 
> -0800)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [GIT PULL] Please pull u-boot-mmc master

2016-12-21 Thread Tom Rini
On Thu, Dec 22, 2016 at 07:12:35AM +0900, Jaehoon Chung wrote:

> Dear Tom,
> 
> Could you pull these patches on your master branch?
> 
> The following changes since commit 0d92f2141ac5ef5c80d13e9501890f914525d43c:
> 
>   arm64: mvebu: Fix A8K memory mapping and add documentation (2016-12-21 
> 09:52:35 +0100)
> 
> are available in the git repository at:
> 
>   http://git.denx.de/u-boot-mmc.git master
> 
> for you to fetch changes up to 9c4132b5267b8d58712077c1c231717c76c4d52f:
> 
>   mmc: Extend dependencies for zynq sdhci (2016-12-22 07:08:52 +0900)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v3 6/6] apalis-tk1: clean-up defconfig

2016-12-21 Thread Marcel Ziswiler
From: Marcel Ziswiler 

Use make savedefconfig to clean-up defconfig.

Signed-off-by: Marcel Ziswiler 

---

Changes in v3:
- Cleaned-up defconfig using savedefconfig.

Changes in v2: None

 configs/apalis-tk1_defconfig | 9 -
 1 file changed, 9 deletions(-)

diff --git a/configs/apalis-tk1_defconfig b/configs/apalis-tk1_defconfig
index 204137b..4204e7f 100644
--- a/configs/apalis-tk1_defconfig
+++ b/configs/apalis-tk1_defconfig
@@ -9,9 +9,7 @@ CONFIG_CONSOLE_MUX=y
 CONFIG_SYS_STDIO_DEREGISTER=y
 CONFIG_VERSION_VARIABLE=y
 # CONFIG_DISPLAY_BOARDINFO is not set
-CONFIG_HUSH_PARSER=y
 CONFIG_SYS_PROMPT="Apalis TK1 # "
-CONFIG_CMD_BOOTZ=y
 # CONFIG_CMD_IMI is not set
 # CONFIG_CMD_IMLS is not set
 # CONFIG_CMD_FLASH is not set
@@ -22,15 +20,8 @@ CONFIG_CMD_DFU=y
 CONFIG_CMD_USB_MASS_STORAGE=y
 # CONFIG_CMD_FPGA is not set
 CONFIG_CMD_GPIO=y
-CONFIG_CMD_DHCP=y
 # CONFIG_CMD_NFS is not set
-CONFIG_CMD_MII=y
-CONFIG_CMD_PING=y
-CONFIG_CMD_EXT2=y
-CONFIG_CMD_EXT4=y
 CONFIG_CMD_EXT4_WRITE=y
-CONFIG_CMD_FAT=y
-CONFIG_CMD_FS_GENERIC=y
 CONFIG_SPL_DM=y
 # CONFIG_BLK is not set
 CONFIG_DFU_MMC=y
-- 
2.9.3

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v3 2/6] mmc: tegra: introduce CONFIG_TEGRA_MMC to Kconfig

2016-12-21 Thread Marcel Ziswiler
From: Marcel Ziswiler 

Signed-off-by: Marcel Ziswiler 
Reviewed-by: Simon Glass 

---

Changes in v3: None
Changes in v2:
- Added Simon's reviewed-by.

 drivers/mmc/Kconfig | 6 ++
 1 file changed, 6 insertions(+)

diff --git a/drivers/mmc/Kconfig b/drivers/mmc/Kconfig
index 5e84a41..18f0e97 100644
--- a/drivers/mmc/Kconfig
+++ b/drivers/mmc/Kconfig
@@ -110,4 +110,10 @@ config SANDBOX_MMC
  improves build coverage for sandbox and makes it easier to detect
  MMC build errors with sandbox.
 
+config TEGRA_MMC
+   bool "Tegra SDHCI aka MMC support"
+   depends on DM_MMC && TEGRA
+   help
+ This selects support for SDHCI on Tegra SoCs.
+
 endmenu
-- 
2.9.3

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v3 5/6] apalis-tk1: disable external clock loopback on SDMMC3

2016-12-21 Thread Marcel Ziswiler
From: Marcel Ziswiler 

Actually make use of that shiny new CONFIG_TEGRA124_MMC_DISABLE_EXT_LOOPBACK.

Signed-off-by: Marcel Ziswiler 
---

Changes in v3: None
Changes in v2: None

 configs/apalis-tk1_defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/configs/apalis-tk1_defconfig b/configs/apalis-tk1_defconfig
index c1caef1..204137b 100644
--- a/configs/apalis-tk1_defconfig
+++ b/configs/apalis-tk1_defconfig
@@ -37,6 +37,7 @@ CONFIG_DFU_MMC=y
 CONFIG_DFU_RAM=y
 # CONFIG_DM_MMC_OPS is not set
 CONFIG_TEGRA_MMC=y
+CONFIG_TEGRA124_MMC_DISABLE_EXT_LOOPBACK=y
 CONFIG_E1000=y
 CONFIG_PCI=y
 CONFIG_DM_PCI=y
-- 
2.9.3

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v3 4/6] mmc: tegra: allow disabling external clock loopback

2016-12-21 Thread Marcel Ziswiler
From: Marcel Ziswiler 

Introduce CONFIG_TEGRA124_MMC_DISABLE_EXT_LOOPBACK to disable the external clock
loopback and use the internal one on SDMMC3 as per the SDMMC_VENDOR_MISC_CNTRL_0
register's SDMMC_SPARE1 bits being set to 0xfffd according to the TRM.

Signed-off-by: Marcel Ziswiler 
Reviewed-by: Simon Glass 

---

Changes in v3: None
Changes in v2:
- Added Simon's reviewed-by.
- Added TODO(email) as suggested by Simon so it is clear this is
  temporary and will be moved to device tree controlled approach once
  proper kernel integration made it mainline.

 arch/arm/include/asm/arch-tegra/tegra_mmc.h |  2 ++
 drivers/mmc/Kconfig | 11 +++
 drivers/mmc/tegra_mmc.c | 16 
 3 files changed, 29 insertions(+)

diff --git a/arch/arm/include/asm/arch-tegra/tegra_mmc.h 
b/arch/arm/include/asm/arch-tegra/tegra_mmc.h
index 64c848a..c40599a 100644
--- a/arch/arm/include/asm/arch-tegra/tegra_mmc.h
+++ b/arch/arm/include/asm/arch-tegra/tegra_mmc.h
@@ -108,6 +108,8 @@ struct tegra_mmc {
 #define TEGRA_MMC_CLKCON_SDCLK_FREQ_SEL_SHIFT  8
 #define TEGRA_MMC_CLKCON_SDCLK_FREQ_SEL_MASK   (0xff << 8)
 
+#define TEGRA_MMC_MISCON_ENABLE_EXT_LOOPBACK   (1 << 17)
+
 #define TEGRA_MMC_SWRST_SW_RESET_FOR_ALL   (1 << 0)
 #define TEGRA_MMC_SWRST_SW_RESET_FOR_CMD_LINE  (1 << 1)
 #define TEGRA_MMC_SWRST_SW_RESET_FOR_DAT_LINE  (1 << 2)
diff --git a/drivers/mmc/Kconfig b/drivers/mmc/Kconfig
index 18f0e97..6372876 100644
--- a/drivers/mmc/Kconfig
+++ b/drivers/mmc/Kconfig
@@ -116,4 +116,15 @@ config TEGRA_MMC
help
  This selects support for SDHCI on Tegra SoCs.
 
+config TEGRA124_MMC_DISABLE_EXT_LOOPBACK
+   bool "Disable external clock loopback"
+   depends on TEGRA_MMC && TEGRA124
+   help
+ Disable the external clock loopback and use the internal one on SDMMC3
+ as per the SDMMC_VENDOR_MISC_CNTRL_0 register's SDMMC_SPARE1 bits
+ being set to 0xfffd according to the TRM.
+
+ TODO(marcel.ziswi...@toradex.com): Move to device tree controlled
+ approach once proper kernel integration made it mainline.
+
 endmenu
diff --git a/drivers/mmc/tegra_mmc.c b/drivers/mmc/tegra_mmc.c
index 97b1154..839b15d 100644
--- a/drivers/mmc/tegra_mmc.c
+++ b/drivers/mmc/tegra_mmc.c
@@ -511,6 +511,22 @@ static int tegra_mmc_init(struct mmc *mmc)
 
tegra_mmc_reset(priv, mmc);
 
+#if defined(CONFIG_TEGRA124_MMC_DISABLE_EXT_LOOPBACK)
+   /*
+* Disable the external clock loopback and use the internal one on
+* SDMMC3 as per the SDMMC_VENDOR_MISC_CNTRL_0 register's SDMMC_SPARE1
+* bits being set to 0xfffd according to the TRM.
+*
+* TODO(marcel.ziswi...@toradex.com): Move to device tree controlled
+* approach once proper kernel integration made it mainline.
+*/
+   if (priv->reg == (void *)0x700b0400) {
+   mask = readl(&priv->reg->venmiscctl);
+   mask &= ~TEGRA_MMC_MISCON_ENABLE_EXT_LOOPBACK;
+   writel(mask, &priv->reg->venmiscctl);
+   }
+#endif
+
priv->version = readw(&priv->reg->hcver);
debug("host version = %x\n", priv->version);
 
-- 
2.9.3

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v3 3/6] mmc: tegra: move CONFIG_TEGRA_MMC from headers to defconfigs

2016-12-21 Thread Marcel Ziswiler
From: Marcel Ziswiler 

Basically running the following script:

tools/moveconfig.py CONFIG_TEGRA_MMC

Note that I left the SPL specific handling in
include/configs/tegra-common-post.h unchanged.

Signed-off-by: Marcel Ziswiler 
Reviewed-by: Simon Glass 

---

Changes in v3:
- My recent fresh install of Fedora 25 was missing an aarch64 toolchain
  which made moveconfig.py not properly handle the 6 newer Tegras. I now
  re-run moveconfig.py after installing an aarch64 toolchain and it all
  looks sane. Sorry about that, Tom.

Changes in v2:
- Added Simon's reviewed-by.

BTW: I will look at converting tegra to use CONFIG_BLK and
CONFIG_DM_MMC_OPS in a later separate step.

---
 configs/apalis-tk1_defconfig | 1 +
 configs/apalis_t30_defconfig | 1 +
 configs/beaver_defconfig | 1 +
 configs/cardhu_defconfig | 1 +
 configs/cei-tk1-som_defconfig| 1 +
 configs/colibri_t20_defconfig| 1 +
 configs/colibri_t30_defconfig| 1 +
 configs/dalmore_defconfig| 1 +
 configs/e2220-1170_defconfig | 9 +
 configs/harmony_defconfig| 1 +
 configs/jetson-tk1_defconfig | 1 +
 configs/medcom-wide_defconfig| 1 +
 configs/nyan-big_defconfig   | 1 +
 configs/p2371-_defconfig | 9 +
 configs/p2371-2180_defconfig | 9 +
 configs/p2571_defconfig  | 9 +
 configs/p2771--000_defconfig | 9 +
 configs/p2771--500_defconfig | 9 +
 configs/paz00_defconfig  | 1 +
 configs/plutux_defconfig | 1 +
 configs/seaboard_defconfig   | 1 +
 configs/tec-ng_defconfig | 1 +
 configs/tec_defconfig| 1 +
 configs/trimslice_defconfig  | 1 +
 configs/venice2_defconfig| 1 +
 configs/ventana_defconfig| 1 +
 configs/whistler_defconfig   | 1 +
 include/configs/apalis-tk1.h | 1 -
 include/configs/apalis_t30.h | 1 -
 include/configs/beaver.h | 1 -
 include/configs/cardhu.h | 1 -
 include/configs/cei-tk1-som.h| 1 -
 include/configs/colibri_t20.h| 1 -
 include/configs/colibri_t30.h| 1 -
 include/configs/dalmore.h| 1 -
 include/configs/e2220-1170.h | 1 -
 include/configs/harmony.h| 1 -
 include/configs/jetson-tk1.h | 1 -
 include/configs/medcom-wide.h| 1 -
 include/configs/nyan-big.h   | 1 -
 include/configs/p2371-.h | 1 -
 include/configs/p2371-2180.h | 1 -
 include/configs/p2571.h  | 1 -
 include/configs/p2771-.h | 1 -
 include/configs/paz00.h  | 1 -
 include/configs/plutux.h | 1 -
 include/configs/seaboard.h   | 1 -
 include/configs/tec-ng.h | 1 -
 include/configs/tec.h| 1 -
 include/configs/trimslice.h  | 1 -
 include/configs/venice2.h| 1 -
 include/configs/ventana.h| 1 -
 include/configs/whistler.h   | 1 -
 scripts/config_whitelist.txt | 1 -
 54 files changed, 27 insertions(+), 75 deletions(-)

diff --git a/configs/apalis-tk1_defconfig b/configs/apalis-tk1_defconfig
index ae95f6b..c1caef1 100644
--- a/configs/apalis-tk1_defconfig
+++ b/configs/apalis-tk1_defconfig
@@ -36,6 +36,7 @@ CONFIG_SPL_DM=y
 CONFIG_DFU_MMC=y
 CONFIG_DFU_RAM=y
 # CONFIG_DM_MMC_OPS is not set
+CONFIG_TEGRA_MMC=y
 CONFIG_E1000=y
 CONFIG_PCI=y
 CONFIG_DM_PCI=y
diff --git a/configs/apalis_t30_defconfig b/configs/apalis_t30_defconfig
index 691148c..0822e2e 100644
--- a/configs/apalis_t30_defconfig
+++ b/configs/apalis_t30_defconfig
@@ -25,6 +25,7 @@ CONFIG_SPL_DM=y
 CONFIG_DFU_MMC=y
 CONFIG_DFU_RAM=y
 # CONFIG_DM_MMC_OPS is not set
+CONFIG_TEGRA_MMC=y
 CONFIG_E1000=y
 CONFIG_PCI=y
 CONFIG_DM_PCI=y
diff --git a/configs/beaver_defconfig b/configs/beaver_defconfig
index 5cb73c0..69c9ccd 100644
--- a/configs/beaver_defconfig
+++ b/configs/beaver_defconfig
@@ -28,6 +28,7 @@ CONFIG_DFU_MMC=y
 CONFIG_DFU_RAM=y
 CONFIG_DFU_SF=y
 # CONFIG_DM_MMC_OPS is not set
+CONFIG_TEGRA_MMC=y
 CONFIG_SPI_FLASH=y
 CONFIG_SPI_FLASH_WINBOND=y
 CONFIG_RTL8169=y
diff --git a/configs/cardhu_defconfig b/configs/cardhu_defconfig
index 0214758..e8e8b20 100644
--- a/configs/cardhu_defconfig
+++ b/configs/cardhu_defconfig
@@ -23,6 +23,7 @@ CONFIG_CMD_EXT4_WRITE=y
 CONFIG_SPL_DM=y
 # CONFIG_BLK is not set
 # CONFIG_DM_MMC_OPS is not set
+CONFIG_TEGRA_MMC=y
 CONFIG_SPI_FLASH=y
 CONFIG_SPI_FLASH_WINBOND=y
 CONFIG_RTL8169=y
diff --git a/configs/cei-tk1-som_defconfig b/configs/cei-tk1-som_defconfig
index 909b367..57b87c9 100644
--- a/configs/cei-tk1-som_defconfig
+++ b/configs/cei-tk1-som_defconfig
@@ -28,6 +28,7 @@ CONFIG_DFU_MMC=y
 CONFIG_DFU_RAM=y
 CONFIG_DFU_SF=y
 # CONFIG_DM_MMC_OPS is not set
+CONFIG_TEGRA_MMC=y
 CONFIG_SPI_FLASH=y
 CONFIG_SPI_FLASH_WINBOND=y
 CONFIG_RTL8169=y
diff --git a/configs/colibri_t20_defconfig b/configs/colibri_t20_defconfig
index a543246..3a4dc31 100644
--- a/configs/colibri_t20_defconfig
+++ b/configs/colibri_t20_defconfig
@@ -28,6 +28,7 @@ CONFIG_SPL_DM=y
 CONFIG_DFU_MMC=y
 CONFIG_DFU_RAM=y
 # CONFIG_DM_MMC_OPS is not set
+CONFIG_TEGRA_MMC=y
 

Re: [U-Boot] [PATCH 3/3] imx: mx6sllevk: add usb support

2016-12-21 Thread Marek Vasut
On 12/22/2016 02:13 AM, Peng Fan wrote:
> 
> 
>> -Original Message-
>> From: Marek Vasut [mailto:ma...@denx.de]
>> Sent: Wednesday, December 21, 2016 10:10 PM
>> To: Peng Fan ; sba...@denx.de
>> Cc: u-boot@lists.denx.de; van.free...@gmail.com
>> Subject: Re: [PATCH 3/3] imx: mx6sllevk: add usb support
>>
>> On 12/21/2016 09:14 AM, Peng Fan wrote:
>>> Add usb support for mx6sllevk board.
>>>
>>> Signed-off-by: Peng Fan 
>>> Cc: Stefano Babic 
>>> ---
> [..]
> 
>>> +
>>> +#define USB_OTHERREGS_OFFSET   0x800
>>> +#define UCTRL_PWR_POL  (1 << 9)
>>> +
>>> +int board_ehci_hcd_init(int port)
>>> +{
>>> +   u32 *usbnc_usb_ctrl;
>>> +
>>> +   if (port > 1)
>>> +   return -EINVAL;
>>> +
>>> +   usbnc_usb_ctrl = (u32 *)(USB_BASE_ADDR + USB_OTHERREGS_OFFSET
>> +
>>> +port * 4);
>>> +
>>> +   /* Set Power polarity */
>>> +   setbits_le32(usbnc_usb_ctrl, UCTRL_PWR_POL);
>>> +   return 0;
>>> +}
>>
>> Is this function similar to what usb_oc_config() does ?
> 
> No, this bit is not for overcurrent. According to RM, this is OTG Power 
> Polarity
> This bit should be set according to power switch's enable polarity.
> 1 Power switch has an active-high enable input
> 0 Power switch has an active-low enable input
> 
> This is board specific.

Great, except it should also be part of the driver , the same way as
polarity is configured, yes ?


-- 
Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] ARM64: zynqmp: Fix i2c node's compatible string

2016-12-21 Thread Moritz Fischer
From: Moritz Fischer 

The Zynq Ultrascale MP uses version 1.4 of the Cadence IP core
which fixes some silicon bugs that needed software workarounds
in Version 1.0 that was used on Zynq systems.

Signed-off-by: Moritz Fischer 
Cc: Michal Simek 
Cc: Sören Brinkmann 
Cc: U-Boot List 
Cc: Rob Herring 
---

Hi Michal,

I think this is a slip up and should be r1p14 for
Ultrascale ZynqMP. drivers/i2c/i2c-cadence.c already uses this.
I Cc'd the u-boot list, because the same change would be required there.

Cheers,

Moritz

---
 arch/arm64/boot/dts/xilinx/zynqmp.dtsi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/xilinx/zynqmp.dtsi 
b/arch/arm64/boot/dts/xilinx/zynqmp.dtsi
index 68a90833..a5a5f91 100644
--- a/arch/arm64/boot/dts/xilinx/zynqmp.dtsi
+++ b/arch/arm64/boot/dts/xilinx/zynqmp.dtsi
@@ -175,7 +175,7 @@
};
 
i2c0: i2c@ff02 {
-   compatible = "cdns,i2c-r1p10";
+   compatible = "cdns,i2c-r1p14";
status = "disabled";
interrupt-parent = <&gic>;
interrupts = <0 17 4>;
@@ -185,7 +185,7 @@
};
 
i2c1: i2c@ff03 {
-   compatible = "cdns,i2c-r1p10";
+   compatible = "cdns,i2c-r1p14";
status = "disabled";
interrupt-parent = <&gic>;
interrupts = <0 18 4>;
-- 
2.4.11

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 3/3] imx: mx6sllevk: add usb support

2016-12-21 Thread Peng Fan

Hi Marek,

> -Original Message-
> From: Marek Vasut [mailto:ma...@denx.de]
> Sent: Thursday, December 22, 2016 12:40 PM
> To: Peng Fan ; sba...@denx.de
> Cc: u-boot@lists.denx.de; van.free...@gmail.com
> Subject: Re: [PATCH 3/3] imx: mx6sllevk: add usb support
> 
> On 12/22/2016 02:13 AM, Peng Fan wrote:
> >
> >
> >> -Original Message-
> >> From: Marek Vasut [mailto:ma...@denx.de]
> >> Sent: Wednesday, December 21, 2016 10:10 PM
> >> To: Peng Fan ; sba...@denx.de
> >> Cc: u-boot@lists.denx.de; van.free...@gmail.com
> >> Subject: Re: [PATCH 3/3] imx: mx6sllevk: add usb support
> >>
> >> On 12/21/2016 09:14 AM, Peng Fan wrote:
> >>> Add usb support for mx6sllevk board.
> >>>
> >>> Signed-off-by: Peng Fan 
> >>> Cc: Stefano Babic 
> >>> ---
> > [..]
> >
> >>> +
> >>> +#define USB_OTHERREGS_OFFSET   0x800
> >>> +#define UCTRL_PWR_POL  (1 << 9)
> >>> +
> >>> +int board_ehci_hcd_init(int port)
> >>> +{
> >>> + u32 *usbnc_usb_ctrl;
> >>> +
> >>> + if (port > 1)
> >>> + return -EINVAL;
> >>> +
> >>> + usbnc_usb_ctrl = (u32 *)(USB_BASE_ADDR + USB_OTHERREGS_OFFSET
> >> +
> >>> +  port * 4);
> >>> +
> >>> + /* Set Power polarity */
> >>> + setbits_le32(usbnc_usb_ctrl, UCTRL_PWR_POL);
> >>> + return 0;
> >>> +}
> >>
> >> Is this function similar to what usb_oc_config() does ?
> >
> > No, this bit is not for overcurrent. According to RM, this is OTG
> > Power Polarity This bit should be set according to power switch's enable
> polarity.
> > 1 Power switch has an active-high enable input
> > 0 Power switch has an active-low enable input
> >
> > This is board specific.
> 
> Great, except it should also be part of the driver , the same way as polarity 
> is
> configured, yes ?

I just found that there is code for mx7, 
http://lists.denx.de/pipermail/u-boot/2016-July/260321.html

Then do you agree I add such piece code in usb_power_config for mx6?
"
#ifdef CONFIG_MXC_USB_OTG_HACTIVE
setbits_le32(ctrl, UCTRL_PWR_POL); 
#else 
clrbits_le32(ctrl, UCTRL_PWR_POL);
#endif
"

Thanks,
Peng
> 
> 
> --
> Best regards,
> Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 3/3] imx: mx6sllevk: add usb support

2016-12-21 Thread Marek Vasut
On 12/22/2016 07:03 AM, Peng Fan wrote:
> 
> Hi Marek,
> 
>> -Original Message-
>> From: Marek Vasut [mailto:ma...@denx.de]
>> Sent: Thursday, December 22, 2016 12:40 PM
>> To: Peng Fan ; sba...@denx.de
>> Cc: u-boot@lists.denx.de; van.free...@gmail.com
>> Subject: Re: [PATCH 3/3] imx: mx6sllevk: add usb support
>>
>> On 12/22/2016 02:13 AM, Peng Fan wrote:
>>>
>>>
 -Original Message-
 From: Marek Vasut [mailto:ma...@denx.de]
 Sent: Wednesday, December 21, 2016 10:10 PM
 To: Peng Fan ; sba...@denx.de
 Cc: u-boot@lists.denx.de; van.free...@gmail.com
 Subject: Re: [PATCH 3/3] imx: mx6sllevk: add usb support

 On 12/21/2016 09:14 AM, Peng Fan wrote:
> Add usb support for mx6sllevk board.
>
> Signed-off-by: Peng Fan 
> Cc: Stefano Babic 
> ---
>>> [..]
>>>
> +
> +#define USB_OTHERREGS_OFFSET   0x800
> +#define UCTRL_PWR_POL  (1 << 9)
> +
> +int board_ehci_hcd_init(int port)
> +{
> + u32 *usbnc_usb_ctrl;
> +
> + if (port > 1)
> + return -EINVAL;
> +
> + usbnc_usb_ctrl = (u32 *)(USB_BASE_ADDR + USB_OTHERREGS_OFFSET
 +
> +  port * 4);
> +
> + /* Set Power polarity */
> + setbits_le32(usbnc_usb_ctrl, UCTRL_PWR_POL);
> + return 0;
> +}

 Is this function similar to what usb_oc_config() does ?
>>>
>>> No, this bit is not for overcurrent. According to RM, this is OTG
>>> Power Polarity This bit should be set according to power switch's enable
>> polarity.
>>> 1 Power switch has an active-high enable input
>>> 0 Power switch has an active-low enable input
>>>
>>> This is board specific.
>>
>> Great, except it should also be part of the driver , the same way as 
>> polarity is
>> configured, yes ?
> 
> I just found that there is code for mx7, 
> http://lists.denx.de/pipermail/u-boot/2016-July/260321.html
> 
> Then do you agree I add such piece code in usb_power_config for mx6?
> "
> #ifdef CONFIG_MXC_USB_OTG_HACTIVE
> setbits_le32(ctrl, UCTRL_PWR_POL); 
> #else 
> clrbits_le32(ctrl, UCTRL_PWR_POL);
> #endif
> "

Didn't you add DT support in a patch sent like ... yesterday ? :) Just
use DT property instead of ifdef.

-- 
Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] usb: storage: Show number of storage devices detected for DM_USB

2016-12-21 Thread Michal Simek
On 21.12.2016 15:38, Michal Simek wrote:
> On 21.12.2016 15:11, Marek Vasut wrote:
>> On 12/21/2016 09:35 AM, Michal Simek wrote:
>>> By enabling DM_USB information about number of storage devices
>>> was lost.
>>> Get this information back simply by printing number of devices detected
>>> via BLK uclass.
>>>
>>> For example:
>>> scanning bus 0 for devices... 7 USB Device(s) found
>>>scanning usb for storage devices... 3 Storage Device(s) found
>>>scanning usb for ethernet devices... 0 Ethernet Device(s) found
>>>
>>> Signed-off-by: Michal Simek 
>>> ---
>>>
>>>  cmd/usb.c| 2 +-
>>>  common/usb_storage.c | 8 
>>>  2 files changed, 5 insertions(+), 5 deletions(-)
>>>
>>> diff --git a/cmd/usb.c b/cmd/usb.c
>>> index 455127c844b9..4fa456e31834 100644
>>> --- a/cmd/usb.c
>>> +++ b/cmd/usb.c
>>> @@ -571,11 +571,11 @@ static void do_usb_start(void)
>>> return;
>>>  
>>> /* Driver model will probe the devices as they are found */
>>> -#ifndef CONFIG_DM_USB
>>>  # ifdef CONFIG_USB_STORAGE
>>> /* try to recognize storage devices immediately */
>>> usb_stor_curr_dev = usb_stor_scan(1);
>>>  # endif
>>> +#ifndef CONFIG_DM_USB
>>>  # ifdef CONFIG_USB_KEYBOARD
>>> drv_usb_kbd_init();
>>>  # endif
>>> diff --git a/common/usb_storage.c b/common/usb_storage.c
>>> index 0345aa22eff5..b524a15e2bf9 100644
>>> --- a/common/usb_storage.c
>>> +++ b/common/usb_storage.c
>>> @@ -303,7 +303,6 @@ void usb_stor_reset(void)
>>> usb_max_devs = 0;
>>>  }
>>>  
>>> -#ifndef CONFIG_DM_USB
>>>  
>>> /***
>>>   * scan the usb and reports device info
>>>   * to the user if mode = 1
>>> @@ -311,11 +310,12 @@ void usb_stor_reset(void)
>>>   */
>>>  int usb_stor_scan(int mode)
>>>  {
>>> -   unsigned char i;
>>> -
>>> if (mode == 1)
>>> printf("   scanning usb for storage devices... ");
>>>  
>>> +#ifndef CONFIG_DM_USB
>>> +   unsigned char i;
>>
>> Won't this complain about mixing variables and code ? I think it will.
>> You can use __maybe_unused if you want to avoid excess ifdeffery.
> 
> I didn't see this issue on my PC. But I will use travis to validate this
> https://travis-ci.org/michalsimek-test/u-boot/builds/185787335

Just a note. Travis is not reporting any issue.

Thanks,
Michal

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 3/3] imx: mx6sllevk: add usb support

2016-12-21 Thread Peng Fan


> -Original Message-
> From: Marek Vasut [mailto:ma...@denx.de]
> Sent: Thursday, December 22, 2016 2:12 PM
> To: Peng Fan ; sba...@denx.de
> Cc: u-boot@lists.denx.de; van.free...@gmail.com
> Subject: Re: [PATCH 3/3] imx: mx6sllevk: add usb support
> 
> On 12/22/2016 07:03 AM, Peng Fan wrote:
> >
> > Hi Marek,
> >
> >> -Original Message-
> >> From: Marek Vasut [mailto:ma...@denx.de]
> >> Sent: Thursday, December 22, 2016 12:40 PM
> >> To: Peng Fan ; sba...@denx.de
> >> Cc: u-boot@lists.denx.de; van.free...@gmail.com
> >> Subject: Re: [PATCH 3/3] imx: mx6sllevk: add usb support
> >>
> >> On 12/22/2016 02:13 AM, Peng Fan wrote:
> >>>
> >>>
>  -Original Message-
>  From: Marek Vasut [mailto:ma...@denx.de]
>  Sent: Wednesday, December 21, 2016 10:10 PM
>  To: Peng Fan ; sba...@denx.de
>  Cc: u-boot@lists.denx.de; van.free...@gmail.com
>  Subject: Re: [PATCH 3/3] imx: mx6sllevk: add usb support
> 
>  On 12/21/2016 09:14 AM, Peng Fan wrote:
> > Add usb support for mx6sllevk board.
> >
> > Signed-off-by: Peng Fan 
> > Cc: Stefano Babic 
> > ---
> >>> [..]
> >>>
> > +
> > +#define USB_OTHERREGS_OFFSET   0x800
> > +#define UCTRL_PWR_POL  (1 << 9)
> > +
> > +int board_ehci_hcd_init(int port) {
> > +   u32 *usbnc_usb_ctrl;
> > +
> > +   if (port > 1)
> > +   return -EINVAL;
> > +
> > +   usbnc_usb_ctrl = (u32 *)(USB_BASE_ADDR +
> USB_OTHERREGS_OFFSET
>  +
> > +port * 4);
> > +
> > +   /* Set Power polarity */
> > +   setbits_le32(usbnc_usb_ctrl, UCTRL_PWR_POL);
> > +   return 0;
> > +}
> 
>  Is this function similar to what usb_oc_config() does ?
> >>>
> >>> No, this bit is not for overcurrent. According to RM, this is OTG
> >>> Power Polarity This bit should be set according to power switch's
> >>> enable
> >> polarity.
> >>> 1 Power switch has an active-high enable input
> >>> 0 Power switch has an active-low enable input
> >>>
> >>> This is board specific.
> >>
> >> Great, except it should also be part of the driver , the same way as
> >> polarity is configured, yes ?
> >
> > I just found that there is code for mx7,
> > http://lists.denx.de/pipermail/u-boot/2016-July/260321.html
> >
> > Then do you agree I add such piece code in usb_power_config for mx6?
> > "
> > #ifdef CONFIG_MXC_USB_OTG_HACTIVE
> > setbits_le32(ctrl, UCTRL_PWR_POL); #else
> > clrbits_le32(ctrl, UCTRL_PWR_POL);
> > #endif
> > "
> 
> Didn't you add DT support in a patch sent like ... yesterday ? :) Just use DT
> property instead of ifdef.

There is no property for otg power polatiry in upstream Linux. I would not like 
to introduce one
in U-Boot. We can drop the ifdef when there is an property in upstream. What do 
you think?

Regards,
Peng.

> 
> --
> Best regards,
> Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 3/3] imx: mx6sllevk: add usb support

2016-12-21 Thread Marek Vasut
On 12/22/2016 07:38 AM, Peng Fan wrote:
> 
> 
>> -Original Message-
>> From: Marek Vasut [mailto:ma...@denx.de]
>> Sent: Thursday, December 22, 2016 2:12 PM
>> To: Peng Fan ; sba...@denx.de
>> Cc: u-boot@lists.denx.de; van.free...@gmail.com
>> Subject: Re: [PATCH 3/3] imx: mx6sllevk: add usb support
>>
>> On 12/22/2016 07:03 AM, Peng Fan wrote:
>>>
>>> Hi Marek,
>>>
 -Original Message-
 From: Marek Vasut [mailto:ma...@denx.de]
 Sent: Thursday, December 22, 2016 12:40 PM
 To: Peng Fan ; sba...@denx.de
 Cc: u-boot@lists.denx.de; van.free...@gmail.com
 Subject: Re: [PATCH 3/3] imx: mx6sllevk: add usb support

 On 12/22/2016 02:13 AM, Peng Fan wrote:
>
>
>> -Original Message-
>> From: Marek Vasut [mailto:ma...@denx.de]
>> Sent: Wednesday, December 21, 2016 10:10 PM
>> To: Peng Fan ; sba...@denx.de
>> Cc: u-boot@lists.denx.de; van.free...@gmail.com
>> Subject: Re: [PATCH 3/3] imx: mx6sllevk: add usb support
>>
>> On 12/21/2016 09:14 AM, Peng Fan wrote:
>>> Add usb support for mx6sllevk board.
>>>
>>> Signed-off-by: Peng Fan 
>>> Cc: Stefano Babic 
>>> ---
> [..]
>
>>> +
>>> +#define USB_OTHERREGS_OFFSET   0x800
>>> +#define UCTRL_PWR_POL  (1 << 9)
>>> +
>>> +int board_ehci_hcd_init(int port) {
>>> +   u32 *usbnc_usb_ctrl;
>>> +
>>> +   if (port > 1)
>>> +   return -EINVAL;
>>> +
>>> +   usbnc_usb_ctrl = (u32 *)(USB_BASE_ADDR +
>> USB_OTHERREGS_OFFSET
>> +
>>> +port * 4);
>>> +
>>> +   /* Set Power polarity */
>>> +   setbits_le32(usbnc_usb_ctrl, UCTRL_PWR_POL);
>>> +   return 0;
>>> +}
>>
>> Is this function similar to what usb_oc_config() does ?
>
> No, this bit is not for overcurrent. According to RM, this is OTG
> Power Polarity This bit should be set according to power switch's
> enable
 polarity.
> 1 Power switch has an active-high enable input
> 0 Power switch has an active-low enable input
>
> This is board specific.

 Great, except it should also be part of the driver , the same way as
 polarity is configured, yes ?
>>>
>>> I just found that there is code for mx7,
>>> http://lists.denx.de/pipermail/u-boot/2016-July/260321.html
>>>
>>> Then do you agree I add such piece code in usb_power_config for mx6?
>>> "
>>> #ifdef CONFIG_MXC_USB_OTG_HACTIVE
>>> setbits_le32(ctrl, UCTRL_PWR_POL); #else
>>> clrbits_le32(ctrl, UCTRL_PWR_POL);
>>> #endif
>>> "
>>
>> Didn't you add DT support in a patch sent like ... yesterday ? :) Just use DT
>> property instead of ifdef.
> 
> There is no property for otg power polatiry in upstream Linux. I would not 
> like to introduce one
> in U-Boot. We can drop the ifdef when there is an property in upstream. What 
> do you think?

So uh, how can the USB work in mainline Linux on that board ?

-- 
Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] ARM64: zynqmp: Fix i2c node's compatible string

2016-12-21 Thread Michal Simek
On 22.12.2016 06:49, Moritz Fischer wrote:
> From: Moritz Fischer 
> 
> The Zynq Ultrascale MP uses version 1.4 of the Cadence IP core
> which fixes some silicon bugs that needed software workarounds
> in Version 1.0 that was used on Zynq systems.
> 
> Signed-off-by: Moritz Fischer 
> Cc: Michal Simek 
> Cc: Sören Brinkmann 
> Cc: U-Boot List 
> Cc: Rob Herring 
> ---
> 
> Hi Michal,
> 
> I think this is a slip up and should be r1p14 for
> Ultrascale ZynqMP. drivers/i2c/i2c-cadence.c already uses this.
> I Cc'd the u-boot list, because the same change would be required there.
> 
> Cheers,
> 
> Moritz
> 
> ---
>  arch/arm64/boot/dts/xilinx/zynqmp.dtsi | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/xilinx/zynqmp.dtsi 
> b/arch/arm64/boot/dts/xilinx/zynqmp.dtsi
> index 68a90833..a5a5f91 100644
> --- a/arch/arm64/boot/dts/xilinx/zynqmp.dtsi
> +++ b/arch/arm64/boot/dts/xilinx/zynqmp.dtsi
> @@ -175,7 +175,7 @@
>   };
>  
>   i2c0: i2c@ff02 {
> - compatible = "cdns,i2c-r1p10";
> + compatible = "cdns,i2c-r1p14";

I was checking this internally and p10 is doing something what p14
doesn't need to do. That's why this should be

compatible = "cdns,i2c-r1p14", "cdns,i2c-r1p10";

The same of course for u-boot where also p14 should be added to the driver.

Thanks,
Michal
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot