* [PATCH] ARM: dts: ti/omap: dra7: fix redundant clock divider definition
@ 2024-10-25 16:46 Andreas Kemnade
2024-10-29 23:44 ` Kevin Hilman
0 siblings, 1 reply; 2+ messages in thread
From: Andreas Kemnade @ 2024-10-25 16:46 UTC (permalink / raw)
To: tony, robh, krzk+dt, conor+dt, linux-omap, devicetree,
linux-kernel, Roger Quadros, Aaro Koskinen, Kevin Hilman
Cc: Andreas Kemnade, kernel test robot
Clock specifies ti,dividers as well as ti,max-div. That is in this case
redundant and not allowed by schema. Both specify possible dividers of
1 or 2. Remove redundant definition.
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202410252030.8B2JoTuq-lkp@intel.com/
Signed-off-by: Andreas Kemnade <andreas@kemnade.info>
---
arch/arm/boot/dts/ti/omap/dra7xx-clocks.dtsi | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm/boot/dts/ti/omap/dra7xx-clocks.dtsi b/arch/arm/boot/dts/ti/omap/dra7xx-clocks.dtsi
index 04f08b8c64d2..0de16ee262cf 100644
--- a/arch/arm/boot/dts/ti/omap/dra7xx-clocks.dtsi
+++ b/arch/arm/boot/dts/ti/omap/dra7xx-clocks.dtsi
@@ -1376,7 +1376,6 @@ optfclk_pciephy_div: clock-optfclk-pciephy-div-8@4a00821c {
clocks = <&apll_pcie_ck>;
#clock-cells = <0>;
reg = <0x021c>;
- ti,dividers = <2>, <1>;
ti,bit-shift = <8>;
ti,max-div = <2>;
};
--
2.39.5
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] ARM: dts: ti/omap: dra7: fix redundant clock divider definition
2024-10-25 16:46 [PATCH] ARM: dts: ti/omap: dra7: fix redundant clock divider definition Andreas Kemnade
@ 2024-10-29 23:44 ` Kevin Hilman
0 siblings, 0 replies; 2+ messages in thread
From: Kevin Hilman @ 2024-10-29 23:44 UTC (permalink / raw)
To: tony, robh, krzk+dt, conor+dt, linux-omap, devicetree,
linux-kernel, Roger Quadros, Aaro Koskinen, Andreas Kemnade
Cc: kernel test robot
On Fri, 25 Oct 2024 18:46:22 +0200, Andreas Kemnade wrote:
> Clock specifies ti,dividers as well as ti,max-div. That is in this case
> redundant and not allowed by schema. Both specify possible dividers of
> 1 or 2. Remove redundant definition.
>
>
Applied, thanks!
[1/1] ARM: dts: ti/omap: dra7: fix redundant clock divider definition
commit: d0bdd8bb7f35a2b4434a3ef665f9cfc3aba886c7
Best regards,
--
Kevin Hilman <khilman@baylibre.com>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2024-10-29 23:44 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-10-25 16:46 [PATCH] ARM: dts: ti/omap: dra7: fix redundant clock divider definition Andreas Kemnade
2024-10-29 23:44 ` Kevin Hilman
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox