* [PATCH v4 0/3] Mark clocks as critical for MT6797
@ 2019-02-14 16:30 matthias.bgg
2019-02-26 12:59 ` Matthias Brugger
0 siblings, 1 reply; 3+ messages in thread
From: matthias.bgg @ 2019-02-14 16:30 UTC (permalink / raw)
To: mturquette, sboyd, kevin-cw.chen, mars.cheng
Cc: sean.wang, matthias.bgg, jasu, linux-clk, linux-kernel,
linux-arm-kernel, linux-mediatek, Matthias Brugger
From: Matthias Brugger <mbrugger@suse.com>
Jasper send this series some month ago. As there was no reaction from
his side, I'll do a friendly take-over.
I tested the patches on my Helios X20 boards and they fix the issue.
I didn't add a Tested-by tag as I added my Signed-off-by.
Changes since v3:
- add comments explaining why the clock muxer and gates are marked
as critical
Changes since v2 (https://patchwork.kernel.org/patch/10686759/):
- axi_sel is not needed, drop CLK_IS_CRITICAL for it.
- update commit message
- add Acked-by from Mars
Changes since v1:
- add a fixes tag.
---
Currently, DRAM-related clocks and the axi_sel MUX are not marked with
CLK_IS_CRITICAL for MT6797. This causes memory corruption when the
system is booted without clk_ignore_unused.
This patchset
1. Makes it possible to mark outputs of MUXes as critical by introducing
a new macro, MUX_FLAGS,
2. Makes it possible to mark gates as critical by adding flags to
mtk_gate, and
3. Marks axi_sel, ddrphycfg_sel, infra_dramc_f26m and infra_dramc_b_f26m
as critical.
The addition of flags to mtk_gate also exists in the patch series "Add
basic and clock support for Mediatek MT8183 SoC" [1]. The type of
flags is unsigned int in that series, but the real type is unsigned
long, so my patch differs from that patch.
[1] https://patchwork.kernel.org/patch/10549953/
Jasper Mattsson (3):
clk: mediatek: Add MUX_FLAGS macro
clk: mediatek: Add flags to mtk_gate
clk: mediatek: Mark bus and DRAM related clocks as critical
drivers/clk/mediatek/clk-gate.c | 4 +-
drivers/clk/mediatek/clk-gate.h | 3 +-
drivers/clk/mediatek/clk-mt6797.c | 68 +++++++++++++++++++------------
drivers/clk/mediatek/clk-mtk.c | 2 +-
drivers/clk/mediatek/clk-mtk.h | 9 +++-
5 files changed, 56 insertions(+), 30 deletions(-)
--
2.20.1
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v4 0/3] Mark clocks as critical for MT6797
2019-02-14 16:30 [PATCH v4 0/3] Mark clocks as critical for MT6797 matthias.bgg
@ 2019-02-26 12:59 ` Matthias Brugger
2019-02-26 17:53 ` Stephen Boyd
0 siblings, 1 reply; 3+ messages in thread
From: Matthias Brugger @ 2019-02-26 12:59 UTC (permalink / raw)
To: matthias.bgg, mturquette, sboyd, kevin-cw.chen, mars.cheng
Cc: sean.wang, jasu, linux-clk, linux-kernel, linux-arm-kernel,
linux-mediatek, Matthias Brugger
On 14/02/2019 17:30, matthias.bgg@kernel.org wrote:
> From: Matthias Brugger <mbrugger@suse.com>
>
> Jasper send this series some month ago. As there was no reaction from
> his side, I'll do a friendly take-over.
> I tested the patches on my Helios X20 boards and they fix the issue.
> I didn't add a Tested-by tag as I added my Signed-off-by.
>
> Changes since v3:
> - add comments explaining why the clock muxer and gates are marked
> as critical
>
> Changes since v2 (https://patchwork.kernel.org/patch/10686759/):
> - axi_sel is not needed, drop CLK_IS_CRITICAL for it.
> - update commit message
> - add Acked-by from Mars
>
> Changes since v1:
> - add a fixes tag.
>
Steven, Mike any comments on this?
Regards,
Matthias
> ---
>
> Currently, DRAM-related clocks and the axi_sel MUX are not marked with
> CLK_IS_CRITICAL for MT6797. This causes memory corruption when the
> system is booted without clk_ignore_unused.
>
> This patchset
>
> 1. Makes it possible to mark outputs of MUXes as critical by introducing
> a new macro, MUX_FLAGS,
> 2. Makes it possible to mark gates as critical by adding flags to
> mtk_gate, and
> 3. Marks axi_sel, ddrphycfg_sel, infra_dramc_f26m and infra_dramc_b_f26m
> as critical.
>
> The addition of flags to mtk_gate also exists in the patch series "Add
> basic and clock support for Mediatek MT8183 SoC" [1]. The type of
> flags is unsigned int in that series, but the real type is unsigned
> long, so my patch differs from that patch.
>
> [1] https://patchwork.kernel.org/patch/10549953/
>
> Jasper Mattsson (3):
> clk: mediatek: Add MUX_FLAGS macro
> clk: mediatek: Add flags to mtk_gate
> clk: mediatek: Mark bus and DRAM related clocks as critical
>
> drivers/clk/mediatek/clk-gate.c | 4 +-
> drivers/clk/mediatek/clk-gate.h | 3 +-
> drivers/clk/mediatek/clk-mt6797.c | 68 +++++++++++++++++++------------
> drivers/clk/mediatek/clk-mtk.c | 2 +-
> drivers/clk/mediatek/clk-mtk.h | 9 +++-
> 5 files changed, 56 insertions(+), 30 deletions(-)
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v4 0/3] Mark clocks as critical for MT6797
2019-02-26 12:59 ` Matthias Brugger
@ 2019-02-26 17:53 ` Stephen Boyd
0 siblings, 0 replies; 3+ messages in thread
From: Stephen Boyd @ 2019-02-26 17:53 UTC (permalink / raw)
To: Matthias Brugger, kevin-cw.chen, mars.cheng, matthias.bgg, mturquette
Cc: sean.wang, jasu, linux-clk, linux-kernel, linux-arm-kernel,
linux-mediatek, Matthias Brugger
Quoting Matthias Brugger (2019-02-26 04:59:44)
>
>
> On 14/02/2019 17:30, matthias.bgg@kernel.org wrote:
> > From: Matthias Brugger <mbrugger@suse.com>
> >
> > Jasper send this series some month ago. As there was no reaction from
> > his side, I'll do a friendly take-over.
> > I tested the patches on my Helios X20 boards and they fix the issue.
> > I didn't add a Tested-by tag as I added my Signed-off-by.
> >
> > Changes since v3:
> > - add comments explaining why the clock muxer and gates are marked
> > as critical
> >
> > Changes since v2 (https://patchwork.kernel.org/patch/10686759/):
> > - axi_sel is not needed, drop CLK_IS_CRITICAL for it.
> > - update commit message
> > - add Acked-by from Mars
> >
> > Changes since v1:
> > - add a fixes tag.
> >
>
> Steven, Mike any comments on this?
>
I got patch #1 and #2 in my queue and they're not threaded properly as
replies to each other. I'll have to go dig out the last patch somehow,
but probably I can apply them now.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-02-26 17:53 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-02-14 16:30 [PATCH v4 0/3] Mark clocks as critical for MT6797 matthias.bgg
2019-02-26 12:59 ` Matthias Brugger
2019-02-26 17:53 ` Stephen Boyd
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).