On Wed, Apr 11, 2012 at 03:44:38PM -0600, Paul Walmsley wrote:
> Hi
Hello & thx for the feedback.
> On Wed, 11 Apr 2012, Mark A. Greer wrote:
>
> > From: "Mark A. Greer" <mgreer@xxxxxxxxxxxxxxx>
> >
> > The am35x family of SoCs do not have an IVA so
> > a parallel set of clockdomain dependencies are
> > required that are simililar to OMAP3 but without
> > any IVA dependencies.
> >
> > Signed-off-by: Mark A. Greer <mgreer@xxxxxxxxxxxxxxx>
> > ---
> > diff --git a/arch/arm/mach-omap2/clockdomains3xxx_data.c b/arch/arm/mach-omap2/clockdomains3xxx_data.c
> > index b84e138..a05a8cb 100644
> > --- a/arch/arm/mach-omap2/clockdomains3xxx_data.c
> > +++ b/arch/arm/mach-omap2/clockdomains3xxx_data.c
> > @@ -59,6 +59,12 @@ static struct clkdm_dep gfx_sgx_3xxx_wkdeps[] = {
> > { NULL },
> > };
> >
> > +static struct clkdm_dep gfx_sgx_am35x_wkdeps[] = {
> > + { .clkdm_name = "mpu_clkdm", },
> > + { .clkdm_name = "wkup_clkdm", },
> > + { NULL },
> > +};
>
> Looks like there are an extra set of commas here, maybe just use:
>
> static struct clkdm_dep gfx_sgx_am35x_wkdeps[] = {
> { .clkdm_name = "mpu_clkdm" },
> { .clkdm_name = "wkup_clkdm" },
> { NULL },
Yes, there are extra commas but they come from the code I'm copying
and removing iva2_clkdm from. I was keeping things consistent so I
kept the commas. Given that, do you still want them removed?
(I'm indifferent.)
> > @@ -366,6 +473,26 @@ static struct clockdomain *clockdomains_omap3430_common[] __initdata = {
> > NULL
> > };
> >
> > +static struct clockdomain *clockdomains_am35x_common[] __initdata = {
> > + &wkup_common_clkdm,
> > + &cm_common_clkdm,
> > + &prm_common_clkdm,
> > + &mpu_am35x_clkdm,
> > + &neon_clkdm,
> > + &sgx_am35x_clkdm,
> > + &core_l3_3xxx_clkdm,
> > + &core_l4_3xxx_clkdm,
> > + &dss_am35x_clkdm,
> > + &per_am35x_clkdm,
> > + &emu_clkdm,
> > + &usbhost_am35x_clkdm,
> > + &dpll1_clkdm,
> > + &dpll3_clkdm,
> > + &dpll4_clkdm,
> > + &dpll5_clkdm,
> > + NULL
> > +};
>
> Looks like many of these clockdomains are shared with the rest of
> OMAP3xxx. Maybe move the shared ones into a common array, remove them
> from clockdomains_omap3430_common, and in the init function below, first
> register the shared ones? That will avoid duplicate listings.
Okay.
Mark
--
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
