From: Fabrizio Castro <fabrizio.castro@bp.renesas.com> To: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Cc: David Airlie <airlied@linux.ie>, Daniel Vetter <daniel@ffwll.ch>, Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>, "dri-devel@lists.freedesktop.org" <dri-devel@lists.freedesktop.org>, "linux-renesas-soc@vger.kernel.org" <linux-renesas-soc@vger.kernel.org>, Simon Horman <horms@verge.net.au>, Geert Uytterhoeven <geert+renesas@glider.be>, Chris Paterson <Chris.Paterson2@renesas.com>, Biju Das <biju.das@bp.renesas.com>, Jacopo Mondi <jacopo+renesas@jmondi.org> Subject: RE: [PATCH v2 6/9] drm: rcar-du: lvds: Fix companion's mode Date: Thu, 15 Aug 2019 13:42:59 +0000 [thread overview] Message-ID: <TY1PR01MB17709614E219524FC4C907A8C0AC0@TY1PR01MB1770.jpnprd01.prod.outlook.com> (raw) In-Reply-To: <20190815115517.GJ5011@pendragon.ideasonboard.com> Hi Laurent, Thank you for your feedback! > From: linux-renesas-soc-owner@vger.kernel.org <linux-renesas-soc-owner@vger.kernel.org> On Behalf Of Laurent Pinchart > Sent: 15 August 2019 12:55 > Subject: Re: [PATCH v2 6/9] drm: rcar-du: lvds: Fix companion's mode > > Hi Fabrizio, > > Thank you for the patch. > > On Thu, Aug 15, 2019 at 12:04:30PM +0100, Fabrizio Castro wrote: > > The companion encoder needs to be told to use the same > > mode as the primary encoder. > > > > Fixes: e9e8798ab7b8 ("drm: rcar-du: lvds: Add support for dual-link mode") > > Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com> > > > > --- > > v1->v2: > > * reworked according to Laurent's feedback > > > > drivers/gpu/drm/rcar-du/rcar_lvds.c | 5 +++++ > > 1 file changed, 5 insertions(+) > > > > diff --git a/drivers/gpu/drm/rcar-du/rcar_lvds.c b/drivers/gpu/drm/rcar-du/rcar_lvds.c > > index 3fe0b86..41d28f4 100644 > > --- a/drivers/gpu/drm/rcar-du/rcar_lvds.c > > +++ b/drivers/gpu/drm/rcar-du/rcar_lvds.c > > @@ -603,6 +603,11 @@ static void rcar_lvds_mode_set(struct drm_bridge *bridge, > > lvds->display_mode = *adjusted_mode; > > > > rcar_lvds_get_lvds_mode(lvds); > > + if (lvds->companion) { > > + struct rcar_lvds *companion_lvds = bridge_to_rcar_lvds( > > + lvds->companion); > > + companion_lvds->mode = lvds->mode; > > How about calling rcar_lvds_mode_set() on the companion instead ? Can do, will send a new version. Cheers, Fab > > > + } > > } > > > > static int rcar_lvds_attach(struct drm_bridge *bridge) > > -- > Regards, > > Laurent Pinchart
WARNING: multiple messages have this Message-ID (diff)
From: Fabrizio Castro <fabrizio.castro@bp.renesas.com> To: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Cc: Chris Paterson <Chris.Paterson2@renesas.com>, Geert Uytterhoeven <geert+renesas@glider.be>, Simon Horman <horms@verge.net.au>, David Airlie <airlied@linux.ie>, "dri-devel@lists.freedesktop.org" <dri-devel@lists.freedesktop.org>, Biju Das <biju.das@bp.renesas.com>, "linux-renesas-soc@vger.kernel.org" <linux-renesas-soc@vger.kernel.org>, Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>, Jacopo Mondi <jacopo+renesas@jmondi.org> Subject: RE: [PATCH v2 6/9] drm: rcar-du: lvds: Fix companion's mode Date: Thu, 15 Aug 2019 13:42:59 +0000 [thread overview] Message-ID: <TY1PR01MB17709614E219524FC4C907A8C0AC0@TY1PR01MB1770.jpnprd01.prod.outlook.com> (raw) In-Reply-To: <20190815115517.GJ5011@pendragon.ideasonboard.com> Hi Laurent, Thank you for your feedback! > From: linux-renesas-soc-owner@vger.kernel.org <linux-renesas-soc-owner@vger.kernel.org> On Behalf Of Laurent Pinchart > Sent: 15 August 2019 12:55 > Subject: Re: [PATCH v2 6/9] drm: rcar-du: lvds: Fix companion's mode > > Hi Fabrizio, > > Thank you for the patch. > > On Thu, Aug 15, 2019 at 12:04:30PM +0100, Fabrizio Castro wrote: > > The companion encoder needs to be told to use the same > > mode as the primary encoder. > > > > Fixes: e9e8798ab7b8 ("drm: rcar-du: lvds: Add support for dual-link mode") > > Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com> > > > > --- > > v1->v2: > > * reworked according to Laurent's feedback > > > > drivers/gpu/drm/rcar-du/rcar_lvds.c | 5 +++++ > > 1 file changed, 5 insertions(+) > > > > diff --git a/drivers/gpu/drm/rcar-du/rcar_lvds.c b/drivers/gpu/drm/rcar-du/rcar_lvds.c > > index 3fe0b86..41d28f4 100644 > > --- a/drivers/gpu/drm/rcar-du/rcar_lvds.c > > +++ b/drivers/gpu/drm/rcar-du/rcar_lvds.c > > @@ -603,6 +603,11 @@ static void rcar_lvds_mode_set(struct drm_bridge *bridge, > > lvds->display_mode = *adjusted_mode; > > > > rcar_lvds_get_lvds_mode(lvds); > > + if (lvds->companion) { > > + struct rcar_lvds *companion_lvds = bridge_to_rcar_lvds( > > + lvds->companion); > > + companion_lvds->mode = lvds->mode; > > How about calling rcar_lvds_mode_set() on the companion instead ? Can do, will send a new version. Cheers, Fab > > > + } > > } > > > > static int rcar_lvds_attach(struct drm_bridge *bridge) > > -- > Regards, > > Laurent Pinchart _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2019-08-15 13:43 UTC|newest] Thread overview: 65+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-08-15 11:04 [PATCH v2 0/9] Add dual-LVDS panel support to EK874 Fabrizio Castro 2019-08-15 11:04 ` [PATCH v2 1/9] dt-bindings: panel: lvds: Add dual-link LVDS display support Fabrizio Castro 2019-08-15 11:45 ` Laurent Pinchart 2019-08-15 11:45 ` Laurent Pinchart 2019-08-15 13:37 ` Fabrizio Castro 2019-08-15 13:37 ` Fabrizio Castro 2019-08-15 11:04 ` [PATCH v2 2/9] dt-bindings: display: Add bindings for Advantech IDK-2121WR Fabrizio Castro 2019-08-15 11:47 ` Laurent Pinchart 2019-08-15 13:38 ` Fabrizio Castro 2019-08-15 13:38 ` Fabrizio Castro 2019-08-15 11:04 ` [PATCH v2 3/9] drm: Rename drm_bridge_timings to drm_timings Fabrizio Castro 2019-08-15 13:18 ` Laurent Pinchart 2019-08-15 13:50 ` Fabrizio Castro 2019-08-15 13:50 ` Fabrizio Castro 2019-08-15 14:04 ` Greg Kroah-Hartman 2019-08-15 14:04 ` Greg Kroah-Hartman 2019-08-15 14:14 ` Laurent Pinchart 2019-08-15 14:31 ` Fabrizio Castro 2019-08-15 14:31 ` Fabrizio Castro 2019-08-15 14:53 ` Greg Kroah-Hartman 2019-08-15 14:53 ` Greg Kroah-Hartman 2019-08-15 15:01 ` Fabrizio Castro 2019-08-15 15:01 ` Fabrizio Castro 2019-08-15 18:06 ` Laurent Pinchart 2019-08-15 18:06 ` Laurent Pinchart 2019-08-15 19:05 ` Greg Kroah-Hartman 2019-08-15 19:05 ` Greg Kroah-Hartman 2019-08-16 8:11 ` Fabrizio Castro 2019-08-16 8:11 ` Fabrizio Castro 2019-08-15 11:04 ` [PATCH v2 4/9] drm/timings: Add link flags Fabrizio Castro 2019-08-15 12:00 ` Laurent Pinchart 2019-08-15 15:40 ` Fabrizio Castro 2019-08-15 15:40 ` Fabrizio Castro 2019-08-15 11:04 ` [PATCH v2 5/9] drm/panel: Add timings field to drm_panel Fabrizio Castro 2019-08-15 12:03 ` Laurent Pinchart 2019-08-15 13:49 ` Fabrizio Castro 2019-08-15 13:49 ` Fabrizio Castro 2019-08-15 14:13 ` Sam Ravnborg 2019-08-15 14:48 ` Fabrizio Castro 2019-08-15 14:48 ` Fabrizio Castro 2019-08-15 11:04 ` [PATCH v2 6/9] drm: rcar-du: lvds: Fix companion's mode Fabrizio Castro 2019-08-15 11:04 ` Fabrizio Castro 2019-08-15 11:55 ` Laurent Pinchart 2019-08-15 11:55 ` Laurent Pinchart 2019-08-15 13:42 ` Fabrizio Castro [this message] 2019-08-15 13:42 ` Fabrizio Castro 2019-08-15 11:04 ` [PATCH v2 7/9] drm: rcar-du: lvds: Add dual-LVDS panels support Fabrizio Castro 2019-08-15 11:04 ` Fabrizio Castro 2019-08-15 13:08 ` Laurent Pinchart 2019-08-15 13:08 ` Laurent Pinchart 2019-08-15 15:36 ` Fabrizio Castro 2019-08-15 15:36 ` Fabrizio Castro 2019-08-20 16:04 ` Laurent Pinchart 2019-08-20 16:04 ` Laurent Pinchart 2019-08-21 17:00 ` Fabrizio Castro 2019-08-21 17:00 ` Fabrizio Castro 2019-08-15 11:04 ` [PATCH v2 8/9] drm/panel: lvds: Add support for the IDK-2121WR Fabrizio Castro 2019-08-15 11:04 ` [PATCH v2 9/9] arm64: dts: renesas: Add EK874 board with idk-2121wr display support Fabrizio Castro 2019-08-19 12:03 ` Geert Uytterhoeven 2019-08-19 12:03 ` Geert Uytterhoeven 2019-08-20 13:59 ` Fabrizio Castro 2019-08-15 14:15 ` [PATCH v2 0/9] Add dual-LVDS panel support to EK874 Sam Ravnborg 2019-08-15 14:15 ` Sam Ravnborg 2019-08-15 14:32 ` Fabrizio Castro 2019-08-15 14:32 ` Fabrizio Castro
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=TY1PR01MB17709614E219524FC4C907A8C0AC0@TY1PR01MB1770.jpnprd01.prod.outlook.com \ --to=fabrizio.castro@bp.renesas.com \ --cc=Chris.Paterson2@renesas.com \ --cc=airlied@linux.ie \ --cc=biju.das@bp.renesas.com \ --cc=daniel@ffwll.ch \ --cc=dri-devel@lists.freedesktop.org \ --cc=geert+renesas@glider.be \ --cc=horms@verge.net.au \ --cc=jacopo+renesas@jmondi.org \ --cc=kieran.bingham+renesas@ideasonboard.com \ --cc=laurent.pinchart@ideasonboard.com \ --cc=linux-renesas-soc@vger.kernel.org \ /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.