Okay,I will submit a new patch immediately.
发自我的企业微信
----------Reply to Message---------- On Wed,Jun 22,2022 2:41 AM Lyude Paul <lyude@redhat.com> wrote: You can send it by adding email addresses to git send-email with the "--to foo@bar.net" argument. If it doesn't come up on patchwork the second try don't worry about it, I'll look and see if I can workaround it. TBH I'm not entirely sure why it didn't show up in the first place
On Wed, 2022-06-22 at 02:38 +0800, 蒋健 wrote: I need resubmit a new patch, how can i make sure dri-devel@ is explicitly?I got to:list by the script name of get_maintainer.pl. Pls help me…
发自我的企业微信
----------Reply to Message---------- On Wed,Jun 22,2022 2:07 AM Lyude Paul <lyude@redhat.com> wrote: Actually I hate to ask but could you resend this and the other patch that you sent and make sure that dri-devel@ is explicitly in the to: list? For some reason these patches don't seem to be coming up on patchwork
On Tue, 2022-06-21 at 13:54 -0400, Lyude Paul wrote: > Reviewed-by: Lyude Paul <lyude@redhat.com> > > Will push to the appropriate branch in a moment > > On Tue, 2022-06-21 at 21:14 +0800, Jiang Jian wrote: > > there is an unexpected word "the" in the comments that need to be dropped > > > > file: drivers/gpu/drm/display/drm_dp_helper.c > > line: 1600 > > * Doesn't account the the "MOT" bit, and instead assumes each > > changed to > > * Doesn't account the "MOT" bit, and instead assumes each > > > > Signed-off-by: Jiang Jian <jiangjian@cdjrlc.com> > > --- > > drivers/gpu/drm/display/drm_dp_helper.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/gpu/drm/display/drm_dp_helper.c > > b/drivers/gpu/drm/display/drm_dp_helper.c > > index e7c22c2ca90c..499f75768523 100644 > > --- a/drivers/gpu/drm/display/drm_dp_helper.c > > +++ b/drivers/gpu/drm/display/drm_dp_helper.c > > @@ -1597,7 +1597,7 @@ static int drm_dp_aux_reply_duration(const struct > > drm_dp_aux_msg *msg) > > * Calculate the length of the i2c transfer in usec, assuming > > * the i2c bus speed is as specified. Gives the the "worst" > > * case estimate, ie. successful while as long as possible. > > - * Doesn't account the the "MOT" bit, and instead assumes each > > + * Doesn't account the "MOT" bit, and instead assumes each > > * message includes a START, ADDRESS and STOP. Neither does it > > * account for additional random variables such as clock stretching. > > */ >
-- Cheers, Lyude Paul (she/her) Software Engineer at Red Hat
I got the list as follow: ~/linux-kernel/linux-next on master ⇡1 ./scripts/get_maintainer.pl 0001-drm-display-Remove-duplicate-the-in-two-places.patch David Airlie <airlied@linux.ie> (maintainer:DRM DRIVERS) Daniel Vetter <daniel@ffwll.ch> (maintainer:DRM DRIVERS) Jani Nikula <jani.nikula@intel.com> (commit_signer:8/21=38%,authored:5/21=24%) Lyude Paul <lyude@redhat.com> (commit_signer:7/21=33%,authored:2/21=10%) Thomas Zimmermann <tzimmermann@suse.de> (commit_signer:5/21=24%,authored:5/21=24%) Douglas Anderson <dianders@chromium.org> (commit_signer:5/21=24%,authored:3/21=14%) "Ville Syrjälä" <ville.syrjala@linux.intel.com> (commit_signer:5/21=24%) Imre Deak <imre.deak@intel.com> (authored:2/21=10%) dri-devel@lists.freedesktop.org (open list:DRM DRIVERS) linux-kernel@vger.kernel.org (open list)
Submit patch CMD as follow: "git send-email 0001-drm-display-Remove-duplicate-the-in-two-places.patch --to airlied@linux.ie,daniel@ffwll.ch --cc jani.nikula@intel.com,lyude@redhat.com,tzimmermann@suse.de,dianders@chromium.org,ville.syrjala@linux.intel.com,imre.deak@intel.com,dri-devel@lists.freedesktop.org,linux-kernel@vger.kernel.org "
------------------ Original ------------------ From: "蒋健"<jiangjian@cdjrlc.com>; Date: Wed, Jun 22, 2022 02:47 AM To: "Lyude Paul"<lyude@redhat.com>; "airlied"<airlied@linux.ie>; "daniel"<daniel@ffwll.ch>; Cc: "jani.nikula"<jani.nikula@intel.com>; "ville.syrjala"<ville.syrjala@linux.intel.com>; "tzimmermann"<tzimmermann@suse.de>; "dianders"<dianders@chromium.org>; "imre.deak"<imre.deak@intel.com>; "dri-devel"<dri-devel@lists.freedesktop.org>; Subject: Re: [PATCH] drm/display: drop unexpected word "the" in the comments
Okay,I will submit a new patch immediately.
发自我的企业微信
----------Reply to Message---------- On Wed,Jun 22,2022 2:41 AM Lyude Paul <lyude@redhat.com> wrote: You can send it by adding email addresses to git send-email with the "--to foo@bar.net" argument. If it doesn't come up on patchwork the second try don't worry about it, I'll look and see if I can workaround it. TBH I'm not entirely sure why it didn't show up in the first place
On Wed, 2022-06-22 at 02:38 +0800, 蒋健 wrote: I need resubmit a new patch, how can i make sure dri-devel@ is explicitly?I got to:list by the script name of get_maintainer.pl. Pls help me…
发自我的企业微信
----------Reply to Message---------- On Wed,Jun 22,2022 2:07 AM Lyude Paul <lyude@redhat.com> wrote: Actually I hate to ask but could you resend this and the other patch that you sent and make sure that dri-devel@ is explicitly in the to: list? For some reason these patches don't seem to be coming up on patchwork
On Tue, 2022-06-21 at 13:54 -0400, Lyude Paul wrote: > Reviewed-by: Lyude Paul <lyude@redhat.com> > > Will push to the appropriate branch in a moment > > On Tue, 2022-06-21 at 21:14 +0800, Jiang Jian wrote: > > there is an unexpected word "the" in the comments that need to be dropped > > > > file: drivers/gpu/drm/display/drm_dp_helper.c > > line: 1600 > > * Doesn't account the the "MOT" bit, and instead assumes each > > changed to > > * Doesn't account the "MOT" bit, and instead assumes each > > > > Signed-off-by: Jiang Jian <jiangjian@cdjrlc.com> > > --- > > drivers/gpu/drm/display/drm_dp_helper.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/gpu/drm/display/drm_dp_helper.c > > b/drivers/gpu/drm/display/drm_dp_helper.c > > index e7c22c2ca90c..499f75768523 100644 > > --- a/drivers/gpu/drm/display/drm_dp_helper.c > > +++ b/drivers/gpu/drm/display/drm_dp_helper.c > > @@ -1597,7 +1597,7 @@ static int drm_dp_aux_reply_duration(const struct > > drm_dp_aux_msg *msg) > > * Calculate the length of the i2c transfer in usec, assuming > > * the i2c bus speed is as specified. Gives the the "worst" > > * case estimate, ie. successful while as long as possible. > > - * Doesn't account the the "MOT" bit, and instead assumes each > > + * Doesn't account the "MOT" bit, and instead assumes each > > * message includes a START, ADDRESS and STOP. Neither does it > > * account for additional random variables such as clock stretching. > > */ >
Yeah I'm not totally sure what's going on here, I wouldn't worry about it too much. I will just push the patches manually
On Wed, 2022-06-22 at 03:01 +0800, 蒋健 wrote:
I got the list as follow: ~/linux-kernel/linux-next on master ⇡1 ./scripts/get_maintainer.pl 0001-drm-display-Remove-duplicate-the-in-two-places.patch David Airlie airlied@linux.ie (maintainer:DRM DRIVERS) Daniel Vetter daniel@ffwll.ch (maintainer:DRM DRIVERS) Jani Nikula jani.nikula@intel.com (commit_signer:8/21=38%,authored:5/21=24%) Lyude Paul lyude@redhat.com (commit_signer:7/21=33%,authored:2/21=10%) Thomas Zimmermann tzimmermann@suse.de (commit_signer:5/21=24%,authored:5/21=24%) Douglas Anderson dianders@chromium.org (commit_signer:5/21=24%,authored:3/21=14%) "Ville Syrjälä" ville.syrjala@linux.intel.com (commit_signer:5/21=24%) Imre Deak imre.deak@intel.com (authored:2/21=10%) dri-devel@lists.freedesktop.org (open list:DRM DRIVERS) linux-kernel@vger.kernel.org (open list)
Submit patch CMD as follow: "git send-email 0001-drm-display-Remove-duplicate-the-in-two-places.patch -- to airlied@linux.ie,daniel@ffwll.ch --cc jani.nikula@intel.com,lyude@redhat.com,tzimmermann@suse.de,dianders@chromium .org,ville.syrjala@linux.intel.com,imre.deak@intel.com,dri- devel@lists.freedesktop.org,linux-kernel@vger.kernel.org "
------------------ Original ------------------ From: "蒋健"jiangjian@cdjrlc.com; Date: Wed, Jun 22, 2022 02:47 AM To: "Lyude Paul"lyude@redhat.com; "airlied"airlied@linux.ie; "daniel"daniel@ffwll.ch; Cc: "jani.nikula"jani.nikula@intel.com; "ville.syrjala"ville.syrjala@linux.intel.com; "tzimmermann"tzimmermann@suse.de; "dianders"dianders@chromium.org; "imre.deak"imre.deak@intel.com; "dri-devel"<dri- devel@lists.freedesktop.org>; Subject: Re: [PATCH] drm/display: drop unexpected word "the" in the comments Okay,I will submit a new patch immediately.
发自我的企业微信
----------Reply to Message---------- On Wed,Jun 22,2022 2:41 AM Lyude Paul lyude@redhat.com wrote: You can send it by adding email addresses to git send-email with the "--to foo@bar.net" argument. If it doesn't come up on patchwork the second try don't worry about it, I'll look and see if I can workaround it. TBH I'm not entirely sure why it didn't show up in the first place
On Wed, 2022-06-22 at 02:38 +0800, 蒋健 wrote:
I need resubmit a new patch, how can i make sure dri-devel@ is explicitly?I got to:list by the script name of get_maintainer.pl. Pls help me…
发自我的企业微信
----------Reply to Message---------- On Wed,Jun 22,2022 2:07 AM Lyude Paul lyude@redhat.com wrote: Actually I hate to ask but could you resend this and the other patch that you sent and make sure that dri-devel@ is explicitly in the to: list? For some reason these patches don't seem to be coming up on patchwork
On Tue, 2022-06-21 at 13:54 -0400, Lyude Paul wrote:
Reviewed-by: Lyude Paul lyude@redhat.com
Will push to the appropriate branch in a moment
On Tue, 2022-06-21 at 21:14 +0800, Jiang Jian wrote:
there is an unexpected word "the" in the comments that need to be
dropped
file: drivers/gpu/drm/display/drm_dp_helper.c line: 1600 * Doesn't account the the "MOT" bit, and instead assumes each changed to * Doesn't account the "MOT" bit, and instead assumes each
Signed-off-by: Jiang Jian jiangjian@cdjrlc.com
drivers/gpu/drm/display/drm_dp_helper.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/display/drm_dp_helper.c b/drivers/gpu/drm/display/drm_dp_helper.c index e7c22c2ca90c..499f75768523 100644 --- a/drivers/gpu/drm/display/drm_dp_helper.c +++ b/drivers/gpu/drm/display/drm_dp_helper.c @@ -1597,7 +1597,7 @@ static int drm_dp_aux_reply_duration(const
struct
drm_dp_aux_msg *msg) * Calculate the length of the i2c transfer in usec, assuming * the i2c bus speed is as specified. Gives the the "worst" * case estimate, ie. successful while as long as possible.
- Doesn't account the the "MOT" bit, and instead assumes each
- Doesn't account the "MOT" bit, and instead assumes each
* message includes a START, ADDRESS and STOP. Neither does it * account for additional random variables such as clock stretching. */
dri-devel@lists.freedesktop.org