Subject: Re: [PATCH 1/2] drm/edid: Add helper to detect whether EDID changed
To: Paul Kocialkowski <paul.kocialkowski@linux.intel.com>,
        dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org,
        intel-gfx@lists.freedesktop.org
Cc: Daniel Vetter <daniel.vetter@intel.com>
References: <20170724145447.23449-1-paul.kocialkowski@linux.intel.com>
From: Harry Wentland <harry.wentland@amd.com>
Message-ID: <92631572-3a27-431a-581a-2e421fb894cd@amd.com>
Date: Mon, 24 Jul 2017 12:00:32 -0400
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101
 Thunderbird/52.2.1
MIME-Version: 1.0
In-Reply-To: <20170724145447.23449-1-paul.kocialkowski@linux.intel.com>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 8bit
SpamDiagnosticOutput: 1:99
SpamDiagnosticMetadata: NSPM
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Jul 2017 16:00:36.8291 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM3PR1201MB1101
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 2842
Lines: 88

On 2017-07-24 10:54 AM, Paul Kocialkowski wrote:
> This adds a common drm helper to detect whether the EDID changed from
> the last known cached one. This is useful help detect that a monitor was
> changed during a suspend/resume cycle.
> 
> When that happens (a monitor is replaced by another one during suspend),
> no hotplug event will be triggered so the change will not be caught at
> resume time. Detecting that the EDID changed allows detecting it.
> 

This makes sense and could be used by other drivers.

Acked-by: Harry Wentland <harry.wentland@amd.com>

Harry

> Signed-off-by: Paul Kocialkowski <paul.kocialkowski@linux.intel.com>
> ---
>  drivers/gpu/drm/drm_edid.c | 45 +++++++++++++++++++++++++++++++++++++++++++++
>  include/drm/drm_edid.h     |  3 +++
>  2 files changed, 48 insertions(+)
> 
> diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
> index 6bb6337be920..f6ce8bc2907a 100644
> --- a/drivers/gpu/drm/drm_edid.c
> +++ b/drivers/gpu/drm/drm_edid.c
> @@ -5036,3 +5036,48 @@ static void drm_get_displayid(struct drm_connector *connector,
>  	}
>  	return;
>  }
> +
> +/**
> + * drm_check_edid_changed - Check whether the EDID changed since the last update
> + * @connector: connector we're probing
> + * @adapter: I2C adapter to use for DDC
> + *
> + * Check whether the EDID changed since the last time it was updated in the
> + * drm blob cache.
> + *
> + * Return: A boolean indicating whether a change happened or not.
> + */
> +bool drm_check_edid_changed(struct drm_connector *connector,
> +			    struct i2c_adapter *adapter)
> +{
> +	struct drm_property_blob *edid_blob;
> +	struct edid *edid_stored;
> +	struct edid *edid_read;
> +	int ret = 0;
> +
> +	if (!connector->edid_blob_ptr)
> +		return false;
> +
> +	edid_blob = drm_property_blob_get(connector->edid_blob_ptr);
> +	if (!edid_blob)
> +		return false;
> +
> +	if (!edid_blob->data || edid_blob->length != sizeof(struct edid))
> +		goto out;
> +
> +	edid_stored = (struct edid *) edid_blob->data;
> +
> +	edid_read = drm_get_edid(connector, adapter);
> +	if (!edid_read)
> +		goto out;
> +
> +	ret = memcmp(edid_stored, edid_read, sizeof(struct edid));
> +
> +	kfree(edid_read);
> +
> +out:
> +	drm_property_blob_put(edid_blob);
> +
> +	return ret != 0;
> +}
> +EXPORT_SYMBOL_GPL(drm_check_edid_changed);
> diff --git a/include/drm/drm_edid.h b/include/drm/drm_edid.h
> index 1e1908a6b1d6..593a97b269c3 100644
> --- a/include/drm/drm_edid.h
> +++ b/include/drm/drm_edid.h
> @@ -485,4 +485,7 @@ void drm_edid_get_monitor_name(struct edid *edid, char *name,
>  struct drm_display_mode *drm_mode_find_dmt(struct drm_device *dev,
>  					   int hsize, int vsize, int fresh,
>  					   bool rb);
> +bool drm_check_edid_changed(struct drm_connector *connector,
> +			    struct i2c_adapter *adapter);
> +
>  #endif /* __DRM_EDID_H__ */
>