diff mbox series

[2/2] media: imx: TODO: Remove items that are already supported

Message ID 20210826183819.105804-2-festevam@denx.de (mailing list archive)
State Accepted, archived
Delegated to: Hans Verkuil
Headers
Series [1/2] media: imx7.rst: Provide an example for imx6ull-evk capture |

Commit Message

Fabio Estevam Aug. 26, 2021, 6:38 p.m. UTC 
  Video capture has been successfully tested using an OV5640 parallel
sensor on a imx6ull-evk board at a 640x480 resolution and UYVY8_2X8
format.

Remove the two items that are no longer pending.

Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 drivers/staging/media/imx/TODO | 5 -----
 1 file changed, 5 deletions(-)

Comments

Rui Miguel Silva Aug. 27, 2021, 6:09 p.m. UTC | #1 
Oi Fabio,
On Thu Aug 26, 2021 at 7:38 PM WEST, Fabio Estevam wrote:

> Video capture has been successfully tested using an OV5640 parallel
> sensor on a imx6ull-evk board at a 640x480 resolution and UYVY8_2X8
> format.
>
> Remove the two items that are no longer pending.
>
> Signed-off-by: Fabio Estevam <festevam@denx.de>

great to see this parallel sensor worked

Acked-by: Rui Miguel Silva <rmfrfs@gmail.com>

Cheers,
  Rui

> ---
>  drivers/staging/media/imx/TODO | 5 -----
>  1 file changed, 5 deletions(-)
>
> diff --git a/drivers/staging/media/imx/TODO b/drivers/staging/media/imx/TODO
> index 2384f4c6b09d..06c94f20ecf8 100644
> --- a/drivers/staging/media/imx/TODO
> +++ b/drivers/staging/media/imx/TODO
> @@ -27,8 +27,3 @@
>  - i.MX7: all of the above, since it uses the imx media core
>  
>  - i.MX7: use Frame Interval Monitor
> -
> -- i.MX7: runtime testing with parallel sensor, links setup and streaming
> -
> -- i.MX7: runtime testing with different formats, for the time only 10-bit bayer
> -  is tested
> -- 
> 2.25.1
Laurent Pinchart Aug. 27, 2021, 9:13 p.m. UTC | #2 
Hi Fabio,

Thank you for the patch.

On Thu, Aug 26, 2021 at 03:38:19PM -0300, Fabio Estevam wrote:
> Video capture has been successfully tested using an OV5640 parallel
> sensor on a imx6ull-evk board at a 640x480 resolution and UYVY8_2X8
> format.
> 
> Remove the two items that are no longer pending.
> 
> Signed-off-by: Fabio Estevam <festevam@denx.de>

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> ---
>  drivers/staging/media/imx/TODO | 5 -----
>  1 file changed, 5 deletions(-)
> 
> diff --git a/drivers/staging/media/imx/TODO b/drivers/staging/media/imx/TODO
> index 2384f4c6b09d..06c94f20ecf8 100644
> --- a/drivers/staging/media/imx/TODO
> +++ b/drivers/staging/media/imx/TODO
> @@ -27,8 +27,3 @@
>  - i.MX7: all of the above, since it uses the imx media core
>  
>  - i.MX7: use Frame Interval Monitor
> -
> -- i.MX7: runtime testing with parallel sensor, links setup and streaming
> -
> -- i.MX7: runtime testing with different formats, for the time only 10-bit bayer
> -  is tested
diff mbox series

Patch

diff --git a/drivers/staging/media/imx/TODO b/drivers/staging/media/imx/TODO
index 2384f4c6b09d..06c94f20ecf8 100644
--- a/drivers/staging/media/imx/TODO
+++ b/drivers/staging/media/imx/TODO
@@ -27,8 +27,3 @@ 
 - i.MX7: all of the above, since it uses the imx media core
 
 - i.MX7: use Frame Interval Monitor
-
-- i.MX7: runtime testing with parallel sensor, links setup and streaming
-
-- i.MX7: runtime testing with different formats, for the time only 10-bit bayer
-  is tested