Re: [PATCH] atomisp2: unify some ifdef cases caused by format changes

2017-03-08 Thread Mauro Carvalho Chehab
Em Wed, 8 Mar 2017 14:55:44 +0100 Hans Verkuil escreveu: > On 08/03/17 14:45, Hans Verkuil wrote: > > On 08/03/17 14:39, Greg KH wrote: > >> On Wed, Mar 08, 2017 at 01:49:23PM +0100, Hans Verkuil wrote: > >>> OK, so I discovered that these patches are for a driver added to > >>> linux-next >

Re: [PATCH] atomisp2: unify some ifdef cases caused by format changes

2017-03-08 Thread Greg KH
On Wed, Mar 08, 2017 at 01:49:23PM +0100, Hans Verkuil wrote: > OK, so I discovered that these patches are for a driver added to linux-next > without it ever been cross-posted to linux-media. > > To be polite, I think that's rather impolite. They were, but got rejected due to the size :( Mauro w

Re: [PATCH] atomisp2: unify some ifdef cases caused by format changes

2017-03-08 Thread Greg KH
On Wed, Mar 08, 2017 at 02:55:44PM +0100, Hans Verkuil wrote: > On 08/03/17 14:45, Hans Verkuil wrote: > > On 08/03/17 14:39, Greg KH wrote: > > > On Wed, Mar 08, 2017 at 01:49:23PM +0100, Hans Verkuil wrote: > > > > OK, so I discovered that these patches are for a driver added to > > > > linux-ne

Re: [PATCH] atomisp2: unify some ifdef cases caused by format changes

2017-03-08 Thread Hans Verkuil
On 08/03/17 14:55, Hans Verkuil wrote: > On 08/03/17 14:45, Hans Verkuil wrote: >> On 08/03/17 14:39, Greg KH wrote: >>> On Wed, Mar 08, 2017 at 01:49:23PM +0100, Hans Verkuil wrote: OK, so I discovered that these patches are for a driver added to linux-next without it ever been cross-pos

Re: [PATCH] atomisp2: unify some ifdef cases caused by format changes

2017-03-08 Thread Hans Verkuil
On 08/03/17 15:20, Greg KH wrote: > On Wed, Mar 08, 2017 at 02:55:44PM +0100, Hans Verkuil wrote: >> On 08/03/17 14:45, Hans Verkuil wrote: >>> On 08/03/17 14:39, Greg KH wrote: On Wed, Mar 08, 2017 at 01:49:23PM +0100, Hans Verkuil wrote: > OK, so I discovered that these patches are for a

Re: [PATCH] atomisp2: unify some ifdef cases caused by format changes

2017-03-08 Thread Hans Verkuil
On 08/03/17 14:45, Hans Verkuil wrote: On 08/03/17 14:39, Greg KH wrote: On Wed, Mar 08, 2017 at 01:49:23PM +0100, Hans Verkuil wrote: OK, so I discovered that these patches are for a driver added to linux-next without it ever been cross-posted to linux-media. To be polite, I think that's rath

Re: [PATCH] atomisp2: unify some ifdef cases caused by format changes

2017-03-08 Thread Hans Verkuil
On 08/03/17 14:39, Greg KH wrote: On Wed, Mar 08, 2017 at 01:49:23PM +0100, Hans Verkuil wrote: OK, so I discovered that these patches are for a driver added to linux-next without it ever been cross-posted to linux-media. To be polite, I think that's rather impolite. They were, but got reject

Re: [PATCH] atomisp2: unify some ifdef cases caused by format changes

2017-03-08 Thread Hans Verkuil
OK, so I discovered that these patches are for a driver added to linux-next without it ever been cross-posted to linux-media. To be polite, I think that's rather impolite. I'm now reviewing patches for a driver I haven't seen, so that makes me rather unhappy. Please cross-post staging/media dri

[PATCH] atomisp2: unify some ifdef cases caused by format changes

2017-03-06 Thread Alan Cox
The two drivers were originally merged by tools, and the tools didn't always spot white space only changes. Fix a few of them found by zero-day and clean up some more by hand. Signed-off-by: Alan Cox --- .../media/atomisp/pci/atomisp2/atomisp_cmd.c | 29 --- .../media/atomisp/pci/ato