Received-SPF: pass (google.com: domain of linux-kernel+bounces-11608-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
From: Josua Mayer <josua@solid-run.com>
Subject: [PATCH v3 0/8] dt-bindings: marvell: a38x: add solidrun armada 388
 clearfog boards
Date: Tue, 26 Dec 2023 19:11:31 +0100
Message-Id: <20231226-support-clearfog-gtr-l8-sfp-v3-0-fd1ac2d6bf2e@solid-run.com>
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
To: Andrew Lunn <andrew@lunn.ch>, 
 Gregory Clement <gregory.clement@bootlin.com>, 
 Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>, 
 Rob Herring <robh+dt@kernel.org>, 
 Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, 
 Conor Dooley <conor+dt@kernel.org>
Cc: linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, 
 linux-kernel@vger.kernel.org, Josua Mayer <josua@solid-run.com>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?Y3pHUDc1ZjYxYnB5V0lLNU8wdEVSN3drdmtaMDd6Y0lqSDA0VENZZThtd09m?=
 =?utf-8?B?dmVGZ1JLVXNkcW9PTHlHZFRaUmZOd0RuQ1JlVDZiVHFEUnFxTlh1MFZVc1FP?=
 =?utf-8?B?QVM2cHMxVnc3N3RwNnFMRjVVQXl3Z0h1emFFY1ZKMHpxeU1YWnp2cEVZS1RD?=
 =?utf-8?B?K2ZGRExRU0FEc1FXSkpJWk4zaGNlS3prNHAyMFhXQkV0OGhHUXZtRmU3Y05V?=
 =?utf-8?B?ekVXd2NYaEZpYzdDeXB2NUNGSkFzU1FNWlBlL1NyaFpvbG54TU5mV0pFVzlK?=
 =?utf-8?B?bkJDTlFRY3l4cHVmcmpCL1RIWnFXWm5nazlxaGh6Z0ljVzVyRXMzZkczcVlX?=
 =?utf-8?B?K05FSWlPaUNDZ3EyREdJUk0wZlRIanF0d01UdUtKUUxHVmZqeEZteWVZY1U0?=
 =?utf-8?B?VXdvNWhHNm1DZFRRWi9NK0xJd2dpdGZidXhtYUxHMWppOVN1eFFWMHNvVmNN?=
 =?utf-8?B?OEJiUU91N3ZIV1J3N2NRc0JFNGdZQzB2NDFITmRLaUF2YWN1WEwydmNPZDR2?=
 =?utf-8?B?bS93SnpYR1picmhOdGs1eG8wTGpPbmk1dkJ4T3Qvb2ZIeHRHTzI2ZE4xeERC?=
 =?utf-8?B?cUR0WGVVYS9sWisrTmZjYXN3RnAwU0lUN1JPMnhoK3llRzU5ZjJqeEZLME9L?=
 =?utf-8?B?M1RPOXBVbWpMMHhqZ3R2a1VJMUR4cjcraUQxdU55RWU1VWhwVjFRREJBanZS?=
 =?utf-8?B?TjhYanBWdE5GY0dzMU1aUUQ3RWQ1ak5CMHF0WGZ4WGQ2Z2FiRUdqcXBPYUcy?=
 =?utf-8?B?M3Fva3lkMGQ5Z2Iva0s4Tm5oZjYyU2p0azhVblZna2U4cHJtMFBUWUh4dm9N?=
 =?utf-8?B?eXZ6OFpJTENVL2s0N1lhaWcxY0xQQ0NRRit3NFN3S2ZTNytWbEVLU2QwVEV2?=
 =?utf-8?B?TFZOSmQ0aXp2YlRKZDhRQWxnZjZ6ejV5aThsazRiODNIQWk4L1NqcnBTV01S?=
 =?utf-8?B?SGdwWVRDZG1IU0d5SkV6aDBtS1VRcFhKYjA2djNmam44Vk1hMldXMGtmRjNF?=
 =?utf-8?B?ZlJlQkFTckZKTU14aktIZ3B0OU00UitmSnI3YWRRMlluY1BQNHV1by9acUVP?=
 =?utf-8?B?eGlkajRrQkpzNmZ5WWZyalM4THdMaUVMTGxpTHR6Z2cveWxRMUlxdTNiTUh1?=
 =?utf-8?B?c3k3VFU2ck0wRHlSeklHRXJvWUtzTjlibGtLTnlyZlYxcVR4K2RqNzdHczhY?=
 =?utf-8?B?OC9hSFA1VURnT05tUEZXbDJmelZNdVpOU2ljTkVYdFFHRGxhNjVLNHBGb2Zt?=
 =?utf-8?B?SWVGdnVnSVRXZS9BNUtvazh5S252amNGcTNaV1M2L0RRb2RtUFMzN0FzaTM2?=
 =?utf-8?B?dTBmNkNBRzNnTjRyckJZOFZPQ1pOTFpXdmZLNzEyczdOblZ0RGEzSnNXK3lq?=
 =?utf-8?B?eC91VmhHVFZ4QVliaEtaOWtwbU9UYTU3MjF1aWRGaWFpR1ExMTh1OEdpSXBM?=
 =?utf-8?B?N0N1OWJnNXFsSHlDMkJFYW40a1V1eWw4bTZIdlI0ZXYxcDN2Mk43YysxVThq?=
 =?utf-8?B?bE5OK3hXSVZGMlBERWJScXQvZncrRy9lYk1vYUE1YUdWWTE5dDM0ZlhscDZF?=
 =?utf-8?B?SmJTZTFJNVdKVjhHRTBNdzBBYVlqR09MTE9QWmxLU1FJMTU4c3ZERGdTdHVF?=
 =?utf-8?B?eXN4N29NNXBOSWR2TEM3WWY2RTBoamV6SGhMdkhuelU2bVd0anR6aEQ1RzRK?=
 =?utf-8?B?elBHclU4R0YrMWdKQ0JHaUl2dTB0OWt3bDlySnNONVBJMzJrZlFLSzZMbEs1?=
 =?utf-8?B?UC9uc1VXak9jSFNFeGw1bFRpVUk3L2NWSkpXcHBROGJpMnJOZW80SWN2cnJQ?=
 =?utf-8?B?a2tISHdQT0pEdkxYNnV2TGw4Z0JpdXBBYVNuZVM1b25GS0hrakN5a1Jkc1Vs?=
 =?utf-8?B?Rng3ek1udEZ1dTBXelJjVU1XWGhtTVY5NmpPVFpJMWVWVHdwUEV6bU9IWUlB?=
 =?utf-8?B?SmZ0RkYxU0xmVXNRWGtVck12bXFUeTZFRFFjVGxnbS9VUWVscDQ4S3R5Ri9W?=
 =?utf-8?B?cEhzNnFmNWNpUmVVYVV2T0huWnVmM3VFbGREd1RoZUZwcCs1V0Y4OVVwSVNL?=
 =?utf-8?B?T3BhOWI3d01TTmVwei90Y2k5bm5lTHpNZHVxUmhpQW15VGpDS0QxTDBBRzJR?=
 =?utf-8?Q?Esp2tPLVW3NLukG4AG6u0VWLh?=
X-OriginatorOrg: solid-run.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 2ae4b52b-e45d-4537-2d80-08dc063e2234
X-MS-Exchange-CrossTenant-AuthSource: AS8PR04MB8963.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Dec 2023 18:11:51.2526
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: a4a8aaf3-fd27-4e27-add2-604707ce5b82
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: MgRWjQ7zMDPnPg4bKYNoTySzBKxVaCowzA4AQa+cHDZK2NLMfX9vnnE6KDnri7UWk2+2vGx5i1iJ1tE/tJur1Q==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9PR04MB9868

Dear Maintainers,

The initially merged device-tree for Clearfog GTR devices contained
various subtle mistakes and omissions:

- missing board-specific compatible strings
- missing pinctrl entries
- missing second sfp connector
- invalid sfp loss-of-signal gpio
- mismatch of labels between dsa ports and enclosure

Most notably this had caused functional issues with the sfp connectors.

This patch-set first converts the armada-38x dt-bindings to yaml.
New bindings are added specifically for the Clearfog GTR boards,
and for the already in-tree clearfog base and pro boards' compatible strings.

Secondly pinctrl nodes are added for all referenced gpios for independence
from bootloader defaults. U-Boot is shared between armada-388 clearfog,
and armada-385 clearfog gtr.

Further remove an invalid io from the first sfp connector description,
and add descriptions for the secondary sfp connector which is driven
by dsa switch port number 9.

Finally labels of dsa switch ports were updated to match the enclosure.
That patch is not suitable for stable.

Signed-off-by: Josua Mayer <josua@solid-run.com>
---
Changes in v3:
- armada-38x.yaml: removed '|', no need to prerserve formatting
  (reported by Conor Dooley conor+dt@kernel.org)
- update commit descriptions to clarify confusing board names and
  compatible strings
  (reported by Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>)
- send to all relevant lists
  (reported by Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>)
- remove duplicate binding for clearfog / clearfog
- Link to v2 (patches 0-2):
  https://lore.kernel.org/r/20231224143750.5604-1-josua@solid-run.com
- Link to v2 (patches 3-8):
  https://lore.kernel.org/r/20231224143850.5671-3-josua@solid-run.com

Changes in v2:
- removed changes changes to gpio numbers because they were wrong
- added bindings documentation
- Link to v1: https://lore.kernel.org/r/20231223212930.14624-1-josua@solid-run.com

---
Josua Mayer (8):
      dt-bindings: marvell: a38x: convert the soc compatibles description to yaml
      dt-bindings: marvell: a38x: add solidrun armada 385 clearfog gtr boards
      dt-bindings: marvell: a38x: add solidrun armada 388 clearfog boards
      arm: dts: marvell: clearfog-gtr: add board-specific compatible strings
      arm: dts: marvell: clearfog-gtr: sort pinctrl nodes alphabetically
      arm: dts: marvell: clearfog-gtr: add missing pinctrl for all used gpios
      arm: dts: marvell: clearfog-gtr-l8: add support for second sfp connector
      arm: dts: marvell: clearfog-gtr-l8: align port numbers with enclosure

 .../devicetree/bindings/arm/marvell/armada-38x.txt | 27 -------
 .../bindings/arm/marvell/armada-38x.yaml           | 53 ++++++++++++++
 .../dts/marvell/armada-385-clearfog-gtr-l8.dts     | 38 +++++++---
 .../dts/marvell/armada-385-clearfog-gtr-s4.dts     |  2 +
 .../boot/dts/marvell/armada-385-clearfog-gtr.dtsi  | 84 ++++++++++++++++------
 5 files changed, 147 insertions(+), 57 deletions(-)
---
base-commit: 861deac3b092f37b2c5e6871732f3e11486f7082
change-id: 20231226-support-clearfog-gtr-l8-sfp-d87ae715a787

Sincerely,
-- 
Josua Mayer <josua@solid-run.com>