Re: [PATCH 5/8] v4l: Switch from V4L2 OF not V4L2 fwnode API

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

 



Hi Benoit,

On Mon, Feb 13, 2017 at 10:57:47AM -0600, Benoit Parrot wrote:
> Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> wrote on Mon [2017-Feb-13 15:28:13 +0200]:
...
> > @@ -421,6 +423,7 @@ config VIDEO_TI_VPE
> >  	select VIDEO_TI_VPDMA
> >  	select VIDEO_TI_SC
> >  	select VIDEO_TI_CSC
> > +	select V4L2_FWNODE
> 
> Sakari,
> 
> TI_VPE does not use async registration, but as you already saw TI_CAL does.
> So adding "select V4L2_FWNODE" should be moved to the VIDEO_TI_CAL section.
> Once that's fixed you can add my acked-by for both the TI_CAL and AM437x_VPFE.

Thanks for the review and for spotting this! I think I just used the name
of the directory, and assumed that the matching Kconfig option is the right
one. There don't seem to be other cases such as that one.

diff:

diff --git a/drivers/media/platform/Kconfig b/drivers/media/platform/Kconfig
index cced276..2f7792c 100644
--- a/drivers/media/platform/Kconfig
+++ b/drivers/media/platform/Kconfig
@@ -129,6 +129,7 @@ config VIDEO_TI_CAL
 	depends on SOC_DRA7XX || COMPILE_TEST
 	depends on HAS_DMA
 	select VIDEOBUF2_DMA_CONTIG
+	select V4L2_FWNODE
 	default n
 	---help---
 	  Support for the TI CAL (Camera Adaptation Layer) block
@@ -423,7 +424,6 @@ config VIDEO_TI_VPE
 	select VIDEO_TI_VPDMA
 	select VIDEO_TI_SC
 	select VIDEO_TI_CSC
-	select V4L2_FWNODE
 	default n
 	---help---
 	  Support for the TI VPE(Video Processing Engine) block

-- 
Kind regards,

Sakari Ailus
e-mail: sakari.ailus@xxxxxx	XMPP: sailus@xxxxxxxxxxxxxx
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux