Re: [PATCHv2 12/23] ARM: socfpga: dts: add a10 clock binding yaml

[Krzysztof Kozlowski <krzk@kernel.org>]

On 25/10/2024 08:59, Lothar Rubusch wrote:
> On Thu, Oct 24, 2024 at 8:24 AM Krzysztof Kozlowski <krzk@kernel.org> wrote:
>>
>> On 24/10/2024 08:10, Lothar Rubusch wrote:
>>> On Mon, Oct 21, 2024 at 9:05 AM Krzysztof Kozlowski <krzk@kernel.org> wrote:
>>>>
> [...]
>>>>> diff --git a/Documentation/devicetree/bindings/clock/altr,socfpga-a10.yaml b/Documentation/devicetree/bindings/clock/altr,socfpga-a10.yaml
>>>>> new file mode 100644
>>>>> index 000000000..795826f53
>>>>> --- /dev/null
>>>>> +++ b/Documentation/devicetree/bindings/clock/altr,socfpga-a10.yaml
>>>>> @@ -0,0 +1,107 @@
>>>>> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>>>>> +%YAML 1.2
> [...]
>> What corresponding txt file? You are adding new binding. Are you saying
>> you duplicated bindings instead of doing conversion?
>>
>> git log -p -- Documentation/devicetree | grep -i convert

This would answer you... You would see hundreds of patches showing what
to do, including patches from me, Rob or Conor.

> 
> Please, try the following:

I know this, what are you asking about?

> $ find ./Documentation/devicetree/bindings -name socfpga-\*.txt
> ./Documentation/devicetree/bindings/net/socfpga-dwmac.txt
> ./Documentation/devicetree/bindings/edac/socfpga-eccmgr.txt
> ./Documentation/devicetree/bindings/arm/altera/socfpga-sdram-edac.txt
> ./Documentation/devicetree/bindings/arm/altera/socfpga-system.txt
> 
> Currently, bindings described in these .txt files are not covered by
> bindings check. Is it supposed to be like that, or is this just
> something "historical"?

These are TXT files, that's nothing to do with dtschema. Everything in
TXT is just unconverted binding.

Best regards,
Krzysztof