Re: [PATCH 0/5] drm/i915: Miscellaneous fixes to reduce dependency for I915_MAX_PIPES

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 2017-10-10 at 12:19 +0300, Ville Syrjälä wrote:
> On Thu, Sep 14, 2017 at 10:53:01AM +0300, Mika Kahola wrote:
> > 
> > This patch series introduces fixes to reduce dependency for
> > I915_MAX_PIPES and minor optimizations to reduce hardcoding.
> > 
> > Kahola, Mika (1):
> >   drm/i915: Remove I915_MAX_PIPES dependency for DDB allocation
> > 
> > Mika Kahola (4):
> >   drm/i915: Don't relay on I915_MAX_PIPES
> >   drm/i915: Fold IRQ pipe masks
> >   drm/i915: Favor for_each_pipe() macro
> >   drm/i915: Cleanup South Error Interrupts
> I replied with a few minor nits to the individual patches. Ttoally up
> to
> you whether you want to do those changes or not.
> 
> For the series
> Reviewed-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
Thanks Ville for review. I will change it 'enum pipe pipe'. It's
cleaner that way.

> 
> > 
> > 
> >  drivers/gpu/drm/i915/i915_irq.c       | 19 +++++++------------
> >  drivers/gpu/drm/i915/i915_reg.h       |  3 ---
> >  drivers/gpu/drm/i915/intel_audio.c    |  2 +-
> >  drivers/gpu/drm/i915/intel_display.c  |  5 ++++-
> >  drivers/gpu/drm/i915/intel_drv.h      |  3 ++-
> >  drivers/gpu/drm/i915/intel_pipe_crc.c |  9 +++++----
> >  drivers/gpu/drm/i915/intel_pm.c       |  6 ++++--
> >  7 files changed, 23 insertions(+), 24 deletions(-)
> > 
> > -- 
> > 2.7.4
> > 
> > _______________________________________________
> > Intel-gfx mailing list
> > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
-- 
Mika Kahola - Intel OTC

_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux