Hi Simon, On 28/09/2022 19:27, Simon Glass wrote: > Hi Roger, > > On Wed, 28 Sept 2022 at 06:12, Roger Quadros <rog...@kernel.org> wrote: >> >> We will need ti-gpmc driver for SPL. Allow memory drivers >> do be built for SPL. >> >> Signed-off-by: Roger Quadros <rog...@kernel.org> >> --- >> scripts/Makefile.spl | 1 + >> 1 file changed, 1 insertion(+) > > Please can you use the existing drivers/ram directory?
The ti-gpmc driver is not actually a RAM only controller. Although it can support SRAM. It is a more general purpose controller that can support different peripherals. It is similar to the drivers already existing in the divers/memory directory. I was just trying to keep the file layout similar to that in the Linux kernel. Do you still see a problem with it? > >> >> diff --git a/scripts/Makefile.spl b/scripts/Makefile.spl >> index 3bafeb4fe9..110076b22f 100644 >> --- a/scripts/Makefile.spl >> +++ b/scripts/Makefile.spl >> @@ -114,6 +114,7 @@ libs-$(CONFIG_PARTITIONS) += disk/ >> endif >> >> libs-y += drivers/ >> +libs-y += drivers/memory/ >> libs-$(CONFIG_SPL_USB_GADGET) += drivers/usb/dwc3/ >> libs-$(CONFIG_SPL_USB_GADGET) += drivers/usb/cdns3/ >> libs-y += dts/ >> -- >> 2.17.1 >> > > Regards, > Simon cheers, -roger