[Intel-gfx] [PATCH 2/2] drm/i915: Change Mipi register definitions

Damien Lespiau damien.lespiau at intel.com
Wed May 21 17:49:51 CEST 2014


On Wed, May 21, 2014 at 06:35:12PM +0300, Ville Syrjälä wrote:
> > +
> > +/* VLV port control */
> >  #define _MIPIA_PORT_CTRL			(VLV_DISPLAY_BASE + 0x61190)
> >  #define _MIPIB_PORT_CTRL			(VLV_DISPLAY_BASE + 0x61700)
> >  #define MIPI_PORT_CTRL(pipe)		_PIPE(pipe, _MIPIA_PORT_CTRL, _MIPIB_PORT_CTRL)
> 
> Why isn't mipi_mmio_base used here? Does the register not need the new
> offset? I htink it would still be cleaner to use the mipi_mmio_offset
> for all the MIPI registers.

On VLV, this register (a at least one other) isn't part of the MIPI IP
block address space (base + 0xbxxx) so shouldn't be defined as
mipi_mmio_base + offset.

-- 
Damien



More information about the Intel-gfx mailing list