date:20160131

Hello Simon,

On Thu, 19 Nov 2015 20:29:11 -0700, Simon Glass 
wrote:
> Hi Albert,
> 
> On 13 November 2015 at 19:35, Simon Glass  wrote:
> > On 9 November 2015 at 14:36, Albert ARIBAUD  
> > wrote:
> >> Hello Simon,
> >>
> >> On Mon, 9 Nov 2015 12:24:55 -0800, Simon Glass  wrote:
> >>> On 9 November 2015 at 06:19, Albert ARIBAUD  
> >>> wrote:
> >>> > A patman series with a 'Series-notes' section causes
> >>> > buildman to crash with:
> >>> >
> >>> > self.series.notes += self.section
> >>> > TypeError: cannot concatenate 'str' and 'list' objects
> >>> >
> >>> > Fix by initializing series.notes as a one-element array
> >>> > rathen than a scalar.
> >>> >
> >>> > Signed-off-by: Albert ARIBAUD 
> >>> > ---
> >>> >
> >>> >  tools/patman/series.py | 2 +-
> >>> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >>>
> >>> Acked-by: Simon Glass 
> >>
> >> Note: just spotted a typo, 'rathen' instead of 'rather'. If this patch
> >> gets applied, please fix that on-the-fly.
> >>
> >> Amicalement,
> >> --
> >> Albert.
> >
> > Fixed typo and
> >
> > Applied to u-boot-dm, thanks!
> 
> Hmm, actually I've had to drop this as it breaks 'Series-version'.
> That currently does not expect a list.

Hmm, I can't reproduce this here. How do you trigger the Series-version
break?

> I wonder if Series-notes needs a special case, instead?
> 
> Regards,
> Simon

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

Re: [U-Boot] [PATCH v2 17/23] tegra: video: Convert tegra20 LCD driver to driver model

On Sat, 30 Jan 2016 16:37:56 -0700
Simon Glass s...@chromium.org wrote:

> Move this driver over to use driver model. This involves rearranging the
> code somewhat. The effect is that everything is run from the probe() method.
> 
> Boards which use this are fixed up, but only seaboard is tested.
> 
> Signed-off-by: Simon Glass 
> ---
> 
> Changes in v2:
> - Fix a checkpatch warning
> 
>  arch/arm/dts/tegra20-seaboard.dts   |   4 +
>  arch/arm/dts/tegra20.dtsi   |   2 +
>  arch/arm/include/asm/arch-tegra20/display.h |  26 --
>  board/compal/paz00/paz00.c  |   2 +-
>  configs/colibri_t20_defconfig   |   4 +
>  configs/harmony_defconfig   |   6 +-
>  configs/medcom-wide_defconfig   |   4 +
>  configs/paz00_defconfig |   4 +
>  configs/seaboard_defconfig  |   4 +
>  configs/tec_defconfig   |   4 +
>  configs/ventana_defconfig   |   4 +
>  drivers/video/tegra.c   | 385 
> +---
>  include/configs/colibri_t20.h   |   2 -
>  include/configs/harmony.h   |   2 -
>  include/configs/medcom-wide.h   |   2 -
>  include/configs/paz00.h |   2 -
>  include/configs/seaboard.h  |   2 -
>  include/configs/tec.h   |   2 -
>  include/configs/ventana.h   |   2 -
>  19 files changed, 208 insertions(+), 255 deletions(-)

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

Re: [U-Boot] [PATCH 1/2] arm: imx6: Add DDR3 calibration code for MX6 Q/D/DL

2016-01-31 Thread Marek Vasut

On Monday, January 25, 2016 at 12:30:00 AM, Tom Rini wrote:
> On Mon, Jan 25, 2016 at 12:00:05AM +0100, Marek Vasut wrote:
> > On Sunday, January 24, 2016 at 11:21:51 PM, Tom Rini wrote:
> > > On Sun, Jan 24, 2016 at 11:07:30PM +0100, Marek Vasut wrote:
> > > > On Sunday, January 24, 2016 at 08:33:57 PM, Tom Rini wrote:
> > > > > On Sun, Jan 24, 2016 at 06:22:10PM +0100, Marek Vasut wrote:
> > > > > > On Sunday, January 24, 2016 at 06:18:23 PM, Stefano Babic wrote:
> > > > > > > On 24/01/2016 18:11, Marek Vasut wrote:
> > > > > > > > It is not clear when the wait_for_bit() will be applied, I am
> > > > > > > > certain there will be another round for so. I do not want to
> > > > > > > > wait for it and I don't see a reason why those patches should
> > > > > > > > block this if the conversion can be done afterward.
> > > > > > > 
> > > > > > > Just wait for a while - if it takes too much, I reconsider to
> > > > > > > apply this first and factorize wait_for_bit() in a follow-up
> > > > > > > patch.
> > > > > > 
> > > > > > I have waited for over a month and I fail to see a reason why
> > > > > > patches which will be applied at uncertain point in the future
> > > > > > shall block this patchset. The wait_for_bit() can be removed by
> > > > > > a subsequent patch, it is already pulled out explicitly in the
> > > > > > code, so I don't see a problem with applying this.
> > > > > 
> > > > > Did I miss something or isn't v4 of wait_for_bit good to go?
> > > > 
> > > > I don't really know if it's good to go, but this patch does not
> > > > depend on it in any way. A subsequent patch can drop the
> > > > wait_for_bit() from here, it's the same as the wait_for_bit() in
> > > > dwc2 and other wait_for_bit() anywhere else, but I don't see a
> > > > reason why this patch should not be applied now.
> > > > 
> > > > If I follow the logic in this thread, it would also be possible to
> > > > say that this patch should wait until Eric submits the MX6S DDR
> > > > support for example. We could indefinitelly wait for new and new
> > > > stuff which might possibly block this.
> > > 
> > > Why don't you ack/test/review the wait_for_bit series and post a
> > > follow-up to this one that uses the common function, which can be
> > > squashed into the original and then this gets picked up?
> > 
> > Why ? I can send subsequent patch which removes the duplicate
> > wait_for_bit() from this code once the wait_for_bit series is applied. I
> > don't see a problem with that and the wait_for_bit() being so explicitly
> > pulled out from the code is done with that in mind. But that does not
> > block this patch from being applied now. Does it?
> 
> If I tell you I'm probably going to have the wait_for_bit stuff applied
> before the next imx PR is ready... ?

So, why is this patchset not applied yet ... ?

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 v2 11/23] tegra: nyan-big: Move the LCD driver to driver model

On Sat, 30 Jan 2016 16:37:50 -0700
Simon Glass s...@chromium.org wrote:

> Adjust the driver to use driver model. The SOR becomes a bridge device. We
> use the normal simple_panel driver to handle the display itself. We also
> need to enable some options such as regulators, PWMs and DM_VIDEO itself.
> 
> Signed-off-by: Simon Glass 
> ---
> 
> Changes in v2: None
> 
>  configs/nyan-big_defconfig   |   9 +++
>  drivers/video/Kconfig|   1 +
>  drivers/video/simple_panel.c |   1 +
>  drivers/video/tegra124/display.c | 169 
> ---
>  drivers/video/tegra124/dp.c  |  53 
>  drivers/video/tegra124/sor.c | 162 +
>  drivers/video/tegra124/sor.h |  45 +--
>  include/configs/nyan-big.h   |   2 -
>  include/fdtdec.h |   1 -
>  lib/fdtdec.c |   1 -
>  10 files changed, 246 insertions(+), 198 deletions(-)

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

Re: [U-Boot] [PATCH v2 22/23] tegra: video: Move LCD driver to use the DM PWM driver

On Sat, 30 Jan 2016 16:38:01 -0700
Simon Glass s...@chromium.org wrote:

> Use the driver-model PWM driver in preference to the old code.
> 
> Signed-off-by: Simon Glass 
> ---
> 
> Changes in v2: None
> 
>  arch/arm/dts/tegra20-colibri.dts |  4 
>  arch/arm/dts/tegra20-harmony.dts |  4 
>  arch/arm/dts/tegra20-medcom-wide.dts |  4 
>  arch/arm/dts/tegra20-paz00.dts   |  4 
>  arch/arm/dts/tegra20-seaboard.dts|  4 
>  arch/arm/dts/tegra20-tec.dts |  4 
>  arch/arm/dts/tegra20-ventana.dts |  4 
>  arch/arm/mach-tegra/Kconfig  |  1 +
>  drivers/pwm/Kconfig  |  1 +
>  drivers/video/tegra.c| 20 
>  10 files changed, 46 insertions(+), 4 deletions(-)

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

[U-Boot] [PATCH] OMAP3: omap3_logic: Enable Android Fastboot

2016-01-31 Thread Adam Ford

Android Fastboot requires USB.  The TWL4030 PMIC on omap3_logic handles USB 
traffic.
This patch sets up the USB gadget and Android Fastboot to match what is done in 
the
omap3_beagle project.

Signed-off-by: Adam Ford 
---
 board/logicpd/omap3som/omap3logic.c | 37 +
 include/configs/omap3_logic.h   | 21 +
 2 files changed, 58 insertions(+)

diff --git a/board/logicpd/omap3som/omap3logic.c 
b/board/logicpd/omap3som/omap3logic.c
index 05f6587..345bc93 100644
--- a/board/logicpd/omap3som/omap3logic.c
+++ b/board/logicpd/omap3som/omap3logic.c
@@ -27,6 +27,11 @@
 #include 
 #include 
 #include 
+#include 
+#include 
+#include 
+#include 
+#include 
 #include "omap3logic.h"
 
 DECLARE_GLOBAL_DATA_PTR;
@@ -105,6 +110,34 @@ void get_board_mem_timings(struct board_sdrc_timings 
*timings)
 }
 #endif
 
+#ifdef CONFIG_USB_MUSB_OMAP2PLUS
+static struct musb_hdrc_config musb_config = {
+   .multipoint = 1,
+   .dyn_fifo   = 1,
+   .num_eps= 16,
+   .ram_bits   = 12,
+};
+
+static struct omap_musb_board_data musb_board_data = {
+   .interface_type = MUSB_INTERFACE_ULPI,
+};
+
+static struct musb_hdrc_platform_data musb_plat = {
+#if defined(CONFIG_USB_MUSB_HOST)
+   .mode   = MUSB_HOST,
+#elif defined(CONFIG_USB_MUSB_GADGET)
+   .mode   = MUSB_PERIPHERAL,
+#else
+#error "Please define either CONFIG_USB_MUSB_HOST or CONFIG_USB_MUSB_GADGET"
+#endif
+   .config = _config,
+   .power  = 100,
+   .platform_ops   = _ops,
+   .board_data = _board_data,
+};
+#endif
+
+
 /*
  * Routine: misc_init_r
  * Description: Configure board specific parts
@@ -126,6 +159,10 @@ int misc_init_r(void)
omap_die_id_display();
putc('\n');
 
+#ifdef CONFIG_USB_MUSB_OMAP2PLUS
+   musb_register(_plat, _board_data, (void *)MUSB_BASE);
+#endif
+
return 0;
 }
 
diff --git a/include/configs/omap3_logic.h b/include/configs/omap3_logic.h
index b2778ca..1e00455 100644
--- a/include/configs/omap3_logic.h
+++ b/include/configs/omap3_logic.h
@@ -74,8 +74,29 @@
 #define EXPANSION_EEPROM_I2C_BUS   2   /* I2C Bus for AT24C64 */
 #define CONFIG_OMAP3_LOGIC_USE_NEW_PRODUCT_ID
 
+/* USB */
+#define CONFIG_USB_MUSB_GADGET
+#define CONFIG_USB_MUSB_OMAP2PLUS
+#define CONFIG_USB_MUSB_PIO_ONLY
+#define CONFIG_USB_GADGET_DUALSPEED
+#define CONFIG_USB_ETHER
+#define CONFIG_USB_ETHER_RNDIS
+#define CONFIG_USB_GADGET
+#define CONFIG_USB_GADGET_VBUS_DRAW0
+#define CONFIG_USB_GADGET_DOWNLOAD
+#define CONFIG_G_DNL_VENDOR_NUM0x0451
+#define CONFIG_G_DNL_PRODUCT_NUM   0xd022
+#define CONFIG_G_DNL_MANUFACTURER  "TI"
+#define CONFIG_USB_FUNCTION_FASTBOOT
+#define CONFIG_CMD_FASTBOOT
+#define CONFIG_ANDROID_BOOT_IMAGE
+#define CONFIG_FASTBOOT_BUF_ADDR   CONFIG_SYS_LOAD_ADDR
+#define CONFIG_FASTBOOT_BUF_SIZE   0x0700
+#define CONFIG_SYS_CACHELINE_SIZE  64
+
 /* TWL4030 */
 #define CONFIG_TWL4030_PWM
+#define CONFIG_TWL4030_USB
 
 /* Board NAND Info. */
 #ifdef CONFIG_NAND
-- 
1.9.1

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

Re: [U-Boot] [PATCH v2 15/23] tegra: video: Move the check for CONFIG_OF_CONTROL to Kconfig

On Sat, 30 Jan 2016 16:37:54 -0700
Simon Glass s...@chromium.org wrote:

> We can check this in Kconfig now.
> 
> Signed-off-by: Simon Glass 
> ---
> 
> Changes in v2: None
> 
>  drivers/video/Kconfig | 1 +
>  drivers/video/tegra.c | 4 
>  2 files changed, 1 insertion(+), 4 deletions(-)

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

Re: [U-Boot] [PATCH v2 14/23] tegra: video: Merge the display driver into one file

On Sat, 30 Jan 2016 16:37:53 -0700
Simon Glass s...@chromium.org wrote:

> At present we have code in arch/arm and code in drivers/video. Move it all
> into drivers/video since it is a display driver and our current approach is
> to put all driver code in drivers/.
> 
> Make a few functions static now that they are not used outside the file.
> Signed-off-by: Simon Glass 
> ---
> 
> Changes in v2:
> - Fix a checkpatch warning
> 
>  arch/arm/include/asm/arch-tegra20/display.h |  21 --
>  arch/arm/mach-tegra/tegra20/Makefile|   2 -
>  arch/arm/mach-tegra/tegra20/display.c   | 378 --
>  drivers/video/tegra.c   | 396 
> 
>  4 files changed, 396 insertions(+), 401 deletions(-)

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

Re: [U-Boot] [PATCH v2 02/23] dm: video: Flush the cache after a puts()

Hi Simon,

On Sat, 30 Jan 2016 16:37:41 -0700
Simon Glass s...@chromium.org wrote:

> This helps keep the display consistent. puts() is used when printing the
> prompt, so is a useful way to make sure the current display contents is
> visible.
> 
> Signed-off-by: Simon Glass 
> ---
> 
> Changes in v2: None
> 
>  drivers/video/vidconsole-uclass.c | 1 +
>  1 file changed, 1 insertion(+)

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

Re: [U-Boot] [PATCH v2 08/23] tegra: lcd: Merge tegra124-lcd.c into display.c

On Sat, 30 Jan 2016 16:37:47 -0700
Simon Glass s...@chromium.org wrote:

> There isn't a lot of benefit of have two separate files. With driver model
> the code needs to be in the same driver, so it's better to have it in the
> same file.
> 
> Signed-off-by: Simon Glass 
> ---
> 
> Changes in v2: None
> 
>  arch/arm/include/asm/arch-tegra/dc.h  |  5 --
>  drivers/video/tegra124/Makefile   |  1 -
>  drivers/video/tegra124/display.c  | 81 -
>  drivers/video/tegra124/tegra124-lcd.c | 95 
> ---
>  4 files changed, 79 insertions(+), 103 deletions(-)
>  delete mode 100644 drivers/video/tegra124/tegra124-lcd.c

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

[U-Boot] Pull request: u-boot-arm/master

Hello Tom,

The following changes since commit 8cdae1dacde7dbe74d53a8ac1a05761a53c4f191:

  video: Correct 'tor' typo in comment (2016-01-30 10:58:47 +0100)

are available in the git repository at:

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

for you to fetch changes up to 735b1a2e5a61401868bb35702b6e5e18bce6eb97:

  arm: novena: Fix EEPROM i2c configuration (2016-01-31 16:32:56 +0100)


Albert ARIBAUD (1):
  armv7: add cacheline sizes where missing

Marek Vasut (4):
  arm: Replace test for CONFIG_ARMV7 with CONFIG_CPU_V7
  arm: Remove S bit from MMU section entry
  arm: cache: Implement cache range check for v7
  arm: novena: Fix EEPROM i2c configuration

Peng Fan (1):
  arm: config: enforce -fno-pic for gcc

Wang Dongsheng (1):
  ARM: Disable "DISCARD" for secure section if CONFIG_ARMV7_SECURE_BASE 
isn't defined

 arch/arm/config.mk  |  1 +
 arch/arm/cpu/armv7/cache_v7.c   | 21 +
 arch/arm/cpu/u-boot.lds | 27 ++-
 arch/arm/include/asm/system.h   |  7 +++
 arch/arm/lib/cache-cp15.c   |  2 +-
 include/configs/am3517_crane.h  |  2 ++
 include/configs/am3517_evm.h|  2 ++
 include/configs/at91-sama5_common.h |  2 ++
 include/configs/bcm_ep_board.h  |  1 +
 include/configs/cm_t35.h|  2 ++
 include/configs/cm_t3517.h  |  2 ++
 include/configs/colibri_vf.h|  2 ++
 include/configs/kzm9g.h |  2 ++
 include/configs/nokia_rx51.h|  2 ++
 include/configs/novena.h|  3 ++-
 include/configs/pcm052.h|  2 ++
 include/configs/rcar-gen2-common.h  |  2 ++
 include/configs/rk3036_common.h |  2 ++
 include/configs/rk3288_common.h |  2 ++
 include/configs/s5p_goni.h  |  2 ++
 include/configs/smdkc100.h  |  2 ++
 include/configs/tao3530.h   |  2 ++
 include/configs/ti814x_evm.h|  2 ++
 include/configs/ti816x_evm.h|  2 ++
 include/configs/ti_omap3_common.h   |  5 +
 include/configs/tricorder.h |  2 ++
 include/configs/vexpress_common.h   |  2 ++
 include/configs/vf610twr.h  |  2 ++
 28 files changed, 88 insertions(+), 19 deletions(-)

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

[U-Boot] [PATCH] OMAP3: omap3_logic: Remove YAFFS support.

2016-01-31 Thread Adam Ford

UBIFS is the preferred OS, and YAFFS isn't officially included in
Linux.  Removing this feature reduces the code size.

Signed-off-by: Adam Ford 
---
 include/configs/omap3_logic.h | 2 --
 1 file changed, 2 deletions(-)

diff --git a/include/configs/omap3_logic.h b/include/configs/omap3_logic.h
index a153928..3cb71f1 100644
--- a/include/configs/omap3_logic.h
+++ b/include/configs/omap3_logic.h
@@ -66,8 +66,6 @@
 #define CONFIG_CMD_PING
 #define CONFIG_CMD_DHCP
 
-#define CONFIG_YAFFS2
-
 /* I2C */
 #define CONFIG_SYS_I2C_OMAP34XX
 #define CONFIG_SYS_I2C_EEPROM_ADDR 0x50/* EEPROM AT24C64  */
-- 
1.9.1

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

Re: [U-Boot] Conversion of serial_pxa.c to driver model

2016-01-31 Thread Marek Vasut

On Sunday, January 31, 2016 at 05:16:37 PM, Simon Glass wrote:
> Hi Marek,

Hi,

> Are you planning to convert this to driver model?

No, I don't plan to convert it any soon. CCing some more interested parties.

> I see that two boards still use it.

That's only because you removed the rest apparently ...

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 v2 20/23] tegra: video: Move all fdt-decoding into a single function

On Sat, 30 Jan 2016 16:37:59 -0700
Simon Glass s...@chromium.org wrote:

> Join the two functions which decode the device tree and put them in the
> ofdata_to_platdata() method.
> 
> Signed-off-by: Simon Glass 
> ---
> 
> Changes in v2: None
> 
>  drivers/video/tegra.c | 264 
> --
>  1 file changed, 105 insertions(+), 159 deletions(-)

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

Re: [U-Boot] [PATCH v2 18/23] tegra: video: Remove the static variables

2016-01-31 Thread Fabio Estevam

On Sat, Jan 30, 2016 at 9:37 PM, Simon Glass  wrote:

> diff --git a/drivers/video/tegra.c b/drivers/video/tegra.c
> index bb2601f..f49f95d 100644
> --- a/drivers/video/tegra.c
> +++ b/drivers/video/tegra.c
> @@ -2,7 +2,7 @@
>   * Copyright (c) 2011 The Chromium OS Authors.
>   * SPDX-License-Identifier:GPL-2.0+
>   */
> -#define DEBUG
> +

This is an unrelated change.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] OMAP3: omap3_logic: Enable CONFIG_SMC911X_32_BIT

2016-01-31 Thread Adam Ford

The interface automatically converts one 32-bit word into two 16 words.
The README said it is permissible to use this flag in that scenario.

Signed-off-by: Adam Ford 
---
 include/configs/omap3_logic.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/configs/omap3_logic.h b/include/configs/omap3_logic.h
index 1e00455..a153928 100644
--- a/include/configs/omap3_logic.h
+++ b/include/configs/omap3_logic.h
@@ -286,7 +286,7 @@
 /* SMSC922x Ethernet */
 #if defined(CONFIG_CMD_NET)
 #define CONFIG_SMC911X
-#define CONFIG_SMC911X_16_BIT
+#define CONFIG_SMC911X_32_BIT
 #define CONFIG_SMC911X_BASE0x0800
 #endif /* (CONFIG_CMD_NET) */
 
-- 
1.9.1

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

Re: [U-Boot] [PATCH v2 19/23] tegra: video: Move LCD enums into the driver

On Sat, 30 Jan 2016 16:37:58 -0700
Simon Glass s...@chromium.org wrote:

> There is no need to have these in a separate file as they are not
> referenced from anywhere else.
> 
> Signed-off-by: Simon Glass 
> ---
> 
> Changes in v2: None
> 
>  arch/arm/include/asm/arch-tegra20/display.h | 22 --
>  drivers/video/tegra.c   | 20 
>  2 files changed, 20 insertions(+), 22 deletions(-)

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

[U-Boot] [PATCH] Revert "arm: Remove wireless_space board"

Revert commit b352182a and complete generic board support.

Signed-off-by: Albert ARIBAUD 
---
 arch/arm/mach-kirkwood/Kconfig  |   4 +
 board/LaCie/wireless_space/Kconfig  |  12 ++
 board/LaCie/wireless_space/MAINTAINERS  |   6 +
 board/LaCie/wireless_space/Makefile |  12 ++
 board/LaCie/wireless_space/kwbimage.cfg |  71 +++
 board/LaCie/wireless_space/wireless_space.c | 165 +
 configs/wireless_space_defconfig|   7 ++
 include/configs/wireless_space.h| 179 
 8 files changed, 456 insertions(+)
 create mode 100644 board/LaCie/wireless_space/Kconfig
 create mode 100644 board/LaCie/wireless_space/MAINTAINERS
 create mode 100644 board/LaCie/wireless_space/Makefile
 create mode 100644 board/LaCie/wireless_space/kwbimage.cfg
 create mode 100644 board/LaCie/wireless_space/wireless_space.c
 create mode 100644 configs/wireless_space_defconfig
 create mode 100644 include/configs/wireless_space.h

diff --git a/arch/arm/mach-kirkwood/Kconfig b/arch/arm/mach-kirkwood/Kconfig
index 9205b1e..9249564 100644
--- a/arch/arm/mach-kirkwood/Kconfig
+++ b/arch/arm/mach-kirkwood/Kconfig
@@ -37,6 +37,9 @@ config TARGET_NET2BIG_V2
 config TARGET_NETSPACE_V2
bool "LaCie netspace_v2 Board"
 
+config TARGET_WIRELESS_SPACE
+   bool "LaCie Wireless_space Board"
+
 config TARGET_IB62X0
bool "ib62x0 Board"
 
@@ -68,6 +71,7 @@ source "board/iomega/iconnect/Kconfig"
 source "board/keymile/km_arm/Kconfig"
 source "board/LaCie/net2big_v2/Kconfig"
 source "board/LaCie/netspace_v2/Kconfig"
+source "board/LaCie/wireless_space/Kconfig"
 source "board/raidsonic/ib62x0/Kconfig"
 source "board/Seagate/dockstar/Kconfig"
 source "board/Seagate/goflexhome/Kconfig"
diff --git a/board/LaCie/wireless_space/Kconfig 
b/board/LaCie/wireless_space/Kconfig
new file mode 100644
index 000..75a2fc5
--- /dev/null
+++ b/board/LaCie/wireless_space/Kconfig
@@ -0,0 +1,12 @@
+if TARGET_WIRELESS_SPACE
+
+config SYS_BOARD
+   default "wireless_space"
+
+config SYS_VENDOR
+   default "LaCie"
+
+config SYS_CONFIG_NAME
+   default "wireless_space"
+
+endif
diff --git a/board/LaCie/wireless_space/MAINTAINERS 
b/board/LaCie/wireless_space/MAINTAINERS
new file mode 100644
index 000..c32ecb8
--- /dev/null
+++ b/board/LaCie/wireless_space/MAINTAINERS
@@ -0,0 +1,6 @@
+WIRELESS_SPACE BOARD
+M: Albert ARIBAUD 
+S: Maintained
+F: board/LaCie/wireless_space/
+F: include/configs/wireless_space.h
+F: configs/wireless_space_defconfig
diff --git a/board/LaCie/wireless_space/Makefile 
b/board/LaCie/wireless_space/Makefile
new file mode 100644
index 000..90a84f4
--- /dev/null
+++ b/board/LaCie/wireless_space/Makefile
@@ -0,0 +1,12 @@
+#
+# Copyright (C) 2011 Simon Guinot 
+#
+# Based on Kirkwood support:
+# (C) Copyright 2009
+# Marvell Semiconductor 
+# Written-by: Prafulla Wadaskar 
+#
+# SPDX-License-Identifier: GPL-2.0+
+#
+
+obj-y  := wireless_space.o ../common/common.o
diff --git a/board/LaCie/wireless_space/kwbimage.cfg 
b/board/LaCie/wireless_space/kwbimage.cfg
new file mode 100644
index 000..037248b
--- /dev/null
+++ b/board/LaCie/wireless_space/kwbimage.cfg
@@ -0,0 +1,71 @@
+#
+# Copyright (C) 2012 Albert ARIBAUD 
+#
+# Based on netspace_v2 kwbimage.cfg:
+# Copyright (C) 2011 Simon Guinot 
+#
+# Based on Kirkwood support:
+# (C) Copyright 2009
+# Marvell Semiconductor 
+# Written-by: Prafulla Wadaskar 
+#
+# SPDX-License-Identifier: GPL-2.0+
+#
+# Refer doc/README.kwbimage for more details about how-to configure
+# and create kirkwood boot image
+#
+
+# Boot Media configurations
+BOOT_FROM  nand# Boot from NAND flash
+NAND_PAGE_SIZE 800
+
+# SOC registers configuration using bootrom header extension
+# Maximum KWBIMAGE_MAX_CONFIG configurations allowed
+
+# Values taken from image original LaCie U-Boot header dump!
+
+# Configure RGMII-0 interface pad voltage to 1.8V
+DATA 0xFFD100e0 0x1B1B1B9B
+
+#Dram initalization for SINGLE x16 CL=5 @ 400MHz
+DATA 0xFFD01400 0x43000c30 # DDR Configuration register
+
+DATA 0xFFD01404 0x37743000 # DDR Controller Control Low
+
+DATA 0xFFD01408 0x11012228 # DDR Timing (Low) (active cycles value +1)
+
+DATA 0xFFD0140C 0x0A19 #  DDR Timing (High)
+
+DATA 0xFFD01410 0x #  DDR Address Control
+
+DATA 0xFFD01414 0x #  DDR Open Pages Control
+
+DATA 0xFFD01418 0x #  DDR Operation
+
+DATA 0xFFD0141C 0x0662 #  DDR Mode
+
+DATA 0xFFD01420 0x0004 #  DDR Extended Mode
+
+DATA 0xFFD01424 0xF07F #  DDR Controller Control High
+
+DATA 0xFFD01428 0x00096630 # DDR2 ODT Read Timing (default values)
+
+DATA 0xFFD0147C 0x9663 # DDR2 ODT Write Timing (default values)
+
+DATA 0xFFD01504 0x0FF1 # CS[0]n Size

Re: [U-Boot] [PATCH v2 12/23] tegra: video: Rename CONFIG_VIDEO_TEGRA to CONFIG_VIDEO_TEGRA20

On Sat, 30 Jan 2016 16:37:51 -0700
Simon Glass s...@chromium.org wrote:

> This option refers only to the tegra20 video driver, so name it as such
> to avoid confusion with tegra124.
> 
> Also move this option to Kconfig.
> 
> Signed-off-by: Simon Glass 
> ---
> 
> Changes in v2: None
> 
>  arch/arm/mach-tegra/tegra20/Makefile| 2 +-
>  board/toradex/colibri_t20/colibri_t20.c | 2 +-
>  configs/colibri_t20_defconfig   | 1 +
>  configs/harmony_defconfig   | 1 +
>  configs/medcom-wide_defconfig   | 1 +
>  configs/paz00_defconfig | 1 +
>  configs/seaboard_defconfig  | 1 +
>  configs/tec_defconfig   | 1 +
>  configs/ventana_defconfig   | 1 +
>  drivers/video/Kconfig   | 8 
>  drivers/video/Makefile  | 2 +-
>  include/configs/colibri_t20.h   | 1 -
>  include/configs/harmony.h   | 1 -
>  include/configs/medcom-wide.h   | 1 -
>  include/configs/paz00.h | 1 -
>  include/configs/seaboard.h  | 1 -
>  include/configs/tec.h   | 1 -
>  include/configs/ventana.h   | 1 -
>  18 files changed, 18 insertions(+), 10 deletions(-)

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

Re: [U-Boot] [PATCH v2 18/23] tegra: video: Remove the static variables

On Sat, 30 Jan 2016 16:37:57 -0700
Simon Glass s...@chromium.org wrote:

> We can move the static variables into the driver-private data.
> 
> Signed-off-by: Simon Glass 
> ---
> 
> Changes in v2: None
> 
>  drivers/video/tegra.c | 31 +++
>  1 file changed, 15 insertions(+), 16 deletions(-)

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

Re: [U-Boot] [PATCH 00/14] EFI payload / application support v2

2016-01-31 Thread Alexander Graf

On 01/31/2016 04:17 PM, Simon Glass wrote:

Hi Alexander,

On 14 January 2016 at 22:06, Alexander Graf wrote:

This is my Christmas present for my openSUSE friends :).

U-Boot is a great project for embedded devices. However, convincing
everyone involved that only for "a few oddball ARM devices" we need to
support different configuration formats from grub2 when all other platforms
(PPC, System Z, x86) are standardized on a single format is a nightmare.

Well some might argue that grub2 and UEFI are their own nightmares :-)

They are, but they are the same nightmare everyone else is dreaming ;).

So we started to explore alternatives. At first, people tried to get
grub2 running using the u-boot api interface. However, FWIW that one
doesn't support relocations, so you need to know where to link grub2 to
at compile time. It also seems to be broken more often than not. And on
top of it all, it's a one-off interface, so yet another thing to maintain.

The API interface is mostly for closed-source work I think.

That led to a nifty idea. What if we can just implement the EFI application
protocol on top of U-Boot? Then we could compile a single grub2 binary for
uEFI based systems and U-Boot based systems and as soon as that one's loaded,
everything looks and feels (almost) the same.

This patch set is the result of pursuing this endeavor.

- I am successfully able to run grub2 and Linux EFI binaries with this code.
- When enabled, the resulting U-Boot binary only grows by ~10kb,
so it's very light weight.
- It works on 32bit ARM and AArch64.
- All storage devices are directly accessible
- No EFI variables
- Removable media booting (search for /efi/boot/boota{a64,arm}.efi)

Of course, there are still a few things one could do on top:

- Improve disk media detection (don't scan, use what information we have)
- Add EFI variable support using NVRAM
- Add GFX support
- Make EFI Shell work ;)
- Network device support
- Support for payload exit

But so far, I'm very happy with the state of the patches. They completely
eliminate potential arguments against U-Boot internally and give users the
chance to run with the same level of comfort on all firmware types.

I'd suggest creating a README with the above info. The cover letter
will vanish pretty fast. Perhaps also update README.efi with this new
option.

Good idea.

Another thing you could list is efi_set_watchdog_timer().

What about it exactly? That it's not supported atm?

Version 2 was successfully tested to boot grub2 and Linux from there on a
HiKey (AArch64, dcache disabled) and on a BeagleBone Black.

Do you have a UEFI image for BBB that I can put on an SD card or otherwise boot?

Phew, I hand-crafted one to play around with. You can use the hip04d01
image from

http://download.opensuse.org/repositories/devel:/ARM:/Factory:/Contrib:/HIP04D01/images/openSUSE-Tumbleweed-ARM-JeOS-hip04d01.armv7l.install.tar.xz

Just extract the tar.xz, to get to the actual image .xz file.

That image obviously has an incorrect kernel for the BBB, but everything
up to grub2 should work with it.

My plan was to slowly move all openSUSE arm targets that use our own
u-boot binaries to EFI once this patch set goes in ;).

For now I've had a play with Minnowboard, which is x86. The main thing
I noticed is that the API function implements should have EFIAPI on
them also.

Yes :). I didn't expect anyone to actually care about running this on
x86 which is the only architecture that has different calling
conventions for EFI. I'm very pleasantly surprised that you are
interested and since you already have a patch to add them, I guess you
can as well just post that once the base support is in :).

I'll make a few other comments on the patches. But overall
it seems to function and I think your implementation is nice.

Thanks :)

I was able to get grub to boot but it just says 'Welcome to GRUB!'
and then 'error: disk ',gpt4' not found'. I'm not sure what that
means.

It might mean memory corruption. I'm not sure where from though :). It
could also mean register clobbering.

U-Boot 2016.01-00860-geb4b602-dirty (Jan 31 2016 - 08:02:54 -0700)

CPU: x86_64, vendor Intel, device 30673h
DRAM: 2 GiB
efi_runtime_relocate: Relocating to offset=7ba5d000
MMC: ValleyView SDHCI: 0, ValleyView SDHCI: 1
SF: Detected W25Q64DW with page size 256 Bytes, erase size 4 KiB, total 8 MiB
*** Warning - bad CRC, using default environment

Video: 1280x1024x16
Model: Intel Minnowboard Max
SF: Detected W25Q64DW with page size 256 Bytes, erase size 4 KiB, total 8 MiB
SCSI: SATA link 0 timeout.
Target spinup took 0 ms.
AHCI 0001.0300 32 slots 2 ports 3 Gbps 0x3 impl SATA mode
flags: 64bit ncq stag pm led clo pio slum part sxs
scanning bus for devices...
Device 0: (1:0) Vendor: ATA Prod.: ADATA SP310 Rev: 5.2
Type: Hard Disk
Capacity: 30533.8 MB = 29.8 GB (62533296 x 512)
Found 1 device(s).
Net:

Re: [U-Boot] [PATCH v2 16/23] tegra: video: Merge the two config structures together

On Sat, 30 Jan 2016 16:37:55 -0700
Simon Glass s...@chromium.org wrote:

> We have a structure for the display panel and another for the controller.
> There is some overlap between them. Merge them to simplify the driver.
> 
> Signed-off-by: Simon Glass 
> ---
> 
> Changes in v2: None
> 
>  arch/arm/include/asm/arch-tegra20/display.h | 39 
>  drivers/video/tegra.c   | 72 
> +
>  2 files changed, 53 insertions(+), 58 deletions(-)

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

Re: [U-Boot] [PATCH] fdt: introudce fdt_get_child_count

2016-01-31 Thread Peng Fan

Hi Simon,

On Sun, Jan 31, 2016 at 08:37:45AM -0700, Simon Glass wrote:
>Hi Peng,
>
>On 30 January 2016 at 23:38, Peng Fan  wrote:
>> Introudce fdt_get_child_count for get the number of subnodes of
>> one parent node.
>>
>> Signed-off-by: Peng Fan 
>> Cc: Simon Glass 
>> ---
>>  include/libfdt.h | 18 ++
>>  1 file changed, 18 insertions(+)
>>
>> diff --git a/include/libfdt.h b/include/libfdt.h
>> index e48c21a..94d29e6 100644
>> --- a/include/libfdt.h
>> +++ b/include/libfdt.h
>> @@ -193,6 +193,24 @@ int fdt_next_subnode(const void *fdt, int offset);
>>  node >= 0; \
>>  node = fdt_next_subnode(fdt, node))
>>
>> +/**
>> + * fdt_get_child_count() - get the number of subnodes of one node
>> + *
>> + * @fdt:   FDT blob
>> + * @node:  Offset of node
>> + * @return number of child nodes of node, or 0 if there is no subnode.
>> + */
>> +static inline int fdt_get_child_count(void *fdt, int node)
>
>Does this need to be inline. It doesn't seem important to me.

I just follow linux kernel and add inline. If inline is not needed, I can
move the piece code to lib/fdtdec.c.

You mean this API is not important?

Thanks,
Peng

>
>> +{
>> +   int subnode;
>> +   int num = 0;
>> +
>> +   fdt_for_each_subnode(fdt, subnode, node)
>> +   num++;
>> +
>> +   return num;
>> +}
>> +
>>  /**/
>>  /* General functions  */
>>  /**/
>> --
>> 2.6.2
>>
>
>Regards,
>Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] armv8/ls1043a: Implement workaround for erratum A009660



> -Original Message-
> From: york sun
> Sent: Saturday, January 30, 2016 4:40 AM
> To: Mingkai Hu; u-boot@lists.denx.de
> Subject: Re: [PATCH] armv8/ls1043a: Implement workaround for erratum
> A009660
> 
> On 01/25/2016 10:12 PM, Mingkai Hu wrote:
> >
> >
> >> -Original Message-
> >> From: york sun
> >> Sent: Saturday, January 23, 2016 1:44 AM
> >> To: Mingkai Hu; Mingkai Hu; u-boot@lists.denx.de
> >> Subject: Re: [PATCH] armv8/ls1043a: Implement workaround for erratum
> >> A009660
> >>
> >> On 01/21/2016 11:50 PM, Mingkai Hu wrote:
> >>>
> >>>
>  -Original Message-
>  From: Mingkai Hu
>  Sent: Thursday, January 21, 2016 11:18 AM
>  To: york sun; Mingkai Hu; u-boot@lists.denx.de
>  Subject: RE: [PATCH] armv8/ls1043a: Implement workaround for
>  erratum
>  A009660
> 
> 
> 
> > -Original Message-
> > From: york sun
> > Sent: Thursday, January 21, 2016 12:21 AM
> > To: Mingkai Hu; u-boot@lists.denx.de
> > Cc: Mingkai Hu
> > Subject: Re: [PATCH] armv8/ls1043a: Implement workaround for
> > erratum
> > A009660
> >
> > On 01/19/2016 10:44 PM, Mingkai Hu wrote:
> >> From: Mingkai Hu 
> >>
> >> Memory controller performance is not optimal with default
> >> internal target queue register value, write required value for
> >> optimal DDR performance.
> >>
> >> Signed-off-by: Mingkai Hu 
> >> ---
> >>  arch/arm/cpu/armv8/fsl-layerscape/soc.c   | 13
> >> +
> >>  arch/arm/include/asm/arch-fsl-layerscape/config.h |  1 +
> >>  2 files changed, 14 insertions(+)
> >>
> >> diff --git a/arch/arm/cpu/armv8/fsl-layerscape/soc.c
> >> b/arch/arm/cpu/armv8/fsl-layerscape/soc.c
> >> index 23d6b73..485f5cd 100644
> >> --- a/arch/arm/cpu/armv8/fsl-layerscape/soc.c
> >> +++ b/arch/arm/cpu/armv8/fsl-layerscape/soc.c
> >> @@ -210,6 +210,18 @@ static void erratum_a009929(void)  #endif  }
> >>
> >> +/*
> >> + * This erratum requires setting a value to eddrtqcr1 to
> >> + * optimal the DDR performance.
> >> + */
> >> +static void erratum_a009660(void) { #ifdef
> >> +CONFIG_SYS_FSL_ERRATUM_A009660
> >> +  u32 *eddrtqcr1 = (void *)CONFIG_SYS_FSL_SCFG_ADDR + 0x20c;
> >> +  out_be32(eddrtqcr1, 0x63b20042); #endif }
> >> +
> >>  void fsl_lsch2_early_init_f(void)  {
> >>struct ccsr_cci400 *cci = (struct ccsr_cci400
> >> *)CONFIG_SYS_CCI400_ADDR; @@ -232,6 +244,7 @@ void
> >> fsl_lsch2_early_init_f(void)
> >>
> >>/* Erratum */
> >>erratum_a009929();
> >> +  erratum_a009660();
> >>  }
> >>  #endif
> >>
> >> diff --git a/arch/arm/include/asm/arch-fsl-layerscape/config.h
> >> b/arch/arm/include/asm/arch-fsl-layerscape/config.h
> >> index 49b113d..66399b2 100644
> >> --- a/arch/arm/include/asm/arch-fsl-layerscape/config.h
> >> +++ b/arch/arm/include/asm/arch-fsl-layerscape/config.h
> >> @@ -167,6 +167,7 @@
> >>  #define GICC_BASE 0x01402000
> >>
> >>  #define CONFIG_SYS_FSL_ERRATUM_A009929
> >> +#define CONFIG_SYS_FSL_ERRATUM_A009660
> >>  #else
> >>  #error SoC not defined
> >>  #endif
> >>
> >
> > NACK.
> >
> > Erratum A009660 is cancelled. The workaround is integrated into
> >> A008514.
> > Please revise workaround for A008514. Besides, you are using ARMv7
> > offset for ARMv8.
> > Please check if this workaround applies to LS2 SoCs. While you are
> > on it, please add a comment to LS1 workaround with the word
> > A008514 so we can grep it.
> >
> 
>  Hi York,
> 
>  I discussed with design team and got the erratum in LS1043A CE
>  before preparing the patch. The value (0x63b20042) is the same one
>  described in
>  A008514 of LS1021A and the register offset of LS1043A is same with
>  the one used for LS1021A which is in the SCFG space.
> 
>  I will double check with the design team about if the erraum number
>  is still A009660 and keep you in the loop.
> 
>  For LS2, I got the A008514 in the LS2085A CE, but 1. The value is
>  63b2_0002 which is different from the value used on ls1043 platform.
>  2. The address is in DCFG space.
>  3. I did not get the A008514 in the LS2080A CE (RevD).
> 
>  So it's better to confirm with design team about the value used and
>  the consistence between LS2085A and LS2080A.
> 
> >>> York,
> >>>
> >>> As discussed with design team, the erratum A009660 is the correct
> >>> one
> >> and the value/offset is also correct.
> >>>
> >>> Do you prefer to submit a new patch to fix ls2085 or fix it on this
> >> patch?
> >>>
> >>
> >> Mingkai,
> >>
> >> If the erratum A009660 is republished, and A008514 is updated for
> >> LS2085/LS1021, you can keep this patch, but adding a check to

Re: [U-Boot] 64-bit x86 U-Boot?

Hello Simon,

On Sun, 31 Jan 2016 19:20:31 -0700, Simon Glass 
wrote:
> Hi Bin,
> 
> At present U-Boot supports booting a 64-bit kernel directly. It can
> also be loaded as a 64-bit payload from EFI. But it cannot be built as
> a 64-bit boot loader.
> 
> I took a bit of a look at this. It looks like we need to stay in
> 32-bit mode until the FSP is loaded. Also, to get to 64-bit mode I'm
> pretty sure we need page tables, which means we need somewhere to put
> them!
> 
> Looking at the board_f init sequence, it seems that arch_cpu_init() is
> the earlist we could switch to 64-bit. We'd need to grab some memory
> from somewhere to do this - I wonder if this can be CAR? There is no
> SRAM on x86 chips I think.
> 
> For non-FSP devices we don't init the RAM until much later -
> dram_init(). That means that a significant portion of the init
> sequence would be 32-bit code. I'm not sure that will work.
> 
> I suppose one option is to only go to 64-bit mode when relocating. But
> then we end up with lots of code that needs to run in 32-bit and
> 64-bit.
> 
> Do you have any ideas on this?

How about starting with implementing the last option, i.e. switch to 64
bits when DDR is available, mainline that, then progressively work your
way toward an earlier switch?

> Regards,
> Simon

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

Re: [U-Boot] [PATCH v4 2/4] QE: add QE support on ls1043ardb



> -Original Message-
> From: Zhao Qiang [mailto:qiang.z...@nxp.com]
> Sent: Friday, January 29, 2016 12:28 PM
> To: Mingkai Hu
> Cc: tr...@konsulko.com; york sun; u-boot@lists.denx.de; Qiang Zhao
> Subject: [PATCH v4 2/4] QE: add QE support on ls1043ardb
> 
> Upload qe microcode on ls1043ardb
> 
> Signed-off-by: Zhao Qiang 
> ---
>  board/freescale/ls1043ardb/ls1043ardb.c | 8 
>  drivers/qe/qe.c | 6 ++
>  include/configs/ls1043ardb.h| 7 +++
>  3 files changed, 17 insertions(+), 4 deletions(-)
> 
> diff --git a/board/freescale/ls1043ardb/ls1043ardb.c
> b/board/freescale/ls1043ardb/ls1043ardb.c
> index c8f723a..834fdff 100644
> --- a/board/freescale/ls1043ardb/ls1043ardb.c
> +++ b/board/freescale/ls1043ardb/ls1043ardb.c
> @@ -21,6 +21,10 @@
>  #include 
>  #include 
>  #include "cpld.h"
> +#ifdef CONFIG_U_QE
> +#include "../../../drivers/qe/qe.h"
> +#endif
> +
> 

The patch submitted by Qianyu is trying to move qe.h to include/fsl_qe.h,
It's better to based on this patch set to avoid the long include statement.
http://patchwork.ozlabs.org/patch/572697/

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

[U-Boot] [PATCH v2] armv8/ls1043a: Implement workaround for erratum A009660

From: Mingkai Hu 

Memory controller performance is not optimal with default internal
target queue register value, write required value for optimal DDR
performance.

Signed-off-by: Mingkai Hu 
---
v2: 
 - Add a check to make sure A009660 and A008514 is are not both enabled.
 - Add comment for the offset of eddrtqcr1.

 arch/arm/cpu/armv8/fsl-layerscape/soc.c   | 15 +++
 arch/arm/include/asm/arch-fsl-layerscape/config.h |  5 +
 2 files changed, 20 insertions(+)

diff --git a/arch/arm/cpu/armv8/fsl-layerscape/soc.c 
b/arch/arm/cpu/armv8/fsl-layerscape/soc.c
index 7ff0148..8b01fd2 100644
--- a/arch/arm/cpu/armv8/fsl-layerscape/soc.c
+++ b/arch/arm/cpu/armv8/fsl-layerscape/soc.c
@@ -213,6 +213,20 @@ static void erratum_a009929(void)
 #endif
 }
 
+/*
+ * This erratum requires setting a value to eddrtqcr1 to optimal
+ * the DDR performance. The eddrtqcr1 register is in SCFG space
+ * of LS1043A and the offset is 0x157_020c.
+ */
+static void erratum_a009660(void)
+{
+#if defined(CONFIG_SYS_FSL_ERRATUM_A009660) \
+   && !defined(CONFIG_SYS_FSL_ERRATUM_A008514)
+   u32 *eddrtqcr1 = (void *)CONFIG_SYS_FSL_SCFG_ADDR + 0x20c;
+   out_be32(eddrtqcr1, 0x63b20042);
+#endif
+}
+
 void fsl_lsch2_early_init_f(void)
 {
struct ccsr_cci400 *cci = (struct ccsr_cci400 *)CONFIG_SYS_CCI400_ADDR;
@@ -238,6 +252,7 @@ void fsl_lsch2_early_init_f(void)
 
/* Erratum */
erratum_a009929();
+   erratum_a009660();
 }
 #endif
 
diff --git a/arch/arm/include/asm/arch-fsl-layerscape/config.h 
b/arch/arm/include/asm/arch-fsl-layerscape/config.h
index f1b164f..854ed0b 100644
--- a/arch/arm/include/asm/arch-fsl-layerscape/config.h
+++ b/arch/arm/include/asm/arch-fsl-layerscape/config.h
@@ -171,6 +171,11 @@
 
 #define CONFIG_SYS_FSL_ERRATUM_A009663
 #define CONFIG_SYS_FSL_ERRATUM_A009929
+#define CONFIG_SYS_FSL_ERRATUM_A009660
+#if defined(CONFIG_SYS_FSL_ERRATUM_A009660) \
+   && defined(CONFIG_SYS_FSL_ERRATUM_A008514)
+#error "A009660 and A008514 are both enabed for LS1043A, please check it."
+#endif
 #else
 #error SoC not defined
 #endif
-- 
2.1.0.27.g96db324

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

Re: [U-Boot] [PATCH v3 0/7] fdt: Replace u-boot-dtb.bin with u-boot.bin

Hello Simon,

Nitpicking:

On Sun, 31 Jan 2016 18:10:48 -0700, Simon Glass 
wrote:

> The original decision to use a separate u-boot-dtb.bin was aimed at allowing
> any device tree file to be concatenated to the u-boot.bin image after the
> build. However this no-longer seems so important. More important is the
> convenience of using the same output file regardless of the setting for
> OF_CONTROL.

Maybe explain in a few words why concatenating the DT no long seems
important, so that readers who were intending to concatenate get a hint
on what option they should follow instead?

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

Re: [U-Boot] [PATCH v4 4/4] qe: assgin pins to qe-hdlc



> -Original Message-
> From: Zhao Qiang [mailto:qiang.z...@nxp.com]
> Sent: Friday, January 29, 2016 12:28 PM
> To: Mingkai Hu
> Cc: tr...@konsulko.com; york sun; u-boot@lists.denx.de; Qiang Zhao
> Subject: [PATCH v4 4/4] qe: assgin pins to qe-hdlc
> 
> qe-hdlc and usb multi-use the pins, when set hwconfig=qe-hdlc, assign the
> pins to qe-hdlc, if not, assgin it to usb
> 
> Signed-off-by: Zhao Qiang 
> ---
>  board/freescale/ls1043ardb/ls1043ardb.c | 53 ++-
> --
>  1 file changed, 43 insertions(+), 10 deletions(-)
> 
> diff --git a/board/freescale/ls1043ardb/ls1043ardb.c
> b/board/freescale/ls1043ardb/ls1043ardb.c
> index c2d4887..7ba72f8 100644
> --- a/board/freescale/ls1043ardb/ls1043ardb.c
> +++ b/board/freescale/ls1043ardb/ls1043ardb.c
> @@ -114,18 +114,24 @@ int config_board_mux(void)
>   struct ccsr_scfg *scfg = (struct ccsr_scfg
> *)CONFIG_SYS_FSL_SCFG_ADDR;
>   u32 usb_pwrfault;
> 
> + if (hwconfig("qe-hdlc")) {
> + out_be32(>rcwpmuxcr0,
> +  (in_be32(>rcwpmuxcr0) & ~0xff00) | 0x6600);
> + printf("Assign to qe-hdlc clk, rcwpmuxcr0=%x\n",
> +in_be32(>rcwpmuxcr0));
> + } else {
>  #ifdef CONFIG_HAS_FSL_XHCI_USB
> - out_be32(>rcwpmuxcr0, 0x);
> - out_be32(>usbdrvvbus_selcr, SCFG_USBDRVVBUS_SELCR_USB1);
> - usb_pwrfault = (SCFG_USBPWRFAULT_DEDICATED <<
> - SCFG_USBPWRFAULT_USB3_SHIFT) |
> - (SCFG_USBPWRFAULT_DEDICATED <<
> - SCFG_USBPWRFAULT_USB2_SHIFT) |
> - (SCFG_USBPWRFAULT_SHARED <<
> -  SCFG_USBPWRFAULT_USB1_SHIFT);
> - out_be32(>usbpwrfault_selcr, usb_pwrfault);
> + out_be32(>rcwpmuxcr0, 0x);
> + out_be32(>usbdrvvbus_selcr, SCFG_USBDRVVBUS_SELCR_USB1);
> + usb_pwrfault = (SCFG_USBPWRFAULT_DEDICATED <<
> + SCFG_USBPWRFAULT_USB3_SHIFT) |
> + (SCFG_USBPWRFAULT_DEDICATED <<
> + SCFG_USBPWRFAULT_USB2_SHIFT) |
> + (SCFG_USBPWRFAULT_SHARED <<
> +  SCFG_USBPWRFAULT_USB1_SHIFT);
> + out_be32(>usbpwrfault_selcr, usb_pwrfault);
>  #endif
> -
> + }
>   return 0;
>  }
> 
> @@ -152,6 +158,16 @@ int misc_init_r(void)  }  #endif
> 
> +void fdt_del_qe(void *blob)
> +{
> + int nodeoff = 0;
> +
> + while ((nodeoff = fdt_node_offset_by_compatible(blob, 0,
> + "fsl,qe")) >= 0) {
> + fdt_del_node(blob, nodeoff);
> + }
> +}
> +
>  int ft_board_setup(void *blob, bd_t *bd)  {
>   u64 base[CONFIG_NR_DRAM_BANKS];
> @@ -169,6 +185,23 @@ int ft_board_setup(void *blob, bd_t *bd)  #ifdef
> CONFIG_SYS_DPAA_FMAN
>   fdt_fixup_fman_ethernet(blob);
>  #endif
> +
> + /*
> +  * qe-hdlc and usb multi-use the pins,
> +  * when set hwconfig to qe-hdlc, delete usb node.
> +  */
> + if (hwconfig("qe-hdlc"))
> +#ifdef CONFIG_HAS_FSL_XHCI_USB
> + fdt_del_node_and_alias(blob, "usb1"); #endif
> + /*
> +  * qe just support qe-uart and qe-hdlc,
> +  * if qe-uart and qe-hdlc are not set in hwconfig,
> +  * delete qe node.
> +  */
> + if (!hwconfig("qe-uart") && !hwconfig("qe-hdlc"))
> + fdt_del_qe(blob);
> +
>   return 0;
>  }
> 
> --
> 2.1.0.27.g96db324

Reviewed-by: Mingkai Hu 

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

Re: [U-Boot] [PATCH v4 3/4] ls1043rdb: move USB mux config to config_board_mux



> -Original Message-
> From: Zhao Qiang [mailto:qiang.z...@nxp.com]
> Sent: Friday, January 29, 2016 12:28 PM
> To: Mingkai Hu
> Cc: tr...@konsulko.com; york sun; u-boot@lists.denx.de; Qiang Zhao
> Subject: [PATCH v4 3/4] ls1043rdb: move USB mux config to
> config_board_mux
> 
> USB pins are muxed with other feature, move USB mux config to
> config_board_mux.
> 
> Signed-off-by: Zhao Qiang 
> ---
>  board/freescale/ls1043ardb/ls1043ardb.c | 30 +++
> ---
>  1 file changed, 15 insertions(+), 15 deletions(-)
> 
> diff --git a/board/freescale/ls1043ardb/ls1043ardb.c
> b/board/freescale/ls1043ardb/ls1043ardb.c
> index 834fdff..c2d4887 100644
> --- a/board/freescale/ls1043ardb/ls1043ardb.c
> +++ b/board/freescale/ls1043ardb/ls1043ardb.c
> @@ -75,23 +75,8 @@ int dram_init(void)
> 
>  int board_early_init_f(void)
>  {
> - struct ccsr_scfg *scfg = (struct ccsr_scfg
> *)CONFIG_SYS_FSL_SCFG_ADDR;
> - u32 usb_pwrfault;
> -
>   fsl_lsch2_early_init_f();
> 
> -#ifdef CONFIG_HAS_FSL_XHCI_USB
> - out_be32(>rcwpmuxcr0, 0x);
> - out_be32(>usbdrvvbus_selcr, SCFG_USBDRVVBUS_SELCR_USB1);
> - usb_pwrfault = (SCFG_USBPWRFAULT_DEDICATED <<
> - SCFG_USBPWRFAULT_USB3_SHIFT) |
> - (SCFG_USBPWRFAULT_DEDICATED <<
> - SCFG_USBPWRFAULT_USB2_SHIFT) |
> - (SCFG_USBPWRFAULT_SHARED <<
> -  SCFG_USBPWRFAULT_USB1_SHIFT);
> - out_be32(>usbpwrfault_selcr, usb_pwrfault);
> -#endif
> -
>   return 0;
>  }
> 
> @@ -126,6 +111,21 @@ int board_init(void)
> 
>  int config_board_mux(void)
>  {
> + struct ccsr_scfg *scfg = (struct ccsr_scfg
> *)CONFIG_SYS_FSL_SCFG_ADDR;
> + u32 usb_pwrfault;
> +
> +#ifdef CONFIG_HAS_FSL_XHCI_USB
> + out_be32(>rcwpmuxcr0, 0x);
> + out_be32(>usbdrvvbus_selcr, SCFG_USBDRVVBUS_SELCR_USB1);
> + usb_pwrfault = (SCFG_USBPWRFAULT_DEDICATED <<
> + SCFG_USBPWRFAULT_USB3_SHIFT) |
> + (SCFG_USBPWRFAULT_DEDICATED <<
> + SCFG_USBPWRFAULT_USB2_SHIFT) |
> + (SCFG_USBPWRFAULT_SHARED <<
> +  SCFG_USBPWRFAULT_USB1_SHIFT);
> + out_be32(>usbpwrfault_selcr, usb_pwrfault); #endif
> +
>   return 0;
>  }
> 
> --
> 2.1.0.27.g96db324

Reviewed-by: Mingkai Hu 

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

[U-Boot] [PATCH 1/2] pinctrl: imx: Introduce pinctrl driver for i.MX6

2016-01-31 Thread Peng Fan

Introduce pinctrl for i.MX6
1. pinctrl-imx.c is for common usage. It's used by i.MX6/7.
2. Add PINCTRL_IMX PINCTRL_IMX6 Kconfig entry.
3. To the pinctrl_ops implementation, only set_state is implemented.
   To i.MX6/7, the pinctrl dts entry is as following:
 {
pinctrl-names = "default";

pinctrl_csi1: csi1grp {
fsl,pins = <
MX6UL_PAD_CSI_MCLK__CSI_MCLK0x1b088
MX6UL_PAD_CSI_PIXCLK__CSI_PIXCLK0x1b088
MX6UL_PAD_CSI_VSYNC__CSI_VSYNC  0x1b088
>;
};

[.]
};
  there is no property named function or groups. So pinctrl_generic_set_state
  can not be used here.
5. This driver is a simple implementation for i.mx iomux controller,
   only parse the fsl,pins property and write value to registers.
6. With DEBUG enabled, we can see log when "i2c bus 0":
   "
   set_state_simple op missing
   imx_pinctrl_set_state: i2c1grp
   mux_reg 0x14c, conf_reg 0x3bc, input_reg 0x5d8, mux_mode 0x0, input_val 0x1, 
config_val 0x407f
   write mux: offset 0x14c val 0x10
   select_input: offset 0x5d8 val 0x1
   write config: offset 0x3bc val 0x7f
   mux_reg 0x148, conf_reg 0x3b8, input_reg 0x5d4, mux_mode 0x0, input_val 0x1, 
config_val 0x407f
   write mux: offset 0x148 val 0x10
   select_input: offset 0x5d4 val 0x1
   write config: offset 0x3b8 val 0x7f
   "
   this means imx6 pinctrl driver works as expected.

Signed-off-by: Peng Fan 
Cc: Simon Glass 
Cc: Stefano Babic 
Cc: Fabio Estevam 
---
 drivers/pinctrl/Kconfig|   1 +
 drivers/pinctrl/Makefile   |   1 +
 drivers/pinctrl/nxp/Kconfig|  10 ++
 drivers/pinctrl/nxp/Makefile   |   2 +
 drivers/pinctrl/nxp/pinctrl-imx.c  | 224 +
 drivers/pinctrl/nxp/pinctrl-imx.h  |  59 ++
 drivers/pinctrl/nxp/pinctrl-imx6.c |  41 +++
 7 files changed, 338 insertions(+)
 create mode 100644 drivers/pinctrl/nxp/Kconfig
 create mode 100644 drivers/pinctrl/nxp/Makefile
 create mode 100644 drivers/pinctrl/nxp/pinctrl-imx.c
 create mode 100644 drivers/pinctrl/nxp/pinctrl-imx.h
 create mode 100644 drivers/pinctrl/nxp/pinctrl-imx6.c

diff --git a/drivers/pinctrl/Kconfig b/drivers/pinctrl/Kconfig
index 57e6142..9fd8f62 100644
--- a/drivers/pinctrl/Kconfig
+++ b/drivers/pinctrl/Kconfig
@@ -133,6 +133,7 @@ config PINCTRL_SANDBOX
 
 endif
 
+source "drivers/pinctrl/nxp/Kconfig"
 source "drivers/pinctrl/uniphier/Kconfig"
 
 endmenu
diff --git a/drivers/pinctrl/Makefile b/drivers/pinctrl/Makefile
index 70d25dc..dcd20bf 100644
--- a/drivers/pinctrl/Makefile
+++ b/drivers/pinctrl/Makefile
@@ -5,6 +5,7 @@
 obj-y  += pinctrl-uclass.o
 obj-$(CONFIG_$(SPL_)PINCTRL_GENERIC)   += pinctrl-generic.o
 
+obj-y  += nxp/
 obj-$(CONFIG_ARCH_ROCKCHIP) += rockchip/
 obj-$(CONFIG_PINCTRL_SANDBOX)  += pinctrl-sandbox.o
 
diff --git a/drivers/pinctrl/nxp/Kconfig b/drivers/pinctrl/nxp/Kconfig
new file mode 100644
index 000..2032588
--- /dev/null
+++ b/drivers/pinctrl/nxp/Kconfig
@@ -0,0 +1,10 @@
+config PINCTRL_IMX
+   bool
+
+config PINCTRL_IMX6
+   bool "IMX6 pinctrl driver"
+   depends on ARCH_MX6
+   select DEVRES
+   select PINCTRL_IMX
+   help
+ Say Y here to enable the imx6ul pinctrl driver
diff --git a/drivers/pinctrl/nxp/Makefile b/drivers/pinctrl/nxp/Makefile
new file mode 100644
index 000..7fd9850
--- /dev/null
+++ b/drivers/pinctrl/nxp/Makefile
@@ -0,0 +1,2 @@
+obj-$(CONFIG_PINCTRL_IMX)  += pinctrl-imx.o
+obj-$(CONFIG_PINCTRL_IMX6) += pinctrl-imx6.o
diff --git a/drivers/pinctrl/nxp/pinctrl-imx.c 
b/drivers/pinctrl/nxp/pinctrl-imx.c
new file mode 100644
index 000..09ffb1b
--- /dev/null
+++ b/drivers/pinctrl/nxp/pinctrl-imx.c
@@ -0,0 +1,224 @@
+/*
+ * Copyright (C) 2016 Peng Fan 
+ *
+ * SPDX-License-Identifier:GPL-2.0+
+ */
+
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+
+#include "pinctrl-imx.h"
+
+DECLARE_GLOBAL_DATA_PTR;
+
+static int imx_pinctrl_set_state(struct udevice *dev, struct udevice *config)
+{
+   struct imx_pinctrl_priv *priv = dev_get_priv(dev);
+   struct imx_pinctrl_soc_info *info = priv->info;
+   int node = config->of_offset;
+   const struct fdt_property *prop;
+   unsigned int *pin_data;
+   int npins, size, pin_size;
+   int mux_reg, conf_reg, input_reg, input_val, mux_mode, config_val;
+   int i, j = 0;
+
+   dev_dbg(dev, "%s: %s\n", __func__, config->name);
+
+   if (info->flags & SHARE_MUX_CONF_REG)
+   pin_size = SHARE_FSL_PIN_SIZE;
+   else
+   pin_size = FSL_PIN_SIZE;
+
+   prop = fdt_get_property(gd->fdt_blob, node, "fsl,pins", );
+   if (!prop) {
+   dev_err(dev, "No fsl,pins property in node %s\n", config->name);
+

[U-Boot] [PATCH 2/2] pinctrl: imx: Support i.MX7D

2016-01-31 Thread Peng Fan

Introudce i.MX7 pinctrl driver support.
For now only i.MX7D supported.
There are two iomux controllers in i.MX7D, iomuxc and iomuxc_lpsr.
To iomuxc_lpsr, ZERO_OFFSET_VALID is set, means offset of mux_reg
and conf_reg can begin at 0.

Signed-off-by: Peng Fan 
Cc: Simon Glass 
Cc: Stefano Babic 
Cc: Fabio Estevam 
---
 drivers/pinctrl/nxp/Kconfig|  8 
 drivers/pinctrl/nxp/Makefile   |  1 +
 drivers/pinctrl/nxp/pinctrl-imx7.c | 42 ++
 3 files changed, 51 insertions(+)
 create mode 100644 drivers/pinctrl/nxp/pinctrl-imx7.c

diff --git a/drivers/pinctrl/nxp/Kconfig b/drivers/pinctrl/nxp/Kconfig
index 2032588..a251d54 100644
--- a/drivers/pinctrl/nxp/Kconfig
+++ b/drivers/pinctrl/nxp/Kconfig
@@ -8,3 +8,11 @@ config PINCTRL_IMX6
select PINCTRL_IMX
help
  Say Y here to enable the imx6ul pinctrl driver
+
+config PINCTRL_IMX7
+   bool "IMX7 pinctrl driver"
+   depends on ARCH_MX7
+   select DEVRES
+   select PINCTRL_IMX
+   help
+ Say Y here to enable the imx6ul pinctrl driver
diff --git a/drivers/pinctrl/nxp/Makefile b/drivers/pinctrl/nxp/Makefile
index 7fd9850..0ee7f2f 100644
--- a/drivers/pinctrl/nxp/Makefile
+++ b/drivers/pinctrl/nxp/Makefile
@@ -1,2 +1,3 @@
 obj-$(CONFIG_PINCTRL_IMX)  += pinctrl-imx.o
 obj-$(CONFIG_PINCTRL_IMX6) += pinctrl-imx6.o
+obj-$(CONFIG_PINCTRL_IMX7) += pinctrl-imx7.o
diff --git a/drivers/pinctrl/nxp/pinctrl-imx7.c 
b/drivers/pinctrl/nxp/pinctrl-imx7.c
new file mode 100644
index 000..150ccf9
--- /dev/null
+++ b/drivers/pinctrl/nxp/pinctrl-imx7.c
@@ -0,0 +1,42 @@
+
+/*
+ * Copyright (C) 2016 Peng Fan 
+ *
+ * SPDX-License-Identifier:GPL-2.0+
+ */
+
+#include 
+#include 
+
+#include "pinctrl-imx.h"
+
+static struct imx_pinctrl_soc_info imx7_pinctrl_soc_info;
+
+static struct imx_pinctrl_soc_info imx7_lpsr_pinctrl_soc_info = {
+   .flags = ZERO_OFFSET_VALID,
+};
+
+static int imx7_pinctrl_probe(struct udevice *dev)
+{
+   struct imx_pinctrl_soc_info *info =
+   (struct imx_pinctrl_soc_info *)dev_get_driver_data(dev);
+
+   return imx_pinctrl_probe(dev, info);
+}
+
+static const struct udevice_id imx7_pinctrl_match[] = {
+   { .compatible = "fsl,imx7d-iomuxc", .data = 
(ulong)_pinctrl_soc_info },
+   { .compatible = "fsl,imx7d-iomuxc-lpsr", .data = 
(ulong)_lpsr_pinctrl_soc_info },
+   { /* sentinel */ }
+};
+
+U_BOOT_DRIVER(imx7_pinctrl) = {
+   .name = "imx7-pinctrl",
+   .id = UCLASS_PINCTRL,
+   .of_match = of_match_ptr(imx7_pinctrl_match),
+   .probe = imx7_pinctrl_probe,
+   .remove = imx_pinctrl_remove,
+   .priv_auto_alloc_size = sizeof(struct imx_pinctrl_priv),
+   .ops = _pinctrl_ops,
+   .flags = DM_FLAG_PRE_RELOC,
+};
-- 
2.6.2

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

Re: [U-Boot] [PATCH] OMAP3: omap3_logic: Remove YAFFS support.