On Fri, Sep 19, 2008 at 12:10:34AM +0100, Russell King - ARM Linux wrote:
> On Fri, Sep 19, 2008 at 01:23:40AM +0300, Felipe Balbi wrote:
> > From: Felipe Balbi <[EMAIL PROTECTED]>
> > 
> > Create a new include/linux/watchdog directory
> > for holding watchdog chips headers, move omap_wdt.h
> > to the new location and update the include patch in
> > the driver source.
> 
> What's the reasoning for this change?  As far as I can see, omap_wdt.h
> is private to omap_wdt.c, and shouldn't be exposed to anyone else.
> In which case, leaving it along side omap_wdt.c is the right thing to
> do.
> 
> We don't have any rules which say all header files must be in the
> include/ sub-tree.

It's gonna be needed on patch 5/5. Maybe I should meld it in the same
patch ?

-- 
balbi
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to