RE: [PATCH-V2 4/4] ARM: OMAP2+: CLEANUP: Add new config option for different DPLL features

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 9 May 2012, Hiremath, Vaibhav wrote:

> On Wed, May 09, 2012 at 04:20:06, Hilman, Kevin wrote:
> 
> > 
> > [1]
> > diff --git a/arch/arm/plat-omap/include/plat/clock.h b/arch/arm/plat-omap/include/plat/clock.h
> > index d0ef57c..656b986 100644
> > --- a/arch/arm/plat-omap/include/plat/clock.h
> > +++ b/arch/arm/plat-omap/include/plat/clock.h
> > @@ -156,7 +156,6 @@ struct dpll_data {
> >  	u8			min_divider;
> >  	u16			max_divider;
> >  	u8			modes;
> > -#if defined(CONFIG_ARCH_OMAP3) || defined(CONFIG_ARCH_OMAP4)
> >  	void __iomem		*autoidle_reg;
> >  	void __iomem		*idlest_reg;
> >  	u32			autoidle_mask;
> > @@ -167,7 +166,6 @@ struct dpll_data {
> >  	u8			auto_recal_bit;
> >  	u8			recal_en_bit;
> >  	u8			recal_st_bit;
> > -#  endif
> >  	u8			flags;
> >  };
> >  
> 
> Honestly, I wanted to propose this first. 
> I am OK, as long as we all agree on this.
> 
> Paul, can you conform on this?

Yes that's a better approach then defining CONFIG_* options for all the 
bitfields, so please go ahead and just send a patch to drop that #ifdef 
and I'll ack it.


- Paul
--
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


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux