On Tue, May 30, 2017 at 09:52:35PM -0600, Simon Glass wrote: > Hi Tom, > > On 28 May 2017 at 05:55, Tom Rini <[email protected]> wrote: > > On Sat, May 27, 2017 at 04:01:29PM -0600, Simon Glass wrote: > > > >> Hi Tom, > >> > >> This is similar to the last attempt, but without this offending patch > >> which will be replaced by a little series sent today: (I'll pull in > >> that series later) > >> > >> 0a2980b dm: mmc: Avoid probing block devices in find_mmc_device() > >> > >> > >> The following changes since commit > >> 380e86f361e4e2aef83295972863654fde157560: > >> > >> Merge git://git.denx.de/u-boot-fsl-qoriq (2017-05-26 11:19:27 -0400) > >> > >> are available in the git repository at: > >> > >> git://git.denx.de/u-boot-dm.git > >> > >> for you to fetch changes up to 40fcab41cbcace3ff5928f146dd15c1ee3bfac65: > >> > >> sandbox: Move to use live tree (2017-05-27 10:38:13 -0600) > >> > > > > Two problems. First, you need to add this to "dm: gpio: Add live tree > > support": > > > > diff --git a/board/st/stm32f746-disco/stm32f746-disco.c > > b/board/st/stm32f746-disco/stm32f746-disco.c > > index aeaa31146a5a..addf82bbae8c 100644 > > --- a/board/st/stm32f746-disco/stm32f746-disco.c > > +++ b/board/st/stm32f746-disco/stm32f746-disco.c > > @@ -101,8 +101,8 @@ int board_late_init(void) > > if (node < 0) > > return -1; > > > > - gpio_request_by_name_nodev(gd->fdt_blob, node, "led-gpio", 0, &gpio, > > - GPIOD_IS_OUT); > > + gpio_request_by_name_nodev(offset_to_ofnode(node), "led-gpio", 0, > > + &gpio, GPIOD_IS_OUT); > > > > if (dm_gpio_is_valid(&gpio)) { > > dm_gpio_set_value(&gpio, 0); > > @@ -115,8 +115,8 @@ int board_late_init(void) > > if (node < 0) > > return -1; > > > > - gpio_request_by_name_nodev(gd->fdt_blob, node, "button-gpio", 0, > > &gpio, > > - GPIOD_IS_IN); > > + gpio_request_by_name_nodev(offset_to_ofnode(node), "button-gpio", 0, > > + &gpio, GPIOD_IS_IN); > > > > if (dm_gpio_is_valid(&gpio)) { > > if (dm_gpio_get_value(&gpio)) > > OK, added, thanks. For me that board fails to build since it has something > like: > > for (int i = ....) > > Is that allowed now?
It probably shouldn't have come in and should be fixed, oh well. -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list [email protected] https://lists.denx.de/listinfo/u-boot
