| Message ID | 20210630164623.3.I7bd7d9a8da5e2894711b700a1127e6902a2b2f1d@changeid |
|---|---|
| State | Superseded |
| Headers | show |
| Series | Add support to the mmsys driver to be a reset controller | expand |
Hi Chun-Kuang, Thank you for the review. On 1/7/21 0:48, Chun-Kuang Hu wrote: > HI, Enric: > > Enric Balletbo i Serra <enric.balletbo@collabora.com> 於 2021年6月30日 週三 下午10:47寫道： >> >> Reset the DSI hardware is needed to prevent different settings between >> the bootloader and the kernel. >> >> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com> >> --- >> >> arch/arm64/boot/dts/mediatek/mt8173.dtsi | 2 ++ >> include/dt-bindings/reset/mt8173-resets.h | 2 ++ >> 2 files changed, 4 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/mediatek/mt8173.dtsi b/arch/arm64/boot/dts/mediatek/mt8173.dtsi >> index e5596fe01a1d..36c3998eb7f1 100644 >> --- a/arch/arm64/boot/dts/mediatek/mt8173.dtsi >> +++ b/arch/arm64/boot/dts/mediatek/mt8173.dtsi >> @@ -1036,6 +1036,7 @@ mmsys: syscon@14000000 { >> assigned-clocks = <&topckgen CLK_TOP_MM_SEL>; >> assigned-clock-rates = <400000000>; >> #clock-cells = <1>; >> + #reset-cells = <1>; >> mboxes = <&gce 0 CMDQ_THR_PRIO_HIGHEST>, >> <&gce 1 CMDQ_THR_PRIO_HIGHEST>; >> mediatek,gce-client-reg = <&gce SUBSYS_1400XXXX 0 0x1000>; >> @@ -1262,6 +1263,7 @@ dsi0: dsi@1401b000 { >> <&mmsys CLK_MM_DSI0_DIGITAL>, >> <&mipi_tx0>; >> clock-names = "engine", "digital", "hs"; >> + resets = <&mmsys MT8173_MMSYS_SW0_RST_B_DISP_DSI0>; > > Add this in binding document. It would be good if the binding document > is in yaml format. > Ack, will add this optional property in the mediatek,dsi.txt binding. I can try to translate it to yaml but IMO would be better if someone that really knows the hardware and has access to the datasheet could do it. From the current .txt file I can deduce most of the things, but still I can lack some kind of information, i.e I know that there are three clocks, but don't know if all three are needed (from .txt yes, are all needed but not sure), also I know that the names of the clocks are "engine", "digital", and "hs", but I don't know the description of these clocks. IMHO would be nice if someone from Mediatek can take care of all the binding migration to yaml. Cheers, Enric > Regards, > Chun-Kuang. > >> phys = <&mipi_tx0>; >> phy-names = "dphy"; >> status = "disabled"; >> diff --git a/include/dt-bindings/reset/mt8173-resets.h b/include/dt-bindings/reset/mt8173-resets.h >> index ba8636eda5ae..6a60c7cecc4c 100644 >> --- a/include/dt-bindings/reset/mt8173-resets.h >> +++ b/include/dt-bindings/reset/mt8173-resets.h >> @@ -27,6 +27,8 @@ >> #define MT8173_INFRA_GCE_FAXI_RST 40 >> #define MT8173_INFRA_MMIOMMURST 47 >> >> +/* MMSYS resets */ >> +#define MT8173_MMSYS_SW0_RST_B_DISP_DSI0 25 >> >> /* PERICFG resets */ >> #define MT8173_PERI_UART0_SW_RST 0 >> -- >> 2.30.2 >>
diff --git a/arch/arm64/boot/dts/mediatek/mt8173.dtsi b/arch/arm64/boot/dts/mediatek/mt8173.dtsi index e5596fe01a1d..36c3998eb7f1 100644 --- a/arch/arm64/boot/dts/mediatek/mt8173.dtsi +++ b/arch/arm64/boot/dts/mediatek/mt8173.dtsi @@ -1036,6 +1036,7 @@ mmsys: syscon@14000000 { assigned-clocks = <&topckgen CLK_TOP_MM_SEL>; assigned-clock-rates = <400000000>; #clock-cells = <1>; + #reset-cells = <1>; mboxes = <&gce 0 CMDQ_THR_PRIO_HIGHEST>, <&gce 1 CMDQ_THR_PRIO_HIGHEST>; mediatek,gce-client-reg = <&gce SUBSYS_1400XXXX 0 0x1000>; @@ -1262,6 +1263,7 @@ dsi0: dsi@1401b000 { <&mmsys CLK_MM_DSI0_DIGITAL>, <&mipi_tx0>; clock-names = "engine", "digital", "hs"; + resets = <&mmsys MT8173_MMSYS_SW0_RST_B_DISP_DSI0>; phys = <&mipi_tx0>; phy-names = "dphy"; status = "disabled"; diff --git a/include/dt-bindings/reset/mt8173-resets.h b/include/dt-bindings/reset/mt8173-resets.h index ba8636eda5ae..6a60c7cecc4c 100644 --- a/include/dt-bindings/reset/mt8173-resets.h +++ b/include/dt-bindings/reset/mt8173-resets.h @@ -27,6 +27,8 @@ #define MT8173_INFRA_GCE_FAXI_RST 40 #define MT8173_INFRA_MMIOMMURST 47 +/* MMSYS resets */ +#define MT8173_MMSYS_SW0_RST_B_DISP_DSI0 25 /* PERICFG resets */ #define MT8173_PERI_UART0_SW_RST 0
Reset the DSI hardware is needed to prevent different settings between the bootloader and the kernel. Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com> --- arch/arm64/boot/dts/mediatek/mt8173.dtsi | 2 ++ include/dt-bindings/reset/mt8173-resets.h | 2 ++ 2 files changed, 4 insertions(+)