Skip to contentSkip to conversation

dt-bindings: soc: ti: am654-serdes-ctrl: Add simple-mfd to compatible items

PATCH In review Andrew Davis 2 participants 3 messages Aug 21, 2024 – Aug 24, 2024

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH v2 1/2 dt-bindings: soc: ti: am654-serdes-ctrl: Add simple-mfd to compatible items

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

---

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(-)
Documentation/devicetree/bindings/soc/ti/ti,am654-serdes-ctrl.yaml +2 -1
index a10a3b89ae05e..94b36943a50ff 100644
@@ -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 {
Signature · 1 line
2.39.2
#1 lore raw

PATCH v2 2/2 arm64: dts: ti: k3-am65: Add simple-mfd compatible to SerDes control nodes

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

Fixes da795dc4f2a0 ("arm64: dts: ti: k3-am65: Move SerDes mux nodes under the control node")

---

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(-)
arch/arm64/boot/dts/ti/k3-am65-main.dtsi +2 -2
index 1af3dedde1f67..06ed74197f893 100644
@@ -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 {
Signature · 1 line
2.39.2
maintainer
#2 lore raw
In reply to Andrew Davis

Hi Andrew Davis,

Quoted 4 lines · Andrew Davis

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

Signature · 3 lines
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D