mbox series

[v6,0/2] dt-bindings: mtd: marvell-nand: Add YAML scheme

Message ID 20230530005337.3687938-1-chris.packham@alliedtelesis.co.nz (mailing list archive)
Headers show
Series dt-bindings: mtd: marvell-nand: Add YAML scheme | expand

Message

Chris Packham May 30, 2023, 12:53 a.m. UTC
Add YAML scheme for the Marvell's NAND controller
to validate it's DT bindings. Old txt file is deleted,
not included the compatibles and properties which were marked as
deprecated.

Also fix node name in cp11x DTSI acording to nand-controller.yaml

I've picked up this series to hopefully get it over the line. I think I've
addressed all the feedback from the last round of review.

Vadym Kochan (2):
  dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
  arm64: dts: marvell: cp11x: Fix nand_controller node name according to
    YAML

 .../bindings/mtd/marvell,nand-controller.yaml | 190 ++++++++++++++++++
 .../devicetree/bindings/mtd/marvell-nand.txt  | 126 ------------
 MAINTAINERS                                   |   1 -
 arch/arm64/boot/dts/marvell/armada-cp11x.dtsi |   2 +-
 4 files changed, 191 insertions(+), 128 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/mtd/marvell,nand-controller.yaml
 delete mode 100644 Documentation/devicetree/bindings/mtd/marvell-nand.txt

Comments

Gregory CLEMENT June 16, 2023, 11:45 a.m. UTC | #1
Chris Packham <chris.packham@alliedtelesis.co.nz> writes:

> Add YAML scheme for the Marvell's NAND controller
> to validate it's DT bindings. Old txt file is deleted,
> not included the compatibles and properties which were marked as
> deprecated.
>
> Also fix node name in cp11x DTSI acording to nand-controller.yaml
>
> I've picked up this series to hopefully get it over the line. I think I've
> addressed all the feedback from the last round of review.
>
> Vadym Kochan (2):
>   dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
>   arm64: dts: marvell: cp11x: Fix nand_controller node name according to
>     YAML


Both patch applied on mvebu/dt64

However if the first patch should be merged through another tree, let me
know.

Thanks,

Gregory


>
>  .../bindings/mtd/marvell,nand-controller.yaml | 190 ++++++++++++++++++
>  .../devicetree/bindings/mtd/marvell-nand.txt  | 126 ------------
>  MAINTAINERS                                   |   1 -
>  arch/arm64/boot/dts/marvell/armada-cp11x.dtsi |   2 +-
>  4 files changed, 191 insertions(+), 128 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/mtd/marvell,nand-controller.yaml
>  delete mode 100644 Documentation/devicetree/bindings/mtd/marvell-nand.txt
>
> -- 
> 2.40.1
>
Gregory CLEMENT June 16, 2023, 11:58 a.m. UTC | #2
Gregory CLEMENT <gregory.clement@bootlin.com> writes:

> Chris Packham <chris.packham@alliedtelesis.co.nz> writes:
>
>> Add YAML scheme for the Marvell's NAND controller
>> to validate it's DT bindings. Old txt file is deleted,
>> not included the compatibles and properties which were marked as
>> deprecated.
>>
>> Also fix node name in cp11x DTSI acording to nand-controller.yaml
>>
>> I've picked up this series to hopefully get it over the line. I think I've
>> addressed all the feedback from the last round of review.
>>
>> Vadym Kochan (2):
>>   dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
>>   arm64: dts: marvell: cp11x: Fix nand_controller node name according to
>>     YAML
>
>
> Both patch applied on mvebu/dt64
>
> However if the first patch should be merged through another tree, let me
> know.

Finally I removed these both patches, I missed the version 9!

I applied the 2 patches for the dts files from v9.

Thanks,

Gregory
>
>
>>
>>  .../bindings/mtd/marvell,nand-controller.yaml | 190 ++++++++++++++++++
>>  .../devicetree/bindings/mtd/marvell-nand.txt  | 126 ------------
>>  MAINTAINERS                                   |   1 -
>>  arch/arm64/boot/dts/marvell/armada-cp11x.dtsi |   2 +-
>>  4 files changed, 191 insertions(+), 128 deletions(-)
>>  create mode 100644 Documentation/devicetree/bindings/mtd/marvell,nand-controller.yaml
>>  delete mode 100644 Documentation/devicetree/bindings/mtd/marvell-nand.txt
>>
>> -- 
>> 2.40.1
>>
>
> -- 
> Gregory Clement, Bootlin
> Embedded Linux and Kernel engineering
> http://bootlin.com
Chris Packham June 18, 2023, 9:12 p.m. UTC | #3
On 16/06/23 23:58, Gregory CLEMENT wrote:
> Gregory CLEMENT <gregory.clement@bootlin.com> writes:
>
>> Chris Packham <chris.packham@alliedtelesis.co.nz> writes:
>>
>>> Add YAML scheme for the Marvell's NAND controller
>>> to validate it's DT bindings. Old txt file is deleted,
>>> not included the compatibles and properties which were marked as
>>> deprecated.
>>>
>>> Also fix node name in cp11x DTSI acording to nand-controller.yaml
>>>
>>> I've picked up this series to hopefully get it over the line. I think I've
>>> addressed all the feedback from the last round of review.
>>>
>>> Vadym Kochan (2):
>>>    dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme
>>>    arm64: dts: marvell: cp11x: Fix nand_controller node name according to
>>>      YAML
>>
>> Both patch applied on mvebu/dt64
>>
>> However if the first patch should be merged through another tree, let me
>> know.
> Finally I removed these both patches, I missed the version 9!
>
> I applied the 2 patches for the dts files from v9.
>
> Thanks,
>
> Gregory
Thanks and sorry for the confusion. There's a v10 on the way but I can 
drop out the dts changes from that now.
>>
>>>   .../bindings/mtd/marvell,nand-controller.yaml | 190 ++++++++++++++++++
>>>   .../devicetree/bindings/mtd/marvell-nand.txt  | 126 ------------
>>>   MAINTAINERS                                   |   1 -
>>>   arch/arm64/boot/dts/marvell/armada-cp11x.dtsi |   2 +-
>>>   4 files changed, 191 insertions(+), 128 deletions(-)
>>>   create mode 100644 Documentation/devicetree/bindings/mtd/marvell,nand-controller.yaml
>>>   delete mode 100644 Documentation/devicetree/bindings/mtd/marvell-nand.txt
>>>
>>> -- 
>>> 2.40.1
>>>
>> -- 
>> Gregory Clement, Bootlin
>> Embedded Linux and Kernel engineering
>> http://scanmail.trustwave.com/?c=20988&d=0c6M5H8PBagWXDuSKsjbXpd1zLX-HHHRk3xSZOdFDw&u=http%3a%2f%2fbootlin%2ecom