On Tuesday 27 September 2011 05:44 PM, Mark Brown wrote: > On Tue, Sep 27, 2011 at 03:42:47PM +0530, Rajendra Nayak wrote: > >> +#ifdef CONFIG_OF >> + char compatible[128]; >> +#endif > > Might it not be better to just make this a pointer to const char? Yes, my bad.