diff mbox series

media: Kconfig: Update help description VIDEO_RENESAS_FCP config

Message ID 20200911101046.20200-1-prabhakar.mahadev-lad.rj@bp.renesas.com
State Accepted
Commit e2d7dac34c02edb283fb34a14621ae4cb18cf38a
Headers show
Series media: Kconfig: Update help description VIDEO_RENESAS_FCP config | expand

Commit Message

Prabhakar Mahadev Lad Sept. 11, 2020, 10:10 a.m. UTC
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(-)

Comments

Laurent Pinchart Sept. 15, 2020, 9:26 p.m. UTC | #1
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
Laurent Pinchart Sept. 15, 2020, 9:30 p.m. UTC | #2
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.
diff mbox series

Patch

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.