[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20220627234729.GA3169775-robh@kernel.org>
Date: Mon, 27 Jun 2022 17:47:29 -0600
From: Rob Herring <robh@...nel.org>
To: Laurent Pinchart <laurent.pinchart@...asonboard.com>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
Chunfeng Yun <chunfeng.yun@...iatek.com>,
Kishon Vijay Abraham I <kishon@...com>,
Vinod Koul <vkoul@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Anurag Kumar Vulisha <anurag.kumar.vulisha@...inx.com>,
Michal Simek <michal.simek@...inx.com>,
Matthias Brugger <matthias.bgg@...il.com>,
linux-arm-kernel@...ts.infradead.org,
linux-mediatek@...ts.infradead.org, linux-phy@...ts.infradead.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
Sean Anderson <sean.anderson@...o.com>
Subject: Re: [PATCH] dt-bindings: phy: make phy-cells description a text
On Sun, Jun 19, 2022 at 02:40:12PM +0300, Laurent Pinchart wrote:
> Hi Krzysztof,
>
> Thank you for the patch.
>
> On Sun, Jun 19, 2022 at 01:33:25PM +0200, Krzysztof Kozlowski wrote:
> > The description field is a string, so using YAML inside phy-cells
> > description is not actually helpful.
>
> Does it hurt though ?
Unfortunately, I see this a bit. It's convenient because the schema
passes all the checks. Doh! And I usually stare at it wondering how it
passed.
Though I probably did review this, so IDK...
> For xlnx,zynqmp-psgtr.yaml I wrote it that way to
> prepare for a future where it could be described using a YAML schema
> (but such future may never come).
There's 2 parts. There's the resolving the defines and then applying the
schema to the cells. I actually think the latter would be easier. At
least from a documenting standpoint, we just need to define our own
keyword to stick the schema under. With the tools doing dtb based
validation now, the tools already get the phandle node and get the cell
size from the DT. It's just another step to extract the
node's compatible, find it's schema, and get its cell format schema. Any
volunteers?
Rob
Powered by blists - more mailing lists