Re: [PATCH] media: video: mx2_camera: Remove ifdef's

2012-02-17 Thread Sascha Hauer
Hi Guennadi, On Thu, Feb 16, 2012 at 08:06:16PM +0100, Guennadi Liakhovetski wrote: > Hi > > On Thu, 16 Feb 2012, Baruch Siach wrote: > > > Hi Fabio, > > > > On Thu, Feb 16, 2012 at 04:25:39PM -0200, Fabio Estevam wrote: > > > As we are able to build a same kernel that supports both mx27 and mx

Re: [PATCH] media: video: mx2_camera: Remove ifdef's

2012-02-17 Thread Sascha Hauer
On Thu, Feb 16, 2012 at 04:25:39PM -0200, Fabio Estevam wrote: > As we are able to build a same kernel that supports both mx27 and mx25, we > should remove > the ifdef's for CONFIG_MACH_MX27 in the mx2_camera driver. It's not that simple. Yes, we are able to build a kernel for both i.MX25 and i.M

Re: [PATCH] media: video: mx2_camera: Remove ifdef's

2012-02-16 Thread Baruch Siach
Hi Fabio, On Thu, Feb 16, 2012 at 04:25:39PM -0200, Fabio Estevam wrote: > As we are able to build a same kernel that supports both mx27 and mx25, we > should remove > the ifdef's for CONFIG_MACH_MX27 in the mx2_camera driver. > > Signed-off-by: Fabio Estevam Acked-by: Baruch Siach baruch

Re: [PATCH] media: video: mx2_camera: Remove ifdef's

2012-02-16 Thread Guennadi Liakhovetski
Hi On Thu, 16 Feb 2012, Baruch Siach wrote: > Hi Fabio, > > On Thu, Feb 16, 2012 at 04:25:39PM -0200, Fabio Estevam wrote: > > As we are able to build a same kernel that supports both mx27 and mx25, we > > should remove > > the ifdef's for CONFIG_MACH_MX27 in the mx2_camera driver. > > > > Sig

[PATCH] media: video: mx2_camera: Remove ifdef's

2012-02-16 Thread Fabio Estevam
As we are able to build a same kernel that supports both mx27 and mx25, we should remove the ifdef's for CONFIG_MACH_MX27 in the mx2_camera driver. Signed-off-by: Fabio Estevam --- drivers/media/video/mx2_camera.c | 22 +++--- 1 files changed, 3 insertions(+), 19 deletions(-)