Re: Re: [Patch] USBVision - Fix NT1005 Bridge detection

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, 14 Apr 2007, Mauro Carvalho Chehab wrote:
> I've did such change on:
> 	http://linuxtv.org/hg/~mchehab/usbvision
>
> It also honors CodingStyle max line size.
>
> I've also removed some duplicated stuff (ID Vendor/Product) and changed
> some flags to use bitmap. As result, It saved about 1K of kernelspace
> area.

There's a bug in this patch, the bitfields must be unsigned if you want to
have the range 0..1 and not -1..0.

Anyway, the structure can be made even smaller, -1700 bytes overall change
by my count (*).  See linked patch.

01/01: usbvision: store the device database more efficiently
http://linuxtv.org/hg/~tap/usbvision?cmd=changeset;node=7957ba367757

 usbvision-cards.c |  431 ++++++++++++++++++++----------------------------------
 usbvision-core.c  |    8 -
 usbvision-video.c |   37 +++-
 usbvision.h       |   31 ++-
 4 files changed, 207 insertions(+), 300 deletions(-)


(*) counted like this:
objdump -h usbvision.ko |
  awk '/bss|text|data/&&!/init/{print x=strtonum("0x"$3),"\t",s+=x,$2;}'

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

[Index of Archives]     [Linux Media]     [Video 4 Linux]     [Asterisk]     [Samba]     [Xorg]     [Xfree86]     [Linux USB]

  Powered by Linux