Em 14-07-2011 14:26, Arnaud Lacombe escreveu: > Cc: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxx> > Cc: Randy Dunlap <rdunlap@xxxxxxxxxxxx> > Signed-off-by: Arnaud Lacombe <lacombar@xxxxxxxxx> Arnaud, The patch is ok for me. You can add my acked-by, if you want. However, with 3.0-rc7, this patch and/or Kconfig fixes don't seem to be needed. Basically, include/generated/autoconf.h has "0x" on it: $ grep CONFIG_RADIO_RTRACK_PORT */*/* include/config/auto.conf:CONFIG_RADIO_RTRACK_PORT=20f include/generated/autoconf.h:#define CONFIG_RADIO_RTRACK_PORT 0x20f Maybe some patch applied at -next from another tree broke that behavior. IMHO, keeping those values without "0x" at Kconfig seems more intuitive for the end-user, as otherwise a value like "200" would be confusing if this means 200d or 200h. Anyway, those drivers are very old, and very few people likely still use those old hardware. So, I'm fine if you either want to preserve the current way or to change it. Cheers, Mauro. > --- > drivers/media/radio/Kconfig | 16 ++++++++-------- > 1 files changed, 8 insertions(+), 8 deletions(-) > > diff --git a/drivers/media/radio/Kconfig b/drivers/media/radio/Kconfig > index e4c97fd..9cab04d 100644 > --- a/drivers/media/radio/Kconfig > +++ b/drivers/media/radio/Kconfig > @@ -58,7 +58,7 @@ config RADIO_RTRACK > config RADIO_RTRACK_PORT > hex "RadioTrack i/o port (0x20f or 0x30f)" > depends on RADIO_RTRACK=y > - default "20f" > + default "0x20f" > help > Enter either 0x30f or 0x20f here. The card default is 0x30f, if you > haven't changed the jumper setting on the card. > @@ -81,7 +81,7 @@ config RADIO_RTRACK2 > config RADIO_RTRACK2_PORT > hex "RadioTrack II i/o port (0x20c or 0x30c)" > depends on RADIO_RTRACK2=y > - default "30c" > + default "0x30c" > help > Enter either 0x30c or 0x20c here. The card default is 0x30c, if you > haven't changed the jumper setting on the card. > @@ -104,7 +104,7 @@ config RADIO_AZTECH > config RADIO_AZTECH_PORT > hex "Aztech/Packard Bell I/O port (0x350 or 0x358)" > depends on RADIO_AZTECH=y > - default "350" > + default "0x350" > help > Enter either 0x350 or 0x358 here. The card default is 0x350, if you > haven't changed the setting of jumper JP3 on the card. Removing the > @@ -133,7 +133,7 @@ config RADIO_GEMTEK > config RADIO_GEMTEK_PORT > hex "Fixed I/O port (0x20c, 0x30c, 0x24c, 0x34c, 0c24c or 0x28c)" > depends on RADIO_GEMTEK=y > - default "34c" > + default "0x34c" > help > Enter either 0x20c, 0x30c, 0x24c or 0x34c here. The card default is > 0x34c, if you haven't changed the jumper setting on the card. On > @@ -237,7 +237,7 @@ config RADIO_TERRATEC > config RADIO_TERRATEC_PORT > hex "Terratec i/o port (normally 0x590)" > depends on RADIO_TERRATEC=y > - default "590" > + default "0x590" > help > Fill in the I/O port of your TerraTec FM radio card. If unsure, go > with the default. > @@ -255,7 +255,7 @@ config RADIO_TRUST > config RADIO_TRUST_PORT > hex "Trust i/o port (usually 0x350 or 0x358)" > depends on RADIO_TRUST=y > - default "350" > + default "0x350" > help > Enter the I/O port of your Trust FM radio card. If unsure, try the > values "0x350" or "0x358". > @@ -278,7 +278,7 @@ config RADIO_TYPHOON > config RADIO_TYPHOON_PORT > hex "Typhoon I/O port (0x316 or 0x336)" > depends on RADIO_TYPHOON=y > - default "316" > + default "0x316" > help > Enter the I/O port of your Typhoon or EcoRadio radio card. > > @@ -312,7 +312,7 @@ config RADIO_ZOLTRIX > config RADIO_ZOLTRIX_PORT > hex "ZOLTRIX I/O port (0x20c or 0x30c)" > depends on RADIO_ZOLTRIX=y > - default "20c" > + default "0x20c" > help > Enter the I/O port of your Zoltrix radio card. > -- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
