Messages in this thread Patch in this message | | | Date | Sun, 21 Dec 2008 15:47:06 +0900 | From | Paul Mundt <> | Subject | Re: [PATCH] sh: maple: add support for Maple controller as a joystick |
| |
On Fri, Dec 19, 2008 at 11:15:42PM +0000, Adrian McMenamin wrote: > Not sure what happened with earlier submissions, so here is an up-to-date > patch for the Dreamcast Maple controlled. > > > Add support for the SEGA Dreamcast Maple controller as a joystick > > Signed-off-by: Adrian McMenamin <adrian@mcmen.demon.co.uk> > --- > > diff --git a/drivers/input/joystick/Kconfig b/drivers/input/joystick/Kconfig > index be5c14a..ca7ce42 100644 > --- a/drivers/input/joystick/Kconfig > +++ b/drivers/input/joystick/Kconfig > @@ -294,4 +294,17 @@ config JOYSTICK_XPAD_LEDS > This option enables support for the LED which surrounds the Big X on > XBox 360 controller. > > +config JOYSTICK_MAPLE > + tristate "Dreamcast control pad" > + depends on SH_DREAMCAST > + select MAPLE > + help
Please don't do this kind of dependency garbage. The only dependency this device has is on the maple bus, there is nothing inherently dreamcast specific about it.
CONFIG_MAPLE already has the dreamcast dependency in this case, and I would speculate the only reason that you added the dreamcast dependency was to work around the fact that select would complain on plaforms where the MAPLE symbol is not visible.
This is an indicator that select is not the thing you should be using here in the first place. So, just change this to depends on MAPLE and be done with it. As soon as you have to start mixing and matching depends/selects that already have a dependency on each other, you have already lost.
| |