Re: [PATCH 32/45] usb: ohci-at91: Check result of clk_get()

2017-03-07 Thread Sam Ravnborg 
On Tue, Mar 07, 2017 at 01:21:40PM -0800, Andrey Smirnov wrote:
> On Tue, Mar 7, 2017 at 8:35 AM, Sam Ravnborg  wrote:
> > On Mon, Mar 06, 2017 at 02:53:43PM -0800, Andrey Smirnov wrote:
> >> Signed-off-by: Andrey Smirnov 
> >> ---
> >>  drivers/usb/host/ohci-at91.c | 9 +
> >>  1 file changed, 9 insertions(+)
> >>
> >> diff --git a/drivers/usb/host/ohci-at91.c b/drivers/usb/host/ohci-at91.c
> >> index 0f5c8f1..c70d898 100644
> >> --- a/drivers/usb/host/ohci-at91.c
> >> +++ b/drivers/usb/host/ohci-at91.c
> >> @@ -47,7 +47,16 @@ static int at91_ohci_probe(struct device_d *dev)
> >>   struct ohci_regs __iomem *regs = (struct ohci_regs __iomem 
> >> *)dev->resource[0].start;
> >>
> >>   iclk = clk_get(NULL, "ohci_clk");
> >> + if (IS_ERR(iclk)) {
> >> + dev_err(dev, "Failed to get 'iclk'\n");
> > iclk is the internal name. Woudl it make more sense to use "ohci_clk" here?
> >
> 
> I was looking at the corresponding kernel driver and that's where I
> got the name for the variable. If you think 'ohci_clk' is a better
> name I have no problem changing it.

It is not something I have strong opinions about.
But I later noticed you sued "interface clk" and "function clk".
So it would be good to be consistent.

Even if the kernel is not consistent we can do better in barebox.

Sam

___
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH 00/45] AT91, at91sam9x5ek updates

2017-03-07 Thread Jean-Christophe PLAGNIOL-VILLARD 

> On Mar 8, 2017, at 5:18 AM, Andrey Smirnov  wrote:
> 
> On Tue, Mar 7, 2017 at 12:58 PM, Sam Ravnborg  wrote:
 
 When booting from an SD-Card/NOR-Flash/USB would I then need
 to use at91bootstrap, or can I use only barebox?
 
 If I move forward with this and I can find an 9263
 evaluation kit I would then also update this.
>>> On 9263 the sram is huge so you can build barebox as first stage
>> 
>> Good - simplicity is good.
>> 
> 
> For  at91sam9x5ek I had to use at91bootstrap since there is first
> stage loader in Barebox for it. I don't know about 9263, but I trust
> Jean-Christophe to be correct on this one.

I’ve patch for all the at91 soc for the pbl and nand and mmc

but I’m really busy those days on EFI boot support 
and secure boot. To be able to pass microsoft certification so we can use 
barebox
to boot linux or even windows.

So I’ll try to dig them later

Best Regards,
J.
> 
>>> 
>>> I add this a long time ago on usb_a9263_bootstrap_defconfig
>>> or other (all the EK).
>>> 
>>> This will build a barebox that can load you image from multiple source in a 
>>> sequence
>>> 
>>> and then use a std barebox
>>> 
>>> To switch to DT you will have to import the DT I did for the kernel
>> OK, should be straighforward.
>> 
>>> and replace the hw i2c by a i2c gpio as the i2c IP on at91 has a lot of 
>>> quirks
>>> and for barebox it will not make sense to port it
>> Seems less straightforward but doable.
>> 
> 
> Not sure what the situation on 9263 is, but FWIW, kernel DT file from
> the kernel for 9x5ek secifies I2C0 as GPIO bitbanged on, and I believe
> I had it working so hopefully it would be just a DT related change
> that you'd need to do.
> 
> Thanks,
> Andrey Smirnov


___
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH 32/45] usb: ohci-at91: Check result of clk_get()

2017-03-07 Thread Andrey Smirnov 
On Tue, Mar 7, 2017 at 8:35 AM, Sam Ravnborg  wrote:
> On Mon, Mar 06, 2017 at 02:53:43PM -0800, Andrey Smirnov wrote:
>> Signed-off-by: Andrey Smirnov 
>> ---
>>  drivers/usb/host/ohci-at91.c | 9 +
>>  1 file changed, 9 insertions(+)
>>
>> diff --git a/drivers/usb/host/ohci-at91.c b/drivers/usb/host/ohci-at91.c
>> index 0f5c8f1..c70d898 100644
>> --- a/drivers/usb/host/ohci-at91.c
>> +++ b/drivers/usb/host/ohci-at91.c
>> @@ -47,7 +47,16 @@ static int at91_ohci_probe(struct device_d *dev)
>>   struct ohci_regs __iomem *regs = (struct ohci_regs __iomem 
>> *)dev->resource[0].start;
>>
>>   iclk = clk_get(NULL, "ohci_clk");
>> + if (IS_ERR(iclk)) {
>> + dev_err(dev, "Failed to get 'iclk'\n");
> iclk is the internal name. Woudl it make more sense to use "ohci_clk" here?
>

I was looking at the corresponding kernel driver and that's where I
got the name for the variable. If you think 'ohci_clk' is a better
name I have no problem changing it.

>> + return PTR_ERR(iclk);
>> + }
>> +
>>   fclk = clk_get(NULL, "uhpck");
>> + if (IS_ERR(fclk)) {
>> + dev_err(dev, "Failed to get 'fclk'\n");
> Likewise - uhpck?

Ditto here.

Thanks,
Andrey Smirnov

___
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH 00/45] AT91, at91sam9x5ek updates

2017-03-07 Thread Andrey Smirnov 
On Tue, Mar 7, 2017 at 12:58 PM, Sam Ravnborg  wrote:
>> >
>> > When booting from an SD-Card/NOR-Flash/USB would I then need
>> > to use at91bootstrap, or can I use only barebox?
>> >
>> > If I move forward with this and I can find an 9263
>> > evaluation kit I would then also update this.
>> On 9263 the sram is huge so you can build barebox as first stage
>
> Good - simplicity is good.
>

For  at91sam9x5ek I had to use at91bootstrap since there is first
stage loader in Barebox for it. I don't know about 9263, but I trust
Jean-Christophe to be correct on this one.

>>
>> I add this a long time ago on usb_a9263_bootstrap_defconfig
>> or other (all the EK).
>>
>> This will build a barebox that can load you image from multiple source in a 
>> sequence
>>
>> and then use a std barebox
>>
>> To switch to DT you will have to import the DT I did for the kernel
> OK, should be straighforward.
>
>> and replace the hw i2c by a i2c gpio as the i2c IP on at91 has a lot of 
>> quirks
>> and for barebox it will not make sense to port it
> Seems less straightforward but doable.
>

Not sure what the situation on 9263 is, but FWIW, kernel DT file from
the kernel for 9x5ek secifies I2C0 as GPIO bitbanged on, and I believe
I had it working so hopefully it would be just a DT related change
that you'd need to do.

Thanks,
Andrey Smirnov

___
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH 00/45] AT91, at91sam9x5ek updates

2017-03-07 Thread Sam Ravnborg 
> > 
> > When booting from an SD-Card/NOR-Flash/USB would I then need
> > to use at91bootstrap, or can I use only barebox?
> > 
> > If I move forward with this and I can find an 9263
> > evaluation kit I would then also update this.
> On 9263 the sram is huge so you can build barebox as first stage

Good - simplicity is good.

> 
> I add this a long time ago on usb_a9263_bootstrap_defconfig
> or other (all the EK).
> 
> This will build a barebox that can load you image from multiple source in a 
> sequence
> 
> and then use a std barebox
> 
> To switch to DT you will have to import the DT I did for the kernel
OK, should be straighforward.

> and replace the hw i2c by a i2c gpio as the i2c IP on at91 has a lot of quirks
> and for barebox it will not make sense to port it
Seems less straightforward but doable.

I will see if I can find time to give it a try and then post patches.

Sam

___
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH 00/45] AT91, at91sam9x5ek updates

2017-03-07 Thread Andrey Smirnov 
On Tue, Mar 7, 2017 at 12:13 AM, Sascha Hauer  wrote:
> Hi Andrey,
>
> On Mon, Mar 06, 2017 at 02:53:11PM -0800, Andrey Smirnov wrote:
>> Hi everyone,
>>
>> I recently had a chance to work with at91sam9x5ek board and as a part
>> of that effort I converted the code to support features found in
>> Barebox for i.MX based SoCs. Most notably this patchset converts
>> at91sam9x5ek to use multi-image PBL build process as well as adds
>> support for board initialization from attached devicetree blob.
>>
>> Sascha, at least for now, all of the patches, in chronological order,
>> are included in this submission. However if this proves to be too
>> burdensome to review, I'd be more than happy to split it into several
>> individual submissions. One such division could be:
>>
>>  - Generic bug fixes/infrastructure enhancements
>>  - Peripheral drivers updates
>>  - at91sam9x5ek specific changes
>>
>> Please let me know if that is preferrable.
>
> I had a short look at all patches, overall they look good. Nevertheless
> it would be good to sort them like you suggested. It just makes it
> easier to apply.
>

OK, will do.

> Thanks for working on this, a general AT91 overhaul was long overdue. I
> hope this series prepares a path to cleanup the other AT91 SoCs aswell.
>

Yeah, I have a bit more AT91 based hardware (at least SAMA5D3 Xplained
board and maybe more), so I was hoping to get those done after this
merge request.

> The clock support you ported over from the kernel is for all AT91 SoCs
> and not only one, right?

Yeah, I belive it should be usable by the whole AT91 family.

Thanks,
Andrey

___
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH 00/45] AT91, at91sam9x5ek updates

2017-03-07 Thread Jean-Christophe PLAGNIOL-VILLARD 

> On Mar 8, 2017, at 12:34 AM, Sam Ravnborg  wrote:
> 
> Hi Andrey.
> 
> On Mon, Mar 06, 2017 at 02:53:11PM -0800, Andrey Smirnov wrote:
>> Hi everyone,
>> 
>> I recently had a chance to work with at91sam9x5ek board and as a part
>> of that effort I converted the code to support features found in
>> Barebox for i.MX based SoCs. Most notably this patchset converts
>> at91sam9x5ek to use multi-image PBL build process as well as adds
>> support for board initialization from attached devicetree blob.
> 
> Impressive series - looks good! And timing was good.
> 
> I have a proprietary at91sam9263 based bord that is familiar
> with the 9263 evaluation kit.
> The current bootloader is latest bootstrap + an ancient U-Boot.
> As we have another board (i.MX6 based) there is some synergy into
> using the same bootloader on both boards.
> 
> Can you give any hints on what needs to be done on top of
> your patch-set to support the 9263 evalutation kit?
> 
> When booting from an SD-Card/NOR-Flash/USB would I then need
> to use at91bootstrap, or can I use only barebox?
> 
> If I move forward with this and I can find an 9263
> evaluation kit I would then also update this.
On 9263 the sram is huge so you can build barebox as first stage

I add this a long time ago on usb_a9263_bootstrap_defconfig
or other (all the EK).

This will build a barebox that can load you image from multiple source in a 
sequence

and then use a std barebox

To switch to DT you will have to import the DT I did for the kernel
and replace the hw i2c by a i2c gpio as the i2c IP on at91 has a lot of quirks
and for barebox it will not make sense to port it

Best Regards,
J.
___
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH 35/45] usb: ohci-at91: Add DT support

2017-03-07 Thread Sam Ravnborg 
On Mon, Mar 06, 2017 at 02:53:46PM -0800, Andrey Smirnov wrote:
> Signed-off-by: Andrey Smirnov 
> ---
>  arch/arm/mach-at91/include/mach/board.h |  6 ++-
>  drivers/usb/host/ohci-at91.c| 84 
> -
>  2 files changed, 86 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arm/mach-at91/include/mach/board.h 
> b/arch/arm/mach-at91/include/mach/board.h
> index 491b220..5d76e00 100644
> --- a/arch/arm/mach-at91/include/mach/board.h
> +++ b/arch/arm/mach-at91/include/mach/board.h
> @@ -32,11 +32,13 @@
>  
>  void at91_set_main_clock(unsigned long rate);
>  
> +#define AT91_MAX_USBH_PORTS  3
> +
>   /* USB Host */
>  struct at91_usbh_data {
>   u8  ports;  /* number of ports on root hub */
> - int vbus_pin[2];/* port power-control pin */
> - u8  vbus_pin_active_low[2]; /* vbus polarity */
> + int vbus_pin[AT91_MAX_USBH_PORTS];  /* port power-control 
> pin */
> + u8  vbus_pin_active_low[AT91_MAX_USBH_PORTS];   /* vbus 
> polarity */
>  };
>  extern void __init at91_add_device_usbh_ohci(struct at91_usbh_data *data);
>  extern void __init at91_add_device_usbh_ehci(struct at91_usbh_data *data);
> diff --git a/drivers/usb/host/ohci-at91.c b/drivers/usb/host/ohci-at91.c
> index f413616..54fe45f 100644
> --- a/drivers/usb/host/ohci-at91.c
> +++ b/drivers/usb/host/ohci-at91.c
> @@ -23,10 +23,18 @@
>  #include 
>  #include 
>  #include 
> +#include 
> +#include 
>  #include 
>  
> +#include 
> +
>  #include "ohci.h"
>  
> +#define at91_for_each_port(index)\
> + for ((index) = 0; (index) < AT91_MAX_USBH_PORTS; (index)++)
> +
> +
>  struct ohci_at91_priv {
>   struct device_d *dev;
>   struct clk *iclk;
> @@ -59,6 +67,53 @@ static void at91_stop_clock(struct ohci_at91_priv 
> *ohci_at91)
>   clk_disable(ohci_at91->iclk);
>  }
>  
> +static void at91_ohci_probe_dt(struct device_d *dev)
> +{
> + u32 ports;
> + int i, ret, gpio;
> + enum of_gpio_flags flags;
> + struct at91_usbh_data *pdata;
> + struct device_node *np = dev->device_node;
> +
> + pdata = xzalloc(sizeof(*pdata));
> + dev->platform_data = pdata;
> +
> + if (!of_property_read_u32(np, "num-ports", ))
> + pdata->ports = ports;
If we fail to read "num-ports" then we will fail in the following when 
dereferencing pdata->ports
> +
> + at91_for_each_port(i) {
> + /*
> +  * do not configure PIO if not in relation with
> +  * real USB port on board
> +  */
> + if (i >= pdata->ports) {
 ^ here

> + pdata->vbus_pin[i] = -EINVAL;
> + continue;
> + }

___
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH 32/45] usb: ohci-at91: Check result of clk_get()

2017-03-07 Thread Sam Ravnborg 
On Mon, Mar 06, 2017 at 02:53:43PM -0800, Andrey Smirnov wrote:
> Signed-off-by: Andrey Smirnov 
> ---
>  drivers/usb/host/ohci-at91.c | 9 +
>  1 file changed, 9 insertions(+)
> 
> diff --git a/drivers/usb/host/ohci-at91.c b/drivers/usb/host/ohci-at91.c
> index 0f5c8f1..c70d898 100644
> --- a/drivers/usb/host/ohci-at91.c
> +++ b/drivers/usb/host/ohci-at91.c
> @@ -47,7 +47,16 @@ static int at91_ohci_probe(struct device_d *dev)
>   struct ohci_regs __iomem *regs = (struct ohci_regs __iomem 
> *)dev->resource[0].start;
>  
>   iclk = clk_get(NULL, "ohci_clk");
> + if (IS_ERR(iclk)) {
> + dev_err(dev, "Failed to get 'iclk'\n");
iclk is the internal name. Woudl it make more sense to use "ohci_clk" here?

> + return PTR_ERR(iclk);
> + }
> +
>   fclk = clk_get(NULL, "uhpck");
> + if (IS_ERR(fclk)) {
> + dev_err(dev, "Failed to get 'fclk'\n");
Likewise - uhpck?


___
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH 00/45] AT91, at91sam9x5ek updates

2017-03-07 Thread Sam Ravnborg 
Hi Andrey.

On Mon, Mar 06, 2017 at 02:53:11PM -0800, Andrey Smirnov wrote:
> Hi everyone,
> 
> I recently had a chance to work with at91sam9x5ek board and as a part
> of that effort I converted the code to support features found in
> Barebox for i.MX based SoCs. Most notably this patchset converts
> at91sam9x5ek to use multi-image PBL build process as well as adds
> support for board initialization from attached devicetree blob.

Impressive series - looks good! And timing was good.

I have a proprietary at91sam9263 based bord that is familiar
with the 9263 evaluation kit.
The current bootloader is latest bootstrap + an ancient U-Boot.
As we have another board (i.MX6 based) there is some synergy into
using the same bootloader on both boards.

Can you give any hints on what needs to be done on top of
your patch-set to support the 9263 evalutation kit?

When booting from an SD-Card/NOR-Flash/USB would I then need
to use at91bootstrap, or can I use only barebox?

If I move forward with this and I can find an 9263
evaluation kit I would then also update this.

Sam

___
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH] scripts: imx/mxs remove mxs-usb-loader

2017-03-07 Thread Oleksij Rempel 
... and use imx-usb-loader instead

Signed-off-by: Oleksij Rempel 
---
 arch/arm/mach-imx/Kconfig |   7 +-
 arch/arm/mach-mxs/Kconfig |   7 +-
 scripts/Makefile  |   7 +-
 scripts/imx/Kconfig   |   6 ++
 scripts/mxs-usb-loader.c  | 236 --
 5 files changed, 11 insertions(+), 252 deletions(-)
 create mode 100644 scripts/imx/Kconfig
 delete mode 100644 scripts/mxs-usb-loader.c

diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig
index 9dbe31c4b..420b52281 100644
--- a/arch/arm/mach-imx/Kconfig
+++ b/arch/arm/mach-imx/Kconfig
@@ -632,12 +632,7 @@ endmenu
 
 menu "i.MX specific settings"
 
-config ARCH_IMX_USBLOADER
-   bool "compile imx-usb-loader"
-   help
- imx-usb-loader is a tool to upload and start imximages to an i.MX SoC
- in ROM boot mode. It requires libusb, so make sure you have the libusb
- devel package installed on your machine.
+source scripts/imx/Kconfig
 
 config IMX_IIM
tristate "IIM fusebox device"
diff --git a/arch/arm/mach-mxs/Kconfig b/arch/arm/mach-mxs/Kconfig
index facab9c25..25e2a8d04 100644
--- a/arch/arm/mach-mxs/Kconfig
+++ b/arch/arm/mach-mxs/Kconfig
@@ -143,11 +143,6 @@ endif
 
 endmenu
 
-config ARCH_MXS_USBLOADER
-   bool "compile mxs-usb-loader"
-   help
- mxs-usb-loader is a tool to upload and start mxs bootstream images to 
an
- i.MX SoC in ROM boot mode. It requires libusb, so make sure you have 
the libusb
- devel package installed on your machine.
+source scripts/imx/Kconfig
 
 endif
diff --git a/scripts/Makefile b/scripts/Makefile
index a5c16b2f3..78bb7d5e2 100644
--- a/scripts/Makefile
+++ b/scripts/Makefile
@@ -22,16 +22,15 @@ hostprogs-$(CONFIG_ARCH_SOCFPGA) += socfpga_mkimage
 hostprogs-$(CONFIG_ARCH_MXS) += mxsimage mxsboot
 HOSTCFLAGS += -I$(srctree)/scripts/include/
 HOSTLOADLIBES_mxsimage  = `pkg-config --libs openssl`
-HOSTCFLAGS_mxs-usb-loader.o = `pkg-config --cflags libusb-1.0`
-HOSTLOADLIBES_mxs-usb-loader  = `pkg-config --libs libusb-1.0`
-hostprogs-$(CONFIG_ARCH_MXS_USBLOADER)  += mxs-usb-loader
 HOSTCFLAGS_omap3-usb-loader.o = `pkg-config --cflags libusb-1.0`
 HOSTLOADLIBES_omap3-usb-loader  = `pkg-config --libs libusb-1.0`
 hostprogs-$(CONFIG_OMAP3_USB_LOADER)  += omap3-usb-loader
 
 subdir-y   += mod
 subdir-$(CONFIG_OMAP4_USBBOOT) += omap4_usbboot
-subdir-$(CONFIG_ARCH_IMX)  += imx
+ifneq ($(filter y,$(CONFIG_ARCH_IMX) $(CONFIG_ARCH_MXS)),)
+subdir-y   += imx
+endif
 subdir-$(CONFIG_X86)   += setupmbr
 subdir-$(CONFIG_DTC)   += dtc
 subdir-$(CONFIG_ARCH_TEGRA)+= tegra
diff --git a/scripts/imx/Kconfig b/scripts/imx/Kconfig
new file mode 100644
index 0..de052d070
--- /dev/null
+++ b/scripts/imx/Kconfig
@@ -0,0 +1,6 @@
+config ARCH_IMX_USBLOADER
+   bool "compile imx-usb-loader"
+   help
+ imx-usb-loader is a tool to upload and start imximages to an i.MX SoC
+ in ROM boot mode. It requires libusb, so make sure you have the libusb
+ devel package installed on your machine.
diff --git a/scripts/mxs-usb-loader.c b/scripts/mxs-usb-loader.c
deleted file mode 100644
index 8529274d6..0
--- a/scripts/mxs-usb-loader.c
+++ /dev/null
@@ -1,236 +0,0 @@
-/***
- * __   __   ___.
- *   Open  \__   \     |  | _\_ |__   ___  ___
- *   Source |   _//  _ \_/ ___\|  |/ /| __ \ /  _ \  \/  /
- *   Jukebox||   (  <_> )  \___|< | \_\ (  <_> > <  <
- *   Firmware   ||_  /\/ \___  >__|_ \|___  /\/__/\_ \
- * \/\/ \/\/\/
- * $Id$
- *
- * Copyright (C) 2010 Amaury Pouly
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License
- * as published by the Free Software Foundation; either version 2
- * of the License, or (at your option) any later version.
- *
- * This software is distributed on an "AS IS" basis, WITHOUT WARRANTY OF ANY
- * KIND, either express or implied.
- *
- /
-#include 
-#include 
-#include 
-#include 
-#include 
-
-#ifndef MIN
-#define MIN(a,b) ((a) < (b) ? (a) : (b))
-#endif
-
-void put32le(uint8_t * buf, uint32_t i)
-{
-   *buf++ = i & 0xff;
-   *buf++ = (i >> 8) & 0xff;
-   *buf++ = (i >> 16) & 0xff;
-   *buf++ = (i >> 24) & 0xff;
-}
-
-void put32be(uint8_t * buf, uint32_t i)
-{
-   *buf++ = (i >> 24) & 0xff;
-   *buf++ = (i >> 16) & 0xff;
-   *buf++ = (i >> 8) & 0xff;
-   *buf++ = i & 0xff;
-}
-
-enum dev_type_t {
-   HID_DEVICE,
-   RECOVERY_DEVICE,
-};
-
-struct dev_info_t {
-   uint16_t vendor_id;
-   uint16_t product_id;
-   unsigned xfer_size;
-   enum dev_type_t

[PATCH] ARM: imx233-olinuxino: add CONFIG_CONSOLE_ACTIVATE_ALL

2017-03-07 Thread Oleksij Rempel 
if not set, barebox will use first console by default.
On this board first console is KEYBOARD_GPIO, so we will end
in unusable state.

Signed-off-by: Oleksij Rempel 
---
 arch/arm/configs/imx233-olinuxino_defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/configs/imx233-olinuxino_defconfig 
b/arch/arm/configs/imx233-olinuxino_defconfig
index 991386b30..47d485b4e 100644
--- a/arch/arm/configs/imx233-olinuxino_defconfig
+++ b/arch/arm/configs/imx233-olinuxino_defconfig
@@ -20,6 +20,7 @@ CONFIG_BOOTM_INITRD=y
 CONFIG_BOOTM_OFTREE=y
 CONFIG_BOOTM_OFTREE_UIMAGE=y
 CONFIG_BLSPEC=y
+CONFIG_CONSOLE_ACTIVATE_ALL=y
 CONFIG_DEFAULT_COMPRESSION_LZO=y
 CONFIG_DEFAULT_ENVIRONMENT_GENERIC_NEW=y
 CONFIG_RESET_SOURCE=y
-- 
2.11.0


___
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 1/2] x86: efi: fix efiexit support

2017-03-07 Thread Jean-Christophe PLAGNIOL-VILLARD 
Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD 
---
 arch/x86/mach-efi/elf_x86_64_efi.lds.S | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/x86/mach-efi/elf_x86_64_efi.lds.S 
b/arch/x86/mach-efi/elf_x86_64_efi.lds.S
index e1bc2120f..93d34d17a 100644
--- a/arch/x86/mach-efi/elf_x86_64_efi.lds.S
+++ b/arch/x86/mach-efi/elf_x86_64_efi.lds.S
@@ -80,6 +80,7 @@ SECTIONS
*(.rela.data*)
*(.rela.barebox*)
*(.rela.initcall*)
+   *(.rela.exitcall*)
*(.rela.got)
*(.rela.stab)
}
-- 
2.11.0


___
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 2/2] x86: efi: fix logo support

2017-03-07 Thread Jean-Christophe PLAGNIOL-VILLARD 
we need to copy the logo section too

Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD 
---
 arch/x86/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/x86/Makefile b/arch/x86/Makefile
index e837a2df9..6921b10a7 100644
--- a/arch/x86/Makefile
+++ b/arch/x86/Makefile
@@ -85,7 +85,7 @@ quiet_cmd_efi_image = EFI-IMG $@
   cmd_efi_image = $(OBJCOPY) -j .text -j .sdata -j .data -j .dynamic \
  -j .dynsym -j .rel -j .rela -j .reloc -j 
__barebox_initcalls \
  -j __barebox_cmd -j .barebox_magicvar -j .bbenv.* \
- --target=$(TARGET) $< $@
+ -j .bblogo.* --target=$(TARGET) $< $@
 
 KBUILD_BINARY := barebox
 
-- 
2.11.0


___
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH] fixup! efi: move x86 efi boot support to x86 arch

2017-03-07 Thread Michael Olbrich 
As I noted in my other mail, this got lost during the move.

Regards,
Michael

---
 arch/x86/Makefile  | 4 ++--
 arch/x86/mach-efi/elf_x86_64_efi.lds.S | 1 +
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/arch/x86/Makefile b/arch/x86/Makefile
index e837a2df972c..c36104c2c07f 100644
--- a/arch/x86/Makefile
+++ b/arch/x86/Makefile
@@ -84,8 +84,8 @@ cmd_barebox__ ?= $(LD) $(LDFLAGS) $(LDFLAGS_barebox) -o $@
\
 quiet_cmd_efi_image = EFI-IMG $@
   cmd_efi_image = $(OBJCOPY) -j .text -j .sdata -j .data -j .dynamic \
  -j .dynsym -j .rel -j .rela -j .reloc -j 
__barebox_initcalls \
- -j __barebox_cmd -j .barebox_magicvar -j .bbenv.* \
- --target=$(TARGET) $< $@
+ -j __barebox_exitcalls -j __barebox_cmd -j 
.barebox_magicvar \
+ -j .bbenv.* --target=$(TARGET) $< $@
 
 KBUILD_BINARY := barebox
 
diff --git a/arch/x86/mach-efi/elf_x86_64_efi.lds.S 
b/arch/x86/mach-efi/elf_x86_64_efi.lds.S
index e1bc2120fabc..93d34d17ab1f 100644
--- a/arch/x86/mach-efi/elf_x86_64_efi.lds.S
+++ b/arch/x86/mach-efi/elf_x86_64_efi.lds.S
@@ -80,6 +80,7 @@ SECTIONS
*(.rela.data*)
*(.rela.barebox*)
*(.rela.initcall*)
+   *(.rela.exitcall*)
*(.rela.got)
*(.rela.stab)
}
-- 
2.11.0


___
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH 00/45] AT91, at91sam9x5ek updates

2017-03-07 Thread Sascha Hauer 
Hi Andrey,

On Mon, Mar 06, 2017 at 02:53:11PM -0800, Andrey Smirnov wrote:
> Hi everyone,
> 
> I recently had a chance to work with at91sam9x5ek board and as a part
> of that effort I converted the code to support features found in
> Barebox for i.MX based SoCs. Most notably this patchset converts
> at91sam9x5ek to use multi-image PBL build process as well as adds
> support for board initialization from attached devicetree blob.
> 
> Sascha, at least for now, all of the patches, in chronological order,
> are included in this submission. However if this proves to be too
> burdensome to review, I'd be more than happy to split it into several
> individual submissions. One such division could be:
> 
>  - Generic bug fixes/infrastructure enhancements
>  - Peripheral drivers updates
>  - at91sam9x5ek specific changes
> 
> Please let me know if that is preferrable.

I had a short look at all patches, overall they look good. Nevertheless
it would be good to sort them like you suggested. It just makes it
easier to apply.

Thanks for working on this, a general AT91 overhaul was long overdue. I
hope this series prepares a path to cleanup the other AT91 SoCs aswell.

The clock support you ported over from the kernel is for all AT91 SoCs
and not only one, right?

Sascha

-- 
Pengutronix e.K.   | |
Industrial Linux Solutions | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0|
Amtsgericht Hildesheim, HRA 2686   | Fax:   +49-5121-206917- |

___
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox