| Message ID | 20231109-amlogic-v6-4-upstream-dsi-ccf-vim3-v8-2-81e4aeeda193@linaro.org |
|---|---|
| State | New |
| Headers | show |
| Series | drm/meson: add support for MIPI DSI Display | expand |
On 10/11/2023 21:12, Rob Herring wrote: > On Thu, Nov 09, 2023 at 10:00:03AM +0100, Neil Armstrong wrote: >> Add a thirst example covering the meson-axg-hhi-sysctrl variant and more >> importantly the phy subnode. >> >> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org> >> --- >> .../soc/amlogic/amlogic,meson-gx-hhi-sysctrl.yaml | 41 ++++++++++++++++++++++ >> 1 file changed, 41 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/soc/amlogic/amlogic,meson-gx-hhi-sysctrl.yaml b/Documentation/devicetree/bindings/soc/amlogic/amlogic,meson-gx-hhi-sysctrl.yaml >> index 16977e4e4357..2edf4ccea845 100644 >> --- a/Documentation/devicetree/bindings/soc/amlogic/amlogic,meson-gx-hhi-sysctrl.yaml >> +++ b/Documentation/devicetree/bindings/soc/amlogic/amlogic,meson-gx-hhi-sysctrl.yaml >> @@ -158,3 +158,44 @@ examples: >> }; >> }; >> }; >> + > > New example should be separate starting with a '-|'. > >> + bus@ff63c000 { >> + compatible = "simple-bus"; >> + reg = <0xff63c000 0x1c00>; >> + #address-cells = <1>; >> + #size-cells = <1>; >> + ranges = <0x0 0xff63c000 0x1c00>; > > Why do you need all this? 1 cell is the default for examples. Bad copy-pasta from previous examples, I'll fix all that and separate examples with '-|' > >> + >> + system-controller@0 { >> + compatible = "amlogic,meson-axg-hhi-sysctrl", "simple-mfd", "syscon"; >> + reg = <0 0x400>; >> + >> + clock-controller { >> + compatible = "amlogic,axg-clkc"; >> + #clock-cells = <1>; >> + clocks = <&xtal>; >> + clock-names = "xtal"; >> + }; >> + >> + power-controller { >> + compatible = "amlogic,meson-axg-pwrc"; >> + #power-domain-cells = <1>; >> + amlogic,ao-sysctrl = <&sysctrl_AO>; >> + >> + resets = <&reset_viu>, >> + <&reset_venc>, >> + <&reset_vcbus>, >> + <&reset_vencl>, >> + <&reset_vid_lock>; >> + reset-names = "viu", "venc", "vcbus", "vencl", "vid_lock"; >> + clocks = <&clk_vpu>, <&clk_vapb>; >> + clock-names = "vpu", "vapb"; >> + }; >> + >> + phy { >> + compatible = "amlogic,axg-mipi-pcie-analog-phy"; >> + #phy-cells = <0>; >> + status = "disabled"; > > Examples should not be disabled. Err, thx I 'll fix this > >> + }; >> + }; >> + }; >> >> -- >> 2.34.1 >>
diff --git a/Documentation/devicetree/bindings/soc/amlogic/amlogic,meson-gx-hhi-sysctrl.yaml b/Documentation/devicetree/bindings/soc/amlogic/amlogic,meson-gx-hhi-sysctrl.yaml index 16977e4e4357..2edf4ccea845 100644 --- a/Documentation/devicetree/bindings/soc/amlogic/amlogic,meson-gx-hhi-sysctrl.yaml +++ b/Documentation/devicetree/bindings/soc/amlogic/amlogic,meson-gx-hhi-sysctrl.yaml @@ -158,3 +158,44 @@ examples: }; }; }; + + bus@ff63c000 { + compatible = "simple-bus"; + reg = <0xff63c000 0x1c00>; + #address-cells = <1>; + #size-cells = <1>; + ranges = <0x0 0xff63c000 0x1c00>; + + system-controller@0 { + compatible = "amlogic,meson-axg-hhi-sysctrl", "simple-mfd", "syscon"; + reg = <0 0x400>; + + clock-controller { + compatible = "amlogic,axg-clkc"; + #clock-cells = <1>; + clocks = <&xtal>; + clock-names = "xtal"; + }; + + power-controller { + compatible = "amlogic,meson-axg-pwrc"; + #power-domain-cells = <1>; + amlogic,ao-sysctrl = <&sysctrl_AO>; + + resets = <&reset_viu>, + <&reset_venc>, + <&reset_vcbus>, + <&reset_vencl>, + <&reset_vid_lock>; + reset-names = "viu", "venc", "vcbus", "vencl", "vid_lock"; + clocks = <&clk_vpu>, <&clk_vapb>; + clock-names = "vpu", "vapb"; + }; + + phy { + compatible = "amlogic,axg-mipi-pcie-analog-phy"; + #phy-cells = <0>; + status = "disabled"; + }; + }; + };
Add a thirst example covering the meson-axg-hhi-sysctrl variant and more importantly the phy subnode. Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org> --- .../soc/amlogic/amlogic,meson-gx-hhi-sysctrl.yaml | 41 ++++++++++++++++++++++ 1 file changed, 41 insertions(+)