Re: [PATCH RESEND] drm: panel-orientation-quirks: Add quirk for Lenovo Ideapad D330
On Tue, 23 Apr 2019, David Santamaría Rogado wrote: > Hi Jani, > > When are the changes in drm-misc-next pushed? > > I'm testing recent kernels that don't work with some models because > the dsi display problem you are aware. Works with 4.15 but not with > more recent kernels. > > The model I have by going to sleep state and awake it gets the display on, > I want to see if this change makes the display behave different in recent > kernels in more ways than only rotating the display. The patches are headed for v5.2. In retrospect could've added them to drm-misc-fixes, but regardless we can still make the backport request to older kernel once it hits upstream. BR, Jani. > > Thanks in advance. > > El vie., 1 mar. 2019 a las 11:12, Jani Nikula > () escribió: >> >> On Sat, 23 Feb 2019, David Santamaría Rogado wrote: >> > Done Jani. I think it's ok now. Don't worry, at least I could get >> > little familiar with git send-email also with multiple patches :) >> >> Sorry about all the hassle, pushed to drm-misc-next, thanks for the >> patch. >> >> BR, >> Jani. >> >> > >> > El sáb., 23 feb. 2019 a las 22:19, David Santamaría Rogado >> > () escribió: >> >> >> >> Lenovo Ideapad D330 Pentium CPU version has 1920x1200 LCD. >> >> Console output gets rotated at boot as Miix 310. >> >> >> >> Signed-off-by: David Santamaría Rogado >> >> --- >> >> drivers/gpu/drm/drm_panel_orientation_quirks.c | 13 + >> >> 1 file changed, 13 insertions(+) >> >> >> >> diff --git a/drivers/gpu/drm/drm_panel_orientation_quirks.c >> >> b/drivers/gpu/drm/drm_panel_orientation_quirks.c >> >> index 52e445bb1aa5..521aff99b08a 100644 >> >> --- a/drivers/gpu/drm/drm_panel_orientation_quirks.c >> >> +++ b/drivers/gpu/drm/drm_panel_orientation_quirks.c >> >> @@ -80,6 +80,12 @@ static const struct drm_dmi_panel_orientation_data >> >> lcd800x1280_rightside_up = { >> >> .orientation = DRM_MODE_PANEL_ORIENTATION_RIGHT_UP, >> >> }; >> >> >> >> +static const struct drm_dmi_panel_orientation_data >> >> lcd1200x1920_rightside_up = { >> >> + .width = 1200, >> >> + .height = 1920, >> >> + .orientation = DRM_MODE_PANEL_ORIENTATION_RIGHT_UP, >> >> +}; >> >> + >> >> static const struct dmi_system_id orientation_data[] = { >> >> { /* Acer One 10 (S1003) */ >> >> .matches = { >> >> @@ -148,6 +154,13 @@ static const struct dmi_system_id orientation_data[] >> >> = { >> >> DMI_EXACT_MATCH(DMI_PRODUCT_VERSION, "Lenovo MIIX >> >> 320-10ICR"), >> >> }, >> >> .driver_data = (void *)_rightside_up, >> >> + }, {/* Lenovo Ideapad D330 */ >> >> + .matches = { >> >> + DMI_EXACT_MATCH(DMI_SYS_VENDOR, "LENOVO"), >> >> + DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "81H3"), >> >> + DMI_EXACT_MATCH(DMI_PRODUCT_VERSION, "Lenovo ideapad >> >> D330-10IGM"), >> >> + }, >> >> + .driver_data = (void *)_rightside_up, >> >> }, {/* VIOS LTH17 */ >> >> .matches = { >> >> DMI_EXACT_MATCH(DMI_SYS_VENDOR, "VIOS"), >> >> -- >> >> 2.20.1 >> >> >> >> -- >> Jani Nikula, Intel Open Source Graphics Center -- Jani Nikula, Intel Open Source Graphics Center ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH RESEND] drm: panel-orientation-quirks: Add quirk for Lenovo Ideapad D330
Hi Jani, When are the changes in drm-misc-next pushed? I'm testing recent kernels that don't work with some models because the dsi display problem you are aware. Works with 4.15 but not with more recent kernels. The model I have by going to sleep state and awake it gets the display on, I want to see if this change makes the display behave different in recent kernels in more ways than only rotating the display. Thanks in advance. El vie., 1 mar. 2019 a las 11:12, Jani Nikula () escribió: > > On Sat, 23 Feb 2019, David Santamaría Rogado wrote: > > Done Jani. I think it's ok now. Don't worry, at least I could get > > little familiar with git send-email also with multiple patches :) > > Sorry about all the hassle, pushed to drm-misc-next, thanks for the > patch. > > BR, > Jani. > > > > > El sáb., 23 feb. 2019 a las 22:19, David Santamaría Rogado > > () escribió: > >> > >> Lenovo Ideapad D330 Pentium CPU version has 1920x1200 LCD. > >> Console output gets rotated at boot as Miix 310. > >> > >> Signed-off-by: David Santamaría Rogado > >> --- > >> drivers/gpu/drm/drm_panel_orientation_quirks.c | 13 + > >> 1 file changed, 13 insertions(+) > >> > >> diff --git a/drivers/gpu/drm/drm_panel_orientation_quirks.c > >> b/drivers/gpu/drm/drm_panel_orientation_quirks.c > >> index 52e445bb1aa5..521aff99b08a 100644 > >> --- a/drivers/gpu/drm/drm_panel_orientation_quirks.c > >> +++ b/drivers/gpu/drm/drm_panel_orientation_quirks.c > >> @@ -80,6 +80,12 @@ static const struct drm_dmi_panel_orientation_data > >> lcd800x1280_rightside_up = { > >> .orientation = DRM_MODE_PANEL_ORIENTATION_RIGHT_UP, > >> }; > >> > >> +static const struct drm_dmi_panel_orientation_data > >> lcd1200x1920_rightside_up = { > >> + .width = 1200, > >> + .height = 1920, > >> + .orientation = DRM_MODE_PANEL_ORIENTATION_RIGHT_UP, > >> +}; > >> + > >> static const struct dmi_system_id orientation_data[] = { > >> { /* Acer One 10 (S1003) */ > >> .matches = { > >> @@ -148,6 +154,13 @@ static const struct dmi_system_id orientation_data[] > >> = { > >> DMI_EXACT_MATCH(DMI_PRODUCT_VERSION, "Lenovo MIIX > >> 320-10ICR"), > >> }, > >> .driver_data = (void *)_rightside_up, > >> + }, {/* Lenovo Ideapad D330 */ > >> + .matches = { > >> + DMI_EXACT_MATCH(DMI_SYS_VENDOR, "LENOVO"), > >> + DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "81H3"), > >> + DMI_EXACT_MATCH(DMI_PRODUCT_VERSION, "Lenovo ideapad > >> D330-10IGM"), > >> + }, > >> + .driver_data = (void *)_rightside_up, > >> }, {/* VIOS LTH17 */ > >> .matches = { > >> DMI_EXACT_MATCH(DMI_SYS_VENDOR, "VIOS"), > >> -- > >> 2.20.1 > >> > > -- > Jani Nikula, Intel Open Source Graphics Center ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH RESEND] drm: panel-orientation-quirks: Add quirk for Lenovo Ideapad D330
On Sat, 23 Feb 2019, David Santamaría Rogado wrote: > Done Jani. I think it's ok now. Don't worry, at least I could get > little familiar with git send-email also with multiple patches :) Sorry about all the hassle, pushed to drm-misc-next, thanks for the patch. BR, Jani. > > El sáb., 23 feb. 2019 a las 22:19, David Santamaría Rogado > () escribió: >> >> Lenovo Ideapad D330 Pentium CPU version has 1920x1200 LCD. >> Console output gets rotated at boot as Miix 310. >> >> Signed-off-by: David Santamaría Rogado >> --- >> drivers/gpu/drm/drm_panel_orientation_quirks.c | 13 + >> 1 file changed, 13 insertions(+) >> >> diff --git a/drivers/gpu/drm/drm_panel_orientation_quirks.c >> b/drivers/gpu/drm/drm_panel_orientation_quirks.c >> index 52e445bb1aa5..521aff99b08a 100644 >> --- a/drivers/gpu/drm/drm_panel_orientation_quirks.c >> +++ b/drivers/gpu/drm/drm_panel_orientation_quirks.c >> @@ -80,6 +80,12 @@ static const struct drm_dmi_panel_orientation_data >> lcd800x1280_rightside_up = { >> .orientation = DRM_MODE_PANEL_ORIENTATION_RIGHT_UP, >> }; >> >> +static const struct drm_dmi_panel_orientation_data >> lcd1200x1920_rightside_up = { >> + .width = 1200, >> + .height = 1920, >> + .orientation = DRM_MODE_PANEL_ORIENTATION_RIGHT_UP, >> +}; >> + >> static const struct dmi_system_id orientation_data[] = { >> { /* Acer One 10 (S1003) */ >> .matches = { >> @@ -148,6 +154,13 @@ static const struct dmi_system_id orientation_data[] = { >> DMI_EXACT_MATCH(DMI_PRODUCT_VERSION, "Lenovo MIIX >> 320-10ICR"), >> }, >> .driver_data = (void *)_rightside_up, >> + }, {/* Lenovo Ideapad D330 */ >> + .matches = { >> + DMI_EXACT_MATCH(DMI_SYS_VENDOR, "LENOVO"), >> + DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "81H3"), >> + DMI_EXACT_MATCH(DMI_PRODUCT_VERSION, "Lenovo ideapad >> D330-10IGM"), >> + }, >> + .driver_data = (void *)_rightside_up, >> }, {/* VIOS LTH17 */ >> .matches = { >> DMI_EXACT_MATCH(DMI_SYS_VENDOR, "VIOS"), >> -- >> 2.20.1 >> -- Jani Nikula, Intel Open Source Graphics Center ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH RESEND] drm: panel-orientation-quirks: Add quirk for Lenovo Ideapad D330
Done Jani. I think it's ok now. Don't worry, at least I could get little familiar with git send-email also with multiple patches :) El sáb., 23 feb. 2019 a las 22:19, David Santamaría Rogado () escribió: > > Lenovo Ideapad D330 Pentium CPU version has 1920x1200 LCD. > Console output gets rotated at boot as Miix 310. > > Signed-off-by: David Santamaría Rogado > --- > drivers/gpu/drm/drm_panel_orientation_quirks.c | 13 + > 1 file changed, 13 insertions(+) > > diff --git a/drivers/gpu/drm/drm_panel_orientation_quirks.c > b/drivers/gpu/drm/drm_panel_orientation_quirks.c > index 52e445bb1aa5..521aff99b08a 100644 > --- a/drivers/gpu/drm/drm_panel_orientation_quirks.c > +++ b/drivers/gpu/drm/drm_panel_orientation_quirks.c > @@ -80,6 +80,12 @@ static const struct drm_dmi_panel_orientation_data > lcd800x1280_rightside_up = { > .orientation = DRM_MODE_PANEL_ORIENTATION_RIGHT_UP, > }; > > +static const struct drm_dmi_panel_orientation_data lcd1200x1920_rightside_up > = { > + .width = 1200, > + .height = 1920, > + .orientation = DRM_MODE_PANEL_ORIENTATION_RIGHT_UP, > +}; > + > static const struct dmi_system_id orientation_data[] = { > { /* Acer One 10 (S1003) */ > .matches = { > @@ -148,6 +154,13 @@ static const struct dmi_system_id orientation_data[] = { > DMI_EXACT_MATCH(DMI_PRODUCT_VERSION, "Lenovo MIIX > 320-10ICR"), > }, > .driver_data = (void *)_rightside_up, > + }, {/* Lenovo Ideapad D330 */ > + .matches = { > + DMI_EXACT_MATCH(DMI_SYS_VENDOR, "LENOVO"), > + DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "81H3"), > + DMI_EXACT_MATCH(DMI_PRODUCT_VERSION, "Lenovo ideapad > D330-10IGM"), > + }, > + .driver_data = (void *)_rightside_up, > }, {/* VIOS LTH17 */ > .matches = { > DMI_EXACT_MATCH(DMI_SYS_VENDOR, "VIOS"), > -- > 2.20.1 > ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH RESEND] drm: panel-orientation-quirks: Add quirk for Lenovo Ideapad D330
Lenovo Ideapad D330 Pentium CPU version has 1920x1200 LCD. Console output gets rotated at boot as Miix 310. Also nuke all the (void *)s in driver_data assignment. Signed-off-by: David Santamaría Rogado --- .../gpu/drm/drm_panel_orientation_quirks.c| 31 +-- 1 file changed, 22 insertions(+), 9 deletions(-) diff --git a/drivers/gpu/drm/drm_panel_orientation_quirks.c b/drivers/gpu/drm/drm_panel_orientation_quirks.c index 52e445bb1aa58..835574e2d5bf7 100644 --- a/drivers/gpu/drm/drm_panel_orientation_quirks.c +++ b/drivers/gpu/drm/drm_panel_orientation_quirks.c @@ -80,19 +80,25 @@ static const struct drm_dmi_panel_orientation_data lcd800x1280_rightside_up = { .orientation = DRM_MODE_PANEL_ORIENTATION_RIGHT_UP, }; +static const struct drm_dmi_panel_orientation_data lcd1200x1920_rightside_up = { + .width = 1200, + .height = 1920, + .orientation = DRM_MODE_PANEL_ORIENTATION_RIGHT_UP, +}; + static const struct dmi_system_id orientation_data[] = { { /* Acer One 10 (S1003) */ .matches = { DMI_EXACT_MATCH(DMI_SYS_VENDOR, "Acer"), DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "One S1003"), }, - .driver_data = (void *)_s1003, + .driver_data = _s1003, }, { /* Asus T100HA */ .matches = { DMI_EXACT_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."), DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "T100HAN"), }, - .driver_data = (void *)_t100ha, + .driver_data = _t100ha, }, { /* * GPD Pocket, note that the the DMI data is less generic then * it seems, devices with a board-vendor of "AMI Corporation" @@ -105,7 +111,7 @@ static const struct dmi_system_id orientation_data[] = { DMI_EXACT_MATCH(DMI_BOARD_SERIAL, "Default string"), DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "Default string"), }, - .driver_data = (void *)_pocket, + .driver_data = _pocket, }, { /* GPD Win (same note on DMI match as GPD Pocket) */ .matches = { DMI_EXACT_MATCH(DMI_BOARD_VENDOR, "AMI Corporation"), @@ -113,7 +119,7 @@ static const struct dmi_system_id orientation_data[] = { DMI_EXACT_MATCH(DMI_BOARD_SERIAL, "Default string"), DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "Default string"), }, - .driver_data = (void *)_win, + .driver_data = _win, }, { /* GPD Win 2 (too generic strings, also match on bios date) */ .matches = { DMI_EXACT_MATCH(DMI_SYS_VENDOR, "Default string"), @@ -121,7 +127,7 @@ static const struct dmi_system_id orientation_data[] = { DMI_EXACT_MATCH(DMI_BOARD_VENDOR, "Default string"), DMI_EXACT_MATCH(DMI_BOARD_NAME, "Default string"), }, - .driver_data = (void *)_win2, + .driver_data = _win2, }, { /* I.T.Works TW891 */ .matches = { DMI_EXACT_MATCH(DMI_SYS_VENDOR, "To be filled by O.E.M."), @@ -129,7 +135,14 @@ static const struct dmi_system_id orientation_data[] = { DMI_EXACT_MATCH(DMI_BOARD_VENDOR, "To be filled by O.E.M."), DMI_EXACT_MATCH(DMI_BOARD_NAME, "TW891"), }, - .driver_data = (void *)_tw891, + .driver_data = _tw891, + }, { /* Lenovo Ideapad D330 */ + .matches = { + DMI_EXACT_MATCH(DMI_SYS_VENDOR, "LENOVO"), + DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "81H3"), + DMI_EXACT_MATCH(DMI_PRODUCT_VERSION, "Lenovo ideapad D330-10IGM"), + }, + .driver_data = _rightside_up, }, { /* * Lenovo Ideapad Miix 310 laptop, only some production batches * have a portrait screen, the resolution checks makes the quirk @@ -140,20 +153,20 @@ static const struct dmi_system_id orientation_data[] = { DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "80SG"), DMI_EXACT_MATCH(DMI_PRODUCT_VERSION, "MIIX 310-10ICR"), }, - .driver_data = (void *)_rightside_up, + .driver_data = _rightside_up, }, { /* Lenovo Ideapad Miix 320 */ .matches = { DMI_EXACT_MATCH(DMI_SYS_VENDOR, "LENOVO"), DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "80XF"), DMI_EXACT_MATCH(DMI_PRODUCT_VERSION, "Lenovo MIIX 320-10ICR"), }, - .driver_data = (void *)_rightside_up, + .driver_data = _rightside_up, }, { /* VIOS LTH17 */ .matches = { DMI_EXACT_MATCH(DMI_SYS_VENDOR, "VIOS"), DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "LTH17"), }, - .driver_data = (void *)_rightside_up, + .driver_data = _rightside_up, }, {} }; El mar., 12 feb. 2019 a las 10:51, Jani Nikula () escribió: > > On Mon, 11 Feb 2019, David Santamaría Rogado wrote: > > Lenovo Ideapad D330 Pentium CPU version has 1920x1200 LCD. Console > > ouput gets rotated at boot as Miix 310. > > > > Signed-off-by: David Santamaría Rogado > > --- > > drivers/gpu/drm/drm_panel_orientation_quirks.c | 13 + > > 1 file changed, 13 insertions(+) > > > > diff --git a/drivers/gpu/drm/drm_panel_orientation_quirks.c > > b/drivers/gpu/drm/drm_panel_orientation_quirks.c > > index 52e445bb1aa58..521aff99b08a6 100644 > > --- a/drivers/gpu/drm/drm_panel_orientation_quirks.c > > +++ b/drivers/gpu/drm/drm_panel_orientation_quirks.c > > @@ -80,6 +80,12 @@ static const struct drm_dmi_panel_orientation_data > > lcd800x1280_rightside_up = { > > .orientation = DRM_MODE_PANEL_ORIENTATION_RIGHT_UP, > > }; > > > > +static const struct drm_dmi_panel_orientation_data > >
Re: [PATCH RESEND] drm: panel-orientation-quirks: Add quirk for Lenovo Ideapad D330
On Tue, 12 Feb 2019, David Santamaría Rogado wrote: > Lenovo Ideapad D330 Pentium CPU version has 1920x1200 LCD. Console > output gets rotated at boot as Miix 310. > > Also nuke all the (void *)s in driver_data assignment. Err, I said I'd ack a patch removing all those superfluous cast, but I didn't mean you should do that in the same patch adding a new quirk. Sorry. Communication is hard. But cleanups and functional changes should be separate. BR, Jani. > > Signed-off-by: David Santamaría Rogado --- >.../gpu/drm/drm_panel_orientation_quirks.c | 31 +-- 1 >file changed, 22 insertions(+), 9 deletions(-) > > diff --git a/drivers/gpu/drm/drm_panel_orientation_quirks.c > b/drivers/gpu/drm/drm_panel_orientation_quirks.c > index 52e445bb1aa58..835574e2d5bf7 100644 > --- a/drivers/gpu/drm/drm_panel_orientation_quirks.c > +++ b/drivers/gpu/drm/drm_panel_orientation_quirks.c > @@ -80,19 +80,25 @@ static const struct drm_dmi_panel_orientation_data > lcd800x1280_rightside_up = { > .orientation = DRM_MODE_PANEL_ORIENTATION_RIGHT_UP, > }; > > +static const struct drm_dmi_panel_orientation_data > lcd1200x1920_rightside_up = { > + .width = 1200, > + .height = 1920, > + .orientation = DRM_MODE_PANEL_ORIENTATION_RIGHT_UP, > +}; > + > static const struct dmi_system_id orientation_data[] = { > { /* Acer One 10 (S1003) */ > .matches = { >DMI_EXACT_MATCH(DMI_SYS_VENDOR, "Acer"), >DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "One S1003"), > }, > - .driver_data = (void *)_s1003, > + .driver_data = _s1003, > }, { /* Asus T100HA */ > .matches = { >DMI_EXACT_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."), >DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "T100HAN"), > }, > - .driver_data = (void *)_t100ha, > + .driver_data = _t100ha, > }, { /* > * GPD Pocket, note that the the DMI data is less generic then > * it seems, devices with a board-vendor of "AMI Corporation" > @@ -105,7 +111,7 @@ static const struct dmi_system_id orientation_data[] = { >DMI_EXACT_MATCH(DMI_BOARD_SERIAL, "Default string"), >DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "Default string"), > }, > - .driver_data = (void *)_pocket, > + .driver_data = _pocket, > }, { /* GPD Win (same note on DMI match as GPD Pocket) */ > .matches = { >DMI_EXACT_MATCH(DMI_BOARD_VENDOR, "AMI Corporation"), > @@ -113,7 +119,7 @@ static const struct dmi_system_id orientation_data[] = { >DMI_EXACT_MATCH(DMI_BOARD_SERIAL, "Default string"), >DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "Default string"), > }, > - .driver_data = (void *)_win, > + .driver_data = _win, > }, { /* GPD Win 2 (too generic strings, also match on bios date) */ > .matches = { >DMI_EXACT_MATCH(DMI_SYS_VENDOR, "Default string"), > @@ -121,7 +127,7 @@ static const struct dmi_system_id orientation_data[] = { >DMI_EXACT_MATCH(DMI_BOARD_VENDOR, "Default string"), >DMI_EXACT_MATCH(DMI_BOARD_NAME, "Default string"), > }, > - .driver_data = (void *)_win2, > + .driver_data = _win2, > }, { /* I.T.Works TW891 */ > .matches = { >DMI_EXACT_MATCH(DMI_SYS_VENDOR, "To be filled by O.E.M."), > @@ -129,7 +135,14 @@ static const struct dmi_system_id orientation_data[] = { >DMI_EXACT_MATCH(DMI_BOARD_VENDOR, "To be filled by O.E.M."), >DMI_EXACT_MATCH(DMI_BOARD_NAME, "TW891"), > }, > - .driver_data = (void *)_tw891, > + .driver_data = _tw891, > + }, { /* Lenovo Ideapad D330 */ > + .matches = { > + DMI_EXACT_MATCH(DMI_SYS_VENDOR, "LENOVO"), > + DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "81H3"), > + DMI_EXACT_MATCH(DMI_PRODUCT_VERSION, "Lenovo ideapad D330-10IGM"), > + }, > + .driver_data = _rightside_up, > }, { /* > * Lenovo Ideapad Miix 310 laptop, only some production batches > * have a portrait screen, the resolution checks makes the quirk > @@ -140,20 +153,20 @@ static const struct dmi_system_id orientation_data[] = { >DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "80SG"), >DMI_EXACT_MATCH(DMI_PRODUCT_VERSION, "MIIX 310-10ICR"), > }, > - .driver_data = (void *)_rightside_up, > + .driver_data = _rightside_up, > }, { /* Lenovo Ideapad Miix 320 */ > .matches = { >DMI_EXACT_MATCH(DMI_SYS_VENDOR, "LENOVO"), >DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "80XF"), >DMI_EXACT_MATCH(DMI_PRODUCT_VERSION, "Lenovo MIIX 320-10ICR"), > }, > - .driver_data = (void *)_rightside_up, > + .driver_data = _rightside_up, > }, { /* VIOS LTH17 */ > .matches = { >DMI_EXACT_MATCH(DMI_SYS_VENDOR, "VIOS"), >DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "LTH17"), > }, > - .driver_data = (void *)_rightside_up, > + .driver_data = _rightside_up, > }, > {} > }; > > El mar., 12 feb. 2019 a las 10:51, Jani Nikula > () escribió: >> >> On Mon, 11 Feb 2019, David Santamaría Rogado wrote: >> > Lenovo Ideapad D330 Pentium CPU version has 1920x1200 LCD. Console >> > ouput gets rotated at boot as Miix 310. >> > >> > Signed-off-by: David Santamaría Rogado >> > --- >> > drivers/gpu/drm/drm_panel_orientation_quirks.c | 13 + >> > 1 file changed, 13 insertions(+) >> > >> >
Re: [PATCH RESEND] drm: panel-orientation-quirks: Add quirk for Lenovo Ideapad D330
On Mon, 11 Feb 2019, David Santamaría Rogado wrote: > Lenovo Ideapad D330 Pentium CPU version has 1920x1200 LCD. Console > ouput gets rotated at boot as Miix 310. > > Signed-off-by: David Santamaría Rogado > --- > drivers/gpu/drm/drm_panel_orientation_quirks.c | 13 + > 1 file changed, 13 insertions(+) > > diff --git a/drivers/gpu/drm/drm_panel_orientation_quirks.c > b/drivers/gpu/drm/drm_panel_orientation_quirks.c > index 52e445bb1aa58..521aff99b08a6 100644 > --- a/drivers/gpu/drm/drm_panel_orientation_quirks.c > +++ b/drivers/gpu/drm/drm_panel_orientation_quirks.c > @@ -80,6 +80,12 @@ static const struct drm_dmi_panel_orientation_data > lcd800x1280_rightside_up = { > .orientation = DRM_MODE_PANEL_ORIENTATION_RIGHT_UP, > }; > > +static const struct drm_dmi_panel_orientation_data > lcd1200x1920_rightside_up = { > + .width = 1200, > + .height = 1920, > + .orientation = DRM_MODE_PANEL_ORIENTATION_RIGHT_UP, > +}; > + > static const struct dmi_system_id orientation_data[] = { > { /* Acer One 10 (S1003) */ > .matches = { > @@ -148,6 +154,13 @@ static const struct dmi_system_id orientation_data[] = { >DMI_EXACT_MATCH(DMI_PRODUCT_VERSION, "Lenovo MIIX 320-10ICR"), > }, > .driver_data = (void *)_rightside_up, > + }, { /* Lenovo Ideapad D330 */ > + .matches = { > + DMI_EXACT_MATCH(DMI_SYS_VENDOR, "LENOVO"), > + DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "81H3"), > + DMI_EXACT_MATCH(DMI_PRODUCT_VERSION, "Lenovo ideapad D330-10IGM"), > + }, > + .driver_data = (void *)_rightside_up, The (void *) cast is superfluous. I'd ack a patch nuking them all. BR, Jani. > }, { /* VIOS LTH17 */ > .matches = { >DMI_EXACT_MATCH(DMI_SYS_VENDOR, "VIOS"), > ___ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Jani Nikula, Intel Open Source Graphics Center ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel