On Wed, Mar 21, 2012 at 16:48, Jesse Barnes <[email protected]>wrote:

> +       } else {

+               DRM_DEBUG_KMS("failed to fetch edid\n");
>        }
>


Wouldn't it be prettier if we also add some:
   ...
   err = -ENXIO;

or similar error in that block, to let the caller know that we failed as
well?


>        return ret;
>

-- 
Eugeni Dodonov
 <http://eugeni.dodonov.net/>
_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to