linux-arm-msm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/1] arm64: defconfig: Enable qcom msm8994 clk drivers
@ 2023-01-30 11:10 Petr Vorel
  2023-01-30 15:27 ` Bjorn Andersson
  0 siblings, 1 reply; 6+ messages in thread
From: Petr Vorel @ 2023-01-30 11:10 UTC (permalink / raw)
  To: linux-arm-msm
  Cc: Petr Vorel, Konrad Dybcio, Andy Gross, Bjorn Andersson,
	Krzysztof Kozlowski, Dominik Kobinski, Jamie Douglass

Enabling the clk drivers on msm8994 allows to boot and test most device
drivers on this SoC.

Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
NOTE: Working as part of SUSE hackweek, that's why I use my work mail
instead of usual private gmail.

Kind regards,
Petr

 arch/arm64/configs/defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index bfafb4313436..c6b08a9d8944 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -1105,6 +1105,7 @@ CONFIG_IPQ_GCC_6018=y
 CONFIG_IPQ_GCC_8074=y
 CONFIG_MSM_GCC_8916=y
 CONFIG_MSM_GCC_8994=y
+CONFIG_MSM_MMCC_8994=y
 CONFIG_MSM_MMCC_8996=y
 CONFIG_MSM_GCC_8998=y
 CONFIG_QCS_GCC_404=y
-- 
2.39.1


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/1] arm64: defconfig: Enable qcom msm8994 clk drivers
  2023-01-30 11:10 [PATCH 1/1] arm64: defconfig: Enable qcom msm8994 clk drivers Petr Vorel
@ 2023-01-30 15:27 ` Bjorn Andersson
  2023-01-30 18:20   ` Petr Vorel
  2023-01-30 19:12   ` Petr Vorel
  0 siblings, 2 replies; 6+ messages in thread
From: Bjorn Andersson @ 2023-01-30 15:27 UTC (permalink / raw)
  To: Petr Vorel
  Cc: linux-arm-msm, Konrad Dybcio, Andy Gross, Bjorn Andersson,
	Krzysztof Kozlowski, Dominik Kobinski, Jamie Douglass

On Mon, Jan 30, 2023 at 12:10:14PM +0100, Petr Vorel wrote:
> Enabling the clk drivers on msm8994 allows to boot and test most device
> drivers on this SoC.
> 
> Signed-off-by: Petr Vorel <pvorel@suse.cz>

Now that we can handle probe defer on the power-domains, can this be
made =m instead?

Regards,
Bjorn

> ---
> NOTE: Working as part of SUSE hackweek, that's why I use my work mail
> instead of usual private gmail.
> 
> Kind regards,
> Petr
> 
>  arch/arm64/configs/defconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> index bfafb4313436..c6b08a9d8944 100644
> --- a/arch/arm64/configs/defconfig
> +++ b/arch/arm64/configs/defconfig
> @@ -1105,6 +1105,7 @@ CONFIG_IPQ_GCC_6018=y
>  CONFIG_IPQ_GCC_8074=y
>  CONFIG_MSM_GCC_8916=y
>  CONFIG_MSM_GCC_8994=y
> +CONFIG_MSM_MMCC_8994=y
>  CONFIG_MSM_MMCC_8996=y
>  CONFIG_MSM_GCC_8998=y
>  CONFIG_QCS_GCC_404=y
> -- 
> 2.39.1
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/1] arm64: defconfig: Enable qcom msm8994 clk drivers
  2023-01-30 15:27 ` Bjorn Andersson
@ 2023-01-30 18:20   ` Petr Vorel
  2023-01-30 19:12   ` Petr Vorel
  1 sibling, 0 replies; 6+ messages in thread
From: Petr Vorel @ 2023-01-30 18:20 UTC (permalink / raw)
  To: Bjorn Andersson
  Cc: linux-arm-msm, Konrad Dybcio, Andy Gross, Bjorn Andersson,
	Krzysztof Kozlowski, Dominik Kobinski, Jamie Douglass

Hi Bjorn,

> On Mon, Jan 30, 2023 at 12:10:14PM +0100, Petr Vorel wrote:
> > Enabling the clk drivers on msm8994 allows to boot and test most device
> > drivers on this SoC.

> > Signed-off-by: Petr Vorel <pvorel@suse.cz>

> Now that we can handle probe defer on the power-domains, can this be
> made =m instead?
Thanks for your tip. Sure, if it's safe to change the rest of CONFIG_MSM_MMCC_.*
to module as well (in separate patch). I'll send v2.

Kind regards,
Petr

> Regards,
> Bjorn

> > ---
> > NOTE: Working as part of SUSE hackweek, that's why I use my work mail
> > instead of usual private gmail.

> > Kind regards,
> > Petr

> >  arch/arm64/configs/defconfig | 1 +
> >  1 file changed, 1 insertion(+)

> > diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> > index bfafb4313436..c6b08a9d8944 100644
> > --- a/arch/arm64/configs/defconfig
> > +++ b/arch/arm64/configs/defconfig
> > @@ -1105,6 +1105,7 @@ CONFIG_IPQ_GCC_6018=y
> >  CONFIG_IPQ_GCC_8074=y
> >  CONFIG_MSM_GCC_8916=y
> >  CONFIG_MSM_GCC_8994=y
> > +CONFIG_MSM_MMCC_8994=y
> >  CONFIG_MSM_MMCC_8996=y
> >  CONFIG_MSM_GCC_8998=y
> >  CONFIG_QCS_GCC_404=y
> > -- 
> > 2.39.1


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/1] arm64: defconfig: Enable qcom msm8994 clk drivers
  2023-01-30 15:27 ` Bjorn Andersson
  2023-01-30 18:20   ` Petr Vorel
@ 2023-01-30 19:12   ` Petr Vorel
  2023-02-08 23:06     ` Bjorn Andersson
  1 sibling, 1 reply; 6+ messages in thread
From: Petr Vorel @ 2023-01-30 19:12 UTC (permalink / raw)
  To: Bjorn Andersson
  Cc: linux-arm-msm, Konrad Dybcio, Andy Gross, Bjorn Andersson,
	Krzysztof Kozlowski, Dominik Kobinski, Jamie Douglass

> On Mon, Jan 30, 2023 at 12:10:14PM +0100, Petr Vorel wrote:
> > Enabling the clk drivers on msm8994 allows to boot and test most device
> > drivers on this SoC.

> > Signed-off-by: Petr Vorel <pvorel@suse.cz>

> Now that we can handle probe defer on the power-domains, can this be
> made =m instead?

Out of curiosity may I know what commit implemented handling probe defer on
power-domains?

Kind regards,
Petr

> Regards,
> Bjorn

> > ---
> > NOTE: Working as part of SUSE hackweek, that's why I use my work mail
> > instead of usual private gmail.

> > Kind regards,
> > Petr

> >  arch/arm64/configs/defconfig | 1 +
> >  1 file changed, 1 insertion(+)

> > diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> > index bfafb4313436..c6b08a9d8944 100644
> > --- a/arch/arm64/configs/defconfig
> > +++ b/arch/arm64/configs/defconfig
> > @@ -1105,6 +1105,7 @@ CONFIG_IPQ_GCC_6018=y
> >  CONFIG_IPQ_GCC_8074=y
> >  CONFIG_MSM_GCC_8916=y
> >  CONFIG_MSM_GCC_8994=y
> > +CONFIG_MSM_MMCC_8994=y
> >  CONFIG_MSM_MMCC_8996=y
> >  CONFIG_MSM_GCC_8998=y
> >  CONFIG_QCS_GCC_404=y
> > -- 
> > 2.39.1


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/1] arm64: defconfig: Enable qcom msm8994 clk drivers
  2023-01-30 19:12   ` Petr Vorel
@ 2023-02-08 23:06     ` Bjorn Andersson
  2023-02-09  8:00       ` Petr Vorel
  0 siblings, 1 reply; 6+ messages in thread
From: Bjorn Andersson @ 2023-02-08 23:06 UTC (permalink / raw)
  To: Petr Vorel
  Cc: linux-arm-msm, Konrad Dybcio, Andy Gross, Bjorn Andersson,
	Krzysztof Kozlowski, Dominik Kobinski, Jamie Douglass

On Mon, Jan 30, 2023 at 08:12:49PM +0100, Petr Vorel wrote:
> > On Mon, Jan 30, 2023 at 12:10:14PM +0100, Petr Vorel wrote:
> > > Enabling the clk drivers on msm8994 allows to boot and test most device
> > > drivers on this SoC.
> 
> > > Signed-off-by: Petr Vorel <pvorel@suse.cz>
> 
> > Now that we can handle probe defer on the power-domains, can this be
> > made =m instead?
> 
> Out of curiosity may I know what commit implemented handling probe defer on
> power-domains?
> 

Probe defer support has been there for a long time, but it was not
reliable after late_initcall.

I believe 2b28a1a84a0e ("driver core: Extend deferred probe timeout on
driver registration") is the change that altered the behavior, so we can
now use it for kernel modules as well.

Regards,
Bjorn

> Kind regards,
> Petr
> 
> > Regards,
> > Bjorn
> 
> > > ---
> > > NOTE: Working as part of SUSE hackweek, that's why I use my work mail
> > > instead of usual private gmail.
> 
> > > Kind regards,
> > > Petr
> 
> > >  arch/arm64/configs/defconfig | 1 +
> > >  1 file changed, 1 insertion(+)
> 
> > > diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> > > index bfafb4313436..c6b08a9d8944 100644
> > > --- a/arch/arm64/configs/defconfig
> > > +++ b/arch/arm64/configs/defconfig
> > > @@ -1105,6 +1105,7 @@ CONFIG_IPQ_GCC_6018=y
> > >  CONFIG_IPQ_GCC_8074=y
> > >  CONFIG_MSM_GCC_8916=y
> > >  CONFIG_MSM_GCC_8994=y
> > > +CONFIG_MSM_MMCC_8994=y
> > >  CONFIG_MSM_MMCC_8996=y
> > >  CONFIG_MSM_GCC_8998=y
> > >  CONFIG_QCS_GCC_404=y
> > > -- 
> > > 2.39.1
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/1] arm64: defconfig: Enable qcom msm8994 clk drivers
  2023-02-08 23:06     ` Bjorn Andersson
@ 2023-02-09  8:00       ` Petr Vorel
  0 siblings, 0 replies; 6+ messages in thread
From: Petr Vorel @ 2023-02-09  8:00 UTC (permalink / raw)
  To: Bjorn Andersson
  Cc: linux-arm-msm, Konrad Dybcio, Andy Gross, Bjorn Andersson,
	Krzysztof Kozlowski, Dominik Kobinski, Jamie Douglass

> On Mon, Jan 30, 2023 at 08:12:49PM +0100, Petr Vorel wrote:
> > > On Mon, Jan 30, 2023 at 12:10:14PM +0100, Petr Vorel wrote:
> > > > Enabling the clk drivers on msm8994 allows to boot and test most device
> > > > drivers on this SoC.

> > > > Signed-off-by: Petr Vorel <pvorel@suse.cz>

> > > Now that we can handle probe defer on the power-domains, can this be
> > > made =m instead?

> > Out of curiosity may I know what commit implemented handling probe defer on
> > power-domains?


> Probe defer support has been there for a long time, but it was not
> reliable after late_initcall.

> I believe 2b28a1a84a0e ("driver core: Extend deferred probe timeout on
> driver registration") is the change that altered the behavior, so we can
> now use it for kernel modules as well.

Thanks for info! I have v2 prepared, but I'd prefer to test it, thus I'll send
it next week after I get time to test it.

Kind regards,
Petr

> Regards,
> Bjorn

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2023-02-09  8:00 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-01-30 11:10 [PATCH 1/1] arm64: defconfig: Enable qcom msm8994 clk drivers Petr Vorel
2023-01-30 15:27 ` Bjorn Andersson
2023-01-30 18:20   ` Petr Vorel
2023-01-30 19:12   ` Petr Vorel
2023-02-08 23:06     ` Bjorn Andersson
2023-02-09  8:00       ` Petr Vorel

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).