On Sun, Jun 19, 2022 at 01:46:17PM +0200, Krzysztof Kozlowski wrote: > On 19/06/2022 13:40, 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 ? 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). > > No, it does not hurt. It is however confusing some folks and they think > schema goes into description. The description should be > readable/descriptive for humans, so if you think your approach is > better, I am perfectly fine with it. I don't mind much. If you think it would be a good idea to eventually describe the #phy-cells elements in YAML schema then I'd rather keep the current description, if there's very little chance it will happen, I don't mind changing it. -- Regards, Laurent Pinchart
