On Tue, 20 Sep 2022, Gustavo Sousa <gustavo.sousa@xxxxxxxxx> wrote:
> Hi, Jani.
>
> On Tue, Sep 20, 2022 at 10:19:53AM +0300, Jani Nikula wrote:
>> On Mon, 19 Sep 2022, Gustavo Sousa <gustavo.sousa@xxxxxxxxx> wrote:
>> > Make the code more readable, which will be more apparent as new
>> > platforms with different hotplug inversion needs are added.
>> >
>> > Signed-off-by: Gustavo Sousa <gustavo.sousa@xxxxxxxxx>
>> > ---
>> > drivers/gpu/drm/i915/i915_irq.c | 25 ++++++++++++++++---------
>> > 1 file changed, 16 insertions(+), 9 deletions(-)
>> >
>> > diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
>> > index de06f293e173..c53d21ae197f 100644
>> > --- a/drivers/gpu/drm/i915/i915_irq.c
>> > +++ b/drivers/gpu/drm/i915/i915_irq.c
>> > @@ -3263,6 +3263,21 @@ static void cherryview_irq_reset(struct drm_i915_private *dev_priv)
>> > spin_unlock_irq(&dev_priv->irq_lock);
>> > }
>> >
>> > +static void setup_hotplug_inversion(struct drm_i915_private *dev_priv)
>> > +{
>> > + u32 invert_bits;
>> > +
>> > + if (HAS_PCH_DG1(dev_priv))
>> > + invert_bits = INVERT_DDIA_HPD |
>> > + INVERT_DDIB_HPD |
>> > + INVERT_DDIC_HPD |
>> > + INVERT_DDID_HPD;
>>
>> Nitpick, the indentation will be off compared to automated indentation.
>
> What do you mean by automated indentation?
For example, hit TAB on the lines using a smart enough editor, which has
been configured to follow kernel coding style. ;)
BR,
Jani.
--
Jani Nikula, Intel Open Source Graphics Center
