From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-bw0-f227.google.com ([209.85.218.227]:49599 "EHLO mail-bw0-f227.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756701AbZKMSTO convert rfc822-to-8bit (ORCPT ); Fri, 13 Nov 2009 13:19:14 -0500 MIME-Version: 1.0 In-Reply-To: <4fb5db50911122132t6ad3b5ccsfc0691a63b9a34ac@mail.gmail.com> References: <4fb5db50911121909h1619e483ga4aa395af8fc1161@mail.gmail.com> <1258082571.7715.13.camel@violet> <4fb5db50911122132t6ad3b5ccsfc0691a63b9a34ac@mail.gmail.com> Date: Fri, 13 Nov 2009 23:49:18 +0530 Message-ID: <4fb5db50911131019x51b88cdevba3d61648b83143c@mail.gmail.com> Subject: Re: [patch 0/1] Adding radio type FM From: Janakiram Sistla To: Marcel Holtmann Cc: linux-wireless , linux-kernel@vger.kernel.org Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-wireless-owner@vger.kernel.org List-ID: Hi Marcel, Let me know if this patch can go ahead...........and i can go ahead with this in rfkill utility....... Regards, Ram. On Fri, Nov 13, 2009 at 11:02 AM, Janakiram Sistla wrote: > HI Marcel, > > Yes as pointed out i am pointing to Toshiba Satellite A305 > notebook.This has an FM key which enables FM. > > Regards, > Ram. > On 11/13/09, Marcel Holtmann wrote: >> Hi Janakiram, >> >> > From 1b1493392faeb6702ff364447b135e481930b397 Mon Sep 17 00:00:00 2001 >> > From: Janakiram Sistla >> > Date: Fri, 13 Nov 2009 08:16:26 +0530 >> > Subject: [PATCH 1/1] Adding radio type FM >> > >> > Adding radio type FM in RFKILL_TYPE_.FM also belongs to >> > same class of with both TX/RX capability. >> > Also Added input type for the above same. >> > >> > Signed-off-by: Janakiram Sistla >> > --- >> >  include/linux/input.h  |    1 + >> >  include/linux/rfkill.h |    1 + >> >  net/rfkill/core.c      |    2 ++ >> >  net/rfkill/input.c     |    9 +++++++++ >> >  4 files changed, 13 insertions(+), 0 deletions(-) >> > >> > diff --git a/include/linux/input.h b/include/linux/input.h >> > index 0ccfc30..f03ae90 100644 >> > --- a/include/linux/input.h >> > +++ b/include/linux/input.h >> > @@ -376,6 +376,7 @@ struct input_absinfo { >> >  #define KEY_DISPLAY_OFF              245     /* display device to off state */ >> > >> >  #define KEY_WIMAX            246 >> > +#define KEY_FM                       247 >> > >> >  /* Range 248 - 255 is reserved for special needs of AT keyboard driver */ >> > >> > diff --git a/include/linux/rfkill.h b/include/linux/rfkill.h >> > index 3392c59..03f5598 100644 >> > --- a/include/linux/rfkill.h >> > +++ b/include/linux/rfkill.h >> > @@ -45,6 +45,7 @@ enum rfkill_type { >> >       RFKILL_TYPE_WIMAX, >> >       RFKILL_TYPE_WWAN, >> >       RFKILL_TYPE_GPS, >> > +     RFKILL_TYPE_FM, >> >       NUM_RFKILL_TYPES, >> >  }; >> > >> > diff --git a/net/rfkill/core.c b/net/rfkill/core.c >> > index ba2efb9..b8ac206 100644 >> > --- a/net/rfkill/core.c >> > +++ b/net/rfkill/core.c >> > @@ -592,6 +592,8 @@ static const char *rfkill_get_type_str(enum >> > rfkill_type type) >> >               return "wwan"; >> >       case RFKILL_TYPE_GPS: >> >               return "gps"; >> > +     case RRFKILL_TYPE_FM: >> > +             return "fm"; >> >       default: >> >               BUG(); >> >       } >> >> this part is fine with me. >> >> > diff --git a/net/rfkill/input.c b/net/rfkill/input.c >> > index a7295ad..f51b16d 100644 >> > --- a/net/rfkill/input.c >> > +++ b/net/rfkill/input.c >> > @@ -212,6 +212,9 @@ static void rfkill_event(struct input_handle >> > *handle, unsigned int type, >> >               case KEY_WIMAX: >> >                       rfkill_schedule_toggle(RFKILL_TYPE_WIMAX); >> >                       break; >> > +             case KEY_FM: >> > +                     rfkill_schedule_toggle(RFKILL_TYPE_FM); >> > +                     break; >> >               } >> >       } else if (type == EV_SW && code == SW_RFKILL_ALL) >> >               rfkill_schedule_evsw_rfkillall(data); >> > @@ -290,6 +293,12 @@ static const struct input_device_id rfkill_ids[] = { >> >               .keybit = { [BIT_WORD(KEY_UWB)] = BIT_MASK(KEY_UWB) }, >> >       }, >> >       { >> > +             .flags = INPUT_DEVICE_ID_MATCH_EVBIT | >> > +                             INPUT_DEVICE_ID_MATCH_KEYBIT, >> > +             .evbit = { BIT_MASK(EV_KEY) }, >> > +             .keybit = { [BIT_WORD(KEY_UWB)] = BIT_MASK(KEY_UWB) }, >> > +     }, >> > +     { >> >               .flags = INPUT_DEVICE_ID_MATCH_EVBIT | INPUT_DEVICE_ID_MATCH_KEYBIT, >> >               .evbit = { BIT_MASK(EV_KEY) }, >> >               .keybit = { [BIT_WORD(KEY_WIMAX)] = BIT_MASK(KEY_WIMAX) }, >> >> So besides the pointed of the copy-and-paste mistake, I don't think we >> should be doing this. Unless you point me to hardware that actually has >> a physical RFKILL button for FM. We also didn't add key define for GPS >> since there is no hardware that has this. >> >> If you think you need this for some weird handling then go one step back >> and read the re-written RFKILL subsystem design. >> >> Regards >> >> Marcel >> >> >> >