Re: [PATCH 8/8] OMAP1 clock: convert mach-omap1/clock.h to mach-omap1/clock_data.c

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

 



On Thu, Dec 03, 2009 at 03:36:48AM -0700, Paul Walmsley wrote:
> Hello Russell,
> 
> On Thu, 3 Dec 2009, Russell King wrote:
> 
> > On Thu, Dec 03, 2009 at 03:07:08AM -0700, Paul Walmsley wrote:
> > > -static int clk_omap1_dummy_enable(struct clk *clk)
> > > +int clk_omap1_dummy_enable(struct clk *clk)
> > >  {
> > >  	return 0;
> > >  }
> > >  
> > > -static void clk_omap1_dummy_disable(struct clk *clk)
> > > +void clk_omap1_dummy_disable(struct clk *clk)
> > >  {
> > >  }
> > >  
> > > -static const struct clkops clkops_dummy = {
> > > -	.enable = clk_omap1_dummy_enable,
> > > -	.disable = clk_omap1_dummy_disable,
> > > -};
> > 
> > I've no idea why you're moving the clkops into these other files - I think
> > this is going far too far.  Why not just make the clkops structures non-
> > static?
> 
> clkops that are shared between multiple platforms, such as clkops_null, 
> are indeed non-static.  However, many of the clkops are only used by one 
> OMAP variant, so it seemed pointless to make them non-static.  No point in 
> polluting the symbol table unnecessarily.  Is there some reason that all 
> of them should be non-static?

You say this, but you're keeping the associated code around as well
which is pointless.

You also say about polluting the symbol table unnecessarily.  What
about the pollution caused by having two (or more) functions made
public rather than just one data structure?
--
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