Re: [igt-dev] [PATCH i-g-t v4 08/10] lib/igt_edid: add edid_get_mfg

["Ville Syrjälä" <ville.syrjala@linux.intel.com>]

On Tue, Jun 25, 2019 at 04:14:29PM +0300, Simon Ser wrote:
> This returns the 3-letter manufacturer identifier of an EDID.
> 
> Signed-off-by: Simon Ser <simon.ser@intel.com>
> ---
>  lib/igt_edid.c        | 13 +++++++++++++
>  lib/igt_edid.h        |  1 +
>  tests/kms_chamelium.c | 10 +++-------
>  3 files changed, 17 insertions(+), 7 deletions(-)
> 
> diff --git a/lib/igt_edid.c b/lib/igt_edid.c
> index 6cc5e7dd42c4..cbb7eefff70d 100644
> --- a/lib/igt_edid.c
> +++ b/lib/igt_edid.c
> @@ -172,6 +172,19 @@ void detailed_timing_set_string(struct detailed_timing *dt,
>  		ds->str[len] = '\n';
>  }
>  
> +/**
> + * edid_get_mfg: reads the 3-letter manufacturer identifier
> + *
> + * The string is *not* NULL-terminated.

Maybe we should make it NULL terminated to avoid mishaps?

Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>

> + */
> +void edid_get_mfg(const struct edid *edid, char out[static 3])
> +{
> +	out[0] = ((edid->mfg_id[0] & 0x7C) >> 2) + '@';
> +	out[1] = (((edid->mfg_id[0] & 0x03) << 3) |
> +		 ((edid->mfg_id[1] & 0xE0) >> 5)) + '@';
> +	out[2] = (edid->mfg_id[1] & 0x1F) + '@';
> +}
> +
>  static void edid_set_mfg(struct edid *edid, const char mfg[static 3])
>  {
>  	edid->mfg_id[0] = (mfg[0] - '@') << 2 | (mfg[1] - '@') >> 3;
> diff --git a/lib/igt_edid.h b/lib/igt_edid.h
> index 8d8e30ec0554..47581bb778b0 100644
> --- a/lib/igt_edid.h
> +++ b/lib/igt_edid.h
> @@ -298,6 +298,7 @@ void edid_init(struct edid *edid);
>  void edid_init_with_mode(struct edid *edid, drmModeModeInfo *mode);
>  void edid_update_checksum(struct edid *edid);
>  size_t edid_get_size(const struct edid *edid);
> +void edid_get_mfg(const struct edid *edid, char out[static 3]);
>  void detailed_timing_set_mode(struct detailed_timing *dt, drmModeModeInfo *mode,
>  			      int width_mm, int height_mm);
>  void detailed_timing_set_monitor_range_mode(struct detailed_timing *dt,
> diff --git a/tests/kms_chamelium.c b/tests/kms_chamelium.c
> index 175e5032c973..1d5c0ded8a56 100644
> --- a/tests/kms_chamelium.c
> +++ b/tests/kms_chamelium.c
> @@ -184,7 +184,7 @@ check_analog_bridge(data_t *data, struct chamelium_port *port)
>  	drmModeConnector *connector = chamelium_port_get_connector(
>  	    data->chamelium, port, false);
>  	uint64_t edid_blob_id;
> -	unsigned char *edid;
> +	const struct edid *edid;
>  	char edid_vendor[3];
>  
>  	if (chamelium_port_get_type(port) != DRM_MODE_CONNECTOR_VGA) {
> @@ -198,12 +198,8 @@ check_analog_bridge(data_t *data, struct chamelium_port *port)
>  	igt_assert(edid_blob = drmModeGetPropertyBlob(data->drm_fd,
>  						      edid_blob_id));
>  
> -	edid = (unsigned char *) edid_blob->data;
> -
> -	edid_vendor[0] = ((edid[8] & 0x7c) >> 2) + '@';
> -	edid_vendor[1] = (((edid[8] & 0x03) << 3) |
> -			  ((edid[9] & 0xe0) >> 5)) + '@';
> -	edid_vendor[2] = (edid[9] & 0x1f) + '@';
> +	edid = (const struct edid *) edid_blob->data;
> +	edid_get_mfg(edid, edid_vendor);
>  
>  	drmModeFreePropertyBlob(edid_blob);
>  	drmModeFreeConnector(connector);
> -- 
> 2.22.0
> 
> _______________________________________________
> igt-dev mailing list
> igt-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/igt-dev

-- 
Ville Syrjälä
Intel
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev