From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from perceval.ideasonboard.com ([95.142.166.194]:40179 "EHLO perceval.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753350Ab3HWNOq (ORCPT ); Fri, 23 Aug 2013 09:14:46 -0400 From: Laurent Pinchart To: Sylwester Nawrocki Cc: Dan Carpenter , Mauro Carvalho Chehab , Sachin Kamat , linux-media@vger.kernel.org, kernel-janitors@vger.kernel.org Subject: Re: [patch] [media] s5k6aa: off by one in s5k6aa_enum_frame_interval() Date: Fri, 23 Aug 2013 15:16:01 +0200 Message-ID: <1950467.dfr4BcOJWF@avalon> In-Reply-To: <52175F95.2040108@samsung.com> References: <20130823093306.GH31293@elgon.mountain> <4319865.UMEVfJshWy@avalon> <52175F95.2040108@samsung.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-media-owner@vger.kernel.org List-ID: Hi Sylwester, On Friday 23 August 2013 15:11:49 Sylwester Nawrocki wrote: > On 08/23/2013 02:54 PM, Laurent Pinchart wrote: > > On Friday 23 August 2013 12:33:06 Dan Carpenter wrote: > >> The check is off by one so we could read one space past the end of the > >> array. > >> > >> Signed-off-by: Dan Carpenter > > > > Acked-by: Laurent Pinchart > > > > Mauro, I have no other pending sensor patches, can you pick this one up > > from the list, or should I send you a pull request ? > > I can handle all pending patches for the Samsung sensors, I was planning > to send a pull request today. I can pick up the ov9650 patch as well. > I'd like to also include the s5k5bafx sensor patch in one of my pull > requests. > > IIRC it was agreed that we as Samsung will handle our stuff ourselves > and be sending it to Mauro directly. That's less work for me, so I will definitely not complain :-) -- Regards, Laurent Pinchart From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Date: Fri, 23 Aug 2013 13:16:01 +0000 Subject: Re: [patch] [media] s5k6aa: off by one in s5k6aa_enum_frame_interval() Message-Id: <1950467.dfr4BcOJWF@avalon> List-Id: References: <20130823093306.GH31293@elgon.mountain> <4319865.UMEVfJshWy@avalon> <52175F95.2040108@samsung.com> In-Reply-To: <52175F95.2040108@samsung.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Sylwester Nawrocki Cc: Dan Carpenter , Mauro Carvalho Chehab , Sachin Kamat , linux-media@vger.kernel.org, kernel-janitors@vger.kernel.org Hi Sylwester, On Friday 23 August 2013 15:11:49 Sylwester Nawrocki wrote: > On 08/23/2013 02:54 PM, Laurent Pinchart wrote: > > On Friday 23 August 2013 12:33:06 Dan Carpenter wrote: > >> The check is off by one so we could read one space past the end of the > >> array. > >> > >> Signed-off-by: Dan Carpenter > > > > Acked-by: Laurent Pinchart > > > > Mauro, I have no other pending sensor patches, can you pick this one up > > from the list, or should I send you a pull request ? > > I can handle all pending patches for the Samsung sensors, I was planning > to send a pull request today. I can pick up the ov9650 patch as well. > I'd like to also include the s5k5bafx sensor patch in one of my pull > requests. > > IIRC it was agreed that we as Samsung will handle our stuff ourselves > and be sending it to Mauro directly. That's less work for me, so I will definitely not complain :-) -- Regards, Laurent Pinchart