diff mbox series

[2/3] spi: dt-bindings: cdns,qspi-nor: Deprecate the Cadence compatible alone

Message ID 20250305200133.2511308-3-miquel.raynal@bootlin.com (mailing list archive)
State New
Headers show
Series spi: dt-bindings: cdns,qspi-nor: Improve description | expand

Commit Message

Miquel Raynal March 5, 2025, 8:01 p.m. UTC
The initial SPI controller IP from Cadence has always been implemented
into controllers from various hardware manufacturers and because of
that, it has always been (rightfully) doubled with a more specific
compatible. There are likely no reasons to keep this compatible
legitimate, alone. Make sure people do not get mislead by officially
deprecating this compatible.

Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
---
 Documentation/devicetree/bindings/spi/cdns,qspi-nor.yaml | 1 +
 1 file changed, 1 insertion(+)

Comments

Krzysztof Kozlowski March 6, 2025, 8:43 a.m. UTC | #1
On Wed, Mar 05, 2025 at 09:01:32PM +0100, Miquel Raynal wrote:
> The initial SPI controller IP from Cadence has always been implemented
> into controllers from various hardware manufacturers and because of
> that, it has always been (rightfully) doubled with a more specific
> compatible. There are likely no reasons to keep this compatible
> legitimate, alone. Make sure people do not get mislead by officially
> deprecating this compatible.
> 
> Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
> ---
>  Documentation/devicetree/bindings/spi/cdns,qspi-nor.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/spi/cdns,qspi-nor.yaml b/Documentation/devicetree/bindings/spi/cdns,qspi-nor.yaml
> index c4315b2e04f2..0104cc5283db 100644
> --- a/Documentation/devicetree/bindings/spi/cdns,qspi-nor.yaml
> +++ b/Documentation/devicetree/bindings/spi/cdns,qspi-nor.yaml
> @@ -81,6 +81,7 @@ properties:
>            # ie. also NAND flashes, not only NOR flashes.
>            - const: cdns,qspi-nor
>        - const: cdns,qspi-nor
> +        deprecated: true
>  

Please update the example, so non-deprecated device will be documented.

Best regards,
Krzysztof
Miquel Raynal March 6, 2025, 8:51 a.m. UTC | #2
Hello,

>> --- a/Documentation/devicetree/bindings/spi/cdns,qspi-nor.yaml
>> +++ b/Documentation/devicetree/bindings/spi/cdns,qspi-nor.yaml
>> @@ -81,6 +81,7 @@ properties:
>>            # ie. also NAND flashes, not only NOR flashes.
>>            - const: cdns,qspi-nor
>>        - const: cdns,qspi-nor
>> +        deprecated: true
>>  
>
> Please update the example, so non-deprecated device will be
> documented.

Good point.

Thanks,
Miquèl
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/spi/cdns,qspi-nor.yaml b/Documentation/devicetree/bindings/spi/cdns,qspi-nor.yaml
index c4315b2e04f2..0104cc5283db 100644
--- a/Documentation/devicetree/bindings/spi/cdns,qspi-nor.yaml
+++ b/Documentation/devicetree/bindings/spi/cdns,qspi-nor.yaml
@@ -81,6 +81,7 @@  properties:
           # ie. also NAND flashes, not only NOR flashes.
           - const: cdns,qspi-nor
       - const: cdns,qspi-nor
+        deprecated: true
 
   reg:
     items: