intel-gfx.lists.freedesktop.org archive mirror
 help / color / mirror / Atom feed
From: Anisse Astier <anisse@astier.eu>
To: Jani Nikula <jani.nikula@intel.com>
Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org,
	"Ville Syrjälä" <ville.syrjala@linux.intel.com>,
	"Hans de Goede" <hdegoede@redhat.com>,
	"Uma Shankar" <uma.shankar@intel.com>,
	"Daniel Dadap" <ddadap@nvidia.com>
Subject: Re: [Intel-gfx] [PATCH v3 0/2] GPD Win Max display fixes
Date: Tue, 31 Aug 2021 11:51:49 +0200	[thread overview]
Message-ID: <CALUN=qJ9DX2nRPXUb08t7orJY1=+O+Y3Th=dgiURc8Z4VYvw8A@mail.gmail.com> (raw)
In-Reply-To: <87pmtuarhh.fsf@intel.com>

On Tue, Aug 31, 2021 at 11:33 AM Jani Nikula <jani.nikula@intel.com> wrote:
>
> On Tue, 17 Aug 2021, Anisse Astier <anisse@astier.eu> wrote:
> > This patch series is for making the GPD Win Max display usable with
> > Linux.
> >
> > The GPD Win Max is a small laptop, and its eDP panel does not send an
> > EDID over DPCD; the EDID is instead available in the intel opregion, in
> > mailbox #5 [1]
> >
> > The first patch is based on Jani's patch series [2] adding support for
> > the opregion, with changes. I've changed authorship, but I'd be glad to
> > revert it
>
> If you don't mind, please just add:
>
> Co-developed-by: Jani Nikula <jani.nikula@intel.com>

I don't mind at all, I think you should be first author for this
series, I just didn't feel like giving you the blame for the bugs
after this many modifications, without asking first. Will be in next
iteration.

Anisse


>
>
> Thanks,
> Jani.
>
> >
> > The second patch is just to fix the orientation of the panel.
> >
> > Changes since v1:
> >  - rebased on drm-tip
> >  - squashed patch 1 & 2
> >  - picked up Reviewed-by from Hans de Goede (thanks for the review)
> >
> > Changes since v2:
> >  - rebased on drm-tip
> >  - updated commit message
> >
> > When v2 was initially sent [3] Ville Syrjälä suggested that it might be
> > a good idea to use the ACPI _DDC method instead to get the EDID, to
> > cover a wider range of hardware. Unfortunately, it doesn't seem
> > available on GPD Win Max, so I think this work should be done
> > independently, and this patch series considered separately.
> >
> > [1]: https://gitlab.freedesktop.org/drm/intel/-/issues/3454
> > [2]: https://patchwork.kernel.org/project/intel-gfx/patch/20200828061941.17051-1-jani.nikula@intel.com/
> > [3]: https://patchwork.kernel.org/project/intel-gfx/patch/20210531204642.4907-2-anisse@astier.eu/
> >
> >
> > Anisse Astier (2):
> >   drm/i915/opregion: add support for mailbox #5 EDID
> >   drm: Add orientation quirk for GPD Win Max
> >
> >  .../gpu/drm/drm_panel_orientation_quirks.c    |  6 ++
> >  drivers/gpu/drm/i915/display/intel_dp.c       |  3 +
> >  drivers/gpu/drm/i915/display/intel_opregion.c | 69 ++++++++++++++++++-
> >  drivers/gpu/drm/i915/display/intel_opregion.h |  8 +++
> >  4 files changed, 85 insertions(+), 1 deletion(-)
>
> --
> Jani Nikula, Intel Open Source Graphics Center

      reply	other threads:[~2021-08-31  9:52 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-08-17 20:43 [Intel-gfx] [PATCH v3 0/2] GPD Win Max display fixes Anisse Astier
2021-08-17 20:43 ` [Intel-gfx] [PATCH v3 1/2] drm/i915/opregion: add support for mailbox #5 EDID Anisse Astier
2021-08-31  9:30   ` Jani Nikula
2021-09-01 12:06     ` Anisse Astier
2021-08-17 20:43 ` [Intel-gfx] [PATCH v3 2/2] drm: Add orientation quirk for GPD Win Max Anisse Astier
2021-08-17 21:30 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for GPD Win Max display fixes (rev4) Patchwork
2021-08-17 21:32 ` [Intel-gfx] ✗ Fi.CI.SPARSE: " Patchwork
2021-08-17 22:01 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2021-08-17 23:22 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
2021-08-30 13:49 ` [Intel-gfx] [PATCH v3 0/2] GPD Win Max display fixes Anisse Astier
2021-08-31  9:33 ` Jani Nikula
2021-08-31  9:51   ` Anisse Astier [this message]

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='CALUN=qJ9DX2nRPXUb08t7orJY1=+O+Y3Th=dgiURc8Z4VYvw8A@mail.gmail.com' \
    --to=anisse@astier.eu \
    --cc=ddadap@nvidia.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=hdegoede@redhat.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=jani.nikula@intel.com \
    --cc=uma.shankar@intel.com \
    --cc=ville.syrjala@linux.intel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).