[PATCH] dt-bindings: phy: make phy-cells description a text
Rob Herring
robh at kernel.org
Mon Jun 27 16:47:29 PDT 2022
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
More information about the Linux-mediatek
mailing list