Re: [linux-sunxi] Re: [PATCH v4 1/4] clk: sunxi-ng: Enable DE2_CCU for SUN8I and SUN50I
On Thu, Sep 06, 2018 at 07:10:06AM +0200, Jernej Škrabec wrote: > Dne sreda, 05. september 2018 ob 09:03:35 CEST je Maxime Ripard napisal(a): > > On Tue, Sep 04, 2018 at 10:06:06PM +0530, Jagan Teki wrote: > > > Allwinner SoC like SUN8I and SUN50I has DE2 CCU so enable them > > > as default. > > > > > > Signed-off-by: Jagan Teki > > > --- > > > Changes for v4, v3: > > > - none > > > Changes for v2: > > > - Enable for MACH_SUN8I > > > > > > drivers/clk/sunxi-ng/Kconfig | 2 ++ > > > 1 file changed, 2 insertions(+) > > > > > > diff --git a/drivers/clk/sunxi-ng/Kconfig b/drivers/clk/sunxi-ng/Kconfig > > > index 826674d090fd..e20182aa9748 100644 > > > --- a/drivers/clk/sunxi-ng/Kconfig > > > +++ b/drivers/clk/sunxi-ng/Kconfig > > > @@ -63,6 +63,8 @@ config SUN8I_V3S_CCU > > > > > > config SUN8I_DE2_CCU > > > > > > bool "Support for the Allwinner SoCs DE2 CCU" > > > > > > + default MACH_SUN8I || (ARM64 && ARCH_SUNXI) > > > + depends on DRM_SUN4I && (MACH_SUN8I || (ARM64 && ARCH_SUNXI)) > > > > I don't see why there's a dependency on the DRM driver. I'm pretty > > sure this would even break simplefb. > > I think best approach would be that SUN8I_MIXER selects (or depends on?) this > symbol, since it can't work if there is no DE2 CCU clock present. This is a runtime dependency, not a build time one. We're not doing it for any other similar driver, so there's no reason to do it in this particular case. Maxime -- Maxime Ripard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com -- You received this message because you are subscribed to the Google Groups "linux-sunxi" group. To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout. signature.asc Description: PGP signature
Re: [linux-sunxi] Re: [PATCH v4 1/4] clk: sunxi-ng: Enable DE2_CCU for SUN8I and SUN50I
Dne sreda, 05. september 2018 ob 09:03:35 CEST je Maxime Ripard napisal(a): > On Tue, Sep 04, 2018 at 10:06:06PM +0530, Jagan Teki wrote: > > Allwinner SoC like SUN8I and SUN50I has DE2 CCU so enable them > > as default. > > > > Signed-off-by: Jagan Teki > > --- > > Changes for v4, v3: > > - none > > Changes for v2: > > - Enable for MACH_SUN8I > > > > drivers/clk/sunxi-ng/Kconfig | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/drivers/clk/sunxi-ng/Kconfig b/drivers/clk/sunxi-ng/Kconfig > > index 826674d090fd..e20182aa9748 100644 > > --- a/drivers/clk/sunxi-ng/Kconfig > > +++ b/drivers/clk/sunxi-ng/Kconfig > > @@ -63,6 +63,8 @@ config SUN8I_V3S_CCU > > > > config SUN8I_DE2_CCU > > > > bool "Support for the Allwinner SoCs DE2 CCU" > > > > + default MACH_SUN8I || (ARM64 && ARCH_SUNXI) > > + depends on DRM_SUN4I && (MACH_SUN8I || (ARM64 && ARCH_SUNXI)) > > I don't see why there's a dependency on the DRM driver. I'm pretty > sure this would even break simplefb. I think best approach would be that SUN8I_MIXER selects (or depends on?) this symbol, since it can't work if there is no DE2 CCU clock present. Best regards, Jernej -- You received this message because you are subscribed to the Google Groups "linux-sunxi" group. To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
[linux-sunxi] Re: [PATCH v4 1/4] clk: sunxi-ng: Enable DE2_CCU for SUN8I and SUN50I
On Wed, Sep 5, 2018 at 12:33 PM, Maxime Ripard wrote: > On Tue, Sep 04, 2018 at 10:06:06PM +0530, Jagan Teki wrote: >> Allwinner SoC like SUN8I and SUN50I has DE2 CCU so enable them >> as default. >> >> Signed-off-by: Jagan Teki >> --- >> Changes for v4, v3: >> - none >> Changes for v2: >> - Enable for MACH_SUN8I >> >> drivers/clk/sunxi-ng/Kconfig | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/drivers/clk/sunxi-ng/Kconfig b/drivers/clk/sunxi-ng/Kconfig >> index 826674d090fd..e20182aa9748 100644 >> --- a/drivers/clk/sunxi-ng/Kconfig >> +++ b/drivers/clk/sunxi-ng/Kconfig >> @@ -63,6 +63,8 @@ config SUN8I_V3S_CCU >> >> config SUN8I_DE2_CCU >> bool "Support for the Allwinner SoCs DE2 CCU" >> + default MACH_SUN8I || (ARM64 && ARCH_SUNXI) >> + depends on DRM_SUN4I && (MACH_SUN8I || (ARM64 && ARCH_SUNXI)) > > I don't see why there's a dependency on the DRM driver. I'm pretty > sure this would even break simplefb. Ok, got it. I will drop the depends on just add default. -- You received this message because you are subscribed to the Google Groups "linux-sunxi" group. To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
[linux-sunxi] Re: [PATCH v4 1/4] clk: sunxi-ng: Enable DE2_CCU for SUN8I and SUN50I
On Tue, Sep 04, 2018 at 10:06:06PM +0530, Jagan Teki wrote: > Allwinner SoC like SUN8I and SUN50I has DE2 CCU so enable them > as default. > > Signed-off-by: Jagan Teki > --- > Changes for v4, v3: > - none > Changes for v2: > - Enable for MACH_SUN8I > > drivers/clk/sunxi-ng/Kconfig | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/clk/sunxi-ng/Kconfig b/drivers/clk/sunxi-ng/Kconfig > index 826674d090fd..e20182aa9748 100644 > --- a/drivers/clk/sunxi-ng/Kconfig > +++ b/drivers/clk/sunxi-ng/Kconfig > @@ -63,6 +63,8 @@ config SUN8I_V3S_CCU > > config SUN8I_DE2_CCU > bool "Support for the Allwinner SoCs DE2 CCU" > + default MACH_SUN8I || (ARM64 && ARCH_SUNXI) > + depends on DRM_SUN4I && (MACH_SUN8I || (ARM64 && ARCH_SUNXI)) I don't see why there's a dependency on the DRM driver. I'm pretty sure this would even break simplefb. Maxime -- Maxime Ripard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com -- You received this message because you are subscribed to the Google Groups "linux-sunxi" group. To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout. signature.asc Description: PGP signature