* [PATCH 0/2] phy: amlogic: Add support for the G12A Analog MIPI D-PHY
@ 2020-11-23 14:51 ` Neil Armstrong
0 siblings, 0 replies; 24+ messages in thread
From: Neil Armstrong @ 2020-11-23 14:51 UTC (permalink / raw)
To: vkoul, kishon
Cc: linux-amlogic, linux-arm-kernel, devicetree, linux-kernel,
Neil Armstrong
The Amlogic G12A SoCs embeds an Analog MIPI D-PHY to communicate with DSI
panels, this adds the bindings.
This Analog D-PHY works with a separate Digital MIPI D-PHY.
This serie adds the Bindings and the PHY driver.
Neil Armstrong (2):
dt-bindings: phy: add Amlogic G12A Analog MIPI D-PHY bindings
phy: amlogic: Add G12A Analog MIPI D-PHY driver
.../phy/amlogic,g12a-mipi-dphy-analog.yaml | 40 ++++
drivers/phy/amlogic/Kconfig | 12 ++
drivers/phy/amlogic/Makefile | 1 +
.../amlogic/phy-meson-g12a-mipi-dphy-analog.c | 177 ++++++++++++++++++
4 files changed, 230 insertions(+)
create mode 100644 Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml
create mode 100644 drivers/phy/amlogic/phy-meson-g12a-mipi-dphy-analog.c
--
2.25.1
^ permalink raw reply [flat|nested] 24+ messages in thread* [PATCH 0/2] phy: amlogic: Add support for the G12A Analog MIPI D-PHY @ 2020-11-23 14:51 ` Neil Armstrong 0 siblings, 0 replies; 24+ messages in thread From: Neil Armstrong @ 2020-11-23 14:51 UTC (permalink / raw) To: vkoul, kishon Cc: linux-amlogic, Neil Armstrong, linux-kernel, linux-arm-kernel, devicetree The Amlogic G12A SoCs embeds an Analog MIPI D-PHY to communicate with DSI panels, this adds the bindings. This Analog D-PHY works with a separate Digital MIPI D-PHY. This serie adds the Bindings and the PHY driver. Neil Armstrong (2): dt-bindings: phy: add Amlogic G12A Analog MIPI D-PHY bindings phy: amlogic: Add G12A Analog MIPI D-PHY driver .../phy/amlogic,g12a-mipi-dphy-analog.yaml | 40 ++++ drivers/phy/amlogic/Kconfig | 12 ++ drivers/phy/amlogic/Makefile | 1 + .../amlogic/phy-meson-g12a-mipi-dphy-analog.c | 177 ++++++++++++++++++ 4 files changed, 230 insertions(+) create mode 100644 Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml create mode 100644 drivers/phy/amlogic/phy-meson-g12a-mipi-dphy-analog.c -- 2.25.1 _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic ^ permalink raw reply [flat|nested] 24+ messages in thread
* [PATCH 0/2] phy: amlogic: Add support for the G12A Analog MIPI D-PHY @ 2020-11-23 14:51 ` Neil Armstrong 0 siblings, 0 replies; 24+ messages in thread From: Neil Armstrong @ 2020-11-23 14:51 UTC (permalink / raw) To: vkoul, kishon Cc: linux-amlogic, Neil Armstrong, linux-kernel, linux-arm-kernel, devicetree The Amlogic G12A SoCs embeds an Analog MIPI D-PHY to communicate with DSI panels, this adds the bindings. This Analog D-PHY works with a separate Digital MIPI D-PHY. This serie adds the Bindings and the PHY driver. Neil Armstrong (2): dt-bindings: phy: add Amlogic G12A Analog MIPI D-PHY bindings phy: amlogic: Add G12A Analog MIPI D-PHY driver .../phy/amlogic,g12a-mipi-dphy-analog.yaml | 40 ++++ drivers/phy/amlogic/Kconfig | 12 ++ drivers/phy/amlogic/Makefile | 1 + .../amlogic/phy-meson-g12a-mipi-dphy-analog.c | 177 ++++++++++++++++++ 4 files changed, 230 insertions(+) create mode 100644 Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml create mode 100644 drivers/phy/amlogic/phy-meson-g12a-mipi-dphy-analog.c -- 2.25.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 24+ messages in thread
* [PATCH 1/2] dt-bindings: phy: add Amlogic G12A Analog MIPI D-PHY bindings 2020-11-23 14:51 ` Neil Armstrong (?) @ 2020-11-23 14:51 ` Neil Armstrong -1 siblings, 0 replies; 24+ messages in thread From: Neil Armstrong @ 2020-11-23 14:51 UTC (permalink / raw) To: vkoul, kishon Cc: linux-amlogic, linux-arm-kernel, devicetree, linux-kernel, Neil Armstrong The Amlogic G12A SoCs embeds an Analog MIPI D-PHY to communicate with DSI panels, this adds the bindings. This Analog D-PHY works with a separate Digital MIPI D-PHY. Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> --- .../phy/amlogic,g12a-mipi-dphy-analog.yaml | 40 +++++++++++++++++++ 1 file changed, 40 insertions(+) create mode 100644 Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml diff --git a/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml new file mode 100644 index 000000000000..28663552f05b --- /dev/null +++ b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml @@ -0,0 +1,40 @@ +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) +%YAML 1.2 +--- +$id: "http://devicetree.org/schemas/phy/amlogic,g12a-mipi-dphy-analog.yaml#" +$schema: "http://devicetree.org/meta-schemas/core.yaml#" + +title: Amlogic G12A MIPI analog PHY + +maintainers: + - Neil Armstrong <narmstrong@baylibre.com> + +description: |+ + The Everything-Else Power Domains node should be the child of a syscon + node with the required property: + + - compatible: Should be the following: + "amlogic,meson-gx-hhi-sysctrl", "simple-mfd", "syscon" + + Refer to the the bindings described in + Documentation/devicetree/bindings/mfd/syscon.yaml + +properties: + compatible: + const: amlogic,g12a-mipi-dphy-analog + + "#phy-cells": + const: 0 + +required: + - compatible + - "#phy-cells" + +additionalProperties: false + +examples: + - | + mpphy: phy { + compatible = "amlogic,g12a-mipi-dphy-analog"; + #phy-cells = <0>; + }; -- 2.25.1 ^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 1/2] dt-bindings: phy: add Amlogic G12A Analog MIPI D-PHY bindings @ 2020-11-23 14:51 ` Neil Armstrong 0 siblings, 0 replies; 24+ messages in thread From: Neil Armstrong @ 2020-11-23 14:51 UTC (permalink / raw) To: vkoul, kishon Cc: linux-amlogic, Neil Armstrong, linux-kernel, linux-arm-kernel, devicetree The Amlogic G12A SoCs embeds an Analog MIPI D-PHY to communicate with DSI panels, this adds the bindings. This Analog D-PHY works with a separate Digital MIPI D-PHY. Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> --- .../phy/amlogic,g12a-mipi-dphy-analog.yaml | 40 +++++++++++++++++++ 1 file changed, 40 insertions(+) create mode 100644 Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml diff --git a/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml new file mode 100644 index 000000000000..28663552f05b --- /dev/null +++ b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml @@ -0,0 +1,40 @@ +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) +%YAML 1.2 +--- +$id: "http://devicetree.org/schemas/phy/amlogic,g12a-mipi-dphy-analog.yaml#" +$schema: "http://devicetree.org/meta-schemas/core.yaml#" + +title: Amlogic G12A MIPI analog PHY + +maintainers: + - Neil Armstrong <narmstrong@baylibre.com> + +description: |+ + The Everything-Else Power Domains node should be the child of a syscon + node with the required property: + + - compatible: Should be the following: + "amlogic,meson-gx-hhi-sysctrl", "simple-mfd", "syscon" + + Refer to the the bindings described in + Documentation/devicetree/bindings/mfd/syscon.yaml + +properties: + compatible: + const: amlogic,g12a-mipi-dphy-analog + + "#phy-cells": + const: 0 + +required: + - compatible + - "#phy-cells" + +additionalProperties: false + +examples: + - | + mpphy: phy { + compatible = "amlogic,g12a-mipi-dphy-analog"; + #phy-cells = <0>; + }; -- 2.25.1 _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic ^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 1/2] dt-bindings: phy: add Amlogic G12A Analog MIPI D-PHY bindings @ 2020-11-23 14:51 ` Neil Armstrong 0 siblings, 0 replies; 24+ messages in thread From: Neil Armstrong @ 2020-11-23 14:51 UTC (permalink / raw) To: vkoul, kishon Cc: linux-amlogic, Neil Armstrong, linux-kernel, linux-arm-kernel, devicetree The Amlogic G12A SoCs embeds an Analog MIPI D-PHY to communicate with DSI panels, this adds the bindings. This Analog D-PHY works with a separate Digital MIPI D-PHY. Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> --- .../phy/amlogic,g12a-mipi-dphy-analog.yaml | 40 +++++++++++++++++++ 1 file changed, 40 insertions(+) create mode 100644 Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml diff --git a/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml new file mode 100644 index 000000000000..28663552f05b --- /dev/null +++ b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml @@ -0,0 +1,40 @@ +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) +%YAML 1.2 +--- +$id: "http://devicetree.org/schemas/phy/amlogic,g12a-mipi-dphy-analog.yaml#" +$schema: "http://devicetree.org/meta-schemas/core.yaml#" + +title: Amlogic G12A MIPI analog PHY + +maintainers: + - Neil Armstrong <narmstrong@baylibre.com> + +description: |+ + The Everything-Else Power Domains node should be the child of a syscon + node with the required property: + + - compatible: Should be the following: + "amlogic,meson-gx-hhi-sysctrl", "simple-mfd", "syscon" + + Refer to the the bindings described in + Documentation/devicetree/bindings/mfd/syscon.yaml + +properties: + compatible: + const: amlogic,g12a-mipi-dphy-analog + + "#phy-cells": + const: 0 + +required: + - compatible + - "#phy-cells" + +additionalProperties: false + +examples: + - | + mpphy: phy { + compatible = "amlogic,g12a-mipi-dphy-analog"; + #phy-cells = <0>; + }; -- 2.25.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] dt-bindings: phy: add Amlogic G12A Analog MIPI D-PHY bindings 2020-11-23 14:51 ` Neil Armstrong (?) @ 2020-12-05 8:04 ` Vinod Koul -1 siblings, 0 replies; 24+ messages in thread From: Vinod Koul @ 2020-12-05 8:04 UTC (permalink / raw) To: Neil Armstrong, Rob Herring Cc: kishon, linux-amlogic, linux-arm-kernel, devicetree, linux-kernel On 23-11-20, 15:51, Neil Armstrong wrote: > The Amlogic G12A SoCs embeds an Analog MIPI D-PHY to communicate with DSI > panels, this adds the bindings. > > This Analog D-PHY works with a separate Digital MIPI D-PHY. Pls cc Rob on dt binding patches (hint get_maintainer.pl would tell you do so) > > Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> > --- > .../phy/amlogic,g12a-mipi-dphy-analog.yaml | 40 +++++++++++++++++++ > 1 file changed, 40 insertions(+) > create mode 100644 Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > > diff --git a/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > new file mode 100644 > index 000000000000..28663552f05b > --- /dev/null > +++ b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > @@ -0,0 +1,40 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: "http://devicetree.org/schemas/phy/amlogic,g12a-mipi-dphy-analog.yaml#" > +$schema: "http://devicetree.org/meta-schemas/core.yaml#" > + > +title: Amlogic G12A MIPI analog PHY > + > +maintainers: > + - Neil Armstrong <narmstrong@baylibre.com> > + > +description: |+ > + The Everything-Else Power Domains node should be the child of a syscon > + node with the required property: > + > + - compatible: Should be the following: > + "amlogic,meson-gx-hhi-sysctrl", "simple-mfd", "syscon" > + > + Refer to the the bindings described in > + Documentation/devicetree/bindings/mfd/syscon.yaml > + > +properties: > + compatible: > + const: amlogic,g12a-mipi-dphy-analog > + > + "#phy-cells": > + const: 0 > + > +required: > + - compatible > + - "#phy-cells" > + > +additionalProperties: false > + > +examples: > + - | > + mpphy: phy { > + compatible = "amlogic,g12a-mipi-dphy-analog"; > + #phy-cells = <0>; > + }; > -- > 2.25.1 -- ~Vinod ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] dt-bindings: phy: add Amlogic G12A Analog MIPI D-PHY bindings @ 2020-12-05 8:04 ` Vinod Koul 0 siblings, 0 replies; 24+ messages in thread From: Vinod Koul @ 2020-12-05 8:04 UTC (permalink / raw) To: Neil Armstrong, Rob Herring Cc: linux-amlogic, devicetree, linux-kernel, linux-arm-kernel, kishon On 23-11-20, 15:51, Neil Armstrong wrote: > The Amlogic G12A SoCs embeds an Analog MIPI D-PHY to communicate with DSI > panels, this adds the bindings. > > This Analog D-PHY works with a separate Digital MIPI D-PHY. Pls cc Rob on dt binding patches (hint get_maintainer.pl would tell you do so) > > Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> > --- > .../phy/amlogic,g12a-mipi-dphy-analog.yaml | 40 +++++++++++++++++++ > 1 file changed, 40 insertions(+) > create mode 100644 Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > > diff --git a/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > new file mode 100644 > index 000000000000..28663552f05b > --- /dev/null > +++ b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > @@ -0,0 +1,40 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: "http://devicetree.org/schemas/phy/amlogic,g12a-mipi-dphy-analog.yaml#" > +$schema: "http://devicetree.org/meta-schemas/core.yaml#" > + > +title: Amlogic G12A MIPI analog PHY > + > +maintainers: > + - Neil Armstrong <narmstrong@baylibre.com> > + > +description: |+ > + The Everything-Else Power Domains node should be the child of a syscon > + node with the required property: > + > + - compatible: Should be the following: > + "amlogic,meson-gx-hhi-sysctrl", "simple-mfd", "syscon" > + > + Refer to the the bindings described in > + Documentation/devicetree/bindings/mfd/syscon.yaml > + > +properties: > + compatible: > + const: amlogic,g12a-mipi-dphy-analog > + > + "#phy-cells": > + const: 0 > + > +required: > + - compatible > + - "#phy-cells" > + > +additionalProperties: false > + > +examples: > + - | > + mpphy: phy { > + compatible = "amlogic,g12a-mipi-dphy-analog"; > + #phy-cells = <0>; > + }; > -- > 2.25.1 -- ~Vinod _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] dt-bindings: phy: add Amlogic G12A Analog MIPI D-PHY bindings @ 2020-12-05 8:04 ` Vinod Koul 0 siblings, 0 replies; 24+ messages in thread From: Vinod Koul @ 2020-12-05 8:04 UTC (permalink / raw) To: Neil Armstrong, Rob Herring Cc: linux-amlogic, devicetree, linux-kernel, linux-arm-kernel, kishon On 23-11-20, 15:51, Neil Armstrong wrote: > The Amlogic G12A SoCs embeds an Analog MIPI D-PHY to communicate with DSI > panels, this adds the bindings. > > This Analog D-PHY works with a separate Digital MIPI D-PHY. Pls cc Rob on dt binding patches (hint get_maintainer.pl would tell you do so) > > Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> > --- > .../phy/amlogic,g12a-mipi-dphy-analog.yaml | 40 +++++++++++++++++++ > 1 file changed, 40 insertions(+) > create mode 100644 Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > > diff --git a/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > new file mode 100644 > index 000000000000..28663552f05b > --- /dev/null > +++ b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > @@ -0,0 +1,40 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: "http://devicetree.org/schemas/phy/amlogic,g12a-mipi-dphy-analog.yaml#" > +$schema: "http://devicetree.org/meta-schemas/core.yaml#" > + > +title: Amlogic G12A MIPI analog PHY > + > +maintainers: > + - Neil Armstrong <narmstrong@baylibre.com> > + > +description: |+ > + The Everything-Else Power Domains node should be the child of a syscon > + node with the required property: > + > + - compatible: Should be the following: > + "amlogic,meson-gx-hhi-sysctrl", "simple-mfd", "syscon" > + > + Refer to the the bindings described in > + Documentation/devicetree/bindings/mfd/syscon.yaml > + > +properties: > + compatible: > + const: amlogic,g12a-mipi-dphy-analog > + > + "#phy-cells": > + const: 0 > + > +required: > + - compatible > + - "#phy-cells" > + > +additionalProperties: false > + > +examples: > + - | > + mpphy: phy { > + compatible = "amlogic,g12a-mipi-dphy-analog"; > + #phy-cells = <0>; > + }; > -- > 2.25.1 -- ~Vinod _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] dt-bindings: phy: add Amlogic G12A Analog MIPI D-PHY bindings 2020-12-05 8:04 ` Vinod Koul (?) @ 2020-12-07 9:50 ` Neil Armstrong -1 siblings, 0 replies; 24+ messages in thread From: Neil Armstrong @ 2020-12-07 9:50 UTC (permalink / raw) To: Vinod Koul, Rob Herring Cc: kishon, linux-amlogic, linux-arm-kernel, devicetree, linux-kernel Hi, On 05/12/2020 09:04, Vinod Koul wrote: > On 23-11-20, 15:51, Neil Armstrong wrote: >> The Amlogic G12A SoCs embeds an Analog MIPI D-PHY to communicate with DSI >> panels, this adds the bindings. >> >> This Analog D-PHY works with a separate Digital MIPI D-PHY. > > Pls cc Rob on dt binding patches (hint get_maintainer.pl would tell you > do so) It's the case. Neil > >> >> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> >> --- >> .../phy/amlogic,g12a-mipi-dphy-analog.yaml | 40 +++++++++++++++++++ >> 1 file changed, 40 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml >> >> diff --git a/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml >> new file mode 100644 >> index 000000000000..28663552f05b >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml >> @@ -0,0 +1,40 @@ >> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) >> +%YAML 1.2 >> +--- >> +$id: "http://devicetree.org/schemas/phy/amlogic,g12a-mipi-dphy-analog.yaml#" >> +$schema: "http://devicetree.org/meta-schemas/core.yaml#" >> + >> +title: Amlogic G12A MIPI analog PHY >> + >> +maintainers: >> + - Neil Armstrong <narmstrong@baylibre.com> >> + >> +description: |+ >> + The Everything-Else Power Domains node should be the child of a syscon >> + node with the required property: >> + >> + - compatible: Should be the following: >> + "amlogic,meson-gx-hhi-sysctrl", "simple-mfd", "syscon" >> + >> + Refer to the the bindings described in >> + Documentation/devicetree/bindings/mfd/syscon.yaml >> + >> +properties: >> + compatible: >> + const: amlogic,g12a-mipi-dphy-analog >> + >> + "#phy-cells": >> + const: 0 >> + >> +required: >> + - compatible >> + - "#phy-cells" >> + >> +additionalProperties: false >> + >> +examples: >> + - | >> + mpphy: phy { >> + compatible = "amlogic,g12a-mipi-dphy-analog"; >> + #phy-cells = <0>; >> + }; >> -- >> 2.25.1 > ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] dt-bindings: phy: add Amlogic G12A Analog MIPI D-PHY bindings @ 2020-12-07 9:50 ` Neil Armstrong 0 siblings, 0 replies; 24+ messages in thread From: Neil Armstrong @ 2020-12-07 9:50 UTC (permalink / raw) To: Vinod Koul, Rob Herring Cc: linux-amlogic, devicetree, linux-kernel, linux-arm-kernel, kishon Hi, On 05/12/2020 09:04, Vinod Koul wrote: > On 23-11-20, 15:51, Neil Armstrong wrote: >> The Amlogic G12A SoCs embeds an Analog MIPI D-PHY to communicate with DSI >> panels, this adds the bindings. >> >> This Analog D-PHY works with a separate Digital MIPI D-PHY. > > Pls cc Rob on dt binding patches (hint get_maintainer.pl would tell you > do so) It's the case. Neil > >> >> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> >> --- >> .../phy/amlogic,g12a-mipi-dphy-analog.yaml | 40 +++++++++++++++++++ >> 1 file changed, 40 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml >> >> diff --git a/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml >> new file mode 100644 >> index 000000000000..28663552f05b >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml >> @@ -0,0 +1,40 @@ >> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) >> +%YAML 1.2 >> +--- >> +$id: "http://devicetree.org/schemas/phy/amlogic,g12a-mipi-dphy-analog.yaml#" >> +$schema: "http://devicetree.org/meta-schemas/core.yaml#" >> + >> +title: Amlogic G12A MIPI analog PHY >> + >> +maintainers: >> + - Neil Armstrong <narmstrong@baylibre.com> >> + >> +description: |+ >> + The Everything-Else Power Domains node should be the child of a syscon >> + node with the required property: >> + >> + - compatible: Should be the following: >> + "amlogic,meson-gx-hhi-sysctrl", "simple-mfd", "syscon" >> + >> + Refer to the the bindings described in >> + Documentation/devicetree/bindings/mfd/syscon.yaml >> + >> +properties: >> + compatible: >> + const: amlogic,g12a-mipi-dphy-analog >> + >> + "#phy-cells": >> + const: 0 >> + >> +required: >> + - compatible >> + - "#phy-cells" >> + >> +additionalProperties: false >> + >> +examples: >> + - | >> + mpphy: phy { >> + compatible = "amlogic,g12a-mipi-dphy-analog"; >> + #phy-cells = <0>; >> + }; >> -- >> 2.25.1 > _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] dt-bindings: phy: add Amlogic G12A Analog MIPI D-PHY bindings @ 2020-12-07 9:50 ` Neil Armstrong 0 siblings, 0 replies; 24+ messages in thread From: Neil Armstrong @ 2020-12-07 9:50 UTC (permalink / raw) To: Vinod Koul, Rob Herring Cc: linux-amlogic, devicetree, linux-kernel, linux-arm-kernel, kishon Hi, On 05/12/2020 09:04, Vinod Koul wrote: > On 23-11-20, 15:51, Neil Armstrong wrote: >> The Amlogic G12A SoCs embeds an Analog MIPI D-PHY to communicate with DSI >> panels, this adds the bindings. >> >> This Analog D-PHY works with a separate Digital MIPI D-PHY. > > Pls cc Rob on dt binding patches (hint get_maintainer.pl would tell you > do so) It's the case. Neil > >> >> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> >> --- >> .../phy/amlogic,g12a-mipi-dphy-analog.yaml | 40 +++++++++++++++++++ >> 1 file changed, 40 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml >> >> diff --git a/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml >> new file mode 100644 >> index 000000000000..28663552f05b >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml >> @@ -0,0 +1,40 @@ >> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) >> +%YAML 1.2 >> +--- >> +$id: "http://devicetree.org/schemas/phy/amlogic,g12a-mipi-dphy-analog.yaml#" >> +$schema: "http://devicetree.org/meta-schemas/core.yaml#" >> + >> +title: Amlogic G12A MIPI analog PHY >> + >> +maintainers: >> + - Neil Armstrong <narmstrong@baylibre.com> >> + >> +description: |+ >> + The Everything-Else Power Domains node should be the child of a syscon >> + node with the required property: >> + >> + - compatible: Should be the following: >> + "amlogic,meson-gx-hhi-sysctrl", "simple-mfd", "syscon" >> + >> + Refer to the the bindings described in >> + Documentation/devicetree/bindings/mfd/syscon.yaml >> + >> +properties: >> + compatible: >> + const: amlogic,g12a-mipi-dphy-analog >> + >> + "#phy-cells": >> + const: 0 >> + >> +required: >> + - compatible >> + - "#phy-cells" >> + >> +additionalProperties: false >> + >> +examples: >> + - | >> + mpphy: phy { >> + compatible = "amlogic,g12a-mipi-dphy-analog"; >> + #phy-cells = <0>; >> + }; >> -- >> 2.25.1 > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] dt-bindings: phy: add Amlogic G12A Analog MIPI D-PHY bindings 2020-11-23 14:51 ` Neil Armstrong (?) @ 2020-12-07 22:44 ` Rob Herring -1 siblings, 0 replies; 24+ messages in thread From: Rob Herring @ 2020-12-07 22:44 UTC (permalink / raw) To: Neil Armstrong Cc: vkoul, kishon, linux-amlogic, linux-arm-kernel, devicetree, linux-kernel On Mon, Nov 23, 2020 at 03:51:56PM +0100, Neil Armstrong wrote: > The Amlogic G12A SoCs embeds an Analog MIPI D-PHY to communicate with DSI > panels, this adds the bindings. > > This Analog D-PHY works with a separate Digital MIPI D-PHY. > > Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> > --- > .../phy/amlogic,g12a-mipi-dphy-analog.yaml | 40 +++++++++++++++++++ > 1 file changed, 40 insertions(+) > create mode 100644 Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > > diff --git a/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > new file mode 100644 > index 000000000000..28663552f05b > --- /dev/null > +++ b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > @@ -0,0 +1,40 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: "http://devicetree.org/schemas/phy/amlogic,g12a-mipi-dphy-analog.yaml#" > +$schema: "http://devicetree.org/meta-schemas/core.yaml#" > + > +title: Amlogic G12A MIPI analog PHY > + > +maintainers: > + - Neil Armstrong <narmstrong@baylibre.com> > + > +description: |+ > + The Everything-Else Power Domains node should be the child of a syscon Everything-Else Power Domains node?? > + node with the required property: > + > + - compatible: Should be the following: > + "amlogic,meson-gx-hhi-sysctrl", "simple-mfd", "syscon" > + > + Refer to the the bindings described in > + Documentation/devicetree/bindings/mfd/syscon.yaml > + > +properties: > + compatible: > + const: amlogic,g12a-mipi-dphy-analog > + > + "#phy-cells": > + const: 0 > + > +required: > + - compatible > + - "#phy-cells" > + > +additionalProperties: false > + > +examples: > + - | > + mpphy: phy { > + compatible = "amlogic,g12a-mipi-dphy-analog"; > + #phy-cells = <0>; > + }; > -- > 2.25.1 > ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] dt-bindings: phy: add Amlogic G12A Analog MIPI D-PHY bindings @ 2020-12-07 22:44 ` Rob Herring 0 siblings, 0 replies; 24+ messages in thread From: Rob Herring @ 2020-12-07 22:44 UTC (permalink / raw) To: Neil Armstrong Cc: devicetree, linux-kernel, kishon, vkoul, linux-amlogic, linux-arm-kernel On Mon, Nov 23, 2020 at 03:51:56PM +0100, Neil Armstrong wrote: > The Amlogic G12A SoCs embeds an Analog MIPI D-PHY to communicate with DSI > panels, this adds the bindings. > > This Analog D-PHY works with a separate Digital MIPI D-PHY. > > Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> > --- > .../phy/amlogic,g12a-mipi-dphy-analog.yaml | 40 +++++++++++++++++++ > 1 file changed, 40 insertions(+) > create mode 100644 Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > > diff --git a/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > new file mode 100644 > index 000000000000..28663552f05b > --- /dev/null > +++ b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > @@ -0,0 +1,40 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: "http://devicetree.org/schemas/phy/amlogic,g12a-mipi-dphy-analog.yaml#" > +$schema: "http://devicetree.org/meta-schemas/core.yaml#" > + > +title: Amlogic G12A MIPI analog PHY > + > +maintainers: > + - Neil Armstrong <narmstrong@baylibre.com> > + > +description: |+ > + The Everything-Else Power Domains node should be the child of a syscon Everything-Else Power Domains node?? > + node with the required property: > + > + - compatible: Should be the following: > + "amlogic,meson-gx-hhi-sysctrl", "simple-mfd", "syscon" > + > + Refer to the the bindings described in > + Documentation/devicetree/bindings/mfd/syscon.yaml > + > +properties: > + compatible: > + const: amlogic,g12a-mipi-dphy-analog > + > + "#phy-cells": > + const: 0 > + > +required: > + - compatible > + - "#phy-cells" > + > +additionalProperties: false > + > +examples: > + - | > + mpphy: phy { > + compatible = "amlogic,g12a-mipi-dphy-analog"; > + #phy-cells = <0>; > + }; > -- > 2.25.1 > _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] dt-bindings: phy: add Amlogic G12A Analog MIPI D-PHY bindings @ 2020-12-07 22:44 ` Rob Herring 0 siblings, 0 replies; 24+ messages in thread From: Rob Herring @ 2020-12-07 22:44 UTC (permalink / raw) To: Neil Armstrong Cc: devicetree, linux-kernel, kishon, vkoul, linux-amlogic, linux-arm-kernel On Mon, Nov 23, 2020 at 03:51:56PM +0100, Neil Armstrong wrote: > The Amlogic G12A SoCs embeds an Analog MIPI D-PHY to communicate with DSI > panels, this adds the bindings. > > This Analog D-PHY works with a separate Digital MIPI D-PHY. > > Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> > --- > .../phy/amlogic,g12a-mipi-dphy-analog.yaml | 40 +++++++++++++++++++ > 1 file changed, 40 insertions(+) > create mode 100644 Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > > diff --git a/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > new file mode 100644 > index 000000000000..28663552f05b > --- /dev/null > +++ b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > @@ -0,0 +1,40 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: "http://devicetree.org/schemas/phy/amlogic,g12a-mipi-dphy-analog.yaml#" > +$schema: "http://devicetree.org/meta-schemas/core.yaml#" > + > +title: Amlogic G12A MIPI analog PHY > + > +maintainers: > + - Neil Armstrong <narmstrong@baylibre.com> > + > +description: |+ > + The Everything-Else Power Domains node should be the child of a syscon Everything-Else Power Domains node?? > + node with the required property: > + > + - compatible: Should be the following: > + "amlogic,meson-gx-hhi-sysctrl", "simple-mfd", "syscon" > + > + Refer to the the bindings described in > + Documentation/devicetree/bindings/mfd/syscon.yaml > + > +properties: > + compatible: > + const: amlogic,g12a-mipi-dphy-analog > + > + "#phy-cells": > + const: 0 > + > +required: > + - compatible > + - "#phy-cells" > + > +additionalProperties: false > + > +examples: > + - | > + mpphy: phy { > + compatible = "amlogic,g12a-mipi-dphy-analog"; > + #phy-cells = <0>; > + }; > -- > 2.25.1 > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] dt-bindings: phy: add Amlogic G12A Analog MIPI D-PHY bindings 2020-12-07 22:44 ` Rob Herring (?) @ 2020-12-08 7:52 ` Neil Armstrong -1 siblings, 0 replies; 24+ messages in thread From: Neil Armstrong @ 2020-12-08 7:52 UTC (permalink / raw) To: Rob Herring Cc: vkoul, kishon, linux-amlogic, linux-arm-kernel, devicetree, linux-kernel On 07/12/2020 23:44, Rob Herring wrote: > On Mon, Nov 23, 2020 at 03:51:56PM +0100, Neil Armstrong wrote: >> The Amlogic G12A SoCs embeds an Analog MIPI D-PHY to communicate with DSI >> panels, this adds the bindings. >> >> This Analog D-PHY works with a separate Digital MIPI D-PHY. >> >> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> >> --- >> .../phy/amlogic,g12a-mipi-dphy-analog.yaml | 40 +++++++++++++++++++ >> 1 file changed, 40 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml >> >> diff --git a/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml >> new file mode 100644 >> index 000000000000..28663552f05b >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml >> @@ -0,0 +1,40 @@ >> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) >> +%YAML 1.2 >> +--- >> +$id: "http://devicetree.org/schemas/phy/amlogic,g12a-mipi-dphy-analog.yaml#" >> +$schema: "http://devicetree.org/meta-schemas/core.yaml#" >> + >> +title: Amlogic G12A MIPI analog PHY >> + >> +maintainers: >> + - Neil Armstrong <narmstrong@baylibre.com> >> + >> +description: |+ >> + The Everything-Else Power Domains node should be the child of a syscon > > Everything-Else Power Domains node?? Indeed, it's a typo > >> + node with the required property: >> + >> + - compatible: Should be the following: >> + "amlogic,meson-gx-hhi-sysctrl", "simple-mfd", "syscon" >> + >> + Refer to the the bindings described in >> + Documentation/devicetree/bindings/mfd/syscon.yaml >> + >> +properties: >> + compatible: >> + const: amlogic,g12a-mipi-dphy-analog >> + >> + "#phy-cells": >> + const: 0 >> + >> +required: >> + - compatible >> + - "#phy-cells" >> + >> +additionalProperties: false >> + >> +examples: >> + - | >> + mpphy: phy { >> + compatible = "amlogic,g12a-mipi-dphy-analog"; >> + #phy-cells = <0>; >> + }; >> -- >> 2.25.1 >> ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] dt-bindings: phy: add Amlogic G12A Analog MIPI D-PHY bindings @ 2020-12-08 7:52 ` Neil Armstrong 0 siblings, 0 replies; 24+ messages in thread From: Neil Armstrong @ 2020-12-08 7:52 UTC (permalink / raw) To: Rob Herring Cc: devicetree, linux-kernel, kishon, vkoul, linux-amlogic, linux-arm-kernel On 07/12/2020 23:44, Rob Herring wrote: > On Mon, Nov 23, 2020 at 03:51:56PM +0100, Neil Armstrong wrote: >> The Amlogic G12A SoCs embeds an Analog MIPI D-PHY to communicate with DSI >> panels, this adds the bindings. >> >> This Analog D-PHY works with a separate Digital MIPI D-PHY. >> >> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> >> --- >> .../phy/amlogic,g12a-mipi-dphy-analog.yaml | 40 +++++++++++++++++++ >> 1 file changed, 40 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml >> >> diff --git a/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml >> new file mode 100644 >> index 000000000000..28663552f05b >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml >> @@ -0,0 +1,40 @@ >> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) >> +%YAML 1.2 >> +--- >> +$id: "http://devicetree.org/schemas/phy/amlogic,g12a-mipi-dphy-analog.yaml#" >> +$schema: "http://devicetree.org/meta-schemas/core.yaml#" >> + >> +title: Amlogic G12A MIPI analog PHY >> + >> +maintainers: >> + - Neil Armstrong <narmstrong@baylibre.com> >> + >> +description: |+ >> + The Everything-Else Power Domains node should be the child of a syscon > > Everything-Else Power Domains node?? Indeed, it's a typo > >> + node with the required property: >> + >> + - compatible: Should be the following: >> + "amlogic,meson-gx-hhi-sysctrl", "simple-mfd", "syscon" >> + >> + Refer to the the bindings described in >> + Documentation/devicetree/bindings/mfd/syscon.yaml >> + >> +properties: >> + compatible: >> + const: amlogic,g12a-mipi-dphy-analog >> + >> + "#phy-cells": >> + const: 0 >> + >> +required: >> + - compatible >> + - "#phy-cells" >> + >> +additionalProperties: false >> + >> +examples: >> + - | >> + mpphy: phy { >> + compatible = "amlogic,g12a-mipi-dphy-analog"; >> + #phy-cells = <0>; >> + }; >> -- >> 2.25.1 >> _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] dt-bindings: phy: add Amlogic G12A Analog MIPI D-PHY bindings @ 2020-12-08 7:52 ` Neil Armstrong 0 siblings, 0 replies; 24+ messages in thread From: Neil Armstrong @ 2020-12-08 7:52 UTC (permalink / raw) To: Rob Herring Cc: devicetree, linux-kernel, kishon, vkoul, linux-amlogic, linux-arm-kernel On 07/12/2020 23:44, Rob Herring wrote: > On Mon, Nov 23, 2020 at 03:51:56PM +0100, Neil Armstrong wrote: >> The Amlogic G12A SoCs embeds an Analog MIPI D-PHY to communicate with DSI >> panels, this adds the bindings. >> >> This Analog D-PHY works with a separate Digital MIPI D-PHY. >> >> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> >> --- >> .../phy/amlogic,g12a-mipi-dphy-analog.yaml | 40 +++++++++++++++++++ >> 1 file changed, 40 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml >> >> diff --git a/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml >> new file mode 100644 >> index 000000000000..28663552f05b >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml >> @@ -0,0 +1,40 @@ >> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) >> +%YAML 1.2 >> +--- >> +$id: "http://devicetree.org/schemas/phy/amlogic,g12a-mipi-dphy-analog.yaml#" >> +$schema: "http://devicetree.org/meta-schemas/core.yaml#" >> + >> +title: Amlogic G12A MIPI analog PHY >> + >> +maintainers: >> + - Neil Armstrong <narmstrong@baylibre.com> >> + >> +description: |+ >> + The Everything-Else Power Domains node should be the child of a syscon > > Everything-Else Power Domains node?? Indeed, it's a typo > >> + node with the required property: >> + >> + - compatible: Should be the following: >> + "amlogic,meson-gx-hhi-sysctrl", "simple-mfd", "syscon" >> + >> + Refer to the the bindings described in >> + Documentation/devicetree/bindings/mfd/syscon.yaml >> + >> +properties: >> + compatible: >> + const: amlogic,g12a-mipi-dphy-analog >> + >> + "#phy-cells": >> + const: 0 >> + >> +required: >> + - compatible >> + - "#phy-cells" >> + >> +additionalProperties: false >> + >> +examples: >> + - | >> + mpphy: phy { >> + compatible = "amlogic,g12a-mipi-dphy-analog"; >> + #phy-cells = <0>; >> + }; >> -- >> 2.25.1 >> _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] dt-bindings: phy: add Amlogic G12A Analog MIPI D-PHY bindings 2020-11-23 14:51 ` Neil Armstrong (?) @ 2020-12-07 22:47 ` Rob Herring -1 siblings, 0 replies; 24+ messages in thread From: Rob Herring @ 2020-12-07 22:47 UTC (permalink / raw) To: Neil Armstrong Cc: vkoul, kishon, linux-amlogic, linux-arm-kernel, devicetree, linux-kernel On Mon, Nov 23, 2020 at 03:51:56PM +0100, Neil Armstrong wrote: > The Amlogic G12A SoCs embeds an Analog MIPI D-PHY to communicate with DSI > panels, this adds the bindings. > > This Analog D-PHY works with a separate Digital MIPI D-PHY. > > Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> > --- > .../phy/amlogic,g12a-mipi-dphy-analog.yaml | 40 +++++++++++++++++++ > 1 file changed, 40 insertions(+) > create mode 100644 Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > > diff --git a/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > new file mode 100644 > index 000000000000..28663552f05b > --- /dev/null > +++ b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > @@ -0,0 +1,40 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: "http://devicetree.org/schemas/phy/amlogic,g12a-mipi-dphy-analog.yaml#" > +$schema: "http://devicetree.org/meta-schemas/core.yaml#" > + > +title: Amlogic G12A MIPI analog PHY > + > +maintainers: > + - Neil Armstrong <narmstrong@baylibre.com> > + > +description: |+ > + The Everything-Else Power Domains node should be the child of a syscon > + node with the required property: > + > + - compatible: Should be the following: > + "amlogic,meson-gx-hhi-sysctrl", "simple-mfd", "syscon" > + > + Refer to the the bindings described in > + Documentation/devicetree/bindings/mfd/syscon.yaml This schema should be referenced from the parent schema and if you have child nodes, you shouldn't be using syscon.yaml. > + > +properties: > + compatible: > + const: amlogic,g12a-mipi-dphy-analog > + > + "#phy-cells": > + const: 0 > + > +required: > + - compatible > + - "#phy-cells" > + > +additionalProperties: false > + > +examples: > + - | > + mpphy: phy { > + compatible = "amlogic,g12a-mipi-dphy-analog"; > + #phy-cells = <0>; Looks like a dedicated subset of registers, add 'reg' even if Linux doesn't use it (currently). > + }; > -- > 2.25.1 > ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] dt-bindings: phy: add Amlogic G12A Analog MIPI D-PHY bindings @ 2020-12-07 22:47 ` Rob Herring 0 siblings, 0 replies; 24+ messages in thread From: Rob Herring @ 2020-12-07 22:47 UTC (permalink / raw) To: Neil Armstrong Cc: devicetree, linux-kernel, kishon, vkoul, linux-amlogic, linux-arm-kernel On Mon, Nov 23, 2020 at 03:51:56PM +0100, Neil Armstrong wrote: > The Amlogic G12A SoCs embeds an Analog MIPI D-PHY to communicate with DSI > panels, this adds the bindings. > > This Analog D-PHY works with a separate Digital MIPI D-PHY. > > Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> > --- > .../phy/amlogic,g12a-mipi-dphy-analog.yaml | 40 +++++++++++++++++++ > 1 file changed, 40 insertions(+) > create mode 100644 Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > > diff --git a/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > new file mode 100644 > index 000000000000..28663552f05b > --- /dev/null > +++ b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > @@ -0,0 +1,40 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: "http://devicetree.org/schemas/phy/amlogic,g12a-mipi-dphy-analog.yaml#" > +$schema: "http://devicetree.org/meta-schemas/core.yaml#" > + > +title: Amlogic G12A MIPI analog PHY > + > +maintainers: > + - Neil Armstrong <narmstrong@baylibre.com> > + > +description: |+ > + The Everything-Else Power Domains node should be the child of a syscon > + node with the required property: > + > + - compatible: Should be the following: > + "amlogic,meson-gx-hhi-sysctrl", "simple-mfd", "syscon" > + > + Refer to the the bindings described in > + Documentation/devicetree/bindings/mfd/syscon.yaml This schema should be referenced from the parent schema and if you have child nodes, you shouldn't be using syscon.yaml. > + > +properties: > + compatible: > + const: amlogic,g12a-mipi-dphy-analog > + > + "#phy-cells": > + const: 0 > + > +required: > + - compatible > + - "#phy-cells" > + > +additionalProperties: false > + > +examples: > + - | > + mpphy: phy { > + compatible = "amlogic,g12a-mipi-dphy-analog"; > + #phy-cells = <0>; Looks like a dedicated subset of registers, add 'reg' even if Linux doesn't use it (currently). > + }; > -- > 2.25.1 > _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] dt-bindings: phy: add Amlogic G12A Analog MIPI D-PHY bindings @ 2020-12-07 22:47 ` Rob Herring 0 siblings, 0 replies; 24+ messages in thread From: Rob Herring @ 2020-12-07 22:47 UTC (permalink / raw) To: Neil Armstrong Cc: devicetree, linux-kernel, kishon, vkoul, linux-amlogic, linux-arm-kernel On Mon, Nov 23, 2020 at 03:51:56PM +0100, Neil Armstrong wrote: > The Amlogic G12A SoCs embeds an Analog MIPI D-PHY to communicate with DSI > panels, this adds the bindings. > > This Analog D-PHY works with a separate Digital MIPI D-PHY. > > Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> > --- > .../phy/amlogic,g12a-mipi-dphy-analog.yaml | 40 +++++++++++++++++++ > 1 file changed, 40 insertions(+) > create mode 100644 Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > > diff --git a/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > new file mode 100644 > index 000000000000..28663552f05b > --- /dev/null > +++ b/Documentation/devicetree/bindings/phy/amlogic,g12a-mipi-dphy-analog.yaml > @@ -0,0 +1,40 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: "http://devicetree.org/schemas/phy/amlogic,g12a-mipi-dphy-analog.yaml#" > +$schema: "http://devicetree.org/meta-schemas/core.yaml#" > + > +title: Amlogic G12A MIPI analog PHY > + > +maintainers: > + - Neil Armstrong <narmstrong@baylibre.com> > + > +description: |+ > + The Everything-Else Power Domains node should be the child of a syscon > + node with the required property: > + > + - compatible: Should be the following: > + "amlogic,meson-gx-hhi-sysctrl", "simple-mfd", "syscon" > + > + Refer to the the bindings described in > + Documentation/devicetree/bindings/mfd/syscon.yaml This schema should be referenced from the parent schema and if you have child nodes, you shouldn't be using syscon.yaml. > + > +properties: > + compatible: > + const: amlogic,g12a-mipi-dphy-analog > + > + "#phy-cells": > + const: 0 > + > +required: > + - compatible > + - "#phy-cells" > + > +additionalProperties: false > + > +examples: > + - | > + mpphy: phy { > + compatible = "amlogic,g12a-mipi-dphy-analog"; > + #phy-cells = <0>; Looks like a dedicated subset of registers, add 'reg' even if Linux doesn't use it (currently). > + }; > -- > 2.25.1 > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 24+ messages in thread
* [PATCH 2/2] phy: amlogic: Add G12A Analog MIPI D-PHY driver 2020-11-23 14:51 ` Neil Armstrong (?) @ 2020-11-23 14:51 ` Neil Armstrong -1 siblings, 0 replies; 24+ messages in thread From: Neil Armstrong @ 2020-11-23 14:51 UTC (permalink / raw) To: vkoul, kishon Cc: linux-amlogic, linux-arm-kernel, devicetree, linux-kernel, Neil Armstrong The Amlogic G12A SoCs embeds an Analog MIPI D-PHY used to communicate with DSI panels. Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> --- drivers/phy/amlogic/Kconfig | 12 ++ drivers/phy/amlogic/Makefile | 1 + .../amlogic/phy-meson-g12a-mipi-dphy-analog.c | 177 ++++++++++++++++++ 3 files changed, 190 insertions(+) create mode 100644 drivers/phy/amlogic/phy-meson-g12a-mipi-dphy-analog.c diff --git a/drivers/phy/amlogic/Kconfig b/drivers/phy/amlogic/Kconfig index db5d0cd757e3..ac898a102bcc 100644 --- a/drivers/phy/amlogic/Kconfig +++ b/drivers/phy/amlogic/Kconfig @@ -49,6 +49,18 @@ config PHY_MESON_G12A_USB3_PCIE in Meson G12A SoCs. If unsure, say N. +config PHY_MESON_G12A_MIPI_DPHY_ANALOG + tristate "Meson G12A MIPI Analog DPHY driver" + default ARCH_MESON + depends on OF && (ARCH_MESON || COMPILE_TEST) + select GENERIC_PHY + select REGMAP_MMIO + select GENERIC_PHY_MIPI_DPHY + help + Enable this to support the Meson MIPI Analog DPHY found in Meson G12A + SoCs. + If unsure, say N. + config PHY_MESON_AXG_PCIE tristate "Meson AXG PCIE PHY driver" default ARCH_MESON diff --git a/drivers/phy/amlogic/Makefile b/drivers/phy/amlogic/Makefile index 8fa07fbd0d92..2eada0a683ca 100644 --- a/drivers/phy/amlogic/Makefile +++ b/drivers/phy/amlogic/Makefile @@ -3,6 +3,7 @@ obj-$(CONFIG_PHY_MESON8B_USB2) += phy-meson8b-usb2.o obj-$(CONFIG_PHY_MESON_GXL_USB2) += phy-meson-gxl-usb2.o obj-$(CONFIG_PHY_MESON_G12A_USB2) += phy-meson-g12a-usb2.o obj-$(CONFIG_PHY_MESON_G12A_USB3_PCIE) += phy-meson-g12a-usb3-pcie.o +obj-$(CONFIG_PHY_MESON_G12A_MIPI_DPHY_ANALOG) += phy-meson-g12a-mipi-dphy-analog.o obj-$(CONFIG_PHY_MESON_AXG_PCIE) += phy-meson-axg-pcie.o obj-$(CONFIG_PHY_MESON_AXG_MIPI_PCIE_ANALOG) += phy-meson-axg-mipi-pcie-analog.o obj-$(CONFIG_PHY_MESON_AXG_MIPI_DPHY) += phy-meson-axg-mipi-dphy.o diff --git a/drivers/phy/amlogic/phy-meson-g12a-mipi-dphy-analog.c b/drivers/phy/amlogic/phy-meson-g12a-mipi-dphy-analog.c new file mode 100644 index 000000000000..6e9d416c0552 --- /dev/null +++ b/drivers/phy/amlogic/phy-meson-g12a-mipi-dphy-analog.c @@ -0,0 +1,177 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Meson G12A MIPI DSI Analog PHY + * + * Copyright (C) 2018 Amlogic, Inc. All rights reserved + * Copyright (C) 2020 BayLibre, SAS + * Author: Neil Armstrong <narmstrong@baylibre.com> + */ +#include <linux/bitfield.h> +#include <linux/bitops.h> +#include <linux/module.h> +#include <linux/phy/phy.h> +#include <linux/regmap.h> +#include <linux/delay.h> +#include <linux/mfd/syscon.h> +#include <linux/platform_device.h> +#include <dt-bindings/phy/phy.h> + +#define HHI_MIPI_CNTL0 0x00 +#define HHI_MIPI_CNTL0_DIF_REF_CTL1 GENMASK(31, 16) +#define HHI_MIPI_CNTL0_DIF_REF_CTL0 GENMASK(15, 0) + +#define HHI_MIPI_CNTL1 0x04 +#define HHI_MIPI_CNTL1_BANDGAP BIT(16) +#define HHI_MIPI_CNTL2_DIF_REF_CTL2 GENMASK(15, 0) + +#define HHI_MIPI_CNTL2 0x08 +#define HHI_MIPI_CNTL2_DIF_TX_CTL1 GENMASK(31, 16) +#define HHI_MIPI_CNTL2_CH_EN GENMASK(15, 11) +#define HHI_MIPI_CNTL2_DIF_TX_CTL0 GENMASK(10, 0) + +#define DSI_LANE_0 BIT(4) +#define DSI_LANE_1 BIT(3) +#define DSI_LANE_CLK BIT(2) +#define DSI_LANE_2 BIT(1) +#define DSI_LANE_3 BIT(0) + +struct phy_g12a_mipi_dphy_analog_priv { + struct phy *phy; + struct regmap *regmap; + struct phy_configure_opts_mipi_dphy config; +}; + +static int phy_g12a_mipi_dphy_analog_configure(struct phy *phy, + union phy_configure_opts *opts) +{ + struct phy_g12a_mipi_dphy_analog_priv *priv = phy_get_drvdata(phy); + int ret; + + ret = phy_mipi_dphy_config_validate(&opts->mipi_dphy); + if (ret) + return ret; + + memcpy(&priv->config, opts, sizeof(priv->config)); + + return 0; +} + +static int phy_g12a_mipi_dphy_analog_power_on(struct phy *phy) +{ + struct phy_g12a_mipi_dphy_analog_priv *priv = phy_get_drvdata(phy); + unsigned int reg; + + regmap_write(priv->regmap, HHI_MIPI_CNTL0, + FIELD_PREP(HHI_MIPI_CNTL0_DIF_REF_CTL0, 0x8) | + FIELD_PREP(HHI_MIPI_CNTL0_DIF_REF_CTL1, 0xa487)); + + regmap_write(priv->regmap, HHI_MIPI_CNTL1, + FIELD_PREP(HHI_MIPI_CNTL2_DIF_REF_CTL2, 0x2e) | + HHI_MIPI_CNTL1_BANDGAP); + + regmap_write(priv->regmap, HHI_MIPI_CNTL2, + FIELD_PREP(HHI_MIPI_CNTL2_DIF_TX_CTL0, 0x459) | + FIELD_PREP(HHI_MIPI_CNTL2_DIF_TX_CTL1, 0x2680)); + + reg = DSI_LANE_CLK; + switch (priv->config.lanes) { + case 4: + reg |= DSI_LANE_3; + fallthrough; + case 3: + reg |= DSI_LANE_2; + fallthrough; + case 2: + reg |= DSI_LANE_1; + fallthrough; + case 1: + reg |= DSI_LANE_0; + break; + default: + reg = 0; + } + + regmap_update_bits(priv->regmap, HHI_MIPI_CNTL2, + HHI_MIPI_CNTL2_CH_EN, + FIELD_PREP(HHI_MIPI_CNTL2_CH_EN, reg)); + + return 0; +} + +static int phy_g12a_mipi_dphy_analog_power_off(struct phy *phy) +{ + struct phy_g12a_mipi_dphy_analog_priv *priv = phy_get_drvdata(phy); + + regmap_write(priv->regmap, HHI_MIPI_CNTL0, 0); + regmap_write(priv->regmap, HHI_MIPI_CNTL1, 0); + regmap_write(priv->regmap, HHI_MIPI_CNTL2, 0); + + return 0; +} + +static const struct phy_ops phy_g12a_mipi_dphy_analog_ops = { + .configure = phy_g12a_mipi_dphy_analog_configure, + .power_on = phy_g12a_mipi_dphy_analog_power_on, + .power_off = phy_g12a_mipi_dphy_analog_power_off, + .owner = THIS_MODULE, +}; + +static int phy_g12a_mipi_dphy_analog_probe(struct platform_device *pdev) +{ + struct phy_provider *phy; + struct device *dev = &pdev->dev; + struct phy_g12a_mipi_dphy_analog_priv *priv; + struct device_node *np = dev->of_node; + struct regmap *map; + int ret; + + priv = devm_kmalloc(dev, sizeof(*priv), GFP_KERNEL); + if (!priv) + return -ENOMEM; + + /* Get the hhi system controller node */ + map = syscon_node_to_regmap(of_get_parent(dev->of_node)); + if (IS_ERR(map)) { + dev_err(dev, + "failed to get HHI regmap\n"); + return PTR_ERR(map); + } + + priv->regmap = map; + + priv->phy = devm_phy_create(dev, np, &phy_g12a_mipi_dphy_analog_ops); + if (IS_ERR(priv->phy)) { + ret = PTR_ERR(priv->phy); + if (ret != -EPROBE_DEFER) + dev_err(dev, "failed to create PHY\n"); + return ret; + } + + phy_set_drvdata(priv->phy, priv); + dev_set_drvdata(dev, priv); + + phy = devm_of_phy_provider_register(dev, of_phy_simple_xlate); + + return PTR_ERR_OR_ZERO(phy); +} + +static const struct of_device_id phy_g12a_mipi_dphy_analog_of_match[] = { + { + .compatible = "amlogic,g12a-mipi-dphy-analog", + }, + { }, +}; +MODULE_DEVICE_TABLE(of, phy_g12a_mipi_dphy_analog_of_match); + +static struct platform_driver phy_g12a_mipi_dphy_analog_driver = { + .probe = phy_g12a_mipi_dphy_analog_probe, + .driver = { + .name = "phy-meson-g12a-mipi-dphy-analog", + .of_match_table = phy_g12a_mipi_dphy_analog_of_match, + }, +}; +module_platform_driver(phy_g12a_mipi_dphy_analog_driver); + +MODULE_AUTHOR("Neil Armstrong <narmstrong@baylibre.com>"); +MODULE_DESCRIPTION("Meson G12A MIPI Analog D-PHY driver"); +MODULE_LICENSE("GPL v2"); -- 2.25.1 ^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 2/2] phy: amlogic: Add G12A Analog MIPI D-PHY driver @ 2020-11-23 14:51 ` Neil Armstrong 0 siblings, 0 replies; 24+ messages in thread From: Neil Armstrong @ 2020-11-23 14:51 UTC (permalink / raw) To: vkoul, kishon Cc: linux-amlogic, Neil Armstrong, linux-kernel, linux-arm-kernel, devicetree The Amlogic G12A SoCs embeds an Analog MIPI D-PHY used to communicate with DSI panels. Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> --- drivers/phy/amlogic/Kconfig | 12 ++ drivers/phy/amlogic/Makefile | 1 + .../amlogic/phy-meson-g12a-mipi-dphy-analog.c | 177 ++++++++++++++++++ 3 files changed, 190 insertions(+) create mode 100644 drivers/phy/amlogic/phy-meson-g12a-mipi-dphy-analog.c diff --git a/drivers/phy/amlogic/Kconfig b/drivers/phy/amlogic/Kconfig index db5d0cd757e3..ac898a102bcc 100644 --- a/drivers/phy/amlogic/Kconfig +++ b/drivers/phy/amlogic/Kconfig @@ -49,6 +49,18 @@ config PHY_MESON_G12A_USB3_PCIE in Meson G12A SoCs. If unsure, say N. +config PHY_MESON_G12A_MIPI_DPHY_ANALOG + tristate "Meson G12A MIPI Analog DPHY driver" + default ARCH_MESON + depends on OF && (ARCH_MESON || COMPILE_TEST) + select GENERIC_PHY + select REGMAP_MMIO + select GENERIC_PHY_MIPI_DPHY + help + Enable this to support the Meson MIPI Analog DPHY found in Meson G12A + SoCs. + If unsure, say N. + config PHY_MESON_AXG_PCIE tristate "Meson AXG PCIE PHY driver" default ARCH_MESON diff --git a/drivers/phy/amlogic/Makefile b/drivers/phy/amlogic/Makefile index 8fa07fbd0d92..2eada0a683ca 100644 --- a/drivers/phy/amlogic/Makefile +++ b/drivers/phy/amlogic/Makefile @@ -3,6 +3,7 @@ obj-$(CONFIG_PHY_MESON8B_USB2) += phy-meson8b-usb2.o obj-$(CONFIG_PHY_MESON_GXL_USB2) += phy-meson-gxl-usb2.o obj-$(CONFIG_PHY_MESON_G12A_USB2) += phy-meson-g12a-usb2.o obj-$(CONFIG_PHY_MESON_G12A_USB3_PCIE) += phy-meson-g12a-usb3-pcie.o +obj-$(CONFIG_PHY_MESON_G12A_MIPI_DPHY_ANALOG) += phy-meson-g12a-mipi-dphy-analog.o obj-$(CONFIG_PHY_MESON_AXG_PCIE) += phy-meson-axg-pcie.o obj-$(CONFIG_PHY_MESON_AXG_MIPI_PCIE_ANALOG) += phy-meson-axg-mipi-pcie-analog.o obj-$(CONFIG_PHY_MESON_AXG_MIPI_DPHY) += phy-meson-axg-mipi-dphy.o diff --git a/drivers/phy/amlogic/phy-meson-g12a-mipi-dphy-analog.c b/drivers/phy/amlogic/phy-meson-g12a-mipi-dphy-analog.c new file mode 100644 index 000000000000..6e9d416c0552 --- /dev/null +++ b/drivers/phy/amlogic/phy-meson-g12a-mipi-dphy-analog.c @@ -0,0 +1,177 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Meson G12A MIPI DSI Analog PHY + * + * Copyright (C) 2018 Amlogic, Inc. All rights reserved + * Copyright (C) 2020 BayLibre, SAS + * Author: Neil Armstrong <narmstrong@baylibre.com> + */ +#include <linux/bitfield.h> +#include <linux/bitops.h> +#include <linux/module.h> +#include <linux/phy/phy.h> +#include <linux/regmap.h> +#include <linux/delay.h> +#include <linux/mfd/syscon.h> +#include <linux/platform_device.h> +#include <dt-bindings/phy/phy.h> + +#define HHI_MIPI_CNTL0 0x00 +#define HHI_MIPI_CNTL0_DIF_REF_CTL1 GENMASK(31, 16) +#define HHI_MIPI_CNTL0_DIF_REF_CTL0 GENMASK(15, 0) + +#define HHI_MIPI_CNTL1 0x04 +#define HHI_MIPI_CNTL1_BANDGAP BIT(16) +#define HHI_MIPI_CNTL2_DIF_REF_CTL2 GENMASK(15, 0) + +#define HHI_MIPI_CNTL2 0x08 +#define HHI_MIPI_CNTL2_DIF_TX_CTL1 GENMASK(31, 16) +#define HHI_MIPI_CNTL2_CH_EN GENMASK(15, 11) +#define HHI_MIPI_CNTL2_DIF_TX_CTL0 GENMASK(10, 0) + +#define DSI_LANE_0 BIT(4) +#define DSI_LANE_1 BIT(3) +#define DSI_LANE_CLK BIT(2) +#define DSI_LANE_2 BIT(1) +#define DSI_LANE_3 BIT(0) + +struct phy_g12a_mipi_dphy_analog_priv { + struct phy *phy; + struct regmap *regmap; + struct phy_configure_opts_mipi_dphy config; +}; + +static int phy_g12a_mipi_dphy_analog_configure(struct phy *phy, + union phy_configure_opts *opts) +{ + struct phy_g12a_mipi_dphy_analog_priv *priv = phy_get_drvdata(phy); + int ret; + + ret = phy_mipi_dphy_config_validate(&opts->mipi_dphy); + if (ret) + return ret; + + memcpy(&priv->config, opts, sizeof(priv->config)); + + return 0; +} + +static int phy_g12a_mipi_dphy_analog_power_on(struct phy *phy) +{ + struct phy_g12a_mipi_dphy_analog_priv *priv = phy_get_drvdata(phy); + unsigned int reg; + + regmap_write(priv->regmap, HHI_MIPI_CNTL0, + FIELD_PREP(HHI_MIPI_CNTL0_DIF_REF_CTL0, 0x8) | + FIELD_PREP(HHI_MIPI_CNTL0_DIF_REF_CTL1, 0xa487)); + + regmap_write(priv->regmap, HHI_MIPI_CNTL1, + FIELD_PREP(HHI_MIPI_CNTL2_DIF_REF_CTL2, 0x2e) | + HHI_MIPI_CNTL1_BANDGAP); + + regmap_write(priv->regmap, HHI_MIPI_CNTL2, + FIELD_PREP(HHI_MIPI_CNTL2_DIF_TX_CTL0, 0x459) | + FIELD_PREP(HHI_MIPI_CNTL2_DIF_TX_CTL1, 0x2680)); + + reg = DSI_LANE_CLK; + switch (priv->config.lanes) { + case 4: + reg |= DSI_LANE_3; + fallthrough; + case 3: + reg |= DSI_LANE_2; + fallthrough; + case 2: + reg |= DSI_LANE_1; + fallthrough; + case 1: + reg |= DSI_LANE_0; + break; + default: + reg = 0; + } + + regmap_update_bits(priv->regmap, HHI_MIPI_CNTL2, + HHI_MIPI_CNTL2_CH_EN, + FIELD_PREP(HHI_MIPI_CNTL2_CH_EN, reg)); + + return 0; +} + +static int phy_g12a_mipi_dphy_analog_power_off(struct phy *phy) +{ + struct phy_g12a_mipi_dphy_analog_priv *priv = phy_get_drvdata(phy); + + regmap_write(priv->regmap, HHI_MIPI_CNTL0, 0); + regmap_write(priv->regmap, HHI_MIPI_CNTL1, 0); + regmap_write(priv->regmap, HHI_MIPI_CNTL2, 0); + + return 0; +} + +static const struct phy_ops phy_g12a_mipi_dphy_analog_ops = { + .configure = phy_g12a_mipi_dphy_analog_configure, + .power_on = phy_g12a_mipi_dphy_analog_power_on, + .power_off = phy_g12a_mipi_dphy_analog_power_off, + .owner = THIS_MODULE, +}; + +static int phy_g12a_mipi_dphy_analog_probe(struct platform_device *pdev) +{ + struct phy_provider *phy; + struct device *dev = &pdev->dev; + struct phy_g12a_mipi_dphy_analog_priv *priv; + struct device_node *np = dev->of_node; + struct regmap *map; + int ret; + + priv = devm_kmalloc(dev, sizeof(*priv), GFP_KERNEL); + if (!priv) + return -ENOMEM; + + /* Get the hhi system controller node */ + map = syscon_node_to_regmap(of_get_parent(dev->of_node)); + if (IS_ERR(map)) { + dev_err(dev, + "failed to get HHI regmap\n"); + return PTR_ERR(map); + } + + priv->regmap = map; + + priv->phy = devm_phy_create(dev, np, &phy_g12a_mipi_dphy_analog_ops); + if (IS_ERR(priv->phy)) { + ret = PTR_ERR(priv->phy); + if (ret != -EPROBE_DEFER) + dev_err(dev, "failed to create PHY\n"); + return ret; + } + + phy_set_drvdata(priv->phy, priv); + dev_set_drvdata(dev, priv); + + phy = devm_of_phy_provider_register(dev, of_phy_simple_xlate); + + return PTR_ERR_OR_ZERO(phy); +} + +static const struct of_device_id phy_g12a_mipi_dphy_analog_of_match[] = { + { + .compatible = "amlogic,g12a-mipi-dphy-analog", + }, + { }, +}; +MODULE_DEVICE_TABLE(of, phy_g12a_mipi_dphy_analog_of_match); + +static struct platform_driver phy_g12a_mipi_dphy_analog_driver = { + .probe = phy_g12a_mipi_dphy_analog_probe, + .driver = { + .name = "phy-meson-g12a-mipi-dphy-analog", + .of_match_table = phy_g12a_mipi_dphy_analog_of_match, + }, +}; +module_platform_driver(phy_g12a_mipi_dphy_analog_driver); + +MODULE_AUTHOR("Neil Armstrong <narmstrong@baylibre.com>"); +MODULE_DESCRIPTION("Meson G12A MIPI Analog D-PHY driver"); +MODULE_LICENSE("GPL v2"); -- 2.25.1 _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic ^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 2/2] phy: amlogic: Add G12A Analog MIPI D-PHY driver @ 2020-11-23 14:51 ` Neil Armstrong 0 siblings, 0 replies; 24+ messages in thread From: Neil Armstrong @ 2020-11-23 14:51 UTC (permalink / raw) To: vkoul, kishon Cc: linux-amlogic, Neil Armstrong, linux-kernel, linux-arm-kernel, devicetree The Amlogic G12A SoCs embeds an Analog MIPI D-PHY used to communicate with DSI panels. Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> --- drivers/phy/amlogic/Kconfig | 12 ++ drivers/phy/amlogic/Makefile | 1 + .../amlogic/phy-meson-g12a-mipi-dphy-analog.c | 177 ++++++++++++++++++ 3 files changed, 190 insertions(+) create mode 100644 drivers/phy/amlogic/phy-meson-g12a-mipi-dphy-analog.c diff --git a/drivers/phy/amlogic/Kconfig b/drivers/phy/amlogic/Kconfig index db5d0cd757e3..ac898a102bcc 100644 --- a/drivers/phy/amlogic/Kconfig +++ b/drivers/phy/amlogic/Kconfig @@ -49,6 +49,18 @@ config PHY_MESON_G12A_USB3_PCIE in Meson G12A SoCs. If unsure, say N. +config PHY_MESON_G12A_MIPI_DPHY_ANALOG + tristate "Meson G12A MIPI Analog DPHY driver" + default ARCH_MESON + depends on OF && (ARCH_MESON || COMPILE_TEST) + select GENERIC_PHY + select REGMAP_MMIO + select GENERIC_PHY_MIPI_DPHY + help + Enable this to support the Meson MIPI Analog DPHY found in Meson G12A + SoCs. + If unsure, say N. + config PHY_MESON_AXG_PCIE tristate "Meson AXG PCIE PHY driver" default ARCH_MESON diff --git a/drivers/phy/amlogic/Makefile b/drivers/phy/amlogic/Makefile index 8fa07fbd0d92..2eada0a683ca 100644 --- a/drivers/phy/amlogic/Makefile +++ b/drivers/phy/amlogic/Makefile @@ -3,6 +3,7 @@ obj-$(CONFIG_PHY_MESON8B_USB2) += phy-meson8b-usb2.o obj-$(CONFIG_PHY_MESON_GXL_USB2) += phy-meson-gxl-usb2.o obj-$(CONFIG_PHY_MESON_G12A_USB2) += phy-meson-g12a-usb2.o obj-$(CONFIG_PHY_MESON_G12A_USB3_PCIE) += phy-meson-g12a-usb3-pcie.o +obj-$(CONFIG_PHY_MESON_G12A_MIPI_DPHY_ANALOG) += phy-meson-g12a-mipi-dphy-analog.o obj-$(CONFIG_PHY_MESON_AXG_PCIE) += phy-meson-axg-pcie.o obj-$(CONFIG_PHY_MESON_AXG_MIPI_PCIE_ANALOG) += phy-meson-axg-mipi-pcie-analog.o obj-$(CONFIG_PHY_MESON_AXG_MIPI_DPHY) += phy-meson-axg-mipi-dphy.o diff --git a/drivers/phy/amlogic/phy-meson-g12a-mipi-dphy-analog.c b/drivers/phy/amlogic/phy-meson-g12a-mipi-dphy-analog.c new file mode 100644 index 000000000000..6e9d416c0552 --- /dev/null +++ b/drivers/phy/amlogic/phy-meson-g12a-mipi-dphy-analog.c @@ -0,0 +1,177 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Meson G12A MIPI DSI Analog PHY + * + * Copyright (C) 2018 Amlogic, Inc. All rights reserved + * Copyright (C) 2020 BayLibre, SAS + * Author: Neil Armstrong <narmstrong@baylibre.com> + */ +#include <linux/bitfield.h> +#include <linux/bitops.h> +#include <linux/module.h> +#include <linux/phy/phy.h> +#include <linux/regmap.h> +#include <linux/delay.h> +#include <linux/mfd/syscon.h> +#include <linux/platform_device.h> +#include <dt-bindings/phy/phy.h> + +#define HHI_MIPI_CNTL0 0x00 +#define HHI_MIPI_CNTL0_DIF_REF_CTL1 GENMASK(31, 16) +#define HHI_MIPI_CNTL0_DIF_REF_CTL0 GENMASK(15, 0) + +#define HHI_MIPI_CNTL1 0x04 +#define HHI_MIPI_CNTL1_BANDGAP BIT(16) +#define HHI_MIPI_CNTL2_DIF_REF_CTL2 GENMASK(15, 0) + +#define HHI_MIPI_CNTL2 0x08 +#define HHI_MIPI_CNTL2_DIF_TX_CTL1 GENMASK(31, 16) +#define HHI_MIPI_CNTL2_CH_EN GENMASK(15, 11) +#define HHI_MIPI_CNTL2_DIF_TX_CTL0 GENMASK(10, 0) + +#define DSI_LANE_0 BIT(4) +#define DSI_LANE_1 BIT(3) +#define DSI_LANE_CLK BIT(2) +#define DSI_LANE_2 BIT(1) +#define DSI_LANE_3 BIT(0) + +struct phy_g12a_mipi_dphy_analog_priv { + struct phy *phy; + struct regmap *regmap; + struct phy_configure_opts_mipi_dphy config; +}; + +static int phy_g12a_mipi_dphy_analog_configure(struct phy *phy, + union phy_configure_opts *opts) +{ + struct phy_g12a_mipi_dphy_analog_priv *priv = phy_get_drvdata(phy); + int ret; + + ret = phy_mipi_dphy_config_validate(&opts->mipi_dphy); + if (ret) + return ret; + + memcpy(&priv->config, opts, sizeof(priv->config)); + + return 0; +} + +static int phy_g12a_mipi_dphy_analog_power_on(struct phy *phy) +{ + struct phy_g12a_mipi_dphy_analog_priv *priv = phy_get_drvdata(phy); + unsigned int reg; + + regmap_write(priv->regmap, HHI_MIPI_CNTL0, + FIELD_PREP(HHI_MIPI_CNTL0_DIF_REF_CTL0, 0x8) | + FIELD_PREP(HHI_MIPI_CNTL0_DIF_REF_CTL1, 0xa487)); + + regmap_write(priv->regmap, HHI_MIPI_CNTL1, + FIELD_PREP(HHI_MIPI_CNTL2_DIF_REF_CTL2, 0x2e) | + HHI_MIPI_CNTL1_BANDGAP); + + regmap_write(priv->regmap, HHI_MIPI_CNTL2, + FIELD_PREP(HHI_MIPI_CNTL2_DIF_TX_CTL0, 0x459) | + FIELD_PREP(HHI_MIPI_CNTL2_DIF_TX_CTL1, 0x2680)); + + reg = DSI_LANE_CLK; + switch (priv->config.lanes) { + case 4: + reg |= DSI_LANE_3; + fallthrough; + case 3: + reg |= DSI_LANE_2; + fallthrough; + case 2: + reg |= DSI_LANE_1; + fallthrough; + case 1: + reg |= DSI_LANE_0; + break; + default: + reg = 0; + } + + regmap_update_bits(priv->regmap, HHI_MIPI_CNTL2, + HHI_MIPI_CNTL2_CH_EN, + FIELD_PREP(HHI_MIPI_CNTL2_CH_EN, reg)); + + return 0; +} + +static int phy_g12a_mipi_dphy_analog_power_off(struct phy *phy) +{ + struct phy_g12a_mipi_dphy_analog_priv *priv = phy_get_drvdata(phy); + + regmap_write(priv->regmap, HHI_MIPI_CNTL0, 0); + regmap_write(priv->regmap, HHI_MIPI_CNTL1, 0); + regmap_write(priv->regmap, HHI_MIPI_CNTL2, 0); + + return 0; +} + +static const struct phy_ops phy_g12a_mipi_dphy_analog_ops = { + .configure = phy_g12a_mipi_dphy_analog_configure, + .power_on = phy_g12a_mipi_dphy_analog_power_on, + .power_off = phy_g12a_mipi_dphy_analog_power_off, + .owner = THIS_MODULE, +}; + +static int phy_g12a_mipi_dphy_analog_probe(struct platform_device *pdev) +{ + struct phy_provider *phy; + struct device *dev = &pdev->dev; + struct phy_g12a_mipi_dphy_analog_priv *priv; + struct device_node *np = dev->of_node; + struct regmap *map; + int ret; + + priv = devm_kmalloc(dev, sizeof(*priv), GFP_KERNEL); + if (!priv) + return -ENOMEM; + + /* Get the hhi system controller node */ + map = syscon_node_to_regmap(of_get_parent(dev->of_node)); + if (IS_ERR(map)) { + dev_err(dev, + "failed to get HHI regmap\n"); + return PTR_ERR(map); + } + + priv->regmap = map; + + priv->phy = devm_phy_create(dev, np, &phy_g12a_mipi_dphy_analog_ops); + if (IS_ERR(priv->phy)) { + ret = PTR_ERR(priv->phy); + if (ret != -EPROBE_DEFER) + dev_err(dev, "failed to create PHY\n"); + return ret; + } + + phy_set_drvdata(priv->phy, priv); + dev_set_drvdata(dev, priv); + + phy = devm_of_phy_provider_register(dev, of_phy_simple_xlate); + + return PTR_ERR_OR_ZERO(phy); +} + +static const struct of_device_id phy_g12a_mipi_dphy_analog_of_match[] = { + { + .compatible = "amlogic,g12a-mipi-dphy-analog", + }, + { }, +}; +MODULE_DEVICE_TABLE(of, phy_g12a_mipi_dphy_analog_of_match); + +static struct platform_driver phy_g12a_mipi_dphy_analog_driver = { + .probe = phy_g12a_mipi_dphy_analog_probe, + .driver = { + .name = "phy-meson-g12a-mipi-dphy-analog", + .of_match_table = phy_g12a_mipi_dphy_analog_of_match, + }, +}; +module_platform_driver(phy_g12a_mipi_dphy_analog_driver); + +MODULE_AUTHOR("Neil Armstrong <narmstrong@baylibre.com>"); +MODULE_DESCRIPTION("Meson G12A MIPI Analog D-PHY driver"); +MODULE_LICENSE("GPL v2"); -- 2.25.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 24+ messages in thread
end of thread, other threads:[~2020-12-08 7:54 UTC | newest] Thread overview: 24+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-11-23 14:51 [PATCH 0/2] phy: amlogic: Add support for the G12A Analog MIPI D-PHY Neil Armstrong 2020-11-23 14:51 ` Neil Armstrong 2020-11-23 14:51 ` Neil Armstrong 2020-11-23 14:51 ` [PATCH 1/2] dt-bindings: phy: add Amlogic G12A Analog MIPI D-PHY bindings Neil Armstrong 2020-11-23 14:51 ` Neil Armstrong 2020-11-23 14:51 ` Neil Armstrong 2020-12-05 8:04 ` Vinod Koul 2020-12-05 8:04 ` Vinod Koul 2020-12-05 8:04 ` Vinod Koul 2020-12-07 9:50 ` Neil Armstrong 2020-12-07 9:50 ` Neil Armstrong 2020-12-07 9:50 ` Neil Armstrong 2020-12-07 22:44 ` Rob Herring 2020-12-07 22:44 ` Rob Herring 2020-12-07 22:44 ` Rob Herring 2020-12-08 7:52 ` Neil Armstrong 2020-12-08 7:52 ` Neil Armstrong 2020-12-08 7:52 ` Neil Armstrong 2020-12-07 22:47 ` Rob Herring 2020-12-07 22:47 ` Rob Herring 2020-12-07 22:47 ` Rob Herring 2020-11-23 14:51 ` [PATCH 2/2] phy: amlogic: Add G12A Analog MIPI D-PHY driver Neil Armstrong 2020-11-23 14:51 ` Neil Armstrong 2020-11-23 14:51 ` Neil Armstrong
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.