Hi sughosh, On Thu, Feb 03, 2011 at 23:31:48, sughosh ganu wrote: > hi Sekhar, > > > On Thu, Feb 3, 2011 at 10:38 PM, Nori, Sekhar <nsek...@ti.com> wrote: > > > > > > We have a common da8xx_omapl_defconfig which is being > currently used by > > multiple boards(including hawkboard). Not all boards have a > nand flash > > mounted as default. So i cannot add this in the above > defconfig file. So > > > On Thu, Feb 03, 2011 at 20:03:35, sughosh ganu wrote: > Why? Boards which do not have NAND flash will not have > NAND platform data registered. Enabling the NAND driver > should cause no harm (except an increase in image size). > > > > Because i thought adding configs for features not used by all boards not > to be a clean way of implementing it. I am confused as to why adding the > configs for the board that uses those features is being frowned upon, > but enabling them for boards which do not use it is fine. Can someone > please elucidate.
Why do we think all Hawkboard users will need NAND? May be most users like to boot using NFS? > But if adding them to the defconfig file is ok, i will respin the patch > accordingly and resubmit. Thanks. This should be a separate patch. I think it is better to not mix defconfig updates with board file changes. Also, when you re-post, please make sure to CC linux-arm-kernel. Thanks, Sekhar _______________________________________________ Davinci-linux-open-source mailing list Davinci-linux-open-source@linux.davincidsp.com http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source