Re: [PATCH] arm64: defconfig: Enable Texas Instruments UDMA driver

From: Peter Ujfalusi <peter.ujfalusi@ti.com>
To: Olof Johansson <olof@lixom.net>
Cc: Alexandre Belloni <alexandre.belloni@bootlin.com>,
	Arnd Bergmann <arnd@arndb.de>, SoC Team <soc@kernel.org>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Tero Kristo <t-kristo@ti.com>, Vinod Koul <vkoul@kernel.org>,
	Will Deacon <will@kernel.org>,
	Linux ARM Mailing List <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH] arm64: defconfig: Enable Texas Instruments UDMA driver
Date: Tue, 28 Jan 2020 10:21:08 +0200	[thread overview]
Message-ID: <a81fa6b0-811c-82af-4cf6-e2f4ac3c0ded@ti.com> (raw)
In-Reply-To: <CAOesGMiFw2V6fdbGCOfgsVq+WK-4ijdzivDdX3hbS2E=bO4zkg@mail.gmail.com>

Hi Olof,

On 27/01/2020 17.30, Olof Johansson wrote:
>>> I also see that this is statically enabling this driver -- we try to keep as
>>> many drivers as possible as modules to avoid the static kernel from growing too
>>> large. Would that be a suitable approach here, or is the driver needed to reach
>>> rootfs for further module loading?
>>
>> We would need built in DMA for nfs rootfs, SD/MMC has it's own buit-in
>> ADMA. We do not have network drivers upstream as they depend on the DMA.
> 
> Ok, so that can either be turned into a ramdisk argument, or into a
> "we really want to enable non-ramdisk rootfs boots on this hardware
> because it's a common use case".

SD/MMC does not need slave DMA, it is self containing with it's own
built-in DMA.
I'm not sure if it is enabled in defconfig. It is not enabled at all in
defconfig atm.

Normally I would use nfs rootfs, but we don't have network drivers
upstream for K3 platform.

I think having the UDMA stack as module should be fine when I have the
dependencies in to be able to build them as modules.

> I find it useful to challenge most of the =y drivers to make people
> think about it, and at some point we'll enough overhead of cruft in
> the static superset kernel that defconfig today is used for such that
> we need to prune more =y -> =m,

Sure, I fully agree on this, we should have non boot needed drivers as
modules.

> but this particular driver is probably
> OK (it's also not large).

Well, it depends how you look at it ;)

UDMA stack is not enabled in defconfig (w/o this patch):
$ size vmlinux
text      data     bss     dec       hex      filename
17853717  9221872  469288  27544877  1a44d2d  vmlinux

UDMA stack is enabled in defconfig (w this patch):
$ size vmlinux
text      data     bss     dec       hex      filename
17890970  9237544  469288  27597802  1a51bea  vmlinux

It would be nice for other driver to enable the DMA if it is acceptable
to have it built in for start and when I can build it as module we can
switch it to module?

>> Imho module would be fine for the DMA stack, but neither ringacc or the
>> UDMA driver can be built as module atm until the following patches got
>> merged:
>> https://lore.kernel.org/lkml/20200122104723.16955-1-peter.ujfalusi@ti.com/
>> https://lore.kernel.org/lkml/20200122104031.15733-1-peter.ujfalusi@ti.com/
>>
>> I have the patches to add back module support, but waiting on these
>> currently.
> 
> -Olof
> 

- Péter

Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel