linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@redhat.com>
To: Stefan Richter <stefanr@s5r6.in-berlin.de>
Cc: Linux Media Mailing List <linux-media@vger.kernel.org>
Subject: Re: [PATCH 1/2] [media] Add a keymap for FireDTV board
Date: Mon, 28 May 2012 10:32:45 -0300	[thread overview]
Message-ID: <4FC37E7D.5020106@redhat.com> (raw)
In-Reply-To: <20120528152937.75049a69@stein>

Em 28-05-2012 10:29, Stefan Richter escreveu:
> On May 28 Mauro Carvalho Chehab wrote:
>> --- a/include/media/rc-map.h
>> +++ b/include/media/rc-map.h
>> @@ -158,6 +158,7 @@ void rc_map_init(void);
>>  #define RC_MAP_VIDEOMATE_TV_PVR          "rc-videomate-tv-pvr"
>>  #define RC_MAP_WINFAST                   "rc-winfast"
>>  #define RC_MAP_WINFAST_USBII_DELUXE      "rc-winfast-usbii-deluxe"
>> +#define RC_MAP_FIREDTV			 "rc-firedtv"
>>  
>>  /*
>>   * Please, do not just append newer Remote Controller names at the end.
> 
> The comment says that names should be inserted in alphabetical order. :-)

Gah! you got me ;)

I'll merge the enclosed diff on it, to fix it.

Regards,
Mauro

diff --git a/include/media/rc-map.h b/include/media/rc-map.h
index adc6f28..29a54db 100644
--- a/include/media/rc-map.h
+++ b/include/media/rc-map.h
@@ -93,6 +93,7 @@ void rc_map_init(void);
 #define RC_MAP_ENCORE_ENLTV              "rc-encore-enltv"
 #define RC_MAP_EVGA_INDTUBE              "rc-evga-indtube"
 #define RC_MAP_EZTV                      "rc-eztv"
+#define RC_MAP_FIREDTV			 "rc-firedtv"
 #define RC_MAP_FLYDVB                    "rc-flydvb"
 #define RC_MAP_FLYVIDEO                  "rc-flyvideo"
 #define RC_MAP_FUSIONHDTV_MCE            "rc-fusionhdtv-mce"
@@ -158,7 +159,6 @@ void rc_map_init(void);
 #define RC_MAP_VIDEOMATE_TV_PVR          "rc-videomate-tv-pvr"
 #define RC_MAP_WINFAST                   "rc-winfast"
 #define RC_MAP_WINFAST_USBII_DELUXE      "rc-winfast-usbii-deluxe"
-#define RC_MAP_FIREDTV			 "rc-firedtv"
 
 /*
  * Please, do not just append newer Remote Controller names at the end.

      reply	other threads:[~2012-05-28 13:32 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-28 13:14 [PATCH 1/2] [media] Add a keymap for FireDTV board Mauro Carvalho Chehab
2012-05-28 13:14 ` [PATCH 2/2] [media] firedtv: Port it to use rc_core Mauro Carvalho Chehab
2012-05-28 14:01   ` Stefan Richter
2012-05-28 14:09     ` Mauro Carvalho Chehab
2012-05-28 14:09     ` Mauro Carvalho Chehab
2012-05-28 14:14     ` Stefan Richter
2012-05-28 14:41       ` Mauro Carvalho Chehab
2012-05-28 13:29 ` [PATCH 1/2] [media] Add a keymap for FireDTV board Stefan Richter
2012-05-28 13:32   ` Mauro Carvalho Chehab [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4FC37E7D.5020106@redhat.com \
    --to=mchehab@redhat.com \
    --cc=linux-media@vger.kernel.org \
    --cc=stefanr@s5r6.in-berlin.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).