Re: [PATCH] [POWERPC] Update TQM5200, CM5200 and Motion-PRO _defconfig and .dts files

[Marian Balakowicz <m8@semihalf.com>]

Grant Likely wrote:
> On 1/17/08, Marian Balakowicz <m8@semihalf.com> wrote:
>> Grant Likely wrote:
>>> On 1/17/08, Marian Balakowicz <m8@semihalf.com> wrote:
...
>>> Can you split the defconfig changes into a separate patch...  That
>>> being said, how do you feel about merging all the 5200 defconfigs into
>>> a single defconfig?  They are all multiplatform after all and it would
>>> make maintenance easier.
>> Ok, I'll split it into two patches.
>>
>> But merging defconfigs won't be a good option, boards differ in which
>> devices they use, some have PCI, some have USB, etc. Having one
>> defconfig, it would be necessary to manually customize kernel
>> configuration and remember which options are to be set/disabled.
> 
> That doesn't matter for defconfigs.  That needs to be done when you're
> tailoring a product regardless.  defconfigs are simply a known good
> configuration; they are not intended to be the deployed config.  If
> the defconfig enables all features used by any of the boards then it
> should be okay.

That's true, defconfigs are only reference configurations, but still,
I would opt for a separate defconfigs as it is more clear which target
given defconfig is meant for and it is more convenient for development
and customer to have it separate. And IMHO it's actually easier to
maintain, as you don't need to consider all three boards each time you
update defconfig and don't need to test each defconfig change on three
(right now) boards, etc.

Best Regards,
m.