All of lore.kernel.org
 help / color / mirror / Atom feed
From: Krzysztof Kozlowski <krzk@kernel.org>
To: Zhen Lei <thunder.leizhen@huawei.com>
Cc: devicetree <devicetree@vger.kernel.org>,
	Kefeng Wang <wangkefeng.wang@huawei.com>,
	linux-kernel <linux-kernel@vger.kernel.org>,
	Wei Xu <xuwei5@hisilicon.com>, Rob Herring <robh+dt@kernel.org>,
	Libin <huawei.libin@huawei.com>,
	Jonathan Cameron <Jonathan.Cameron@Huawei.com>,
	linux-arm-kernel <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH v6 03/17] dt-bindings: arm: hisilicon: split the dt-bindings of each controller into a separate file
Date: Thu, 1 Oct 2020 08:32:36 +0200	[thread overview]
Message-ID: <20201001063236.GB3018@kozik-lap> (raw)
In-Reply-To: <20200930031712.2365-4-thunder.leizhen@huawei.com>

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@huawei.com>
> ---
>  .../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

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

WARNING: multiple messages have this Message-ID (diff)
From: Krzysztof Kozlowski <krzk@kernel.org>
To: Zhen Lei <thunder.leizhen@huawei.com>
Cc: Wei Xu <xuwei5@hisilicon.com>, Rob Herring <robh+dt@kernel.org>,
	Jonathan Cameron <Jonathan.Cameron@Huawei.com>,
	devicetree <devicetree@vger.kernel.org>,
	linux-arm-kernel <linux-arm-kernel@lists.infradead.org>,
	linux-kernel <linux-kernel@vger.kernel.org>,
	Libin <huawei.libin@huawei.com>,
	Kefeng Wang <wangkefeng.wang@huawei.com>
Subject: Re: [PATCH v6 03/17] dt-bindings: arm: hisilicon: split the dt-bindings of each controller into a separate file
Date: Thu, 1 Oct 2020 08:32:36 +0200	[thread overview]
Message-ID: <20201001063236.GB3018@kozik-lap> (raw)
In-Reply-To: <20200930031712.2365-4-thunder.leizhen@huawei.com>

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@huawei.com>
> ---
>  .../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

  reply	other threads:[~2020-10-01  6:34 UTC|newest]

Thread overview: 66+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-09-30  3:16 [PATCH v6 00/17] add support for Hisilicon SD5203 SoC Zhen Lei
2020-09-30  3:16 ` Zhen Lei
2020-09-30  3:16 ` [PATCH v6 01/17] dt-bindings: mfd: syscon: add some compatible strings for Hisilicon Zhen Lei
2020-09-30  3:16   ` Zhen Lei
2020-09-30  7:11   ` Lee Jones
2020-09-30  7:11     ` Lee Jones
2020-09-30  7:21     ` Leizhen (ThunderTown)
2020-09-30  7:21       ` Leizhen (ThunderTown)
2020-10-01  6:59       ` Lee Jones
2020-10-01  6:59         ` Lee Jones
2020-10-10 10:01         ` Leizhen (ThunderTown)
2020-10-10 10:01           ` Leizhen (ThunderTown)
2020-10-01  6:27   ` Krzysztof Kozlowski
2020-10-01  6:27     ` Krzysztof Kozlowski
2020-09-30  3:16 ` [PATCH v6 02/17] dt-bindings: arm: hisilicon: delete the descriptions of HiP05/HiP06 controllers Zhen Lei
2020-09-30  3:16   ` Zhen Lei
2020-10-01  6:28   ` Krzysztof Kozlowski
2020-10-01  6:28     ` Krzysztof Kozlowski
2020-09-30  3:16 ` [PATCH v6 03/17] dt-bindings: arm: hisilicon: split the dt-bindings of each controller into a separate file Zhen Lei
2020-09-30  3:16   ` Zhen Lei
2020-10-01  6:32   ` Krzysztof Kozlowski [this message]
2020-10-01  6:32     ` Krzysztof Kozlowski
2020-09-30  3:16 ` [PATCH v6 04/17] dt-bindings: arm: hisilicon: convert Hisilicon board/soc bindings to json-schema Zhen Lei
2020-09-30  3:16   ` Zhen Lei
2020-09-30  3:17 ` [PATCH v6 05/17] dt-bindings: arm: hisilicon: add binding for SD5203 SoC Zhen Lei
2020-09-30  3:17   ` Zhen Lei
2020-09-30  3:17 ` [PATCH v6 06/17] ARM: hisi: add support " Zhen Lei
2020-09-30  3:17   ` Zhen Lei
2020-09-30  3:17 ` [PATCH v6 07/17] ARM: debug: add UART early console support for SD5203 Zhen Lei
2020-09-30  3:17   ` Zhen Lei
2020-09-30  3:17 ` [PATCH v6 08/17] ARM: dts: add SD5203 dts Zhen Lei
2020-09-30  3:17   ` Zhen Lei
2020-10-01  6:36   ` Krzysztof Kozlowski
2020-10-01  6:36     ` Krzysztof Kozlowski
2020-09-30  3:17 ` [PATCH v6 09/17] ARM: dts: hisilicon: fix ststem controller compatible node Zhen Lei
2020-09-30  3:17   ` Zhen Lei
2020-09-30  3:17 ` [PATCH v6 10/17] dt-bindings: arm: hisilicon: convert system controller bindings to json-schema Zhen Lei
2020-09-30  3:17   ` Zhen Lei
2020-09-30  3:17 ` [PATCH v6 11/17] dt-bindings: arm: hisilicon: convert hisilicon, cpuctrl " Zhen Lei
2020-09-30  3:17   ` [PATCH v6 11/17] dt-bindings: arm: hisilicon: convert hisilicon,cpuctrl " Zhen Lei
2020-10-01  6:40   ` Krzysztof Kozlowski
2020-10-01  6:40     ` Krzysztof Kozlowski
2020-10-10 10:54     ` Leizhen (ThunderTown)
2020-10-10 10:54       ` Leizhen (ThunderTown)
2020-09-30  3:17 ` [PATCH v6 12/17] dt-bindings: arm: hisilicon: convert hisilicon, pctrl " Zhen Lei
2020-09-30  3:17   ` [PATCH v6 12/17] dt-bindings: arm: hisilicon: convert hisilicon,pctrl " Zhen Lei
2020-09-30  3:17 ` [PATCH v6 13/17] dt-bindings: arm: hisilicon: convert hisilicon, hip04-fabric " Zhen Lei
2020-09-30  3:17   ` [PATCH v6 13/17] dt-bindings: arm: hisilicon: convert hisilicon,hip04-fabric " Zhen Lei
2020-09-30  3:17 ` [PATCH v6 14/17] dt-bindings: arm: hisilicon: convert hisilicon, hip04-bootwrapper " Zhen Lei
2020-09-30  3:17   ` [PATCH v6 14/17] dt-bindings: arm: hisilicon: convert hisilicon,hip04-bootwrapper " Zhen Lei
2020-10-01  6:41   ` Krzysztof Kozlowski
2020-10-01  6:41     ` Krzysztof Kozlowski
2020-10-10 10:07     ` Leizhen (ThunderTown)
2020-10-10 10:07       ` Leizhen (ThunderTown)
2020-09-30  3:17 ` [PATCH v6 15/17] dt-bindings: arm: hisilicon: convert Hi6220 domain controller " Zhen Lei
2020-09-30  3:17   ` Zhen Lei
2020-10-06 19:04   ` Rob Herring
2020-10-06 19:04     ` Rob Herring
2020-09-30  3:17 ` [PATCH v6 16/17] dt-bindings: arm: hisilicon: convert hisilicon, hi3798cv200-perictrl " Zhen Lei
2020-09-30  3:17   ` [PATCH v6 16/17] dt-bindings: arm: hisilicon: convert hisilicon,hi3798cv200-perictrl " Zhen Lei
2020-10-01  6:35   ` Krzysztof Kozlowski
2020-10-01  6:35     ` Krzysztof Kozlowski
2020-10-10 10:56     ` Leizhen (ThunderTown)
2020-10-10 10:56       ` Leizhen (ThunderTown)
2020-09-30  3:17 ` [PATCH v6 17/17] dt-bindings: arm: hisilicon: convert LPC controller " Zhen Lei
2020-09-30  3:17   ` Zhen Lei

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20201001063236.GB3018@kozik-lap \
    --to=krzk@kernel.org \
    --cc=Jonathan.Cameron@Huawei.com \
    --cc=devicetree@vger.kernel.org \
    --cc=huawei.libin@huawei.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=robh+dt@kernel.org \
    --cc=thunder.leizhen@huawei.com \
    --cc=wangkefeng.wang@huawei.com \
    --cc=xuwei5@hisilicon.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.