public inbox for stable@vger.kernel.org
 help / color / mirror / Atom feed
* FAILED: patch "[PATCH] dt-bindings: pwm: adi,axi-pwmgen: Fix clocks" failed to apply to 6.12-stable tree
@ 2025-06-07  9:38 gregkh
  2025-06-08 15:50 ` [PATCH 6.12.y] dt-bindings: pwm: adi,axi-pwmgen: Fix clocks David Lechner
  0 siblings, 1 reply; 4+ messages in thread
From: gregkh @ 2025-06-07  9:38 UTC (permalink / raw)
  To: dlechner, krzysztof.kozlowski, ukleinek; +Cc: stable


The patch below does not apply to the 6.12-stable tree.
If someone wants it applied there, or to any other stable or longterm
tree, then please email the backport, including the original git commit
id to <stable@vger.kernel.org>.

To reproduce the conflict and resubmit, you may use the following commands:

git fetch https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/ linux-6.12.y
git checkout FETCH_HEAD
git cherry-pick -x e683131e64f71e957ca77743cb3d313646157329
# <resolve conflicts, build, test, etc.>
git commit -s
git send-email --to '<stable@vger.kernel.org>' --in-reply-to '2025060734-elated-juvenile-da5c@gregkh' --subject-prefix 'PATCH 6.12.y' HEAD^..

Possible dependencies:



thanks,

greg k-h

------------------ original commit in Linus's tree ------------------

From e683131e64f71e957ca77743cb3d313646157329 Mon Sep 17 00:00:00 2001
From: David Lechner <dlechner@baylibre.com>
Date: Thu, 29 May 2025 11:53:19 -0500
Subject: [PATCH] dt-bindings: pwm: adi,axi-pwmgen: Fix clocks
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Fix a shortcoming in the bindings that doesn't allow for a separate
external clock.

The AXI PWMGEN IP block has a compile option ASYNC_CLK_EN that allows
the use of an external clock for the PWM output separate from the AXI
clock that runs the peripheral.

This was missed in the original bindings and so users were writing dts
files where the one and only clock specified would be the external
clock, if there was one, incorrectly missing the separate AXI clock.

The correct bindings are that the AXI clock is always required and the
external clock is optional (must be given only when HDL compile option
ASYNC_CLK_EN=1).

Fixes: 1edf2c2a2841 ("dt-bindings: pwm: Add AXI PWM generator")
Cc: stable@vger.kernel.org
Signed-off-by: David Lechner <dlechner@baylibre.com>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Link: https://lore.kernel.org/r/20250529-pwm-axi-pwmgen-add-external-clock-v3-2-5d8809a7da91@baylibre.com
Signed-off-by: Uwe Kleine-König <ukleinek@kernel.org>

diff --git a/Documentation/devicetree/bindings/pwm/adi,axi-pwmgen.yaml b/Documentation/devicetree/bindings/pwm/adi,axi-pwmgen.yaml
index 45e112d0efb4..5575c58357d6 100644
--- a/Documentation/devicetree/bindings/pwm/adi,axi-pwmgen.yaml
+++ b/Documentation/devicetree/bindings/pwm/adi,axi-pwmgen.yaml
@@ -30,11 +30,19 @@ properties:
     const: 3
 
   clocks:
-    maxItems: 1
+    minItems: 1
+    maxItems: 2
+
+  clock-names:
+    minItems: 1
+    items:
+      - const: axi
+      - const: ext
 
 required:
   - reg
   - clocks
+  - clock-names
 
 unevaluatedProperties: false
 
@@ -43,6 +51,7 @@ examples:
     pwm@44b00000 {
         compatible = "adi,axi-pwmgen-2.00.a";
         reg = <0x44b00000 0x1000>;
-        clocks = <&spi_clk>;
+        clocks = <&fpga_clk>, <&spi_clk>;
+        clock-names = "axi", "ext";
         #pwm-cells = <3>;
     };


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

* [PATCH 6.12.y] dt-bindings: pwm: adi,axi-pwmgen: Fix clocks
  2025-06-07  9:38 FAILED: patch "[PATCH] dt-bindings: pwm: adi,axi-pwmgen: Fix clocks" failed to apply to 6.12-stable tree gregkh
@ 2025-06-08 15:50 ` David Lechner
  2025-06-09  2:34   ` Sasha Levin
  0 siblings, 1 reply; 4+ messages in thread
From: David Lechner @ 2025-06-08 15:50 UTC (permalink / raw)
  To: stable; +Cc: David Lechner, Krzysztof Kozlowski, Uwe Kleine-König

Fix a shortcoming in the bindings that doesn't allow for a separate
external clock.

The AXI PWMGEN IP block has a compile option ASYNC_CLK_EN that allows
the use of an external clock for the PWM output separate from the AXI
clock that runs the peripheral.

This was missed in the original bindings and so users were writing dts
files where the one and only clock specified would be the external
clock, if there was one, incorrectly missing the separate AXI clock.

The correct bindings are that the AXI clock is always required and the
external clock is optional (must be given only when HDL compile option
ASYNC_CLK_EN=1).

Fixes: 1edf2c2a2841 ("dt-bindings: pwm: Add AXI PWM generator")
Cc: stable@vger.kernel.org
Signed-off-by: David Lechner <dlechner@baylibre.com>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Link: https://lore.kernel.org/r/20250529-pwm-axi-pwmgen-add-external-clock-v3-2-5d8809a7da91@baylibre.com
Signed-off-by: Uwe Kleine-König <ukleinek@kernel.org>
(cherry picked from commit e683131e64f71e957ca77743cb3d313646157329)
Signed-off-by: David Lechner <dlechner@baylibre.com>
---
 .../devicetree/bindings/pwm/adi,axi-pwmgen.yaml     | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/pwm/adi,axi-pwmgen.yaml b/Documentation/devicetree/bindings/pwm/adi,axi-pwmgen.yaml
index ec6115d3796b..f4eb851b2bc8 100644
--- a/Documentation/devicetree/bindings/pwm/adi,axi-pwmgen.yaml
+++ b/Documentation/devicetree/bindings/pwm/adi,axi-pwmgen.yaml
@@ -30,11 +30,19 @@ properties:
     const: 2
 
   clocks:
-    maxItems: 1
+    minItems: 1
+    maxItems: 2
+
+  clock-names:
+    minItems: 1
+    items:
+      - const: axi
+      - const: ext
 
 required:
   - reg
   - clocks
+  - clock-names
 
 unevaluatedProperties: false
 
@@ -43,6 +51,7 @@ examples:
     pwm@44b00000 {
        compatible = "adi,axi-pwmgen-2.00.a";
        reg = <0x44b00000 0x1000>;
-       clocks = <&spi_clk>;
+       clocks = <&fpga_clk>, <&spi_clk>;
+       clock-names = "axi", "ext";
        #pwm-cells = <2>;
     };
-- 
2.43.0


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

* Re: [PATCH 6.12.y] dt-bindings: pwm: adi,axi-pwmgen: Fix clocks
  2025-06-08 15:50 ` [PATCH 6.12.y] dt-bindings: pwm: adi,axi-pwmgen: Fix clocks David Lechner
@ 2025-06-09  2:34   ` Sasha Levin
  2025-06-10 14:15     ` David Lechner
  0 siblings, 1 reply; 4+ messages in thread
From: Sasha Levin @ 2025-06-09  2:34 UTC (permalink / raw)
  To: stable, dlechner; +Cc: Sasha Levin

[ Sasha's backport helper bot ]

Hi,

Summary of potential issues:
⚠️ Found matching upstream commit but patch is missing proper reference to it

Found matching upstream commit: e683131e64f71e957ca77743cb3d313646157329

Status in newer kernel trees:
6.15.y | Not found
6.14.y | Not found

Note: The patch differs from the upstream commit:
---
1:  e683131e64f71 ! 1:  9a2b766b7ef19 dt-bindings: pwm: adi,axi-pwmgen: Fix clocks
    @@ Commit message
         Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
         Link: https://lore.kernel.org/r/20250529-pwm-axi-pwmgen-add-external-clock-v3-2-5d8809a7da91@baylibre.com
         Signed-off-by: Uwe Kleine-König <ukleinek@kernel.org>
    +    (cherry picked from commit e683131e64f71e957ca77743cb3d313646157329)
    +    Signed-off-by: David Lechner <dlechner@baylibre.com>
     
      ## Documentation/devicetree/bindings/pwm/adi,axi-pwmgen.yaml ##
     @@ Documentation/devicetree/bindings/pwm/adi,axi-pwmgen.yaml: properties:
    -     const: 3
    +     const: 2
      
        clocks:
     -    maxItems: 1
    @@ Documentation/devicetree/bindings/pwm/adi,axi-pwmgen.yaml: properties:
      
     @@ Documentation/devicetree/bindings/pwm/adi,axi-pwmgen.yaml: examples:
          pwm@44b00000 {
    -         compatible = "adi,axi-pwmgen-2.00.a";
    -         reg = <0x44b00000 0x1000>;
    --        clocks = <&spi_clk>;
    -+        clocks = <&fpga_clk>, <&spi_clk>;
    -+        clock-names = "axi", "ext";
    -         #pwm-cells = <3>;
    +        compatible = "adi,axi-pwmgen-2.00.a";
    +        reg = <0x44b00000 0x1000>;
    +-       clocks = <&spi_clk>;
    ++       clocks = <&fpga_clk>, <&spi_clk>;
    ++       clock-names = "axi", "ext";
    +        #pwm-cells = <2>;
          };
---

Results of testing on various branches:

| Branch                    | Patch Apply | Build Test |
|---------------------------|-------------|------------|
| stable/linux-6.12.y       |  Success    |  Success   |

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

* Re: [PATCH 6.12.y] dt-bindings: pwm: adi,axi-pwmgen: Fix clocks
  2025-06-09  2:34   ` Sasha Levin
@ 2025-06-10 14:15     ` David Lechner
  0 siblings, 0 replies; 4+ messages in thread
From: David Lechner @ 2025-06-10 14:15 UTC (permalink / raw)
  To: Sasha Levin, stable

On 6/8/25 9:34 PM, Sasha Levin wrote:
> [ Sasha's backport helper bot ]
> 
> Hi,
> 
> Summary of potential issues:
> ⚠️ Found matching upstream commit but patch is missing proper reference to it
> 
> Found matching upstream commit: e683131e64f71e957ca77743cb3d313646157329
> 
> Status in newer kernel trees:
> 6.15.y | Not found
> 6.14.y | Not found
> 
This patch landed in these two trees today, so issue no longer exists.

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

end of thread, other threads:[~2025-06-10 14:15 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-06-07  9:38 FAILED: patch "[PATCH] dt-bindings: pwm: adi,axi-pwmgen: Fix clocks" failed to apply to 6.12-stable tree gregkh
2025-06-08 15:50 ` [PATCH 6.12.y] dt-bindings: pwm: adi,axi-pwmgen: Fix clocks David Lechner
2025-06-09  2:34   ` Sasha Levin
2025-06-10 14:15     ` David Lechner

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox