All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jani Nikula <jani.nikula@intel.com>
To: Anisse Astier <anisse@astier.eu>,
	intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org
Cc: Daniel Dadap <ddadap@nvidia.com>
Subject: Re: [Intel-gfx] [PATCH v5 0/2] GPD Win Max display fixes
Date: Fri, 31 Dec 2021 12:51:10 +0200	[thread overview]
Message-ID: <8735m9jbdt.fsf@intel.com> (raw)
In-Reply-To: <20211229222200.53128-1-anisse@astier.eu>

On Wed, 29 Dec 2021, Anisse Astier <anisse@astier.eu> wrote:
> This patch series is for making the GPD Win Max display usable with
> Linux.

Thanks, pushed to drm-intel-next. Sorry for the delay in dealing with
the patches.

BR,
Jani.


>
> 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 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.
>
> Change since v3:
>  - edits following Jani's review:
>     - The EDID from the opregion is now only used as a fallback: if we
>       cannot get any edid from the edp connector, then we attempt to get
>       it from the opregion. This works for the GPD Win Max.
>     - all other remarks should have been taken into account
>  - rebased on drm-tip
>  - added Co-developed-by
>  - reordered signed-off-by and reviewed-by in second patch (thanks
>    Maarten!)
>
> Changes since v4:
>  - checkpatch.pl fixes
>  - rebased on drm-tip
>  - Note: patch #1 is incomplete, still missing Jani's signed-off-by
>
>
> [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       |  7 +++
>  drivers/gpu/drm/i915/display/intel_opregion.c | 55 ++++++++++++++++++-
>  drivers/gpu/drm/i915/display/intel_opregion.h | 10 ++++
>  4 files changed, 77 insertions(+), 1 deletion(-)

-- 
Jani Nikula, Intel Open Source Graphics Center

WARNING: multiple messages have this Message-ID (diff)
From: Jani Nikula <jani.nikula@intel.com>
To: Anisse Astier <anisse@astier.eu>,
	intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org
Cc: Hans de Goede <hdegoede@redhat.com>,
	Uma Shankar <uma.shankar@intel.com>,
	Anisse Astier <anisse@astier.eu>,
	Daniel Dadap <ddadap@nvidia.com>
Subject: Re: [PATCH v5 0/2] GPD Win Max display fixes
Date: Fri, 31 Dec 2021 12:51:10 +0200	[thread overview]
Message-ID: <8735m9jbdt.fsf@intel.com> (raw)
In-Reply-To: <20211229222200.53128-1-anisse@astier.eu>

On Wed, 29 Dec 2021, Anisse Astier <anisse@astier.eu> wrote:
> This patch series is for making the GPD Win Max display usable with
> Linux.

Thanks, pushed to drm-intel-next. Sorry for the delay in dealing with
the patches.

BR,
Jani.


>
> 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 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.
>
> Change since v3:
>  - edits following Jani's review:
>     - The EDID from the opregion is now only used as a fallback: if we
>       cannot get any edid from the edp connector, then we attempt to get
>       it from the opregion. This works for the GPD Win Max.
>     - all other remarks should have been taken into account
>  - rebased on drm-tip
>  - added Co-developed-by
>  - reordered signed-off-by and reviewed-by in second patch (thanks
>    Maarten!)
>
> Changes since v4:
>  - checkpatch.pl fixes
>  - rebased on drm-tip
>  - Note: patch #1 is incomplete, still missing Jani's signed-off-by
>
>
> [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       |  7 +++
>  drivers/gpu/drm/i915/display/intel_opregion.c | 55 ++++++++++++++++++-
>  drivers/gpu/drm/i915/display/intel_opregion.h | 10 ++++
>  4 files changed, 77 insertions(+), 1 deletion(-)

-- 
Jani Nikula, Intel Open Source Graphics Center

  parent reply	other threads:[~2021-12-31 10:51 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-12-29 22:21 [Intel-gfx] [PATCH v5 0/2] GPD Win Max display fixes Anisse Astier
2021-12-29 22:21 ` Anisse Astier
2021-12-29 22:21 ` [Intel-gfx] [PATCH v5 1/2] drm/i915/opregion: add support for mailbox #5 EDID Anisse Astier
2021-12-29 22:21   ` Anisse Astier
2021-12-29 22:22 ` [Intel-gfx] [PATCH v5 2/2] drm: Add orientation quirk for GPD Win Max Anisse Astier
2021-12-29 22:22   ` Anisse Astier
2021-12-29 22:30 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for GPD Win Max display fixes (rev7) Patchwork
2021-12-29 22:32 ` [Intel-gfx] ✗ Fi.CI.SPARSE: " Patchwork
2021-12-29 22:58 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2021-12-30  0:29 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
2021-12-31 10:51 ` Jani Nikula [this message]
2021-12-31 10:51   ` [PATCH v5 0/2] GPD Win Max display fixes Jani Nikula

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=8735m9jbdt.fsf@intel.com \
    --to=jani.nikula@intel.com \
    --cc=anisse@astier.eu \
    --cc=ddadap@nvidia.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=intel-gfx@lists.freedesktop.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: 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.