[U-Boot] pull request: u-boot-uniphier/master

[Masahiro Yamada]

Hi Tom,

Please pull some more UniPhier updates for v2016.09-rc3.
Thanks!


The following changes since commit da968c7bfa4cc1203a4f9f61a97f55c85dfbb3b6:

  Merge branch 'master' of git://git.denx.de/u-boot-i2c (2016-08-26
07:42:06 -0400)

are available in the git repository at:


  git://git.denx.de/u-boot-uniphier master

for you to fetch changes up to 8d11f804130d27a8f5ed6954f3a8a7f62a10ac45:

  ARM: uniphier: enable CONFIG_CMD_CACHE (2016-08-28 13:12:18 +0900)


Masahiro Yamada (11):
  mmc: uniphier-sd: add static qualifiers to probe and remove callbacks
  mmc: uniphier-sd: migrate to CONFIG_DM_MMC_OPS
  mmc: uniphier-sd: move uniphier_sd_init() below
  mmc: uniphier-sd: return error code if unsupported width is given
  mmc: uniphier-sd: just return if already set to desired clock rate
  ARM: uniphier: support system reset functionality for PSCI
  ARM: uniphier: display revision of Micro Support Card 3.6.x kindly
  ARM: uniphier: increase CONFIG_SYS_MALLOC_F_LEN for sLD3
  ARM: dts: uniphier: add u-boot, dm-pre-reloc to use eMMC boot on sLD3
  ARM: uniphier: change UNIPHIER_SERIAL to default y option
  ARM: uniphier: enable CONFIG_CMD_CACHE

 arch/arm/dts/uniphier-ph1-sld3-ref.dts  |   8 ++
 arch/arm/dts/uniphier-ph1-sld3.dtsi |   1 +
 arch/arm/mach-uniphier/arm32/psci.c |   5 ++
 arch/arm/mach-uniphier/micro-support-card.c |  11 ++-
 arch/arm/mach-uniphier/reset.c  |  14 +++-
 configs/uniphier_ld11_defconfig |   2 +-
 configs/uniphier_ld20_defconfig |   2 +-
 configs/uniphier_ld4_sld8_defconfig |   2 +-
 configs/uniphier_pro4_defconfig |   2 +-
 configs/uniphier_pxs2_ld6b_defconfig|   2 +-
 configs/uniphier_sld3_defconfig |   3 +-
 drivers/mmc/Kconfig |   1 +
 drivers/mmc/uniphier-sd.c   | 148
+
 drivers/serial/Kconfig  |   1 +
 include/configs/uniphier.h  |   1 +
 15 files changed, 123 insertions(+), 80 deletions(-)

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

Re: [U-Boot] [PATCH] ARM: uniphier: support system reset functionality for PSCI

[Masahiro Yamada]

2016-08-25 21:03 GMT+09:00 Masahiro Yamada :
> This supports the system reset via PSCI for ARMv7 SoCs.
>
> Because the system reset is not supported on PSCI 0.1, let's define
> CONFIG_ARMV7_PSCI_1_0. (it is supported since PSCI 0.2, but there
> is no CONFIG to enable it in U-Boot for now.)
>
> Signed-off-by: Masahiro Yamada 
> ---
>
>  arch/arm/mach-uniphier/arm32/psci.c |  5 +
>  arch/arm/mach-uniphier/reset.c  | 10 +-
>  include/configs/uniphier.h  |  1 +
>  3 files changed, 15 insertions(+), 1 deletion(-)
>


Applied to u-boot-uniphier/master.


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

Re: [U-Boot] [PATCH 0/3] ARM: uniphier: some more fixes for v2016.09

[Masahiro Yamada]

2016-08-25 17:02 GMT+09:00 Masahiro Yamada :
>
>
>
> Masahiro Yamada (3):
>   ARM: uniphier: display revision of Micro Support Card 3.6.x kindly
>   ARM: uniphier: increase CONFIG_SYS_MALLOC_F_LEN for sLD3
>   ARM: dts: uniphier: add u-boot,dm-pre-reloc to use eMMC boot on sLD3

Series, applied to u-boot-uniphier/master.



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

Re: [U-Boot] [PATCH] ARM: uniphier: change UNIPHIER_SERIAL to default y option

[Masahiro Yamada]

2016-08-25 19:00 GMT+09:00 Masahiro Yamada :
> This is very likely to be necessary for normal use cases.
> Set its default to 'y' for shorter defconfig files.
>
> Signed-off-by: Masahiro Yamada 


Applied to u-boot-uniphier/master.

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

Re: [U-Boot] [PATCH 0/5] mmc: uniphier: migrate to CONFIG_DM_MMC_OPS and CONFIG_BLK

[Masahiro Yamada]

2016-08-25 14:52 GMT+09:00 Masahiro Yamada :
>
>
>
> Masahiro Yamada (5):
>   mmc: uniphier-sd: migrate to CONFIG_DM_MMC_OPS
>   mmc: uniphier-sd: move uniphier_sd_init() below
>   mmc: uniphier-sd: return error code if unsupported width is given
>   mmc: uniphier-sd: just return if already set to desired clock rate
>   mmc: uniphier-sd: migrate to CONFIG_BLK
>


Series except 05, Applied to u-boot-uniphier/master.
05 has been deferred until USB is converted to CONFIG_BLK.

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

Re: [U-Boot] [PATCH] mmc: uniphier-sd: add static qualifiers to probe and remove callbacks

[Masahiro Yamada]

2016-08-12 19:19 GMT+09:00 Masahiro Yamada :
> They are both only referenced in this file.
>
> Signed-off-by: Masahiro Yamada 
> ---


Applied to u-boot-uniphier/master.



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

[U-Boot] [RFC] mmc: fsl_esdhc: fix misaligned cache operation warning

[Clemens Gruber]

When using gzwrite to eMMC on an i.MX6Q board, the following warning
occurs repeatedly:
CACHE: Misaligned operation at range [4fd63318, 4fe63318]

I tried to cache-line align the start and end parameter for
flush_dcache_range in esdhc_setup_data.
(Same approach as in https://patchwork.ozlabs.org/patch/656470/)

After that, the "Misaligned operation" error message disappeared and
everything still works for me.

But is this safe and should I send the patch again with [PATCH] ?

Signed-off-by: Clemens Gruber 
---
 drivers/mmc/fsl_esdhc.c | 9 ++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/drivers/mmc/fsl_esdhc.c b/drivers/mmc/fsl_esdhc.c
index 103b32e..f40b424 100644
--- a/drivers/mmc/fsl_esdhc.c
+++ b/drivers/mmc/fsl_esdhc.c
@@ -222,6 +222,7 @@ static int esdhc_setup_data(struct mmc *mmc, struct 
mmc_data *data)
dma_addr_t addr;
 #endif
uint wml_value;
+   ulong start, end;
 
wml_value = data->blocksize/4;
 
@@ -243,9 +244,11 @@ static int esdhc_setup_data(struct mmc *mmc, struct 
mmc_data *data)
 #endif
} else {
 #ifndef CONFIG_SYS_FSL_ESDHC_USE_PIO
-   flush_dcache_range((ulong)data->src,
-  (ulong)data->src+data->blocks
-*data->blocksize);
+   start = (ulong)data->src;
+   start &= ~(CONFIG_SYS_CACHELINE_SIZE - 1);
+   end = ALIGN((ulong)data->src + data->blocks * data->blocksize,
+   CONFIG_SYS_CACHELINE_SIZE);
+   flush_dcache_range(start, end);
 #endif
if (wml_value > WML_WR_WML_MAX)
wml_value = WML_WR_WML_MAX_VAL;
-- 
2.9.3

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

Re: [U-Boot] [PATCH v2 4/4] config: rk3399: add support for dwc3 gadget

[Steve Rae]

Hi Kever,

On Aug 23, 2016 20:48, "Kever Yang"  wrote:
>
> To support fastboot, we need to enable the controller first.
> rk3399 is using dwc3 as usb device controller, this patch enable
> the configs for dwc3 gadget.
>
> Signed-off-by: Kever Yang 
> Reviewed-by: Simon Glass 
> ---
>
> Changes in v2:
> - remove config for USB2PHY UTMI BITS
>
>  include/configs/rk3399_common.h | 20 
>  1 file changed, 20 insertions(+)
>
> diff --git a/include/configs/rk3399_common.h
b/include/configs/rk3399_common.h
> index 6875308..fcd1700 100644
> --- a/include/configs/rk3399_common.h
> +++ b/include/configs/rk3399_common.h
> @@ -27,6 +27,26 @@
>
>  #define CONFIG_SYS_BOOTM_LEN   (64 << 20)  /* 64M */
>
> +/*  usb device */
> +#define CONFIG_USB_DWC3
> +#define CONFIG_USB_DWC3_GADGET
> +#define CONFIG_USB_GADGET
> +#define CONFIG_USB_GADGET_DUALSPEED
> +#define CONFIG_USB_GADGET_VBUS_DRAW0
> +
> +/*  fastboot  */
> +#define CONFIG_CMD_FASTBOOT
> +#define CONFIG_USB_FUNCTION_FASTBOOT
> +#define CONFIG_FASTBOOT_FLASH
> +#define CONFIG_FASTBOOT_FLASH_MMC_DEV  1   /* eMMC */
> +#define CONFIG_FASTBOOT_BUF_ADDR   CONFIG_SYS_LOAD_ADDR
> +#define CONFIG_FASTBOOT_BUF_SIZE   0x0800
> +
> +#define CONFIG_USB_GADGET_DOWNLOAD
> +#define CONFIG_G_DNL_MANUFACTURER  "Rockchip"
> +#define CONFIG_G_DNL_VENDOR_NUM0x2207
> +#define CONFIG_G_DNL_PRODUCT_NUM   0x330a
> +
FYI - All of these new defines are now in Kconfig
Thanks, Steve

>  /* MMC/SD IP block */
>  #define CONFIG_MMC
>  #define CONFIG_GENERIC_MMC
> --
> 1.9.1
>
> ___
> U-Boot mailing list
> U-Boot@lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] Kconfig: update FASTBOOT_FLASH_MMC_DEV

[Steve Rae]

handle FASTBOOT_FLASH_MMC_DEV default properly

Signed-off-by: Steve Rae 
---
I was hoping that the FASTBOOT_FLASH_MMC_DEV Kconfig option could be
an integer (eg. 0, 1, or 2 etc.) or undefined (to signify that it
is not being used). However, it seems that (Kconfig experts please!)
this is not correct within Kconfig.
Therefore, I have implemented "-1" to signify that it is not used.
Is this the "best practice" for handling this scenario?

 cmd/fastboot/Kconfig|  4 +++-
 common/Makefile |  4 +++-
 drivers/usb/gadget/f_fastboot.c | 12 
 3 files changed, 14 insertions(+), 6 deletions(-)

diff --git a/cmd/fastboot/Kconfig b/cmd/fastboot/Kconfig
index a93d1c0..fdd5475 100644
--- a/cmd/fastboot/Kconfig
+++ b/cmd/fastboot/Kconfig
@@ -50,10 +50,12 @@ config FASTBOOT_FLASH
 
 config FASTBOOT_FLASH_MMC_DEV
int "Define FASTBOOT MMC FLASH default device"
+   default -1
help
  The fastboot "flash" command requires additional information
  regarding the non-volatile storage device. Define this to
- the eMMC device that fastboot should use to store the image.
+ the eMMC device that fastboot should use to store the image,
+ or define '-1' to disable storing to an eMMC device.
 
 endif # USB_FUNCTION_FASTBOOT
 
diff --git a/common/Makefile b/common/Makefile
index 21619b3..56e95b3 100644
--- a/common/Makefile
+++ b/common/Makefile
@@ -142,12 +142,14 @@ obj-$(CONFIG_IO_TRACE) += iotrace.o
 obj-y += memsize.o
 obj-y += stdio.o
 
-# This option is not just y/n - it can have a numeric value
 ifdef CONFIG_FASTBOOT_FLASH
 obj-y += image-sparse.o
+# This option is not just y/n - it is a numeric value
 ifdef CONFIG_FASTBOOT_FLASH_MMC_DEV
+ifneq ($(CONFIG_FASTBOOT_FLASH_MMC_DEV), -1)
 obj-y += fb_mmc.o
 endif
+endif
 ifdef CONFIG_FASTBOOT_FLASH_NAND_DEV
 obj-y += fb_nand.o
 endif
diff --git a/drivers/usb/gadget/f_fastboot.c b/drivers/usb/gadget/f_fastboot.c
index 2160b1c..5df6b8b 100644
--- a/drivers/usb/gadget/f_fastboot.c
+++ b/drivers/usb/gadget/f_fastboot.c
@@ -21,7 +21,8 @@
 #include 
 #include 
 #include 
-#ifdef CONFIG_FASTBOOT_FLASH_MMC_DEV
+#if defined(CONFIG_FASTBOOT_FLASH_MMC_DEV) && \
+  (CONFIG_FASTBOOT_FLASH_MMC_DEV != -1)
 #include 
 #endif
 #ifdef CONFIG_FASTBOOT_FLASH_NAND_DEV
@@ -594,7 +595,8 @@ static void cb_flash(struct usb_ep *ep, struct usb_request 
*req)
fb_response_str = response;
 
fastboot_fail("no flash device defined");
-#ifdef CONFIG_FASTBOOT_FLASH_MMC_DEV
+#if defined(CONFIG_FASTBOOT_FLASH_MMC_DEV) && \
+  (CONFIG_FASTBOOT_FLASH_MMC_DEV != -1)
fb_mmc_flash_write(cmd, (void *)CONFIG_FASTBOOT_BUF_ADDR,
   download_bytes);
 #endif
@@ -610,7 +612,8 @@ static void cb_flash(struct usb_ep *ep, struct usb_request 
*req)
 static void cb_oem(struct usb_ep *ep, struct usb_request *req)
 {
char *cmd = req->buf;
-#ifdef CONFIG_FASTBOOT_FLASH_MMC_DEV
+#if defined(CONFIG_FASTBOOT_FLASH_MMC_DEV) && \
+  (CONFIG_FASTBOOT_FLASH_MMC_DEV != -1)
if (strncmp("format", cmd + 4, 6) == 0) {
char cmdbuf[32];
 sprintf(cmdbuf, "gpt write mmc %x $partitions",
@@ -646,7 +649,8 @@ static void cb_erase(struct usb_ep *ep, struct usb_request 
*req)
fb_response_str = response;
 
fastboot_fail("no flash device defined");
-#ifdef CONFIG_FASTBOOT_FLASH_MMC_DEV
+#if defined(CONFIG_FASTBOOT_FLASH_MMC_DEV) && \
+  (CONFIG_FASTBOOT_FLASH_MMC_DEV != -1)
fb_mmc_erase(cmd);
 #endif
 #ifdef CONFIG_FASTBOOT_FLASH_NAND_DEV
-- 
1.8.5

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

[U-Boot] [PATCH v2 1/2] Disable SPL_MMC_SUPPORT if ROCKCHIP_SPL_BACK_TO_BROM is enabled.

[Sandy Patterson]

Default SPL_MMC_SUPPORT to false when ROCKCHIP_SPL_BACK_TO_BROM is enabled.

Signed-off-by: Sandy Patterson 
---

Changes in v2:
 - Rebase after "Kconfig: Move CONFIG_SPL_..._SUPPORT to Kconfig."
 - Remove all the refactoring in the configs.
 - Split enabling featuring in rock2 into separate commit.

 arch/arm/mach-rockchip/Kconfig | 3 +++
 configs/rock2_defconfig| 2 +-
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/arch/arm/mach-rockchip/Kconfig b/arch/arm/mach-rockchip/Kconfig
index 1aac3c8..8a5d62a 100644
--- a/arch/arm/mach-rockchip/Kconfig
+++ b/arch/arm/mach-rockchip/Kconfig
@@ -42,6 +42,9 @@ config ROCKCHIP_SPL_BACK_TO_BROM
   SPL will return to the boot rom, which will then load the U-Boot
   binary to keep going on.
 
+config SPL_MMC_SUPPORT
+   default y if !ROCKCHIP_SPL_BACK_TO_BROM
+
 source "arch/arm/mach-rockchip/rk3036/Kconfig"
 source "arch/arm/mach-rockchip/rk3288/Kconfig"
 source "arch/arm/mach-rockchip/rk3399/Kconfig"
diff --git a/configs/rock2_defconfig b/configs/rock2_defconfig
index b1294f0..e34d8b3 100644
--- a/configs/rock2_defconfig
+++ b/configs/rock2_defconfig
@@ -10,7 +10,7 @@ CONFIG_SPL_STACK_R_MALLOC_SIMPLE_LEN=0x2000
 CONFIG_SPL_DRIVERS_MISC_SUPPORT=y
 CONFIG_SPL_LIBCOMMON_SUPPORT=y
 CONFIG_SPL_LIBGENERIC_SUPPORT=y
-CONFIG_SPL_MMC_SUPPORT=y
+CONFIG_ROCKCHIP_SPL_BACK_TO_BROM=y
 CONFIG_SPL_PINCTRL_SUPPORT=y
 CONFIG_SPL_SERIAL_SUPPORT=y
 CONFIG_HUSH_PARSER=y
-- 
1.9.1

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

[U-Boot] [PATCH v2 0/2] With this change the SPL binary shrinks from 32k to 23k when

[Sandy Patterson]

ROCKCHIP_SPL_BACK_TO_BROM is enabled. This has been tested on the rock2
board. It should work with any rk3288 though since the bootrom is
loading the SPL already it should load the main U-Boot as well.

Rock2 has been tested with ROCKCHIP_SPL_BACK_TO_BROM.

Also, this moves around the images on the disk, so I had to move the
environment so it doesn't overwrite u-boot. The new env location works
with both boot methods.

Changes in v2:
 - Rebase after "Kconfig: Move CONFIG_SPL_..._SUPPORT to Kconfig."
 - Remove all the refactoring in the configs.
 - Split enabling featuring in rock2 into separate commit.

Sandy Patterson (2):
  Disable SPL_MMC_SUPPORT if ROCKCHIP_SPL_BACK_TO_BROM is enabled.
  Enable ROCKCHIP_SPL_BACK_TO_BROM for rock2 board

 arch/arm/mach-rockchip/Kconfig | 3 +++
 configs/rock2_defconfig| 2 +-
 include/configs/rock2.h| 8 
 3 files changed, 8 insertions(+), 5 deletions(-)

-- 
1.9.1

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

[U-Boot] [PATCH v2 2/2] Enable ROCKCHIP_SPL_BACK_TO_BROM for rock2 board

[Sandy Patterson]

Rock2 has been tested with back to brom feature. The tricky part is that
with this feature the default environment is inside u-boot, and it's
defined for every rk3288 board independetly. So I just changed it for
rock2 here.

Solve by moving environment after u-boot before 1M boundary

Signed-off-by: Sandy Patterson 
---

Changes in v2: None

 include/configs/rock2.h | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/include/configs/rock2.h b/include/configs/rock2.h
index ee924c3..946367f 100644
--- a/include/configs/rock2.h
+++ b/include/configs/rock2.h
@@ -16,11 +16,11 @@
 
 #define CONFIG_ENV_IS_IN_MMC
 #define CONFIG_SYS_MMC_ENV_DEV 0
-/* SPL @ 32k for ~36k
- * ENV @ 96k
- * u-boot @ 128K
+/* SPL @ 32k for 34k
+ * u-boot directly after @ 68k for 400k or so
+ * ENV @ 992k
  */
-#define CONFIG_ENV_OFFSET (96 * 1024)
+#define CONFIG_ENV_OFFSET ((1024-32) * 1024)
 
 #define CONFIG_SYS_WHITE_ON_BLACK
 #define CONFIG_CONSOLE_SCROLL_LINES10
-- 
1.9.1

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

Re: [U-Boot] [PATCH 4/4] RK3288 needs fdt and initrd below 256M now.

[Sandy Patterson]

On Sat, Aug 27, 2016 at 12:06 PM, Simon Glass  wrote:

> Hi Sandy,
>
> On 11 July 2016 at 11:38, Sandy Patterson 
> wrote:
> > I am not sure why this limit is changing. But my kernel
> > doesn't load when it's above 256. This was testing on the
> > rock2 board.
> >
> > Signed-off-by: Sandy Patterson 
> > ---
> >
> >  include/configs/rk3288_common.h | 6 +++---
> >  1 file changed, 3 insertions(+), 3 deletions(-)
>
> Is this patch not needed now?
>

I just retested with the latest from u-boot.git/master . This patch is
still required for booing on rock2. I tested with the same mainline kernel
version as before (4.7.0-rc6). I don't know why, and I haven't tried
searching back through the history to see if it used to work and what
changed, sorry.


>
> >
> > diff --git a/include/configs/rk3288_common.h b/include/configs/rk3288_
> common.h
> > index 9d50d83..b88e7e5 100644
> > --- a/include/configs/rk3288_common.h
> > +++ b/include/configs/rk3288_common.h
> > @@ -97,11 +97,11 @@
> >
> >  #include 
> >
> > -/* Linux fails to load the fdt if it's loaded above 512M on a Rock 2
> board, so
> > +/* Linux fails to load the fdt if it's loaded above 256M on a Rock 2
> board, so
> >   * limit the fdt reallocation to that */
> >  #define CONFIG_EXTRA_ENV_SETTINGS \
> > -   "fdt_high=0x1fff\0" \
> > -   "initrd_high=0x1fff\0" \
> > +   "fdt_high=0x0fff\0" \
> > +   "initrd_high=0x0fff\0" \
> > ENV_MEM_LAYOUT_SETTINGS \
> > ROCKCHIP_DEVICE_SETTINGS \
> > BOOTENV
> > --
> > 1.9.1
> >
>
> Regards,
> Simon
>
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] u-boot - raspberry pi

[dh]

Thanks for the response. We like what you do with u-boot, we believe it is a 
better solution.

We've reviewed the PI netboot project and have come to the conclusion that even 
with modifications it will not provide the flexibility of u-boot, due to the 
proprietary nature of the PI firmware.

With the current PI implementation we have to configure the hardware on each 
and every PI, and cannot ship them and the sd card to a customer and have them 
installed. There is the extra step of configuring the hardware.
After configuring the hardware a customer has the potential of having two 
identical devices which work differently, non-netboot and netboot, with no 
visible way to tell the difference.
Back to what it would take for u-boot to fill this need?
The u-boot portion of the kernel parms needs to be appended to the firmware 
portion of the kernel pams which are in the directory: 

/proc/device-tree/chosen/bootargs
 where device-tree is a link to
/sys/firmware/devicetree/base
it appears the parsing requirement may be limited, although I do not know how 
these files are written, by firmware or by the kernel.
How can we get your help?
Thanks

Duncan Hare

714 931 7952

  From: Stephen Warren 
 To: Simon Glass ; d...@synoia.com 
Cc: "u-boot@lists.denx.de" ; Stephen Warren 

 Sent: Friday, August 26, 2016 9:00 AM
 Subject: Re: [U-Boot] u-boot - raspberry pi
   
On 08/26/2016 08:39 AM, Simon Glass wrote:
> +Stephen
>
> On 25 August 2016 at 22:12,   wrote:
>>  We have u-boot working on a raspberry pi, but need to append our kernel
>>  parms to those built by the firmware.
>>
>> is there a version of u-boot for the pi 3, with this support, and some doc
>> (the variable name for the firmware built kernel parms), a github link would
>> be perfect.
>>
>> We have an order of 400 PIs for a hospital in S America, and want to supply
>> net boot, including kernels. Duncan Hare

The current port of U-Boot to the Pi is intended to replace the 
operations that the binary FW performs rather than add to them. As such, 
there's currently no easy way to do what you need with U-Boot.

Two potential options exist:

1) Update U-Boot so that it saves the DTB pointer the binary FW passes 
at boot, parses this DTB, and exposes various properties (its address, 
the command-line embodied within it) as environment variables. U-Boot 
scripts could then use those environment variables as they see fit (e.g. 
ignore them to be compatible with current U-Boot behaviour, or set 
$bootargs by appending the extracted cmdline to whatever you want to add).

2) The binary FW recently grew a network boot feature itself. Perhaps 
this will fulfil your network boot needs by itself, so you don't need to 
use U-Boot.

> https://www.raspberrypi.org/documentation/hardware/raspberrypi/bootmodes/net.md

I haven't tried this yet, but it sounds good.


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

Re: [U-Boot] [PATCH 3/5] rk_pwm: use clock framework API to get module clock

[Simon Glass]

On 12 August 2016 at 11:21, Simon Glass  wrote:
> Hi Kever,
>
> On 12 August 2016 at 03:57, Kever Yang  wrote:
>> This patch use clock API instead of hardcode for get pwm clock.
>>
>> Signed-off-by: Kever Yang 
>> ---
>>
>>  drivers/pwm/rk_pwm.c | 17 ++---
>>  1 file changed, 14 insertions(+), 3 deletions(-)
>
> Acked-by: Simon Glass 
>
> nit below
>
>>
>> diff --git a/drivers/pwm/rk_pwm.c b/drivers/pwm/rk_pwm.c
>> index 2d289a4..d69aab5 100644
>> --- a/drivers/pwm/rk_pwm.c
>> +++ b/drivers/pwm/rk_pwm.c
>> @@ -6,6 +6,7 @@
>>   */
>>
>>  #include 
>> +#include 
>>  #include 
>>  #include 
>>  #include 
>> @@ -13,9 +14,9 @@
>>  #include 
>>  #include 
>>  #include 
>> -#include 
>>  #include 
>>  #include 
>> +#include 
>>  #include 
>>
>>  DECLARE_GLOBAL_DATA_PTR;
>> @@ -23,6 +24,7 @@ DECLARE_GLOBAL_DATA_PTR;
>>  struct rk_pwm_priv {
>> struct rk3288_pwm *regs;
>> struct rk3288_grf *grf;
>> +   ulong freq;
>>  };
>>
>>  static int rk_pwm_set_config(struct udevice *dev, uint channel, uint 
>> period_ns,
>> @@ -38,8 +40,8 @@ static int rk_pwm_set_config(struct udevice *dev, uint 
>> channel, uint period_ns,
>> RK_PWM_DISABLE,
>> >ctrl);
>>
>> -   period = lldiv((uint64_t)(PD_BUS_PCLK_HZ / 1000) * period_ns, 
>> 100);
>> -   duty = lldiv((uint64_t)(PD_BUS_PCLK_HZ / 1000) * duty_ns, 100);
>> +   period = lldiv((uint64_t)(priv->freq / 1000) * period_ns, 100);
>> +   duty = lldiv((uint64_t)(priv->freq / 1000) * duty_ns, 100);
>>
>> writel(period, >period_hpr);
>> writel(duty, >duty_lpr);
>> @@ -76,9 +78,18 @@ static int rk_pwm_ofdata_to_platdata(struct udevice *dev)
>>  static int rk_pwm_probe(struct udevice *dev)
>>  {
>> struct rk_pwm_priv *priv = dev_get_priv(dev);
>> +   struct clk clk;
>> +   int ret = 0;
>>
>> rk_setreg(>grf->soc_con2, 1 << 0);
>>
>> +   ret = clk_get_by_index(dev, 0, );
>> +   if (ret < 0) {
>> +   printf("%s get clock fail!\n", __func__);
>
> debug() please.
>
>> +   return -EINVAL;
>> +   }
>> +   priv->freq = clk_get_rate();
>> +
>> return 0;
>>  }
>>
>> --
>> 1.9.1
>>
>>
>
> Regards,
> Simon

Fixed nit and:

Applied to u-boot-rockchip/next, thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 4/4] RK3288 needs fdt and initrd below 256M now.

[Simon Glass]

Hi Sandy,

On 11 July 2016 at 11:38, Sandy Patterson  wrote:
> I am not sure why this limit is changing. But my kernel
> doesn't load when it's above 256. This was testing on the
> rock2 board.
>
> Signed-off-by: Sandy Patterson 
> ---
>
>  include/configs/rk3288_common.h | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)

Is this patch not needed now?

>
> diff --git a/include/configs/rk3288_common.h b/include/configs/rk3288_common.h
> index 9d50d83..b88e7e5 100644
> --- a/include/configs/rk3288_common.h
> +++ b/include/configs/rk3288_common.h
> @@ -97,11 +97,11 @@
>
>  #include 
>
> -/* Linux fails to load the fdt if it's loaded above 512M on a Rock 2 board, 
> so
> +/* Linux fails to load the fdt if it's loaded above 256M on a Rock 2 board, 
> so
>   * limit the fdt reallocation to that */
>  #define CONFIG_EXTRA_ENV_SETTINGS \
> -   "fdt_high=0x1fff\0" \
> -   "initrd_high=0x1fff\0" \
> +   "fdt_high=0x0fff\0" \
> +   "initrd_high=0x0fff\0" \
> ENV_MEM_LAYOUT_SETTINGS \
> ROCKCHIP_DEVICE_SETTINGS \
> BOOTENV
> --
> 1.9.1
>

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

Re: [U-Boot] [RESEND PATCH v2 3/5] config: evb-rk3399: enable pinctrl driver

[Simon Glass]

On 16 August 2016 at 03:58, Kever Yang  wrote:
> This patch enable rk3399 pinctrl driver and gpio driver which is sub-node
> of pinctrl.
>
> Signed-off-by: Kever Yang 
> Acked-by: Simon Glass 
> ---
>
> Changes in v2: None
>
>  configs/evb-rk3399_defconfig | 2 ++
>  1 file changed, 2 insertions(+)

Applied to u-boot-rockchip/next, thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 5/5] rk3288: add arch_cpu_init for rk3288

[Simon Glass]

On 15 August 2016 at 22:50, Simon Glass  wrote:
> On 12 August 2016 at 03:58, Kever Yang  wrote:
>>
>> We do some SoC level one time setting initialization in
>> arch_cpu_init.
>>
>> Signed-off-by: Kever Yang 
>> ---
>>
>>  arch/arm/mach-rockchip/rk3288/Makefile |  1 +
>>  arch/arm/mach-rockchip/rk3288/rk3288.c | 19 +++
>>  2 files changed, 20 insertions(+)
>>  create mode 100644 arch/arm/mach-rockchip/rk3288/rk3288.c
>
>
> Acked-by: Simon Glass 

Applied to u-boot-rockchip/next, thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] rockchip: use dummy byte only enable OF_PLATDATA

[Simon Glass]

On 12 August 2016 at 11:20, Simon Glass  wrote:
> On 12 August 2016 at 01:43, Ziyuan Xu  wrote:
>> Add a condition to determine the rk3288_sdram_channel size.
>>
>> This patch fixes read sdram_channel property failed from DT on rk3288
>> boards, which not enable OF_PLATDATA.
>>
>> Signed-off-by: Ziyuan Xu 
>> ---
>>
>>  arch/arm/include/asm/arch-rockchip/sdram.h | 4 
>>  1 file changed, 4 insertions(+)
>
> Acked-by: Simon Glass 

Applied to u-boot-rockchip/next, thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 4/5] rk_pwm: remove grf setting code from driver

[Simon Glass]

On 12 August 2016 at 11:21, Simon Glass  wrote:
> On 12 August 2016 at 03:58, Kever Yang  wrote:
>> We consider the grf setting for pwm controller select as the system
>> operation instead of driver operation, move it to soc init, let's
>> remove it from pwm driver first.
>>
>> Signed-off-by: Kever Yang 
>> ---
>>
>>  drivers/pwm/rk_pwm.c | 11 ---
>>  1 file changed, 11 deletions(-)
>
> Acked-by: Simon Glass 

Applied to u-boot-rockchip/next, thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [RESEND PATCH v2 2/5] pinctrl: add driver for rk3399

[Simon Glass]

Hi,

On 16 August 2016 at 22:15, Simon Glass  wrote:
> On 16 August 2016 at 03:58, Kever Yang  wrote:
>>
>> This patch add pinctrl driver for rk3399.
>>
>> Signed-off-by: Kever Yang 
>> ---
>>
>> Changes in v2:
>> - move and reg value MACRO in C source, and use MASK/SHIFT
>>
>>  arch/arm/include/asm/arch-rockchip/grf_rk3399.h | 321 +
>>  drivers/pinctrl/Kconfig |   9 +
>>  drivers/pinctrl/rockchip/Makefile   |   1 +
>>  drivers/pinctrl/rockchip/pinctrl_rk3399.c   | 439 
>> 
>>  4 files changed, 770 insertions(+)
>>  create mode 100644 arch/arm/include/asm/arch-rockchip/grf_rk3399.h
>>  create mode 100644 drivers/pinctrl/rockchip/pinctrl_rk3399.c
>
> Acked-by: Simon Glass 

I'm applying these since there seem to be no further comments.

Applied to u-boot-rockchip/next, thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [RESEND PATCH v2 4/5] rk3399: enable the pwm2/3 pinctrl in board init

[Simon Glass]

On 16 August 2016 at 03:58, Kever Yang  wrote:
> There is no interrupt line for each PWM which used by pinctrl to get the
> periph_id, so it's not able to enable the default pinctrl setting by pinctrl
> framework, let's enable it at board_init().
>
> Signed-off-by: Kever Yang 
> Acked-by: Simon Glass 
> ---
>
> Changes in v2: None
>
>  board/rockchip/evb_rk3399/evb-rk3399.c | 31 ++-
>  1 file changed, 30 insertions(+), 1 deletion(-)

Applied to u-boot-rockchip/next, thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [RESEND PATCH v2 5/5] dts: rk3399: add pinctrl for sdmmc

[Simon Glass]

On 16 August 2016 at 03:58, Kever Yang  wrote:
> This patch add pinctrl for sdcard which may not be initialized before
> uboot.
>
> Signed-off-by: Kever Yang 
> Acked-by: Simon Glass 
> ---
>
> Changes in v2: None
>
>  arch/arm/dts/rk3399.dtsi | 37 +
>  1 file changed, 37 insertions(+)

Applied to u-boot-rockchip/next, thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [RFC PATCH] rockchip: rk3288: skip lowlevel_init process

[Simon Glass]

On 27 August 2016 at 07:53, Ziyuan Xu  wrote:
> lowlevel_init() is never needed for rk3288, so drop it.
>
> Signed-off-by: Ziyuan Xu 
>
> ---
>
>  arch/arm/mach-rockchip/board.c| 4 
>  arch/arm/mach-rockchip/rk3288-board-spl.c | 4 
>  include/configs/rk3288_common.h   | 1 +
>  3 files changed, 1 insertion(+), 8 deletions(-)

Acked-by: Simon Glass 

Tested on firefly-rk3288:
Tested-by: Simon Glass 
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [RESEND PATCH v2 1/5] rk3399: syscon: add support for pmugrf

[Simon Glass]

On 16 August 2016 at 03:58, Kever Yang  wrote:
> pmugrf is a module like grf which contain some of the iomux registers
> and other registers.
>
> Signed-off-by: Kever Yang 
> Acked-by: Simon Glass 
> ---
>
> Changes in v2: None
>
>  arch/arm/include/asm/arch-rockchip/clock.h| 1 +
>  arch/arm/mach-rockchip/rk3399/syscon_rk3399.c | 1 +
>  2 files changed, 2 insertions(+)

Applied to u-boot-rockchip/next, thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] README: add cmd directory description

[Simon Glass]

On 26 August 2016 at 05:54, Ziyuan Xu  wrote:
> All of the command files have moved to cmd directory, add description to
> Directory Hierarchy.
>
> Signed-off-by: Ziyuan Xu 
> ---
>
>  README | 1 +
>  1 file changed, 1 insertion(+)

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

Re: [U-Boot] [PATCH v2] sunxi: display: Use PWM to drive backlight where applicable

[Simon Glass]

Hi Hans,

On 26 August 2016 at 08:21, Hans de Goede  wrote:
> Hi,
>
>
> On 26-08-16 16:15, Peter Korsgaard wrote:
>>>
>>> "Hans" == Hans de Goede  writes:
>>
>>
>>  > When the backlight's pwm input is connected to a pwm output of the SoC,
>>  > actually use pwm to drive the backlight.
>>
>>  > The mean reason for doing this is to fix the backlight turning off
>>  > for aprox. 1 second while the kernel is booting. This is caused by
>>  > the kernel actually using pwm to drive the backlight, so that it
>>  > can dim the backlight. First the pwm driver loads and switches the
>>  > pinmux for the pin driving the backlight's pwm input to the pwm
>>  > controller. Then about 1s later the actual backlight driver loads
>>  > and tells the pwm driver to actually update the pwm settings, which
>>  > have a power-on-reset value of "off".
>>
>>  > An additional advantage is that this allows us to initatiate the
>>  > backlight at 80%, which is the kernel default, avoiding a brightness
>>  > change while the kernel loads.
>>
>>  > +++ b/drivers/video/sunxi_display.c
>>  > @@ -12,6 +12,7 @@
>>  >  #include 
>>  >  #include 
>>  >  #include 
>>  > +#include 
>>  >  #include 
>>  >  #include 
>>  >  #include 
>>  > @@ -34,7 +35,6 @@
>>  >  #define PWM_ON 1
>>  >  #define PWM_OFF 0
>>  >  #endif
>>  > -
>>
>> Unrelated white space change. Other than that:
>
>
> Oops, fixed locally.
>
>> Reviewed by: Peter Korsgaard 
>
>
> Thank you.

Can this move to driver model for the PWM, and for the display for that matter?

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

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

Re: [U-Boot] [PATCH] rockchip: rk3288-firefly: enable boot from eMMC

[Simon Glass]

Hi,

On 12 August 2016 at 11:20, Simon Glass  wrote:
> On 9 August 2016 at 19:56, Jacob Chen  wrote:
>> Add eMMC dt node and define fallback boot devices.
>>
>> Signed-off-by: Jacob Chen 
>> ---
>>
>>  arch/arm/dts/rk3288-firefly.dts   | 5 +
>>  board/firefly/firefly-rk3288/firefly-rk3288.c | 8 
>>  2 files changed, 13 insertions(+)
>
> Acked-by: Simon Glass 

Unfortunately I cannot apply this. It appears to actually force
booting from eMMC. So if the eMMC is blank, it hangs at 'Trying to
boot from MMC2'.

I think it should continue to boot from whatever media it started
with. So if it booted SPL from SD then it should load U-Boot from SD.

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

Re: [U-Boot] [PATCH 22/30] dtoc: Add methods for reading data from properties

[Simon Glass]

On 25 July 2016 at 18:59, Simon Glass  wrote:
> Provide easy helpers for reading integer, string and boolean values from
> device-tree properties.
>
> Signed-off-by: Simon Glass 
> ---
>
>  tools/dtoc/fdt_util.py | 25 +
>  1 file changed, 25 insertions(+)

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

Re: [U-Boot] [PATCH 19/30] dtoc: Support packing the device tree

[Simon Glass]

On 25 July 2016 at 18:59, Simon Glass  wrote:
> After any node/property deletion the device tree can be packed to remove
> spare space. Add a way to perform this operation.
>
> Note that for fdt_fallback, fdtput automatically packs the device tree after
> deletion, so no action is required here.
>
> Signed-off-by: Simon Glass 
> ---
>
>  lib/libfdt/libfdt.swig   |  1 +
>  tools/dtoc/fdt.py| 16 
>  tools/dtoc/fdt_normal.py | 11 +++
>  3 files changed, 28 insertions(+)

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

Re: [U-Boot] [PATCH] rockchip: Disable SPL_MMC_SUPPORT if ROCKCHIP_SPL_BACK_TO_BROM is enabled.

[Simon Glass]

Hi Sandy,

On 25 August 2016 at 06:05, Simon Glass  wrote:
> Hi Sandy,
>
> On 24 August 2016 at 15:00, Sandy Patterson  wrote:
>>
>> On Wed, Aug 24, 2016 at 9:27 AM, Sandy Patterson 
>> wrote:

[...]

>>>
>>> Hi Simon,
>>>
>>> Just wondering if this commit got lost, or maybe there is some advice you
>>> could give me on how to make this more helpful.
>>>
>>> Sandy
>>
>>
>> Ah, I see why you didn't grab this. I'll resubmit after I've had a chance to
>> digest the SPL Kconfig changeset.
>
> No it's not that. You patch comes before that series.
>
> But I would like to see review/ tags from others on Rockchip patches -
> just so that I know that people are on-board with the change.
>
> Will add my tag to encourage people!
>
> Acked-by: Simon Glass 

But on closer inspection, yes this will be a pain to update after the
SPL series. So please can you take a look and resend? It looks like
you should do something like this in arch/arm/mach-rockchip/Kconfig or
similar:

config SPL_MMC_SUPPORT
   default y if !ROCKCHIP_SPL_BACK_TO_BROM

>
> Regards,
> Simon

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

Re: [U-Boot] [PATCH 21/30] dtoc: Correct quotes in fdt_util

[Simon Glass]

On 25 July 2016 at 18:59, Simon Glass  wrote:
> The style is to use single quotes for strings where possible. Adjust this
> function.
>
> Signed-off-by: Simon Glass 
> ---
>
>  tools/dtoc/fdt_util.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

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

Re: [U-Boot] [PATCH 20/30] dtoc: Support finding the offset of a property

[Simon Glass]

On 25 July 2016 at 18:59, Simon Glass  wrote:
> Add a way to find the byte offset of a property within the device tree. This
> is only supported with the normal libfdt implementation since fdtget does
> not provide this information.
>
> Signed-off-by: Simon Glass 
> ---
>
>  lib/libfdt/libfdt.swig   |  3 +++
>  tools/dtoc/fdt.py| 11 +++
>  tools/dtoc/fdt_normal.py | 18 ++
>  3 files changed, 32 insertions(+)

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

Re: [U-Boot] [PATCH 18/30] dtoc: Support deleting device tree properties

[Simon Glass]

On 25 July 2016 at 18:59, Simon Glass  wrote:
> Add support for deleting a device tree property. With the fallback
> implementation this uses fdtput. With libfdt it uses the API call and
> updates the offsets afterwards.
>
> Signed-off-by: Simon Glass 
> ---
>
>  lib/libfdt/libfdt.swig | 12 
>  tools/dtoc/fdt.py  | 10 ++
>  tools/dtoc/fdt_fallback.py | 13 +
>  tools/dtoc/fdt_normal.py   | 20 
>  4 files changed, 55 insertions(+)

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

Re: [U-Boot] [PATCH 15/30] dtoc: Drop the convert_dash parameter to GetProps()

[Simon Glass]

On 25 July 2016 at 18:59, Simon Glass  wrote:
> This is not used anywhere in dtoc, so drop it.
>
> Signed-off-by: Simon Glass 
> ---
>
>  tools/dtoc/fdt_fallback.py | 5 +
>  1 file changed, 1 insertion(+), 4 deletions(-)

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

Re: [U-Boot] [PATCH 17/30] dtoc: Move to using bytearray

[Simon Glass]

On 25 July 2016 at 18:59, Simon Glass  wrote:
> Since we want to be able to change the in-memory device tree using libfdt,
> use a bytearray instead of a string. This makes interfacing from Python
> easier.
>
> Signed-off-by: Simon Glass 
> ---
>
>  lib/libfdt/libfdt.swig   | 8 
>  tools/dtoc/fdt_normal.py | 2 +-
>  2 files changed, 9 insertions(+), 1 deletion(-)

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

Re: [U-Boot] [PATCH 16/30] dtoc: Prepare for supporting changing of device trees

[Simon Glass]

On 25 July 2016 at 18:59, Simon Glass  wrote:
> For binman we need to support deleting properties in the device tree. This
> will change the offsets of nodes after the deletion. In preparation, add
> code to keep track of when the offsets are invalid, and regenerate them.
>
> Signed-off-by: Simon Glass 
> ---
>
>  tools/dtoc/fdt_normal.py | 46 ++
>  1 file changed, 42 insertions(+), 4 deletions(-)

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

Re: [U-Boot] [PATCH 13/30] patman: Add a library to handle logging and progress

[Simon Glass]

On 25 July 2016 at 18:59, Simon Glass  wrote:
> When tools want to display information of varying levels of importance, it
> helps to provide the user with control over the verbosity of these messages.
> Progress messages work best if they are displayed and then removed from the
> display when no-longer relevant.
>
> Add a new tout library (terminal out) to handle these tasks.
>
> Signed-off-by: Simon Glass 
> ---
>
>  tools/patman/tout.py | 166 
> +++
>  1 file changed, 166 insertions(+)
>  create mode 100644 tools/patman/tout.py

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

Re: [U-Boot] [PATCH 14/30] dtoc: Allow the device tree to be compiled from source

[Simon Glass]

On 25 July 2016 at 18:59, Simon Glass  wrote:
> If a source device tree is provide to the Fdt() constructors, compile it
> automatically. This will be used in tests, where we want to build a
> particular test .dts file and check that it works correctly in binman.
>
> Signed-off-by: Simon Glass 
> ---
>
>  tools/dtoc/fdt_fallback.py |  2 ++
>  tools/dtoc/fdt_normal.py   |  7 +--
>  tools/dtoc/fdt_util.py | 41 +
>  3 files changed, 48 insertions(+), 2 deletions(-)

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

Re: [U-Boot] [PATCH 09/30] dtoc: Move BytesToValue() and GetEmpty() into PropBase

[Simon Glass]

On 25 July 2016 at 18:59, Simon Glass  wrote:
> These functions are currently in a separate fdt_util file. Since they are
> only used from PropBase and subclasses, it makes sense for them to be in the
> PropBase class.
>
> Move these functions into fdt.py along with the list of types.
>
> Signed-off-by: Simon Glass 
> ---
>
>  tools/dtoc/dtoc.py | 17 ++--
>  tools/dtoc/fdt.py  | 66 
> ++
>  tools/dtoc/fdt_fallback.py |  4 +--
>  tools/dtoc/fdt_normal.py   |  6 ++---
>  tools/dtoc/fdt_util.py | 66 
> --
>  5 files changed, 80 insertions(+), 79 deletions(-)

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

Re: [U-Boot] [PATCH 10/30] dtoc: Move Widen() and GetPhandle() into the base class

[Simon Glass]

On 25 July 2016 at 18:59, Simon Glass  wrote:
> These functions are identical in both subclasses. Move them into the base
> class.
>
> Note: In fact there is a bug in one version, which was fixed by this patch:
>
> https://patchwork.ozlabs.org/patch/651697/
>
> Signed-off-by: Simon Glass 
> ---
>
>  tools/dtoc/fdt.py  | 41 +
>  tools/dtoc/fdt_fallback.py | 36 
>  tools/dtoc/fdt_normal.py   | 42 --
>  3 files changed, 41 insertions(+), 78 deletions(-)

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

Re: [U-Boot] [PATCH 11/30] dtoc: Move a few more common functions into fdt.py

[Simon Glass]

On 25 July 2016 at 18:59, Simon Glass  wrote:
> Some functions have the same code in the subclasses. Move these into the
> superclass to avoid duplication.
>
> Signed-off-by: Simon Glass 
> ---
>
>  tools/dtoc/fdt.py  | 57 
> ++
>  tools/dtoc/fdt_fallback.py | 16 -
>  tools/dtoc/fdt_normal.py   | 16 -
>  3 files changed, 57 insertions(+), 32 deletions(-)

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

Re: [U-Boot] [PATCH 12/30] patman: Add a tools library for using temporary files

[Simon Glass]

On 25 July 2016 at 18:59, Simon Glass  wrote:
> For tools which want to use input files and temporary output, it is useful
> to have the handling of these dealt with in one place. Add a new library
> which allows input files to be read, and output files to be written, all
> based on a common directory structure.
>
> Signed-off-by: Simon Glass 
> ---
>
>  tools/patman/tools.py | 120 
> ++
>  1 file changed, 120 insertions(+)
>  create mode 100644 tools/patman/tools.py

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

Re: [U-Boot] [PATCH 08/30] dtoc: Create a base class for Fdt

[Simon Glass]

On 25 July 2016 at 18:59, Simon Glass  wrote:
> At present we have two separate implementations of the Fdt library, one which
> uses fdtget/fdtput and one which uses libfdt (via swig).
>
> Before adding more functionality it makes sense to create a base class for
> these. This will allow common functions to be shared, and make the Fdt API
> a little clearer.
>
> Create a new fdt.py file with the base class, and adjust fdt_normal.py and
> fdt_fallback.py to use it.
>
> Signed-off-by: Simon Glass 
> ---
>
>  tools/dtoc/fdt.py  | 68 
> ++
>  tools/dtoc/fdt_fallback.py | 61 +
>  tools/dtoc/fdt_normal.py   | 58 +--
>  tools/dtoc/fdt_select.py   |  9 --
>  4 files changed, 148 insertions(+), 48 deletions(-)
>  create mode 100644 tools/dtoc/fdt.py

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

Re: [U-Boot] [PATCH 07/30] dtoc: Rename fdt.py to fdt_normal.py

[Simon Glass]

On 25 July 2016 at 18:59, Simon Glass  wrote:
> In preparation for creating an Fdt base class, rename this file to indicate
> it is the normal Fdt implementation.
>
> Signed-off-by: Simon Glass 
> ---
>
>  tools/dtoc/{fdt.py => fdt_normal.py} | 0
>  tools/dtoc/fdt_select.py | 2 +-
>  2 files changed, 1 insertion(+), 1 deletion(-)
>  rename tools/dtoc/{fdt.py => fdt_normal.py} (100%)

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

Re: [U-Boot] [PATCH 06/30] dtoc: Move the fdt library selection into fdt_select

[Simon Glass]

On 25 July 2016 at 18:59, Simon Glass  wrote:
> Rather than have dtc worry about which fdt library to use, move this into
> a helper file. Add a function which creates a new Fdt object and scans it,
> regardless of the implementation.
>
> Signed-off-by: Simon Glass 
> ---
>
>  tools/dtoc/dtoc.py   | 18 +++---
>  tools/dtoc/fdt_select.py | 23 +++
>  2 files changed, 26 insertions(+), 15 deletions(-)
>  create mode 100644 tools/dtoc/fdt_select.py

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

Re: [U-Boot] [PATCH 05/30] dtoc: Move the struct import into the correct order

[Simon Glass]

On 25 July 2016 at 18:59, Simon Glass  wrote:
> This should be in with the other system includes. Move it.
>
> Signed-off-by: Simon Glass 
> ---
>
>  tools/dtoc/dtoc.py | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)

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

Re: [U-Boot] [PATCH 04/30] patman: Adjust command.Output() to raise an error by default

[Simon Glass]

On 25 July 2016 at 18:59, Simon Glass  wrote:
> It is more useful to have this method raise an error when something goes
> wrong. Make this the default and adjust the few callers that don't want to
> use it this way.
>
> Signed-off-by: Simon Glass 
> ---
>
>  tools/buildman/control.py  | 2 +-
>  tools/patman/checkpatch.py | 3 ++-
>  tools/patman/command.py| 5 +++--
>  tools/patman/gitutil.py| 3 ++-
>  4 files changed, 8 insertions(+), 5 deletions(-)

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

[U-Boot] Please pull u-boot-rockchip

[Simon Glass]

Hi Tom,

The following changes since commit da968c7bfa4cc1203a4f9f61a97f55c85dfbb3b6:

  Merge branch 'master' of git://git.denx.de/u-boot-i2c (2016-08-26
07:42:06 -0400)

are available in the git repository at:

  git://git.denx.de/u-boot-rockchip.git

for you to fetch changes up to bc2f8a5406350def670bb2dbfdfd83e4b495:

  rockchip: rk3399: update MAINTAINER file (2016-08-27 08:48:23 -0600)


Kever Yang (1):
  rockchip: rk3399: update MAINTAINER file

 board/rockchip/evb_rk3399/MAINTAINERS | 6 ++
 1 file changed, 6 insertions(+)

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

Re: [U-Boot] [PATCH 00/30] binman: A tool for creating firmware images

[Simon Glass]

Hi,

On 25 July 2016 at 18:58, Simon Glass  wrote:
> This series introduces binman, a tool designed to create firmware images.
> It provides a way to bring together various binaries and place them in an
> image, at particular positions and with configurable alignment.
>
> Packaging of firmware is quite a different task from building the various
> parts. In many cases the various binaries which go into the image come from
> separate build systems. For example, ARM Trusted Firmware is used on ARMv8
> devices but is not built in the U-Boot tree. If a Linux kernel is included
> in the firmware image, it is built elsewhere.
>
> It is of course possible to add more and more build rules to the U-Boot
> build system to cover these cases. It can shell out to other Makefiles and
> build scripts. But it seems better to create a clear divide between building
> software and packaging it.
>
> U-Boot supports a very large number of boards. Many of these have their own
> specific rules for how an image should be put together so that it boots
> correctly. At present these rules are described by manual instructions,
> different for each board. By turning these instructions into a standard
> format, we can support making valid images for any board without manual
> effort, lots of READMEs, etc.
>
> Images consist of a number of entries which are combined to make up the
> final image. The image is described in the device tree for the board, meaning
> that it can be accessed at run-time if desired.
>
> Binman is an extensible tool. A set of standard entries is provides, but
> new entries can be created fairly easily. Entries can be as simple as
> providing the name of a file to include. They can also handle more complex
> requirements, such as adjusting the input file or reading header information
> from one entry to control the position of another.
>
> U-Boot's mkimage builds FIT images and various other binaries. Binman
> augments this by allowing these binaries to be packed together. While FIT
> should be used where possible, it cannot be used everywhere. For example,
> many devices require executable code at a particular offset in the image.
> X86 machines require lots of binary blobs at particular places, and a
> microcode collection easily accessible at boot.
>
> So far binman has enough functionality to be useful, but apart from a few RFC
> patches, no attempt is made to switch boards over to use it. There should be
> enough material to permit review and comments.
>
> The series is available at u-boot-dm/binman-working
>
> Future work and missing features are documented in the README.

I'm going to apply the clean-up parts of this series to u-boot-dm/next
and then resend just the tool as a shorter series with some updates.

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

Re: [U-Boot] [PATCH 1/3 v2] Sandbox: document support of block device emulation

[Simon Glass]

On 11 August 2016 at 14:52, Stefan Brüns  wrote:
> Signed-off-by: Stefan Brüns 
> Acked-by: Simon Glass 
> ---
>  board/sandbox/README.sandbox | 19 +++
>  1 file changed, 19 insertions(+)

Applied to u-boot-dm/next, thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 2/3 v2] sandbox: Add "host size" hostfs command for fs test

[Simon Glass]

On 11 August 2016 at 14:52, Stefan Brüns  wrote:
> This complements the size/fatsize/ext4size commands added in
> commit cf6598193aed5de8855eaf70c1994f2bc437255a
> load, save and ls are already implemented for hostfs, now tests can
> cover the same operations on hostfs and emulated block devices.
>
> Signed-off-by: Stefan Brüns 
> Acked-by: Simon Glass 
> ---
>  cmd/host.c | 8 
>  1 file changed, 8 insertions(+)

Applied to u-boot-dm/next, thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [RFC PATCH] rockchip: rk3288: skip lowlevel_init process

[Ziyuan Xu]

lowlevel_init() is never needed for rk3288, so drop it.

Signed-off-by: Ziyuan Xu 

---

 arch/arm/mach-rockchip/board.c| 4 
 arch/arm/mach-rockchip/rk3288-board-spl.c | 4 
 include/configs/rk3288_common.h   | 1 +
 3 files changed, 1 insertion(+), 8 deletions(-)

diff --git a/arch/arm/mach-rockchip/board.c b/arch/arm/mach-rockchip/board.c
index bec756d..6c36bf9 100644
--- a/arch/arm/mach-rockchip/board.c
+++ b/arch/arm/mach-rockchip/board.c
@@ -81,10 +81,6 @@ void enable_caches(void)
 }
 #endif
 
-void lowlevel_init(void)
-{
-}
-
 #if defined(CONFIG_USB_GADGET) && defined(CONFIG_USB_GADGET_DWC2_OTG)
 #include 
 #include 
diff --git a/arch/arm/mach-rockchip/rk3288-board-spl.c 
b/arch/arm/mach-rockchip/rk3288-board-spl.c
index e0d92a6..ae509ff 100644
--- a/arch/arm/mach-rockchip/rk3288-board-spl.c
+++ b/arch/arm/mach-rockchip/rk3288-board-spl.c
@@ -280,7 +280,3 @@ err:
/* No way to report error here */
hang();
 }
-
-void lowlevel_init(void)
-{
-}
diff --git a/include/configs/rk3288_common.h b/include/configs/rk3288_common.h
index d3d4c68..e909ef9 100644
--- a/include/configs/rk3288_common.h
+++ b/include/configs/rk3288_common.h
@@ -11,6 +11,7 @@
 
 #include 
 
+#define CONFIG_SKIP_LOWLEVEL_INIT_ONLY
 #define CONFIG_SYS_NO_FLASH
 #define CONFIG_NR_DRAM_BANKS   1
 #define CONFIG_ENV_SIZE0x2000
-- 
2.9.2


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

Re: [U-Boot] [PATCH] TI: Rework SRAM definitions and maximums

[Lokesh Vutla]

On Friday 26 August 2016 11:00 PM, Tom Rini wrote:
> On all TI platforms the ROM defines a "downloaded image" area at or near
> the start of SRAM which is followed by a reserved area.  As it is at
> best bad form and at worst possibly harmful in corner cases to write in
> this reserved area, we stop doing that by adding in the define
> NON_SECURE_SRAM_IMG_END to say where the end of the downloaded image
> area is and make SRAM_SCRATCH_SPACE_ADDR be one kilobyte before this.
> At current we define the end of scratch space at 0x228 bytes past the
> start of scratch space this this gives us a lot of room to grow.  As
> these scratch uses are non-optional today, all targets are modified to
> respect this boundary.
> 
> Tested on OMAP4 Pandaboard, OMAP3 Beagle xM

Verified on AM437x-GP EVM.

Tested-by: Lokesh Vutla 
Acked-by: Lokesh Vutla 

Thanks and regards,
Lokesh


> 
> Cc: Albert Aribaud 
> Cc: Nagendra T S 
> Cc: Vaibhav Hiremath 
> Cc: Lokesh Vutla 
> Cc: Felipe Balbi 
> Cc: Igor Grinberg 
> Cc: Nikita Kiryanov 
> Cc: Paul Kocialkowski 
> Cc: Enric Balletbo i Serra 
> Cc: Adam Ford 
> Cc: Steve Sakoman 
> Cc: Stefan Roese 
> Cc: Thomas Weber 
> Cc: Hannes Schmelzer 
> Cc: Thomas Chou 
> Cc: Masahiro Yamada 
> Cc: Simon Glass 
> Cc: Joe Hershberger 
> Cc: Sam Protsenko 
> Cc: Heiko Schocher 
> Cc: Samuel Egli 
> Cc: Michal Simek 
> Cc: Wolfgang Denk 
> Cc: Mateusz Kulikowski 
> Cc: Ben Whitten 
> Cc: Stefano Babic 
> Cc: Bin Meng 
> Cc: Sekhar Nori 
> Cc: Mugunthan V N 
> Cc: "B, Ravi" 
> Cc: "Matwey V. Kornilov" 
> Cc: Ladislav Michl 
> Cc: Ash Charles 
> Cc: "Kipisz, Steven" 
> Cc: Daniel Allred 
> Signed-off-by: Tom Rini 
> ---
>  arch/arm/include/asm/arch-am33xx/omap.h | 13 ++---
>  arch/arm/include/asm/arch-omap3/omap.h  |  5 -
>  arch/arm/include/asm/arch-omap4/omap.h  |  5 -
>  arch/arm/include/asm/arch-omap5/omap.h  |  6 +-
>  include/configs/am3517_crane.h  |  3 ++-
>  include/configs/am3517_evm.h|  3 ++-
>  include/configs/am43xx_evm.h|  3 ---
>  include/configs/am57xx_evm.h|  2 --
>  include/configs/bur_am335x_common.h |  6 --
>  include/configs/cm_t35.h|  3 ++-
>  include/configs/cm_t43.h|  1 -
>  include/configs/dra7xx_evm.h|  2 --
>  include/configs/kc1.h   |  3 ++-
>  include/configs/omap3_evm.h |  3 ++-
>  include/configs/omap3_igep00x0.h|  8 
>  include/configs/omap3_logic.h   | 10 ++
>  include/configs/omap3_overo.h   |  8 +---
>  include/configs/siemens-am33x-common.h  |  3 ++-
>  include/configs/sniper.h|  3 ++-
>  include/configs/tam3517-common.h|  3 ++-
>  include/configs/tao3530.h   |  3 ++-
>  include/configs/ti814x_evm.h|  3 ++-
>  include/configs/ti816x_evm.h|  3 ++-
>  include/configs/ti_am335x_common.h  |  1 -
>  include/configs/ti_armv7_common.h   | 21 +
>  include/configs/ti_omap3_common.h   |  1 -
>  include/configs/ti_omap4_common.h   |  1 -
>  include/configs/ti_omap5_common.h   |  7 ---
>  include/configs/tricorder.h |  3 ++-
>  29 files changed, 75 insertions(+), 61 deletions(-)
> 
> diff --git a/arch/arm/include/asm/arch-am33xx/omap.h 
> b/arch/arm/include/asm/arch-am33xx/omap.h
> index 47962dadf59c..3293caaca4ab 100644
> --- a/arch/arm/include/asm/arch-am33xx/omap.h
> +++ b/arch/arm/include/asm/arch-am33xx/omap.h
> @@ -15,24 +15,23 @@
>  #ifndef _OMAP_H_
>  #define _OMAP_H_
>  
> +#include 
> +
>  #ifdef CONFIG_AM33XX
>  #define NON_SECURE_SRAM_START0x402F0400
>  #define NON_SECURE_SRAM_END  0x4031
> -#define SRAM_SCRATCH_SPACE_ADDR  0x4030B800
> +#define NON_SECURE_SRAM_IMG_END  0x4030B800
>  #elif defined(CONFIG_TI81XX)
>  #define NON_SECURE_SRAM_START0x4030
>  #define NON_SECURE_SRAM_END  0x4032
> -#define SRAM_SCRATCH_SPACE_ADDR  0x4031B800
> +#define NON_SECURE_SRAM_IMG_END  0x4031B800
>  #elif defined(CONFIG_AM43XX)
>  #define NON_SECURE_SRAM_START0x402F0400
>  #define NON_SECURE_SRAM_END  0x4034
> -#define 

Re: [U-Boot] [PATCH] TI: Rework SRAM definitions and maximums

[Adam Ford]

On Fri, Aug 26, 2016 at 12:30 PM, Tom Rini  wrote:
> On all TI platforms the ROM defines a "downloaded image" area at or near
> the start of SRAM which is followed by a reserved area.  As it is at
> best bad form and at worst possibly harmful in corner cases to write in
> this reserved area, we stop doing that by adding in the define
> NON_SECURE_SRAM_IMG_END to say where the end of the downloaded image
> area is and make SRAM_SCRATCH_SPACE_ADDR be one kilobyte before this.
> At current we define the end of scratch space at 0x228 bytes past the
> start of scratch space this this gives us a lot of room to grow.  As
> these scratch uses are non-optional today, all targets are modified to
> respect this boundary.
>
> Tested on OMAP4 Pandaboard, OMAP3 Beagle xM
>

Tested on LogicPD DM3730 Torpedo
Tested by Adam Ford 

> Cc: Albert Aribaud 
> Cc: Nagendra T S 
> Cc: Vaibhav Hiremath 
> Cc: Lokesh Vutla 
> Cc: Felipe Balbi 
> Cc: Igor Grinberg 
> Cc: Nikita Kiryanov 
> Cc: Paul Kocialkowski 
> Cc: Enric Balletbo i Serra 
> Cc: Adam Ford 
> Cc: Steve Sakoman 
> Cc: Stefan Roese 
> Cc: Thomas Weber 
> Cc: Hannes Schmelzer 
> Cc: Thomas Chou 
> Cc: Masahiro Yamada 
> Cc: Simon Glass 
> Cc: Joe Hershberger 
> Cc: Sam Protsenko 
> Cc: Heiko Schocher 
> Cc: Samuel Egli 
> Cc: Michal Simek 
> Cc: Wolfgang Denk 
> Cc: Mateusz Kulikowski 
> Cc: Ben Whitten 
> Cc: Stefano Babic 
> Cc: Bin Meng 
> Cc: Sekhar Nori 
> Cc: Mugunthan V N 
> Cc: "B, Ravi" 
> Cc: "Matwey V. Kornilov" 
> Cc: Ladislav Michl 
> Cc: Ash Charles 
> Cc: "Kipisz, Steven" 
> Cc: Daniel Allred 
> Signed-off-by: Tom Rini 
> ---
>  arch/arm/include/asm/arch-am33xx/omap.h | 13 ++---
>  arch/arm/include/asm/arch-omap3/omap.h  |  5 -
>  arch/arm/include/asm/arch-omap4/omap.h  |  5 -
>  arch/arm/include/asm/arch-omap5/omap.h  |  6 +-
>  include/configs/am3517_crane.h  |  3 ++-
>  include/configs/am3517_evm.h|  3 ++-
>  include/configs/am43xx_evm.h|  3 ---
>  include/configs/am57xx_evm.h|  2 --
>  include/configs/bur_am335x_common.h |  6 --
>  include/configs/cm_t35.h|  3 ++-
>  include/configs/cm_t43.h|  1 -
>  include/configs/dra7xx_evm.h|  2 --
>  include/configs/kc1.h   |  3 ++-
>  include/configs/omap3_evm.h |  3 ++-
>  include/configs/omap3_igep00x0.h|  8 
>  include/configs/omap3_logic.h   | 10 ++
>  include/configs/omap3_overo.h   |  8 +---
>  include/configs/siemens-am33x-common.h  |  3 ++-
>  include/configs/sniper.h|  3 ++-
>  include/configs/tam3517-common.h|  3 ++-
>  include/configs/tao3530.h   |  3 ++-
>  include/configs/ti814x_evm.h|  3 ++-
>  include/configs/ti816x_evm.h|  3 ++-
>  include/configs/ti_am335x_common.h  |  1 -
>  include/configs/ti_armv7_common.h   | 21 +
>  include/configs/ti_omap3_common.h   |  1 -
>  include/configs/ti_omap4_common.h   |  1 -
>  include/configs/ti_omap5_common.h   |  7 ---
>  include/configs/tricorder.h |  3 ++-
>  29 files changed, 75 insertions(+), 61 deletions(-)
>
> diff --git a/arch/arm/include/asm/arch-am33xx/omap.h 
> b/arch/arm/include/asm/arch-am33xx/omap.h
> index 47962dadf59c..3293caaca4ab 100644
> --- a/arch/arm/include/asm/arch-am33xx/omap.h
> +++ b/arch/arm/include/asm/arch-am33xx/omap.h
> @@ -15,24 +15,23 @@
>  #ifndef _OMAP_H_
>  #define _OMAP_H_
>
> +#include 
> +
>  #ifdef CONFIG_AM33XX
>  #define NON_SECURE_SRAM_START  0x402F0400
>  #define NON_SECURE_SRAM_END0x4031
> -#define SRAM_SCRATCH_SPACE_ADDR0x4030B800
> +#define NON_SECURE_SRAM_IMG_END0x4030B800
>  #elif defined(CONFIG_TI81XX)
>  #define NON_SECURE_SRAM_START  0x4030
>  #define NON_SECURE_SRAM_END0x4032
> -#define SRAM_SCRATCH_SPACE_ADDR0x4031B800
> +#define NON_SECURE_SRAM_IMG_END0x4031B800
>  #elif defined(CONFIG_AM43XX)
>  #define NON_SECURE_SRAM_START  0x402F0400
>  #define NON_SECURE_SRAM_END0x4034
> -#define SRAM_SCRATCH_SPACE_ADDR0x40337C00
> -#define AM4372_BOARD_NAME_START 

Re: [U-Boot] [PATCH 8/8] armv8: ls1046ardb: Add LS1046ARDB board support

[Prabhakar Kushwaha]

> -Original Message-
> From: Gong Qianyu [mailto:qianyu.g...@nxp.com]
> Sent: Friday, August 26, 2016 6:29 AM
> To: u-boot@lists.denx.de; york sun 
> Cc: Prabhakar Kushwaha ; Mingkai Hu
> ; Shaohui Xie ; Zhiqiang Hou
> ; Wenbin Song ; Mingkai Hu
> ; Qianyu Gong 
> Subject: [PATCH 8/8] armv8: ls1046ardb: Add LS1046ARDB board support
> 
> From: Mingkai Hu 
> 
> LS1046ARDB Specification:
> -
> Memory subsystem:
>  * 8GByte DDR4 SDRAM (64bit bus)
>  * 512 Mbyte NAND flash
>  * Two 64 Mbyte high-speed SPI flash
>  * SD connector to interface with the SD memory card
>  * On-board 4G eMMC
> 
> Ethernet:
>  * Two XFI 10G ports
>  * Two SGMII ports
>  * Two RGMII ports
> 
> PCIe:
>  * PCIe1 (SerDes2 Lane0) to miniPCIe slot
>  * PCIe2 (SerDes2 Lane1) to x2 PCIe slot
>  * PCIe3 (SerDes2 Lane2) to x4 PCIe slot
> 
> SATA:
>  * SerDes2 Lane3 to SATA port
> 
> USB 3.0: one super speed USB 3.0 type A port
>one Micro-AB port
> 
> UART: supports two UARTs up to 115200 bps for console
> 
> Signed-off-by: Gong Qianyu 
> Signed-off-by: Mingkai Hu 
> ---
>  arch/arm/Kconfig   |   9 +
>  arch/arm/dts/Makefile  |   1 +
>  arch/arm/dts/fsl-ls1046a-rdb.dts   |  44 
>  arch/arm/dts/fsl-ls1046a.dtsi  | 220 +++
>  board/freescale/ls1046ardb/Kconfig |  16 ++
>  board/freescale/ls1046ardb/MAINTAINERS |   8 +
>  board/freescale/ls1046ardb/Makefile|  10 +
>  board/freescale/ls1046ardb/README  |  67 ++
>  board/freescale/ls1046ardb/cpld.c  | 158 ++
>  board/freescale/ls1046ardb/cpld.h  |  49 +
>  board/freescale/ls1046ardb/ddr.c   | 140 
>  board/freescale/ls1046ardb/ddr.h   |  44 
>  board/freescale/ls1046ardb/eth.c   |  77 +++
>  board/freescale/ls1046ardb/ls1046ardb.c| 173 +++
>  board/freescale/ls1046ardb/ls1046ardb_pbi.cfg  |  22 ++
>  board/freescale/ls1046ardb/ls1046ardb_rcw_emmc.cfg |   7 +
>  board/freescale/ls1046ardb/ls1046ardb_rcw_sd.cfg   |   7 +
>  .../ls1046ardb/ls1046ardb_rcw_sd_1200.cfg  |   7 +
>  .../ls1046ardb/ls1046ardb_rcw_sd_1400.cfg  |   7 +
>  .../ls1046ardb/ls1046ardb_rcw_sd_5506.cfg  |   7 +
>  configs/ls1046ardb_qspi_defconfig  |  25 +++
>  configs/ls1046ardb_sdcard_defconfig|  26 +++
>  include/configs/ls1046a_common.h   | 181 
>  include/configs/ls1046ardb.h   | 237 
> +
>  24 files changed, 1542 insertions(+)
> 
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> index aef901c..d343995 100644
> --- a/arch/arm/Kconfig
> +++ b/arch/arm/Kconfig
> @@ -811,6 +811,14 @@ config TARGET_LS1043ARDB
>   help
> Support for Freescale LS1043ARDB platform.
> 
> +config TARGET_LS1046ARDB
> + bool "Support ls1046ardb"
> + select ARM64
> + select ARMV8_MULTIENTRY
> + select SUPPORT_SPL
> + help
> +   Support for Freescale LS1046ARDB platform.
> +

Have you run checkpatch.
I guess we get warning if help is  < 60 characters. 

>  config TARGET_H2200
>   bool "Support h2200"
>   select CPU_PXA
> @@ -954,6 +962,7 @@ source "board/freescale/ls1021aqds/Kconfig"
>  source "board/freescale/ls1043aqds/Kconfig"
>  source "board/freescale/ls1021atwr/Kconfig"
>  source "board/freescale/ls1043ardb/Kconfig"
> +source "board/freescale/ls1046ardb/Kconfig"
>  source "board/freescale/ls1012aqds/Kconfig"
>  source "board/freescale/ls1012ardb/Kconfig"
>  source "board/freescale/ls1012afrdm/Kconfig"
> diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
> index 223124e..95b40f6 100644
> --- a/arch/arm/dts/Makefile
> +++ b/arch/arm/dts/Makefile
> @@ -142,6 +142,7 @@ dtb-$(CONFIG_FSL_LSCH3) += fsl-ls2080a-qds.dtb \
>  dtb-$(CONFIG_FSL_LSCH2) += fsl-ls1043a-qds-duart.dtb \
>   fsl-ls1043a-qds-lpuart.dtb \
>   fsl-ls1043a-rdb.dtb \
> + fsl-ls1046a-rdb.dtb \
>   fsl-ls1012a-qds.dtb \
>   fsl-ls1012a-rdb.dtb \
>   fsl-ls1012a-frdm.dtb
> diff --git a/arch/arm/dts/fsl-ls1046a-rdb.dts 
> b/arch/arm/dts/fsl-ls1046a-rdb.dts
> new file mode 100644
> index 000..91f6ced
> --- /dev/null
> +++ b/arch/arm/dts/fsl-ls1046a-rdb.dts
> @@ -0,0 +1,44 @@
> +/*
> + * Device Tree Include file for Freescale Layerscape-1043A family SoC.

LS1043?
 I guess this patch is for LS1046

> + *
> + * Copyright 2016, Freescale Semiconductor
> + *
> + * Mingkai Hu 
> + *
> + * This file is licensed under the terms of the GNU General Public
> + * License version 2.  This 

[U-Boot] [PATCH 0/2] ARM: TI: Enable DISPLAY_{CPU/BOARD}INFO

[Lokesh Vutla]

This series adds ID code detection support for AM437x and enables
CONFIG_DISPLAY_{CPU/BOARD}INFO on all TI platforms.

Lokesh Vutla (2):
  ARM: AM437X: Add Silicon ID support
  configs: ti: armv7_common: Enable DISPLAY_{CPU/BOARD}INFO

 arch/arm/cpu/armv7/am33xx/sys_info.c   | 5 -
 arch/arm/include/asm/arch-am33xx/cpu.h | 3 ++-
 include/configs/ti_armv7_common.h  | 4 
 include/configs/ti_armv7_keystone2.h   | 1 -
 include/configs/ti_omap4_common.h  | 2 --
 include/configs/ti_omap5_common.h  | 3 ---
 6 files changed, 10 insertions(+), 8 deletions(-)

-- 
2.9.3

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

[U-Boot] [PATCH 2/2] configs: ti: armv7_common: Enable DISPLAY_{CPU/BOARD}INFO

[Lokesh Vutla]

Enable CONFIG_DISPLAY_{CPU/BOARD}INFO on all TI platforms.
Also removing the duplicated configs.

Signed-off-by: Lokesh Vutla 
---
 include/configs/ti_armv7_common.h| 4 
 include/configs/ti_armv7_keystone2.h | 1 -
 include/configs/ti_omap4_common.h| 2 --
 include/configs/ti_omap5_common.h| 3 ---
 4 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/include/configs/ti_armv7_common.h 
b/include/configs/ti_armv7_common.h
index 9f947ee..b11ddb2 100644
--- a/include/configs/ti_armv7_common.h
+++ b/include/configs/ti_armv7_common.h
@@ -284,6 +284,10 @@
 #endif
 #endif /* !CONFIG_NOR_BOOT */
 
+/* Display CPU and Board info */
+#define CONFIG_DISPLAY_CPUINFO
+#define CONFIG_DISPLAY_BOARDINFO
+
 /* Generic Environment Variables */
 
 #ifdef CONFIG_CMD_NET
diff --git a/include/configs/ti_armv7_keystone2.h 
b/include/configs/ti_armv7_keystone2.h
index 4aa262e..9241323 100644
--- a/include/configs/ti_armv7_keystone2.h
+++ b/include/configs/ti_armv7_keystone2.h
@@ -15,7 +15,6 @@
 /* U-Boot Build Configuration */
 #define CONFIG_SKIP_LOWLEVEL_INIT  /* U-Boot is a 2nd stage loader */
 #define CONFIG_BOARD_EARLY_INIT_F
-#define CONFIG_DISPLAY_CPUINFO
 
 /* SoC Configuration */
 #define CONFIG_ARCH_CPU_INIT
diff --git a/include/configs/ti_omap4_common.h 
b/include/configs/ti_omap4_common.h
index 5fad3c1..24d4c83 100644
--- a/include/configs/ti_omap4_common.h
+++ b/include/configs/ti_omap4_common.h
@@ -17,8 +17,6 @@
  */
 #define CONFIG_OMAP44301   /* which is in a 4430 */
 #define CONFIG_MISC_INIT_R
-#define CONFIG_DISPLAY_CPUINFO 1
-#define CONFIG_DISPLAY_BOARDINFO   1
 
 #define CONFIG_SYS_THUMB_BUILD
 
diff --git a/include/configs/ti_omap5_common.h 
b/include/configs/ti_omap5_common.h
index 3589cdc..2d859e1 100644
--- a/include/configs/ti_omap5_common.h
+++ b/include/configs/ti_omap5_common.h
@@ -17,9 +17,6 @@
 #ifndef __CONFIG_TI_OMAP5_COMMON_H
 #define __CONFIG_TI_OMAP5_COMMON_H
 
-#define CONFIG_DISPLAY_CPUINFO
-#define CONFIG_DISPLAY_BOARDINFO
-
 /* Common ARM Erratas */
 #define CONFIG_ARM_ERRATA_798870
 
-- 
2.9.3

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

[U-Boot] [PATCH 1/2] ARM: AM437X: Add Silicon ID support

[Lokesh Vutla]

Add silicon ID code for AM437x silicon. This can be used to print
the cpu info using CONFIG_DISPLAY_CPUINFO.
Also printing "CPU :" along with cpu name in order to be consistent
with other OMAP platforms.

Signed-off-by: Lokesh Vutla 
---
 arch/arm/cpu/armv7/am33xx/sys_info.c   | 5 -
 arch/arm/include/asm/arch-am33xx/cpu.h | 3 ++-
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/arch/arm/cpu/armv7/am33xx/sys_info.c 
b/arch/arm/cpu/armv7/am33xx/sys_info.c
index 52a6824..808faf4 100644
--- a/arch/arm/cpu/armv7/am33xx/sys_info.c
+++ b/arch/arm/cpu/armv7/am33xx/sys_info.c
@@ -97,6 +97,9 @@ int print_cpuinfo(void)
case TI81XX:
cpu_s = "TI81XX";
break;
+   case AM437X:
+   cpu_s = "AM437X";
+   break;
default:
cpu_s = "Unknown CPU type";
break;
@@ -112,7 +115,7 @@ int print_cpuinfo(void)
else
sec_s = "?";
 
-   printf("%s-%s rev %s\n", cpu_s, sec_s, rev_s);
+   printf("CPU  : %s-%s rev %s\n", cpu_s, sec_s, rev_s);
 
return 0;
 }
diff --git a/arch/arm/include/asm/arch-am33xx/cpu.h 
b/arch/arm/include/asm/arch-am33xx/cpu.h
index 62bca8c..4a0dce9 100644
--- a/arch/arm/include/asm/arch-am33xx/cpu.h
+++ b/arch/arm/include/asm/arch-am33xx/cpu.h
@@ -43,7 +43,8 @@
 #define HS_DEVICE  0x2
 #define GP_DEVICE  0x3
 
-/* cpu-id for AM33XX and TI81XX family */
+/* cpu-id for AM43XX AM33XX and TI81XX family */
+#define AM437X 0xB98C
 #define AM335X 0xB944
 #define TI81XX 0xB81E
 #define DEVICE_ID  (CTRL_BASE + 0x0600)
-- 
2.9.3

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

[U-Boot] [PATCH 2/2] board: k2g: Enable ECC byte lane

[Lokesh Vutla]

Enable ECC byte lane for k2g-evm

Signed-off-by: Lokesh Vutla 
---
 arch/arm/mach-keystone/ddr3.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/arch/arm/mach-keystone/ddr3.c b/arch/arm/mach-keystone/ddr3.c
index 34606f4..6b92530 100644
--- a/arch/arm/mach-keystone/ddr3.c
+++ b/arch/arm/mach-keystone/ddr3.c
@@ -65,9 +65,8 @@ void ddr3_init_ddrphy(u32 base, struct ddr3_phy_config 
*phy_cfg)
while ((__raw_readl(base + KS2_DDRPHY_PGSR0_OFFSET) & 0x1) != 0x1)
;
 
-   /* Disable ECC for K2G */
if (cpu_is_k2g()) {
-   clrbits_le32(base + KS2_DDRPHY_DATX8_4_OFFSET, 0x1);
+   setbits_le32(base + KS2_DDRPHY_DATX8_4_OFFSET, 0x1);
clrbits_le32(base + KS2_DDRPHY_DATX8_5_OFFSET, 0x1);
clrbits_le32(base + KS2_DDRPHY_DATX8_6_OFFSET, 0x1);
clrbits_le32(base + KS2_DDRPHY_DATX8_7_OFFSET, 0x1);
-- 
2.9.3

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

[U-Boot] [PATCH 0/2] board: k2g: Enable ECC support

[Lokesh Vutla]

This series enables ECC support on k2g-evm.

Lokesh Vutla (2):
  board: ks2: Enable ECC using detected DDR size
  board: k2g: Enable ECC byte lane

 arch/arm/mach-keystone/ddr3.c | 3 +--
 board/ti/ks2_evm/board.c  | 3 +++
 2 files changed, 4 insertions(+), 2 deletions(-)

-- 
2.9.3

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

[U-Boot] [PATCH 1/2] board: ks2: Enable ECC using detected DDR size

[Lokesh Vutla]

EEC is being enabled based on the ddr size populated by SPD data.
But not all keystone platforms have SPD data to detect ddr3 size.
So, enable ECC using the detected DDR size.

Signed-off-by: Lokesh Vutla 
---
 board/ti/ks2_evm/board.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/board/ti/ks2_evm/board.c b/board/ti/ks2_evm/board.c
index 1de7df0..03254e1 100644
--- a/board/ti/ks2_evm/board.c
+++ b/board/ti/ks2_evm/board.c
@@ -50,6 +50,9 @@ int dram_init(void)
 
if (ddr3_size)
ddr3_init_ecc(KS2_DDR3A_EMIF_CTRL_BASE, ddr3_size);
+   else
+   ddr3_init_ecc(KS2_DDR3A_EMIF_CTRL_BASE, gd->ram_size >> 30);
+
return 0;
 }
 
-- 
2.9.3

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

[U-Boot] x86: How to run UEFI BIOS + u-boot + linux on haswell/broadwell board?

[Yan Phenix]

Hi,

I'd like to run u-boot+linux on a haswell+pch board which uses AMI UEFI BIOS. 
Which board in u-boot may I take as a porting template?
Or if any guys with such experience, pls share with me . 
Any suggestion is greatly appreciated.

Thank you.

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

Re: [U-Boot] [PATCH 3/8] armv8: fsl-layerscape: Increase L2 Data RAM latency and L2 Tag RAM latency

[Prabhakar Kushwaha]

> -Original Message-
> From: Gong Qianyu [mailto:qianyu.g...@nxp.com]
> Sent: Friday, August 26, 2016 6:29 AM
> To: u-boot@lists.denx.de; york sun 
> Cc: Prabhakar Kushwaha ; Mingkai Hu
> ; Shaohui Xie ; Zhiqiang Hou
> ; Wenbin Song ; Mingkai Hu
> ; Qianyu Gong 
> Subject: [PATCH 3/8] armv8: fsl-layerscape: Increase L2 Data RAM latency and
> L2 Tag RAM latency
> 
> From: Mingkai Hu 
> 
> Use 3 cycles.
> 

Will it be possible to share more details on this.

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