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

[Peter Robinson]

Hi Nicolas,

> > > 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?

It's just the list of configs we build for Fedora.

> >   /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
>