Re: [PATCH 2/3] Alchemy: Move evalboard code to common directory

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 2008-11-12 at 13:35 +0300, Sergei Shtylyov wrote:
> Hello.
> 
> Kevin Hickey wrote:
> 
> > And in keeping with my other email, I think that the evalboards
> > directory is a good idea (though I would recommend the name
> > "develboards" as that is what DB stands for), but it should contain
> >   
>    I'd prefer devel-boards, or dev-boards.
dev-boards works for me.

> 
> > subdirectories for each board and a common directory for common DB code.
> > Smashing all of the board code into one file doesn't leave any room to
> > grow if that file gets too big.
> 
>    I doubt that this could be the case here. And I don't think anybody 
> has placed limits on the source file size so far.
I agree from a technical standpoint.  I meant that if the files
logically grew too much and it was desirable to split them, there would
be nowhere to put the split files.  On the other hand, I suppose that we
don't need that today and we can always create the directory later if
necessary...

> 
> > Also, a single common.c will not be sufficient in the future.
> >   
> 
>    Wait, the file only includes prom_init() for now, so might be worth 
> renaming it...
> 
> > =Kevin
> >
> > On Sat, 2008-11-08 at 13:08 +0100, Manuel Lauss wrote:
> >   
> >> Move all code of the Pb/Db boards to a single subdirectory and extract
> >> some common code.
> >>
> >> Signed-off-by: Manuel Lauss <mano@xxxxxxxxxxxxxxxxxxxxxxx>
> >>     
> 
>     And now I'll have to give Kevin two lessons of the network etiquette:
> 
> - don't top-post (your comments should be below the quoted text you're 
> replying to);
> - above all, don't leave tens of KBs of uncommented patch behind, do 
> spend several seconds to delete it!
My apologies.  I will comply in this and future posts.

> 
> WBR, Sergei
> 
> 
=Kevin
-- 
Kevin Hickey
Alchemy Solutions
RMI Corporation
khickey@xxxxxxxxxxx
P: 512.691.8044


[Index of Archives]     [Linux MIPS Home]     [LKML Archive]     [Linux ARM Kernel]     [Linux ARM]     [Linux]     [Git]     [Yosemite News]     [Linux SCSI]     [Linux Hams]

  Powered by Linux