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

[Marek Vasut]

On 7/5/24 9:06 AM, Alexandre TORGUE wrote:

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.

Maybe someone else will chime in, let's see what the overall opinion is.

btw I didn't write it is a _good_ compromise, I wrote it is _a_ 
compromise. I think nobody is particularly happy about it one way or the 
other in this case :)