Re: [PATCH v7 0/8] Add a panel API to set orientation properly

[Doug Anderson]

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