public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 1/2] dt-bindings: soc: ti: am654-serdes-ctrl: Add simple-mfd to compatible items
@ 2024-08-21 16:23 Andrew Davis
  2024-08-21 16:23 ` [PATCH v2 2/2] arm64: dts: ti: k3-am65: Add simple-mfd compatible to SerDes control nodes Andrew Davis
  2024-08-24 19:53 ` [PATCH v2 1/2] dt-bindings: soc: ti: am654-serdes-ctrl: Add simple-mfd to compatible items Nishanth Menon
  0 siblings, 2 replies; 3+ messages in thread
From: Andrew Davis @ 2024-08-21 16:23 UTC (permalink / raw)
  To: Nishanth Menon, Vignesh Raghavendra, Tero Kristo, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Jan Kiszka
  Cc: linux-arm-kernel, devicetree, linux-kernel, Andrew Davis

This node contains a child which is only probed if simple-mfd is in the
compatible list. Add this here.

Signed-off-by: Andrew Davis <afd@ti.com>
Acked-by: Rob Herring (Arm) <robh@kernel.org>
---

Changes for v2:
 - Added tags
 - Rebased on v6.11-rc4

 .../devicetree/bindings/soc/ti/ti,am654-serdes-ctrl.yaml       | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/soc/ti/ti,am654-serdes-ctrl.yaml b/Documentation/devicetree/bindings/soc/ti/ti,am654-serdes-ctrl.yaml
index a10a3b89ae05e..94b36943a50ff 100644
--- a/Documentation/devicetree/bindings/soc/ti/ti,am654-serdes-ctrl.yaml
+++ b/Documentation/devicetree/bindings/soc/ti/ti,am654-serdes-ctrl.yaml
@@ -14,6 +14,7 @@ properties:
     items:
       - const: ti,am654-serdes-ctrl
       - const: syscon
+      - const: simple-mfd
 
   reg:
     maxItems: 1
@@ -31,7 +32,7 @@ additionalProperties: false
 examples:
   - |
     clock@4080 {
-        compatible = "ti,am654-serdes-ctrl", "syscon";
+        compatible = "ti,am654-serdes-ctrl", "syscon", "simple-mfd";
         reg = <0x4080 0x4>;
 
         mux-controller {
-- 
2.39.2


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

* [PATCH v2 2/2] arm64: dts: ti: k3-am65: Add simple-mfd compatible to SerDes control nodes
  2024-08-21 16:23 [PATCH v2 1/2] dt-bindings: soc: ti: am654-serdes-ctrl: Add simple-mfd to compatible items Andrew Davis
@ 2024-08-21 16:23 ` Andrew Davis
  2024-08-24 19:53 ` [PATCH v2 1/2] dt-bindings: soc: ti: am654-serdes-ctrl: Add simple-mfd to compatible items Nishanth Menon
  1 sibling, 0 replies; 3+ messages in thread
From: Andrew Davis @ 2024-08-21 16:23 UTC (permalink / raw)
  To: Nishanth Menon, Vignesh Raghavendra, Tero Kristo, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Jan Kiszka
  Cc: linux-arm-kernel, devicetree, linux-kernel, Andrew Davis

The SerDes control nodes contain both a clock and clock mux, this is
a simple MFD. Add this to the compatible string list.

Reported-by: Jan Kiszka <jan.kiszka@siemens.com>
Fixes: da795dc4f2a0 ("arm64: dts: ti: k3-am65: Move SerDes mux nodes under the control node")
Signed-off-by: Andrew Davis <afd@ti.com>
Tested-by: Jan Kiszka <jan.kiszka@siemens.com>
---

Changes for v2:
 - Added tags
 - Rebased on v6.11-rc4

 arch/arm64/boot/dts/ti/k3-am65-main.dtsi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
index 1af3dedde1f67..06ed74197f893 100644
--- a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
@@ -478,7 +478,7 @@ scm_conf: scm-conf@100000 {
 		ranges = <0x0 0x0 0x00100000 0x1c000>;
 
 		serdes0_clk: clock@4080 {
-			compatible = "ti,am654-serdes-ctrl", "syscon";
+			compatible = "ti,am654-serdes-ctrl", "syscon", "simple-mfd";
 			reg = <0x4080 0x4>;
 
 			serdes0_mux: mux-controller {
@@ -489,7 +489,7 @@ serdes0_mux: mux-controller {
 		};
 
 		serdes1_clk: clock@4090 {
-			compatible = "ti,am654-serdes-ctrl", "syscon";
+			compatible = "ti,am654-serdes-ctrl", "syscon", "simple-mfd";
 			reg = <0x4090 0x4>;
 
 			serdes1_mux: mux-controller {
-- 
2.39.2


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

* Re: [PATCH v2 1/2] dt-bindings: soc: ti: am654-serdes-ctrl: Add simple-mfd to compatible items
  2024-08-21 16:23 [PATCH v2 1/2] dt-bindings: soc: ti: am654-serdes-ctrl: Add simple-mfd to compatible items Andrew Davis
  2024-08-21 16:23 ` [PATCH v2 2/2] arm64: dts: ti: k3-am65: Add simple-mfd compatible to SerDes control nodes Andrew Davis
@ 2024-08-24 19:53 ` Nishanth Menon
  1 sibling, 0 replies; 3+ messages in thread
From: Nishanth Menon @ 2024-08-24 19:53 UTC (permalink / raw)
  To: Vignesh Raghavendra, Tero Kristo, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Jan Kiszka, Andrew Davis
  Cc: Nishanth Menon, linux-arm-kernel, devicetree, linux-kernel

Hi Andrew Davis,

On Wed, 21 Aug 2024 11:23:36 -0500, Andrew Davis wrote:
> This node contains a child which is only probed if simple-mfd is in the
> compatible list. Add this here.
> 
> 

I have applied the following to branch ti-k3-dts-next on [1].
NOTE: I edited your second patch to include a closes link (used the patch from
Jan as reference), Let me know if you disagree and I can drop the series (for
future reference: Reported-by is usually followed by Closes tag)
Thank you!

[1/2] dt-bindings: soc: ti: am654-serdes-ctrl: Add simple-mfd to compatible items
      commit: f92ed4e4c8454144158bda90614109373e210676
[2/2] arm64: dts: ti: k3-am65: Add simple-mfd compatible to SerDes control nodes
      commit: 73f7ec38556e51f32627fd18c28e56aac4da2646

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent up the chain during
the next merge window (or sooner if it is a relevant bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

[1] https://git.kernel.org/pub/scm/linux/kernel/git/ti/linux.git
-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D


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

end of thread, other threads:[~2024-08-24 19:53 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-08-21 16:23 [PATCH v2 1/2] dt-bindings: soc: ti: am654-serdes-ctrl: Add simple-mfd to compatible items Andrew Davis
2024-08-21 16:23 ` [PATCH v2 2/2] arm64: dts: ti: k3-am65: Add simple-mfd compatible to SerDes control nodes Andrew Davis
2024-08-24 19:53 ` [PATCH v2 1/2] dt-bindings: soc: ti: am654-serdes-ctrl: Add simple-mfd to compatible items Nishanth Menon

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