devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ASoC: dt-bindings: rename wm8750.yaml to wlf,wm8750.yaml
@ 2024-06-19  7:03 Animesh Agarwal
  2024-06-19  7:26 ` Krzysztof Kozlowski
  0 siblings, 1 reply; 5+ messages in thread
From: Animesh Agarwal @ 2024-06-19  7:03 UTC (permalink / raw)
  Cc: animeshagarwal28, Daniel Baluta, Liam Girdwood, Mark Brown,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley, patches,
	linux-sound, devicetree, linux-kernel

Add vendor prefix to wlf,wm8750 dt-schema.

Signed-off-by: Animesh Agarwal <animeshagarwal28@gmail.com>
Cc: Daniel Baluta <daniel.baluta@nxp.com>
---
 .../devicetree/bindings/sound/{wm8750.yaml => wlf,wm8750.yaml}  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 rename Documentation/devicetree/bindings/sound/{wm8750.yaml => wlf,wm8750.yaml} (92%)

diff --git a/Documentation/devicetree/bindings/sound/wm8750.yaml b/Documentation/devicetree/bindings/sound/wlf,wm8750.yaml
similarity index 92%
rename from Documentation/devicetree/bindings/sound/wm8750.yaml
rename to Documentation/devicetree/bindings/sound/wlf,wm8750.yaml
index 24246ac7bbdf..96859e38315b 100644
--- a/Documentation/devicetree/bindings/sound/wm8750.yaml
+++ b/Documentation/devicetree/bindings/sound/wlf,wm8750.yaml
@@ -1,7 +1,7 @@
 # SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
 %YAML 1.2
 ---
-$id: http://devicetree.org/schemas/sound/wm8750.yaml#
+$id: http://devicetree.org/schemas/sound/wlf,wm8750.yaml#
 $schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: WM8750 and WM8987 audio CODECs
-- 
2.45.2


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] ASoC: dt-bindings: rename wm8750.yaml to wlf,wm8750.yaml
  2024-06-19  7:03 [PATCH] ASoC: dt-bindings: rename wm8750.yaml to wlf,wm8750.yaml Animesh Agarwal
@ 2024-06-19  7:26 ` Krzysztof Kozlowski
  2024-06-20  6:08   ` Daniel Baluta
  0 siblings, 1 reply; 5+ messages in thread
From: Krzysztof Kozlowski @ 2024-06-19  7:26 UTC (permalink / raw)
  To: Animesh Agarwal
  Cc: Daniel Baluta, Liam Girdwood, Mark Brown, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, patches, linux-sound,
	devicetree, linux-kernel

On 19/06/2024 09:03, Animesh Agarwal wrote:
> Add vendor prefix to wlf,wm8750 dt-schema.
> 
> Signed-off-by: Animesh Agarwal <animeshagarwal28@gmail.com>
> Cc: Daniel Baluta <daniel.baluta@nxp.com>
> ---
>  .../devicetree/bindings/sound/{wm8750.yaml => wlf,wm8750.yaml}  | 2 +-

Same as in other cases (why they started now?): why only one file not
all of them? Anyway, that's not necessary.

Best regards,
Krzysztof


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] ASoC: dt-bindings: rename wm8750.yaml to wlf,wm8750.yaml
  2024-06-19  7:26 ` Krzysztof Kozlowski
@ 2024-06-20  6:08   ` Daniel Baluta
  2024-06-20  6:37     ` Krzysztof Kozlowski
  0 siblings, 1 reply; 5+ messages in thread
From: Daniel Baluta @ 2024-06-20  6:08 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Animesh Agarwal, Daniel Baluta, Liam Girdwood, Mark Brown,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley, patches,
	linux-sound, devicetree, linux-kernel

On Wed, Jun 19, 2024 at 10:29 AM Krzysztof Kozlowski <krzk@kernel.org> wrote:
>
> On 19/06/2024 09:03, Animesh Agarwal wrote:
> > Add vendor prefix to wlf,wm8750 dt-schema.
> >
> > Signed-off-by: Animesh Agarwal <animeshagarwal28@gmail.com>
> > Cc: Daniel Baluta <daniel.baluta@nxp.com>
> > ---
> >  .../devicetree/bindings/sound/{wm8750.yaml => wlf,wm8750.yaml}  | 2 +-
>
> Same as in other cases (why they started now?): why only one file not
> all of them? Anyway, that's not necessary.

There is only one yaml file from wolfson which doesnt have the wlf, prefix.
For consistency we think that we should rename the file :).

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] ASoC: dt-bindings: rename wm8750.yaml to wlf,wm8750.yaml
  2024-06-20  6:08   ` Daniel Baluta
@ 2024-06-20  6:37     ` Krzysztof Kozlowski
  2024-06-20  6:59       ` Krzysztof Kozlowski
  0 siblings, 1 reply; 5+ messages in thread
From: Krzysztof Kozlowski @ 2024-06-20  6:37 UTC (permalink / raw)
  To: Daniel Baluta
  Cc: Animesh Agarwal, Daniel Baluta, Liam Girdwood, Mark Brown,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley, patches,
	linux-sound, devicetree, linux-kernel

On 20/06/2024 08:08, Daniel Baluta wrote:
> On Wed, Jun 19, 2024 at 10:29 AM Krzysztof Kozlowski <krzk@kernel.org> wrote:
>>
>> On 19/06/2024 09:03, Animesh Agarwal wrote:
>>> Add vendor prefix to wlf,wm8750 dt-schema.
>>>
>>> Signed-off-by: Animesh Agarwal <animeshagarwal28@gmail.com>
>>> Cc: Daniel Baluta <daniel.baluta@nxp.com>
>>> ---
>>>  .../devicetree/bindings/sound/{wm8750.yaml => wlf,wm8750.yaml}  | 2 +-
>>
>> Same as in other cases (why they started now?): why only one file not
>> all of them? Anyway, that's not necessary.
> 
> There is only one yaml file from wolfson which doesnt have the wlf, prefix.
> For consistency we think that we should rename the file :).

And several other files have the same issue. That's just churn. No, let
me be clear. Rule is for all files, not for wlf or asahi only. If you
want that consistency, then fix *all* files, not selected one schema.

Best regards,
Krzysztof


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] ASoC: dt-bindings: rename wm8750.yaml to wlf,wm8750.yaml
  2024-06-20  6:37     ` Krzysztof Kozlowski
@ 2024-06-20  6:59       ` Krzysztof Kozlowski
  0 siblings, 0 replies; 5+ messages in thread
From: Krzysztof Kozlowski @ 2024-06-20  6:59 UTC (permalink / raw)
  To: Daniel Baluta
  Cc: Animesh Agarwal, Daniel Baluta, Liam Girdwood, Mark Brown,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley, patches,
	linux-sound, devicetree, linux-kernel

On 20/06/2024 08:37, Krzysztof Kozlowski wrote:
> On 20/06/2024 08:08, Daniel Baluta wrote:
>> On Wed, Jun 19, 2024 at 10:29 AM Krzysztof Kozlowski <krzk@kernel.org> wrote:
>>>
>>> On 19/06/2024 09:03, Animesh Agarwal wrote:
>>>> Add vendor prefix to wlf,wm8750 dt-schema.
>>>>
>>>> Signed-off-by: Animesh Agarwal <animeshagarwal28@gmail.com>
>>>> Cc: Daniel Baluta <daniel.baluta@nxp.com>
>>>> ---
>>>>  .../devicetree/bindings/sound/{wm8750.yaml => wlf,wm8750.yaml}  | 2 +-
>>>
>>> Same as in other cases (why they started now?): why only one file not
>>> all of them? Anyway, that's not necessary.
>>
>> There is only one yaml file from wolfson which doesnt have the wlf, prefix.
>> For consistency we think that we should rename the file :).
> 
> And several other files have the same issue. That's just churn. No, let
> me be clear. Rule is for all files, not for wlf or asahi only. If you
> want that consistency, then fix *all* files, not selected one schema.

I see now Kuninori went by my previous suggestion, so this patch is not
needed/obsolete.

Best regards,
Krzysztof


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2024-06-20  7:00 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-06-19  7:03 [PATCH] ASoC: dt-bindings: rename wm8750.yaml to wlf,wm8750.yaml Animesh Agarwal
2024-06-19  7:26 ` Krzysztof Kozlowski
2024-06-20  6:08   ` Daniel Baluta
2024-06-20  6:37     ` Krzysztof Kozlowski
2024-06-20  6:59       ` Krzysztof Kozlowski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).