From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mx3.redhat.com (mx3.redhat.com [172.16.48.32]) by int-mx1.corp.redhat.com (8.13.1/8.13.1) with ESMTP id m5A03K8u015726 for ; Mon, 9 Jun 2008 20:03:20 -0400 Received: from QMTA05.westchester.pa.mail.comcast.net (qmta05.westchester.pa.mail.comcast.net [76.96.62.48]) by mx3.redhat.com (8.13.8/8.13.8) with ESMTP id m5A036Bm004734 for ; Mon, 9 Jun 2008 20:03:07 -0400 Message-ID: <484DC498.7070503@personnelware.com> Date: Mon, 09 Jun 2008 19:02:32 -0500 From: Carl Karsten MIME-Version: 1.0 To: video4linux-list@redhat.com References: <47C8A0C9.4020107@personnelware.com> <20080304112519.6f4c748c@gaivota> <483DBD67.8090508@personnelware.com> <20080528173755.594ea08b@gaivota> <483DD6AA.1070203@personnelware.com> <1212015588.5745.9.camel@pc10.localdom.local> <483DEB53.40604@personnelware.com> <1212021382.5745.13.camel@pc10.localdom.local> In-Reply-To: <1212021382.5745.13.camel@pc10.localdom.local> Content-Type: multipart/mixed; boundary="------------080207040807000203040602" Cc: Mauro Carvalho Chehab Subject: Re: [patch] vivi: registered as /dev/video%d List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: video4linux-list-bounces@redhat.com Errors-To: video4linux-list-bounces@redhat.com List-ID: This is a multi-part message in MIME format. --------------080207040807000203040602 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit hermann pitton wrote: > Am Mittwoch, den 28.05.2008, 18:31 -0500 schrieb Carl Karsten: >> hermann pitton wrote: >>> Hi Carl, >>> >>> Am Mittwoch, den 28.05.2008, 17:03 -0500 schrieb Carl Karsten: >>>> Mauro Carvalho Chehab wrote: >>>>> On Wed, 28 May 2008 15:15:35 -0500 >>>>> Carl Karsten wrote: >>>>> >>>>>> I posted a week ago and haven't heard anything. >>>>> I was on vacations last week. >>>>> >>>>>> How long should I wait before >>>>>> posting this? :) >>>>> There are a few issues on your patch: >>>>> >>>>>> - else >>>>>> + printk(KERN_INFO "%s: /dev/video%d unregistered.\n", MODULE_NAME, >>>>>> dev->vfd->minor); >>>>> Your patch got word wrapped. So, it didn't apply. >>>>> >>>>>> + } >>>>>> + else { >>>>> CodingStyle is wrong. It should be: >>>>> } else { >>>>> >>>>> (at the same line) >>>>> >>>>> Also, on some places, you used space, instead of tabs. >>>>> >>>>> Please, check your patch with checkpatch.pl (or, inside Mercurial, make >>>>> checkpatch) before sending it. >>>>> >>>>> Also, be sure that your emailer won't add line breaks at the wrong places. >>>>>> } else >>>>>> printk(KERN_INFO "Video Technology Magazine Virtual Video " >>>>>> - "Capture Board successfully loaded.\n"); >>>>>> + "Capture Board ver %u.%u.%u successfully loaded.\n", >>>>>> + (VIVI_VERSION >> 16) & 0xFF, (VIVI_VERSION >> 8) & 0xFF, VIVI_VERSION & >>>>>> 0xFF); >>>> Fixed what you mentioned, make checkpatch doesn't report anything now. It was >>>> reporting "warning: line over 80 characters" so now that those are fixed maybe >>>> t-bird won't wrap them. >>> your hope is in vain for the one space indent in front of every line >>> tbird adds. We have that issue already seen with your previous patches >>> and I told you about, it is a very well known tbird flaw ;) >>> >>> Please use always attachments with thunderbird. >>> It is no fun to fix all lines for indentation. >> Sorry about that - I meant to attach it too. >> >> What is the proper way to supply: >> >> Signed-off-by: Carl Karsten >> >> Carl K > > Hi, > > no, at least I did not receive it as attachment. > > Last known status, years back here, is that tbird attachments are always > OK and unbroken. > Same patch I have been trying to correctly submit, Sorry all for posts to the list, just trying to do it right so the busy people don't have to fuss with fixing it. Signed-off-by: Carl Karsten Carl K --------------080207040807000203040602 Content-Type: text/x-diff; name="vivi.c.diff" Content-Transfer-Encoding: 7bit Content-Disposition: inline; filename="vivi.c.diff" diff -r 9d04bba82511 linux/drivers/media/video/vivi.c --- a/linux/drivers/media/video/vivi.c Wed May 14 23:14:04 2008 +0000 +++ b/linux/drivers/media/video/vivi.c Thu May 29 14:16:27 2008 -0500 @@ -48,6 +48,8 @@ #include #endif +#define MODULE_NAME "vivi" + /* Wake up at about 30 fps */ #define WAKE_NUMERATOR 30 #define WAKE_DENOMINATOR 1001 @@ -56,7 +58,7 @@ #include "font.h" #define VIVI_MAJOR_VERSION 0 -#define VIVI_MINOR_VERSION 4 +#define VIVI_MINOR_VERSION 5 #define VIVI_RELEASE 0 #define VIVI_VERSION \ KERNEL_VERSION(VIVI_MAJOR_VERSION, VIVI_MINOR_VERSION, VIVI_RELEASE) @@ -1086,10 +1088,15 @@ static int vivi_release(void) list_del(list); dev = list_entry(list, struct vivi_dev, vivi_devlist); - if (-1 != dev->vfd->minor) + if (-1 != dev->vfd->minor) { video_unregister_device(dev->vfd); - else + printk(KERN_INFO "%s: /dev/video%d unregistered.\n", + MODULE_NAME, dev->vfd->minor); + } else { video_device_release(dev->vfd); + printk(KERN_INFO "%s: /dev/video%d released.\n", + MODULE_NAME, dev->vfd->minor); + } kfree(dev); } @@ -1202,6 +1209,8 @@ static int __init vivi_init(void) video_nr++; dev->vfd = vfd; + printk(KERN_INFO "%s: V4L2 device registered as /dev/video%d\n", + MODULE_NAME, vfd->minor); } if (ret < 0) { @@ -1209,7 +1218,9 @@ static int __init vivi_init(void) printk(KERN_INFO "Error %d while loading vivi driver\n", ret); } else printk(KERN_INFO "Video Technology Magazine Virtual Video " - "Capture Board successfully loaded.\n"); + "Capture Board ver %u.%u.%u successfully loaded.\n", + (VIVI_VERSION >> 16) & 0xFF, (VIVI_VERSION >> 8) & 0xFF, + VIVI_VERSION & 0xFF); return ret; } --------------080207040807000203040602 Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline -- video4linux-list mailing list Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe https://www.redhat.com/mailman/listinfo/video4linux-list --------------080207040807000203040602--