On 08/05/17 14:32, Laurent Pinchart wrote:
> The DPI code only needs to differentiate between major OMAP revisions,
> which can be obtained from the DSS compatible string. Replace the OMAP
> SoC model checks to prepare for removal of the OMAP SoC version platform
> data.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> ---
>  drivers/gpu/drm/omapdrm/dss/dpi.c | 59 +++++++++++++++++----------------------
>  drivers/gpu/drm/omapdrm/dss/dss.c | 10 ++++++-
>  drivers/gpu/drm/omapdrm/dss/dss.h | 13 +++++++--
>  3 files changed, 45 insertions(+), 37 deletions(-)
> 
> diff --git a/drivers/gpu/drm/omapdrm/dss/dpi.c b/drivers/gpu/drm/omapdrm/dss/dpi.c
> index 3d87f3af72bb..b5cb23c167db 100644
> --- a/drivers/gpu/drm/omapdrm/dss/dpi.c
> +++ b/drivers/gpu/drm/omapdrm/dss/dpi.c
> @@ -39,6 +39,7 @@
>  
>  struct dpi_data {
>  	struct platform_device *pdev;
> +	enum dss_device_type type;

I really don't like "dss_device_type" or "type" here... "dss_version"?
Or maybe it should be tied to DPI, so "dpi_version"?

 Tomi