* [PATCH] media: Kconfig: Update help description VIDEO_RENESAS_FCP config
@ 2020-09-11 10:10 Lad Prabhakar
2020-09-11 11:33 ` Sergei Shtylyov
2020-09-15 21:26 ` Laurent Pinchart
0 siblings, 2 replies; 4+ messages in thread
From: Lad Prabhakar @ 2020-09-11 10:10 UTC (permalink / raw)
To: Geert Uytterhoeven, Laurent Pinchart, Mauro Carvalho Chehab,
Hans Verkuil, linux-media
Cc: linux-kernel, linux-renesas-soc, Lad Prabhakar, Prabhakar
rcar-fcp driver is also used on Renesas RZ/G2 SoC's, update the same
to reflect help description for VIDEO_RENESAS_FCP config.
Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
Reviewed-by: Chris Paterson <Chris.Paterson2@renesas.com>
---
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 bbf32086b607..a5716e9f463a 100644
--- a/drivers/media/platform/Kconfig
+++ b/drivers/media/platform/Kconfig
@@ -426,8 +426,8 @@ config VIDEO_RENESAS_FCP
help
This is a driver for the Renesas Frame Compression Processor (FCP).
The FCP is a companion module of video processing modules in the
- Renesas R-Car Gen3 SoCs. It handles memory access for the codec,
- VSP and FDP modules.
+ Renesas R-Car Gen3 and RZ/G2 SoCs. It handles memory access for
+ the codec, VSP and FDP modules.
To compile this driver as a module, choose M here: the module
will be called rcar-fcp.
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] media: Kconfig: Update help description VIDEO_RENESAS_FCP config
2020-09-11 10:10 [PATCH] media: Kconfig: Update help description VIDEO_RENESAS_FCP config Lad Prabhakar
@ 2020-09-11 11:33 ` Sergei Shtylyov
2020-09-15 21:26 ` Laurent Pinchart
1 sibling, 0 replies; 4+ messages in thread
From: Sergei Shtylyov @ 2020-09-11 11:33 UTC (permalink / raw)
To: Lad Prabhakar, Geert Uytterhoeven, Laurent Pinchart,
Mauro Carvalho Chehab, Hans Verkuil, linux-media
Cc: linux-kernel, linux-renesas-soc, Prabhakar
Hello!
On 9/11/20 1:10 PM, Lad Prabhakar wrote:
> rcar-fcp driver is also used on Renesas RZ/G2 SoC's, update the same
What same, the driver?
> to reflect help description for VIDEO_RENESAS_FCP config.
>
> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> Reviewed-by: Chris Paterson <Chris.Paterson2@renesas.com>
> ---
> 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 bbf32086b607..a5716e9f463a 100644
> --- a/drivers/media/platform/Kconfig
> +++ b/drivers/media/platform/Kconfig
> @@ -426,8 +426,8 @@ config VIDEO_RENESAS_FCP
> help
> This is a driver for the Renesas Frame Compression Processor (FCP).
> The FCP is a companion module of video processing modules in the
> - Renesas R-Car Gen3 SoCs. It handles memory access for the codec,
> - VSP and FDP modules.
> + Renesas R-Car Gen3 and RZ/G2 SoCs. It handles memory access for
> + the codec, VSP and FDP modules.
>
> To compile this driver as a module, choose M here: the module
> will be called rcar-fcp.
MBR, Sergei
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] media: Kconfig: Update help description VIDEO_RENESAS_FCP config
2020-09-11 10:10 [PATCH] media: Kconfig: Update help description VIDEO_RENESAS_FCP config Lad Prabhakar
2020-09-11 11:33 ` Sergei Shtylyov
@ 2020-09-15 21:26 ` Laurent Pinchart
2020-09-15 21:30 ` Laurent Pinchart
1 sibling, 1 reply; 4+ messages in thread
From: Laurent Pinchart @ 2020-09-15 21:26 UTC (permalink / raw)
To: Lad Prabhakar
Cc: Geert Uytterhoeven, Mauro Carvalho Chehab, Hans Verkuil,
linux-media, linux-kernel, linux-renesas-soc, Prabhakar
Hi Prabhakar,
Thank you for the patch.
On Fri, Sep 11, 2020 at 11:10:46AM +0100, Lad Prabhakar wrote:
> rcar-fcp driver is also used on Renesas RZ/G2 SoC's, update the same
I'm not sure what you mean by "the same" here. I'd propose
The rcar_dw_hdmi driver is also used on Renesas RZ/G2 SoCs. Update the
Kconfig entry description to reflect this.
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
If you're fine with that, there's no need to resubmit the patch.
> to reflect help description for VIDEO_RENESAS_FCP config.
>
> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> Reviewed-by: Chris Paterson <Chris.Paterson2@renesas.com>
> ---
> 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 bbf32086b607..a5716e9f463a 100644
> --- a/drivers/media/platform/Kconfig
> +++ b/drivers/media/platform/Kconfig
> @@ -426,8 +426,8 @@ config VIDEO_RENESAS_FCP
> help
> This is a driver for the Renesas Frame Compression Processor (FCP).
> The FCP is a companion module of video processing modules in the
> - Renesas R-Car Gen3 SoCs. It handles memory access for the codec,
> - VSP and FDP modules.
> + Renesas R-Car Gen3 and RZ/G2 SoCs. It handles memory access for
> + the codec, VSP and FDP modules.
>
> To compile this driver as a module, choose M here: the module
> will be called rcar-fcp.
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] media: Kconfig: Update help description VIDEO_RENESAS_FCP config
2020-09-15 21:26 ` Laurent Pinchart
@ 2020-09-15 21:30 ` Laurent Pinchart
0 siblings, 0 replies; 4+ messages in thread
From: Laurent Pinchart @ 2020-09-15 21:30 UTC (permalink / raw)
To: Lad Prabhakar
Cc: Geert Uytterhoeven, Mauro Carvalho Chehab, Hans Verkuil,
linux-media, linux-kernel, linux-renesas-soc, Prabhakar
On Wed, Sep 16, 2020 at 12:26:08AM +0300, Laurent Pinchart wrote:
> Hi Prabhakar,
>
> Thank you for the patch.
>
> On Fri, Sep 11, 2020 at 11:10:46AM +0100, Lad Prabhakar wrote:
> > rcar-fcp driver is also used on Renesas RZ/G2 SoC's, update the same
>
> I'm not sure what you mean by "the same" here. I'd propose
>
> The rcar_dw_hdmi driver is also used on Renesas RZ/G2 SoCs. Update the
> Kconfig entry description to reflect this.
And this was meant to be sent as a review of the DRM_RCAR_DW_HDMI patch
:-S For this patch, you could write
"The rcar-fcp driver is also used on Renesas RZ/G2 SoCs. Update the
Kconfig entry description to reflect this."
I would also replace the subject line with
"media: rcar-fcp: Update description for VIDEO_RENESAS_FCP Kconfig entry"
> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
>
> If you're fine with that, there's no need to resubmit the patch.
>
> > to reflect help description for VIDEO_RENESAS_FCP config.
> >
> > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> > Reviewed-by: Chris Paterson <Chris.Paterson2@renesas.com>
> > ---
> > 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 bbf32086b607..a5716e9f463a 100644
> > --- a/drivers/media/platform/Kconfig
> > +++ b/drivers/media/platform/Kconfig
> > @@ -426,8 +426,8 @@ config VIDEO_RENESAS_FCP
> > help
> > This is a driver for the Renesas Frame Compression Processor (FCP).
> > The FCP is a companion module of video processing modules in the
> > - Renesas R-Car Gen3 SoCs. It handles memory access for the codec,
> > - VSP and FDP modules.
> > + Renesas R-Car Gen3 and RZ/G2 SoCs. It handles memory access for
> > + the codec, VSP and FDP modules.
> >
> > To compile this driver as a module, choose M here: the module
> > will be called rcar-fcp.
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-09-15 21:48 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-11 10:10 [PATCH] media: Kconfig: Update help description VIDEO_RENESAS_FCP config Lad Prabhakar
2020-09-11 11:33 ` Sergei Shtylyov
2020-09-15 21:26 ` Laurent Pinchart
2020-09-15 21:30 ` Laurent Pinchart
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.