Hi, On Wed, Jun 15, 2022 at 6:36 AM Doug Anderson <dianders@xxxxxxxxxxxx> wrote: > > Hi, > > On Tue, Jun 14, 2022 at 10:50 PM Hsin-Yi Wang <hsinyi@xxxxxxxxxxxx> wrote: > > > > On Thu, Jun 9, 2022 at 3:27 PM Hsin-Yi Wang <hsinyi@xxxxxxxxxxxx> wrote: > > > > > > Panels usually call drm_connector_set_panel_orientation(), which is > > > later than drm/kms driver calling drm_dev_register(). This leads to a > > > WARN()[1]. > > > > > > The orientation property is known earlier. For example, some panels > > > parse the property through device tree during probe. > > > > > > The series add a panel API drm_connector_set_orientation_from_panel() > > > for drm/kms drivers. The drivers can call the API to set panel's > > > orientation before drm_dev_register(). > > > > > > Panel needs to implement .get_orientation callback to return the property. > > > > > > [1] https://patchwork.kernel.org/project/linux-mediatek/patch/20220530081910.3947168-2-hsinyi@xxxxxxxxxxxx/ > > > > > > Hsin-Yi Wang (8): > > > drm/panel: Add an API to allow drm to set orientation from panel > > > drm/panel: boe-tv101wum-nl6: Implement .get_orientation callback > > > drm/panel: panel-edp: Implement .get_orientation callback > > > drm/panel: lvds: Implement .get_orientation callback > > > drm/panel: panel-simple: Implement .get_orientation callback > > > drm/panel: ili9881c: Implement .get_orientation callback > > > drm/panel: elida-kd35t133: Implement .get_orientation callback > > > drm: Config orientation property if panel provides it > > > > > hi Maintainers, > > > > All the patches are reviewed. If there's no other comments, will this > > series be picked? Thanks. > > Unless someone beat me to it or yells, my plan was to land them to > drm-misc-next next week. Since it touches core code I wanted to give a > little extra time. Also at the moment patch #8 is all Chromium (all > author and reviewers are chromium.org) at the moment so that's another > reason to make sure it has sufficient time on the lists. As promised, I've pushed to drm-misc-next. Note: I had to fix whitespace problems in almost every patch. Please run checkpatch on your patches. 15b9ca1641f0 drm: Config orientation property if panel provides it a64af136771a drm/panel: elida-kd35t133: Implement .get_orientation callback db9b4ffbfa82 drm/panel: ili9881c: Implement .get_orientation callback a960e35ac011 drm/panel: panel-simple: Implement .get_orientation callback c0398df2a195 drm/panel: lvds: Implement .get_orientation callback 47bef230225b drm/panel: panel-edp: Implement .get_orientation callback 3fa214eea43a drm/panel: boe-tv101wum-nl6: Implement .get_orientation callback 5e41b01a7808 drm/panel: Add an API to allow drm to set orientation from panel -Doug
