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 :)
