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

Damien Lespiau damien.lespiau at intel.com
Mon Jun 2 13:23:26 CEST 2014


On Sun, Jun 01, 2014 at 07:24:47PM +0530, Shashank Sharma wrote:
>  
>  /* XXX: all bits reserved */
> -#define _MIPIA_AUTOPWG				(VLV_DISPLAY_BASE + 0x611a0)
> +#define _MIPIA_AUTOPWG			(dev_priv->mipi_mmio_base + 0x611a0)

This one isn't part of the MIPI block address space IIUC, so we
shouldn't express it as mipi_mmio_base + offset. Let's leave
VLV_DISPLAY_BASE there.

> -#define _MIPIB_READ_DATA_VALID			(VLV_DISPLAY_BASE + 0xb938)
> +#define _MIPIA_READ_DATA_VALID		(dev_priv->mipi_mmio_base + 0xb138)
> +#define _MIPIB_READ_DATA_VALID		(dev_priv->mipi_mmio_base + 0xb938)
>  #define MIPI_READ_DATA_VALID(pipe)	_PIPE(pipe, _MIPIA_READ_DATA_VALID, _MIPIB_READ_DATA_VALID)
>  #define  READ_DATA_VALID(n)				(1 << (n))
>  
> +
>  /* For UMS only (deprecated): */

That's an extra line here. Details, sure, but that's the rigour expected
from submissions.

HTH,

-- 
Damien



More information about the Intel-gfx mailing list