* [PATCH] #1: test capitalizing PATCH @ 2017-02-28 11:45 Dan Carpenter 2017-02-28 16:31 ` Jani Nikula 0 siblings, 1 reply; 6+ messages in thread From: Dan Carpenter @ 2017-02-28 11:45 UTC (permalink / raw) To: intel-gfx The new version patchwork is filtering out my patches for some reason. Test if it's because it now insists on a capital [PATCH] in the subject. Patchwork should be more version control agnostic than that. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c index d1670b8afbf5..9173548ba601 100644 --- a/drivers/gpu/drm/i915/intel_dp.c +++ b/drivers/gpu/drm/i915/intel_dp.c @@ -1637,10 +1637,8 @@ intel_dp_compute_config(struct intel_encoder *encoder, adjusted_mode); if (INTEL_GEN(dev_priv) >= 9) { - int ret; - ret = skl_update_scaler_crtc(pipe_config); - if (ret) - return ret; + if (skl_update_scaler_crtc(pipe_config)) + return false; } if (HAS_GMCH_DISPLAY(dev_priv)) _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] #1: test capitalizing PATCH 2017-02-28 11:45 [PATCH] #1: test capitalizing PATCH Dan Carpenter @ 2017-02-28 16:31 ` Jani Nikula 2017-02-28 21:57 ` Dan Carpenter 2017-02-28 23:38 ` Hans de Goede 0 siblings, 2 replies; 6+ messages in thread From: Jani Nikula @ 2017-02-28 16:31 UTC (permalink / raw) To: Dan Carpenter, intel-gfx; +Cc: Hans de Goede, Daniel Vetter Cc: Hans, this probably applies to you as well. On Tue, 28 Feb 2017, Dan Carpenter <dan.carpenter@oracle.com> wrote: > The new version patchwork is filtering out my patches for some reason. > Test if it's because it now insists on a capital [PATCH] in the subject. > > Patchwork should be more version control agnostic than that. The patchwork flavor at freedesktop.org expects patches are sent using git send-email [1]. I hear otherwise there were too many false positives. Please either use git send-email, or add X-Mailer: git-send-email haha only kidding header into your patch mails. I'm sorry for the inconvenience and lack of transparency on this. Apparently git send-email is so ubiquitous nowadays that this hasn't been much of a problem. BR, Jani. [1] https://github.com/dlespiau/patchwork/blob/master/patchwork/bin/parsemail.py#L323 -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] #1: test capitalizing PATCH 2017-02-28 16:31 ` Jani Nikula @ 2017-02-28 21:57 ` Dan Carpenter 2017-02-28 23:38 ` Hans de Goede 1 sibling, 0 replies; 6+ messages in thread From: Dan Carpenter @ 2017-02-28 21:57 UTC (permalink / raw) To: Jani Nikula; +Cc: Hans de Goede, intel-gfx, Daniel Vetter On Tue, Feb 28, 2017 at 06:31:48PM +0200, Jani Nikula wrote: > > Cc: Hans, this probably applies to you as well. > > On Tue, 28 Feb 2017, Dan Carpenter <dan.carpenter@oracle.com> wrote: > > The new version patchwork is filtering out my patches for some reason. > > Test if it's because it now insists on a capital [PATCH] in the subject. > > > > Patchwork should be more version control agnostic than that. > > The patchwork flavor at freedesktop.org expects patches are sent using > git send-email [1]. I hear otherwise there were too many false > positives. > > Please either use git send-email, or add > > X-Mailer: git-send-email haha only kidding > > header into your patch mails. Ah. Great. Thanks! regards, dan carpenter _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] #1: test capitalizing PATCH 2017-02-28 16:31 ` Jani Nikula 2017-02-28 21:57 ` Dan Carpenter @ 2017-02-28 23:38 ` Hans de Goede 2017-03-01 9:57 ` Jani Nikula 1 sibling, 1 reply; 6+ messages in thread From: Hans de Goede @ 2017-02-28 23:38 UTC (permalink / raw) To: Jani Nikula, Dan Carpenter, intel-gfx; +Cc: Daniel Vetter Hi, On 28-02-17 17:31, Jani Nikula wrote: > > Cc: Hans, this probably applies to you as well. I'm already always using git-send-email, so whatever the reason why the CI system is not picking up my patches, this aint it. Regards, Hans > > On Tue, 28 Feb 2017, Dan Carpenter <dan.carpenter@oracle.com> wrote: >> The new version patchwork is filtering out my patches for some reason. >> Test if it's because it now insists on a capital [PATCH] in the subject. >> >> Patchwork should be more version control agnostic than that. > > The patchwork flavor at freedesktop.org expects patches are sent using > git send-email [1]. I hear otherwise there were too many false > positives. > > Please either use git send-email, or add > > X-Mailer: git-send-email haha only kidding > > header into your patch mails. > > I'm sorry for the inconvenience and lack of transparency on > this. Apparently git send-email is so ubiquitous nowadays that this > hasn't been much of a problem. > > > BR, > Jani. > > > [1] https://github.com/dlespiau/patchwork/blob/master/patchwork/bin/parsemail.py#L323 > _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] #1: test capitalizing PATCH 2017-02-28 23:38 ` Hans de Goede @ 2017-03-01 9:57 ` Jani Nikula 2017-03-01 10:14 ` Hans de Goede 0 siblings, 1 reply; 6+ messages in thread From: Jani Nikula @ 2017-03-01 9:57 UTC (permalink / raw) To: Hans de Goede, Dan Carpenter, intel-gfx Cc: Daniel Vetter, Lespiau, Damien, Heikkila, Juha-pekka On Wed, 01 Mar 2017, Hans de Goede <hdegoede@redhat.com> wrote: > Hi, > > On 28-02-17 17:31, Jani Nikula wrote: >> >> Cc: Hans, this probably applies to you as well. > > I'm already always using git-send-email, so whatever the > reason why the CI system is not picking up my patches, this > aint it. It doesn't look like your patches match [1], though. Do you have the sendemail.xmailer config option set to false? BR, Jani. > > Regards, > > Hans > >> >> On Tue, 28 Feb 2017, Dan Carpenter <dan.carpenter@oracle.com> wrote: >>> The new version patchwork is filtering out my patches for some reason. >>> Test if it's because it now insists on a capital [PATCH] in the subject. >>> >>> Patchwork should be more version control agnostic than that. >> >> The patchwork flavor at freedesktop.org expects patches are sent using >> git send-email [1]. I hear otherwise there were too many false >> positives. >> >> Please either use git send-email, or add >> >> X-Mailer: git-send-email haha only kidding >> >> header into your patch mails. >> >> I'm sorry for the inconvenience and lack of transparency on >> this. Apparently git send-email is so ubiquitous nowadays that this >> hasn't been much of a problem. >> >> >> BR, >> Jani. >> >> >> [1] https://github.com/dlespiau/patchwork/blob/master/patchwork/bin/parsemail.py#L323 >> -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] #1: test capitalizing PATCH 2017-03-01 9:57 ` Jani Nikula @ 2017-03-01 10:14 ` Hans de Goede 0 siblings, 0 replies; 6+ messages in thread From: Hans de Goede @ 2017-03-01 10:14 UTC (permalink / raw) To: Jani Nikula, Dan Carpenter, intel-gfx Cc: Daniel Vetter, Lespiau, Damien, Heikkila, Juha-pekka Hi, On 01-03-17 10:57, Jani Nikula wrote: > On Wed, 01 Mar 2017, Hans de Goede <hdegoede@redhat.com> wrote: >> Hi, >> >> On 28-02-17 17:31, Jani Nikula wrote: >>> >>> Cc: Hans, this probably applies to you as well. >> >> I'm already always using git-send-email, so whatever the >> reason why the CI system is not picking up my patches, this >> aint it. > > It doesn't look like your patches match [1], though. Do you have the > sendemail.xmailer config option set to false? Nope, just send myself a test patch: Date: Wed, 1 Mar 2017 11:05:32 +0100 Message-Id: <20170301100532.26692-1-hdegoede@redhat.com> X-Mailer: git-send-email 2.9.3 Result: 250 But indeed the header is gone when it hits my own Inbox, seems this gets stripped by the Red Hat smtp server somehow ? I will file a ticket with our it department for this. Regards, Hans > > BR, > Jani. > > >> >> Regards, >> >> Hans >> >>> >>> On Tue, 28 Feb 2017, Dan Carpenter <dan.carpenter@oracle.com> wrote: >>>> The new version patchwork is filtering out my patches for some reason. >>>> Test if it's because it now insists on a capital [PATCH] in the subject. >>>> >>>> Patchwork should be more version control agnostic than that. >>> >>> The patchwork flavor at freedesktop.org expects patches are sent using >>> git send-email [1]. I hear otherwise there were too many false >>> positives. >>> >>> Please either use git send-email, or add >>> >>> X-Mailer: git-send-email haha only kidding >>> >>> header into your patch mails. >>> >>> I'm sorry for the inconvenience and lack of transparency on >>> this. Apparently git send-email is so ubiquitous nowadays that this >>> hasn't been much of a problem. >>> >>> >>> BR, >>> Jani. >>> >>> >>> [1] https://github.com/dlespiau/patchwork/blob/master/patchwork/bin/parsemail.py#L323 >>> > _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-03-01 10:14 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2017-02-28 11:45 [PATCH] #1: test capitalizing PATCH Dan Carpenter 2017-02-28 16:31 ` Jani Nikula 2017-02-28 21:57 ` Dan Carpenter 2017-02-28 23:38 ` Hans de Goede 2017-03-01 9:57 ` Jani Nikula 2017-03-01 10:14 ` Hans de Goede
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.