Re: [PATCH v6 03/17] dt-bindings: arm: hisilicon: split the dt-bindings of each controller into a separate file

From: Krzysztof Kozlowski
Date: Thu Oct 01 2020 - 02:32:46 EST


On Wed, Sep 30, 2020 at 11:16:58AM +0800, Zhen Lei wrote:
> Split the devicetree bindings of each Hisilicon controller from
> hisilicon.txt into a separate file, the file name is the compatible name
> attach the .txt file name extension.
>
> Signed-off-by: Zhen Lei <thunder.leizhen@xxxxxxxxxx>
> ---
> .../arm/hisilicon/controller/hisilicon,cpuctrl.txt | 8 +
> .../controller/hisilicon,hi3798cv200-perictrl.txt | 21 +++
> .../controller/hisilicon,hi6220-aoctrl.txt | 18 ++
> .../controller/hisilicon,hi6220-mediactrl.txt | 18 ++
> .../controller/hisilicon,hi6220-pmctrl.txt | 18 ++
> .../controller/hisilicon,hi6220-sysctrl.txt | 19 ++
> .../controller/hisilicon,hip01-sysctrl.txt | 19 ++
> .../controller/hisilicon,hip04-bootwrapper.txt | 9 +
> .../controller/hisilicon,hip04-fabric.txt | 5 +
> .../arm/hisilicon/controller/hisilicon,pctrl.txt | 13 ++
> .../arm/hisilicon/controller/hisilicon,sysctrl.txt | 25 +++
> .../bindings/arm/hisilicon/hisilicon.txt | 194 ---------------------
> 12 files changed, 173 insertions(+), 194 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/arm/hisilicon/controller/hisilicon,cpuctrl.txt
> create mode 100644 Documentation/devicetree/bindings/arm/hisilicon/controller/hisilicon,hi3798cv200-perictrl.txt
> create mode 100644 Documentation/devicetree/bindings/arm/hisilicon/controller/hisilicon,hi6220-aoctrl.txt
> create mode 100644 Documentation/devicetree/bindings/arm/hisilicon/controller/hisilicon,hi6220-mediactrl.txt
> create mode 100644 Documentation/devicetree/bindings/arm/hisilicon/controller/hisilicon,hi6220-pmctrl.txt
> create mode 100644 Documentation/devicetree/bindings/arm/hisilicon/controller/hisilicon,hi6220-sysctrl.txt
> create mode 100644 Documentation/devicetree/bindings/arm/hisilicon/controller/hisilicon,hip01-sysctrl.txt
> create mode 100644 Documentation/devicetree/bindings/arm/hisilicon/controller/hisilicon,hip04-bootwrapper.txt
> create mode 100644 Documentation/devicetree/bindings/arm/hisilicon/controller/hisilicon,hip04-fabric.txt
> create mode 100644 Documentation/devicetree/bindings/arm/hisilicon/controller/hisilicon,pctrl.txt
> create mode 100644 Documentation/devicetree/bindings/arm/hisilicon/controller/hisilicon,sysctrl.txt

If I understand correctly, first you create multiple TXT bindings files
in this patch and then convert them one by one to JSON.

No, it does not make sense creating a temporary file.

Split out the bindings one by one from hisilicon.txt into JSON directly.

Best regards,
Krzysztof