Re: [PATCH v6 2/2] dt-bindings: add xilinx clocking wizard bindings

[Michal Simek <michal.simek@xilinx.com>]

Hi,

On 6/16/21 5:26 PM, Zhengxun Li wrote:
> Hi Sean,
> 
> Thank you for your reply.
> 
> Sean Anderson <sean.anderson@seco.com> 於 2021年6月15日 週二 下午2:52寫道：
>>
>>
>>
>> On 6/15/21 1:21 PM, Zhengxun Li wrote:
>>  > Hi Michal,
>>  >
>>  > Sean Anderson <sean.anderson@seco.com> 於 2021年6月11日 週五 下午3:07寫道：
>>  >>
>>  >>
>>  >>
>>  >> On 6/11/21 11:10 AM, Zhengxun wrote:
>>  >>   > Add the devicetree binding for the xilinx clocking wizard.
>>  >>   >
>>  >>   > Signed-off-by: Zhengxun <zhengxunli.mxic@gmail.com>
>>  >>   > ---
>>  >>   >   .../clock/xlnx,clocking-wizard.txt            | 43 +++++++++++++++++++
>>  >>   >   1 file changed, 43 insertions(+)
>>  >>   >   create mode 100644 doc/device-tree-bindings/clock/xlnx,clocking-wizard.txt
>>  >>
>>  >> Should this be .yml? In general, I believe we only need to document
>>  >> bindings in U-Boot if they differ from Linux. In Linux, I see that
>>  >> Documentation/devicetree/bindings/clock/xlnx,clocking-wizard.txt exists.
>>
>> Because the binding is documented in Linux, we do not need to document
>> it in U-Boot.
>>
>>  >> Will you be upstreaming this conversion to yaml?
>>
>> This patch documents the binding in yaml. However, the version in Linux
>> is in old-style txt. Will you be sending a patch to Linux to convert
>> this binding to yaml?
> 
> In linux, I saw Shubhrajyoti Datta do this, although it has not been
> accepted yet. The link is below.
> 
> https://patchwork.kernel.org/project/linux-clk/patch/1617886723-27117-2-git-send-email-shubhrajyoti.datta@xilinx.com/
> 
> Should I do this again?

I prefer not to do it. Linux binding will be reviewed by Rob and that
should be enough.

Thanks,
Michal