[U-Boot] u-boot-usb/master build failure on dra7xx_evm

Hi,
can you look at this build problem in u-boot-usb/master on dra7xx_evm and send me a patch?
https://travis-ci.org/marex/u-boot-usb/jobs/367566783
u-boot-usb$ bu dra7xx_evm In file included from drivers/usb/host/dwc3-of-simple.c:17:0: include/reset.h:246:62: warning: ‘struct clk_bulk’ declared inside parameter list will not be visible outside of this definition or declaration static inline int reset_get_bulk(struct udevice *dev, struct clk_bulk *bulk) ^~~~~~~~ include/reset.h:287:45: warning: ‘struct clk_bulk’ declared inside parameter list will not be visible outside of this definition or declaration static inline int reset_release_bulk(struct clk_bulk *bulk) ^~~~~~~~ drivers/usb/host/dwc3-of-simple.c: In function ‘dwc3_of_simple_reset_init’: drivers/usb/host/dwc3-of-simple.c:32:28: warning: passing argument 2 of ‘reset_get_bulk’ from incompatible pointer type [-Wincompatible-pointer-types] ret = reset_get_bulk(dev, &simple->resets); ^ In file included from drivers/usb/host/dwc3-of-simple.c:17:0: include/reset.h:246:19: note: expected ‘struct clk_bulk *’ but argument is of type ‘struct reset_ctl_bulk *’ static inline int reset_get_bulk(struct udevice *dev, struct clk_bulk *bulk) ^~~~~~~~~~~~~~ drivers/usb/host/dwc3-of-simple.c:40:22: warning: passing argument 1 of ‘reset_release_bulk’ from incompatible pointer type [-Wincompatible-pointer-types] reset_release_bulk(&simple->resets); ^ In file included from drivers/usb/host/dwc3-of-simple.c:17:0: include/reset.h:287:19: note: expected ‘struct clk_bulk *’ but argument is of type ‘struct reset_ctl_bulk *’ static inline int reset_release_bulk(struct clk_bulk *bulk) ^~~~~~~~~~~~~~~~~~ drivers/usb/host/dwc3-of-simple.c: In function ‘dwc3_of_simple_remove’: drivers/usb/host/dwc3-of-simple.c:89:21: warning: passing argument 1 of ‘reset_release_bulk’ from incompatible pointer type [-Wincompatible-pointer-types] reset_release_bulk(&simple->resets); ^ In file included from drivers/usb/host/dwc3-of-simple.c:17:0: include/reset.h:287:19: note: expected ‘struct clk_bulk *’ but argument is of type ‘struct reset_ctl_bulk *’ static inline int reset_release_bulk(struct clk_bulk *bulk)

Hi Marek,
that should be fixed by applying https://patchwork.ozlabs.org/patch/897532/
JJ
On 17/04/2018 20:25, Marek Vasut wrote:
Hi,
can you look at this build problem in u-boot-usb/master on dra7xx_evm and send me a patch?
https://travis-ci.org/marex/u-boot-usb/jobs/367566783
u-boot-usb$ bu dra7xx_evm In file included from drivers/usb/host/dwc3-of-simple.c:17:0: include/reset.h:246:62: warning: ‘struct clk_bulk’ declared inside parameter list will not be visible outside of this definition or declaration static inline int reset_get_bulk(struct udevice *dev, struct clk_bulk *bulk) ^~~~~~~~ include/reset.h:287:45: warning: ‘struct clk_bulk’ declared inside parameter list will not be visible outside of this definition or declaration static inline int reset_release_bulk(struct clk_bulk *bulk) ^~~~~~~~ drivers/usb/host/dwc3-of-simple.c: In function ‘dwc3_of_simple_reset_init’: drivers/usb/host/dwc3-of-simple.c:32:28: warning: passing argument 2 of ‘reset_get_bulk’ from incompatible pointer type [-Wincompatible-pointer-types] ret = reset_get_bulk(dev, &simple->resets); ^ In file included from drivers/usb/host/dwc3-of-simple.c:17:0: include/reset.h:246:19: note: expected ‘struct clk_bulk *’ but argument is of type ‘struct reset_ctl_bulk *’ static inline int reset_get_bulk(struct udevice *dev, struct clk_bulk *bulk) ^~~~~~~~~~~~~~ drivers/usb/host/dwc3-of-simple.c:40:22: warning: passing argument 1 of ‘reset_release_bulk’ from incompatible pointer type [-Wincompatible-pointer-types] reset_release_bulk(&simple->resets); ^ In file included from drivers/usb/host/dwc3-of-simple.c:17:0: include/reset.h:287:19: note: expected ‘struct clk_bulk *’ but argument is of type ‘struct reset_ctl_bulk *’ static inline int reset_release_bulk(struct clk_bulk *bulk) ^~~~~~~~~~~~~~~~~~ drivers/usb/host/dwc3-of-simple.c: In function ‘dwc3_of_simple_remove’: drivers/usb/host/dwc3-of-simple.c:89:21: warning: passing argument 1 of ‘reset_release_bulk’ from incompatible pointer type [-Wincompatible-pointer-types] reset_release_bulk(&simple->resets); ^ In file included from drivers/usb/host/dwc3-of-simple.c:17:0: include/reset.h:287:19: note: expected ‘struct clk_bulk *’ but argument is of type ‘struct reset_ctl_bulk *’ static inline int reset_release_bulk(struct clk_bulk *bulk)

On 17/04/2018 20:35, Jean-Jacques Hiblot wrote:
Hi Marek,
that should be fixed by applying https://patchwork.ozlabs.org/patch/897532/
You should also apply https://patchwork.ozlabs.org/patch/897537/ Without it the probe fails on dra7xx-evm
JJ
On 17/04/2018 20:25, Marek Vasut wrote:
Hi,
can you look at this build problem in u-boot-usb/master on dra7xx_evm and send me a patch?
https://travis-ci.org/marex/u-boot-usb/jobs/367566783
u-boot-usb$ bu dra7xx_evm In file included from drivers/usb/host/dwc3-of-simple.c:17:0: include/reset.h:246:62: warning: ‘struct clk_bulk’ declared inside parameter list will not be visible outside of this definition or declaration static inline int reset_get_bulk(struct udevice *dev, struct clk_bulk *bulk) ^~~~~~~~ include/reset.h:287:45: warning: ‘struct clk_bulk’ declared inside parameter list will not be visible outside of this definition or declaration static inline int reset_release_bulk(struct clk_bulk *bulk) ^~~~~~~~ drivers/usb/host/dwc3-of-simple.c: In function ‘dwc3_of_simple_reset_init’: drivers/usb/host/dwc3-of-simple.c:32:28: warning: passing argument 2 of ‘reset_get_bulk’ from incompatible pointer type [-Wincompatible-pointer-types] ret = reset_get_bulk(dev, &simple->resets); ^ In file included from drivers/usb/host/dwc3-of-simple.c:17:0: include/reset.h:246:19: note: expected ‘struct clk_bulk *’ but argument is of type ‘struct reset_ctl_bulk *’ static inline int reset_get_bulk(struct udevice *dev, struct clk_bulk *bulk) ^~~~~~~~~~~~~~ drivers/usb/host/dwc3-of-simple.c:40:22: warning: passing argument 1 of ‘reset_release_bulk’ from incompatible pointer type [-Wincompatible-pointer-types] reset_release_bulk(&simple->resets); ^ In file included from drivers/usb/host/dwc3-of-simple.c:17:0: include/reset.h:287:19: note: expected ‘struct clk_bulk *’ but argument is of type ‘struct reset_ctl_bulk *’ static inline int reset_release_bulk(struct clk_bulk *bulk) ^~~~~~~~~~~~~~~~~~ drivers/usb/host/dwc3-of-simple.c: In function ‘dwc3_of_simple_remove’: drivers/usb/host/dwc3-of-simple.c:89:21: warning: passing argument 1 of ‘reset_release_bulk’ from incompatible pointer type [-Wincompatible-pointer-types] reset_release_bulk(&simple->resets); ^ In file included from drivers/usb/host/dwc3-of-simple.c:17:0: include/reset.h:287:19: note: expected ‘struct clk_bulk *’ but argument is of type ‘struct reset_ctl_bulk *’ static inline int reset_release_bulk(struct clk_bulk *bulk)
U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

On 04/17/2018 08:38 PM, Jean-Jacques Hiblot wrote:
On 17/04/2018 20:35, Jean-Jacques Hiblot wrote:
Hi Marek,
that should be fixed by applying https://patchwork.ozlabs.org/patch/897532/
You should also apply https://patchwork.ozlabs.org/patch/897537/ Without it the probe fails on dra7xx-evm
That one is already applied.
The reset fix is for Tom to pick I think.
participants (2)
-
Jean-Jacques Hiblot
-
Marek Vasut