[ARM] pxa/em-x270: don't use pxa_camera init() callback
authorAntonio Ospite <ospite@studenti.unina.it>
Fri, 27 Nov 2009 20:30:23 +0000 (21:30 +0100)
committerEric Miao <eric.y.miao@gmail.com>
Tue, 1 Dec 2009 01:03:14 +0000 (09:03 +0800)
pxa_camera init() is ambiguous, it's better to statically configure the sensor.

Signed-off-by: Antonio Ospite <ospite@studenti.unina.it>
Acked-by: Mike Rapoport <mike@compulab.co.il>
Signed-off-by: Eric Miao <eric.y.miao@gmail.com>
arch/arm/mach-pxa/em-x270.c

index e0b0fda9e67e1ce45946657fcaca25ff60972f58..1c0de808b54d456e9c780146efc26b2168e0a85f 100644 (file)
@@ -967,7 +967,7 @@ static inline void em_x270_init_gpio_keys(void) {}
 #if defined(CONFIG_VIDEO_PXA27x) || defined(CONFIG_VIDEO_PXA27x_MODULE)
 static struct regulator *em_x270_camera_ldo;
 
-static int em_x270_sensor_init(struct device *dev)
+static int em_x270_sensor_init(void)
 {
        int ret;
 
@@ -996,7 +996,6 @@ static int em_x270_sensor_init(struct device *dev)
 }
 
 struct pxacamera_platform_data em_x270_camera_platform_data = {
-       .init   = em_x270_sensor_init,
        .flags  = PXA_CAMERA_MASTER | PXA_CAMERA_DATAWIDTH_8 |
                PXA_CAMERA_PCLK_EN | PXA_CAMERA_MCLK_EN,
        .mclk_10khz = 2600,
@@ -1049,8 +1048,10 @@ static struct platform_device em_x270_camera = {
 
 static void  __init em_x270_init_camera(void)
 {
-       pxa_set_camera_info(&em_x270_camera_platform_data);
-       platform_device_register(&em_x270_camera);
+       if (em_x270_sensor_init() == 0) {
+               pxa_set_camera_info(&em_x270_camera_platform_data);
+               platform_device_register(&em_x270_camera);
+       }
 }
 #else
 static inline void em_x270_init_camera(void) {}