From: Brian Norris <briannorris@chromium.org>
To: Philippe CORNU <philippe.cornu@st.com>
Cc: "mark.rutland@arm.com" <mark.rutland@arm.com>,
"airlied@linux.ie" <airlied@linux.ie>,
"hoegsberg@gmail.com" <hoegsberg@gmail.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"dri-devel@lists.freedesktop.org"
<dri-devel@lists.freedesktop.org>,
Yannick FERTRE <yannick.fertre@st.com>,
"linux-rockchip@lists.infradead.org"
<linux-rockchip@lists.infradead.org>,
Nickey Yang <nickey.yang@rock-chips.com>,
"robh+dt@kernel.org" <robh+dt@kernel.org>,
"laurent.pinchart@ideasonboard.com"
<laurent.pinchart@ideasonboard.com>,
"zyw@rock-chips.com" <zyw@rock-chips.com>,
"xbl@rock-chips.com" <xbl@rock-chips.com>,
"mka@chromium.org" <mka@chromium.org>,
"hl@rock-chips.com" <hl@rock-chips.com>
Subject: Re: [PATCH v6 1/3] drm/bridge/synopsys: dsi: stop clobbering drvdata
Date: Thu, 7 Dec 2017 09:24:26 -0800 [thread overview]
Message-ID: <20171207172424.GA41020@google.com> (raw)
In-Reply-To: <e24ca009-6ac2-94f8-2f9d-53d339da1e8d@st.com>
On Thu, Dec 07, 2017 at 11:41:17AM +0000, Philippe CORNU wrote:
> platform_set_drvdata() is still missing in your version.
Ugh...indeed.
> On 12/06/2017 10:39 PM, Brian Norris wrote:
> > On Wed, Dec 06, 2017 at 05:08:19PM +0800, Nickey Yang wrote:
> >> From: Brian Norris <briannorris@chromium.org>
> >>
> >> Bridge drivers/helpers shouldn't be clobbering the drvdata, since a
> >> parent driver might need to own this. Instead, let's return our
> >> 'dw_mipi_dsi' object and have callers pass that back to us for removal.
> >>
> >> Signed-off-by: Brian Norris <briannorris@chromium.org>
> >> Signed-off-by: Nickey Yang <nickey.yang@rock-chips.com>
> >> Link:https://patchwork.kernel.org/patch/10078493/
> >>
> >> ---
> >> Changes
> >>
> >> v4:
> >> - Add From tag,update subject line
> >> - keep patch "drm/stm: dsi: Adjust dw_mipi_dsi_probe and remove"
> >> in this piece together.
> >>
> >> v5:
> >> - remove Review & Ack tag
> >> - fix remove() directly referencing the static
> >> dw_mipi_dsi_stm_plat_data struct.
> >
> > You've gotten this back into basically the shape I sent v1 in, so I
> > guess this deserves an:
> >
> > Acked-by: Brian Norris <briannorris@chromium.org>
...and I even diffed the two directly and didn't notice that one missing
at first :(
So instead, make that:
Nacked-by: Brian Norris <briannorris@chromium.org>
Can you *please* just use my patch and reword the commit message/subject
per the original comments on version 1? You're screwing it up every time
you try to change it...
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
WARNING: multiple messages have this Message-ID (diff)
From: Brian Norris <briannorris@chromium.org>
To: Philippe CORNU <philippe.cornu@st.com>
Cc: Nickey Yang <nickey.yang@rock-chips.com>,
"robh+dt@kernel.org" <robh+dt@kernel.org>,
"heiko@sntech.de" <heiko@sntech.de>,
"mark.rutland@arm.com" <mark.rutland@arm.com>,
"airlied@linux.ie" <airlied@linux.ie>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"dri-devel@lists.freedesktop.org"
<dri-devel@lists.freedesktop.org>,
"linux-rockchip@lists.infradead.org"
<linux-rockchip@lists.infradead.org>,
"laurent.pinchart@ideasonboard.com"
<laurent.pinchart@ideasonboard.com>,
"seanpaul@chromium.org" <seanpaul@chromium.org>,
"mka@chromium.org" <mka@chromium.org>,
"hoegsberg@gmail.com" <hoegsberg@gmail.com>,
"architt@codeaurora.org" <architt@codeaurora.org>,
Yannick FERTRE <yannick.fertre@st.com>,
"hl@rock-chips.com" <hl@rock-chips.com>,
"zyw@rock-chips.com" <zyw@rock-chips.com>,
"xbl@rock-chips.com" <xbl@rock-chips.com>
Subject: Re: [PATCH v6 1/3] drm/bridge/synopsys: dsi: stop clobbering drvdata
Date: Thu, 7 Dec 2017 09:24:26 -0800 [thread overview]
Message-ID: <20171207172424.GA41020@google.com> (raw)
In-Reply-To: <e24ca009-6ac2-94f8-2f9d-53d339da1e8d@st.com>
On Thu, Dec 07, 2017 at 11:41:17AM +0000, Philippe CORNU wrote:
> platform_set_drvdata() is still missing in your version.
Ugh...indeed.
> On 12/06/2017 10:39 PM, Brian Norris wrote:
> > On Wed, Dec 06, 2017 at 05:08:19PM +0800, Nickey Yang wrote:
> >> From: Brian Norris <briannorris@chromium.org>
> >>
> >> Bridge drivers/helpers shouldn't be clobbering the drvdata, since a
> >> parent driver might need to own this. Instead, let's return our
> >> 'dw_mipi_dsi' object and have callers pass that back to us for removal.
> >>
> >> Signed-off-by: Brian Norris <briannorris@chromium.org>
> >> Signed-off-by: Nickey Yang <nickey.yang@rock-chips.com>
> >> Link:https://patchwork.kernel.org/patch/10078493/
> >>
> >> ---
> >> Changes
> >>
> >> v4:
> >> - Add From tag,update subject line
> >> - keep patch "drm/stm: dsi: Adjust dw_mipi_dsi_probe and remove"
> >> in this piece together.
> >>
> >> v5:
> >> - remove Review & Ack tag
> >> - fix remove() directly referencing the static
> >> dw_mipi_dsi_stm_plat_data struct.
> >
> > You've gotten this back into basically the shape I sent v1 in, so I
> > guess this deserves an:
> >
> > Acked-by: Brian Norris <briannorris@chromium.org>
...and I even diffed the two directly and didn't notice that one missing
at first :(
So instead, make that:
Nacked-by: Brian Norris <briannorris@chromium.org>
Can you *please* just use my patch and reword the commit message/subject
per the original comments on version 1? You're screwing it up every time
you try to change it...
next prev parent reply other threads:[~2017-12-07 17:24 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-12-06 9:08 [PATCH v6 0/3] Update ROCKCHIP DSI driver that uses dw-mipi-dsi bridge Nickey Yang
2017-12-06 9:08 ` [PATCH v6 1/3] drm/bridge/synopsys: dsi: stop clobbering drvdata Nickey Yang
2017-12-06 9:08 ` Nickey Yang
2017-12-06 21:39 ` Brian Norris
2017-12-06 21:39 ` Brian Norris
2017-12-07 11:41 ` Philippe CORNU
2017-12-07 11:41 ` Philippe CORNU
2017-12-07 17:24 ` Brian Norris [this message]
2017-12-07 17:24 ` Brian Norris
2017-12-06 9:08 ` [PATCH v6 2/3] dt-bindings: display: rockchip: update DSI controller Nickey Yang
2017-12-06 9:08 ` [PATCH v6 3/3] drm/rockchip: Add ROCKCHIP DW MIPI DSI controller driver Nickey Yang
2017-12-06 21:52 ` Brian Norris
2017-12-06 21:52 ` Brian Norris
2017-12-07 14:29 ` Philippe CORNU
2017-12-07 14:29 ` Philippe CORNU
2017-12-12 1:42 ` Nickey Yang
2017-12-12 1:42 ` Nickey Yang
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=20171207172424.GA41020@google.com \
--to=briannorris@chromium.org \
--cc=airlied@linux.ie \
--cc=dri-devel@lists.freedesktop.org \
--cc=hl@rock-chips.com \
--cc=hoegsberg@gmail.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=mark.rutland@arm.com \
--cc=mka@chromium.org \
--cc=nickey.yang@rock-chips.com \
--cc=philippe.cornu@st.com \
--cc=robh+dt@kernel.org \
--cc=xbl@rock-chips.com \
--cc=yannick.fertre@st.com \
--cc=zyw@rock-chips.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: link
Be 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.