Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1732369 > unrolled thread
| Started by | "Levin, Alexander (Sasha Levin)" <alexander.levin@verizon.com> |
|---|---|
| First post | 2017-09-14 18:00 +0200 |
| Last post | 2017-09-18 12:30 +0200 |
| Articles | 4 — 2 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
[PATCH for 4.9 39/59] arm: dts: mt2701: Add subsystem clock controller device nodes "Levin, Alexander (Sasha Levin)" <alexander.levin@verizon.com> - 2017-09-14 18:00 +0200
Re: [PATCH for 4.9 39/59] arm: dts: mt2701: Add subsystem clock controller device nodes Matthias Brugger <matthias.bgg@gmail.com> - 2017-09-15 13:20 +0200
Re: [PATCH for 4.9 39/59] arm: dts: mt2701: Add subsystem clock controller device nodes "Levin, Alexander (Sasha Levin)" <alexander.levin@verizon.com> - 2017-09-18 01:30 +0200
Re: [PATCH for 4.9 39/59] arm: dts: mt2701: Add subsystem clock controller device nodes Matthias Brugger <matthias.bgg@gmail.com> - 2017-09-18 12:30 +0200
| From | "Levin, Alexander (Sasha Levin)" <alexander.levin@verizon.com> |
|---|---|
| Date | 2017-09-14 18:00 +0200 |
| Subject | [PATCH for 4.9 39/59] arm: dts: mt2701: Add subsystem clock controller device nodes |
| Message-ID | <upEIy-6V9-19@gated-at.bofh.it> |
From: James Liao <jamesjj.liao@mediatek.com>
[ Upstream commit f235c7e7a75325f28a33559a71f25a0eca6112db ]
Add MT2701 subsystem clock controllers, inlcude mmsys, imgsys,
vdecsys, hifsys, ethsys and bdpsys.
Signed-off-by: James Liao <jamesjj.liao@mediatek.com>
Signed-off-by: Matthias Brugger <matthias.bgg@gmail.com>
Signed-off-by: Sasha Levin <alexander.levin@verizon.com>
---
arch/arm/boot/dts/mt2701.dtsi | 36 ++++++++++++++++++++++++++++++++++++
1 file changed, 36 insertions(+)
diff --git a/arch/arm/boot/dts/mt2701.dtsi b/arch/arm/boot/dts/mt2701.dtsi
index 18596a2c58a1..77c6b931dc24 100644
--- a/arch/arm/boot/dts/mt2701.dtsi
+++ b/arch/arm/boot/dts/mt2701.dtsi
@@ -174,4 +174,40 @@
clocks = <&uart_clk>;
status = "disabled";
};
+
+ mmsys: syscon@14000000 {
+ compatible = "mediatek,mt2701-mmsys", "syscon";
+ reg = <0 0x14000000 0 0x1000>;
+ #clock-cells = <1>;
+ };
+
+ imgsys: syscon@15000000 {
+ compatible = "mediatek,mt2701-imgsys", "syscon";
+ reg = <0 0x15000000 0 0x1000>;
+ #clock-cells = <1>;
+ };
+
+ vdecsys: syscon@16000000 {
+ compatible = "mediatek,mt2701-vdecsys", "syscon";
+ reg = <0 0x16000000 0 0x1000>;
+ #clock-cells = <1>;
+ };
+
+ hifsys: syscon@1a000000 {
+ compatible = "mediatek,mt2701-hifsys", "syscon";
+ reg = <0 0x1a000000 0 0x1000>;
+ #clock-cells = <1>;
+ };
+
+ ethsys: syscon@1b000000 {
+ compatible = "mediatek,mt2701-ethsys", "syscon";
+ reg = <0 0x1b000000 0 0x1000>;
+ #clock-cells = <1>;
+ };
+
+ bdpsys: syscon@1c000000 {
+ compatible = "mediatek,mt2701-bdpsys", "syscon";
+ reg = <0 0x1c000000 0 0x1000>;
+ #clock-cells = <1>;
+ };
};
--
2.11.0
[toc] | [next] | [standalone]
| From | Matthias Brugger <matthias.bgg@gmail.com> |
|---|---|
| Date | 2017-09-15 13:20 +0200 |
| Message-ID | <upWP8-22A-5@gated-at.bofh.it> |
| In reply to | #1732369 |
On 09/14/2017 05:51 PM, Levin, Alexander (Sasha Levin) wrote:
> From: James Liao <jamesjj.liao@mediatek.com>
>
> [ Upstream commit f235c7e7a75325f28a33559a71f25a0eca6112db ]
>
> Add MT2701 subsystem clock controllers, inlcude mmsys, imgsys,
> vdecsys, hifsys, ethsys and bdpsys.
>
> Signed-off-by: James Liao <jamesjj.liao@mediatek.com>
> Signed-off-by: Matthias Brugger <matthias.bgg@gmail.com>
> Signed-off-by: Sasha Levin <alexander.levin@verizon.com>
> ---
> arch/arm/boot/dts/mt2701.dtsi | 36 ++++++++++++++++++++++++++++++++++++
> 1 file changed, 36 insertions(+)
>
It's not clear to me which bug in v4.9.y you are fixing with this. Can you
please explain.
Thanks,
Matthias
> diff --git a/arch/arm/boot/dts/mt2701.dtsi b/arch/arm/boot/dts/mt2701.dtsi
> index 18596a2c58a1..77c6b931dc24 100644
> --- a/arch/arm/boot/dts/mt2701.dtsi
> +++ b/arch/arm/boot/dts/mt2701.dtsi
> @@ -174,4 +174,40 @@
> clocks = <&uart_clk>;
> status = "disabled";
> };
> +
> + mmsys: syscon@14000000 {
> + compatible = "mediatek,mt2701-mmsys", "syscon";
> + reg = <0 0x14000000 0 0x1000>;
> + #clock-cells = <1>;
> + };
> +
> + imgsys: syscon@15000000 {
> + compatible = "mediatek,mt2701-imgsys", "syscon";
> + reg = <0 0x15000000 0 0x1000>;
> + #clock-cells = <1>;
> + };
> +
> + vdecsys: syscon@16000000 {
> + compatible = "mediatek,mt2701-vdecsys", "syscon";
> + reg = <0 0x16000000 0 0x1000>;
> + #clock-cells = <1>;
> + };
> +
> + hifsys: syscon@1a000000 {
> + compatible = "mediatek,mt2701-hifsys", "syscon";
> + reg = <0 0x1a000000 0 0x1000>;
> + #clock-cells = <1>;
> + };
> +
> + ethsys: syscon@1b000000 {
> + compatible = "mediatek,mt2701-ethsys", "syscon";
> + reg = <0 0x1b000000 0 0x1000>;
> + #clock-cells = <1>;
> + };
> +
> + bdpsys: syscon@1c000000 {
> + compatible = "mediatek,mt2701-bdpsys", "syscon";
> + reg = <0 0x1c000000 0 0x1000>;
> + #clock-cells = <1>;
> + };
> };
>
[toc] | [prev] | [next] | [standalone]
| From | "Levin, Alexander (Sasha Levin)" <alexander.levin@verizon.com> |
|---|---|
| Date | 2017-09-18 01:30 +0200 |
| Message-ID | <uqRaF-6ie-1@gated-at.bofh.it> |
| In reply to | #1732830 |
On Fri, Sep 15, 2017 at 01:15:43PM +0200, Matthias Brugger wrote: >On 09/14/2017 05:51 PM, Levin, Alexander (Sasha Levin) wrote: >>From: James Liao <jamesjj.liao@mediatek.com> >> >>[ Upstream commit f235c7e7a75325f28a33559a71f25a0eca6112db ] >> >>Add MT2701 subsystem clock controllers, inlcude mmsys, imgsys, >>vdecsys, hifsys, ethsys and bdpsys. >> >>Signed-off-by: James Liao <jamesjj.liao@mediatek.com> >>Signed-off-by: Matthias Brugger <matthias.bgg@gmail.com> >>Signed-off-by: Sasha Levin <alexander.levin@verizon.com> >>--- >> arch/arm/boot/dts/mt2701.dtsi | 36 ++++++++++++++++++++++++++++++++++++ >> 1 file changed, 36 insertions(+) >> > >It's not clear to me which bug in v4.9.y you are fixing with this. Can >you please explain. Hi Matthias, Note that beyond bug fixes, stable kernel rules also allow for: """ - New device IDs and quirks are also accepted. """ In general, patches that enable devices which use existing in-kernel drivers are also accepted to the stable kernel tree. If this is not the case here, or you have a reason to keep it out, please let me know. -- Thanks, Sasha
[toc] | [prev] | [next] | [standalone]
| From | Matthias Brugger <matthias.bgg@gmail.com> |
|---|---|
| Date | 2017-09-18 12:30 +0200 |
| Message-ID | <ur1to-56M-9@gated-at.bofh.it> |
| In reply to | #1733551 |
On 09/18/2017 01:20 AM, Levin, Alexander (Sasha Levin) wrote:
> On Fri, Sep 15, 2017 at 01:15:43PM +0200, Matthias Brugger wrote:
>> On 09/14/2017 05:51 PM, Levin, Alexander (Sasha Levin) wrote:
>>> From: James Liao <jamesjj.liao@mediatek.com>
>>>
>>> [ Upstream commit f235c7e7a75325f28a33559a71f25a0eca6112db ]
>>>
>>> Add MT2701 subsystem clock controllers, inlcude mmsys, imgsys,
>>> vdecsys, hifsys, ethsys and bdpsys.
>>>
>>> Signed-off-by: James Liao <jamesjj.liao@mediatek.com>
>>> Signed-off-by: Matthias Brugger <matthias.bgg@gmail.com>
>>> Signed-off-by: Sasha Levin <alexander.levin@verizon.com>
>>> ---
>>> arch/arm/boot/dts/mt2701.dtsi | 36 ++++++++++++++++++++++++++++++++++++
>>> 1 file changed, 36 insertions(+)
>>>
>>
>> It's not clear to me which bug in v4.9.y you are fixing with this. Can
>> you please explain.
>
> Hi Matthias,
>
> Note that beyond bug fixes, stable kernel rules also allow for:
>
> """
> - New device IDs and quirks are also accepted.
> """
>
> In general, patches that enable devices which use existing in-kernel drivers are also accepted to the stable kernel tree.
>
> If this is not the case here, or you have a reason to keep it out, please let me know.
>
Hm, I checked linux-4.9.y and it lacks commit:
e9862118272a ("clk: mediatek: Add MT2701 clock support")
Which got's merged in v4.10, so this patch has no effect at all.
What do I miss?
Regards,
Matthias
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web