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 <felipe.balbi@xxxxxxxxx> > > > > 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
