Hi,
On 01-12-17 19:02, Bartlomiej Zolnierkiewicz wrote:
On Saturday, November 25, 2017 08:35:46 PM Hans de Goede wrote:
Here is v7 of my series to add a "panel orientation" property to
the drm-connector for the LCD panel to let userspace know about LCD
panels which are not mounted upright, as well as detecting upside-down
panels without needing quirks (like we do for 90 degree rotated screens).
Bartlomiej, can we please have your Acked-by for merging patches 1,
6 and 7 through the drm tree?
Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@xxxxxxxxxxx>
Thank you.
Given that you add (can be in a incremental patch) a comment to:
drivers/gpu/drm/drm_panel_orientation_quirks.c [ at the top ]
I've fixed this up before merging, adding this comment:
* Note the quirks in this file are shared with fbdev/efifb and as such
* must not depend on other drm code.
and
drivers/gpu/drm/Kconfig [ to config DRM_PANEL_ORIENTATION_QUIRKS ]
Not everyone likes adding comments to Kconfig files I've had
push-back against that in the past. So I will post a follow-up patch
for this and then we will see from there.
explaining that the code in question is shared with fbdev/efifb (so
hopefully people will be careful and not try to make this code DRM
specific at some point in a future).
Regards,
Hans
New in v7:
-Fix embarrassing build error in efifb due to me only rebuilding modules
after cleanups from v6
New in v6:
-Fix / reference kernel-doc comments
-Don't export the DRM_MODE_PANEL_ORIENTATION_* defines in the UAPI
-Move i915 dsi hardware rotation state read-out to intel_dsi_init()
New in v5:
-Add kernel-doc comment documenting drm_get_panel_orientation_quirk()
-drm_fb_helper: Only use hardware (crtc primary plane) rotation for
180 degrees for now as 9-/270 degrees rotation requires special handling
New in v4:
-Fix drm_fb_helper code setting an invalid rotation value on the primary
plane of disabled/unused crtcs (caught by Fi.CI)
New in v3:
-As requested by Daniel v3 moves the quirks over from the fbdev
subsys to the drm subsys. I've done this by simpy starting with a copy of
the quirk table and eventually removing the fbdev version.
The 1st patch in this series is a small fbdev/fbcon patch, patches 2-5
are all drm patches and patches 6-7 are fbdev/fbcon patches again. As
discussed previously the plan is to merge all 7 patches through the
drm tree.
Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
