Date: Tue, 16 Jan 2007 10:16:33 -0800
From: Randy Dunlap <randy.dunlap@oracle.com>
To: "Robert P. J. Day" <rpjday@mindspring.com>
Cc: "Ahmed S. Darwish" <darwish.07@gmail.com>, isely@pobox.com,
       video4linux-list@redhat.com, linux-kernel@vger.kernel.org,
       trivial@kernel.org
Subject: Re: [PATCH 2.6.20-rc5 2/4] pvrusb2: Use ARRAY_SIZE macro
Message-Id: <20070116101633.39e57884.randy.dunlap@oracle.com>
In-Reply-To: <Pine.LNX.4.64.0701160334350.20244@CPE00045a9c397f-CM001225dbafb6>
References: <20070116080136.GA30133@Ahmed>
	<Pine.LNX.4.64.0701160334350.20244@CPE00045a9c397f-CM001225dbafb6>
Organization: Oracle Linux Eng.
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 944
Lines: 28

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@gmail.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.

---
~Randy
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/