* [PATCH] media: platform: rcar_jpu, sh_vou, vsp1: Use ARCH_RENESAS
@ 2016-02-24 2:22 Simon Horman
2016-02-24 6:01 ` Laurent Pinchart
2016-02-24 7:46 ` Geert Uytterhoeven
0 siblings, 2 replies; 6+ messages in thread
From: Simon Horman @ 2016-02-24 2:22 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.
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
drivers/media/platform/Kconfig | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
Based on media_tree/master
diff --git a/drivers/media/platform/Kconfig b/drivers/media/platform/Kconfig
index 201f5c296a95..662c029400de 100644
--- a/drivers/media/platform/Kconfig
+++ b/drivers/media/platform/Kconfig
@@ -37,7 +37,7 @@ config VIDEO_SH_VOU
tristate "SuperH VOU video output driver"
depends on MEDIA_CAMERA_SUPPORT
depends on VIDEO_DEV && I2C && HAS_DMA
- depends on ARCH_SHMOBILE || COMPILE_TEST
+ depends on ARCH_RENESAS || COMPILE_TEST
select VIDEOBUF2_DMA_CONTIG
help
Support for the Video Output Unit (VOU) on SuperH SoCs.
@@ -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] 6+ messages in thread
* Re: [PATCH] media: platform: rcar_jpu, sh_vou, vsp1: Use ARCH_RENESAS
2016-02-24 2:22 [PATCH] media: platform: rcar_jpu, sh_vou, vsp1: Use ARCH_RENESAS Simon Horman
@ 2016-02-24 6:01 ` Laurent Pinchart
2016-02-24 7:46 ` Geert Uytterhoeven
1 sibling, 0 replies; 6+ messages in thread
From: Laurent Pinchart @ 2016-02-24 6:01 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 24 February 2016 11:22:22 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.
>
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> ---
> drivers/media/platform/Kconfig | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> Based on media_tree/master
>
> diff --git a/drivers/media/platform/Kconfig b/drivers/media/platform/Kconfig
> index 201f5c296a95..662c029400de 100644
> --- a/drivers/media/platform/Kconfig
> +++ b/drivers/media/platform/Kconfig
> @@ -37,7 +37,7 @@ config VIDEO_SH_VOU
> tristate "SuperH VOU video output driver"
> depends on MEDIA_CAMERA_SUPPORT
> depends on VIDEO_DEV && I2C && HAS_DMA
> - depends on ARCH_SHMOBILE || COMPILE_TEST
> + depends on ARCH_RENESAS || COMPILE_TEST
> select VIDEOBUF2_DMA_CONTIG
> help
> Support for the Video Output Unit (VOU) on SuperH SoCs.
> @@ -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] 6+ messages in thread
* Re: [PATCH] media: platform: rcar_jpu, sh_vou, vsp1: Use ARCH_RENESAS
2016-02-24 2:22 [PATCH] media: platform: rcar_jpu, sh_vou, vsp1: Use ARCH_RENESAS Simon Horman
2016-02-24 6:01 ` Laurent Pinchart
@ 2016-02-24 7:46 ` Geert Uytterhoeven
2016-02-24 23:56 ` Simon Horman
1 sibling, 1 reply; 6+ messages in thread
From: Geert Uytterhoeven @ 2016-02-24 7:46 UTC (permalink / raw)
To: Simon Horman
Cc: Mauro Carvalho Chehab, Laurent Pinchart, Mikhail Ulyanov,
Magnus Damm, Linux Media Mailing List, linux-renesas-soc
Hi Simon,
On Wed, Feb 24, 2016 at 3:22 AM, Simon Horman
<horms+renesas@verge.net.au> 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.
>
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> ---
> drivers/media/platform/Kconfig | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> Based on media_tree/master
>
> diff --git a/drivers/media/platform/Kconfig b/drivers/media/platform/Kconfig
> index 201f5c296a95..662c029400de 100644
> --- a/drivers/media/platform/Kconfig
> +++ b/drivers/media/platform/Kconfig
> @@ -37,7 +37,7 @@ config VIDEO_SH_VOU
> tristate "SuperH VOU video output driver"
> depends on MEDIA_CAMERA_SUPPORT
> depends on VIDEO_DEV && I2C && HAS_DMA
> - depends on ARCH_SHMOBILE || COMPILE_TEST
> + depends on ARCH_RENESAS || COMPILE_TEST
This driver is used on sh7722/sh7723/sh7724 only.
While these are Renesas parts, ARCH_RENESAS isn't set for SuperH SoCs,
making this driver unavailable where needed.
> select VIDEOBUF2_DMA_CONTIG
> help
> Support for the Video Output Unit (VOU) on SuperH SoCs.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] media: platform: rcar_jpu, sh_vou, vsp1: Use ARCH_RENESAS
2016-02-24 7:46 ` Geert Uytterhoeven
@ 2016-02-24 23:56 ` Simon Horman
2016-02-25 0:32 ` Simon Horman
0 siblings, 1 reply; 6+ messages in thread
From: Simon Horman @ 2016-02-24 23:56 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Mauro Carvalho Chehab, Laurent Pinchart, Mikhail Ulyanov,
Magnus Damm, Linux Media Mailing List, linux-renesas-soc
On Wed, Feb 24, 2016 at 08:46:31AM +0100, Geert Uytterhoeven wrote:
> Hi Simon,
>
> On Wed, Feb 24, 2016 at 3:22 AM, Simon Horman
> <horms+renesas@verge.net.au> 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.
> >
> > Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> > ---
> > drivers/media/platform/Kconfig | 6 +++---
> > 1 file changed, 3 insertions(+), 3 deletions(-)
> >
> > Based on media_tree/master
> >
> > diff --git a/drivers/media/platform/Kconfig b/drivers/media/platform/Kconfig
> > index 201f5c296a95..662c029400de 100644
> > --- a/drivers/media/platform/Kconfig
> > +++ b/drivers/media/platform/Kconfig
> > @@ -37,7 +37,7 @@ config VIDEO_SH_VOU
> > tristate "SuperH VOU video output driver"
> > depends on MEDIA_CAMERA_SUPPORT
> > depends on VIDEO_DEV && I2C && HAS_DMA
> > - depends on ARCH_SHMOBILE || COMPILE_TEST
> > + depends on ARCH_RENESAS || COMPILE_TEST
>
> This driver is used on sh7722/sh7723/sh7724 only.
> While these are Renesas parts, ARCH_RENESAS isn't set for SuperH SoCs,
> making this driver unavailable where needed.
Thanks for pointing that out, I had missed that detail.
Ideally I would like to stop setting ARCH_SHMOBILE for ARM Based
SoCs. So perhaps the following would be best?
depends on ARCH_SHMOBILE || ARCH_RENESAS || COMPILE_TEST
> > select VIDEOBUF2_DMA_CONTIG
> > help
> > Support for the Video Output Unit (VOU) on SuperH SoCs.
>
> Gr{oetje,eeting}s,
>
> Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
> -- Linus Torvalds
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] media: platform: rcar_jpu, sh_vou, vsp1: Use ARCH_RENESAS
2016-02-24 23:56 ` Simon Horman
@ 2016-02-25 0:32 ` Simon Horman
2016-02-25 7:57 ` Geert Uytterhoeven
0 siblings, 1 reply; 6+ messages in thread
From: Simon Horman @ 2016-02-25 0:32 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Mauro Carvalho Chehab, Laurent Pinchart, Mikhail Ulyanov,
Magnus Damm, Linux Media Mailing List, linux-renesas-soc
On Thu, Feb 25, 2016 at 08:56:22AM +0900, Simon Horman wrote:
> On Wed, Feb 24, 2016 at 08:46:31AM +0100, Geert Uytterhoeven wrote:
> > Hi Simon,
> >
> > On Wed, Feb 24, 2016 at 3:22 AM, Simon Horman
> > <horms+renesas@verge.net.au> 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.
> > >
> > > Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> > > ---
> > > drivers/media/platform/Kconfig | 6 +++---
> > > 1 file changed, 3 insertions(+), 3 deletions(-)
> > >
> > > Based on media_tree/master
> > >
> > > diff --git a/drivers/media/platform/Kconfig b/drivers/media/platform/Kconfig
> > > index 201f5c296a95..662c029400de 100644
> > > --- a/drivers/media/platform/Kconfig
> > > +++ b/drivers/media/platform/Kconfig
> > > @@ -37,7 +37,7 @@ config VIDEO_SH_VOU
> > > tristate "SuperH VOU video output driver"
> > > depends on MEDIA_CAMERA_SUPPORT
> > > depends on VIDEO_DEV && I2C && HAS_DMA
> > > - depends on ARCH_SHMOBILE || COMPILE_TEST
> > > + depends on ARCH_RENESAS || COMPILE_TEST
> >
> > This driver is used on sh7722/sh7723/sh7724 only.
> > While these are Renesas parts, ARCH_RENESAS isn't set for SuperH SoCs,
> > making this driver unavailable where needed.
>
> Thanks for pointing that out, I had missed that detail.
>
> Ideally I would like to stop setting ARCH_SHMOBILE for ARM Based
> SoCs. So perhaps the following would be best?
>
> depends on ARCH_SHMOBILE || ARCH_RENESAS || COMPILE_TEST
Sorry, I think I misread your comment. If the driver is not
used by any ARM Based SoCs then perhaps this patch should be simply
withdrawn.
> > > select VIDEOBUF2_DMA_CONTIG
> > > help
> > > Support for the Video Output Unit (VOU) on SuperH SoCs.
> >
> > Gr{oetje,eeting}s,
> >
> > Geert
> >
> > --
> > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
> >
> > In personal conversations with technical people, I call myself a hacker. But
> > when I'm talking to journalists I just say "programmer" or something like that.
> > -- Linus Torvalds
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] media: platform: rcar_jpu, sh_vou, vsp1: Use ARCH_RENESAS
2016-02-25 0:32 ` Simon Horman
@ 2016-02-25 7:57 ` Geert Uytterhoeven
0 siblings, 0 replies; 6+ messages in thread
From: Geert Uytterhoeven @ 2016-02-25 7:57 UTC (permalink / raw)
To: Simon Horman
Cc: Mauro Carvalho Chehab, Laurent Pinchart, Mikhail Ulyanov,
Magnus Damm, Linux Media Mailing List, linux-renesas-soc
Hi Simon,
On Thu, Feb 25, 2016 at 1:32 AM, Simon Horman <horms@verge.net.au> wrote:
> On Thu, Feb 25, 2016 at 08:56:22AM +0900, Simon Horman wrote:
>> On Wed, Feb 24, 2016 at 08:46:31AM +0100, Geert Uytterhoeven wrote:
>> > On Wed, Feb 24, 2016 at 3:22 AM, Simon Horman
>> > <horms+renesas@verge.net.au> 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.
>> > >
>> > > Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
>> > > ---
>> > > drivers/media/platform/Kconfig | 6 +++---
>> > > 1 file changed, 3 insertions(+), 3 deletions(-)
>> > >
>> > > Based on media_tree/master
>> > >
>> > > diff --git a/drivers/media/platform/Kconfig b/drivers/media/platform/Kconfig
>> > > index 201f5c296a95..662c029400de 100644
>> > > --- a/drivers/media/platform/Kconfig
>> > > +++ b/drivers/media/platform/Kconfig
>> > > @@ -37,7 +37,7 @@ config VIDEO_SH_VOU
>> > > tristate "SuperH VOU video output driver"
>> > > depends on MEDIA_CAMERA_SUPPORT
>> > > depends on VIDEO_DEV && I2C && HAS_DMA
>> > > - depends on ARCH_SHMOBILE || COMPILE_TEST
>> > > + depends on ARCH_RENESAS || COMPILE_TEST
>> >
>> > This driver is used on sh7722/sh7723/sh7724 only.
>> > While these are Renesas parts, ARCH_RENESAS isn't set for SuperH SoCs,
>> > making this driver unavailable where needed.
>>
>> Thanks for pointing that out, I had missed that detail.
>>
>> Ideally I would like to stop setting ARCH_SHMOBILE for ARM Based
>> SoCs. So perhaps the following would be best?
>>
>> depends on ARCH_SHMOBILE || ARCH_RENESAS || COMPILE_TEST
>
> Sorry, I think I misread your comment. If the driver is not
> used by any ARM Based SoCs then perhaps this patch should be simply
> withdrawn.
Just the first chunk. If you drop that one, you can have my
Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-02-25 7:57 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-02-24 2:22 [PATCH] media: platform: rcar_jpu, sh_vou, vsp1: Use ARCH_RENESAS Simon Horman
2016-02-24 6:01 ` Laurent Pinchart
2016-02-24 7:46 ` Geert Uytterhoeven
2016-02-24 23:56 ` Simon Horman
2016-02-25 0:32 ` Simon Horman
2016-02-25 7:57 ` Geert Uytterhoeven
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.