On 10/11/23 09:16, Peter Griffin wrote:

Hi Tudor,

Thanks for your reply.

On Wed, 11 Oct 2023 at 07:10, Tudor Ambarus [off-list ref] wrote:

quoted

Hi, Peter,

On 10/10/23 23:49, Peter Griffin wrote:

quoted

Note 3: In `dt-bindings: pinctrl: samsung: add google,gs101-pinctrl
compatible` I tried to narrow the interrupts check to
google,gs101-pinctrl but I still see a warning: gs101-oriole.dtb:
pinctrl@174d0000: interrupts: [[0, 0, 4],[..] is too long If anyone can
educate me on what I've done wrong here it would be most appreciated!

I guess the initial definition of the number of interrupts should
include the largest min/maxItems. I no longer see the warning with this
change:

Yes that is how it was in v1. The review feedback though was to narrow
the scope to just google,gs101-pinctrl compatible using if: then: else: which
is what I can't get to work properly.

Right. The diff that I sent is on top of your changes (patch 6/20).
I expect that when the interrupts property is defined it should include
the min/maxItems of all the available SoCs. Then use "if Soc" to narrow
the range.

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