[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <Pine.LNX.4.64.0701190003020.26643@cnc.isely.net>
Date: Fri, 19 Jan 2007 00:05:58 -0600 (CST)
From: Mike Isely <isely@...ly.net>
To: "Ahmed S. Darwish" <darwish.07@...il.com>
cc: Randy Dunlap <randy.dunlap@...cle.com>,
"Robert P. J. Day" <rpjday@...dspring.com>,
video4linux-list@...hat.com,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
trivial@...nel.org, Mike Isely at pobox <isely@...ox.com>
Subject: Re: [PATCH 2.6.20-rc5 2/4] pvrusb2: Use ARRAY_SIZE macro
On Tue, 16 Jan 2007, Ahmed S. Darwish wrote:
> On Tue, Jan 16, 2007 at 10:16:33AM -0800, Randy Dunlap wrote:
> > On Tue, 16 Jan 2007 03:36:16 -0500 (EST) Robert P. J. Day wrote:
> >
> > > On Tue, 16 Jan 2007, Ahmed S. Darwish wrote:
> > >
> > > > Use ARRAY_SIZE macro in pvrusb2-hdw.c file
> > > >
> > > > Signed-off-by: Ahmed S. Darwish <darwish.07@...il.com>
> > >
> > > ... snip ...
> > >
> > > i'm not sure it's worth submitting multiple patches to convert code
> > > expressions to the ARRAY_SIZE() macro since i was going to wait for
> > > the next kernel release, and do that in one fell swoop with a single
> > > patch.
> > >
> > > but if people higher up the food chain think it's a better idea to do
> > > it a little at a time, that's fine.
> >
> > I'm not strictly on the food chain, but these 4 patches to
> > pvrusb2 should have been sent as one patch IMO.
>
> Here's the same patch in one file as suggested.
>
> A patch to use ARRAY_SIZE macro when appropriate.
>
> Signed-off-by: Ahmed S. Darwish <darwish.07@...il.com>
> ---
Ahmed:
I've pulled your patch into my driver source and will propagate it up
appropriately as part of the next batch of pvrusb2 changes to come out
of the v4l-dvb repository.
-Mike
--
| Mike Isely | PGP fingerprint
Spammers Die!! | | 03 54 43 4D 75 E5 CC 92
| isely @ pobox (dot) com | 71 16 01 E2 B5 F5 C1 E8
| |
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists