The patch number 9173 was added via Steven Toth <[EMAIL PROTECTED]>
to http://linuxtv.org/hg/v4l-dvb master development tree.

Kernel patches in this development tree may be modified to be backward
compatible with older kernels. Compatibility modifications will be
removed before inclusion into the mainstream Kernel

If anyone has any objections, please let us know by sending a message to:
        [EMAIL PROTECTED]

------

From: Steven Toth  <[EMAIL PROTECTED]>
S2API: Remove the hardcoded command limit during validation


This means that when developers add new commands then they'll be see
the DTV_MAX_COMMAND define and will be more likely to modify it, without
having to modify the command validation code.

Priority: normal

Signed-off-by: Steven Toth <[EMAIL PROTECTED]>


---

 linux/drivers/media/dvb/dvb-core/dvb_frontend.c |    3 +--
 linux/include/linux/dvb/frontend.h              |    2 ++
 2 files changed, 3 insertions(+), 2 deletions(-)

diff -r f1f3ff0e5b0b -r dde3cace78bc 
linux/drivers/media/dvb/dvb-core/dvb_frontend.c
--- a/linux/drivers/media/dvb/dvb-core/dvb_frontend.c   Fri Oct 03 10:53:18 
2008 -0400
+++ b/linux/drivers/media/dvb/dvb-core/dvb_frontend.c   Mon Oct 06 19:44:04 
2008 -0400
@@ -918,8 +918,7 @@ void dtv_property_dump(struct dtv_proper
 {
        int i;
 
-       if( (tvp->cmd <= 0 || tvp->cmd > DTV_DELIVERY_SYSTEM) &&
-                       tvp->cmd != DTV_API_VERSION) {
+       if (tvp->cmd <= 0 || tvp->cmd > DTV_MAX_COMMAND) {
                printk("%s: tvp.cmd = 0x%08x (undefined/unknown/invalid)\n",
                        __func__, tvp->cmd);
                return;
diff -r f1f3ff0e5b0b -r dde3cace78bc linux/include/linux/dvb/frontend.h
--- a/linux/include/linux/dvb/frontend.h        Fri Oct 03 10:53:18 2008 -0400
+++ b/linux/include/linux/dvb/frontend.h        Mon Oct 06 19:44:04 2008 -0400
@@ -301,6 +301,8 @@ struct dvb_frontend_event {
 #endif
 #define DTV_API_VERSION                                35
 
+#define DTV_MAX_COMMAND                                DTV_API_VERSION
+
 typedef enum fe_pilot {
        PILOT_ON,
        PILOT_OFF,


---

Patch is available at: 
http://linuxtv.org/hg/v4l-dvb/rev/dde3cace78bcdf142ddc621398ed89bd08ae8592

_______________________________________________
linuxtv-commits mailing list
linuxtv-commits@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to