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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAFXKEHZhGEJhOxqX04fAR6qs-vee4+-DWC4_pNGaDCzEumMsiw@mail.gmail.com>
Date: Fri, 25 Oct 2024 08:59:49 +0200
From: Lothar Rubusch <l.rubusch@...il.com>
To: Krzysztof Kozlowski <krzk@...nel.org>
Cc: robh@...nel.org, krzk+dt@...nel.org, conor+dt@...nel.org, 
	dinguyen@...nel.org, marex@...x.de, s.trumtrar@...gutronix.de, 
	devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCHv2 12/23] ARM: socfpga: dts: add a10 clock binding yaml

On Thu, Oct 24, 2024 at 8:24 AM Krzysztof Kozlowski <krzk@...nel.org> wrote:
>
> On 24/10/2024 08:10, Lothar Rubusch wrote:
> > On Mon, Oct 21, 2024 at 9:05 AM Krzysztof Kozlowski <krzk@...nel.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

Please, try the following:
$ 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"?

I would appreciate to take the opportunity to learn more about
bindings and bindings check, and may try to convert the files to
.yaml. Is this ok or do you have plans with those files?

If noone speaks up, I'll put my name under the maintainers in the
.yaml. Let's see what happens (I hope it's ok, due to the high demand
in cyclone5 these days...).

Best regards,
Lothar