[regression] 2023.01 breaks u-boot-initial-env for aarch64

2023-01-06 Thread Otavio Salvador
Hello,

When using v2023.01-rc4-49-g582e3c9fb23 for aarch64
(technically imx8mm-lpddr4-evk) I get following error:

| /bin/false)
|   cat u-boot-nodtb.bin dts/dt.dtb > u-boot-dtb.bin
|   cp u-boot-dtb.bin u-boot.bin
|   aarch64-oel-linux-objcopy --dump-section
.rodata.default_environment=u-boot-initial-env env/common.o; sed --in-place
-e 's/\x00/\x0A/g' u-boot-initial-env; sed --in-place -e '/^\s*$/d'
u-boot-initial-env; sort --field-separator== -k1,1 --stable
u-boot-initial-env -o u-boot-initial-env
| aarch64-oel-linux-objcopy: env/common.o: can't dump section
'.rodata.default_environment' - it does not exist: file format not
recognized
| sed: can't read u-boot-initial-env: No such file or directory

The very same works just fine with 2022.10 it works. I didn't investigate
more yet.

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854  Mobile: +1 (347) 903-9750


Re: [regression] 2023.01 breaks u-boot-initial-env for aarch64

2023-01-07 Thread Otavio Salvador
Hello Sean,

Em sex., 6 de jan. de 2023 às 22:59, Sean Anderson 
escreveu:

> On 1/6/23 19:44, Otavio Salvador wrote:
> > The very same works just fine with 2022.10 it works. I didn't investigate
> > more yet.
> >
>
> Do you have LTO enabled?
>

I didn't enable it explicitly.


> Does 486aef08de ("u-boot-initial-env: rework make target") fix things?
>

Yes, this fixed it. @Tom Rini  please add this to
v2023.01 release.

Thanks for the quick response.

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854  Mobile: +1 (347) 903-9750


Re: [regression] 2023.01 breaks u-boot-initial-env for aarch64

2023-01-07 Thread Otavio Salvador
Thanks!

Em sáb., 7 de jan. de 2023 às 10:44, Fabio Estevam 
escreveu:

> Hi Otavio,
>
> On Sat, Jan 7, 2023 at 10:22 AM Otavio Salvador
>  wrote:
>
> > I didn't enable it explicitly.
>
> I enable LTO in this commit:
>
> https://source.denx.de/u-boot/u-boot/-/commit/750d7ddf2c514949228d991e6dd4f7982bfb27f6
>
> > > Does 486aef08de ("u-boot-initial-env: rework make target") fix things?
> > >
> >
> > Yes, this fixed it. @Tom Rini  please add this to
> > v2023.01 release.
>
> Adding the correct Tom's email.
>


-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854  Mobile: +1 (347) 903-9750


Re: [PATCH v3] rockchip: rv1108: Convert to OF_UPSTREAM

2024-04-24 Thread Otavio Salvador
Reviewed-by: Otavio Salvador 
Tested-by: Otavio Salvador 

Em qua., 24 de abr. de 2024 às 11:18, Fabio Estevam 
escreveu:

> Instead of using the local rv1108 devicetree copies from U-Boot,
> let's convert the rv1108 boards to OF_UPSTREAM so that the upstream kernel
> devicetrees can be used instead.
>
> Tested on a rv1108-elgin-r1 board.
>
> Signed-off-by: Fabio Estevam 
> ---
> Changes since v2:
> - Also removed the arch/arm/dts/Makefile entries.
>
>  arch/arm/dts/Makefile|   4 -
>  arch/arm/dts/rv1108-elgin-r1.dts |  59 
>  arch/arm/dts/rv1108-evb.dts  |  79 -
>  arch/arm/dts/rv1108.dtsi | 581 ---
>  arch/arm/mach-rockchip/Kconfig   |   1 +
>  configs/elgin-rv1108_defconfig   |   2 +-
>  configs/evb-rv1108_defconfig |   2 +-
>  7 files changed, 3 insertions(+), 725 deletions(-)
>  delete mode 100644 arch/arm/dts/rv1108-elgin-r1.dts
>  delete mode 100644 arch/arm/dts/rv1108-evb.dts
>  delete mode 100644 arch/arm/dts/rv1108.dtsi
>
> diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
> index b1c9c6222e5d..e600f9c6ed25 100644
> --- a/arch/arm/dts/Makefile
> +++ b/arch/arm/dts/Makefile
> @@ -182,10 +182,6 @@ dtb-$(CONFIG_ROCKCHIP_RK3588) += \
> rk3588-rock-5b.dtb \
> rk3588-turing-rk1.dtb
>
> -dtb-$(CONFIG_ROCKCHIP_RV1108) += \
> -   rv1108-elgin-r1.dtb \
> -   rv1108-evb.dtb
> -
>  dtb-$(CONFIG_ROCKCHIP_RV1126) += \
> rv1126-edgeble-neu2-io.dtb
>
> diff --git a/arch/arm/dts/rv1108-elgin-r1.dts
> b/arch/arm/dts/rv1108-elgin-r1.dts
> deleted file mode 100644
> index 83e8b3183847..
> --- a/arch/arm/dts/rv1108-elgin-r1.dts
> +++ /dev/null
> @@ -1,59 +0,0 @@
> -// SPDX-License-Identifier: GPL-2.0+
> -/*
> - * (C) Copyright 2016 Rockchip Electronics Co., Ltd
> - */
> -
> -/dts-v1/;
> -
> -#include "rv1108.dtsi"
> -
> -/ {
> -   model = "Elgin RV1108 R1 board";
> -   compatible = "elgin,rv1108-elgin", "rockchip,rv1108";
> -
> -   memory@6000 {
> -   device_type = "memory";
> -   reg = <0x6000 0x0800>;
> -   };
> -
> -   chosen {
> -   stdout-path = "serial2:150n8";
> -   };
> -};
> -
> -&emmc {
> -   pinctrl-names = "default";
> -   pinctrl-0 = <&emmc_clk &emmc_cmd &emmc_bus8>;
> -   bus-width = <8>;
> -   cap-mmc-highspeed;
> -   disable-wp;
> -   non-removable;
> -   status = "okay";
> -};
> -
> -&u2phy {
> -   status = "okay";
> -
> -   u2phy_otg: otg-port {
> -   status = "okay";
> -   };
> -};
> -
> -&uart2 {
> -   pinctrl-names = "default";
> -   pinctrl-0 = <&uart2m0_xfer_pullup>;
> -   status = "okay";
> -};
> -
> -&usb20_otg {
> -   status = "okay";
> -};
> -
> -&pinctrl {
> -   uart2m0 {
> -   uart2m0_xfer_pullup: uart2m0-xfer-pullup {
> -   rockchip,pins = <2 RK_PD2 RK_FUNC_1
> &pcfg_pull_up_drv_8ma>,
> -   <2 RK_PD1 RK_FUNC_1
> &pcfg_pull_up_drv_8ma>;
> -   };
> -   };
> -};
> diff --git a/arch/arm/dts/rv1108-evb.dts b/arch/arm/dts/rv1108-evb.dts
> deleted file mode 100644
> index c91776bc106e..
> --- a/arch/arm/dts/rv1108-evb.dts
> +++ /dev/null
> @@ -1,79 +0,0 @@
> -// SPDX-License-Identifier: GPL-2.0+
> -/*
> - * (C) Copyright 2016 Rockchip Electronics Co., Ltd
> - */
> -
> -/dts-v1/;
> -
> -#include "rv1108.dtsi"
> -
> -/ {
> -   model = "Rockchip RV1108 Evaluation board";
> -   compatible = "rockchip,rv1108-evb", "rockchip,rv1108";
> -
> -   memory@6000 {
> -   device_type = "memory";
> -   reg = <0x6000 0x0800>;
> -   };
> -
> -   chosen {
> -   stdout-path = "serial2:150n8";
> -   };
> -
> -   vcc5v0_otg: vcc5v0-otg-drv {
> -   compatible = "regulator-fixed";
> -   enable-active-high;
> -   regulator-name = "vcc5v0_otg";
> -   gpio = <&gpio0 RK_PB0 GPIO_ACTIVE_HIGH>;
> -   regulator-min-microvolt = <500>;
> -   regulator-max-microvolt = <500>;
> -   };
> -};
> -
> -&gmac {
> -   status = "okay";
> -   clock_

[PATCH] spl: Avoid printing boot device if silent console is enabled

2020-09-03 Thread Otavio Salvador
Signed-off-by: Otavio Salvador 
---

 common/spl/spl.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/common/spl/spl.c b/common/spl/spl.c
index 4840d1d367..63c48fbf33 100644
--- a/common/spl/spl.c
+++ b/common/spl/spl.c
@@ -552,7 +552,9 @@ static int boot_from_devices(struct spl_image_info 
*spl_image,
struct spl_image_loader *loader;
 
loader = spl_ll_find_loader(spl_boot_list[i]);
-#if defined(CONFIG_SPL_SERIAL_SUPPORT) && defined(CONFIG_SPL_LIBCOMMON_SUPPORT)
+#if defined(CONFIG_SPL_SERIAL_SUPPORT) \
+&& defined(CONFIG_SPL_LIBCOMMON_SUPPORT)\
+&& !defined(CONFIG_SILENT_CONSOLE)
if (loader)
printf("Trying to boot from %s\n", loader->name);
else
-- 
2.28.0



[PATCH] spl: Add SPL_SERIAL as requirement for SDP_USB_SDP

2020-09-05 Thread Otavio Salvador
Signed-off-by: Otavio Salvador 
---

 common/spl/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/common/spl/Kconfig b/common/spl/Kconfig
index af8255a8d6..20c5c4af3c 100644
--- a/common/spl/Kconfig
+++ b/common/spl/Kconfig
@@ -1237,6 +1237,7 @@ endchoice
 
 config SPL_USB_SDP_SUPPORT
bool "Support SDP (Serial Download Protocol)"
+   depends on SPL_SERIAL_SUPPORT
help
  Enable Serial Download Protocol (SDP) device support in SPL. This
  allows to download images into memory and execute (jump to) them
-- 
2.28.0



Re: [PATCH 1/4] ARM: dts: Activate pullups in the console pins on rv1108-elgin-r1【请注意,邮件由u-boot-boun...@lists.denx.de代发】

2020-03-26 Thread Otavio Salvador
Ping?

On Tue, Mar 17, 2020 at 10:50 PM Kever Yang  wrote:
>
> Hi Otavio,
>
>
> On 2020/3/14 上午1:42, Otavio Salvador wrote:
> > In order to make the console pins more robust to noise, activate
> > the pullups and increase its drive strength.
> >
> > Signed-off-by: Otavio Salvador 
>
>
> Reviewed-by: Kever Yang 
>
> Thanks,
> - Kever
> > ---
> >
> >   arch/arm/dts/rv1108-elgin-r1.dts | 11 +++
> >   1 file changed, 11 insertions(+)
> >
> > diff --git a/arch/arm/dts/rv1108-elgin-r1.dts 
> > b/arch/arm/dts/rv1108-elgin-r1.dts
> > index 32b95940b0..83e8b31838 100644
> > --- a/arch/arm/dts/rv1108-elgin-r1.dts
> > +++ b/arch/arm/dts/rv1108-elgin-r1.dts
> > @@ -40,9 +40,20 @@
> >   };
> >
> >   &uart2 {
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&uart2m0_xfer_pullup>;
> >   status = "okay";
> >   };
> >
> >   &usb20_otg {
> >   status = "okay";
> >   };
> > +
> > +&pinctrl {
> > + uart2m0 {
> > + uart2m0_xfer_pullup: uart2m0-xfer-pullup {
> > + rockchip,pins = <2 RK_PD2 RK_FUNC_1 
> > &pcfg_pull_up_drv_8ma>,
> > + <2 RK_PD1 RK_FUNC_1 
> > &pcfg_pull_up_drv_8ma>;
> > + };
> > + };
> > +};
>
>


-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854  Mobile: +1 (347) 903-9750


Re: [PATCH 2/4] elgin-rv1108: Use rk_board_late_init() for GPIO settings【请注意,邮件由u-boot-boun...@lists.denx.de代发】

2020-03-26 Thread Otavio Salvador
Ping?

On Tue, Mar 17, 2020 at 10:52 PM Kever Yang  wrote:
>
>
> On 2020/3/14 上午1:42, Otavio Salvador wrote:
> > Since commit 8e9a8d0d0c8c ("rockchip: elgin-rv1108: use board_early_init_f
> > for per-boar init") the function that configure the board GPIOs is no
> > longer called since CONFIG_BOARD_EARLY_INIT_F=y is not selected.
> >
> > These GPIOs do not need to be configured in such early stagem, so change it
> > to rk_board_late_init() and also select CONFIG_BOARD_LATE_INIT=y
> > to fix the regression.
> >
> > Signed-off-by: Otavio Salvador 
> > Signed-off-by: Fabio Berton 
>
>
> Reviewed-by: Kever Yang 
>
> Thanks,
> - Kever
> > ---
> >
> >   board/elgin/elgin_rv1108/elgin_rv1108.c | 2 +-
> >   configs/elgin-rv1108_defconfig  | 1 +
> >   2 files changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/board/elgin/elgin_rv1108/elgin_rv1108.c 
> > b/board/elgin/elgin_rv1108/elgin_rv1108.c
> > index 607667ac63..950ab2 100644
> > --- a/board/elgin/elgin_rv1108/elgin_rv1108.c
> > +++ b/board/elgin/elgin_rv1108/elgin_rv1108.c
> > @@ -50,7 +50,7 @@ int mach_cpu_init(void)
> >
> >   #define MODEM_ENABLE_GPIO 111
> >
> > -int board_early_init_f(void)
> > +int rk_board_late_init(void)
> >   {
> >   gpio_request(MODEM_ENABLE_GPIO, "modem_enable");
> >   gpio_direction_output(MODEM_ENABLE_GPIO, 0);
> > diff --git a/configs/elgin-rv1108_defconfig b/configs/elgin-rv1108_defconfig
> > index 62af7634a3..80d53f3c10 100644
> > --- a/configs/elgin-rv1108_defconfig
> > +++ b/configs/elgin-rv1108_defconfig
> > @@ -10,6 +10,7 @@ CONFIG_DEBUG_UART_CLOCK=2400
> >   CONFIG_DEBUG_UART=y
> >   # CONFIG_USE_BOOTCOMMAND is not set
> >   CONFIG_DEFAULT_FDT_FILE="rv1108-elgin-r1.dtb"
> > +CONFIG_BOARD_LATE_INIT=y
> >   # CONFIG_DISPLAY_CPUINFO is not set
> >   CONFIG_DISPLAY_BOARDINFO_LATE=y
> >   CONFIG_CMD_GPIO=y
>
>


-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854  Mobile: +1 (347) 903-9750


Re: [PATCH 3/4] elgin-rv1108: Avoid adc_channel_single_shot error【请注意,邮件由u-boot-boun...@lists.denx.de代发】

2020-03-26 Thread Otavio Salvador
Ping?

On Tue, Mar 17, 2020 at 10:52 PM Kever Yang  wrote:
>
>
> On 2020/3/14 上午1:42, Otavio Salvador wrote:
> > Currently the following error message is seen during boot:
> >
> > U-Boot 2020.01-08751-g55759ae141 (Mar 09 2020 - 14:44:52 -0300)
> >
> > Model: Elgin RV1108 R1 board
> > DRAM:  128 MiB
> > APLL: 6 DPLL:12 GPLL:118800
> > ACLK_BUS: 14850 ACLK_PERI:14850 HCLK_PERI:14850 
> > PCLK_PERI:7425
> > MMC:   dwmmc@3011: 0
> > Loading Environment from MMC... OK
> > In:serial@1021
> > Out:   serial@1021
> > Err:   serial@1021
> > Model: Elgin RV1108 R1 board
> > rockchip_dnl_key_pressed: adc_channel_single_shot fail!
> > 
> >
> > Since the elgin-rv1108 does not use ADC to read the download
> > key status, select CONFIG_ROCKCHIP_BOOT_MODE_REG=0 to avoid
> > such error.
> >
> > Signed-off-by: Otavio Salvador 
>
>
> Reviewed-by: Kever Yang 
>
> Thanks,
> - Kever
> > ---
> >
> >   configs/elgin-rv1108_defconfig | 1 +
> >   1 file changed, 1 insertion(+)
> >
> > diff --git a/configs/elgin-rv1108_defconfig b/configs/elgin-rv1108_defconfig
> > index 80d53f3c10..b6682994f5 100644
> > --- a/configs/elgin-rv1108_defconfig
> > +++ b/configs/elgin-rv1108_defconfig
> > @@ -4,6 +4,7 @@ CONFIG_SYS_TEXT_BASE=0x6000
> >   CONFIG_ENV_OFFSET=0x3F8000
> >   CONFIG_ROCKCHIP_RV1108=y
> >   CONFIG_TARGET_ELGIN_RV1108=y
> > +CONFIG_ROCKCHIP_BOOT_MODE_REG=0
> >   CONFIG_NR_DRAM_BANKS=1
> >   CONFIG_DEBUG_UART_BASE=0x1021
> >   CONFIG_DEBUG_UART_CLOCK=2400
>
>


-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854  Mobile: +1 (347) 903-9750


Re: [PATCH 4/4] rv1108: Fix boot regression

2020-03-26 Thread Otavio Salvador
Ping?

On Tue, Mar 17, 2020 at 10:52 PM Kever Yang  wrote:
>
>
> On 2020/3/14 上午1:42, Otavio Salvador wrote:
> > Since commit 79030a486128 ("rockchip: Add Single boot image
> > (with binman, pad_cat)") the following boot regression is seen:
> >
> > U-Boot 2020.04-rc3-00050-gd16e18ca6c-dirty (Mar 09 2020 - 11:40:07 -0300)
> >
> > Model: Elgin RV1108 R1 board
> > DRAM:  128 MiB
> > initcall sequence 67fd12a0 failed at call 6000b927 (err=-22)
> >
> > This happens because the above commit missed to include the
> > "rockchip-u-boot.dtsi" for rv1108, so include this file
> > like it done for other Rockchip SoC dtsi's.
> >
> > Fixes: 79030a486128 ("rockchip: Add Single boot image (with binman, 
> > pad_cat)")
> > Signed-off-by: Otavio Salvador 
>
>
> Reviewed-by: Kever Yang 
>
> Thanks,
> - Kever
> > ---
> >
> >   arch/arm/dts/rv1108-u-boot.dtsi | 6 ++
> >   1 file changed, 6 insertions(+)
> >   create mode 100644 arch/arm/dts/rv1108-u-boot.dtsi
> >
> > diff --git a/arch/arm/dts/rv1108-u-boot.dtsi 
> > b/arch/arm/dts/rv1108-u-boot.dtsi
> > new file mode 100644
> > index 00..41ac054b81
> > --- /dev/null
> > +++ b/arch/arm/dts/rv1108-u-boot.dtsi
> > @@ -0,0 +1,6 @@
> > +// SPDX-License-Identifier: GPL-2.0+
> > +/*
> > + * Copyright (C) 2019 Jagan Teki 
> > + */
> > +
> > +#include "rockchip-u-boot.dtsi"
>
>


-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854  Mobile: +1 (347) 903-9750


Re: [PATCH 0/4] Fixes for rv1108 and rv1108-elgin-r1

2020-03-26 Thread Otavio Salvador
Hello,

On Fri, Mar 13, 2020 at 2:42 PM Otavio Salvador  wrote:
> Those are fixes we've been using and we'd like to upstream.
>
> They are fixes and would be great to have them included on 2020.04
> release.
>
>
> Otavio Salvador (4):
>   ARM: dts: Activate pullups in the console pins on rv1108-elgin-r1
>   elgin-rv1108: Use rk_board_late_init() for GPIO settings
>   elgin-rv1108: Avoid adc_channel_single_shot error
>   rv1108: Fix boot regression

The patches on this serie are all bug fixes so please make sure they
go on 2020.04. Maybe Tom may apply them directly?

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854  Mobile: +1 (347) 903-9750


Re: [PATCH 1/2] rockchip: elgin-rv1108: Use syscon API to get grf base

2020-02-20 Thread Otavio Salvador
On Thu, Feb 20, 2020 at 12:05 AM Simon Glass  wrote:
>
> On Tue, 18 Feb 2020 at 18:46, Kever Yang  wrote:
> >
> > Use syscon API to get grf base instead of get from dts.
> >
> > Signed-off-by: Kever Yang 
> > ---
> >
> >  board/elgin/elgin_rv1108/elgin_rv1108.c | 7 +++
> >  1 file changed, 3 insertions(+), 4 deletions(-)
>
> Reviewed-by: Simon Glass 

Acked-by: Otavio Salvador 

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854  Mobile: +1 (347) 903-9750


[PATCH 1/4] ARM: dts: Activate pullups in the console pins on rv1108-elgin-r1

2020-03-13 Thread Otavio Salvador
In order to make the console pins more robust to noise, activate
the pullups and increase its drive strength.

Signed-off-by: Otavio Salvador 
---

 arch/arm/dts/rv1108-elgin-r1.dts | 11 +++
 1 file changed, 11 insertions(+)

diff --git a/arch/arm/dts/rv1108-elgin-r1.dts b/arch/arm/dts/rv1108-elgin-r1.dts
index 32b95940b0..83e8b31838 100644
--- a/arch/arm/dts/rv1108-elgin-r1.dts
+++ b/arch/arm/dts/rv1108-elgin-r1.dts
@@ -40,9 +40,20 @@
 };
 
 &uart2 {
+   pinctrl-names = "default";
+   pinctrl-0 = <&uart2m0_xfer_pullup>;
status = "okay";
 };
 
 &usb20_otg {
status = "okay";
 };
+
+&pinctrl {
+   uart2m0 {
+   uart2m0_xfer_pullup: uart2m0-xfer-pullup {
+   rockchip,pins = <2 RK_PD2 RK_FUNC_1 
&pcfg_pull_up_drv_8ma>,
+   <2 RK_PD1 RK_FUNC_1 
&pcfg_pull_up_drv_8ma>;
+   };
+   };
+};
-- 
2.25.1



[PATCH 0/4] Fixes for rv1108 and rv1108-elgin-r1

2020-03-13 Thread Otavio Salvador
Those are fixes we've been using and we'd like to upstream.

They are fixes and would be great to have them included on 2020.04
release.


Otavio Salvador (4):
  ARM: dts: Activate pullups in the console pins on rv1108-elgin-r1
  elgin-rv1108: Use rk_board_late_init() for GPIO settings
  elgin-rv1108: Avoid adc_channel_single_shot error
  rv1108: Fix boot regression

 arch/arm/dts/rv1108-elgin-r1.dts| 11 +++
 arch/arm/dts/rv1108-u-boot.dtsi |  6 ++
 board/elgin/elgin_rv1108/elgin_rv1108.c |  2 +-
 configs/elgin-rv1108_defconfig  |  2 ++
 4 files changed, 20 insertions(+), 1 deletion(-)
 create mode 100644 arch/arm/dts/rv1108-u-boot.dtsi

-- 
2.25.1



[PATCH 3/4] elgin-rv1108: Avoid adc_channel_single_shot error

2020-03-13 Thread Otavio Salvador
Currently the following error message is seen during boot:

U-Boot 2020.01-08751-g55759ae141 (Mar 09 2020 - 14:44:52 -0300)

Model: Elgin RV1108 R1 board
DRAM:  128 MiB
APLL: 6 DPLL:12 GPLL:118800
ACLK_BUS: 14850 ACLK_PERI:14850 HCLK_PERI:14850 PCLK_PERI:7425
MMC:   dwmmc@3011: 0
Loading Environment from MMC... OK
In:serial@1021
Out:   serial@1021
Err:   serial@1021
Model: Elgin RV1108 R1 board
rockchip_dnl_key_pressed: adc_channel_single_shot fail!


Since the elgin-rv1108 does not use ADC to read the download
key status, select CONFIG_ROCKCHIP_BOOT_MODE_REG=0 to avoid
such error.

Signed-off-by: Otavio Salvador 
---

 configs/elgin-rv1108_defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/configs/elgin-rv1108_defconfig b/configs/elgin-rv1108_defconfig
index 80d53f3c10..b6682994f5 100644
--- a/configs/elgin-rv1108_defconfig
+++ b/configs/elgin-rv1108_defconfig
@@ -4,6 +4,7 @@ CONFIG_SYS_TEXT_BASE=0x6000
 CONFIG_ENV_OFFSET=0x3F8000
 CONFIG_ROCKCHIP_RV1108=y
 CONFIG_TARGET_ELGIN_RV1108=y
+CONFIG_ROCKCHIP_BOOT_MODE_REG=0
 CONFIG_NR_DRAM_BANKS=1
 CONFIG_DEBUG_UART_BASE=0x1021
 CONFIG_DEBUG_UART_CLOCK=2400
-- 
2.25.1



[PATCH 2/4] elgin-rv1108: Use rk_board_late_init() for GPIO settings

2020-03-13 Thread Otavio Salvador
Since commit 8e9a8d0d0c8c ("rockchip: elgin-rv1108: use board_early_init_f
for per-boar init") the function that configure the board GPIOs is no
longer called since CONFIG_BOARD_EARLY_INIT_F=y is not selected.

These GPIOs do not need to be configured in such early stagem, so change it
to rk_board_late_init() and also select CONFIG_BOARD_LATE_INIT=y
to fix the regression.

Signed-off-by: Otavio Salvador 
Signed-off-by: Fabio Berton 
---

 board/elgin/elgin_rv1108/elgin_rv1108.c | 2 +-
 configs/elgin-rv1108_defconfig  | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/board/elgin/elgin_rv1108/elgin_rv1108.c 
b/board/elgin/elgin_rv1108/elgin_rv1108.c
index 607667ac63..950ab2 100644
--- a/board/elgin/elgin_rv1108/elgin_rv1108.c
+++ b/board/elgin/elgin_rv1108/elgin_rv1108.c
@@ -50,7 +50,7 @@ int mach_cpu_init(void)
 
 #define MODEM_ENABLE_GPIO 111
 
-int board_early_init_f(void)
+int rk_board_late_init(void)
 {
gpio_request(MODEM_ENABLE_GPIO, "modem_enable");
gpio_direction_output(MODEM_ENABLE_GPIO, 0);
diff --git a/configs/elgin-rv1108_defconfig b/configs/elgin-rv1108_defconfig
index 62af7634a3..80d53f3c10 100644
--- a/configs/elgin-rv1108_defconfig
+++ b/configs/elgin-rv1108_defconfig
@@ -10,6 +10,7 @@ CONFIG_DEBUG_UART_CLOCK=2400
 CONFIG_DEBUG_UART=y
 # CONFIG_USE_BOOTCOMMAND is not set
 CONFIG_DEFAULT_FDT_FILE="rv1108-elgin-r1.dtb"
+CONFIG_BOARD_LATE_INIT=y
 # CONFIG_DISPLAY_CPUINFO is not set
 CONFIG_DISPLAY_BOARDINFO_LATE=y
 CONFIG_CMD_GPIO=y
-- 
2.25.1



[PATCH 4/4] rv1108: Fix boot regression

2020-03-13 Thread Otavio Salvador
Since commit 79030a486128 ("rockchip: Add Single boot image
(with binman, pad_cat)") the following boot regression is seen:

U-Boot 2020.04-rc3-00050-gd16e18ca6c-dirty (Mar 09 2020 - 11:40:07 -0300)

Model: Elgin RV1108 R1 board
DRAM:  128 MiB
initcall sequence 67fd12a0 failed at call 6000b927 (err=-22)

This happens because the above commit missed to include the
"rockchip-u-boot.dtsi" for rv1108, so include this file
like it done for other Rockchip SoC dtsi's.

Fixes: 79030a486128 ("rockchip: Add Single boot image (with binman, pad_cat)")
Signed-off-by: Otavio Salvador 
---

 arch/arm/dts/rv1108-u-boot.dtsi | 6 ++
 1 file changed, 6 insertions(+)
 create mode 100644 arch/arm/dts/rv1108-u-boot.dtsi

diff --git a/arch/arm/dts/rv1108-u-boot.dtsi b/arch/arm/dts/rv1108-u-boot.dtsi
new file mode 100644
index 00..41ac054b81
--- /dev/null
+++ b/arch/arm/dts/rv1108-u-boot.dtsi
@@ -0,0 +1,6 @@
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * Copyright (C) 2019 Jagan Teki 
+ */
+
+#include "rockchip-u-boot.dtsi"
-- 
2.25.1



Re: [PATCH 3/3] rockchip: evb-rv1108: enable board early init

2020-04-13 Thread Otavio Salvador
On Sun, Apr 12, 2020 at 10:39 PM Kever Yang  wrote:
>
> Enable board early init callback to init board specific hardware.
>
> Signed-off-by: Kever Yang 

Please send the same patch to elgin-rv1108 config.


-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854  Mobile: +1 (347) 903-9750


Re: [U-Boot] [PATCH] pico-imx7d: Remove specific code for fitImage

2019-10-13 Thread Otavio Salvador
On Sun, Oct 13, 2019 at 9:30 AM Joris Offouga  wrote:
>
> This implementation is a specific utilisation with CONFIG_FIT.
> This feature use script for boot. With distrobootcmd is possible
> to load script.
>
> Signed-off-by: Joris Offouga 

Reviewed-by: Otavio Salvador 

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854  Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


[PATCH 1/3] mx6ul_14x14_evk: Enable SPL USB and SDP support

2020-06-09 Thread Otavio Salvador
This fixes the boot from USB loader, which is critical to easy the
manufacture process.

Signed-off-by: Otavio Salvador 
---

 configs/mx6ul_14x14_evk_defconfig | 11 +++
 1 file changed, 11 insertions(+)

diff --git a/configs/mx6ul_14x14_evk_defconfig 
b/configs/mx6ul_14x14_evk_defconfig
index 2906782b126..cad766443c2 100644
--- a/configs/mx6ul_14x14_evk_defconfig
+++ b/configs/mx6ul_14x14_evk_defconfig
@@ -22,6 +22,9 @@ CONFIG_BOUNCE_BUFFER=y
 CONFIG_BOARD_EARLY_INIT_F=y
 CONFIG_SPL_FS_EXT4=y
 CONFIG_SPL_I2C_SUPPORT=y
+CONFIG_SPL_USB_HOST_SUPPORT=y
+CONFIG_SPL_USB_GADGET=y
+CONFIG_SPL_USB_SDP_SUPPORT=y
 CONFIG_SPL_WATCHDOG_SUPPORT=y
 CONFIG_HUSH_PARSER=y
 CONFIG_CMD_BOOTZ=y
@@ -32,6 +35,8 @@ CONFIG_CMD_GPIO=y
 CONFIG_CMD_I2C=y
 CONFIG_CMD_MMC=y
 CONFIG_CMD_USB=y
+CONFIG_CMD_USB_SDP=y
+CONFIG_CMD_USB_MASS_STORAGE=y
 CONFIG_CMD_DHCP=y
 CONFIG_CMD_PING=y
 CONFIG_CMD_BMP=y
@@ -46,6 +51,7 @@ CONFIG_DEFAULT_DEVICE_TREE="imx6ul-14x14-evk"
 CONFIG_ENV_IS_IN_MMC=y
 CONFIG_SYS_RELOC_GD_ENV_ADDR=y
 CONFIG_ENV_VARS_UBOOT_RUNTIME_CONFIG=y
+CONFIG_USB_FUNCTION_FASTBOOT=y
 CONFIG_DM_74X164=y
 CONFIG_DM_I2C=y
 CONFIG_DM_MMC=y
@@ -73,5 +79,10 @@ CONFIG_SOFT_SPI=y
 CONFIG_USB=y
 CONFIG_DM_USB=y
 CONFIG_USB_STORAGE=y
+CONFIG_USB_GADGET=y
+CONFIG_USB_GADGET_MANUFACTURER="FSL"
+CONFIG_USB_GADGET_VENDOR_NUM=0x0525
+CONFIG_USB_GADGET_PRODUCT_NUM=0xa4a5
+CONFIG_CI_UDC=y
 CONFIG_DM_VIDEO=y
 CONFIG_SYS_WHITE_ON_BLACK=y
-- 
2.27.0



[PATCH 2/3] mx6ul_14x14_evk: Enable eMMC boot support

2020-06-09 Thread Otavio Salvador
Signed-off-by: Otavio Salvador 
---

 configs/mx6ul_14x14_evk_defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/configs/mx6ul_14x14_evk_defconfig 
b/configs/mx6ul_14x14_evk_defconfig
index cad766443c2..a2cb3215d44 100644
--- a/configs/mx6ul_14x14_evk_defconfig
+++ b/configs/mx6ul_14x14_evk_defconfig
@@ -55,6 +55,7 @@ CONFIG_USB_FUNCTION_FASTBOOT=y
 CONFIG_DM_74X164=y
 CONFIG_DM_I2C=y
 CONFIG_DM_MMC=y
+CONFIG_SUPPORT_EMMC_BOOT=y
 CONFIG_FSL_USDHC=y
 CONFIG_MTD=y
 CONFIG_DM_SPI_FLASH=y
-- 
2.27.0



[PATCH 3/3] mx6ul_14x14_evk: Change environment offset to overlap with U-Boot

2020-06-09 Thread Otavio Salvador
We need to change the environment offset to avoid corrupting the U-Boot
binary when saving it.

Signed-off-by: Otavio Salvador 
---

 configs/mx6ul_14x14_evk_defconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/configs/mx6ul_14x14_evk_defconfig 
b/configs/mx6ul_14x14_evk_defconfig
index a2cb3215d44..5086dac4569 100644
--- a/configs/mx6ul_14x14_evk_defconfig
+++ b/configs/mx6ul_14x14_evk_defconfig
@@ -5,7 +5,7 @@ CONFIG_SPL_GPIO_SUPPORT=y
 CONFIG_SPL_LIBCOMMON_SUPPORT=y
 CONFIG_SPL_LIBGENERIC_SUPPORT=y
 CONFIG_ENV_SIZE=0x2000
-CONFIG_ENV_OFFSET=0x8
+CONFIG_ENV_OFFSET=0xC
 CONFIG_TARGET_MX6UL_14X14_EVK=y
 CONFIG_DM_GPIO=y
 CONFIG_SPL_MMC_SUPPORT=y
-- 
2.27.0



[PATCH 1/2] mx6ul_14x14_evk: Enable SPL USB and SDP support

2020-06-09 Thread Otavio Salvador
This fixes the boot from USB loader, which is critical to easy the
manufacture process.

Signed-off-by: Otavio Salvador 
---

(no changes since v1)

 configs/mx6ul_14x14_evk_defconfig | 11 +++
 1 file changed, 11 insertions(+)

diff --git a/configs/mx6ul_14x14_evk_defconfig 
b/configs/mx6ul_14x14_evk_defconfig
index 2906782b126..cad766443c2 100644
--- a/configs/mx6ul_14x14_evk_defconfig
+++ b/configs/mx6ul_14x14_evk_defconfig
@@ -22,6 +22,9 @@ CONFIG_BOUNCE_BUFFER=y
 CONFIG_BOARD_EARLY_INIT_F=y
 CONFIG_SPL_FS_EXT4=y
 CONFIG_SPL_I2C_SUPPORT=y
+CONFIG_SPL_USB_HOST_SUPPORT=y
+CONFIG_SPL_USB_GADGET=y
+CONFIG_SPL_USB_SDP_SUPPORT=y
 CONFIG_SPL_WATCHDOG_SUPPORT=y
 CONFIG_HUSH_PARSER=y
 CONFIG_CMD_BOOTZ=y
@@ -32,6 +35,8 @@ CONFIG_CMD_GPIO=y
 CONFIG_CMD_I2C=y
 CONFIG_CMD_MMC=y
 CONFIG_CMD_USB=y
+CONFIG_CMD_USB_SDP=y
+CONFIG_CMD_USB_MASS_STORAGE=y
 CONFIG_CMD_DHCP=y
 CONFIG_CMD_PING=y
 CONFIG_CMD_BMP=y
@@ -46,6 +51,7 @@ CONFIG_DEFAULT_DEVICE_TREE="imx6ul-14x14-evk"
 CONFIG_ENV_IS_IN_MMC=y
 CONFIG_SYS_RELOC_GD_ENV_ADDR=y
 CONFIG_ENV_VARS_UBOOT_RUNTIME_CONFIG=y
+CONFIG_USB_FUNCTION_FASTBOOT=y
 CONFIG_DM_74X164=y
 CONFIG_DM_I2C=y
 CONFIG_DM_MMC=y
@@ -73,5 +79,10 @@ CONFIG_SOFT_SPI=y
 CONFIG_USB=y
 CONFIG_DM_USB=y
 CONFIG_USB_STORAGE=y
+CONFIG_USB_GADGET=y
+CONFIG_USB_GADGET_MANUFACTURER="FSL"
+CONFIG_USB_GADGET_VENDOR_NUM=0x0525
+CONFIG_USB_GADGET_PRODUCT_NUM=0xa4a5
+CONFIG_CI_UDC=y
 CONFIG_DM_VIDEO=y
 CONFIG_SYS_WHITE_ON_BLACK=y
-- 
2.27.0



[PATCH 2/2] mx6ul_14x14_evk: Avoid overlap of environment over U-Boot proper

2020-06-09 Thread Otavio Salvador
We need to change the environment offset to avoid corrupting the U-Boot
binary when saving it.

Signed-off-by: Otavio Salvador 
---

Changes in v2:
- rework commit log

 configs/mx6ul_14x14_evk_defconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/configs/mx6ul_14x14_evk_defconfig 
b/configs/mx6ul_14x14_evk_defconfig
index cad766443c2..4ec5f80ee69 100644
--- a/configs/mx6ul_14x14_evk_defconfig
+++ b/configs/mx6ul_14x14_evk_defconfig
@@ -5,7 +5,7 @@ CONFIG_SPL_GPIO_SUPPORT=y
 CONFIG_SPL_LIBCOMMON_SUPPORT=y
 CONFIG_SPL_LIBGENERIC_SUPPORT=y
 CONFIG_ENV_SIZE=0x2000
-CONFIG_ENV_OFFSET=0x8
+CONFIG_ENV_OFFSET=0xC
 CONFIG_TARGET_MX6UL_14X14_EVK=y
 CONFIG_DM_GPIO=y
 CONFIG_SPL_MMC_SUPPORT=y
-- 
2.27.0



[PATCH 1/2] spl: Avoid printing boot device if silent console is enabled

2020-09-29 Thread Otavio Salvador
This avoid emitting message if silent console is in use, otherwise it
pollutes the output and user desires a quiet console

Signed-off-by: Otavio Salvador 
---

 common/spl/spl.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/common/spl/spl.c b/common/spl/spl.c
index 4840d1d3670..63c48fbf33d 100644
--- a/common/spl/spl.c
+++ b/common/spl/spl.c
@@ -552,7 +552,9 @@ static int boot_from_devices(struct spl_image_info 
*spl_image,
struct spl_image_loader *loader;
 
loader = spl_ll_find_loader(spl_boot_list[i]);
-#if defined(CONFIG_SPL_SERIAL_SUPPORT) && defined(CONFIG_SPL_LIBCOMMON_SUPPORT)
+#if defined(CONFIG_SPL_SERIAL_SUPPORT) \
+&& defined(CONFIG_SPL_LIBCOMMON_SUPPORT)\
+&& !defined(CONFIG_SILENT_CONSOLE)
if (loader)
printf("Trying to boot from %s\n", loader->name);
else
-- 
2.28.0



[PATCH 2/2] spl: Add SPL_SERIAL as requirement for SDP_USB_SDP

2020-09-29 Thread Otavio Salvador
The USB SDP protocol require the SPL serial support to allow the build
to succeed.

Signed-off-by: Otavio Salvador 
---

 common/spl/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/common/spl/Kconfig b/common/spl/Kconfig
index af8255a8d6a..20c5c4af3c3 100644
--- a/common/spl/Kconfig
+++ b/common/spl/Kconfig
@@ -1237,6 +1237,7 @@ endchoice
 
 config SPL_USB_SDP_SUPPORT
bool "Support SDP (Serial Download Protocol)"
+   depends on SPL_SERIAL_SUPPORT
help
  Enable Serial Download Protocol (SDP) device support in SPL. This
  allows to download images into memory and execute (jump to) them
-- 
2.28.0



Re: [PATCH 1/2] spl: Avoid printing boot device if silent console is enabled

2020-09-30 Thread Otavio Salvador
Em qua., 30 de set. de 2020 às 02:23, Stefan Roese  escreveu:
> On 30.09.20 04:14, Otavio Salvador wrote:
> Wouldn't it be better, to add this CONFIG_SILENT_CONSOLE check to
> the console / printf function itself instead of adding it to all
> callers?

I believe in critical errors and like, it is important to print so I'd
say that using a more controlled "hide" mechanism is good.

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854  Mobile: +1 (347) 903-9750


Re: RPi3: serial console

2020-10-12 Thread Otavio Salvador
Hello,

Em seg., 12 de out. de 2020 às 15:39, Heinrich Schuchardt
 escreveu:
> On 10/12/20 6:50 PM, Peter Robinson wrote:
> > On Mon, Oct 12, 2020 at 3:56 PM Heinrich Schuchardt  
> > wrote:
> >>
> >> Hello Matthias,
> >>
> >> I have upgraded the Raspberry 3 firmware from raspi3-firmware
> >> (1.20190215-1+deb10u4) to raspi-firmware (1.20200601-3)
> >> [https://packages.debian.org/bullseye/raspi-firmware].
> >>
> >> After the upgrade the output of U-Boot on the serial console is complete
> >> gibberish as if the baudrate were incorrect. The output from the Linux
> >> kernel is fine at 115200 baud.
> >
> > I've seen similar on all firmware since around mid April up until
> > recently, it seems to be fixed with releases in Oct (I'm using one
> > from Oct 8th), I'm not sure if it can be fixed in U-Boot but it seems
> > to be due to a change in the firmware.
>
> Thanks for confirming the problem.
>
> Using the most current files from
> https://github.com/raspberrypi/firmware does not solve the problem for me.
>
> In bcm283x_mu_serial_setbrg() we divide a clock rate by the baudrate.
> divider = plat->clock / (baudrate * 8);
>
> If we do not have the correct clock rate, the actual baudrate will be wrong.

It seems if you have the dtb files next to the firmware, the firmware
loads it and the baudrate seems right. We've been struggling with this
at our side as well and this was the only way we got it to work.

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854  Mobile: +1 (347) 903-9750


Re: [PATCH] pico-imx7d: Convert to DM_ETH

2020-01-05 Thread Otavio Salvador
On Sun, Jan 5, 2020 at 7:32 AM Stefano Babic  wrote:
>
> On 04/01/20 13:03, Joris Offouga wrote:
> > Hi Stefano,
> >
> > Sorry for the delay,
> >
> > Le 28/12/2019 à 12:31, Stefano Babic a écrit :
> >> Hi Joris,
> >>
> >>
> >>
> >> On 08/12/19 18:02, Joris Offouga wrote:
> >>> Signed-off-by: Joris Offouga 
> >>> ---
> >>>   board/technexion/pico-imx7d/pico-imx7d.c | 46 
> >>>   configs/pico-dwarf-imx7d_defconfig   |  5 +++
> >>>   configs/pico-hobbit-imx7d_defconfig  |  5 +++
> >>>   configs/pico-imx7d_bl33_defconfig|  5 +++
> >>>   configs/pico-imx7d_defconfig |  5 +++
> >>>   configs/pico-nymph-imx7d_defconfig   |  5 +++
> >>>   configs/pico-pi-imx7d_defconfig  |  5 +++
> >>>   include/configs/pico-imx7d.h | 11 --
> >>>   8 files changed, 30 insertions(+), 57 deletions(-)
> >>>
> >> I get build errors with pico-imx7d_bl33.
> >>
> >> https://travis-ci.org/sbabic/u-boot-imx/jobs/630056283
> >>
> >>
> >>   Could you take a look, please ?
> >
> > I tried to correct the compilation error but I did not find good ways to
> > do it, I suggest you remove the usb ethernet support from the
> > configuration :
> >
> > -CONFIG_USB_ETHER=y
> > -CONFIG_USB_ETH_CDC=y
> > -CONFIG_USBNET_HOST_ADDR="de:ad:be:af:00:00"
> >
> > If you agree I send V2
>
> If the board maintainer (Otavio) agrees it, I am fine.

I agree. However, do this as a commit prior to this one so if someone
ever wants to revert it is easy. Also, explain the reasoning on the
commit log.

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854  Mobile: +1 (347) 903-9750


Re: [U-Boot] [PATCH] Wandboard: Add support for Wandboard quad

2013-05-27 Thread Otavio Salvador
On Mon, May 27, 2013 at 1:13 AM, Tapani Utriainen wrote:

>
> Add support for the Wandboard quad.
>
> Signed-off-by: Tapani Utriainen 
>
> ---
>  boards.cfg  | 1 +
>  include/configs/wandboard.h | 2 ++
>  2 files changed, 3 insertions(+)
>
> diff --git a/boards.cfg b/boards.cfg
> index 31483d6..b907409 100644
> --- a/boards.cfg
> +++ b/boards.cfg
> @@ -268,6 +268,7 @@ nitrogen6s   arm armv7
> nitrogen6x  boundar
>  nitrogen6s1g arm armv7   nitrogen6x
>  boundary   mx6
>  
> nitrogen6x:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6s1g.cfg,MX6S,DDR_MB=1024
>  wandboard_dlarm armv7   wandboard   -
>  mx6
> wandboard:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6dl.cfg,MX6DL,DDR_MB=1024
>  wandboard_solo  arm armv7   wandboard   -
>  mx6
> wandboard:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6s.cfg,MX6S,DDR_MB=512
> +wandboard_quad  arm armv7   wandboard   -
>  mx6
> wandboard:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6q2g.cfg,MX6Q,DDR_MB=2048
>

Please put quad, dl and solo in order. It is now dl, solo, quad.


>  cm_t35   arm armv7   cm_t35
>  -  omap3
>  omap3_overo  arm armv7   overo
> -  omap3
>  omap3_pandoraarm armv7   pandora
> -  omap3
> diff --git a/include/configs/wandboard.h b/include/configs/wandboard.h
> index 120e3f6..cf22042 100644
> --- a/include/configs/wandboard.h
> +++ b/include/configs/wandboard.h
> @@ -85,6 +85,8 @@
>  #define CONFIG_DEFAULT_FDT_FILE"imx6dl-wandboard.dtb"
>  #elif defined(CONFIG_MX6S)
>  #define CONFIG_DEFAULT_FDT_FILE"imx6s-wandboard.dtb"
> +#elif defined(CONFIG_MX6Q)
> +#define CONFIG_DEFAULT_FDT_FILE"imx6q-wandboard.dtb"
>  #endif
>

Same here.


>  #define CONFIG_EXTRA_ENV_SETTINGS \
> --
> 1.8.0.3
> ___
> U-Boot mailing list
> U-Boot@lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>



-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 1/2] wandboard: Use 'wandboard' vendor for the Wandboard boards

2013-05-27 Thread Otavio Salvador
The board needs to be under a 'wandboard' vendor so we can use it to
choose the proper Boot Splash image.

Signed-off-by: Otavio Salvador 
---
 board/wandboard/{ => }/wandboard/Makefile| 0
 board/wandboard/{ => }/wandboard/README  | 0
 board/wandboard/{ => }/wandboard/wandboard.c | 0
 boards.cfg   | 5 +++--
 4 files changed, 3 insertions(+), 2 deletions(-)
 rename board/wandboard/{ => }/wandboard/Makefile (100%)
 rename board/wandboard/{ => }/wandboard/README (100%)
 rename board/wandboard/{ => }/wandboard/wandboard.c (100%)

diff --git a/board/wandboard/Makefile b/board/wandboard/wandboard/Makefile
similarity index 100%
rename from board/wandboard/Makefile
rename to board/wandboard/wandboard/Makefile
diff --git a/board/wandboard/README b/board/wandboard/wandboard/README
similarity index 100%
rename from board/wandboard/README
rename to board/wandboard/wandboard/README
diff --git a/board/wandboard/wandboard.c b/board/wandboard/wandboard/wandboard.c
similarity index 100%
rename from board/wandboard/wandboard.c
rename to board/wandboard/wandboard/wandboard.c
diff --git a/boards.cfg b/boards.cfg
index 2f39f26..10fe2f9 100644
--- a/boards.cfg
+++ b/boards.cfg
@@ -269,9 +269,8 @@ nitrogen6q   arm armv7   
nitrogen6x  boundar
 nitrogen6q2g arm armv7   nitrogen6x  
boundary   mx6
nitrogen6x:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6q2g.cfg,MX6Q,DDR_MB=2048
 nitrogen6s   arm armv7   nitrogen6x  
boundary   mx6
nitrogen6x:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6s.cfg,MX6S,DDR_MB=512
 nitrogen6s1g arm armv7   nitrogen6x  
boundary   mx6
nitrogen6x:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6s1g.cfg,MX6S,DDR_MB=1024
-wandboard_dlarm armv7   wandboard   -  
mx6 
wandboard:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6dl.cfg,MX6DL,DDR_MB=1024
-wandboard_solo  arm armv7   wandboard   -  
mx6 
wandboard:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6s.cfg,MX6S,DDR_MB=512
+wandboard_dl arm armv7   wandboard   
wandboard  mx6 
wandboard:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6dl.cfg,MX6DL,DDR_MB=1024
+wandboard_solo   arm armv7   wandboard   
wandboard  mx6 
wandboard:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6s.cfg,MX6S,DDR_MB=512
 omap3_overo  arm armv7   overo   - 
 omap3
 omap3_pandoraarm armv7   pandora - 
 omap3
 dig297   arm armv7   dig297  
comelitomap3
-- 
1.8.1

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


[U-Boot] [PATCH 2/2] wandboard: Add Boot Splash image with Wandboard logo

2013-05-27 Thread Otavio Salvador
Signed-off-by: Otavio Salvador 
---
 include/configs/wandboard.h |   3 +++
 tools/Makefile  |   3 +++
 tools/logos/wandboard.bmp   | Bin 0 -> 22390 bytes
 3 files changed, 6 insertions(+)
 create mode 100644 tools/logos/wandboard.bmp

diff --git a/include/configs/wandboard.h b/include/configs/wandboard.h
index d4ac086..3f90663 100644
--- a/include/configs/wandboard.h
+++ b/include/configs/wandboard.h
@@ -235,6 +235,9 @@
 #define CONFIG_OF_LIBFDT
 #define CONFIG_CMD_BOOTZ
 
+#define CONFIG_SPLASH_SCREEN_ALIGN
+#define CONFIG_VIDEO_BMP_LOGO
+
 #ifndef CONFIG_SYS_DCACHE_OFF
 #define CONFIG_CMD_CACHE
 #endif
diff --git a/tools/Makefile b/tools/Makefile
index 889c897..5d7db14 100644
--- a/tools/Makefile
+++ b/tools/Makefile
@@ -146,6 +146,9 @@ endif
 ifeq ($(VENDOR),syteco)
 LOGO_BMP= logos/syteco.bmp
 endif
+ifeq ($(VENDOR),wandboard)
+LOGO_BMP= logos/wandboard.bmp
+endif
 ifeq ($(VENDOR),intercontrol)
 LOGO_BMP= logos/intercontrol.bmp
 endif
diff --git a/tools/logos/wandboard.bmp b/tools/logos/wandboard.bmp
new file mode 100644
index 
..7f288a8e8eee052c96b9c2c2699e2e3d3efe1758
GIT binary patch
literal 22390
zcmeHPKaBH6b|3zcL~*sEC~_rAD{-}4%c96z4!InpF_0YM;u09CT%>U6)}@ig#UX%8
zmj;r+#icM1BrXntfdhgZ;3N)k4&c&A5Wu*&aA)8Uz=bc(A-^{xuBh+Rw@dAQ{?7i^
z1FT36-+MFj=Djy>-aPIf|Jh&sIg{lNOvYl|#TsLOf~&?XS^wi#8T+`e{!mFTWa(HI
zJ32aIp69XC(^K}s3oo$MDrRY#GR|{WmL+@XrI*+r{o#+<%P+soe)5w)Wv{&Q3VZd{
zSJ`W?y~bXD{dM-n8*i{T-+Yt3_10VLr$7Bm_V(Lvvv=NkhrRpmyX>$2`fu2K@4d%<
z_Orib@4x>(`@6sY2ll}SAF!YQ{2$qeAAZPw@r!?A|NP5evX4Ic7xvM={u}%F<9}zr
z`s6>@C%^iHefsIA?6c24WB>VI|II%C{B!o}Uw^^==Zi1cmtTI#zWVAb_M6}QFZ=rI
zuh}=>{FZ(5+y7&~``z!Ic(%
z9uZ7y5JX&vl4nW0IzO9FgJ3e9htVotXQ&rE8V{@o&MT{b5?zTbj%Fjb-)$M2wMNQ=
zX0*C}cQlV;F7k7~-+CUUq|uoAlSGK+*y)6U3riPkMDW-hlCZz3MHvh)?XB
zShO9m*&C&ThxWGSf!tYBp3fclevOOiEb=0FVA|KuuIBB7)yQ6y>*L0ZeGUMRId|_g
z3@^`)?u7WgMCv5GcSy%uTeraFi-)6n6EQr5P3F?yGRiN82kYx0jeVD3vvspoZqsM)
zYP983E1eyL{9PLNp?%@29SkQ@A&#BO?SqlTwRO>0pVgkWP|xGtD%){{hQhgNlmkB8
ziW9SWPtij%gpd991d4fn=eENh;@A~u-Osqw0HUW#(5lr!L4>vg;mlNy6>XyH5e
zsypu7p-=im@Gatqx0yEe>WoU$VR}i7(pOQXg4StX_|?Oc8*RIu=Mpy!$Mc57WX1PP!`}{axn=ZYz!f>Tu;0w7hd(%rT@}i-)Kc9{{ko7NU?!JK`cm6DWPug)PXl
zm=WX5NhxwPU`3)V5d)7}d{ko9EPJ?h%EUUZq5l_|aYpv1+59brVs}
zm_DMpf%N+TSX<0o;Hv256N70zM3BUV&GaF)k!8fICGa&LVBnJme33xL3_=vydX1J-
zqEE#R5Q8gLqrv3@q95lxPK8kO&c^E&Tb$|oBJ)P8qU7;tBdH4Y$-fqjNK8#;Kvpo<
zght(tNfSK0jBRFuPb5zkhAT=XPa^RL{v3?RG41U5QWVqUV|RuTMJ?6>LK!(`cY^+d
zfbM}W0qN$n5Lm7aj1&CR`oO=$xT(R+A9y5$#Tfh>;~4^U4EFO;r{$s_Wgz8A@`2pVMCe2hR>hcy0W{y70egh^%mQR;k=}``zI+mdXA!?he;FXx
zbln!icA~6jkNHYx0Wn>c5Kj0gF>F3ZbX;13Eyj&^Fi&YJ3WqtFNG`z^Vm!%xnM*y&
zXu1`j`f{)ppCW2n_rN-MvfvZ+6M8U_^5bK)D#AewT0ya1byKn%
z)IdUf)m_OMqp$1Ev1Eht21y_bSLvN+pA!|rL#>#DU3~|Jpe9dvMa3!Bn6MMAlN2K|
z8<1EKDR=azG364zS7Nm+wn-t<$DqiMWJ+O=H74ZpX2R~3s6e&+~!RlO!5<{V)u|66M
z)ubO$!%!QbaKcNZ_XBbz#5mx*+{dRjEP5dV5>k=K1t;J~%q{wC9B;iR+1PDA)(R%u
zO+;v6Z0_(km9(RFp^AZsWA;p19m$ndqaH6
z7bn3IVnF+JX@;ddnV=Wd`7}{uu%NLMX`e(m3YK`nS|<4f@pQ&00qOlCDE_!x?NGsTHg@l#L2`5D9ck9~2@Tz@E^9(tmiOH8r+^MxlS<$D<}Db
ztR0CWf_p0MHpvt5iS@~Et=T677KaF_(uQzxX~3`{ks9VhVoAbXbQk`EgHByXkw}3b
zDcu5nt9HKdI5C;zQDe;_BAe2FVSVcn9xZIx0;-h^JRV}a)PL3j1JMa(jY*zR);-q2
zNH=;%>wcU3qZLL1c_P2SJjM92&=Fa37o<2z$CDX8hFENtLGm++PvmFP!cPs5U(g{R
zE|r|2P7qkRfIw-GMzJ!LJdxjo`6$6BgB&_IrYN1+iY~jwNT=}WH+64L~?Hzcjt*5Z0r=EPUs&xK+Cui$uWAYthgEm280MQZCEyZ%H~112GAur0Hk-3
zKf)QK{9>=Y4pyp91SCysF5RKW32bVNuoQB`34N2M
z$gB*M5;5h&$ze#jlO8$v)TQyO5=#lbw&UR@3;DQHsT3aNJWvOU_!w5VEA3A7&*_Kks+
zcEq{#o85%-5jw1oGfpcmM0$oqn1%8(j499Zh}5Ao=NaU!6H<;0&(fu@Y+x_OBNAt_
z*i#dzyD|ARc`~kMPWJ$@mP*YG+$;E%gg2*fDu~|2h931m7HgV;*(F~}_QA=MDLPrU
zsv;rwT<-qzY*8~g6*lYT%H6%LkT-g=BWCR}F?(^tc38J;Tb`d9UE7wy2-CJVI^054
zb++2FaZgR?9l)szURx@DLG>Wi?b==J(yLRPzVN)AbCJi-1@NH!;v^E;ByH>^1
ziUzJ)d*J)m;Tqr(Vz*Nn9YAyjXlSs1&4L54@d;`q;gyFqe7tc;;V&!qp$_^$?zQkb
z9EZuQcD+{qsQYBiA+=fkVBvu{xPpWwfR%V?`4|fDuOl5^jC!2yixwCGeBu{x
zbvc`JZI5Maj{J-v%@y2J)(%A(&ysXJ^aoJFFxx9O`ryS<{7A*E@83<$8EhA|`;+^&
zDNIY^NsnXuh#9z$EJ(=H^UkuwW@23A0Xe;T
zy4-aOw*4@CIQa66U|V;`J9|yXJ>Q#RW4{TnzKo|MK2yfWOl2^1{}6gG=NC^QX^cYr
zH*bdaxBvU9)#It|hSLDUua-MAz3GtLMEK=iMoQCl4UzD8ovL8%?`z}-wmVBrUBmg*
zg5vnHbkm!2!Uk)R-^1lJyeNur8s?4482Rh=o=wJqLwF|O?#2nQFXd4Z+H`I+9M)-3
z&h~GFnsgcGBV~N%o7N1+wYSG?nvi@3Fzt*E7_m_tR+&5dg9!DmXo$nzoMl#re(b)p
z5NgrH&3fPV{4tF0Ld9ycy6M$=KY5Z{Q0r~~MWnvm*-^B@oET*W`6fH2M){yy{YeY+
zK~$OnPp%I+H}KI;ROJao%oEP5*bP8zm51^K;K!D`Dq
z+(a8IAboY(XqE;)sx9n(xYvcRXg+#%(vdyx_tu+JwMSn!eeNXDon^Xd8`p5%?E#Lg
zYbTFI3$9^zZmhGb`6d|4g0g->vjev8;UtTbGFv1hRd+9%b0w3t?oQzlOg%JFw@P6up$$t%>@H)*C58hTELH6YnA)KbhCTg9K-)t
z#kf8}IBW=-ZnoOPJk-0{CM)GI{JTNV>c>#}@<37UMxk?obcrEfc;i`g5m(1hcj%06
zHE{FIQM_8GSyn)6k;ifwF9L7iR4d9IT5XEr=CufimB^#}rnrDJrzxdf>NuUdOp#F~
zvsn+ouYtBF@))vKv*%Vr)@Bihi_pgjc<&rvek0Ht?F+%~LB3B_I~!Mu0OxN

[U-Boot] [PATCH v2 1/2] wandboard: Use 'wandboard' vendor for the Wandboard boards

2013-05-27 Thread Otavio Salvador
The board needs to be under a 'wandboard' vendor so we can use it to
choose the proper Boot Splash image.

Signed-off-by: Otavio Salvador 
---
Changes in v2:
- Fix indenting in boards.cfg

 board/wandboard/{ => }/wandboard/Makefile| 0
 board/wandboard/{ => }/wandboard/README  | 0
 board/wandboard/{ => }/wandboard/wandboard.c | 0
 boards.cfg   | 4 ++--
 4 files changed, 2 insertions(+), 2 deletions(-)
 rename board/wandboard/{ => }/wandboard/Makefile (100%)
 rename board/wandboard/{ => }/wandboard/README (100%)
 rename board/wandboard/{ => }/wandboard/wandboard.c (100%)

diff --git a/board/wandboard/Makefile b/board/wandboard/wandboard/Makefile
similarity index 100%
rename from board/wandboard/Makefile
rename to board/wandboard/wandboard/Makefile
diff --git a/board/wandboard/README b/board/wandboard/wandboard/README
similarity index 100%
rename from board/wandboard/README
rename to board/wandboard/wandboard/README
diff --git a/board/wandboard/wandboard.c b/board/wandboard/wandboard/wandboard.c
similarity index 100%
rename from board/wandboard/wandboard.c
rename to board/wandboard/wandboard/wandboard.c
diff --git a/boards.cfg b/boards.cfg
index 2f39f26..a51165c 100644
--- a/boards.cfg
+++ b/boards.cfg
@@ -269,8 +269,8 @@ nitrogen6q   arm armv7   
nitrogen6x  boundar
 nitrogen6q2g arm armv7   nitrogen6x  
boundary   mx6
nitrogen6x:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6q2g.cfg,MX6Q,DDR_MB=2048
 nitrogen6s   arm armv7   nitrogen6x  
boundary   mx6
nitrogen6x:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6s.cfg,MX6S,DDR_MB=512
 nitrogen6s1g arm armv7   nitrogen6x  
boundary   mx6
nitrogen6x:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6s1g.cfg,MX6S,DDR_MB=1024
-wandboard_dlarm armv7   wandboard   -  
mx6 
wandboard:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6dl.cfg,MX6DL,DDR_MB=1024
-wandboard_solo  arm armv7   wandboard   -  
mx6 
wandboard:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6s.cfg,MX6S,DDR_MB=512
+wandboard_dl arm armv7   wandboard   
wandboard  mx6
wandboard:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6dl.cfg,MX6DL,DDR_MB=1024
+wandboard_solo   arm armv7   wandboard   
wandboard  mx6
wandboard:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6s.cfg,MX6S,DDR_MB=512
 omap3_overo  arm armv7   overo   - 
 omap3
 omap3_pandoraarm armv7   pandora - 
 omap3
 dig297   arm armv7   dig297  
comelitomap3
-- 
1.8.1

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


[U-Boot] [PATCH v2 2/2] wandboard: Add Boot Splash image with Wandboard logo

2013-05-27 Thread Otavio Salvador
Signed-off-by: Otavio Salvador 
---
Changes in v2:
- Move config changes to video block

 include/configs/wandboard.h |   2 ++
 tools/Makefile  |   3 +++
 tools/logos/wandboard.bmp   | Bin 0 -> 22390 bytes
 3 files changed, 5 insertions(+)
 create mode 100644 tools/logos/wandboard.bmp

diff --git a/include/configs/wandboard.h b/include/configs/wandboard.h
index d4ac086..823e17b 100644
--- a/include/configs/wandboard.h
+++ b/include/configs/wandboard.h
@@ -95,8 +95,10 @@
 #define CONFIG_SYS_CONSOLE_OVERWRITE_ROUTINE
 #define CONFIG_VIDEO_BMP_RLE8
 #define CONFIG_SPLASH_SCREEN
+#define CONFIG_SPLASH_SCREEN_ALIGN
 #define CONFIG_BMP_16BPP
 #define CONFIG_VIDEO_LOGO
+#define CONFIG_VIDEO_BMP_LOGO
 #define CONFIG_IPUV3_CLK 26000
 
 #if defined(CONFIG_MX6DL)
diff --git a/tools/Makefile b/tools/Makefile
index 889c897..5d7db14 100644
--- a/tools/Makefile
+++ b/tools/Makefile
@@ -146,6 +146,9 @@ endif
 ifeq ($(VENDOR),syteco)
 LOGO_BMP= logos/syteco.bmp
 endif
+ifeq ($(VENDOR),wandboard)
+LOGO_BMP= logos/wandboard.bmp
+endif
 ifeq ($(VENDOR),intercontrol)
 LOGO_BMP= logos/intercontrol.bmp
 endif
diff --git a/tools/logos/wandboard.bmp b/tools/logos/wandboard.bmp
new file mode 100644
index 
..7f288a8e8eee052c96b9c2c2699e2e3d3efe1758
GIT binary patch
literal 22390
zcmeHPKaBH6b|3zcL~*sEC~_rAD{-}4%c96z4!InpF_0YM;u09CT%>U6)}@ig#UX%8
zmj;r+#icM1BrXntfdhgZ;3N)k4&c&A5Wu*&aA)8Uz=bc(A-^{xuBh+Rw@dAQ{?7i^
z1FT36-+MFj=Djy>-aPIf|Jh&sIg{lNOvYl|#TsLOf~&?XS^wi#8T+`e{!mFTWa(HI
zJ32aIp69XC(^K}s3oo$MDrRY#GR|{WmL+@XrI*+r{o#+<%P+soe)5w)Wv{&Q3VZd{
zSJ`W?y~bXD{dM-n8*i{T-+Yt3_10VLr$7Bm_V(Lvvv=NkhrRpmyX>$2`fu2K@4d%<
z_Orib@4x>(`@6sY2ll}SAF!YQ{2$qeAAZPw@r!?A|NP5evX4Ic7xvM={u}%F<9}zr
z`s6>@C%^iHefsIA?6c24WB>VI|II%C{B!o}Uw^^==Zi1cmtTI#zWVAb_M6}QFZ=rI
zuh}=>{FZ(5+y7&~``z!Ic(%
z9uZ7y5JX&vl4nW0IzO9FgJ3e9htVotXQ&rE8V{@o&MT{b5?zTbj%Fjb-)$M2wMNQ=
zX0*C}cQlV;F7k7~-+CUUq|uoAlSGK+*y)6U3riPkMDW-hlCZz3MHvh)?XB
zShO9m*&C&ThxWGSf!tYBp3fclevOOiEb=0FVA|KuuIBB7)yQ6y>*L0ZeGUMRId|_g
z3@^`)?u7WgMCv5GcSy%uTeraFi-)6n6EQr5P3F?yGRiN82kYx0jeVD3vvspoZqsM)
zYP983E1eyL{9PLNp?%@29SkQ@A&#BO?SqlTwRO>0pVgkWP|xGtD%){{hQhgNlmkB8
ziW9SWPtij%gpd991d4fn=eENh;@A~u-Osqw0HUW#(5lr!L4>vg;mlNy6>XyH5e
zsypu7p-=im@Gatqx0yEe>WoU$VR}i7(pOQXg4StX_|?Oc8*RIu=Mpy!$Mc57WX1PP!`}{axn=ZYz!f>Tu;0w7hd(%rT@}i-)Kc9{{ko7NU?!JK`cm6DWPug)PXl
zm=WX5NhxwPU`3)V5d)7}d{ko9EPJ?h%EUUZq5l_|aYpv1+59brVs}
zm_DMpf%N+TSX<0o;Hv256N70zM3BUV&GaF)k!8fICGa&LVBnJme33xL3_=vydX1J-
zqEE#R5Q8gLqrv3@q95lxPK8kO&c^E&Tb$|oBJ)P8qU7;tBdH4Y$-fqjNK8#;Kvpo<
zght(tNfSK0jBRFuPb5zkhAT=XPa^RL{v3?RG41U5QWVqUV|RuTMJ?6>LK!(`cY^+d
zfbM}W0qN$n5Lm7aj1&CR`oO=$xT(R+A9y5$#Tfh>;~4^U4EFO;r{$s_Wgz8A@`2pVMCe2hR>hcy0W{y70egh^%mQR;k=}``zI+mdXA!?he;FXx
zbln!icA~6jkNHYx0Wn>c5Kj0gF>F3ZbX;13Eyj&^Fi&YJ3WqtFNG`z^Vm!%xnM*y&
zXu1`j`f{)ppCW2n_rN-MvfvZ+6M8U_^5bK)D#AewT0ya1byKn%
z)IdUf)m_OMqp$1Ev1Eht21y_bSLvN+pA!|rL#>#DU3~|Jpe9dvMa3!Bn6MMAlN2K|
z8<1EKDR=azG364zS7Nm+wn-t<$DqiMWJ+O=H74ZpX2R~3s6e&+~!RlO!5<{V)u|66M
z)ubO$!%!QbaKcNZ_XBbz#5mx*+{dRjEP5dV5>k=K1t;J~%q{wC9B;iR+1PDA)(R%u
zO+;v6Z0_(km9(RFp^AZsWA;p19m$ndqaH6
z7bn3IVnF+JX@;ddnV=Wd`7}{uu%NLMX`e(m3YK`nS|<4f@pQ&00qOlCDE_!x?NGsTHg@l#L2`5D9ck9~2@Tz@E^9(tmiOH8r+^MxlS<$D<}Db
ztR0CWf_p0MHpvt5iS@~Et=T677KaF_(uQzxX~3`{ks9VhVoAbXbQk`EgHByXkw}3b
zDcu5nt9HKdI5C;zQDe;_BAe2FVSVcn9xZIx0;-h^JRV}a)PL3j1JMa(jY*zR);-q2
zNH=;%>wcU3qZLL1c_P2SJjM92&=Fa37o<2z$CDX8hFENtLGm++PvmFP!cPs5U(g{R
zE|r|2P7qkRfIw-GMzJ!LJdxjo`6$6BgB&_IrYN1+iY~jwNT=}WH+64L~?Hzcjt*5Z0r=EPUs&xK+Cui$uWAYthgEm280MQZCEyZ%H~112GAur0Hk-3
zKf)QK{9>=Y4pyp91SCysF5RKW32bVNuoQB`34N2M
z$gB*M5;5h&$ze#jlO8$v)TQyO5=#lbw&UR@3;DQHsT3aNJWvOU_!w5VEA3A7&*_Kks+
zcEq{#o85%-5jw1oGfpcmM0$oqn1%8(j499Zh}5Ao=NaU!6H<;0&(fu@Y+x_OBNAt_
z*i#dzyD|ARc`~kMPWJ$@mP*YG+$;E%gg2*fDu~|2h931m7HgV;*(F~}_QA=MDLPrU
zsv;rwT<-qzY*8~g6*lYT%H6%LkT-g=BWCR}F?(^tc38J;Tb`d9UE7wy2-CJVI^054
zb++2FaZgR?9l)szURx@DLG>Wi?b==J(yLRPzVN)AbCJi-1@NH!;v^E;ByH>^1
ziUzJ)d*J)m;Tqr(Vz*Nn9YAyjXlSs1&4L54@d;`q;gyFqe7tc;;V&!qp$_^$?zQkb
z9EZuQcD+{qsQYBiA+=fkVBvu{xPpWwfR%V?`4|fDuOl5^jC!2yixwCGeBu{x
zbvc`JZI5Maj{J-v%@y2J)(%A(&ysXJ^aoJFFxx9O`ryS<{7A*E@83<$8EhA|`;+^&
zDNIY^NsnXuh#9z$EJ(=H^UkuwW@23A0Xe;T
zy4-aOw*4@CIQa66U|V;`J9|yXJ>Q#RW4{TnzKo|MK2yfWOl2^1{}6gG=NC^QX^cYr
zH*bdaxBvU9)#It|hSLDUua-MAz3GtLMEK=iMoQCl4UzD8ovL8%?`z}-wmVBrUBmg*
zg5vnHbkm!2!Uk)R-^1lJyeNur8s?4482Rh=o=wJqLwF|O?#2nQFXd4Z+H`I+9M)-3
z&h~GFnsgcGBV~N%o7N1+wYSG?nvi@3Fzt*E7_m_tR+&5dg9!DmXo$nzoMl#re(b)p
z5NgrH&3fPV{4tF0Ld9ycy6M$=KY5Z{Q0r~~MWnvm*-^B@oET*W`6fH2M){yy{YeY+
zK~$OnPp%I+H}KI;ROJao%oEP5*bP8zm51^K;K!D`Dq
z+(a8IAboY(XqE;)sx9n(xYvcRXg+#%(vdyx_tu+JwMSn!eeNXDon^Xd8`p5%?E#Lg
zYbTFI3$9^zZmhGb`6d|4g0g->vjev8;UtTbGFv1hRd+9%b0w3t?oQzlOg%JFw@P6up$$t%>@H)*C58hTELH6YnA)KbhCTg9K-)t
z#kf8}IBW=-ZnoOPJk-0{CM)GI{JTNV>c>#}@<37UMxk?obcrEfc;i`g5m(

Re: [U-Boot] [PATCH] Wandboard: Add support for Wandboard quad

2013-05-27 Thread Otavio Salvador
On Mon, May 27, 2013 at 9:13 AM, Otavio Salvador wrote:

> On Mon, May 27, 2013 at 1:13 AM, Tapani Utriainen 
> wrote:
>
>>
>> Add support for the Wandboard quad.
>>
>> Signed-off-by: Tapani Utriainen 
>>
>> ---
>>  boards.cfg  | 1 +
>>  include/configs/wandboard.h | 2 ++
>>  2 files changed, 3 insertions(+)
>>
>> diff --git a/boards.cfg b/boards.cfg
>> index 31483d6..b907409 100644
>> --- a/boards.cfg
>> +++ b/boards.cfg
>> @@ -268,6 +268,7 @@ nitrogen6s   arm armv7
>> nitrogen6x  boundar
>>  nitrogen6s1g arm armv7   nitrogen6x
>>  boundary   mx6
>>  
>> nitrogen6x:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6s1g.cfg,MX6S,DDR_MB=1024
>>  wandboard_dlarm armv7   wandboard
>> -  mx6
>> wandboard:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6dl.cfg,MX6DL,DDR_MB=1024
>>  wandboard_solo  arm armv7   wandboard
>> -  mx6
>> wandboard:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6s.cfg,MX6S,DDR_MB=512
>> +wandboard_quad  arm armv7   wandboard
>> -  mx6
>> wandboard:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6q2g.cfg,MX6Q,DDR_MB=2048
>>
>
> Please put quad, dl and solo in order. It is now dl, solo, quad.
>
>
>>  cm_t35   arm armv7   cm_t35
>>  -  omap3
>>  omap3_overo  arm armv7   overo
>> -  omap3
>>  omap3_pandoraarm armv7   pandora
>> -  omap3
>> diff --git a/include/configs/wandboard.h b/include/configs/wandboard.h
>> index 120e3f6..cf22042 100644
>> --- a/include/configs/wandboard.h
>> +++ b/include/configs/wandboard.h
>> @@ -85,6 +85,8 @@
>>  #define CONFIG_DEFAULT_FDT_FILE"imx6dl-wandboard.dtb"
>>  #elif defined(CONFIG_MX6S)
>>  #define CONFIG_DEFAULT_FDT_FILE"imx6s-wandboard.dtb"
>> +#elif defined(CONFIG_MX6Q)
>> +#define CONFIG_DEFAULT_FDT_FILE"imx6q-wandboard.dtb"
>>  #endif
>>
>
> Same here.
>
>
>>  #define CONFIG_EXTRA_ENV_SETTINGS \
>>
>
I have sent two patches for adding the Boot Splash with Wandboard logo.
Please take a look on them and if you agree please base on this to avoid
conflicts.

(For easyness: http://patchwork.ozlabs.org/bundle/otavio/wandboard-20130527/
)

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 1/2] wandboard: Use 'wandboard' vendor for the Wandboard boards

2013-05-27 Thread Otavio Salvador
On Mon, May 27, 2013 at 6:23 PM, Wolfgang Denk  wrote:

> Dear Otavio Salvador,
>
> In message <1369673122-20455-1-git-send-email-ota...@ossystems.com.br>
> you wrote:
> > The board needs to be under a 'wandboard' vendor so we can use it to
> > choose the proper Boot Splash image.
> >
> > Signed-off-by: Otavio Salvador 
> > ---
> > Changes in v2:
> > - Fix indenting in boards.cfg
>
> NAK as for v1.


How do you suggest me to handle it? I'd like to have the logo file used
when building wandboard.

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 2/2] wandboard: Add Boot Splash image with Wandboard logo

2013-05-27 Thread Otavio Salvador
On Mon, May 27, 2013 at 6:22 PM, Wolfgang Denk  wrote:

> Dear Otavio Salvador,
>
> In message <1369672312-18774-2-git-send-email-ota...@ossystems.com.br>
> you wrote:
> > Signed-off-by: Otavio Salvador 
> > ---
> >  include/configs/wandboard.h |   3 +++
> >  tools/Makefile  |   3 +++
> >  tools/logos/wandboard.bmp   | Bin 0 -> 22390 bytes
> >  3 files changed, 6 insertions(+)
> >  create mode 100644 tools/logos/wandboard.bmp
> ...
> > --- a/tools/Makefile
> > +++ b/tools/Makefile
> > @@ -146,6 +146,9 @@ endif
> >  ifeq ($(VENDOR),syteco)
> >  LOGO_BMP= logos/syteco.bmp
> >  endif
> > +ifeq ($(VENDOR),wandboard)
> > +LOGO_BMP= logos/wandboard.bmp
> > +endif
>
> This does not scale; it never did, but now we're running out of
> control.  Can we please fix the logo selection? Thanks.
>

I can work on this but how you think we can make it?

Like check for vendor, than board?

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v3 1/2] build: Use generic boot logo matching

2013-05-27 Thread Otavio Salvador
The boot logo matching is now done in following way:

 - use LOGO_BMP if it is set, or
 - use $(BOARD).bmp if it exists in tools/logos, or
 - use $(VENDOR).bmp if it exists in tools/logos, or
 - use denx.bmp otherwise.

Signed-off-by: Otavio Salvador 
---
Changes in v3:
- New patch, which replaces the 1/2 from v1 and v2

Changes in v2: None

 tools/Makefile | 27 ++-
 1 file changed, 10 insertions(+), 17 deletions(-)

diff --git a/tools/Makefile b/tools/Makefile
index 889c897..93f4a84 100644
--- a/tools/Makefile
+++ b/tools/Makefile
@@ -128,28 +128,21 @@ LOGO-$(CONFIG_LCD_LOGO) += $(LOGO_DATA_H)
 LOGO-$(CONFIG_VIDEO_LOGO) += $(LOGO_H)
 LOGO-$(CONFIG_VIDEO_LOGO) += $(LOGO_DATA_H)
 
+# Generic logo
 ifeq ($(LOGO_BMP),)
 LOGO_BMP= logos/denx.bmp
+
+# Use board logo and fallback to vendor
+ifneq ($(wildcard logos/$(BOARD).bmp),)
+LOGO_BMP= logos/$(BOARD).bmp
+else
+ifneq ($(wildcard logos/$(VENDOR).bmp),)
+LOGO_BMP= logos/$(VENDOR).bmp
 endif
-ifeq ($(VENDOR),atmel)
-LOGO_BMP= logos/atmel.bmp
-endif
-ifeq ($(VENDOR),esd)
-LOGO_BMP= logos/esd.bmp
-endif
-ifeq ($(VENDOR),freescale)
-LOGO_BMP= logos/freescale.bmp
-endif
-ifeq ($(VENDOR),ronetix)
-LOGO_BMP= logos/ronetix.bmp
-endif
-ifeq ($(VENDOR),syteco)
-LOGO_BMP= logos/syteco.bmp
-endif
-ifeq ($(VENDOR),intercontrol)
-LOGO_BMP= logos/intercontrol.bmp
 endif
 
+endif # !LOGO_BMP
+
 # now $(obj) is defined
 HOSTSRCS += $(addprefix $(SRCTREE)/,$(EXT_OBJ_FILES-y:.o=.c))
 HOSTSRCS += $(addprefix $(SRCTREE)/tools/,$(OBJ_FILES-y:.o=.c))
-- 
1.8.1

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


[U-Boot] [PATCH v3 2/2] wandboard: Add Boot Splash image with Wandboard logo

2013-05-27 Thread Otavio Salvador
Signed-off-by: Otavio Salvador 
---
Changes in v3:
- Remove Makefile change as it is not need anymore

Changes in v2:
- Move config changes to video block

 include/configs/wandboard.h |   2 ++
 tools/logos/wandboard.bmp   | Bin 0 -> 22390 bytes
 2 files changed, 2 insertions(+)
 create mode 100644 tools/logos/wandboard.bmp

diff --git a/include/configs/wandboard.h b/include/configs/wandboard.h
index d4ac086..823e17b 100644
--- a/include/configs/wandboard.h
+++ b/include/configs/wandboard.h
@@ -95,8 +95,10 @@
 #define CONFIG_SYS_CONSOLE_OVERWRITE_ROUTINE
 #define CONFIG_VIDEO_BMP_RLE8
 #define CONFIG_SPLASH_SCREEN
+#define CONFIG_SPLASH_SCREEN_ALIGN
 #define CONFIG_BMP_16BPP
 #define CONFIG_VIDEO_LOGO
+#define CONFIG_VIDEO_BMP_LOGO
 #define CONFIG_IPUV3_CLK 26000
 
 #if defined(CONFIG_MX6DL)
diff --git a/tools/logos/wandboard.bmp b/tools/logos/wandboard.bmp
new file mode 100644
index 
..7f288a8e8eee052c96b9c2c2699e2e3d3efe1758
GIT binary patch
literal 22390
zcmeHPKaBH6b|3zcL~*sEC~_rAD{-}4%c96z4!InpF_0YM;u09CT%>U6)}@ig#UX%8
zmj;r+#icM1BrXntfdhgZ;3N)k4&c&A5Wu*&aA)8Uz=bc(A-^{xuBh+Rw@dAQ{?7i^
z1FT36-+MFj=Djy>-aPIf|Jh&sIg{lNOvYl|#TsLOf~&?XS^wi#8T+`e{!mFTWa(HI
zJ32aIp69XC(^K}s3oo$MDrRY#GR|{WmL+@XrI*+r{o#+<%P+soe)5w)Wv{&Q3VZd{
zSJ`W?y~bXD{dM-n8*i{T-+Yt3_10VLr$7Bm_V(Lvvv=NkhrRpmyX>$2`fu2K@4d%<
z_Orib@4x>(`@6sY2ll}SAF!YQ{2$qeAAZPw@r!?A|NP5evX4Ic7xvM={u}%F<9}zr
z`s6>@C%^iHefsIA?6c24WB>VI|II%C{B!o}Uw^^==Zi1cmtTI#zWVAb_M6}QFZ=rI
zuh}=>{FZ(5+y7&~``z!Ic(%
z9uZ7y5JX&vl4nW0IzO9FgJ3e9htVotXQ&rE8V{@o&MT{b5?zTbj%Fjb-)$M2wMNQ=
zX0*C}cQlV;F7k7~-+CUUq|uoAlSGK+*y)6U3riPkMDW-hlCZz3MHvh)?XB
zShO9m*&C&ThxWGSf!tYBp3fclevOOiEb=0FVA|KuuIBB7)yQ6y>*L0ZeGUMRId|_g
z3@^`)?u7WgMCv5GcSy%uTeraFi-)6n6EQr5P3F?yGRiN82kYx0jeVD3vvspoZqsM)
zYP983E1eyL{9PLNp?%@29SkQ@A&#BO?SqlTwRO>0pVgkWP|xGtD%){{hQhgNlmkB8
ziW9SWPtij%gpd991d4fn=eENh;@A~u-Osqw0HUW#(5lr!L4>vg;mlNy6>XyH5e
zsypu7p-=im@Gatqx0yEe>WoU$VR}i7(pOQXg4StX_|?Oc8*RIu=Mpy!$Mc57WX1PP!`}{axn=ZYz!f>Tu;0w7hd(%rT@}i-)Kc9{{ko7NU?!JK`cm6DWPug)PXl
zm=WX5NhxwPU`3)V5d)7}d{ko9EPJ?h%EUUZq5l_|aYpv1+59brVs}
zm_DMpf%N+TSX<0o;Hv256N70zM3BUV&GaF)k!8fICGa&LVBnJme33xL3_=vydX1J-
zqEE#R5Q8gLqrv3@q95lxPK8kO&c^E&Tb$|oBJ)P8qU7;tBdH4Y$-fqjNK8#;Kvpo<
zght(tNfSK0jBRFuPb5zkhAT=XPa^RL{v3?RG41U5QWVqUV|RuTMJ?6>LK!(`cY^+d
zfbM}W0qN$n5Lm7aj1&CR`oO=$xT(R+A9y5$#Tfh>;~4^U4EFO;r{$s_Wgz8A@`2pVMCe2hR>hcy0W{y70egh^%mQR;k=}``zI+mdXA!?he;FXx
zbln!icA~6jkNHYx0Wn>c5Kj0gF>F3ZbX;13Eyj&^Fi&YJ3WqtFNG`z^Vm!%xnM*y&
zXu1`j`f{)ppCW2n_rN-MvfvZ+6M8U_^5bK)D#AewT0ya1byKn%
z)IdUf)m_OMqp$1Ev1Eht21y_bSLvN+pA!|rL#>#DU3~|Jpe9dvMa3!Bn6MMAlN2K|
z8<1EKDR=azG364zS7Nm+wn-t<$DqiMWJ+O=H74ZpX2R~3s6e&+~!RlO!5<{V)u|66M
z)ubO$!%!QbaKcNZ_XBbz#5mx*+{dRjEP5dV5>k=K1t;J~%q{wC9B;iR+1PDA)(R%u
zO+;v6Z0_(km9(RFp^AZsWA;p19m$ndqaH6
z7bn3IVnF+JX@;ddnV=Wd`7}{uu%NLMX`e(m3YK`nS|<4f@pQ&00qOlCDE_!x?NGsTHg@l#L2`5D9ck9~2@Tz@E^9(tmiOH8r+^MxlS<$D<}Db
ztR0CWf_p0MHpvt5iS@~Et=T677KaF_(uQzxX~3`{ks9VhVoAbXbQk`EgHByXkw}3b
zDcu5nt9HKdI5C;zQDe;_BAe2FVSVcn9xZIx0;-h^JRV}a)PL3j1JMa(jY*zR);-q2
zNH=;%>wcU3qZLL1c_P2SJjM92&=Fa37o<2z$CDX8hFENtLGm++PvmFP!cPs5U(g{R
zE|r|2P7qkRfIw-GMzJ!LJdxjo`6$6BgB&_IrYN1+iY~jwNT=}WH+64L~?Hzcjt*5Z0r=EPUs&xK+Cui$uWAYthgEm280MQZCEyZ%H~112GAur0Hk-3
zKf)QK{9>=Y4pyp91SCysF5RKW32bVNuoQB`34N2M
z$gB*M5;5h&$ze#jlO8$v)TQyO5=#lbw&UR@3;DQHsT3aNJWvOU_!w5VEA3A7&*_Kks+
zcEq{#o85%-5jw1oGfpcmM0$oqn1%8(j499Zh}5Ao=NaU!6H<;0&(fu@Y+x_OBNAt_
z*i#dzyD|ARc`~kMPWJ$@mP*YG+$;E%gg2*fDu~|2h931m7HgV;*(F~}_QA=MDLPrU
zsv;rwT<-qzY*8~g6*lYT%H6%LkT-g=BWCR}F?(^tc38J;Tb`d9UE7wy2-CJVI^054
zb++2FaZgR?9l)szURx@DLG>Wi?b==J(yLRPzVN)AbCJi-1@NH!;v^E;ByH>^1
ziUzJ)d*J)m;Tqr(Vz*Nn9YAyjXlSs1&4L54@d;`q;gyFqe7tc;;V&!qp$_^$?zQkb
z9EZuQcD+{qsQYBiA+=fkVBvu{xPpWwfR%V?`4|fDuOl5^jC!2yixwCGeBu{x
zbvc`JZI5Maj{J-v%@y2J)(%A(&ysXJ^aoJFFxx9O`ryS<{7A*E@83<$8EhA|`;+^&
zDNIY^NsnXuh#9z$EJ(=H^UkuwW@23A0Xe;T
zy4-aOw*4@CIQa66U|V;`J9|yXJ>Q#RW4{TnzKo|MK2yfWOl2^1{}6gG=NC^QX^cYr
zH*bdaxBvU9)#It|hSLDUua-MAz3GtLMEK=iMoQCl4UzD8ovL8%?`z}-wmVBrUBmg*
zg5vnHbkm!2!Uk)R-^1lJyeNur8s?4482Rh=o=wJqLwF|O?#2nQFXd4Z+H`I+9M)-3
z&h~GFnsgcGBV~N%o7N1+wYSG?nvi@3Fzt*E7_m_tR+&5dg9!DmXo$nzoMl#re(b)p
z5NgrH&3fPV{4tF0Ld9ycy6M$=KY5Z{Q0r~~MWnvm*-^B@oET*W`6fH2M){yy{YeY+
zK~$OnPp%I+H}KI;ROJao%oEP5*bP8zm51^K;K!D`Dq
z+(a8IAboY(XqE;)sx9n(xYvcRXg+#%(vdyx_tu+JwMSn!eeNXDon^Xd8`p5%?E#Lg
zYbTFI3$9^zZmhGb`6d|4g0g->vjev8;UtTbGFv1hRd+9%b0w3t?oQzlOg%JFw@P6up$$t%>@H)*C58hTELH6YnA)KbhCTg9K-)t
z#kf8}IBW=-ZnoOPJk-0{CM)GI{JTNV>c>#}@<37UMxk?obcrEfc;i`g5m(1hcj%06
zHE{FIQM_8GSyn)6k;ifwF9L7iR4d9IT5XEr=CufimB^#}rnrDJrzxdf>NuUdOp#F~
zvsn+ouYtBF@))vKv*%Vr)@Bihi_pgjc<&rvek0Ht?F+%~LB3B_I~!Mu0OxNf2je!f
znrTyZiS!EE{hp`7g?6lDy
zPFBcmT1{o*IK9$?Cb{N2M5+hV84Nmx{s1D=3RQV=Pru@|292Y8k9R&6B
zS`NnT4dApYcDCCbFP8~nx8v=DFk~LENE^krjEPKmrBsu&xxcrs*#qi~6=e;{V#dvO
zc

Re: [U-Boot] [PATCH v2 1/2] wandboard: Use 'wandboard' vendor for the Wandboard boards

2013-05-27 Thread Otavio Salvador
On Mon, May 27, 2013 at 6:27 PM, Otavio Salvador wrote:

>
>
>
> On Mon, May 27, 2013 at 6:23 PM, Wolfgang Denk  wrote:
>
>> Dear Otavio Salvador,
>>
>> In message <1369673122-20455-1-git-send-email-ota...@ossystems.com.br>
>> you wrote:
>> > The board needs to be under a 'wandboard' vendor so we can use it to
>> > choose the proper Boot Splash image.
>> >
>> > Signed-off-by: Otavio Salvador 
>> > ---
>> > Changes in v2:
>> > - Fix indenting in boards.cfg
>>
>> NAK as for v1.
>
>
> How do you suggest me to handle it? I'd like to have the logo file used
> when building wandboard.
>

I sent a v3 doing this. Please give it a look.

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2] Add support for Wandboard quad.

2013-05-28 Thread Otavio Salvador
On Tue, May 28, 2013 at 5:04 AM, Tapani Utriainen wrote:

>
> Add support for Wandboard quad.
>
> Signed-off-by: Tapani Utriainen 
>

Acked-by: Otavio Salvador 

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] Wandboard: Add support for Wandboard quad

2013-05-28 Thread Otavio Salvador
On Tue, May 28, 2013 at 4:44 AM, Tapani Utriainen wrote:

> On Mon, 27 May 2013 13:56:30 -0300
> Otavio Salvador  wrote:
>
> > I have sent two patches for adding the Boot Splash with Wandboard logo.
> > Please take a look on them and if you agree please base on this to avoid
> > conflicts.
> >
> > (For easyness:
> http://patchwork.ozlabs.org/bundle/otavio/wandboard-20130527/
> > )
> >
>
> Otavio,
>
> while I do agree with the work you are doing (HDMI bootlogo into mainline),
> and would not have minded to base my patches on top of yours.
> However, the patches you sent seem to have been NAK:ed. The v3 does not
> apply
> on top of 2013.04 (maybe I am basing on the wrong release!?).
>
> So for now, I am basing the resubmission on the 2013.04 release.
>

You must base on imx/master as this is the custodian tree for upcoming
2013.07 release.

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] mmc: fsl_esdhc: Fix hang after 'save' command

2013-05-28 Thread Otavio Salvador
On Tue, May 28, 2013 at 3:09 PM, Fabio Estevam
wrote:

> Since commit 48e0b2bd (powerpc/esdhc: Correct judgement for DATA PIO mode)
> we see mx6 systems to hang after doing a 'save' command.
>
> Revert this commit since the original 'ifdef' logic from 7b43db92
> (drivers/mmc/fsl_esdhc.c: fix compiler warnings) was the correct one.
>
> Reported-by: Tapani Utriainen 
> Signed-off-by: Fabio Estevam 
>

This seem to affect 'master' branch, not 'imx' tree. Is that correct?

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2] Add support for Wandboard quad.

2013-06-03 Thread Otavio Salvador
On Mon, Jun 3, 2013 at 8:21 AM, Wolfgang Denk  wrote:

> Dear Stefano Babic,
>
> In message <51ac61a6.9050...@denx.de> you wrote:
> >
> > It is better to fix it directly into this patch. Can you fix it by
> > casting to phys_size_t and resubmit ?
> >
> >   gd->ram_size = (phys_size_t)CONFIG_DDR_MB * SZ_1M;
>
> Please do NOT use these terrible  SZ_*  defines.  Never, ever.
>
> They are dangerous and should be avoided.
>

Do you mind to explain why?

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] Maintain U-Boot splash until application is loaded

2013-06-04 Thread Otavio Salvador
Hello,

I am working in a product and we'd like to keep the U-Boot splash until the
application has been loaded so we have a graceful initialization. This
project uses MX23 (so mxs SoC family).

I tried to check the U-Boot mailing list archive for it but I couldn't find
anything useful. I also did look at kernel source and it seems
CONFIG_FB_PRE_INIT_FB is available for only *one* board.

Could someone advice me how to do that?

Regards,

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] vf610twr: Drop unneeded 'or' operator

2013-06-05 Thread Otavio Salvador
On Wed, Jun 5, 2013 at 6:24 PM, Benoît Thébaudeau <
benoit.thebaud...@advansee.com> wrote:

> Hi Fabio,
>
> On Wednesday, June 5, 2013 11:17:45 PM, Fabio Estevam wrote:
> > From: Fabio Estevam 
> >
> > No need to use the or operator, so just remove it.
> >
> > Signed-off-by: Fabio Estevam 
> > ---
> >  board/freescale/vf610twr/vf610twr.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/board/freescale/vf610twr/vf610twr.c
> > b/board/freescale/vf610twr/vf610twr.c
> > index 04fa882..1156194 100644
> > --- a/board/freescale/vf610twr/vf610twr.c
> > +++ b/board/freescale/vf610twr/vf610twr.c
> > @@ -308,7 +308,7 @@ int board_mmc_init(bd_t *bis)
> >   imx_iomux_v3_setup_multiple_pads(
> >   esdhc1_pads, ARRAY_SIZE(esdhc1_pads));
> >
> > - status |= fsl_esdhc_initialize(bis, &esdhc_cfg[0]);
> > + status = fsl_esdhc_initialize(bis, &esdhc_cfg[0]);
> >
> >   return status;
> >  }
> > --
> > 1.8.1.2
>
> Or even better, completely drop status and use
> "return fsl_esdhc_initialize(bis, &esdhc_cfg[0]);".
>

+1 :)

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2] vf610twr: Drop unneeded 'status' variable

2013-06-05 Thread Otavio Salvador
On Wed, Jun 5, 2013 at 6:34 PM, Fabio Estevam  wrote:

> From: Fabio Estevam 
>
> No need to use the 'status' variable, so just remove it.
>
> Signed-off-by: Fabio Estevam 
>

Reviewed-by: Otavio Salvador 

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] patches never reviewed

2013-06-06 Thread Otavio Salvador
On Thu, Jun 6, 2013 at 11:20 AM, Angelo Dureghello wrote:
...

> I don't want to accuse anyone of any form of discrimination, but i
> stop to contribute for now, almost for m68k branch.
>

I understand your disappointment and I agree in most of your complains. At
same time, I also think we should turn the page and start over. May you
reconsider and do a last try?

 * fix the board addition as already pointed out
 * check your other patches to see if you spot something wrong
 * send it again.

Please also ensure you add (again) the maintainer in Cc so you'll have your
side done and the duty of review and check it will be in the custodian side.

Friendly,

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] bmp: Respect "splashpos" if it is defined

2013-06-06 Thread Otavio Salvador
Signed-off-by: Otavio Salvador 
---
 common/cmd_bmp.c | 23 +++
 1 file changed, 23 insertions(+)

diff --git a/common/cmd_bmp.c b/common/cmd_bmp.c
index 5a52edd..48e045e 100644
--- a/common/cmd_bmp.c
+++ b/common/cmd_bmp.c
@@ -33,6 +33,10 @@
 #include 
 #include 
 
+#ifdef CONFIG_SPLASH_SCREEN_ALIGN
+#define BMP_ALIGN_CENTER   0x7FFF
+#endif
+
 static int bmp_info (ulong addr);
 
 /*
@@ -112,6 +116,25 @@ static int do_bmp_display(cmd_tbl_t * cmdtp, int flag, int 
argc, char * const ar
 {
ulong addr;
int x = 0, y = 0;
+   __maybe_unused char *s;
+
+#ifdef CONFIG_SPLASH_SCREEN_ALIGN
+   s = getenv("splashpos");
+   if (s != NULL) {
+   if (s[0] == 'm')
+   x = BMP_ALIGN_CENTER;
+   else
+   x = simple_strtol(s, NULL, 0);
+
+   s = strchr(s + 1, ',');
+   if (s != NULL) {
+   if (s[1] == 'm')
+   y = BMP_ALIGN_CENTER;
+   else
+   y = simple_strtol(s + 1, NULL, 0);
+   }
+   }
+#endif /* CONFIG_SPLASH_SCREEN_ALIGN */
 
switch (argc) {
case 1: /* use load_addr as default address */
-- 
1.8.1

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


Re: [U-Boot] [PATCH] bmp: Respect "splashpos" if it is defined

2013-06-06 Thread Otavio Salvador
Dear Anatolij,

On Thu, Jun 6, 2013 at 2:57 PM, Otavio Salvador wrote:

> Signed-off-by: Otavio Salvador 
>

I forgot to add you in Cc; sorry.

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] mmc: mx6qsabrelite: fsl_esdhc: Define maximum bus width supported by SabreLite board

2013-06-06 Thread Otavio Salvador
On Thu, Jun 6, 2013 at 5:25 PM, Eric Nelson  wrote:

> Hi Abbas,
>
> On 03/25/2013 04:02 AM, Abbas Raza wrote:
>
>> Hi,
>>
>> On 03/21/2013 09:17 PM, Eric Nelson wrote:
>>
>>> Thanks Abbas,
>>>
>>> Tested-by: Eric Nelson 
>>> 
>>> >
>>>
>>> Note that the board settings also apply to nitrogen6x and wandboard
>>> if you want to expand the patch a bit:
>>>
>>>  http://git.denx.de/u-boot.git/**?p=u-boot/u-boot-imx.git;a=**
>>> blob;f=board/boundary/**nitrogen6x/nitrogen6x.c;h=**
>>> 229c2378396d43a365d6682af35c0e**519ccd69d4;hb=HEAD#l304<http://git.denx.de/u-boot.git/?p=u-boot/u-boot-imx.git;a=blob;f=board/boundary/nitrogen6x/nitrogen6x.c;h=229c2378396d43a365d6682af35c0e519ccd69d4;hb=HEAD#l304>
>>>  http://git.denx.de/u-boot.git/**?p=u-boot/u-boot-imx.git;a=**
>>> blob;f=board/wandboard/**wandboard.c;h=**d95189f77de4c3573ec30c53e04fe1*
>>> *b083c1d2b5;hb=HEAD#l107<http://git.denx.de/u-boot.git/?p=u-boot/u-boot-imx.git;a=blob;f=board/wandboard/wandboard.c;h=d95189f77de4c3573ec30c53e04fe1b083c1d2b5;hb=HEAD#l107>
>>>
>>
>> I would like to expand the patch but i cannot test it for boards you
>> mentioned above. Are you sure these boards have 4-bit mmc buses?
>>
>>
> I'm certain about the Nitrogen6x. Less so about Wandboard.
>
> I think I looked it up, but can't recall precisely.


I can test a patch for Wandboard :)

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] 2013.07-rc1 plans?

2013-06-09 Thread Otavio Salvador
Hello,

I'd like to know what the plains regarding rc releases?

Regards,

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] 2013.07-rc1 plans?

2013-06-10 Thread Otavio Salvador
On Mon, Jun 10, 2013 at 6:26 AM, Albert ARIBAUD
wrote:

> Hi Otavio,
>
> On Sun, 9 Jun 2013 12:54:31 -0300, Otavio Salvador
>  wrote:
>
> > Hello,
> >
> > I'd like to know what the plains regarding rc releases?
>
> Dunno right now what Tom is planning, but before it goes out, I would
> like to first finish going through my current todo on patchwork (and
> also sort out the current TI PR) so that rc1 has all ARM changes known
> to be meant for 2013.07.
>

Right; I am specially interested on this but I also believe a more clear
schedule or plan would be good.

Maybe U-Boot could adopt a more or less 'rc' release as kernel. I don't
think it needs to be weekly but every two weeks could be interesting?

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] 2013.07-rc1 plans?

2013-06-10 Thread Otavio Salvador
On Mon, Jun 10, 2013 at 12:55 PM, Tom Rini  wrote:

> On Mon, Jun 10, 2013 at 08:43:20AM -0300, Otavio Salvador wrote:
>
> > On Mon, Jun 10, 2013 at 6:26 AM, Albert ARIBAUD
> > wrote:
> >
> > > Hi Otavio,
> > >
> > > On Sun, 9 Jun 2013 12:54:31 -0300, Otavio Salvador
> > >  wrote:
> > >
> > > > Hello,
> > > >
> > > > I'd like to know what the plains regarding rc releases?
> > >
> > > Dunno right now what Tom is planning, but before it goes out, I would
> > > like to first finish going through my current todo on patchwork (and
> > > also sort out the current TI PR) so that rc1 has all ARM changes known
> > > to be meant for 2013.07.
> > >
> >
> > Right; I am specially interested on this but I also believe a more clear
> > schedule or plan would be good.
> >
> > Maybe U-Boot could adopt a more or less 'rc' release as kernel. I don't
> > think it needs to be weekly but every two weeks could be interesting?
>
> My intention has been to do more frequent rc releases.  I started out
> doing that, but have slipped a bit.  I've honestly intented to do a rc1
> for a while, but have just kept telling myself "one more thing", and not
> tagging and pushing.  I will do -rc1 this week as a reminder to everyone
> that hasn't posted pull requests that the time is coming for things
> being just too late.
>

Great; maybe we could have some cadence here? every two week? personally I
think it'd be a good time window but any cadence works for me.

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] wandboard: Fix typo in README

2013-06-13 Thread Otavio Salvador
On Thu, Jun 13, 2013 at 1:28 AM, Wolfgang Denk  wrote:

> Dear Shigeru Yoshida,
>
> In message <
> camqom_0gl2czu_xm-c9_zonmgmyi1m6w_ysxuv2etaebuvr...@mail.gmail.com> you
> wrote:
> >
> > diff --git a/board/wandboard/README b/board/wandboard/README
> > index e0b0b33..8bf646d 100644
> > --- a/board/wandboard/README
> > +++ b/board/wandboard/README
> > @@ -14,12 +14,12 @@ Building U-boot for Wandboard
> >
> >  To build U-Boot for the Wandboard Dual Lite version:
> >
> > -$ make wanboard_dl_config
> > +$ make WANBOARD_DL_config
> >  $ make
> >
> >  To build U-Boot for the Wandboard Solo version:
> >
> > -$ make wanboard_solo_config
> > +$ make WANBOARD_SOLO_config
> >  $ make
>
> Both "fixes" are wrong.  First, the target names are _really lower
> case.  Second, the name is wan>d missing.  Please change into:
>
> $ make wandboard_dl_config
> resp.
>     $ make wandboard_solo_config
>

http://git.denx.de/?p=u-boot/u-boot-imx.git;a=commitdiff;h=8c4983779ed50090b74d56498be17cde31892a64


Already fixed.

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] ARM: mxs: Make the console buffer smaller

2013-06-17 Thread Otavio Salvador
On Mon, Jun 17, 2013 at 10:07 AM, Stefano Babic  wrote:
> Hallo Marek,
>
> On 17/06/2013 14:51, Marek Vasut wrote:
>
>>> I am missing something: which is the real advantage to reduce the
>>> console buffer ? I do not think that the saved memory is worth, and on
>>> the other side more elaborated scripts (usings nested if-then-else) are
>>> quite long nowadays.
>>
>> True, but so far they didn't overflow this limit I believe. Some of them are
>> hanging on the verge of blowing it though, good point.
>
> I think that a point to consider is if all scripts are already provided
> or we let the user/owner of the board to add his own scripts, as I
> presume. It is pity if he cannot do what u-boot really supports only to
> save some bytes.

I think this is more than enough to justify a bigger buffer.


--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] bmp: Respect "splashpos" if it is defined

2013-06-17 Thread Otavio Salvador
On Thu, Jun 6, 2013 at 2:57 PM, Otavio Salvador  wrote:
> Signed-off-by: Otavio Salvador 

Ping?

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] gpio: Question about gpio_set_value() implementation in spear_gpio.c

2013-06-19 Thread Otavio Salvador
On Wed, Jun 19, 2013 at 10:44 AM, Axel Lin  wrote:
> Current code looks strange because no matter the value argument is 0 or 1
> it always calls
> writel(1 << gpio, ®s->gpiodata[DATA_REG_ADDR(gpio)]);
>
> And then gpio_get_value() always return 1.
>
> I'm wondering if it needs to be fixed, something like below change:
>
> diff --git a/drivers/gpio/spear_gpio.c b/drivers/gpio/spear_gpio.c
> index d3c728e..8878608 100644
> --- a/drivers/gpio/spear_gpio.c
> +++ b/drivers/gpio/spear_gpio.c
> @@ -52,7 +52,10 @@ int gpio_set_value(unsigned gpio, int value)
>  {
> struct gpio_regs *regs = (struct gpio_regs *)CONFIG_GPIO_BASE;
>
> -   writel(1 << gpio, ®s->gpiodata[DATA_REG_ADDR(gpio)]);
> +   if (value)
> +   writel(1 << gpio, ®s->gpiodata[DATA_REG_ADDR(gpio)]);
> +   else
> +   writel(0, ®s->gpiodata[DATA_REG_ADDR(gpio)]);
>
> return 0;
>  }

writel(value << gpio, ®s->gpiodata[DATA_REG_ADDR(gpio)]);

Should do no? This would avoid the if block.

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 1/2] PHY: micrel: helper function for KSZ9031

2013-06-20 Thread Otavio Salvador
On Thu, Jun 20, 2013 at 8:26 AM, SARTRE Leo  wrote:
> micrel.c: Add function ksz9031_phy_extended_write, helper to MDD
>   register write
> micrel.h: Prototype and define MDD registers
>
> Signed-off-by: Leo Sartre 
> ---
>  drivers/net/phy/micrel.c |   29 +++--
>  include/micrel.h |7 +++
>  2 files changed, 34 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/net/phy/micrel.c b/drivers/net/phy/micrel.c
> index 2a8b6cb..5b5c11b 100644
> --- a/drivers/net/phy/micrel.c
> +++ b/drivers/net/phy/micrel.c
> @@ -85,7 +85,7 @@ static int ksz90xx_startup(struct phy_device *phydev)
>  }
>  #ifdef CONFIG_PHY_MICREL_KSZ9021
>
> -/*
> +/**
>   * KSZ9021
>   */

Please drop this change; not need.

> @@ -148,9 +148,34 @@ static struct phy_driver ksz9021_driver = {
>  };
>  #endif
>
> -/*
> +/**
>   * KSZ9031
>   */

Please drop this change; not need.

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 2/2] ARM: Congatec: Ethernet: Add support for cgtqmx6qeval

2013-06-20 Thread Otavio Salvador
IO_NR(3, 23), 0);
> +   gpio_direction_output(IMX_GPIO_NR(6, 30), 1);
> +   gpio_direction_output(IMX_GPIO_NR(6, 25), 1);
> +   gpio_direction_output(IMX_GPIO_NR(6, 27), 1);
> +   gpio_direction_output(IMX_GPIO_NR(6, 28), 1);
> +   gpio_direction_output(IMX_GPIO_NR(6, 29), 1);
> +   imx_iomux_v3_setup_multiple_pads(enet_pads1, ARRAY_SIZE(enet_pads1));
> +   gpio_direction_output(IMX_GPIO_NR(6, 24), 1);
> +
> +   /* Need delay 10ms according to KSZ9031 spec */
> +   udelay(1000 * 10);
> +   gpio_set_value(IMX_GPIO_NR(3, 23), 1);
> +   imx_iomux_v3_setup_multiple_pads(enet_pads2, ARRAY_SIZE(enet_pads2));
> +}
> +
> +int board_phy_config(struct phy_device *phydev)
> +{
> +   ksz9031_phy_extended_write(phydev, 2,
> +   MII_KSZ9031_EXT_RGMII_CTRL_SIG_SKEW,
> +   0x);
> +   ksz9031_phy_extended_write(phydev, 2,
> +   MII_KSZ9031_EXT_RGMII_RX_DATA_SKEW,
> +   0x);
> +   ksz9031_phy_extended_write(phydev, 2,
> +   MII_KSZ9031_EXT_RGMII_TX_DATA_SKEW,
> +   0x);
> +   ksz9031_phy_extended_write(phydev, 2,
> +   MII_KSZ9031_EXT_RGMII_CLOCK_SKEW,
> +   0x3FFF);
> +
> +   phydev->drv->config(phydev);
> +
> +   return 0;
> +}
> +
> +int board_eth_init(bd_t *bis)
> +{
> +   uint32_t base = IMX_FEC_BASE;
> +   struct mii_dev *bus = NULL;
> +   struct phy_device *phydev = NULL;
> +   int ret;
> +
> +   setup_iomux_enet();
> +
> +#ifdef CONFIG_FEC_MXC

Shouldn't this cover the whole function?

> +   bus = fec_get_miibus(base, -1);
> +   if (!bus)
> +   return 0;
> +   /* scan phy 6*/
> +   phydev = phy_find_by_mask(bus, (0xf << 6), PHY_INTERFACE_MODE_RGMII);
> +
> +   if (!phydev) {
> +   free(bus);
> +   printf("No phy found\n");
> +   return 0;
> +   }
> +   printf("using phy at %d\n", phydev->addr);
> +   ret  = fec_probe(bis, -1, base, bus, phydev);
> +   if (ret) {
> +   printf("FEC MXC: %s:failed\n", __func__);
> +   free(phydev);
> +   free(bus);
> +   }
> +#endif
> +   return 0;
> +}


--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2] Add support for Wandboard quad.

2013-06-20 Thread Otavio Salvador
Hello Tapani,

On Tue, May 28, 2013 at 5:04 AM, Tapani Utriainen  wrote:
>
> Add support for Wandboard quad.
>
> Signed-off-by: Tapani Utriainen 

Can you handle the comments done by Wolfgang and Stefano?

It'd be very good to have it merged for 2013.07 but we're getting short on time.

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 3/3] ARM: Congatec: README update

2013-06-20 Thread Otavio Salvador
On Thu, Jun 20, 2013 at 10:46 AM, SARTRE Leo
 wrote:
> README: U-boot works both on SPI-NOR and SDcard
>
> Signed-off-by: Leo Sartre 

Acked-by: Otavio Salvador 

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 1/3] PHY: micrel: helper function for KSZ9031

2013-06-20 Thread Otavio Salvador
On Thu, Jun 20, 2013 at 10:46 AM, SARTRE Leo
 wrote:
> micrel.c: Add function ksz9031_phy_extended_write, helper to MDD
>   register write
> micrel.h: Prototype and define MDD registers
>
> Signed-off-by: Leo Sartre 

Acked-by: Otavio Salvador 

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 2/3] ARM: Congatec: Ethernet: Add support for cgtqmx6qeval

2013-06-20 Thread Otavio Salvador
On Thu, Jun 20, 2013 at 10:46 AM, SARTRE Leo
 wrote:
> cgtqmx6eval.c: add Micrel KSZ9031  Ethernet transceiver
>
> Signed-off-by: Leo Sartre 

Acked-by: Otavio Salvador 

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 5/6] mx28evk: Move README file inside board directory

2013-06-26 Thread Otavio Salvador
On Wed, Jun 26, 2013 at 12:43 PM, Stefano Babic  wrote:
> On 03/05/2013 20:08, Fabio Estevam wrote:
>> From: Fabio Estevam 
>>
>> Board specific READMEs should be located inside the respective board 
>> directory.
>>
>> Signed-off-by: Fabio Estevam 
>> ---
>>  .../freescale/mx28evk/README   |0
>>  1 file changed, 0 insertions(+), 0 deletions(-)
>>  rename doc/README.mx28evk => board/freescale/mx28evk/README (100%)
>>
>> diff --git a/doc/README.mx28evk b/board/freescale/mx28evk/README
>> similarity index 100%
>> rename from doc/README.mx28evk
>> rename to board/freescale/mx28evk/README
>>
>
> Strange, I cannot find this patch in patchworks..anyway
>
> Applied to u-boot-imx, thanks.

This is because it was  a rename so patchwork does not track it. It is a bug ...

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v3] Add support for Wandboard Quad

2013-06-26 Thread Otavio Salvador
On Wed, Jun 26, 2013 at 11:14 AM, Stefano Babic  wrote:
> Patch does not apply cleanly because your mailer or your editor replaced
> original  with whitespaces. I change back again, but please take
> care of this issue for the next time.

Tapani, the easiest way of sending the patches is using git send-email
command. This avoid this problems with the e-mail client.

> Apart of that:
>
> Acked-by: Stefano Babic 
>
> This patch was pending for a while, I will push it still for the release.

Thanks Stefano.

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 2/2] vf610twr: Remove SoC name from U-Boot prompt

2013-06-26 Thread Otavio Salvador
We've been dropping SoC name from U-Boot prompt as it increase
complexity for automatic testing and makes line longer for no good
reason.

Signed-off-by: Otavio Salvador 
---
 include/configs/vf610twr.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/configs/vf610twr.h b/include/configs/vf610twr.h
index bda5ddf..f45d73c 100644
--- a/include/configs/vf610twr.h
+++ b/include/configs/vf610twr.h
@@ -147,7 +147,7 @@
 #define CONFIG_SYS_LONGHELP/* undef to save memory */
 #define CONFIG_SYS_HUSH_PARSER /* use "hush" command parser */
 #define CONFIG_SYS_PROMPT_HUSH_PS2 "> "
-#define CONFIG_SYS_PROMPT  "Vybrid U-Boot > "
+#define CONFIG_SYS_PROMPT  "=> "
 #undef CONFIG_AUTO_COMPLETE
 #define CONFIG_SYS_CBSIZE  256 /* Console I/O Buffer Size */
 #define CONFIG_SYS_PBSIZE  \
-- 
1.8.3.1

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


[U-Boot] [PATCH 1/2] vf610twr: Add default environment in line with other Freescale boards

2013-06-26 Thread Otavio Salvador
Signed-off-by: Otavio Salvador 
---
 include/configs/vf610twr.h | 58 ++
 1 file changed, 58 insertions(+)

diff --git a/include/configs/vf610twr.h b/include/configs/vf610twr.h
index 77fe893..bda5ddf 100644
--- a/include/configs/vf610twr.h
+++ b/include/configs/vf610twr.h
@@ -85,6 +85,64 @@
 
 #define CONFIG_SYS_TEXT_BASE   0x3f008000
 
+#define CONFIG_EXTRA_ENV_SETTINGS \
+   "script=boot.scr\0" \
+   "uimage=uImage\0" \
+   "console=ttymxc0\0" \
+   "initrd_high=0x\0" \
+   "ip_dyn=yes\0" \
+   "mmcdev=" __stringify(CONFIG_SYS_MMC_ENV_DEV) "\0" \
+   "mmcpart=1\0" \
+   "mmcroot=/dev/mmcblk0p2 rootwait rw\0" \
+   "update_sd_firmware_filename=u-boot.imx\0" \
+   "update_sd_firmware=" \
+   "if test ${ip_dyn} = yes; then " \
+   "setenv get_cmd dhcp; " \
+   "else " \
+   "setenv get_cmd tftp; " \
+   "fi; " \
+   "if mmc dev ${mmcdev}; then "   \
+   "if ${get_cmd} ${update_sd_firmware_filename}; then " \
+   "setexpr fw_sz ${filesize} / 0x200; " \
+   "setexpr fw_sz ${fw_sz} + 1; "  \
+   "mmc write ${loadaddr} 0x2 ${fw_sz}; " \
+   "fi; "  \
+   "fi\0" \
+   "mmcargs=setenv bootargs console=${console},${baudrate} " \
+   "root=${mmcroot}\0" \
+   "loadbootscript=" \
+   "fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${script};\0" \
+   "bootscript=echo Running bootscript from mmc ...; " \
+   "source\0" \
+   "loaduimage=fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${uimage}\0" \
+   "mmcboot=echo Booting from mmc ...; " \
+   "run mmcargs; " \
+   "bootm\0" \
+   "netargs=setenv bootargs console=${console},${baudrate} " \
+   "root=/dev/nfs " \
+   "ip=dhcp nfsroot=${serverip}:${nfsroot},v3,tcp\0" \
+   "netboot=echo Booting from net ...; " \
+   "run netargs; " \
+   "if test ${ip_dyn} = yes; then " \
+   "setenv get_cmd dhcp; " \
+   "else " \
+   "setenv get_cmd tftp; " \
+   "fi; " \
+   "${get_cmd} ${uimage}; " \
+   "bootm\0"
+
+#define CONFIG_BOOTCOMMAND \
+  "mmc dev ${mmcdev}; if mmc rescan; then " \
+  "if run loadbootscript; then " \
+  "run bootscript; " \
+  "else " \
+  "if run loaduimage; then " \
+  "run mmcboot; " \
+  "else run netboot; " \
+  "fi; " \
+  "fi; " \
+  "else run netboot; fi"
+
 /* Miscellaneous configurable options */
 #define CONFIG_SYS_LONGHELP/* undef to save memory */
 #define CONFIG_SYS_HUSH_PARSER /* use "hush" command parser */
-- 
1.8.3.1

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


Re: [U-Boot] [PATCH] MAINTAINERS: Add an entry to the mx6q wandboard variant

2013-06-26 Thread Otavio Salvador
On Wed, Jun 26, 2013 at 1:53 PM, Fabio Estevam
 wrote:
> Signed-off-by: Fabio Estevam 

Acked-by: Otavio Salvador 

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 1/2] vf610twr: Add default environment in line with other Freescale boards

2013-06-26 Thread Otavio Salvador
On Wed, Jun 26, 2013 at 1:59 PM, Fabio Estevam  wrote:
> On Wed, Jun 26, 2013 at 1:40 PM, Otavio Salvador
>  wrote:
>> Signed-off-by: Otavio Salvador 
>> ---
>>  include/configs/vf610twr.h | 58 
>> ++
>>  1 file changed, 58 insertions(+)
>>
>> diff --git a/include/configs/vf610twr.h b/include/configs/vf610twr.h
>> index 77fe893..bda5ddf 100644
>> --- a/include/configs/vf610twr.h
>> +++ b/include/configs/vf610twr.h
>> @@ -85,6 +85,64 @@
>>
>>  #define CONFIG_SYS_TEXT_BASE   0x3f008000
>>
>> +#define CONFIG_EXTRA_ENV_SETTINGS \
>> +   "script=boot.scr\0" \
>> +   "uimage=uImage\0" \
>> +   "console=ttymxc0\0" \
>
> Vybrid does not use ttymxc0.

Andy can you confirm which one we should use here?

It should be /dev/ttymxc1 right?

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v2 1/2] vf610twr: Add default environment in line with other Freescale boards

2013-06-26 Thread Otavio Salvador
Signed-off-by: Otavio Salvador 
---
Changes in v2:
- fix console for ttymxc1 (Fabio)

 include/configs/vf610twr.h | 58 ++
 1 file changed, 58 insertions(+)

diff --git a/include/configs/vf610twr.h b/include/configs/vf610twr.h
index 77fe893..a944984 100644
--- a/include/configs/vf610twr.h
+++ b/include/configs/vf610twr.h
@@ -85,6 +85,64 @@
 
 #define CONFIG_SYS_TEXT_BASE   0x3f008000
 
+#define CONFIG_EXTRA_ENV_SETTINGS \
+   "script=boot.scr\0" \
+   "uimage=uImage\0" \
+   "console=ttymxc1\0" \
+   "initrd_high=0x\0" \
+   "ip_dyn=yes\0" \
+   "mmcdev=" __stringify(CONFIG_SYS_MMC_ENV_DEV) "\0" \
+   "mmcpart=1\0" \
+   "mmcroot=/dev/mmcblk0p2 rootwait rw\0" \
+   "update_sd_firmware_filename=u-boot.imx\0" \
+   "update_sd_firmware=" \
+   "if test ${ip_dyn} = yes; then " \
+   "setenv get_cmd dhcp; " \
+   "else " \
+   "setenv get_cmd tftp; " \
+   "fi; " \
+   "if mmc dev ${mmcdev}; then "   \
+   "if ${get_cmd} ${update_sd_firmware_filename}; then " \
+   "setexpr fw_sz ${filesize} / 0x200; " \
+   "setexpr fw_sz ${fw_sz} + 1; "  \
+   "mmc write ${loadaddr} 0x2 ${fw_sz}; " \
+   "fi; "  \
+   "fi\0" \
+   "mmcargs=setenv bootargs console=${console},${baudrate} " \
+   "root=${mmcroot}\0" \
+   "loadbootscript=" \
+   "fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${script};\0" \
+   "bootscript=echo Running bootscript from mmc ...; " \
+   "source\0" \
+   "loaduimage=fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${uimage}\0" \
+   "mmcboot=echo Booting from mmc ...; " \
+   "run mmcargs; " \
+   "bootm\0" \
+   "netargs=setenv bootargs console=${console},${baudrate} " \
+   "root=/dev/nfs " \
+   "ip=dhcp nfsroot=${serverip}:${nfsroot},v3,tcp\0" \
+   "netboot=echo Booting from net ...; " \
+   "run netargs; " \
+   "if test ${ip_dyn} = yes; then " \
+   "setenv get_cmd dhcp; " \
+   "else " \
+   "setenv get_cmd tftp; " \
+   "fi; " \
+   "${get_cmd} ${uimage}; " \
+   "bootm\0"
+
+#define CONFIG_BOOTCOMMAND \
+  "mmc dev ${mmcdev}; if mmc rescan; then " \
+  "if run loadbootscript; then " \
+  "run bootscript; " \
+  "else " \
+  "if run loaduimage; then " \
+  "run mmcboot; " \
+  "else run netboot; " \
+  "fi; " \
+  "fi; " \
+  "else run netboot; fi"
+
 /* Miscellaneous configurable options */
 #define CONFIG_SYS_LONGHELP/* undef to save memory */
 #define CONFIG_SYS_HUSH_PARSER /* use "hush" command parser */
-- 
1.8.3.1

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


[U-Boot] [PATCH v2 2/2] vf610twr: Remove SoC name from U-Boot prompt

2013-06-26 Thread Otavio Salvador
We've been dropping SoC name from U-Boot prompt as it increase
complexity for automatic testing and makes line longer for no good
reason.

Signed-off-by: Otavio Salvador 
---
Changes in v2: None

 include/configs/vf610twr.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/configs/vf610twr.h b/include/configs/vf610twr.h
index a944984..bb1ccd6 100644
--- a/include/configs/vf610twr.h
+++ b/include/configs/vf610twr.h
@@ -147,7 +147,7 @@
 #define CONFIG_SYS_LONGHELP/* undef to save memory */
 #define CONFIG_SYS_HUSH_PARSER /* use "hush" command parser */
 #define CONFIG_SYS_PROMPT_HUSH_PS2 "> "
-#define CONFIG_SYS_PROMPT  "Vybrid U-Boot > "
+#define CONFIG_SYS_PROMPT  "=> "
 #undef CONFIG_AUTO_COMPLETE
 #define CONFIG_SYS_CBSIZE  256 /* Console I/O Buffer Size */
 #define CONFIG_SYS_PBSIZE  \
-- 
1.8.3.1

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


Re: [U-Boot] [PATCH v2 1/2] vf610twr: Add default environment in line with other Freescale boards

2013-06-26 Thread Otavio Salvador
On Wed, Jun 26, 2013 at 2:16 PM, Fabio Estevam  wrote:
> On Wed, Jun 26, 2013 at 2:12 PM, Otavio Salvador
>  wrote:
>
>> +   "console=ttymxc1\0" \
>
> Vybrid does not use the same serial driver as i.MX.
>
> In the kernel the serial driver is this one:
> https://git.kernel.org/cgit/linux/kernel/git/gregkh/tty.git/tree/drivers/tty/serial/fsl_lpuart.c?h=tty-next
>
> ,and the driver name is "ttyLP".
>
> Vybrid starts numbering the UART ports from 0, so the device number in
> Linux matches the hardware port number.
>
> This means that the correct is "console=ttyLP1" for this board to boot
> a mainline kernel.

This is being done using Timesys kernel as reference; so I did used
his kernel as reference.

This is the kernel which has been send for inclusion in Yocto BSP so I
think I will stay with it for now.

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 1/2] vf610twr: Add default environment in line with other Freescale boards

2013-06-26 Thread Otavio Salvador
On Wed, Jun 26, 2013 at 2:27 PM, Fabio Estevam  wrote:
> On Wed, Jun 26, 2013 at 2:19 PM, Otavio Salvador
>  wrote:
>
>> This is being done using Timesys kernel as reference; so I did used
>> his kernel as reference.
>
> So it is better to fix the tty driver name in the Timesys kernel to
> ttyLP then, instead of adding a console option that is incorrect.

Yes; I think I agree. Mostly like we did for mxs in Freescale kernel.
Andy, can you check it?

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v3 2/2] vf610twr: Remove SoC name from U-Boot prompt

2013-06-26 Thread Otavio Salvador
We've been dropping SoC name from U-Boot prompt as it increase
complexity for automatic testing and makes line longer for no good
reason.

Signed-off-by: Otavio Salvador 
---
Changes in v3: None
Changes in v2: None

 include/configs/vf610twr.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/configs/vf610twr.h b/include/configs/vf610twr.h
index 94c69a6..6848b3f 100644
--- a/include/configs/vf610twr.h
+++ b/include/configs/vf610twr.h
@@ -177,7 +177,7 @@
 #define CONFIG_SYS_LONGHELP/* undef to save memory */
 #define CONFIG_SYS_HUSH_PARSER /* use "hush" command parser */
 #define CONFIG_SYS_PROMPT_HUSH_PS2 "> "
-#define CONFIG_SYS_PROMPT  "Vybrid U-Boot > "
+#define CONFIG_SYS_PROMPT  "=> "
 #undef CONFIG_AUTO_COMPLETE
 #define CONFIG_SYS_CBSIZE  256 /* Console I/O Buffer Size */
 #define CONFIG_SYS_PBSIZE  \
-- 
1.8.3.1

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


[U-Boot] [PATCH v3 1/2] vf610twr: Add default environment in line with other Freescale boards

2013-06-26 Thread Otavio Salvador
This adds a default environment which should be able to support both
3.0.15 from Timesys and upcoming 3.11.

Signed-off-by: Otavio Salvador 
---
Changes in v3:
- add dt boot support
- use ttyLP1 as console
- change loadaddr and sys text as iMX6SL
- NOTE: It needs to be boot tested as I don't have the board

Changes in v2:
- fix console for ttymxc1 (Fabio)

 include/configs/vf610twr.h | 93 --
 1 file changed, 90 insertions(+), 3 deletions(-)

diff --git a/include/configs/vf610twr.h b/include/configs/vf610twr.h
index 77fe893..94c69a6 100644
--- a/include/configs/vf610twr.h
+++ b/include/configs/vf610twr.h
@@ -83,7 +83,95 @@
 
 #define CONFIG_BOOTDELAY   3
 
-#define CONFIG_SYS_TEXT_BASE   0x3f008000
+#define CONFIG_LOADADDR0x8200
+#define CONFIG_SYS_TEXT_BASE   0x8780
+
+#define CONFIG_EXTRA_ENV_SETTINGS \
+   "script=boot.scr\0" \
+   "uimage=uImage\0" \
+   "console=ttyLP1\0" \
+   "fdt_high=0x\0" \
+   "initrd_high=0x\0" \
+   "fdt_file=vf610-twr.dtb\0" \
+   "fdt_addr=0x8100\0" \
+   "boot_fdt=try\0" \
+   "ip_dyn=yes\0" \
+   "mmcdev=" __stringify(CONFIG_SYS_MMC_ENV_DEV) "\0" \
+   "mmcpart=1\0" \
+   "mmcroot=/dev/mmcblk0p2 rootwait rw\0" \
+   "update_sd_firmware_filename=u-boot.imx\0" \
+   "update_sd_firmware=" \
+   "if test ${ip_dyn} = yes; then " \
+   "setenv get_cmd dhcp; " \
+   "else " \
+   "setenv get_cmd tftp; " \
+   "fi; " \
+   "if mmc dev ${mmcdev}; then "   \
+   "if ${get_cmd} ${update_sd_firmware_filename}; then " \
+   "setexpr fw_sz ${filesize} / 0x200; " \
+   "setexpr fw_sz ${fw_sz} + 1; "  \
+   "mmc write ${loadaddr} 0x2 ${fw_sz}; " \
+   "fi; "  \
+   "fi\0" \
+   "mmcargs=setenv bootargs console=${console},${baudrate} " \
+   "root=${mmcroot}\0" \
+   "loadbootscript=" \
+   "fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${script};\0" \
+   "bootscript=echo Running bootscript from mmc ...; " \
+   "source\0" \
+   "loaduimage=fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${uimage}\0" \
+   "loadfdt=fatload mmc ${mmcdev}:${mmcpart} ${fdt_addr} ${fdt_file}\0" \
+   "mmcboot=echo Booting from mmc ...; " \
+   "run mmcargs; " \
+   "if test ${boot_fdt} = yes || test ${boot_fdt} = try; then " \
+   "if run loadfdt; then " \
+   "bootm ${loadaddr} - ${fdt_addr}; " \
+   "else " \
+   "if test ${boot_fdt} = try; then " \
+   "bootm; " \
+   "else " \
+   "echo WARN: Cannot load the DT; " \
+   "fi; " \
+   "fi; " \
+   "else " \
+   "bootm; " \
+   "fi;\0" \
+   "netargs=setenv bootargs console=${console},${baudrate} " \
+   "root=/dev/nfs " \
+   "ip=dhcp nfsroot=${serverip}:${nfsroot},v3,tcp\0" \
+   "netboot=echo Booting from net ...; " \
+   "run netargs; " \
+   "if test ${ip_dyn} = yes; then " \
+   "setenv get_cmd dhcp; " \
+   "else " \
+   "setenv get_cmd tftp; " \
+   "fi; " \
+   "${get_cmd} ${uimage}; " \
+   "if test ${boot_fdt} = yes || test ${boot_fdt} = try; then " \
+   "if ${get_cmd} ${fdt_addr} ${fdt_file}; then " \
+   "bootm ${loadaddr} - ${fdt_addr}; " \
+   "else " \
+   "if test ${boot_fdt} = try; then " \
+   "bootm; " \
+   "else " \
+   "echo WARN: Cannot load the DT; " \
+   "fi; " \
+   "fi; " \
+   

Re: [U-Boot] [PATCH v2 1/2] vf610twr: Add default environment in line with other Freescale boards

2013-06-27 Thread Otavio Salvador
On Thu, Jun 27, 2013 at 5:58 AM, Stefano Babic  wrote:
> On 27/06/2013 10:00, Wang Huan-B18965 wrote:
>> Hi, stefano,
>>
>> I am ok with the default environment changes. But the sys text base 
>> should not be changed, otherwise the u-boot could not work. The reason I 
>> have explained in the other email.
>
> Of course, and I will not apply the patches until there are open comments.

I will fix this. As I don't have the board can you, Alison, provide me
the right loadaddr and fdt_addr? So we avoid many revisions.

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v4 2/2] vf610twr: Remove SoC name from U-Boot prompt

2013-06-27 Thread Otavio Salvador
We've been dropping SoC name from U-Boot prompt as it increase
complexity for automatic testing and makes line longer for no good
reason.

Signed-off-by: Otavio Salvador 
---
Changes in v4: None
Changes in v3: None
Changes in v2: None

 include/configs/vf610twr.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/configs/vf610twr.h b/include/configs/vf610twr.h
index 9474aeb..41d7d9a 100644
--- a/include/configs/vf610twr.h
+++ b/include/configs/vf610twr.h
@@ -177,7 +177,7 @@
 #define CONFIG_SYS_LONGHELP/* undef to save memory */
 #define CONFIG_SYS_HUSH_PARSER /* use "hush" command parser */
 #define CONFIG_SYS_PROMPT_HUSH_PS2 "> "
-#define CONFIG_SYS_PROMPT  "Vybrid U-Boot > "
+#define CONFIG_SYS_PROMPT  "=> "
 #undef CONFIG_AUTO_COMPLETE
 #define CONFIG_SYS_CBSIZE  256 /* Console I/O Buffer Size */
 #define CONFIG_SYS_PBSIZE  \
-- 
1.8.3.1

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


[U-Boot] [PATCH v4 1/2] vf610twr: Add default environment in line with other Freescale boards

2013-06-27 Thread Otavio Salvador
This adds a default environment which should be able to support both
3.0.15 from Timesys and upcoming 3.11.

Signed-off-by: Otavio Salvador 
---
Changes in v4:
- revert sys text address change (Alison)
- add mem=128M to bootargs/netargs (Andy)

Changes in v3:
- add dt boot support
- use ttyLP1 as console
- change loadaddr and sys text as iMX6SL
- NOTE: It needs to be boot tested as I don't have the board

Changes in v2:
- fix console for ttymxc1 (Fabio)

 include/configs/vf610twr.h | 91 +-
 1 file changed, 89 insertions(+), 2 deletions(-)

diff --git a/include/configs/vf610twr.h b/include/configs/vf610twr.h
index 77fe893..9474aeb 100644
--- a/include/configs/vf610twr.h
+++ b/include/configs/vf610twr.h
@@ -83,8 +83,96 @@
 
 #define CONFIG_BOOTDELAY   3
 
+#define CONFIG_LOADADDR0x8200
 #define CONFIG_SYS_TEXT_BASE   0x3f008000
 
+#define CONFIG_EXTRA_ENV_SETTINGS \
+   "script=boot.scr\0" \
+   "uimage=uImage\0" \
+   "console=ttyLP1\0" \
+   "fdt_high=0x\0" \
+   "initrd_high=0x\0" \
+   "fdt_file=vf610-twr.dtb\0" \
+   "fdt_addr=0x8100\0" \
+   "boot_fdt=try\0" \
+   "ip_dyn=yes\0" \
+   "mmcdev=" __stringify(CONFIG_SYS_MMC_ENV_DEV) "\0" \
+   "mmcpart=1\0" \
+   "mmcroot=/dev/mmcblk0p2 rootwait rw\0" \
+   "update_sd_firmware_filename=u-boot.imx\0" \
+   "update_sd_firmware=" \
+   "if test ${ip_dyn} = yes; then " \
+   "setenv get_cmd dhcp; " \
+   "else " \
+   "setenv get_cmd tftp; " \
+   "fi; " \
+   "if mmc dev ${mmcdev}; then "   \
+   "if ${get_cmd} ${update_sd_firmware_filename}; then " \
+   "setexpr fw_sz ${filesize} / 0x200; " \
+   "setexpr fw_sz ${fw_sz} + 1; "  \
+   "mmc write ${loadaddr} 0x2 ${fw_sz}; " \
+   "fi; "  \
+   "fi\0" \
+   "mmcargs=setenv bootargs console=${console},${baudrate} " \
+   "root=${mmcroot} mem=128M\0" \
+   "loadbootscript=" \
+   "fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${script};\0" \
+   "bootscript=echo Running bootscript from mmc ...; " \
+   "source\0" \
+   "loaduimage=fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${uimage}\0" \
+   "loadfdt=fatload mmc ${mmcdev}:${mmcpart} ${fdt_addr} ${fdt_file}\0" \
+   "mmcboot=echo Booting from mmc ...; " \
+   "run mmcargs; " \
+   "if test ${boot_fdt} = yes || test ${boot_fdt} = try; then " \
+   "if run loadfdt; then " \
+   "bootm ${loadaddr} - ${fdt_addr}; " \
+   "else " \
+   "if test ${boot_fdt} = try; then " \
+   "bootm; " \
+   "else " \
+   "echo WARN: Cannot load the DT; " \
+   "fi; " \
+   "fi; " \
+   "else " \
+   "bootm; " \
+   "fi;\0" \
+   "netargs=setenv bootargs console=${console},${baudrate} " \
+   "root=/dev/nfs mem=128M " \
+   "ip=dhcp nfsroot=${serverip}:${nfsroot},v3,tcp\0" \
+   "netboot=echo Booting from net ...; " \
+   "run netargs; " \
+   "if test ${ip_dyn} = yes; then " \
+   "setenv get_cmd dhcp; " \
+   "else " \
+   "setenv get_cmd tftp; " \
+   "fi; " \
+   "${get_cmd} ${uimage}; " \
+   "if test ${boot_fdt} = yes || test ${boot_fdt} = try; then " \
+   "if ${get_cmd} ${fdt_addr} ${fdt_file}; then " \
+   "bootm ${loadaddr} - ${fdt_addr}; " \
+   "else " \
+   "if test ${boot_fdt} = try; then " \
+   "bootm; " \
+   "else " \
+   "echo WARN: Cannot load the DT; " \
+   "

Re: [U-Boot] [PATCH v4 1/2] vf610twr: Add default environment in line with other Freescale boards

2013-06-27 Thread Otavio Salvador
On Thu, Jun 27, 2013 at 11:03 PM, Fabio Estevam  wrote:
> On Thu, Jun 27, 2013 at 10:59 PM, Otavio Salvador
>  wrote:
>> This adds a default environment which should be able to support both
>> 3.0.15 from Timesys and upcoming 3.11.
>>
>> Signed-off-by: Otavio Salvador 
>> ---
>> Changes in v4:
>> - revert sys text address change (Alison)
>> - add mem=128M to bootargs/netargs (Andy)
>
> Just curious: why do we need to pass mem=128M?

Board fails to boot otherwise ... Andy did try it so he can explain it
with more detail.

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v4 1/2] vf610twr: Add default environment in line with other Freescale boards

2013-06-28 Thread Otavio Salvador
On Fri, Jun 28, 2013 at 5:59 AM, Wang Huan-B18965  wrote:
>
>
>> -Original Message-
>> From: Fabio Estevam [mailto:feste...@gmail.com]
>> Sent: Friday, June 28, 2013 10:15 AM
>> To: Otavio Salvador
>> Cc: Andy Voltz; U-Boot Mailing List; Estevam Fabio-R49496; Stefano
>> Babic; Wang Huan-B18965
>> Subject: Re: [U-Boot] [PATCH v4 1/2] vf610twr: Add default environment
>> in line with other Freescale boards
>>
>> On Thu, Jun 27, 2013 at 11:12 PM, Otavio Salvador
>>  wrote:
>>
>> >> Just curious: why do we need to pass mem=128M?
>> >
>> > Board fails to boot otherwise ... Andy did try it so he can explain
>> it
>> > with more detail.
>>
>> Hmm...Is DDR being incorrectly configured then? We need to understand
>> why this happens.
>>
>> Andy, can you comment on this, please?
>
> [Alison Wang] mem=128M is not needed with the latest kernel.

Latest? Freescale one or mainline?

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v4 1/2] vf610twr: Add default environment in line with other Freescale boards

2013-06-28 Thread Otavio Salvador
On Fri, Jun 28, 2013 at 9:48 AM, Andy Voltz  wrote:
> Hi Fabio,
>
> On Thu, Jun 27, 2013 at 10:14:40PM -0400, Fabio Estevam wrote:
>> On Thu, Jun 27, 2013 at 11:12 PM, Otavio Salvador
>>  wrote:
>>
>> >> Just curious: why do we need to pass mem=128M?
>> >
>> > Board fails to boot otherwise ... Andy did try it so he can explain it
>> > with more detail.
>>
>> Hmm...Is DDR being incorrectly configured then? We need to understand
>> why this happens.
>>
>> Andy, can you comment on this, please?
>
> I haven't looked closely at the memory setup, but you are likely correct.
>
> When I boot our kernel on twr-vf610 without mem, nothing is seen after
> 'Starting kernel...'
>
> For comparison I booted linux-next on the twr-vf610 without the mem argument,
> and it works. I haven't gotten the board to mount an RFS over net or mmc, but 
> I
> suspect that is a kernel configuration problem.
>
> It probably makes the most sense to just drop mem arg from the environment, 
> and
> then we can patch our kernel to fix the memory setup issue.

Ok; I can cook a v5 without mem param and send.

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v5 2/2] vf610twr: Remove SoC name from U-Boot prompt

2013-06-28 Thread Otavio Salvador
We've been dropping SoC name from U-Boot prompt as it increase
complexity for automatic testing and makes line longer for no good
reason.

Signed-off-by: Otavio Salvador 
---
Changes in v5: None
Changes in v4: None
Changes in v3: None
Changes in v2: None

 include/configs/vf610twr.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/configs/vf610twr.h b/include/configs/vf610twr.h
index af772ca..5012fc8 100644
--- a/include/configs/vf610twr.h
+++ b/include/configs/vf610twr.h
@@ -177,7 +177,7 @@
 #define CONFIG_SYS_LONGHELP/* undef to save memory */
 #define CONFIG_SYS_HUSH_PARSER /* use "hush" command parser */
 #define CONFIG_SYS_PROMPT_HUSH_PS2 "> "
-#define CONFIG_SYS_PROMPT  "Vybrid U-Boot > "
+#define CONFIG_SYS_PROMPT  "=> "
 #undef CONFIG_AUTO_COMPLETE
 #define CONFIG_SYS_CBSIZE  256 /* Console I/O Buffer Size */
 #define CONFIG_SYS_PBSIZE  \
-- 
1.8.3.1

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


[U-Boot] [PATCH v5 1/2] vf610twr: Add default environment in line with other Freescale boards

2013-06-28 Thread Otavio Salvador
This adds a default environment which should be able to support both
3.0.15 from Timesys and upcoming 3.11.

Signed-off-by: Otavio Salvador 
---
Changes in v5:
- remove mem=128M param (Alison/Andy/Fabio)

Changes in v4:
- revert sys text address change (Alison)
- add mem=128M to bootargs/netargs (Andy)

Changes in v3:
- add dt boot support
- use ttyLP1 as console
- change loadaddr and sys text as iMX6SL
- NOTE: It needs to be boot tested as I don't have the board

Changes in v2:
- fix console for ttymxc1 (Fabio)

 include/configs/vf610twr.h | 91 +-
 1 file changed, 89 insertions(+), 2 deletions(-)

diff --git a/include/configs/vf610twr.h b/include/configs/vf610twr.h
index 77fe893..af772ca 100644
--- a/include/configs/vf610twr.h
+++ b/include/configs/vf610twr.h
@@ -83,8 +83,96 @@
 
 #define CONFIG_BOOTDELAY   3
 
+#define CONFIG_LOADADDR0x8200
 #define CONFIG_SYS_TEXT_BASE   0x3f008000
 
+#define CONFIG_EXTRA_ENV_SETTINGS \
+   "script=boot.scr\0" \
+   "uimage=uImage\0" \
+   "console=ttyLP1\0" \
+   "fdt_high=0x\0" \
+   "initrd_high=0x\0" \
+   "fdt_file=vf610-twr.dtb\0" \
+   "fdt_addr=0x8100\0" \
+   "boot_fdt=try\0" \
+   "ip_dyn=yes\0" \
+   "mmcdev=" __stringify(CONFIG_SYS_MMC_ENV_DEV) "\0" \
+   "mmcpart=1\0" \
+   "mmcroot=/dev/mmcblk0p2 rootwait rw\0" \
+   "update_sd_firmware_filename=u-boot.imx\0" \
+   "update_sd_firmware=" \
+   "if test ${ip_dyn} = yes; then " \
+   "setenv get_cmd dhcp; " \
+   "else " \
+   "setenv get_cmd tftp; " \
+   "fi; " \
+   "if mmc dev ${mmcdev}; then "   \
+   "if ${get_cmd} ${update_sd_firmware_filename}; then " \
+   "setexpr fw_sz ${filesize} / 0x200; " \
+   "setexpr fw_sz ${fw_sz} + 1; "  \
+   "mmc write ${loadaddr} 0x2 ${fw_sz}; " \
+   "fi; "  \
+   "fi\0" \
+   "mmcargs=setenv bootargs console=${console},${baudrate} " \
+   "root=${mmcroot}\0" \
+   "loadbootscript=" \
+   "fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${script};\0" \
+   "bootscript=echo Running bootscript from mmc ...; " \
+   "source\0" \
+   "loaduimage=fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${uimage}\0" \
+   "loadfdt=fatload mmc ${mmcdev}:${mmcpart} ${fdt_addr} ${fdt_file}\0" \
+   "mmcboot=echo Booting from mmc ...; " \
+   "run mmcargs; " \
+   "if test ${boot_fdt} = yes || test ${boot_fdt} = try; then " \
+   "if run loadfdt; then " \
+   "bootm ${loadaddr} - ${fdt_addr}; " \
+   "else " \
+   "if test ${boot_fdt} = try; then " \
+   "bootm; " \
+   "else " \
+   "echo WARN: Cannot load the DT; " \
+   "fi; " \
+   "fi; " \
+   "else " \
+   "bootm; " \
+   "fi;\0" \
+   "netargs=setenv bootargs console=${console},${baudrate} " \
+   "root=/dev/nfs " \
+   "ip=dhcp nfsroot=${serverip}:${nfsroot},v3,tcp\0" \
+   "netboot=echo Booting from net ...; " \
+   "run netargs; " \
+   "if test ${ip_dyn} = yes; then " \
+   "setenv get_cmd dhcp; " \
+   "else " \
+   "setenv get_cmd tftp; " \
+   "fi; " \
+   "${get_cmd} ${uimage}; " \
+   "if test ${boot_fdt} = yes || test ${boot_fdt} = try; then " \
+   "if ${get_cmd} ${fdt_addr} ${fdt_file}; then " \
+   "bootm ${loadaddr} - ${fdt_addr}; " \
+   "else " \
+   "if test ${boot_fdt} = try; then " \
+   "bootm; " \
+   "else " \
+   "echo WARN: Cannot load the DT;

[U-Boot] [PATCH 0/6] Upstream environment changes being used in Yocto/OE

2013-06-28 Thread Otavio Salvador
This changes were being done in every version of U-Boot and it
makes sense to try to merge them upstream.

Please review them and ack/nack them.


Otavio Salvador (6):
  mx53loco: Change default environment to cope with OE changes
  mx6qsabrelite: Change default environment to cope with OE changes
  mx6slevk: Change default environment to cope with OE changes
  wandboard: Change default environment to cope with OE changes
  mx51evk: Change default environment to cope with OE changes
  mx53ard: Change default environment to cope with OE changes

 include/configs/mx51evk.h   | 4 ++--
 include/configs/mx53ard.h   | 4 ++--
 include/configs/mx53loco.h  | 4 ++--
 include/configs/mx6qsabrelite.h | 4 ++--
 include/configs/mx6slevk.h  | 4 ++--
 include/configs/wandboard.h | 4 ++--
 6 files changed, 12 insertions(+), 12 deletions(-)

-- 
1.8.3.1

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


[U-Boot] [PATCH 1/6] mx53loco: Change default environment to cope with OE changes

2013-06-28 Thread Otavio Salvador
OpenEmbedded has change partitioning layout of generated image so it
does not raise warnings during the boot regarding unkown partition
being used for U-Boot.

Signed-off-by: Otavio Salvador 
---
 include/configs/mx53loco.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/configs/mx53loco.h b/include/configs/mx53loco.h
index a4b610f..cc31e9b 100644
--- a/include/configs/mx53loco.h
+++ b/include/configs/mx53loco.h
@@ -124,8 +124,8 @@
"boot_fdt=try\0" \
"ip_dyn=yes\0" \
"mmcdev=0\0" \
-   "mmcpart=2\0" \
-   "mmcroot=/dev/mmcblk0p3 rw rootwait\0" \
+   "mmcpart=1\0" \
+   "mmcroot=/dev/mmcblk0p2 rw rootwait\0" \
"mmcargs=setenv bootargs console=ttymxc0,${baudrate} root=${mmcroot}\0" 
\
"loadbootscript=" \
"fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${script};\0" \
-- 
1.8.3.1

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


[U-Boot] [PATCH 2/6] mx6qsabrelite: Change default environment to cope with OE changes

2013-06-28 Thread Otavio Salvador
OpenEmbedded has change partitioning layout of generated image so it
does not raise warnings during the boot regarding unkown partition
being used for U-Boot.

Signed-off-by: Otavio Salvador 
---
 include/configs/mx6qsabrelite.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/configs/mx6qsabrelite.h b/include/configs/mx6qsabrelite.h
index b814418..c7db81d 100644
--- a/include/configs/mx6qsabrelite.h
+++ b/include/configs/mx6qsabrelite.h
@@ -170,8 +170,8 @@
"boot_fdt=try\0" \
"ip_dyn=yes\0" \
"mmcdev=0\0" \
-   "mmcpart=2\0" \
-   "mmcroot=/dev/mmcblk0p3 rootwait rw\0" \
+   "mmcpart=1\0" \
+   "mmcroot=/dev/mmcblk0p2 rootwait rw\0" \
"mmcargs=setenv bootargs console=${console},${baudrate} " \
"root=${mmcroot}\0" \
"loadbootscript=" \
-- 
1.8.3.1

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


[U-Boot] [PATCH 3/6] mx6slevk: Change default environment to cope with OE changes

2013-06-28 Thread Otavio Salvador
OpenEmbedded has change partitioning layout of generated image so it
does not raise warnings during the boot regarding unkown partition
being used for U-Boot.

Signed-off-by: Otavio Salvador 
---
 include/configs/mx6slevk.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/configs/mx6slevk.h b/include/configs/mx6slevk.h
index 19dcdd6..55e3ad9 100644
--- a/include/configs/mx6slevk.h
+++ b/include/configs/mx6slevk.h
@@ -73,8 +73,8 @@
"boot_fdt=try\0" \
"ip_dyn=yes\0" \
"mmcdev=0\0" \
-   "mmcpart=2\0" \
-   "mmcroot=/dev/mmcblk0p3 rootwait rw\0" \
+   "mmcpart=1\0" \
+   "mmcroot=/dev/mmcblk0p2 rootwait rw\0" \
"mmcargs=setenv bootargs console=${console},${baudrate} " \
"root=${mmcroot}\0" \
"loadbootscript=" \
-- 
1.8.3.1

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


[U-Boot] [PATCH 6/6] mx53ard: Change default environment to cope with OE changes

2013-06-28 Thread Otavio Salvador
OpenEmbedded has change partitioning layout of generated image so it
does not raise warnings during the boot regarding unkown partition
being used for U-Boot.

Signed-off-by: Otavio Salvador 
---
 include/configs/mx53ard.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/configs/mx53ard.h b/include/configs/mx53ard.h
index b0a965f..fa160e4 100644
--- a/include/configs/mx53ard.h
+++ b/include/configs/mx53ard.h
@@ -118,8 +118,8 @@
"boot_fdt=try\0" \
"ip_dyn=yes\0" \
"mmcdev=" __stringify(CONFIG_SYS_MMC_ENV_DEV) "\0" \
-   "mmcpart=2\0" \
-   "mmcroot=/dev/mmcblk0p3 rootwait rw\0" \
+   "mmcpart=1\0" \
+   "mmcroot=/dev/mmcblk0p2 rootwait rw\0" \
"update_sd_firmware_filename=u-boot.imx\0" \
"update_sd_firmware=" \
"if test ${ip_dyn} = yes; then " \
-- 
1.8.3.1

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


[U-Boot] [PATCH 4/6] wandboard: Change default environment to cope with OE changes

2013-06-28 Thread Otavio Salvador
OpenEmbedded has change partitioning layout of generated image so it
does not raise warnings during the boot regarding unkown partition
being used for U-Boot.

Signed-off-by: Otavio Salvador 
---
 include/configs/wandboard.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/configs/wandboard.h b/include/configs/wandboard.h
index b2995d8..ee6bf21 100644
--- a/include/configs/wandboard.h
+++ b/include/configs/wandboard.h
@@ -120,8 +120,8 @@
"boot_fdt=try\0" \
"ip_dyn=yes\0" \
"mmcdev=" __stringify(CONFIG_SYS_MMC_ENV_DEV) "\0" \
-   "mmcpart=2\0" \
-   "mmcroot=/dev/mmcblk0p3 rootwait rw\0" \
+   "mmcpart=1\0" \
+   "mmcroot=/dev/mmcblk0p2 rootwait rw\0" \
"update_sd_firmware_filename=u-boot.imx\0" \
"update_sd_firmware=" \
"if test ${ip_dyn} = yes; then " \
-- 
1.8.3.1

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


[U-Boot] [PATCH 5/6] mx51evk: Change default environment to cope with OE changes

2013-06-28 Thread Otavio Salvador
OpenEmbedded has change partitioning layout of generated image so it
does not raise warnings during the boot regarding unkown partition
being used for U-Boot.

Signed-off-by: Otavio Salvador 
---
 include/configs/mx51evk.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/configs/mx51evk.h b/include/configs/mx51evk.h
index 13d1839..4383375d 100644
--- a/include/configs/mx51evk.h
+++ b/include/configs/mx51evk.h
@@ -162,8 +162,8 @@
"boot_fdt=try\0" \
"ip_dyn=yes\0" \
"mmcdev=0\0" \
-   "mmcpart=2\0" \
-   "mmcroot=/dev/mmcblk0p3 rootwait rw\0" \
+   "mmcpart=1\0" \
+   "mmcroot=/dev/mmcblk0p2 rootwait rw\0" \
"mmcargs=setenv bootargs console=ttymxc0,${baudrate} " \
"root=${mmcroot}\0" \
"loadbootscript=" \
-- 
1.8.3.1

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


Re: [U-Boot] [PATCH 0/6] Upstream environment changes being used in Yocto/OE

2013-07-01 Thread Otavio Salvador
On Mon, Jul 1, 2013 at 10:20 AM, Tom Rini  wrote:
> On Fri, Jun 28, 2013 at 06:56:46PM -0300, Fabio Estevam wrote:
>
>> On Fri, Jun 28, 2013 at 6:52 PM, Otavio Salvador
>>  wrote:
>> > This changes were being done in every version of U-Boot and it
>> > makes sense to try to merge them upstream.
>> >
>> > Please review them and ack/nack them.
>>
>> I think the series look good:
>>
>> Reviewed-by: Fabio Estevam 
>
> We aren't breaking compat with other distros, are we?  In TI-land where
> (for better or worse) we often have a FAT partition we're dealing with
> too, we use (and can have OE spit out) a uEnv.txt file that's
> user-editable and imported to the environment that would update these
> variables.  I suspect there's not an easy analog here but I thought I'd
> bring it up.

We've not used uEnv.txt files for this in Freescale's Yocto support
but patched the U-Boot source code to make it work out of box. I think
these patches could go in now that Freescale is using Yocto more
extensivelly for ARM platforms so we avoid  same patches to be ported
for every new U-Boot release.

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 0/6] Upstream environment changes being used in Yocto/OE

2013-07-01 Thread Otavio Salvador
On Mon, Jul 1, 2013 at 10:30 AM, Tom Rini  wrote:
> On Mon, Jul 01, 2013 at 10:25:46AM -0300, Otavio Salvador wrote:
>> On Mon, Jul 1, 2013 at 10:20 AM, Tom Rini  wrote:
>> > On Fri, Jun 28, 2013 at 06:56:46PM -0300, Fabio Estevam wrote:
>> >
>> >> On Fri, Jun 28, 2013 at 6:52 PM, Otavio Salvador
>> >>  wrote:
>> >> > This changes were being done in every version of U-Boot and it
>> >> > makes sense to try to merge them upstream.
>> >> >
>> >> > Please review them and ack/nack them.
>> >>
>> >> I think the series look good:
>> >>
>> >> Reviewed-by: Fabio Estevam 
>> >
>> > We aren't breaking compat with other distros, are we?  In TI-land where
>> > (for better or worse) we often have a FAT partition we're dealing with
>> > too, we use (and can have OE spit out) a uEnv.txt file that's
>> > user-editable and imported to the environment that would update these
>> > variables.  I suspect there's not an easy analog here but I thought I'd
>> > bring it up.
>>
>> We've not used uEnv.txt files for this in Freescale's Yocto support
>> but patched the U-Boot source code to make it work out of box. I think
>> these patches could go in now that Freescale is using Yocto more
>> extensivelly for ARM platforms so we avoid  same patches to be ported
>> for every new U-Boot release.
>
> Right well and good.  But part of the reason we use it here is that
> meta-various-ti-things may use a different layout from
> meta-community-oriented-distro and this lets us handle both with just a
> tweak in the various-ti-things layer to add a text file.  Again, may not
> apply here and that's fine.  Heck, we're trying to move more towards
> always just using the community layout.

Yes and we also have support to read a user provided file, but we
leave it for customizations not to use by default.

... yes, community layout is the way to go ;)

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] video: consolidate splash screen alignment code

2013-07-01 Thread Otavio Salvador
On Mon, Jul 1, 2013 at 7:04 PM, Anatolij Gustschin  wrote:
> Code for checking "splashpos" environment variable is
> duplicated in drivers, move it to the common function.
> Call this function also in the bmp display command to
> consider "splashpos" settings.
>
> Signed-off-by: Anatolij Gustschin 

Awesome!

Acked-by: Otavio Salvador 

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] bmp: Respect "splashpos" if it is defined

2013-07-01 Thread Otavio Salvador
On Mon, Jul 1, 2013 at 7:01 PM, Anatolij Gustschin  wrote:
> Hello Otavio,
>
> Sorry for delay.
>
> On Thu,  6 Jun 2013 14:57:23 -0300
> Otavio Salvador  wrote:
> ...
>> +#ifdef CONFIG_SPLASH_SCREEN_ALIGN
>> + s = getenv("splashpos");
>> + if (s != NULL) {
>> + if (s[0] == 'm')
>> + x = BMP_ALIGN_CENTER;
>> + else
>> + x = simple_strtol(s, NULL, 0);
>> +
>> + s = strchr(s + 1, ',');
>> + if (s != NULL) {
>> + if (s[1] == 'm')
>> + y = BMP_ALIGN_CENTER;
>> + else
>> + y = simple_strtol(s + 1, NULL, 0);
>> + }
>> + }
>> +#endif /* CONFIG_SPLASH_SCREEN_ALIGN */
>
> This code is already there in two other drivers, we should better
> add a common function for "splashpos" checking and use it here.
> I'll submit a patch shortly.

Agreed; I just acked the patch ... it looks great, thanks!

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2013-07-01 Thread Otavio Salvador
On Mon, Jul 1, 2013 at 7:13 PM, Anatolij Gustschin  wrote:
> Hello Tom,
>
> The following changes since commit e6bf18dba2a21bebf2c421b1c2e188225f6485a1:
>
>   Prepare v2013.07-rc2 (2013-06-28 18:03:51 -0400)
>
> are available in the git repository at:
>
>   git://git.denx.de/u-boot-video.git master
>
> for you to fetch changes up to 327598945b1365ca6ba3fe96c9bd45320999:
>
>   omap: cm_t35: Fix cm_t35 for weak splash_screen_prepare (2013-07-01 
> 21:47:40 +0200)
>
> 
> Piotr Wilczek (2):
>   drivers:video:s6e8ax0: change data_to_send array to static
>   lcd: align bmp header when uncopmressing image
>
> Robert Winkler (3):
>   video: lcd: Add CONFIG_SPLASH_SCREEN_PREPARE support to CONFIG_VIDEO
>   video: lcd: Make splash_screen_prepare weak, remove config macro
>   omap: cm_t35: Fix cm_t35 for weak splash_screen_prepare
>
> Stephen Warren (1):
>   lcd: remove unaligned access in lcd_dt_simplefb_configure_node()

You didn' t put the splashpos fix.

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2013-07-02 Thread Otavio Salvador
On Tue, Jul 2, 2013 at 3:00 AM, Anatolij Gustschin  wrote:
> On Mon, 1 Jul 2013 23:11:07 -0300
> Otavio Salvador  wrote:
> ...
>> You didn' t put the splashpos fix.
>
> I would prefer to wait a few days, so other people will have a chance
> to review and comment. Also I didn't test the splashpos fix yet, it
> was only compile-tested.

Ok.

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v1 0/7] The patchset fixes some issue in the generation of the imx image

2013-07-11 Thread Otavio Salvador
On Thu, Jul 11, 2013 at 10:06 AM, Stefano Babic  wrote:
> (header for Freescale's i.MX processors) to allow the usage of
> Freescale's tools to sign the u-boot image and provide a secure boot.
>
> This has nothing to do with the Secure Boot extensions implemented by
> Simon Glass, that can be in any case used to boot later a secure image.
> Freescale's secure boot ensures that a signed bootloader
> is started only if it is verified with a key that is burned into the iMX 
> fuses.
> Documentation about the Freescale's secure process can be read from the
> AN4591, available on the Freescale's Website.
>
> The patchset allows to add to the imx Header the CSF (command Sequence File)
> generated by the tools provided by Freescale. The CSF is then simply 
> concatenated
> to the u-boot image, making a signed bootloader, that the processor can verify
> if the fuses for the keys are burned. The processor (i.MX53 / i.MX6x) will not
> start a bootloader that cannot be verified - further infos how to configure
> the SOC to verify the bootloader can be found in the User Manual of the 
> specific
> SOC.
>
> Next step is to verify the kernel, that can be still done using Simon's 
> patches for
> verified boot (CONFIG_OF_CONTROL must be set in the board configuarion file).

I didn't yet test or reviewed the patchset (will do) but I must to say
a big Thank you :-) Awesome!

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [ANN] v2013.07 _not_ on Monday

2013-07-12 Thread Otavio Salvador
On Fri, Jul 12, 2013 at 3:03 PM, Tom Rini  wrote:
> Hey all,
>
> Looking things over, right now:
> - We've got one more bootm/related fixup posted today (I've confirmed it
>   works)
> - The OMAP3 SPL issue (I need to confirm just how bad it is)
> - The exynos4 clock issue
>
> Given the above, and that I'm build testing a rather large set of
> changes right now, I'm _not_ going to do the next release on Monday.  If
> all of my local tests pass, I will tag -rc3 today.  Thanks!

Agreed; it is better to delay it and have a solid release than make a
bugfix release short after it... :-)


--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] mx6qsabrelite: Remove mx6qsabrelite code in favor of nitrogen6x

2013-07-14 Thread Otavio Salvador
On Sun, Jul 14, 2013 at 11:40 PM, Fabio Estevam  wrote:
> From: Fabio Estevam 
>
> mx6qsabrelite and nitrogen6q boards are hardware compatible, so let's avoid 
> the
> code duplication and only use the nitrogen6x source code to make board code
> maintainance easier.
>
> Tested booting a mainline device tree kernel on a mx6qsabrelite board.
>
> Signed-off-by: Fabio Estevam 

I think this is a huge improvement I am just not sure about the
environment preserving for sabrelite. I think most people using
sabrelite end using the U-Boot from SPI NOR.

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


  1   2   3   4   5   6   7   8   9   10   >