lkml.org 
[lkml]   [2008]   [Oct]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] Make ATNGW100 serial ports configurable
Anders Blomdell <anders.blomdell@control.lth.se> wrote:
> Make configuration of ATNGW100 serial ports selectable in kernel configuration.
>
> Signed-off-by: Anders Blomdell <anders.blomdell@control.lth.se>

Gak. If we're going down this path, why not add #ifdefs for every other
conceivable hardware mod as well and turn the ATNGW100 board code into
an even worse mess than the ATSTK1000 board code?

The ATNGW100 has one serial port on board, so IMO the standard board
code should only initialize that one port.

However, it might be sensible to add some sort of interface for
expansion board code. For example something like this:

#ifdef CONFIG_ATNGW100_EXPANSION
atngw100_setup_expansion_board();
#endif

This will allow people with hardware mods to add all the extra devices
they need, including serial ports, by simply adding another file with
expansion board code. People who aren't afraid to solder stuff on their
boards shouldn't be afraid of writing some board code too, right?

What do you think?

Haavard


\
 
 \ /
  Last update: 2008-10-10 17:01    [W:0.077 / U:1.076 seconds]
©2003-2014 Jasper Spaans. Advertise on this site