From: Laurent Pinchart <laurent.pinchart@ideasonboard.com> To: Biju Das <biju.das.jz@bp.renesas.com> Cc: David Airlie <airlied@linux.ie>, Daniel Vetter <daniel@ffwll.ch>, Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>, dri-devel@lists.freedesktop.org, linux-renesas-soc@vger.kernel.org, Geert Uytterhoeven <geert+renesas@glider.be>, Chris Paterson <Chris.Paterson2@renesas.com>, Biju Das <biju.das@bp.renesas.com>, Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@bp.renesas.com> Subject: Re: [PATCH v2 3/7] drm: rcar-du: Add max_width and max_height to struct rcar_du_device_info Date: Fri, 15 Apr 2022 14:20:04 +0300 [thread overview] Message-ID: <YllU5CReP7Y6iRVE@pendragon.ideasonboard.com> (raw) In-Reply-To: <20220316131100.30685-4-biju.das.jz@bp.renesas.com> Hi Biju, Thank you for the patch. On Wed, Mar 16, 2022 at 01:10:56PM +0000, Biju Das wrote: > There are some differences related to max frame size supported by different > R-Car/RZ-G family of SoC's > > Max frame size supported by R-Car Gen1 & R-Car Gen2 is 4095x2047 > Max frame size supported by R-Car Gen3 is 8190x8190 > Max frame size supported by RZ/G2L is 1920x1080 > > Add max_width and max_height to struct rcar_du_device_info to support later > SoC without any code changes. > > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com> > --- > v1->v2: > * No Change > RFC->V1: > * No Change > RFC: > * https://patchwork.kernel.org/project/linux-renesas-soc/patch/20220112174612.10773-13-biju.das.jz@bp.renesas.com/ > --- > drivers/gpu/drm/rcar-du/rcar_du_drv.c | 36 +++++++++++++++++++++++++++ > drivers/gpu/drm/rcar-du/rcar_du_drv.h | 4 +++ > drivers/gpu/drm/rcar-du/rcar_du_kms.c | 17 +++++-------- > 3 files changed, 46 insertions(+), 11 deletions(-) > > diff --git a/drivers/gpu/drm/rcar-du/rcar_du_drv.c b/drivers/gpu/drm/rcar-du/rcar_du_drv.c > index 1bc7325aa356..4640c356a532 100644 > --- a/drivers/gpu/drm/rcar-du/rcar_du_drv.c > +++ b/drivers/gpu/drm/rcar-du/rcar_du_drv.c > @@ -56,6 +56,8 @@ static const struct rcar_du_device_info rzg1_du_r8a7743_info = { > }, > .num_lvds = 1, > .num_rpf = 4, > + .max_width = 4095, > + .max_height = 2047, > }; > > static const struct rcar_du_device_info rzg1_du_r8a7745_info = { > @@ -79,6 +81,8 @@ static const struct rcar_du_device_info rzg1_du_r8a7745_info = { > }, > }, > .num_rpf = 4, > + .max_width = 4095, > + .max_height = 2047, > }; > > static const struct rcar_du_device_info rzg1_du_r8a77470_info = { > @@ -107,6 +111,8 @@ static const struct rcar_du_device_info rzg1_du_r8a77470_info = { > }, > }, > .num_rpf = 4, > + .max_width = 4095, > + .max_height = 2047, > }; > > static const struct rcar_du_device_info rcar_du_r8a774a1_info = { > @@ -137,6 +143,8 @@ static const struct rcar_du_device_info rcar_du_r8a774a1_info = { > }, > .num_lvds = 1, > .num_rpf = 5, > + .max_width = 8190, > + .max_height = 8190, > .dpll_mask = BIT(1), > }; > > @@ -168,6 +176,8 @@ static const struct rcar_du_device_info rcar_du_r8a774b1_info = { > }, > .num_lvds = 1, > .num_rpf = 5, > + .max_width = 8190, > + .max_height = 8190, > .dpll_mask = BIT(1), > }; > > @@ -196,6 +206,8 @@ static const struct rcar_du_device_info rcar_du_r8a774c0_info = { > }, > .num_lvds = 2, > .num_rpf = 4, > + .max_width = 8190, > + .max_height = 8190, > .lvds_clk_mask = BIT(1) | BIT(0), > }; > > @@ -227,6 +239,8 @@ static const struct rcar_du_device_info rcar_du_r8a774e1_info = { > }, > .num_lvds = 1, > .num_rpf = 5, > + .max_width = 8190, > + .max_height = 8190, > .dpll_mask = BIT(1), > }; > > @@ -249,6 +263,8 @@ static const struct rcar_du_device_info rcar_du_r8a7779_info = { > .port = 1, > }, > }, > + .max_width = 4095, > + .max_height = 2047, > }; > > static const struct rcar_du_device_info rcar_du_r8a7790_info = { > @@ -280,6 +296,8 @@ static const struct rcar_du_device_info rcar_du_r8a7790_info = { > }, > .num_lvds = 2, > .num_rpf = 4, > + .max_width = 4095, > + .max_height = 2047, > }; > > /* M2-W (r8a7791) and M2-N (r8a7793) are identical */ > @@ -306,6 +324,8 @@ static const struct rcar_du_device_info rcar_du_r8a7791_info = { > }, > .num_lvds = 1, > .num_rpf = 4, > + .max_width = 4095, > + .max_height = 2047, > }; > > static const struct rcar_du_device_info rcar_du_r8a7792_info = { > @@ -327,6 +347,8 @@ static const struct rcar_du_device_info rcar_du_r8a7792_info = { > }, > }, > .num_rpf = 4, > + .max_width = 4095, > + .max_height = 2047, > }; > > static const struct rcar_du_device_info rcar_du_r8a7794_info = { > @@ -351,6 +373,8 @@ static const struct rcar_du_device_info rcar_du_r8a7794_info = { > }, > }, > .num_rpf = 4, > + .max_width = 4095, > + .max_height = 2047, > }; > > static const struct rcar_du_device_info rcar_du_r8a7795_info = { > @@ -385,6 +409,8 @@ static const struct rcar_du_device_info rcar_du_r8a7795_info = { > }, > .num_lvds = 1, > .num_rpf = 5, > + .max_width = 8190, > + .max_height = 8190, > .dpll_mask = BIT(2) | BIT(1), > }; > > @@ -416,6 +442,8 @@ static const struct rcar_du_device_info rcar_du_r8a7796_info = { > }, > .num_lvds = 1, > .num_rpf = 5, > + .max_width = 8190, > + .max_height = 8190, > .dpll_mask = BIT(1), > }; > > @@ -447,6 +475,8 @@ static const struct rcar_du_device_info rcar_du_r8a77965_info = { > }, > .num_lvds = 1, > .num_rpf = 5, > + .max_width = 8190, > + .max_height = 8190, > .dpll_mask = BIT(1), > }; > > @@ -474,6 +504,8 @@ static const struct rcar_du_device_info rcar_du_r8a77970_info = { > }, > .num_lvds = 1, > .num_rpf = 5, > + .max_width = 8190, > + .max_height = 8190, > }; > > static const struct rcar_du_device_info rcar_du_r8a7799x_info = { > @@ -502,6 +534,8 @@ static const struct rcar_du_device_info rcar_du_r8a7799x_info = { > }, > .num_lvds = 2, > .num_rpf = 5, > + .max_width = 8190, > + .max_height = 8190, > .lvds_clk_mask = BIT(1) | BIT(0), > }; > > @@ -522,6 +556,8 @@ static const struct rcar_du_device_info rcar_du_r8a779a0_info = { > }, > }, > .num_rpf = 5, > + .max_width = 8190, > + .max_height = 8190, > .dsi_clk_mask = BIT(1) | BIT(0), > }; > > diff --git a/drivers/gpu/drm/rcar-du/rcar_du_drv.h b/drivers/gpu/drm/rcar-du/rcar_du_drv.h > index 68c5de59d18d..b0553b43363b 100644 > --- a/drivers/gpu/drm/rcar-du/rcar_du_drv.h > +++ b/drivers/gpu/drm/rcar-du/rcar_du_drv.h > @@ -70,6 +70,8 @@ struct rcar_du_output_routing { > * @routes: array of CRTC to output routes, indexed by output (RCAR_DU_OUTPUT_*) > * @num_lvds: number of internal LVDS encoders > * @num_rpf: max number of RPFs in VSP > + * @max_width: max frame width > + * @max_height: max frame height > * @dpll_mask: bit mask of DU channels equipped with a DPLL > * @dsi_clk_mask: bitmask of channels that can use the DSI clock as dot clock > * @lvds_clk_mask: bitmask of channels that can use the LVDS clock as dot clock > @@ -82,6 +84,8 @@ struct rcar_du_device_info { > struct rcar_du_output_routing routes[RCAR_DU_OUTPUT_MAX]; > unsigned int num_lvds; > unsigned int num_rpf; > + unsigned int max_width; > + unsigned int max_height; > unsigned int dpll_mask; > unsigned int dsi_clk_mask; > unsigned int lvds_clk_mask; > diff --git a/drivers/gpu/drm/rcar-du/rcar_du_kms.c b/drivers/gpu/drm/rcar-du/rcar_du_kms.c > index 190dbb7f15dd..5857705aac20 100644 > --- a/drivers/gpu/drm/rcar-du/rcar_du_kms.c > +++ b/drivers/gpu/drm/rcar-du/rcar_du_kms.c > @@ -834,17 +834,12 @@ int rcar_du_modeset_init(struct rcar_du_device *rcdu) > dev->mode_config.funcs = &rcar_du_mode_config_funcs; > dev->mode_config.helper_private = &rcar_du_mode_config_helper; > > - if (rcdu->info->gen < 3) { > - dev->mode_config.max_width = 4095; > - dev->mode_config.max_height = 2047; > - } else { > - /* > - * The Gen3 DU uses the VSP1 for memory access, and is limited > - * to frame sizes of 8190x8190. > - */ > - dev->mode_config.max_width = 8190; > - dev->mode_config.max_height = 8190; > - } > + /* > + * The Gen3 DU uses the VSP1 for memory access, and is limited > + * to frame sizes of 8190x8190. > + */ This comment doesn't make much sense here anymore, but I think it's useful to keep the information. How about defining macros for the maximum width and height on different platforms, and moving the comment just before the macro definitions ? > + dev->mode_config.max_width = rcdu->info->max_width; > + dev->mode_config.max_height = rcdu->info->max_height; > > rcdu->num_crtcs = hweight8(rcdu->info->channels_mask); > -- Regards, Laurent Pinchart
WARNING: multiple messages have this Message-ID (diff)
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com> To: Biju Das <biju.das.jz@bp.renesas.com> Cc: Chris Paterson <Chris.Paterson2@renesas.com>, Geert Uytterhoeven <geert+renesas@glider.be>, David Airlie <airlied@linux.ie>, Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@bp.renesas.com>, dri-devel@lists.freedesktop.org, Biju Das <biju.das@bp.renesas.com>, linux-renesas-soc@vger.kernel.org, Kieran Bingham <kieran.bingham+renesas@ideasonboard.com> Subject: Re: [PATCH v2 3/7] drm: rcar-du: Add max_width and max_height to struct rcar_du_device_info Date: Fri, 15 Apr 2022 14:20:04 +0300 [thread overview] Message-ID: <YllU5CReP7Y6iRVE@pendragon.ideasonboard.com> (raw) In-Reply-To: <20220316131100.30685-4-biju.das.jz@bp.renesas.com> Hi Biju, Thank you for the patch. On Wed, Mar 16, 2022 at 01:10:56PM +0000, Biju Das wrote: > There are some differences related to max frame size supported by different > R-Car/RZ-G family of SoC's > > Max frame size supported by R-Car Gen1 & R-Car Gen2 is 4095x2047 > Max frame size supported by R-Car Gen3 is 8190x8190 > Max frame size supported by RZ/G2L is 1920x1080 > > Add max_width and max_height to struct rcar_du_device_info to support later > SoC without any code changes. > > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com> > --- > v1->v2: > * No Change > RFC->V1: > * No Change > RFC: > * https://patchwork.kernel.org/project/linux-renesas-soc/patch/20220112174612.10773-13-biju.das.jz@bp.renesas.com/ > --- > drivers/gpu/drm/rcar-du/rcar_du_drv.c | 36 +++++++++++++++++++++++++++ > drivers/gpu/drm/rcar-du/rcar_du_drv.h | 4 +++ > drivers/gpu/drm/rcar-du/rcar_du_kms.c | 17 +++++-------- > 3 files changed, 46 insertions(+), 11 deletions(-) > > diff --git a/drivers/gpu/drm/rcar-du/rcar_du_drv.c b/drivers/gpu/drm/rcar-du/rcar_du_drv.c > index 1bc7325aa356..4640c356a532 100644 > --- a/drivers/gpu/drm/rcar-du/rcar_du_drv.c > +++ b/drivers/gpu/drm/rcar-du/rcar_du_drv.c > @@ -56,6 +56,8 @@ static const struct rcar_du_device_info rzg1_du_r8a7743_info = { > }, > .num_lvds = 1, > .num_rpf = 4, > + .max_width = 4095, > + .max_height = 2047, > }; > > static const struct rcar_du_device_info rzg1_du_r8a7745_info = { > @@ -79,6 +81,8 @@ static const struct rcar_du_device_info rzg1_du_r8a7745_info = { > }, > }, > .num_rpf = 4, > + .max_width = 4095, > + .max_height = 2047, > }; > > static const struct rcar_du_device_info rzg1_du_r8a77470_info = { > @@ -107,6 +111,8 @@ static const struct rcar_du_device_info rzg1_du_r8a77470_info = { > }, > }, > .num_rpf = 4, > + .max_width = 4095, > + .max_height = 2047, > }; > > static const struct rcar_du_device_info rcar_du_r8a774a1_info = { > @@ -137,6 +143,8 @@ static const struct rcar_du_device_info rcar_du_r8a774a1_info = { > }, > .num_lvds = 1, > .num_rpf = 5, > + .max_width = 8190, > + .max_height = 8190, > .dpll_mask = BIT(1), > }; > > @@ -168,6 +176,8 @@ static const struct rcar_du_device_info rcar_du_r8a774b1_info = { > }, > .num_lvds = 1, > .num_rpf = 5, > + .max_width = 8190, > + .max_height = 8190, > .dpll_mask = BIT(1), > }; > > @@ -196,6 +206,8 @@ static const struct rcar_du_device_info rcar_du_r8a774c0_info = { > }, > .num_lvds = 2, > .num_rpf = 4, > + .max_width = 8190, > + .max_height = 8190, > .lvds_clk_mask = BIT(1) | BIT(0), > }; > > @@ -227,6 +239,8 @@ static const struct rcar_du_device_info rcar_du_r8a774e1_info = { > }, > .num_lvds = 1, > .num_rpf = 5, > + .max_width = 8190, > + .max_height = 8190, > .dpll_mask = BIT(1), > }; > > @@ -249,6 +263,8 @@ static const struct rcar_du_device_info rcar_du_r8a7779_info = { > .port = 1, > }, > }, > + .max_width = 4095, > + .max_height = 2047, > }; > > static const struct rcar_du_device_info rcar_du_r8a7790_info = { > @@ -280,6 +296,8 @@ static const struct rcar_du_device_info rcar_du_r8a7790_info = { > }, > .num_lvds = 2, > .num_rpf = 4, > + .max_width = 4095, > + .max_height = 2047, > }; > > /* M2-W (r8a7791) and M2-N (r8a7793) are identical */ > @@ -306,6 +324,8 @@ static const struct rcar_du_device_info rcar_du_r8a7791_info = { > }, > .num_lvds = 1, > .num_rpf = 4, > + .max_width = 4095, > + .max_height = 2047, > }; > > static const struct rcar_du_device_info rcar_du_r8a7792_info = { > @@ -327,6 +347,8 @@ static const struct rcar_du_device_info rcar_du_r8a7792_info = { > }, > }, > .num_rpf = 4, > + .max_width = 4095, > + .max_height = 2047, > }; > > static const struct rcar_du_device_info rcar_du_r8a7794_info = { > @@ -351,6 +373,8 @@ static const struct rcar_du_device_info rcar_du_r8a7794_info = { > }, > }, > .num_rpf = 4, > + .max_width = 4095, > + .max_height = 2047, > }; > > static const struct rcar_du_device_info rcar_du_r8a7795_info = { > @@ -385,6 +409,8 @@ static const struct rcar_du_device_info rcar_du_r8a7795_info = { > }, > .num_lvds = 1, > .num_rpf = 5, > + .max_width = 8190, > + .max_height = 8190, > .dpll_mask = BIT(2) | BIT(1), > }; > > @@ -416,6 +442,8 @@ static const struct rcar_du_device_info rcar_du_r8a7796_info = { > }, > .num_lvds = 1, > .num_rpf = 5, > + .max_width = 8190, > + .max_height = 8190, > .dpll_mask = BIT(1), > }; > > @@ -447,6 +475,8 @@ static const struct rcar_du_device_info rcar_du_r8a77965_info = { > }, > .num_lvds = 1, > .num_rpf = 5, > + .max_width = 8190, > + .max_height = 8190, > .dpll_mask = BIT(1), > }; > > @@ -474,6 +504,8 @@ static const struct rcar_du_device_info rcar_du_r8a77970_info = { > }, > .num_lvds = 1, > .num_rpf = 5, > + .max_width = 8190, > + .max_height = 8190, > }; > > static const struct rcar_du_device_info rcar_du_r8a7799x_info = { > @@ -502,6 +534,8 @@ static const struct rcar_du_device_info rcar_du_r8a7799x_info = { > }, > .num_lvds = 2, > .num_rpf = 5, > + .max_width = 8190, > + .max_height = 8190, > .lvds_clk_mask = BIT(1) | BIT(0), > }; > > @@ -522,6 +556,8 @@ static const struct rcar_du_device_info rcar_du_r8a779a0_info = { > }, > }, > .num_rpf = 5, > + .max_width = 8190, > + .max_height = 8190, > .dsi_clk_mask = BIT(1) | BIT(0), > }; > > diff --git a/drivers/gpu/drm/rcar-du/rcar_du_drv.h b/drivers/gpu/drm/rcar-du/rcar_du_drv.h > index 68c5de59d18d..b0553b43363b 100644 > --- a/drivers/gpu/drm/rcar-du/rcar_du_drv.h > +++ b/drivers/gpu/drm/rcar-du/rcar_du_drv.h > @@ -70,6 +70,8 @@ struct rcar_du_output_routing { > * @routes: array of CRTC to output routes, indexed by output (RCAR_DU_OUTPUT_*) > * @num_lvds: number of internal LVDS encoders > * @num_rpf: max number of RPFs in VSP > + * @max_width: max frame width > + * @max_height: max frame height > * @dpll_mask: bit mask of DU channels equipped with a DPLL > * @dsi_clk_mask: bitmask of channels that can use the DSI clock as dot clock > * @lvds_clk_mask: bitmask of channels that can use the LVDS clock as dot clock > @@ -82,6 +84,8 @@ struct rcar_du_device_info { > struct rcar_du_output_routing routes[RCAR_DU_OUTPUT_MAX]; > unsigned int num_lvds; > unsigned int num_rpf; > + unsigned int max_width; > + unsigned int max_height; > unsigned int dpll_mask; > unsigned int dsi_clk_mask; > unsigned int lvds_clk_mask; > diff --git a/drivers/gpu/drm/rcar-du/rcar_du_kms.c b/drivers/gpu/drm/rcar-du/rcar_du_kms.c > index 190dbb7f15dd..5857705aac20 100644 > --- a/drivers/gpu/drm/rcar-du/rcar_du_kms.c > +++ b/drivers/gpu/drm/rcar-du/rcar_du_kms.c > @@ -834,17 +834,12 @@ int rcar_du_modeset_init(struct rcar_du_device *rcdu) > dev->mode_config.funcs = &rcar_du_mode_config_funcs; > dev->mode_config.helper_private = &rcar_du_mode_config_helper; > > - if (rcdu->info->gen < 3) { > - dev->mode_config.max_width = 4095; > - dev->mode_config.max_height = 2047; > - } else { > - /* > - * The Gen3 DU uses the VSP1 for memory access, and is limited > - * to frame sizes of 8190x8190. > - */ > - dev->mode_config.max_width = 8190; > - dev->mode_config.max_height = 8190; > - } > + /* > + * The Gen3 DU uses the VSP1 for memory access, and is limited > + * to frame sizes of 8190x8190. > + */ This comment doesn't make much sense here anymore, but I think it's useful to keep the information. How about defining macros for the maximum width and height on different platforms, and moving the comment just before the macro definitions ? > + dev->mode_config.max_width = rcdu->info->max_width; > + dev->mode_config.max_height = rcdu->info->max_height; > > rcdu->num_crtcs = hweight8(rcdu->info->channels_mask); > -- Regards, Laurent Pinchart
next prev parent reply other threads:[~2022-04-15 11:20 UTC|newest] Thread overview: 44+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-03-16 13:10 [PATCH v2 0/7] Add RZ/G2L Display support Biju Das 2022-03-16 13:10 ` Biju Das 2022-03-16 13:10 ` [PATCH v2 1/7] dt-bindings: display: renesas,du: Document r9a07g044l bindings Biju Das 2022-03-16 13:10 ` [PATCH v2 1/7] dt-bindings: display: renesas, du: " Biju Das 2022-03-23 20:05 ` [PATCH v2 1/7] dt-bindings: display: renesas,du: " Rob Herring 2022-03-23 20:05 ` Rob Herring 2022-04-12 10:01 ` Geert Uytterhoeven 2022-04-12 10:01 ` Geert Uytterhoeven 2022-04-12 10:40 ` Biju Das 2022-04-12 10:40 ` Biju Das 2022-04-15 11:14 ` Laurent Pinchart 2022-04-15 11:14 ` Laurent Pinchart 2022-04-20 15:58 ` Biju Das 2022-04-20 15:58 ` Biju Das 2022-03-16 13:10 ` [PATCH v2 2/7] drm: rcar-du: Add num_rpf to struct rcar_du_device_info Biju Das 2022-03-16 13:10 ` Biju Das 2022-04-15 11:16 ` Laurent Pinchart 2022-04-15 11:16 ` Laurent Pinchart 2022-04-20 15:59 ` Biju Das 2022-04-20 15:59 ` Biju Das 2022-03-16 13:10 ` [PATCH v2 3/7] drm: rcar-du: Add max_width and max_height " Biju Das 2022-03-16 13:10 ` Biju Das 2022-04-15 11:20 ` Laurent Pinchart [this message] 2022-04-15 11:20 ` Laurent Pinchart 2022-04-20 16:02 ` Biju Das 2022-04-20 16:02 ` Biju Das 2022-03-16 13:10 ` [PATCH v2 4/7] drm: rcar-du: Move rcar_du_output_name() to rcar_du_common.c Biju Das 2022-03-16 13:10 ` Biju Das 2022-04-15 11:22 ` Laurent Pinchart 2022-04-15 11:22 ` Laurent Pinchart 2022-04-20 16:03 ` Biju Das 2022-04-20 16:03 ` Biju Das 2022-03-16 13:10 ` [PATCH v2 5/7] drm: rcar-du: Factorise rcar_du_{atomic_check,modeset_init} Biju Das 2022-03-16 13:10 ` [PATCH v2 5/7] drm: rcar-du: Factorise rcar_du_{atomic_check, modeset_init} Biju Das 2022-04-15 15:45 ` [PATCH v2 5/7] drm: rcar-du: Factorise rcar_du_{atomic_check,modeset_init} Laurent Pinchart 2022-04-15 15:45 ` Laurent Pinchart 2022-04-20 16:05 ` Biju Das 2022-04-20 16:05 ` Biju Das 2022-03-16 13:10 ` [PATCH v2 6/7] drm: rcar-du: Factorise rcar_du_vsp{complete,enable,plane_atomic_check} Biju Das 2022-03-16 13:10 ` [PATCH v2 6/7] drm: rcar-du: Factorise rcar_du_vsp{complete, enable, plane_atomic_check} Biju Das 2022-03-16 13:11 ` [PATCH v2 7/7] drm: rcar-du: Add RZ/G2L DU Support Biju Das 2022-03-16 13:11 ` Biju Das 2022-04-14 7:13 ` [PATCH v2 0/7] Add RZ/G2L Display support Biju Das 2022-04-14 7:13 ` Biju Das
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=YllU5CReP7Y6iRVE@pendragon.ideasonboard.com \ --to=laurent.pinchart@ideasonboard.com \ --cc=Chris.Paterson2@renesas.com \ --cc=airlied@linux.ie \ --cc=biju.das.jz@bp.renesas.com \ --cc=biju.das@bp.renesas.com \ --cc=daniel@ffwll.ch \ --cc=dri-devel@lists.freedesktop.org \ --cc=geert+renesas@glider.be \ --cc=kieran.bingham+renesas@ideasonboard.com \ --cc=linux-renesas-soc@vger.kernel.org \ --cc=prabhakar.mahadev-lad.rj@bp.renesas.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.