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

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 mx25, we

[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 fabio.este...@freescale.com --- drivers/media/video/mx2_camera.c | 22 +++--- 1 files changed, 3

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. Signed-off-by:

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 fabio.este...@freescale.com Acked-by: