Re: [PATCH] dt-bindings: pinctrl: samsung: drop unused header with register constants

From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
To: Conor Dooley <conor@kernel.org>
Cc: Linus Walleij <linus.walleij@linaro.org>,
	Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Conor Dooley <conor+dt@kernel.org>,
	linux-kernel@vger.kernel.org, linux-gpio@vger.kernel.org,
	devicetree@vger.kernel.org
Subject: Re: [PATCH] dt-bindings: pinctrl: samsung: drop unused header with register constants
Date: Tue, 12 Mar 2024 20:08:31 +0100	[thread overview]
Message-ID: <05a3763c-1855-4028-9f2a-b125c7ed58cc@linaro.org> (raw)
In-Reply-To: <20240312-disobey-playset-f3d451adf41c@spud>

On 12/03/2024 19:46, Conor Dooley wrote:
> On Tue, Mar 12, 2024 at 06:44:17PM +0000, Conor Dooley wrote:
>> On Tue, Mar 12, 2024 at 05:44:28PM +0100, Krzysztof Kozlowski wrote:
>>> The bindings header for Samsung pin controller DTS pin values (holding
>>> register values in fact) was deprecated in v6.1 kernel in
>>> commit 9d9292576810 ("dt-bindings: pinctrl: samsung: deprecate header
>>> with register constants").  This was enough of time for users to switch
>>> to in-DTS headers, so drop the bindings header.
>>>
>>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>>
>> Have you checked whether U-Boot has also dropped use (or never did use)
>> of this header?
> 
> nvm, I checked it myself and Caleb's series that moves things to use
> upstream headers does't seem to use this either.
> 

U-Boot did not use it, but even if they did, they had more than one year
to change. The header had a big-fat warning.

Best regards,
Krzysztof