Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Thu, 10 Nov 2022 00:45:25 +0800
From:   Chester Lin <clin@suse.com>
To:     Linus Walleij <linus.walleij@linaro.org>
Cc:     Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        s32@nxp.com, linux-gpio@vger.kernel.org,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org,
        Larisa Grigore <larisa.grigore@nxp.com>,
        Ghennadi Procopciuc <Ghennadi.Procopciuc@nxp.com>,
        Andreas =?iso-8859-1?Q?F=E4rber?= <afaerber@suse.de>,
        Matthias Brugger <mbrugger@suse.com>
Subject: Re: [PATCH 1/2] dt-bindings: pinctrl: add schema for NXP S32 SoCs
Message-ID: <Y2vZJaoWThP2aipE@linux-8mug>
References: <20221031100843.14579-1-clin@suse.com>
 <20221031100843.14579-2-clin@suse.com>
 <CACRpkdY-uaQ--vFM+vVPbwa-q9nbSU0rQB+qbL=9m0wVMwA3Aw@mail.gmail.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <CACRpkdY-uaQ--vFM+vVPbwa-q9nbSU0rQB+qbL=9m0wVMwA3Aw@mail.gmail.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?OJDusVB0MYMJSXOHfgs5r1vuTsUpPeWnq6Ubnn909EIoEz1JS/DOudIfu4kl?=
 =?us-ascii?Q?EMXMjPKS4VzLTavCTlXdUHKCTDYEhgl99i02/ZC+WiFnXU2bMHrZ2k3E4JCw?=
 =?us-ascii?Q?kM1rtEmdAgHFHQguTMd93uzq2qi5n+NfPZnvpp1jqipSJMyKN34D/oGVAt+3?=
 =?us-ascii?Q?qZWpTpB1XxL1afwOJ3lOqThxCrl4wDFVAVRzeyq6a2Aa9Il6SrTLwnhOdPlm?=
 =?us-ascii?Q?1INNnb8SR+lQ2d9Wifd8R0jZ5dfiXWjEQCfDAkT7wZLYa4d5r+rqwverqgKf?=
 =?us-ascii?Q?eSF4QV8r/E1nXGPmDCMVR0DvmMhyfs5D0Jz/wwwX6CfJYfyJUgc8k+sUUXcn?=
 =?us-ascii?Q?OGRtXKPX9DPMolCM6kNuHoW8eT6HhYaBpolWhkK1CTIByKz9IdjRUyGYqH4s?=
 =?us-ascii?Q?9tbg3cFSRCxLwCGf8gsWPKYdbVvjZjWxTjhcxW53jPN7yraIkfWE+RlXnl+L?=
 =?us-ascii?Q?6IywRXidF/HHUS2CNw7i0xp/ITsBjhbGexhjJiol1LHe8iyfSjxYVU4g5CY4?=
 =?us-ascii?Q?afHbzUoPc8XmsR7GwNYegmck51dSEKwtEEvxyg+hxyalvu/5Ubrd4OS06ZOH?=
 =?us-ascii?Q?z3jV13VKTwYaIn1f0z2WPef1i/5q6bUogeIsXqOh1/gkF0p5oVYRS3F61IfC?=
 =?us-ascii?Q?jqOfHyvSw19G8Yol/GccSagv4uVcxPXXhC0fAT22kI/Px2Btgw3kjE3RJNh+?=
 =?us-ascii?Q?w2yNmAvh+aC1EChGS0H93zLvk1gXBY7At6udOh7tNbkBfWzKkD5CNrahZcfP?=
 =?us-ascii?Q?ibSEt77pT/xI9sRa1Q7yzri4+nMI5XyGvjn0sQyQpVNIaP+wKn0uwmv9/wd6?=
 =?us-ascii?Q?/2qyoj11phuZ5i7FOo/REFOLCDTXbEXPEPC2gTom6kmnfzQvnph7NfXDHYZ1?=
 =?us-ascii?Q?evSig+hnfHe6l/5y0POJcLmq9i9JV0oRFpL3ZqIUcI3PPuDipuemVn+7Cg7r?=
 =?us-ascii?Q?GFzucqDlHZvENvZ1wkEwwxgqcw9sQ8QJxuehYDw24Q3p9Nyh9qkHdVw2TULS?=
 =?us-ascii?Q?PTXeGvvL7YCF2rPTsioc47GZXfgk8eGQ3wdv1IEgCHS85wxZIfPINdCgTgZY?=
 =?us-ascii?Q?pHnwpYh2fx1xaJD4CQ6wJ/X2tBGglFVTaOk3VQ63NPdcmgmZtODqzY76teoP?=
 =?us-ascii?Q?CLVCPmZ2SLyQyWDUJdFI16dwUbAVVVfV8DZaN5eABoVkok8Iea39fWfZ9EOM?=
 =?us-ascii?Q?9Cy16rziilZ7QfTSDUXnUkNWgqGqJYiIJ21e9wTw3Qka6ilQAchklZGFkEjl?=
 =?us-ascii?Q?c3ib9zCnqmqbUqij37O56yhDCQq8l9jCER5wAC44XuPs4/sRIXjtkxLeHYa3?=
 =?us-ascii?Q?T+ettzdbyIBWdnxgcYfcugANB5FIHlGqeB6FiDMuxoRrXmj98qTsVOWWSOe6?=
 =?us-ascii?Q?3CpVXEJjszbAv2yoKJ56/VwyRfZ4Ygo4ZNVhMOqPdF5HhVLLi7G9Q4U/cDb2?=
 =?us-ascii?Q?w+psTqGKngaa9fkdTLvmm+SAUsq4eDIZ4nYgcGiPol+bM03I96UCm26Ys7aB?=
 =?us-ascii?Q?bntVekb8RbbmeQzcjrzmNTgSdnTQA32TuuWvl2O/xp/29A+HJvi+gMLPQeJI?=
 =?us-ascii?Q?fsVQ+Q9MCWn4H3h3hos=3D?=
X-OriginatorOrg: suse.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 733d373e-51d2-490d-57e1-08dac271d50c
X-MS-Exchange-CrossTenant-AuthSource: VI1PR0402MB3439.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Nov 2022 16:45:41.4824
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: f7a17af6-1c5c-4a36-aa8b-f5be247aa4ba
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: XsqpQggptyZg9OM8ufhRddL84uJBkdDDKpFluBqzdYjOAjF5EbW65HRyloJsTBTD
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DBBPR04MB7804
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Linus,

On Tue, Nov 08, 2022 at 01:31:16PM +0100, Linus Walleij wrote:
> Hi Chester,
> 
> thanks for your patch!
> 

Thanks for reviewing this patch series as well!

> On Mon, Oct 31, 2022 at 11:09 AM Chester Lin <clin@suse.com> wrote:
> 
> > Add DT schema for the pinctrl driver of NXP S32 SoC family.
> >
> > Signed-off-by: Larisa Grigore <larisa.grigore@nxp.com>
> > Signed-off-by: Ghennadi Procopciuc <Ghennadi.Procopciuc@nxp.com>
> > Signed-off-by: Chester Lin <clin@suse.com>
> (...)
> > +  nxp,pins:
> > +    $ref: /schemas/types.yaml#/definitions/uint32-array
> > +    description:
> > +      A list of [start, end] pin ID boundaries that correspond to each of
> > +      the register regions reserved.
> 
> That's an interesting approach.
> 
> But what about just hardcoding this into the driver instead?
> 
> If you have the compatible, surely you know these indexes from
> that compatible string?
> 

The nxp,pins property is more like a common language that both kernel and
u-boot can share with. Of course hardcoding indexes can be achieved as well
but that also means the index matrices in both kernel driver and u-boot driver
must be revised and synchronized if new boards may have different reg and
nxp,pins. IMHO, having these ID ranges in DT could be more intuitive, developers
can easily compare every pair with the corresponding reg entry in the same dts
file since reg maps are fragmented as I mentioned in Rob's code review[1].
Based on this approach, both kernel driver and u-boot driver can reuse the
same parser to acquire pin ID ranges from different boards, which can also
simplify codes and have less patches in the future.

Regards,
Chester

[1] https://lkml.org/lkml/2022/11/9/722

> Yours,
> Linus Walleij