[PATCH 4/8] dm: Introduce xxx_get_dma_range()

[Nicolas Saenz Julienne]

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
> <nsaenzjulienne at 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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: This is a digitally signed message part
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20201210/83f7dafe/attachment.sig>