[U-Boot] [PATCH v2] board/BuR/brppt1: fix MMC boot

2017-07-05 Thread Hannes Schmelzer
since commit 'd5abcf94c7123167725fc22ace342f0d455093c1' - ti: boot: Register the MMC controllers in SPL in the same way as in u-boot MMC boot on brppt1 board is broken, with this commit we make our board working again. Signed-off-by: Hannes Schmelzer --- Changes in v2: - fix also the 'altb

[U-Boot] [PATCH] board/BuR/brppt1: fix MMC boot

2017-07-05 Thread Hannes Schmelzer
since commit 'd5abcf94c7123167725fc22ace342f0d455093c1' - ti: boot: Register the MMC controllers in SPL in the same way as in u-boot MMC boot on brppt1 board is broken, with this commit we make our board working again. Signed-off-by: Hannes Schmelzer --- include/configs/brppt1.h | 16

Re: [U-Boot] [PATCH v1 3/3] wdt: Unify option of timeout value

2017-07-05 Thread Heiko Schocher
Hello Andy, Am 05.07.2017 um 19:44 schrieb Andy Shevchenko: There is no need to duplicate same option with different name. Kill HW_WATCHDOG_TIMEOUT_MS in favor of WATCHDOG_TIMEOUT_MSECS. Signed-off-by: Andy Shevchenko --- drivers/watchdog/designware_wdt.c | 2 +- include/configs/socfpga_co

Re: [U-Boot] [PATCH 07/16] usb: hub: Translate USB 3.0 hub port status into old version

2017-07-05 Thread Bin Meng
Hi Marek, On Thu, Jun 29, 2017 at 1:25 AM, Marek Vasut wrote: > On 06/28/2017 10:28 AM, Bin Meng wrote: >> Hi Marek, >> >> On Tue, Jun 27, 2017 at 7:57 AM, Bin Meng wrote: >>> Hi Marek, >>> >>> On Tue, Jun 27, 2017 at 2:06 AM, Marek Vasut wrote: On 06/24/2017 03:53 AM, Bin Meng wrote:

Re: [U-Boot] [PATCH 13/16] usb: hub: Call usb_update_hub_device() after hub descriptor is fetched

2017-07-05 Thread Simon Glass
On 23 June 2017 at 03:54, Bin Meng wrote: > After fetching hub descriptor, we need call USB uclass operation need to call > update_hub_device() to notify HCD to do some preparation work. > > Signed-off-by: Bin Meng > --- > > common/usb_hub.c | 11 +++ > 1 file changed, 11 insertions(+)

Re: [U-Boot] [PATCH 08/16] usb: hub: Support 'set hub depth' request for USB 3.0 hubs

2017-07-05 Thread Simon Glass
On 23 June 2017 at 03:54, Bin Meng wrote: > USB 3.0 hub uses a hub depth value multiplied by four as an offset > into the 'route string' to locate the bits it uses to determine the > downstream port number. We shall set the hub depth value of a USB > 3.0 hub after it is configured. > > Signed-off-

Re: [U-Boot] [PATCH 06/16] usb: xhci: Change to use usb_hub_is_root_hub() API

2017-07-05 Thread Simon Glass
On 23 June 2017 at 03:54, Bin Meng wrote: > Now that we have a generic public API, remove the xHCI driver's own > version is_root_hub() and use the new API. > > While we are here, remove the unused/commented out get_usb_device(). > > Signed-off-by: Bin Meng > --- > > drivers/usb/host/xhci.c | 24

Re: [U-Boot] [PATCH 15/16] usb: xhci: Correct TT_SLOT and TT_PORT macros

2017-07-05 Thread Simon Glass
On 23 June 2017 at 03:54, Bin Meng wrote: > These two macros really need a parameter to make them useful. > > Signed-off-by: Bin Meng > --- > > drivers/usb/host/xhci.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Reviewed-by: Simon Glass _

Re: [U-Boot] [PATCH 3/7] rockchip: rk3328: add sdram driver in U-Boot

2017-07-05 Thread Simon Glass
Hi Kever, On 23 June 2017 at 02:22, Kever Yang wrote: > Hi Simon, > > > On 06/17/2017 11:41 AM, Simon Glass wrote: >> >> On 13 June 2017 at 03:29, Kever Yang wrote: >>> >>> Add sdram driver in U-Boot for get the correct sdram size from >>> sys_reg, so that U-Boot can co-work with Rockchip loader

Re: [U-Boot] [PATCH 01/16] usb: xhci-pci: Drop non-DM version of xhci-pci driver

2017-07-05 Thread Simon Glass
On 23 June 2017 at 03:54, Bin Meng wrote: > As there is no board that currently uses xhci-pci driver without DM > USB, drop its support and leave only DM support. > > Signed-off-by: Bin Meng > --- > > drivers/usb/host/xhci-pci.c | 52 > - > 1 file cha

Re: [U-Boot] [PATCH] dm: video: fix abuse of enum

2017-07-05 Thread Simon Glass
Hi Lothar, On 23 June 2017 at 00:30, Lothar Waßmann wrote: > Hi, > > On Wed, 21 Jun 2017 09:59:05 +0200 Lothar Waßmann wrote: >> Hi, >> >> On Tue, 20 Jun 2017 12:26:29 -0600 Simon Glass wrote: >> > Hi Lothar, >> > >> > On 20 June 2017 at 04:25, Lothar Waßmann wrote: >> > > LCD_MAX_WIDTH, LCD_MAX

Re: [U-Boot] [PATCH 14/16] usb: xhci: Implement update_hub_device() operation

2017-07-05 Thread Simon Glass
Hi Bin, On 23 June 2017 at 03:54, Bin Meng wrote: > There is no way to know whether the attached device is a hub or > not in advance before device's descriptor is fetched. But once nits: before the device's > we know it's a high speed hub, per xHCI spec, we need tell xHC per the xHCI we need

Re: [U-Boot] [PATCH 11/16] usb: hub: Parse and save TT details from device descriptor

2017-07-05 Thread Simon Glass
On 23 June 2017 at 03:54, Bin Meng wrote: > A high speed hub has a special responsibility to handle full speed/ > low speed devices connected on downstream ports. In this case, the > hub must isolate the high speed signaling environment from the full > speed/low speed signaling environment with th

Re: [U-Boot] [PATCH] serial: make serial_stub_* to static functions

2017-07-05 Thread Simon Glass
On 22 June 2017 at 01:48, Masahiro Yamada wrote: > Add missing static to serial_stub_puts(). > > Unexport serial_stub_{getc,tstc} because they are used locally. > > Signed-off-by: Masahiro Yamada > --- > > drivers/serial/serial-uclass.c | 11 +++ > drivers/serial/serial.c| 4 ++-

Re: [U-Boot] [PATCH 2/2] atmel, at91: fix smartweb board

2017-07-05 Thread Simon Glass
On 22 June 2017 at 00:49, Heiko Schocher wrote: > since commit: f8b7fff1d5c5 "serial: atmel_usart: Add clk support" > > smartweb board comes not up anymore. Fix it. > > Signed-off-by: Heiko Schocher > --- > > arch/arm/dts/at91sam9260-smartweb.dts | 2 ++ > board/siemens/smartweb/smartweb.c |

Re: [U-Boot] mpc85xx boards

2017-07-05 Thread York Sun
I am not in a position to decide which board to drop. I can try my best to convert them. York On 07/06/2017 12:50 PM, Simon Glass wrote: > Hi York, > > OK...so all 250 are needed? Is there a plan to convert to Kconfig / DM? > > Regards, > Simon > > On 23 June 2017 at 07:10, York Sun wrote: >

Re: [U-Boot] mpc85xx boards

2017-07-05 Thread Simon Glass
Hi York, OK...so all 250 are needed? Is there a plan to convert to Kconfig / DM? Regards, Simon On 23 June 2017 at 07:10, York Sun wrote: > I tried to drop some old 85xx boards but got resistance from our silicon > support team. There is a demand to maintain those SoCs still shipping. > > York

Re: [U-Boot] [PATCH 02/16] usb: xhci-pci: Clean up the driver a little bit

2017-07-05 Thread Simon Glass
On 23 June 2017 at 11:52, Marek Vasut wrote: > On 06/23/2017 11:54 AM, Bin Meng wrote: >> This cleans up the driver a little bit. >> >> Signed-off-by: Bin Meng >> --- >> >> drivers/usb/host/xhci-pci.c | 16 ++-- >> 1 file changed, 2 insertions(+), 14 deletions(-) >> >> diff --git a/d

Re: [U-Boot] [PATCH v9 08/10] usb: host: ohci-generic: add CLOCK support

2017-07-05 Thread Simon Glass
On 21 June 2017 at 06:42, wrote: > From: Patrice Chotard > > use array to save enabled clocks reference in order to > disabled them in case of error during probe() or during > driver removal. > > Signed-off-by: Patrice Chotard > --- > v9: _ remove useless clk_free() when a clock is correctl

Re: [U-Boot] [PATCH] dm: ofnode: use fdt32_t for DT property value to fix sparse warning

2017-07-05 Thread Simon Glass
On 22 June 2017 at 02:57, Masahiro Yamada wrote: > DTB is encoded in big endian. When we retrieve property values, > we need to use fdt32_to_cpu (aka be32_to_cpu) for endian conversion. > This is a bit error-prone, but sparse is useful to detect endian > mismatch. > > We need to use (fdt32_t *) i

Re: [U-Boot] [PATCH 4/5] dm: ofnode: rename dev_read_prop() to dev_get_property()

2017-07-05 Thread Simon Glass
On 22 June 2017 at 01:54, Masahiro Yamada wrote: > The previous commit renamed ofnode_read_prop() to ofnode_get_propery() > and fixed its return type. Likewise, rename dev_read_prop() and fix > its return type. > > Signed-off-by: Masahiro Yamada > --- > > drivers/core/read.c| 3 ++- > d

Re: [U-Boot] [PATCH 04/16] usb: hub: Use 'struct usb_hub_device' as hub device's uclass_priv

2017-07-05 Thread Simon Glass
On 23 June 2017 at 03:54, Bin Meng wrote: > Use USB hub device's dev->uclass_priv to point to 'usb_hub_device' > so that with driver model usb_hub_reset() and usb_hub_allocate() > are no longer needed. > > Signed-off-by: Bin Meng > --- > > common/usb_hub.c | 10 +- > drivers

Re: [U-Boot] [PATCH 5/5] dm: ofnode: simplify ofnode_read_bool()

2017-07-05 Thread Simon Glass
On 22 June 2017 at 01:54, Masahiro Yamada wrote: > Reuse ofnode_get_property() to simplify the implementation. > > Signed-off-by: Masahiro Yamada > --- > > drivers/core/ofnode.c | 14 +- > 1 file changed, 5 insertions(+), 9 deletions(-) Acked-by: Simon Glass ___

Re: [U-Boot] [PATCH] dm: include from drivers/core/util.c

2017-07-05 Thread Simon Glass
Hi Masahiro, On 22 June 2017 at 01:50, Masahiro Yamada wrote: > Fix sparse warnings "... was not declared. Should it be static?" > > Also, fix redefinition of dm_warn/dm_dbg. > > Signed-off-by: Masahiro Yamada > --- > > drivers/core/util.c | 5 + > 1 file changed, 5 insertions(+) > > diff -

Re: [U-Boot] [PATCH 16/16] usb: xhci: Enable TT to support LS/FS devices behind a HS hub

2017-07-05 Thread Simon Glass
On 23 June 2017 at 03:54, Bin Meng wrote: > So far LS/FS devices directly attached to xHC root port can be > successfully enumerated by xHCI driver, but if they are connected > behind a hub, the enumeration process fails to address the device. > > It turns out xHCI driver still misses a part that

Re: [U-Boot] [PATCH 3/5] dm: ofnode: rename ofnode_read_prop() to ofnode_get_property()

2017-07-05 Thread Simon Glass
On 22 June 2017 at 01:54, Masahiro Yamada wrote: > This function returns the pointer to the value of a node property. > The current name ofnode_read_prop() is confusing. Follow the naming > of_get_property() from Linux. > > The return type (const u32 *) is wrong. DT property values can be > stri

Re: [U-Boot] [PATCH 12/16] dm: usb: Add a new USB controller operation 'update_hub_device'

2017-07-05 Thread Simon Glass
On 23 June 2017 at 03:54, Bin Meng wrote: > For USB host controllers like xHC, its internal representation of > hub needs to be updated after the hub descriptor is fetched. This > adds a new op that does this. > > Signed-off-by: Bin Meng > --- > > drivers/usb/host/usb-uclass.c | 11 +++ >

Re: [U-Boot] [PATCH] dm: include from driver/core/dump.c

2017-07-05 Thread Simon Glass
On 22 June 2017 at 02:10, Masahiro Yamada wrote: > Include to fix sparse warnings: > symbol 'dm_dump_all' was not declared. Should it be static? > symbol 'dm_dump_uclass' was not declared. Should it be static? > > Signed-off-by: Masahiro Yamada > --- > > drivers/core/dump.c | 1 + > 1 file chan

Re: [U-Boot] [PATCH 09/16] usb: xhci: Change xhci_setup_addressable_virt_dev() signature

2017-07-05 Thread Simon Glass
On 23 June 2017 at 03:54, Bin Meng wrote: > For future extension, change xhci_setup_addressable_virt_dev() > signature to accept a pointer to 'struct usb_device', instead > of its members slot_id & speed, as the struct already contains > these two plus some other useful information of the device.

Re: [U-Boot] [PATCH v8 08/10] usb: host: ohci-generic: add CLOCK support

2017-07-05 Thread Simon Glass
On 21 June 2017 at 01:50, wrote: > From: Patrice Chotard > > use array to save enabled clocks reference in order to > disabled them in case of error during probe() or during > driver removal. > > Signed-off-by: Patrice Chotard > --- > v8: _ rework error path by propagating the initial error

Re: [U-Boot] [PATCH v9 07/10] usb: host: ehci-generic: add generic PHY support

2017-07-05 Thread Simon Glass
On 21 June 2017 at 06:42, wrote: > From: Patrice Chotard > > Extend ehci-generic driver with generic PHY framework > > Signed-off-by: Patrice Chotard > --- > v9: _ none > > v8: _ rework error path by propagating the initial error code until the > end of probe() > > v7: _ none > > v

Re: [U-Boot] [PATCH 2/5] dm: ofnode: simplify ofnode_read_prop()

2017-07-05 Thread Simon Glass
On 22 June 2017 at 01:54, Masahiro Yamada wrote: > The code inside the if-block is the same as of_get_property(). > > Signed-off-by: Masahiro Yamada > --- > > drivers/core/ofnode.c | 12 +++- > 1 file changed, 3 insertions(+), 9 deletions(-) Acked-by: Simon Glass __

Re: [U-Boot] [PATCH v9 03/10] clk: add clk_release_all()

2017-07-05 Thread Simon Glass
On 21 June 2017 at 06:42, wrote: > From: Patrice Chotard > > Add clk_release_all() method which Disable/Free an > array of clocks that has been previously requested by > clk_request/get_by_*() > > Signed-off-by: Patrice Chotard > --- > v9: _ to avoid confusion, rename clk_disable_all() in c

Re: [U-Boot] [PATCH v9 09/10] usb: host: ohci-generic: add RESET support

2017-07-05 Thread Simon Glass
On 21 June 2017 at 06:42, wrote: > From: Patrice Chotard > > use array to save deasserted resets reference in order to > assert them in case of error during probe() or during driver > removal. > > Signed-off-by: Patrice Chotard > --- > v9: _ remove useless reset_free() when a reset is corre

Re: [U-Boot] [PATCH] gpio: add static to get_function()

2017-07-05 Thread Simon Glass
On 22 June 2017 at 01:50, Masahiro Yamada wrote: > Signed-off-by: Masahiro Yamada > --- > > drivers/gpio/gpio-uclass.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Reviewed-by: Simon Glass ___ U-Boot mailing list U-Boot@lists.denx.de h

Re: [U-Boot] [PATCH 1/5] dm: ofnode: use ofnode_read_bool() to check property existence

2017-07-05 Thread Simon Glass
On 22 June 2017 at 01:54, Masahiro Yamada wrote: > This will clarify the code. > > Signed-off-by: Masahiro Yamada > --- > > drivers/core/ofnode.c | 10 +- > 1 file changed, 5 insertions(+), 5 deletions(-) Acked-by: Simon Glass ___ U-Boot mail

Re: [U-Boot] [PATCH v9 10/10] usb: host: ohci-generic: add generic PHY support

2017-07-05 Thread Simon Glass
On 21 June 2017 at 06:42, wrote: > From: Patrice Chotard > > Extend ohci-generic driver with generic PHY framework > > Signed-off-by: Patrice Chotard > --- > v9: _ none > > v8: _ rework error path by propagating the initial error code until the > end of probe() > > v7: _ none > > v

Re: [U-Boot] [PATCH v9 02/10] reset: add reset_release_all()

2017-07-05 Thread Simon Glass
On 21 June 2017 at 06:42, wrote: > From: Patrice Chotard > > Add reset_release_all() method which Assert/Free an > array of resets signal that has been previously successfully > requested by reset_get_by_*() > > Signed-off-by: Patrice Chotard > --- > v9: _ to avoid confusion, rename reset_a

Re: [U-Boot] [PATCH v2 08/13] usb: gadget: ether: Provide a way to read MAC address

2017-07-05 Thread Simon Glass
Hi Lukasz, On 21 June 2017 at 01:33, Lukasz Majewski wrote: > Hi Vignesh, > >> Hi Lukasz, >> >> On Thursday 15 June 2017 10:28 PM, Marek Vasut wrote: >> > On 06/14/2017 02:24 PM, Vignesh R wrote: >> >> >> >> >> >> On Tuesday 13 June 2017 07:36 PM, Marek Vasut wrote: >> >>> On 06/13/2017 02:10 PM,

Re: [U-Boot] at91: smartweb board fails with current HEAD

2017-07-05 Thread Simon Glass
Hi Heiko, On 20 June 2017 at 22:26, Heiko Schocher wrote: > Hello Wenyou, > > Am 21.06.2017 um 02:52 schrieb wenyou.y...@microchip.com: >> >> >> >>> -Original Message- >>> From: Heiko Schocher [mailto:h...@denx.de] >>> Sent: 2017年6月20日 22:05 >>> To: Wenyou Yang - A41535 >>> Cc: u-boot@li

Re: [U-Boot] [PATCH v1 2/3] wdt: Timeout better to be in microseconds

2017-07-05 Thread Heiko Schocher
Hello Andy, Am 05.07.2017 um 19:44 schrieb Andy Shevchenko: Timeout in some abstract ticks is not what we are applying to get deterministic behaviour. Convert name to show explicitly that we are using microseconds (for watchdog it's more than precise). No functional change intended. Signed-of

Re: [U-Boot] [PATCH v1 1/3] wdt: Fix spelling Resettting -> Resetting

2017-07-05 Thread Heiko Schocher
Hello Andy, Am 05.07.2017 um 19:44 schrieb Andy Shevchenko: Fix spelling Resettting -> Resetting. No functional change intended. Signed-off-by: Andy Shevchenko --- drivers/watchdog/wdt-uclass.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Thanks! Reviewed-by: Heiko Schocher by

Re: [U-Boot] [PATCH v1 2/3] wdt: Timeout better to be in microseconds

2017-07-05 Thread Bin Meng
On Thu, Jul 6, 2017 at 1:44 AM, Andy Shevchenko wrote: > Timeout in some abstract ticks is not what we are applying to get > deterministic behaviour. > > Convert name to show explicitly that we are using microseconds (for > watchdog it's more than precise). > > No functional change intended. > > S

Re: [U-Boot] [PATCH v1 3/3] wdt: Unify option of timeout value

2017-07-05 Thread Bin Meng
On Thu, Jul 6, 2017 at 1:44 AM, Andy Shevchenko wrote: > There is no need to duplicate same option with different name. > > Kill HW_WATCHDOG_TIMEOUT_MS in favor of WATCHDOG_TIMEOUT_MSECS. > > Signed-off-by: Andy Shevchenko > --- > drivers/watchdog/designware_wdt.c | 2 +- > include/configs/socfp

Re: [U-Boot] [PATCH v1 1/3] wdt: Fix spelling Resettting -> Resetting

2017-07-05 Thread Bin Meng
On Thu, Jul 6, 2017 at 1:44 AM, Andy Shevchenko wrote: > Fix spelling Resettting -> Resetting. > No functional change intended. > > Signed-off-by: Andy Shevchenko > --- > drivers/watchdog/wdt-uclass.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Reviewed-by: Bin Meng _

Re: [U-Boot] [PATCH v2 3/3] x86: Add Intel Edison board files

2017-07-05 Thread Bin Meng
Hi Andy, On Thu, Jul 6, 2017 at 4:56 AM, Andy Shevchenko wrote: > Add Intel Edison board which is using U-Boot. > > The patch is based on work done by the following people (in alphabetical > order): > Aiden Park > Dukjoon Jeon > eric.park > Fabien Chereau >

Re: [U-Boot] [PATCH v2 2/3] x86: Add Intel Tangier support

2017-07-05 Thread Bin Meng
On Thu, Jul 6, 2017 at 4:56 AM, Andy Shevchenko wrote: > From: Felipe Balbi > > Add Intel Tangier SoC support. > > Intel Tangier SoC is a core part of Intel Merrifield platform. For > example, Intel Edison board is based on such platform. > > The patch is based on work done by the following peopl

Re: [U-Boot] [PATCH v2 1/3] x86: Add dma-mapping.h to architectural code

2017-07-05 Thread Bin Meng
On Thu, Jul 6, 2017 at 4:56 AM, Andy Shevchenko wrote: > Some cross-platform drivers rely on this header present. > Make it so for x86. > > It's just a copy'n'paste of arch/arm/include/asm/dma-mapping.h. > > Suggested-by: Bin Meng > Reviewed-by: Simon Glass > Signed-off-by: Andy Shevchenko > --

Re: [U-Boot] [RFC/PATCH 1/2] ni: zynq: Add support for NI Ettus Research Project Sulfur Rev2 SDR

2017-07-05 Thread Moritz Fischer
On Thu, Jun 29, 2017 at 3:22 AM, Michal Simek wrote: > On 23.6.2017 22:57, Moritz Fischer wrote: >> Add support for second revision of NI Ettus Research Project Sulfur >> Revision 2 SDR board. >> >> Signed-off-by: Moritz Fischer >> --- >> arch/arm/dts/Makefile| 1 +

[U-Boot] [PATCH v2 2/3] x86: Add Intel Tangier support

2017-07-05 Thread Andy Shevchenko
From: Felipe Balbi Add Intel Tangier SoC support. Intel Tangier SoC is a core part of Intel Merrifield platform. For example, Intel Edison board is based on such platform. The patch is based on work done by the following people (in alphabetical order): Aiden Park Dukjoon Jeon

[U-Boot] [PATCH v2 3/3] x86: Add Intel Edison board files

2017-07-05 Thread Andy Shevchenko
Add Intel Edison board which is using U-Boot. The patch is based on work done by the following people (in alphabetical order): Aiden Park Dukjoon Jeon eric.park Fabien Chereau Felipe Balbi Scott D Phillips Sebastien Colleur Ste

[U-Boot] [PATCH v2 0/3] x86: Introduce Intel Tangier SoC and Edison board

2017-07-05 Thread Andy Shevchenko
This brings support for Intel Tangier SoC and Intel Edison board which is using U-Boot as a main bootloader. The series has implicit dependency to watchdog driver (V3 of which had been sent earlier today). Since v1: - split patch 1 to 2 (Bin) - move enum to sfi.h (Bin) - remove unnecessary header

[U-Boot] [PATCH v2 1/3] x86: Add dma-mapping.h to architectural code

2017-07-05 Thread Andy Shevchenko
Some cross-platform drivers rely on this header present. Make it so for x86. It's just a copy'n'paste of arch/arm/include/asm/dma-mapping.h. Suggested-by: Bin Meng Reviewed-by: Simon Glass Signed-off-by: Andy Shevchenko --- arch/x86/include/asm/dma-mapping.h | 41 +

Re: [U-Boot] [U-Boot,RFC,v1] wdt: Add a glue driver for watchdog

2017-07-05 Thread Heinrich Schuchardt
On 07/05/2017 07:45 PM, Andy Shevchenko wrote: > This driver enables first found watchdog as a system one and uses it as a > system hardware watchdog device by providing hw_watchdog_reset(), > hw_watchdog_disable() and void hw_watchdog_init() hooks. > > Signed-off-by: Andy Shevchenko > --- > > T

Re: [U-Boot] [PATCH 10/14] dm: mmc: sunxi: Add support for driver model

2017-07-05 Thread Maxime Ripard
On Wed, Jul 05, 2017 at 04:57:40PM +0200, Maxime Ripard wrote: > On Tue, Jul 04, 2017 at 01:33:25PM -0600, Simon Glass wrote: > > Hi Maxime, > > > > On 21 June 2017 at 01:31, Maxime Ripard > > wrote: > > > Hi Simon, > > > > > > On Mon, Jun 19, 2017 at 11:11:27AM -0600, Simon Glass wrote: > > >> A

Re: [U-Boot] [PATCH v1] cpu: Add Intel Tangier support

2017-07-05 Thread Andy Shevchenko
On Thu, 2017-04-20 at 17:41 +0800, Bin Meng wrote: > On Thu, Apr 20, 2017 at 3:23 PM, Andy Shevchenko > wrote: > > On Thu, Apr 20, 2017 at 5:59 AM, Bin Meng > > > On Tue, Apr 18, 2017 at 10:21 PM, Andy Shevchenko > > > wrote: > > > > +#define SFI_BASE_ADDR  0x000E > > > > +#define

Re: [U-Boot] [U-Boot,RFC,v1] wdt: Add a glue driver for watchdog

2017-07-05 Thread Andy Shevchenko
On Wed, 2017-07-05 at 21:56 +0200, Heinrich Schuchardt wrote: > On 07/05/2017 07:45 PM, Andy Shevchenko wrote: > > This is not a working solution, just an opinion to which direction > > we might go > > with ugly broken WDT class. > > > > Feel free to ignore, drop, modify, etc. ^^ DISCLAIMER

Re: [U-Boot] [U-Boot,RFC,v1] wdt: Add a glue driver for watchdog

2017-07-05 Thread Heinrich Schuchardt
On 07/05/2017 07:45 PM, Andy Shevchenko wrote: > This driver enables first found watchdog as a system one and uses it as a > system hardware watchdog device by providing hw_watchdog_reset(), > hw_watchdog_disable() and void hw_watchdog_init() hooks. > > Signed-off-by: Andy Shevchenko > --- > > T

Re: [U-Boot] [PATCH v1] cpu: Add Intel Tangier support

2017-07-05 Thread Simon Glass
Hi Andy, On 5 July 2017 at 13:27, Andy Shevchenko wrote: > On Tue, 2017-06-20 at 13:35 -0600, Simon Glass wrote: >> Hi Andy, >> >> On 20 June 2017 at 13:02, Andy Shevchenko >> wrote: >> > On Tue, Jun 20, 2017 at 9:51 PM, Simon Glass >> > wrote: >> > > On 20 April 2017 at 03:41, Bin Meng wrote:

Re: [U-Boot] [PATCH v1] cpu: Add Intel Tangier support

2017-07-05 Thread Andy Shevchenko
On Tue, 2017-06-20 at 13:35 -0600, Simon Glass wrote: > Hi Andy, > > On 20 June 2017 at 13:02, Andy Shevchenko > wrote: > > On Tue, Jun 20, 2017 at 9:51 PM, Simon Glass > > wrote: > > > On 20 April 2017 at 03:41, Bin Meng wrote: > > > Andy, any update on this please? Is it still in progress? >

Re: [U-Boot] [PATCH v1 3/3] wdt: Unify option of timeout value

2017-07-05 Thread Andy Shevchenko
On Wed, 2017-07-05 at 20:44 +0300, Andy Shevchenko wrote: > There is no need to duplicate same option with different name. > > Kill HW_WATCHDOG_TIMEOUT_MS in favor of WATCHDOG_TIMEOUT_MSECS. > Yes, I know about WATCHDOG_TIMEOUT, I spent too much time already on this crap. Feel free to make a fol

[U-Boot] [PATCH 3/3] efi_loader: no debug message on wait for key

2017-07-05 Thread Heinrich Schuchardt
If efi_check_event is called in a loop waiting for keyboard input the screen is flooded with debug messages. So only write debug message for other events. Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_boottime.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git

[U-Boot] [PATCH 2/3] efi_loader: implement WaitForKey

2017-07-05 Thread Heinrich Schuchardt
The EFI_SIMPLE_TEXT_INPUT_PROTOCOL requires an event WaitForKey. We can easily signal the event in the efi_timer_check function. Signed-off-by: Heinrich Schuchardt --- include/efi_loader.h | 5 - lib/efi_loader/efi_boottime.c | 17 ++--- lib/efi_loader/efi_console.c |

[U-Boot] [PATCH 0/3] efi_loader: event services

2017-07-05 Thread Heinrich Schuchardt
Currently U-Boot only allows a single event. This is sufficient to run grub but not for other workloads as iPXE. The EFI_SIMPLE_TEXT_INPUT_PROTOCOL requires an event WaitForKey. This is used for instance by the iPXE shell. With the 1st patch events are stored in an array. Four events seems to be

[U-Boot] [PATCH 1/3] efi_loader: implement multiple event support

2017-07-05 Thread Heinrich Schuchardt
Up to now the boot time supported only a single event. This patch now allows four events. Signed-off-by: Heinrich Schuchardt --- include/efi_api.h | 8 ++- include/efi_loader.h | 23 ++ lib/efi_loader/efi_boottime.c | 161 ++ 3

[U-Boot] [RFC PATCH v1] wdt: Add a glue driver for watchdog

2017-07-05 Thread Andy Shevchenko
This driver enables first found watchdog as a system one and uses it as a system hardware watchdog device by providing hw_watchdog_reset(), hw_watchdog_disable() and void hw_watchdog_init() hooks. Signed-off-by: Andy Shevchenko --- This is not a working solution, just an opinion to which directi

[U-Boot] [PATCH v1 3/3] wdt: Unify option of timeout value

2017-07-05 Thread Andy Shevchenko
There is no need to duplicate same option with different name. Kill HW_WATCHDOG_TIMEOUT_MS in favor of WATCHDOG_TIMEOUT_MSECS. Signed-off-by: Andy Shevchenko --- drivers/watchdog/designware_wdt.c | 2 +- include/configs/socfpga_common.h | 2 +- scripts/config_whitelist.txt | 1 - 3 files

[U-Boot] [PATCH v1 2/3] wdt: Timeout better to be in microseconds

2017-07-05 Thread Andy Shevchenko
Timeout in some abstract ticks is not what we are applying to get deterministic behaviour. Convert name to show explicitly that we are using microseconds (for watchdog it's more than precise). No functional change intended. Signed-off-by: Andy Shevchenko --- drivers/watchdog/wdt-uclass.c | 4 +

[U-Boot] [PATCH v1 1/3] wdt: Fix spelling Resettting -> Resetting

2017-07-05 Thread Andy Shevchenko
Fix spelling Resettting -> Resetting. No functional change intended. Signed-off-by: Andy Shevchenko --- drivers/watchdog/wdt-uclass.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/watchdog/wdt-uclass.c b/drivers/watchdog/wdt-uclass.c index ab8a64c354..bb9ae80866 100

[U-Boot] [PATCH v3] watchdog: Introduce watchdog driver for Intel Tangier

2017-07-05 Thread Andy Shevchenko
From: Felipe Balbi Add watchdog driver for Intel Tangier based platforms. Signed-off-by: Vincent Tinelli Signed-off-by: Felipe Balbi Signed-off-by: Andy Shevchenko --- In v3: - protect against overflow in hw_watchdog_reset() drivers/watchdog/Kconfig | 11 ++- drivers/watchdog/Mak

[U-Boot] [PATCH 3/9] efi_loader: implement InstallProtocolInterface

2017-07-05 Thread Heinrich Schuchardt
efi_install_protocol_interface up to now only returned an error code. The patch implements the UEFI specification for InstallProtocolInterface with the exception that it will not create new handles. Signed-off-by: Heinrich Schuchardt --- include/efi_api.h | 2 +- lib/efi_loader/efi

[U-Boot] [PATCH 8/9] efi_loader: refactor efi_locate_handle

2017-07-05 Thread Heinrich Schuchardt
To implement LocateHandleBuffer we need to call efi_locate_handle internally without running through EFI_EXIT. So put EFI_ENTRY and EFI_EXIT into a new wrapper function efi_locate_handle_ext. Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_boottime.c | 21 +++-- 1 file

[U-Boot] [PATCH 9/9] efi_loader: implement LocateHandleBuffer

2017-07-05 Thread Heinrich Schuchardt
UEFI boot service LocateHandleBuffer is implemented by calling efi_allocate_handle and efi_locate_handle. Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_boottime.c | 25 - 1 file changed, 24 insertions(+), 1 deletion(-) diff --git a/lib/efi_loader/efi_boottime

[U-Boot] [PATCH 0/9] EFI protocol services

2017-07-05 Thread Heinrich Schuchardt
Currently only a small fraction of the UEFI specification is implemented in U-Boot. It is sufficient to load grub but it fails on iPXE. With this patch series all EFI protocol services are implemented that are needed to reach the console prompt of the bin--efi/snponly.efi target of iPXE. Futher p

[U-Boot] [PATCH 7/9] efi_loader: implement InstallMultipleProtocolInterfaces

2017-07-05 Thread Heinrich Schuchardt
Implement InstallMultipleProtocolInterfaces in function efi_install_multiple_protocol_interfaces by repeatedly calling efi_install_protocol_interface. Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_boottime.c | 39 ++- 1 file changed, 38 insertions(

[U-Boot] [PATCH 5/9] efi_loader: refactor efi_install_protocol_interface

2017-07-05 Thread Heinrich Schuchardt
For the implementation of InstallMultipleProtocolInterfaces we need to call efi_install_protocol_interface. In internal calls we should not pass through EFI_EXIT. The patch introduces a wrapper function efi_install_protocol_interface_ext. Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/ef

[U-Boot] [PATCH 6/9] efi_loader: refactor efi_uninstall_protocol_interface

2017-07-05 Thread Heinrich Schuchardt
For the implementation of UninstallMultipleProtocolInterfaces we need to call efi_uninstall_protocol_interface. In internal calls we should not pass through EFI_EXIT. The patch introduces a wrapper function efi_uninstall_protocol_interface_ext. Signed-off-by: Heinrich Schuchardt --- lib/efi_loa

[U-Boot] [PATCH 1/9] efi_loader: refactor efi_open_protocol

2017-07-05 Thread Heinrich Schuchardt
efi_open_protocol was implemented to call a protocol specific 'open' function to retrieve the protocol interface. The UEFI specification does not know of such a function. It is not possible to implement InstallProtocolInterface with the current design. With the patch the protocol interface itsel

[U-Boot] [PATCH 2/9] efi_loader: efi_open_protocol: parameter checks

2017-07-05 Thread Heinrich Schuchardt
Add all parameter checks for function efi_open_protocol that do not depend on a locking table. Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_boottime.c | 31 +++ 1 file changed, 27 insertions(+), 4 deletions(-) diff --git a/lib/efi_loader/efi_boottime.c b

[U-Boot] [PATCH 4/9] efi_loader: implement UninstallProtocolInterface

2017-07-05 Thread Heinrich Schuchardt
Without the patch efi_uninstall_protocol_interface always returns an error. With the patch protocols without interface can be uninstalled. Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_boottime.c | 37 +++-- 1 file changed, 35 insertions(+), 2 deletio

Re: [U-Boot] [PATCH v1] watchdog: Introduce watchdog driver for Intel Tangier

2017-07-05 Thread Andy Shevchenko
On Tue, 2017-07-04 at 16:31 -0400, Tom Rini wrote: > On Tue, Jul 04, 2017 at 10:48:36PM +0300, Andy Shevchenko wrote: > > On Tue, 2017-04-18 at 16:49 +0200, Lukasz Majewski wrote: > > So, it looks to me now as bikeshedding, otherwise where is the > > documentation which describes how this all stuf

[U-Boot] [PATCH] Makefile: add dependencies to regenerate u-boot.cfg when lost

2017-07-05 Thread Philipp Tomsich
When running a 'make clean' or carelessly removing u-boot.cfg, all future make invocations (until autoconf is regenerated) will print an error for a missing u-boot.cfg due to missing rules and dependencies. This commit adds (i) an explicit rule dependency from all (which will invokes the configura

Re: [U-Boot] [PATCH 10/14] dm: mmc: sunxi: Add support for driver model

2017-07-05 Thread Maxime Ripard
On Tue, Jul 04, 2017 at 01:33:25PM -0600, Simon Glass wrote: > Hi Maxime, > > On 21 June 2017 at 01:31, Maxime Ripard > wrote: > > Hi Simon, > > > > On Mon, Jun 19, 2017 at 11:11:27AM -0600, Simon Glass wrote: > >> Add a driver-model version of this driver which mostly uses the existing > >> code

Re: [U-Boot] [PATCH v2 14/15] dm: sata: sunxi: Add support for driver model

2017-07-05 Thread Maxime Ripard
On Tue, Jul 04, 2017 at 01:31:31PM -0600, Simon Glass wrote: > Adjust SATA setup to support driver model. > > Signed-off-by: Simon Glass Acked-by: Maxime Ripard Thanks! Maxime -- Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com signature.asc Des

Re: [U-Boot] [PATCH v2 13/15] dm: sunxi: sata: Don't build sata support into SPL

2017-07-05 Thread Maxime Ripard
On Tue, Jul 04, 2017 at 01:31:30PM -0600, Simon Glass wrote: > This is not used in SPL so we do not need to compile it. Make this change > before adding driver-model support to the driver, to avoid build errors. > With driver model we define a U_BOOT_DRIVER() which would otherwise be > present in S

Re: [U-Boot] [PATCH v2 11/15] dm: sunxi: Linksprite_pcDuino3: Correct polarity of MMC card detect

2017-07-05 Thread Maxime Ripard
Hi, On Tue, Jul 04, 2017 at 01:31:28PM -0600, Simon Glass wrote: > This is shown as active high in the schematics[1], so fix it. > > [1] https://patchwork.ozlabs.org/patch/777890/ > > Signed-off-by: Simon Glass > Reported-by: Maxime Ripard > --- > > Changes in v2: > - Add new patch to correct

Re: [U-Boot] [PATCH v2 09/15] dm: mmc: sunxi: Drop mmc_clk_io_on()

2017-07-05 Thread Maxime Ripard
On Tue, Jul 04, 2017 at 01:31:26PM -0600, Simon Glass wrote: > This function has #ifdefs in it which we want to avoid for driver model. > Instead we should use different compatible strings and the .data field. > It also uses the MMC device number which is not available in driver > model except thro

Re: [U-Boot] [PATCH v2 08/15] dm: mmc: sunxi: Pass private data around explicitly

2017-07-05 Thread Maxime Ripard
On Tue, Jul 04, 2017 at 01:31:25PM -0600, Simon Glass wrote: > At present the driver-private data is obtained in various functions by > various means. With driver model this is provided automatically. Without > driver model it comes from a C array declared at the top of the file. > > Adjust intern

Re: [U-Boot] [PATCH v2 07/15] dm: mmc: sunxi: Rename mmchost to priv

2017-07-05 Thread Maxime Ripard
On Tue, Jul 04, 2017 at 01:31:24PM -0600, Simon Glass wrote: > Use the driver-model naming convention for this structure. It is data > private to the driver so the local variable should be called 'priv'. > > Signed-off-by: Simon Glass Acked-by: Maxime Ripard Maxime -- Maxime Ripard, Free Ele

Re: [U-Boot] [PATCH v2 06/15] dm: mmc: sunxi: Rename struct sunxi_mmc_host to sunxi_mmc_priv

2017-07-05 Thread Maxime Ripard
On Tue, Jul 04, 2017 at 01:31:23PM -0600, Simon Glass wrote: > Use the driver-model naming convention for this structure. It is data > private to the driver. > > Signed-off-by: Simon Glass Acked-by: Maxime Ripard Maxime -- Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering h

Re: [U-Boot] [PATCH v1] arch/x86: Select USB before selecting host driver

2017-07-05 Thread Tom Rini
On Wed, Jul 05, 2017 at 08:19:27PM +0800, Bin Meng wrote: > +Marek, > > Hi Andy, > > On Wed, Jul 5, 2017 at 5:36 PM, Andy Shevchenko > wrote: > > On Wed, Jul 5, 2017 at 12:36 PM, Andy Shevchenko > > wrote: > >> On Wed, Jul 5, 2017 at 8:23 AM, Bin Meng wrote: > > > >> So, options are: > >> 1) a

[U-Boot] [PATCH 1/1] board: ls2080ardb: Add fsl_fdt_fixup_flash

2017-07-05 Thread Santan Kumar
IFC and QSPI are muxed on board. Add fsl_fdt_fixup_flash() -To disable IFC node in dts if QSPI is enabled. -Or disable QSPI node in dts if IFC is enabled. Signed-off-by: Santan Kumar Signed-off-by: Priyanka Jain --- board/freescale/ls2080ardb/ls2080ardb.c | 28 1 f

Re: [U-Boot] [PATCH v1] arch/x86: Select USB before selecting host driver

2017-07-05 Thread Bin Meng
+Marek, Hi Andy, On Wed, Jul 5, 2017 at 5:36 PM, Andy Shevchenko wrote: > On Wed, Jul 5, 2017 at 12:36 PM, Andy Shevchenko > wrote: >> On Wed, Jul 5, 2017 at 8:23 AM, Bin Meng wrote: > >> So, options are: >> 1) apply my patch >> 2) remove selection of the host at least for X86. > > 3) move hos

Re: [U-Boot] [PATCH v2] avr32: Retire AVR32 for good

2017-07-05 Thread Heiko Schocher
Hello Andy, Am 05.07.2017 um 13:27 schrieb Andy Shevchenko: AVR32 is gone. It's already more than two years for no support in Buildroot, even longer there is no support in GCC (last version is heavily patched 4.2.4). Linux kernel v4.12 got rid of it (and v4.11 didn't build successfully). There

Re: [U-Boot] [BUG] checkpatch: false positive storage class location

2017-07-05 Thread Joe Perches
On Tue, 2017-07-04 at 23:08 +0200, Heinrich Schuchardt wrote: > On 07/04/2017 10:44 PM, Joe Perches wrote: > > On Tue, 2017-07-04 at 21:29 +0200, Heinrich Schuchardt wrote: > > > The U-Boot project uses the same scripts/checkpatch.pl as the Linux > > > kernel. I ran upon the problem below when work

Re: [U-Boot] [PATCH next 3/3] mx6sabreauto: Update to SPL only mode

2017-07-05 Thread Stefano Babic
On 29/06/2017 14:33, Vanessa Maegima wrote: > As mx6sabreauto supports SPL now, all variants can boot using the same > defconfig. > > This patch: > - Removes non-SPL targets. > - Renames target to mx6sabreauto_defconfig. > - Renames folder and board files to mx6sabreauto. > - Updates MAINTAINERS,

Re: [U-Boot] [PATCH v1] avr32: Retire AVR32 for good

2017-07-05 Thread Heiko Schocher
Hello Andy, Am 05.07.2017 um 11:49 schrieb Andy Shevchenko: On Wed, Jul 5, 2017 at 6:08 AM, Heiko Schocher wrote: Am 04.07.2017 um 18:25 schrieb Andy Shevchenko: AVR32 is gone. It's already more than two years for no support in Buildroot, even longer there is no support in GCC (last version

Re: [U-Boot] [PATCH v1] avr32: Retire AVR32 for good

2017-07-05 Thread Andy Shevchenko
On Tue, Jul 4, 2017 at 7:25 PM, Andy Shevchenko wrote: > AVR32 is gone. It's already more than two years for no support in Buildroot, > even longer there is no support in GCC (last version is heavily patched > 4.2.4). > > Linux kernel v4.12 got rid of it (and v4.11 didn't build successfully). > >

Re: [U-Boot] [PATCH] mx6cuboxi: Add support for sata

2017-07-05 Thread Stefano Babic
On 01/07/2017 20:04, Fabio Estevam wrote: > On Sat, Jul 1, 2017 at 2:44 PM, Peter Robinson wrote: >> The Cubox-i and Hummingboard series of devices have an option of >> SATA on board, and depending on how the fuses are blown even the >> option to boot SPL from SATA. So enable support for it so it

Re: [U-Boot] [PATCH v3 1/1] mx7dsabresd: Set VLD04 output to 2.8V in PMIC initialization.

2017-07-05 Thread Stefano Babic
On 03/07/2017 00:09, Fabio Estevam wrote: > On Sun, Jul 2, 2017 at 4:20 PM, Gautam Bhat wrote: >> This change sets the VLDO4 settings output to 2.8V in PMIC >> initialization so that the MIPI DSI/CSI input voltage is 2.8V >> as per the schematics. The original code provides an output of >> 3.3V wh

  1   2   >