public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] media: platform: rcar_jpu, sh_vou, vsp1: Use ARCH_RENESAS
@ 2016-03-02  1:14 Simon Horman
  2016-03-02 11:00 ` Laurent Pinchart
  2016-03-02 13:17 ` Sergei Shtylyov
  0 siblings, 2 replies; 5+ messages in thread
From: Simon Horman @ 2016-03-02  1:14 UTC (permalink / raw)
  To: Mauro Carvalho Chehab
  Cc: Laurent Pinchart, Mikhail Ulyanov, Magnus Damm, linux-media,
	linux-renesas-soc, Simon Horman

Make use of ARCH_RENESAS in place of ARCH_SHMOBILE.

This is part of an ongoing process to migrate from ARCH_SHMOBILE to
ARCH_RENESAS the motivation for which being that RENESAS seems to be a more
appropriate name than SHMOBILE for the majority of Renesas ARM based SoCs.

Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
Based on media-tree/master

v2
* Do not update VIDEO_SH_VOU to use ARCH_RENESAS as this is
  used by some SH-based platforms and is not used by any ARM-based platforms
  so a dependency on ARCH_SHMOBILE is correct for that driver
* Added Geert Uytterhoeven's Ack
---
 drivers/media/platform/Kconfig | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/media/platform/Kconfig b/drivers/media/platform/Kconfig
index 201f5c296a95..84e041c0a70e 100644
--- a/drivers/media/platform/Kconfig
+++ b/drivers/media/platform/Kconfig
@@ -238,7 +238,7 @@ config VIDEO_SH_VEU
 config VIDEO_RENESAS_JPU
 	tristate "Renesas JPEG Processing Unit"
 	depends on VIDEO_DEV && VIDEO_V4L2 && HAS_DMA
-	depends on ARCH_SHMOBILE || COMPILE_TEST
+	depends on ARCH_RENESAS || COMPILE_TEST
 	select VIDEOBUF2_DMA_CONTIG
 	select V4L2_MEM2MEM_DEV
 	---help---
@@ -250,7 +250,7 @@ config VIDEO_RENESAS_JPU
 config VIDEO_RENESAS_VSP1
 	tristate "Renesas VSP1 Video Processing Engine"
 	depends on VIDEO_V4L2 && VIDEO_V4L2_SUBDEV_API && HAS_DMA
-	depends on (ARCH_SHMOBILE && OF) || COMPILE_TEST
+	depends on (ARCH_RENESAS && OF) || COMPILE_TEST
 	select VIDEOBUF2_DMA_CONTIG
 	---help---
 	  This is a V4L2 driver for the Renesas VSP1 video processing engine.
-- 
2.1.4


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH v2] media: platform: rcar_jpu, sh_vou, vsp1: Use ARCH_RENESAS
  2016-03-02  1:14 [PATCH v2] media: platform: rcar_jpu, sh_vou, vsp1: Use ARCH_RENESAS Simon Horman
@ 2016-03-02 11:00 ` Laurent Pinchart
  2016-03-02 11:32   ` Hans Verkuil
  2016-03-02 13:17 ` Sergei Shtylyov
  1 sibling, 1 reply; 5+ messages in thread
From: Laurent Pinchart @ 2016-03-02 11:00 UTC (permalink / raw)
  To: Simon Horman
  Cc: Mauro Carvalho Chehab, Mikhail Ulyanov, Magnus Damm, linux-media,
	linux-renesas-soc

Hi Simon,

Thank you for the patch.

On Wednesday 02 March 2016 10:14:51 Simon Horman wrote:
> Make use of ARCH_RENESAS in place of ARCH_SHMOBILE.
> 
> This is part of an ongoing process to migrate from ARCH_SHMOBILE to
> ARCH_RENESAS the motivation for which being that RENESAS seems to be a more
> appropriate name than SHMOBILE for the majority of Renesas ARM based SoCs.
> 
> Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>

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

> ---
> Based on media-tree/master
> 
> v2
> * Do not update VIDEO_SH_VOU to use ARCH_RENESAS as this is
>   used by some SH-based platforms and is not used by any ARM-based platforms
> so a dependency on ARCH_SHMOBILE is correct for that driver
> * Added Geert Uytterhoeven's Ack
> ---
>  drivers/media/platform/Kconfig | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/media/platform/Kconfig b/drivers/media/platform/Kconfig
> index 201f5c296a95..84e041c0a70e 100644
> --- a/drivers/media/platform/Kconfig
> +++ b/drivers/media/platform/Kconfig
> @@ -238,7 +238,7 @@ config VIDEO_SH_VEU
>  config VIDEO_RENESAS_JPU
>  	tristate "Renesas JPEG Processing Unit"
>  	depends on VIDEO_DEV && VIDEO_V4L2 && HAS_DMA
> -	depends on ARCH_SHMOBILE || COMPILE_TEST
> +	depends on ARCH_RENESAS || COMPILE_TEST
>  	select VIDEOBUF2_DMA_CONTIG
>  	select V4L2_MEM2MEM_DEV
>  	---help---
> @@ -250,7 +250,7 @@ config VIDEO_RENESAS_JPU
>  config VIDEO_RENESAS_VSP1
>  	tristate "Renesas VSP1 Video Processing Engine"
>  	depends on VIDEO_V4L2 && VIDEO_V4L2_SUBDEV_API && HAS_DMA
> -	depends on (ARCH_SHMOBILE && OF) || COMPILE_TEST
> +	depends on (ARCH_RENESAS && OF) || COMPILE_TEST
>  	select VIDEOBUF2_DMA_CONTIG
>  	---help---
>  	  This is a V4L2 driver for the Renesas VSP1 video processing engine.

-- 
Regards,

Laurent Pinchart


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH v2] media: platform: rcar_jpu, sh_vou, vsp1: Use ARCH_RENESAS
  2016-03-02 11:00 ` Laurent Pinchart
@ 2016-03-02 11:32   ` Hans Verkuil
  2016-03-03  0:25     ` Simon Horman
  0 siblings, 1 reply; 5+ messages in thread
From: Hans Verkuil @ 2016-03-02 11:32 UTC (permalink / raw)
  To: Laurent Pinchart, Simon Horman
  Cc: Mauro Carvalho Chehab, Mikhail Ulyanov, Magnus Damm, linux-media,
	linux-renesas-soc

Hi Simon,

Note that the patch subject still mentions sh_vou.

Otherwise:

Acked-by: Hans Verkuil <hans.verkuil@cisco.com>

Regards,

	Hans

On 03/02/16 12:00, Laurent Pinchart wrote:
> Hi Simon,
> 
> Thank you for the patch.
> 
> On Wednesday 02 March 2016 10:14:51 Simon Horman wrote:
>> Make use of ARCH_RENESAS in place of ARCH_SHMOBILE.
>>
>> This is part of an ongoing process to migrate from ARCH_SHMOBILE to
>> ARCH_RENESAS the motivation for which being that RENESAS seems to be a more
>> appropriate name than SHMOBILE for the majority of Renesas ARM based SoCs.
>>
>> Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
>> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> 
> Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> 
>> ---
>> Based on media-tree/master
>>
>> v2
>> * Do not update VIDEO_SH_VOU to use ARCH_RENESAS as this is
>>   used by some SH-based platforms and is not used by any ARM-based platforms
>> so a dependency on ARCH_SHMOBILE is correct for that driver
>> * Added Geert Uytterhoeven's Ack
>> ---
>>  drivers/media/platform/Kconfig | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/media/platform/Kconfig b/drivers/media/platform/Kconfig
>> index 201f5c296a95..84e041c0a70e 100644
>> --- a/drivers/media/platform/Kconfig
>> +++ b/drivers/media/platform/Kconfig
>> @@ -238,7 +238,7 @@ config VIDEO_SH_VEU
>>  config VIDEO_RENESAS_JPU
>>  	tristate "Renesas JPEG Processing Unit"
>>  	depends on VIDEO_DEV && VIDEO_V4L2 && HAS_DMA
>> -	depends on ARCH_SHMOBILE || COMPILE_TEST
>> +	depends on ARCH_RENESAS || COMPILE_TEST
>>  	select VIDEOBUF2_DMA_CONTIG
>>  	select V4L2_MEM2MEM_DEV
>>  	---help---
>> @@ -250,7 +250,7 @@ config VIDEO_RENESAS_JPU
>>  config VIDEO_RENESAS_VSP1
>>  	tristate "Renesas VSP1 Video Processing Engine"
>>  	depends on VIDEO_V4L2 && VIDEO_V4L2_SUBDEV_API && HAS_DMA
>> -	depends on (ARCH_SHMOBILE && OF) || COMPILE_TEST
>> +	depends on (ARCH_RENESAS && OF) || COMPILE_TEST
>>  	select VIDEOBUF2_DMA_CONTIG
>>  	---help---
>>  	  This is a V4L2 driver for the Renesas VSP1 video processing engine.
> 

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH v2] media: platform: rcar_jpu, sh_vou, vsp1: Use ARCH_RENESAS
  2016-03-02  1:14 [PATCH v2] media: platform: rcar_jpu, sh_vou, vsp1: Use ARCH_RENESAS Simon Horman
  2016-03-02 11:00 ` Laurent Pinchart
@ 2016-03-02 13:17 ` Sergei Shtylyov
  1 sibling, 0 replies; 5+ messages in thread
From: Sergei Shtylyov @ 2016-03-02 13:17 UTC (permalink / raw)
  To: Simon Horman, Mauro Carvalho Chehab
  Cc: Laurent Pinchart, Mikhail Ulyanov, Magnus Damm, linux-media,
	linux-renesas-soc

Hello.

On 3/2/2016 4:14 AM, Simon Horman wrote:

> Make use of ARCH_RENESAS in place of ARCH_SHMOBILE.
>
> This is part of an ongoing process to migrate from ARCH_SHMOBILE to
> ARCH_RENESAS the motivation for which being that RENESAS seems to be a more
> appropriate name than SHMOBILE for the majority of Renesas ARM based SoCs.
>
> Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> ---
> Based on media-tree/master
>
> v2
> * Do not update VIDEO_SH_VOU to use ARCH_RENESAS as this is
>    used by some SH-based platforms and is not used by any ARM-based platforms
>    so a dependency on ARCH_SHMOBILE is correct for that driver

    You forgot to remove it from the subject though.

> * Added Geert Uytterhoeven's Ack
[...]

MBR, Sergei


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH v2] media: platform: rcar_jpu, sh_vou, vsp1: Use ARCH_RENESAS
  2016-03-02 11:32   ` Hans Verkuil
@ 2016-03-03  0:25     ` Simon Horman
  0 siblings, 0 replies; 5+ messages in thread
From: Simon Horman @ 2016-03-03  0:25 UTC (permalink / raw)
  To: Hans Verkuil, Sergei Shtylyov
  Cc: Laurent Pinchart, Mauro Carvalho Chehab, Mikhail Ulyanov,
	Magnus Damm, linux-media, linux-renesas-soc

On Wed, Mar 02, 2016 at 12:32:39PM +0100, Hans Verkuil wrote:
> Hi Simon,
> 
> Note that the patch subject still mentions sh_vou.
> 
> Otherwise:
> 
> Acked-by: Hans Verkuil <hans.verkuil@cisco.com>

[snip]

On Wed, Mar 02, 2016 at 04:17:10PM +0300, Sergei Shtylyov wrote:

[snip]

> >v2
> >* Do not update VIDEO_SH_VOU to use ARCH_RENESAS as this is
> >   used by some SH-based platforms and is not used by any ARM-based platforms
> >   so a dependency on ARCH_SHMOBILE is correct for that driver
> 
>    You forgot to remove it from the subject though.

[snip]


Thanks, I have posted v3 with sh_vou removed from the subject line.


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2016-03-03  0:25 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-03-02  1:14 [PATCH v2] media: platform: rcar_jpu, sh_vou, vsp1: Use ARCH_RENESAS Simon Horman
2016-03-02 11:00 ` Laurent Pinchart
2016-03-02 11:32   ` Hans Verkuil
2016-03-03  0:25     ` Simon Horman
2016-03-02 13:17 ` Sergei Shtylyov

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox