arch/powerpc/sysdev: dumping ground or only for shared drivers?

Kumar Gala galak at kernel.crashing.org
Wed May 16 07:06:40 EST 2007


On May 11, 2007, at 12:05 PM, Linas Vepstas wrote:

> On Fri, May 11, 2007 at 10:26:27AM +1000, Paul Mackerras wrote:
>> Olof Johansson writes:
>>
>>> This adds yet another set of chipset drivers under sysdev, that are
>>> only used by one platform (several board ports under that platform,
>>> but only one platforms/* directory).
>>>
>>> In my opinion, they really should go under the platform directory  
>>> instead,
>>> and not clutter the shared directory.
>>
>> I disagree, actually.  Having these things in a shared directory  
>> makes
>> it more likely that people will look at the code.  That means that
>> it's more likely that bugs will be found, and more likely that parts
>> of the code can get reused when people are doing the port to a new
>> chip or board.
>>
>> If there were hundreds of files in arch/powerpc/sysdev then I  
>> would be
>> more likely to agree with you, but there aren't.
>
> I like Paul's take, it matches my gut instincts.

If this is how we are going we should move some code from arch/ 
powerpc/platforms into sysdev (for example the 5200 platform has its  
pic code and some other bits that would be candidate to move into  
sysdev).

- k



More information about the Linuxppc-dev mailing list