Re: [PATCH v2 1/2] arm64: dts: imx8mm: Add pinctrl config definitions

[Krzysztof Kozlowski <krzk@kernel.org>]

On 24/02/2025 08:39, Maud Spierings | GOcontroll wrote:
> From: Krzysztof Kozlowski <krzk@kernel.org>
> Sent: Sunday, February 23, 2025 11:27 AM
>  
>> On Tue, Feb 18, 2025 at 09:39:16AM +0100, Maud Spierings wrote:
>>> Currently to configure each IOMUXC_SW_PAD_CTL_PAD the raw value of this
>>> register is written in the dts, these values are not obvious. Add defines
>>> which describe the fields of this register which can be or-ed together to
>>> produce readable settings.
>>>
>>> Signed-off-by: Maud Spierings <maudspierings@gocontroll.com>
>>> ---
>>>   arch/arm64/boot/dts/freescale/imx8mm-pinfunc.h | 27 ++++++++++++++++++++++++++
>>>   1 file changed, 27 insertions(+)
>>>
> 
>> Why are you adding defines which have no users? This is just dead code.
> 
> I will be sending more patches in the near future that will use these bindings, the total stack of patches is starting to grow though so I'm trying to spread it out in multiple easy to manage bits.
> It is also a bit of a test if I can continue using these defines as I plan, or if that will not be accepted and I should instead revert to using raw hex values.

Please wrap your replies.

> 
> If this is a problem then I will keep it in the larger patchset that is to come.

Defines without users is dead code. It's not even easy to validate if
they make any sense without seeing how they are used.

Best regards,
Krzysztof