On Thu, 2 Apr 2009, Darius Augulis wrote: > Guennadi Liakhovetski wrote: > > Use platform power hook to turn the camera on and off. > > > > Signed-off-by: Guennadi Liakhovetski <lg@xxxxxxx> > > --- > > diff --git a/drivers/media/video/mt9t031.c b/drivers/media/video/mt9t031.c > > index 23f9ce9..2b0927b 100644 > > --- a/drivers/media/video/mt9t031.c > > +++ b/drivers/media/video/mt9t031.c > > @@ -141,8 +141,19 @@ static int get_shutter(struct soc_camera_device *icd, > > u32 *data) > > static int mt9t031_init(struct soc_camera_device *icd) > > { > > + struct mt9t031 *mt9t031 = container_of(icd, struct mt9t031, icd); > > + struct soc_camera_link *icl = mt9t031->client->dev.platform_data; > > int ret; > > + if (icl->power) { > > + ret = icl->power(&mt9t031->client->dev, 1); > > + if (ret < 0) { > > + dev_err(icd->vdev->parent, > > + "Platform failed to power-on the camera.\n"); > > + return ret; > > + } > > + } > > probably you would have to call icl->reset there too? > I guess this camera sensor does have a reset pin? Reset is not implemented on this board, and I do not want to implement something that noone can test ATM. Thanks Guennadi > > > > + > > /* Disable chip output, synchronous option update */ > > ret = reg_write(icd, MT9T031_RESET, 1); > > if (ret >= 0) > > @@ -150,13 +161,23 @@ static int mt9t031_init(struct soc_camera_device *icd) > > if (ret >= 0) > > ret = reg_clear(icd, MT9T031_OUTPUT_CONTROL, 2); > > + if (ret < 0 && icl->power) > > + icl->power(&mt9t031->client->dev, 0); > > + > > return ret >= 0 ? 0 : -EIO; > > } > > static int mt9t031_release(struct soc_camera_device *icd) > > { > > + struct mt9t031 *mt9t031 = container_of(icd, struct mt9t031, icd); > > + struct soc_camera_link *icl = mt9t031->client->dev.platform_data; > > + > > /* Disable the chip */ > > reg_clear(icd, MT9T031_OUTPUT_CONTROL, 2); > > + > > + if (icl->power) > > + icl->power(&mt9t031->client->dev, 0); > > + > > return 0; > > } > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-media" in > > the body of a message to majordomo@xxxxxxxxxxxxxxx > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > > > > --- Guennadi Liakhovetski, Ph.D. DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office@xxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html