Hi Peter, On Thu, 2020-12-10 at 17:03 +0000, Peter Robinson wrote: > Hi Nicolas, > > > On Thu, Nov 19, 2020 at 5:50 PM Nicolas Saenz Julienne > <nsaenzjulie...@suse.de> wrote: > > > > Add the follwing functions to get a specific device's DMA ranges: > > - dev_get_dma_range() > > - ofnode_get_dma_range() > > - of_get_dma_range() > > - fdt_get_dma_range() > > They are specially useful in oder to be able validate a physical address > > space range into a bus's and to convert addresses from and to address > > spaces. > > I'm seeing issues with this patch, with the just posted v2 as well > (it's not arrived in my inbox as yet).
I think didn't directly CC you, sorry for that. I'll keep it in mind for the next round. > I get the following linking error when building across a number of devices > such as Jetson boards like jetson-tk1, p3450-0000, as well as puma-rk3399, > marsboard, udoo, udoo_neo. The error is: Thanks for having a lookg at this. I've been strugling to test my series against all posible configurations. What are you using catch the regressions? Is something I can run locally? > /usr/bin/arm-linux-gnu-ld.bfd -pie --gc-sections -Bstatic > --no-dynamic-linker -Ttext 0x87800000 -o u-boot -T u-boot.lds > arch/arm/cpu/armv7/start.o --start-group arch/arm/cpu/built-in.o > arch/arm/cpu/armv7/built-in.o arch/arm/lib/built-in.o > arch/arm/mach-imx/built-in.o board/udoo/neo/built-in.o > cmd/built-in.o common/built-in.o disk/built-in.o drivers/built-in.o > drivers/dma/built-in.o drivers/gpio/built-in.o > drivers/i2c/built-in.o drivers/net/built-in.o > drivers/net/phy/built-in.o drivers/power/built-in.o > drivers/power/battery/built-in.o drivers/power/domain/built-in.o > drivers/power/fuel_gauge/built-in.o drivers/power/mfd/built-in.o > drivers/power/pmic/built-in.o drivers/power/regulator/built-in.o > drivers/serial/built-in.o drivers/spi/built-in.o > drivers/usb/cdns3/built-in.o drivers/usb/common/built-in.o > drivers/usb/dwc3/built-in.o drivers/usb/emul/built-in.o > drivers/usb/eth/built-in.o drivers/usb/host/built-in.o > drivers/usb/mtu3/built-in.o drivers/usb/musb-new/built-in.o > drivers/usb/musb/built-in.o drivers/usb/phy/built-in.o > drivers/usb/ulpi/built-in.o env/built-in.o fs/built-in.o > lib/built-in.o net/built-in.o --end-group arch/arm/lib/eabi_compat.o > arch/arm/lib/lib.a -Map u-boot.map; true > /usr/bin/arm-linux-gnu-ld.bfd: drivers/built-in.o: in function > `dev_get_dma_range': > /home/perobins/fedora/packages/uboot-tools/u-boot-2021.01-rc3/include/dm/read.h:1025: > undefined reference to `ofnode_get_dma_range' > make[1]: *** > [/home/perobins/fedora/packages/uboot-tools/u-boot-2021.01-rc3/Makefile:1757: > u-boot] Error 1 > make[1]: Leaving directory > '/home/perobins/fedora/packages/uboot-tools/u-boot-2021.01-rc3/builds/udoo_neo' I reproduced it locally, thanks! I'll fit it. Regards, Nicolas
signature.asc
Description: This is a digitally signed message part