Add documentation for the MDIO bus multiplexer found on the Amlogic GXL
SoC family
Signed-off-by: Jerome Brunet <[email protected]>
---
.../bindings/net/amlogic,gxl-mdio-mux.yaml | 64 +++++++++++++++++++
1 file changed, 64 insertions(+)
create mode 100644 Documentation/devicetree/bindings/net/amlogic,gxl-mdio-mux.yaml
diff --git a/Documentation/devicetree/bindings/net/amlogic,gxl-mdio-mux.yaml b/Documentation/devicetree/bindings/net/amlogic,gxl-mdio-mux.yaml
new file mode 100644
index 000000000000..d21bce695fa9
--- /dev/null
+++ b/Documentation/devicetree/bindings/net/amlogic,gxl-mdio-mux.yaml
@@ -0,0 +1,64 @@
+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/net/amlogic,gxl-mdio-mux.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Amlogic GXL MDIO bus multiplexer
+
+maintainers:
+ - Jerome Brunet <[email protected]>
+
+description:
+ This is a special case of a MDIO bus multiplexer. It allows to choose between
+ the internal mdio bus leading to the embedded 10/100 PHY or the external
+ MDIO bus on the Amlogic GXL SoC family.
+
+allOf:
+ - $ref: mdio-mux.yaml#
+
+properties:
+ compatible:
+ const: amlogic,gxl-mdio-mux
+
+ reg:
+ maxItems: 1
+
+ clocks:
+ maxItems: 1
+
+ clock-names:
+ items:
+ - const: ref
+
+required:
+ - compatible
+ - reg
+ - clocks
+ - clock-names
+
+unevaluatedProperties: false
+
+examples:
+ - |
+ eth_phy_mux: mdio@558 {
+ #address-cells = <1>;
+ #size-cells = <0>;
+ compatible = "amlogic,gxl-mdio-mux";
+ clocks = <&refclk>;
+ clock-names = "ref";
+ reg = <0x558 0xc>;
+ mdio-parent-bus = <&mdio0>;
+
+ external_mdio: mdio@0 {
+ #address-cells = <1>;
+ #size-cells = <0>;
+ reg = <0x0>;
+ };
+
+ internal_mdio: mdio@1 {
+ #address-cells = <1>;
+ #size-cells = <0>;
+ reg = <0x1>;
+ };
+ };
--
2.39.0
On 16/01/2023 10:16, Jerome Brunet wrote:
> Add documentation for the MDIO bus multiplexer found on the Amlogic GXL
> SoC family
Please use scripts/get_maintainers.pl to get a list of necessary people
and lists to CC. It might happen, that command when run on an older
kernel, gives you outdated entries. Therefore please be sure you base
your patches on recent Linux kernel.
>
> Signed-off-by: Jerome Brunet <[email protected]>
> ---
> .../bindings/net/amlogic,gxl-mdio-mux.yaml | 64 +++++++++++++++++++
> 1 file changed, 64 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/net/amlogic,gxl-mdio-mux.yaml
>
> diff --git a/Documentation/devicetree/bindings/net/amlogic,gxl-mdio-mux.yaml b/Documentation/devicetree/bindings/net/amlogic,gxl-mdio-mux.yaml
> new file mode 100644
> index 000000000000..d21bce695fa9
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/net/amlogic,gxl-mdio-mux.yaml
> @@ -0,0 +1,64 @@
> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/net/amlogic,gxl-mdio-mux.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Amlogic GXL MDIO bus multiplexer
> +
> +maintainers:
> + - Jerome Brunet <[email protected]>
> +
> +description:
> + This is a special case of a MDIO bus multiplexer. It allows to choose between
> + the internal mdio bus leading to the embedded 10/100 PHY or the external
> + MDIO bus on the Amlogic GXL SoC family.
> +
> +allOf:
> + - $ref: mdio-mux.yaml#
> +
> +properties:
> + compatible:
> + const: amlogic,gxl-mdio-mux
> +
> + reg:
> + maxItems: 1
> +
> + clocks:
> + maxItems: 1
> +
> + clock-names:
> + items:
> + - const: ref
> +
> +required:
> + - compatible
> + - reg
> + - clocks
> + - clock-names
> +
> +unevaluatedProperties: false
> +
> +examples:
> + - |
> + eth_phy_mux: mdio@558 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> + compatible = "amlogic,gxl-mdio-mux";
compatible, then reg then the rest.
> + clocks = <&refclk>;
> + clock-names = "ref";
> + reg = <0x558 0xc>;
> + mdio-parent-bus = <&mdio0>;
> +
> + external_mdio: mdio@0 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> + reg = <0x0>;
reg is before other properties.
> + };
> +
> + internal_mdio: mdio@1 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> + reg = <0x1>;
Ditto. If you resend, keep my tag and finally use get_maintainers.pl
Reviewed-by: Krzysztof Kozlowski <[email protected]>
Best regards,
Krzysztof
On Tue 17 Jan 2023 at 09:31, Krzysztof Kozlowski <[email protected]> wrote:
> On 16/01/2023 10:16, Jerome Brunet wrote:
>> Add documentation for the MDIO bus multiplexer found on the Amlogic GXL
>> SoC family
>
> Please use scripts/get_maintainers.pl to get a list of necessary people
> and lists to CC. It might happen, that command when run on an older
> kernel, gives you outdated entries. Therefore please be sure you base
> your patches on recent Linux kernel.
>
Hi Krzysztof,
I do use get_maintainers.pl but I also filter based on past experience
to avoid spamming to much. It seems I stayed on the pre-2015
requirement to send only to devicetree list (I was actually making an
exception specifically for DT) ... and there was no complain so far ;)
I've read documentation again and it is explicit. This will be fixed for
v2.
Thanks for pointing this out.
On 17/01/2023 10:05, Jerome Brunet wrote:
>
> On Tue 17 Jan 2023 at 09:31, Krzysztof Kozlowski <[email protected]> wrote:
>
>> On 16/01/2023 10:16, Jerome Brunet wrote:
>>> Add documentation for the MDIO bus multiplexer found on the Amlogic GXL
>>> SoC family
>>
>> Please use scripts/get_maintainers.pl to get a list of necessary people
>> and lists to CC. It might happen, that command when run on an older
>> kernel, gives you outdated entries. Therefore please be sure you base
>> your patches on recent Linux kernel.
>>
>
> Hi Krzysztof,
>
> I do use get_maintainers.pl but I also filter based on past experience
> to avoid spamming to much. It seems I stayed on the pre-2015
> requirement to send only to devicetree list (I was actually making an
> exception specifically for DT) ... and there was no complain so far ;)
>
> I've read documentation again and it is explicit. This will be fixed for
> v2.
>
> Thanks for pointing this out.
For regular patchsets not spanning over 10 different subsystems (so
total number of CCs should be 5-10), please Cc all
maintainers/reviewers/supporters/lists pointed by maintainers.pl. Skip
git fallback. How your patch should appear in my mailbox if you skip me?
Not everyone are using Patchwork.
Best regards,
Krzysztof