Re: [PATCH] ARM: dts: stm32: omit unused pinctrl groups from stm32mp13 dtb files

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 7/4/24 19:56, Marek Vasut wrote:
On 7/4/24 6:40 PM, Alexandre TORGUE wrote:
Hi Marek

Hi,

On 6/29/24 22:36, Marek Vasut wrote:
stm32mp13-pinctrl.dtsi contains nearly all pinctrl groups collected from
all boards. Most of them end up unused by a board and only waste binary
space. Add /omit-if-no-ref/ to the groups to scrub the unused groups
from the dtbs.

Use the following regex to update the file and drop two useless newlines too:
s@^\t[^:]\+: [^ ]\+ {$@\t/omit-if-no-ref/\r&@

I understand the aim of this patch but I'm just wondering about DT overlay that would need one of those configurations. IMO, in this case the DT overlay will not apply.

It is always a compromise, either we will have massive ever-growing base DTs or DTOs would have to ship their own pinmux settings. I think I am fine with the later option, also that's what the MP15 is doing already.

Yes, it is probably the best solution to define pinmux in DTSO that would need a new pin config. For some use case, we will have pinconfig duplication in dtso but as you said it is a good compromise.




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux