diff mbox series

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

Message ID 20210826183819.105804-2-festevam@denx.de
State Accepted
Commit b94b551050b2aad898f87cb78c5e84d3e3e049f2
Headers show
Series [1/2] media: imx7.rst: Provide an example for imx6ull-evk capture | expand

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


-- 
Regards,

Laurent Pinchart
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