[Intel-gfx] [PATCH 4/9] drm/i915/display: start description-based ddi initialization
Lucas De Marchi
lucas.de.marchi at gmail.com
Mon Jun 22 23:50:01 UTC 2020
On Wed, Jan 1, 2020 at 11:19 PM Lucas De Marchi
<lucas.de.marchi at gmail.com> wrote:
>
> On Tue, Dec 31, 2019 at 1:58 AM Jani Nikula <jani.nikula at linux.intel.com> wrote:
> >
> > On Mon, 23 Dec 2019, Lucas De Marchi <lucas.demarchi at intel.com> wrote:
> > > For the latest platforms we can share the logic to initialize the the
> > > ddi, so start moving the most trivial ones to a new setup_outputs_desc()
> > > function that will be responsible for initialization according to a
> > > static const table.
> > >
> > > Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>
> > > ---
> > > drivers/gpu/drm/i915/display/intel_display.c | 96 +++++++++++++------
> > > .../drm/i915/display/intel_display_types.h | 4 +
> > > 2 files changed, 73 insertions(+), 27 deletions(-)
> > >
> > > diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
> > > index 04819b0bd494..b3fb1e03cb0b 100644
> > > --- a/drivers/gpu/drm/i915/display/intel_display.c
> > > +++ b/drivers/gpu/drm/i915/display/intel_display.c
> > > @@ -16221,6 +16221,72 @@ static void intel_pps_init(struct drm_i915_private *dev_priv)
> > > intel_pps_unlock_regs_wa(dev_priv);
> > > }
> > >
> > > +struct intel_output {
> > > + /* Initialize DSI if present */
> > > + void (*dsi_init)(struct drm_i915_private *i915);
> >
> > We'll need to be able to initialize DSI on multiple ports too. I've
> > already drafted a series to do so, maybe I've even sent it to the
> > list. Basically you'd pass the port to icl_dsi_init() too.
> >
> > I don't want that development to get any more complicated than it
> > already is.
Did you make progress on that series? Anywhere for me to look at?
I'd like to respin this series.
Lucas De Marchi
>
> Right now this is just working with what is there. I don't see a problem
> to update this series with the additional port argument, but let's
> keep the series
> independent from each other.
>
> Lucas De Marchi
>
> >
> > BR,
> > Jani.
> >
> >
> > > + struct intel_ddi_port_info ddi_ports[];
> > > +};
> > > +
> > > +static const struct intel_output tgl_output = {
> > > + .dsi_init = icl_dsi_init,
> > > + .ddi_ports = {
> > > + { .port = PORT_A },
> > > + { .port = PORT_B },
> > > + { .port = PORT_D },
> > > + { .port = PORT_E },
> > > + { .port = PORT_F },
> > > + { .port = PORT_G },
> > > + { .port = PORT_H },
> > > + { .port = PORT_I },
> > > + { .port = PORT_NONE }
> > > + }
> > > +};
> > > +
> > > +static const struct intel_output ehl_output = {
> > > + .dsi_init = icl_dsi_init,
> > > + .ddi_ports = {
> > > + { .port = PORT_A },
> > > + { .port = PORT_B },
> > > + { .port = PORT_C },
> > > + { .port = PORT_D },
> > > + { .port = PORT_NONE }
> > > + }
> > > +};
> > > +
> > > +static const struct intel_output gen9lp_output = {
> > > + .dsi_init = vlv_dsi_init,
> > > + .ddi_ports = {
> > > + { .port = PORT_A },
> > > + { .port = PORT_B },
> > > + { .port = PORT_C },
> > > + { .port = PORT_NONE }
> > > + },
> > > +};
> > > +
> > > +/*
> > > + * Use a description-based approach for platforms that can be supported with a
> > > + * static table
> > > + */
> > > +static void setup_ddi_outputs_desc(struct drm_i915_private *i915)
> > > +{
> > > + const struct intel_output *output;
> > > + const struct intel_ddi_port_info *port_info;
> > > +
> > > + if (INTEL_GEN(i915) >= 12)
> > > + output = &tgl_output;
> > > + else if (IS_ELKHARTLAKE(i915))
> > > + output = &ehl_output;
> > > + else if (IS_GEN9_LP(i915))
> > > + output = &gen9lp_output;
> > > +
> > > + for (port_info = output->ddi_ports;
> > > + port_info->port != PORT_NONE; port_info++)
> > > + intel_ddi_init(i915, port_info->port);
> > > +
> > > + if (output->dsi_init)
> > > + output->dsi_init(i915);
> > > +}
> > > +
> > > static void intel_setup_outputs(struct drm_i915_private *dev_priv)
> > > {
> > > struct intel_encoder *encoder;
> > > @@ -16231,22 +16297,9 @@ static void intel_setup_outputs(struct drm_i915_private *dev_priv)
> > > if (!HAS_DISPLAY(dev_priv) || !INTEL_DISPLAY_ENABLED(dev_priv))
> > > return;
> > >
> > > - if (INTEL_GEN(dev_priv) >= 12) {
> > > - intel_ddi_init(dev_priv, PORT_A);
> > > - intel_ddi_init(dev_priv, PORT_B);
> > > - intel_ddi_init(dev_priv, PORT_D);
> > > - intel_ddi_init(dev_priv, PORT_E);
> > > - intel_ddi_init(dev_priv, PORT_F);
> > > - intel_ddi_init(dev_priv, PORT_G);
> > > - intel_ddi_init(dev_priv, PORT_H);
> > > - intel_ddi_init(dev_priv, PORT_I);
> > > - icl_dsi_init(dev_priv);
> > > - } else if (IS_ELKHARTLAKE(dev_priv)) {
> > > - intel_ddi_init(dev_priv, PORT_A);
> > > - intel_ddi_init(dev_priv, PORT_B);
> > > - intel_ddi_init(dev_priv, PORT_C);
> > > - intel_ddi_init(dev_priv, PORT_D);
> > > - icl_dsi_init(dev_priv);
> > > + if (INTEL_GEN(dev_priv) >= 12 || IS_ELKHARTLAKE(dev_priv) ||
> > > + IS_GEN9_LP(dev_priv)) {
> > > + setup_ddi_outputs_desc(dev_priv);
> > > } else if (IS_GEN(dev_priv, 11)) {
> > > intel_ddi_init(dev_priv, PORT_A);
> > > intel_ddi_init(dev_priv, PORT_B);
> > > @@ -16263,17 +16316,6 @@ static void intel_setup_outputs(struct drm_i915_private *dev_priv)
> > > intel_ddi_init(dev_priv, PORT_F);
> > >
> > > icl_dsi_init(dev_priv);
> > > - } else if (IS_GEN9_LP(dev_priv)) {
> > > - /*
> > > - * FIXME: Broxton doesn't support port detection via the
> > > - * DDI_BUF_CTL_A or SFUSE_STRAP registers, find another way to
> > > - * detect the ports.
> > > - */
> > > - intel_ddi_init(dev_priv, PORT_A);
> > > - intel_ddi_init(dev_priv, PORT_B);
> > > - intel_ddi_init(dev_priv, PORT_C);
> > > -
> > > - vlv_dsi_init(dev_priv);
> > > } else if (HAS_DDI(dev_priv)) {
> > > int found;
> > >
> > > diff --git a/drivers/gpu/drm/i915/display/intel_display_types.h b/drivers/gpu/drm/i915/display/intel_display_types.h
> > > index a3a067dacf84..4d2f4ee35812 100644
> > > --- a/drivers/gpu/drm/i915/display/intel_display_types.h
> > > +++ b/drivers/gpu/drm/i915/display/intel_display_types.h
> > > @@ -1376,6 +1376,10 @@ struct intel_dp_mst_encoder {
> > > struct intel_connector *connector;
> > > };
> > >
> > > +struct intel_ddi_port_info {
> > > + enum port port;
> > > +};
> > > +
> > > static inline enum dpio_channel
> > > vlv_dig_port_to_channel(struct intel_digital_port *dig_port)
> > > {
> >
> > --
> > Jani Nikula, Intel Open Source Graphics Center
> > _______________________________________________
> > Intel-gfx mailing list
> > Intel-gfx at lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>
>
>
> --
> Lucas De Marchi
--
Lucas De Marchi
More information about the Intel-gfx
mailing list