On Tue, 17 Nov 2009, Antonio Ospite wrote:
> pxa_camera init() is going to be removed.
> Configure PXA CIF pins directly in machine init function.
Same comment as to patch 1/3.
>
> Signed-off-by: Antonio Ospite <osp...@studenti.unina.it>
Even though the change seems obvious, it better be tested - in case
someone reconfigures camera pins somewhere after
pcm990_baseboard_init()... Juergen, would you be able to test it?
Thanks
Guennadi
> ---
> arch/arm/mach-pxa/pcm990-baseboard.c | 8 +-------
> 1 files changed, 1 insertions(+), 7 deletions(-)
>
> diff --git a/arch/arm/mach-pxa/pcm990-baseboard.c
> b/arch/arm/mach-pxa/pcm990-baseboard.c
> index bbda570..d5255ae 100644
> --- a/arch/arm/mach-pxa/pcm990-baseboard.c
> +++ b/arch/arm/mach-pxa/pcm990-baseboard.c
> @@ -359,19 +359,12 @@ static unsigned long pcm990_camera_pin_config[] = {
> GPIO44_CIF_LV,
> };
>
> -static int pcm990_pxacamera_init(struct device *dev)
> -{
> - pxa2xx_mfp_config(ARRAY_AND_SIZE(pcm990_camera_pin_config));
> - return 0;
> -}
> -
> /*
> * CICR4: PCLK_EN: Pixel clock is supplied by the sensor
> * MCLK_EN: Master clock is generated by PXA
> * PCP: Data sampled on the falling edge of pixel clock
> */
> struct pxacamera_platform_data pcm990_pxacamera_platform_data = {
> - .init = pcm990_pxacamera_init,
> .flags = PXA_CAMERA_MASTER | PXA_CAMERA_DATAWIDTH_8 |
> PXA_CAMERA_DATAWIDTH_10 |
> PXA_CAMERA_PCLK_EN | PXA_CAMERA_MCLK_EN/* | PXA_CAMERA_PCP*/,
> .mclk_10khz = 1000,
> @@ -532,6 +525,7 @@ void __init pcm990_baseboard_init(void)
> pxa_set_ac97_info(NULL);
>
> #if defined(CONFIG_VIDEO_PXA27x) || defined(CONFIG_VIDEO_PXA27x_MODULE)
> + pxa2xx_mfp_config(ARRAY_AND_SIZE(pcm990_camera_pin_config));
> pxa_set_camera_info(&pcm990_pxacamera_platform_data);
>
> i2c_register_board_info(0, ARRAY_AND_SIZE(pcm990_i2c_devices));
> --
> 1.6.5.2
>
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
