Re: [PATCH v7 0/4] DCMI bridge support

[Hugues FRUCHET]

Hi Sakari, Hans,

st-mipid02 changes are already merged, thanks Sakari and sorry for 
disturbance.

Still remain the V4L2_CID_LINK_FREQ for OV5640.


On 8/19/19 11:13 AM, Hugues FRUCHET wrote:
> Hi Hans, Sakari,
> 
> OK to push separately the 80 char fix.
> 
> There was pending related changes on st-mipid02 and ov5640 (listed 
> below), do you think it's possible to take them also ?
> 
> 
> media: st-mipid02: add support of V4L2_CID_LINK_FREQ 
> https://patchwork.linuxtv.org/patch/56969/
> StateÂÂÂ Accepted
> 
> [v2,1/3] media: st-mipid02: add support of RGB565
> https://patchwork.linuxtv.org/patch/56970/
> StateÂÂÂ Accepted
> 
> [v2,2/3] media: st-mipid02: add support of YUYV8 and UYVY8
> https://patchwork.linuxtv.org/patch/56971/
> StateÂÂÂ Accepted
> 
> [v2,3/3] media: st-mipid02: add support of JPEG 
> https://patchwork.linuxtv.org/patch/56973/
> StateÂÂÂ Accepted
> 
> 
> [v2] media: ov5640: add support of V4L2_CID_LINK_FREQ
> https://patchwork.linuxtv.org/patch/57215/
> StateÂÂÂ Changes Requested
> => This change is needed to make it work the whole setup.
> => I don't know what to change here, even if this 384MHz fixed value 
> seems strange, it works fine on my setup, on my opinion it's better than 
> nothing. We could come back on this later on when other OV5640 CSI 
> interfaces will require V4L2_CID_LINK_FREQ value.
> 
> Sakari, what do you think about this ?
> 
> 
> BR,
> Hugues.

BR,
Hugues.