Re: [PATCH] resistor setting sequence fix on ov772x

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Dear Guennadi

Thank you for checking patch.

> ====================
> Subject: [PATCH] ov772x: move configuration from start_capture() to set_fmt()
> soc_camera framework requires, that camera configuration is performed in
> set_fmt, and start_capture and stop_capture only turn the camera on/off.
> This patch modifies ov772x to comply to this requirement.
> ====================
> Agree? I just have no idea whether it's resistors or capacitors or 
> anything else that gest set by those i2c commands:-)

Agree. Thank you.

> One more comment to your ov772x driver: at present S_CROP is not supported 
> and it would just fail if anyone attempts to crop the image. Could you 
> please fix? You have to process the pixfmt == 0 case in set_fmt for this.

I will try to add this support.

Best regards
Kuninori Morimoto

video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@xxxxxxxxxx?subject=unsubscribe

[Linux Media]     [Older V4L]     [Linux DVB]     [Video Disk Recorder]     [Linux Kernel]     [Asterisk]     [Photo]     [DCCP]     [Netdev]     [Xorg]     [Util Linux NG]     [Xfree86]     [Free Photo Albums]     [Fedora Users]     [Fedora Women]     [ALSA Users]     [ALSA Devel]     [SSH]     [DVB Maintainers]     [Linux USB]     [Yosemite Information]

Add to Google Powered by Linux