On Wed, Mar 21, 2012 at 16:48, Jesse Barnes <jbarnes at virtuousgeek.org>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/> -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20120321/fe54b6cb/attachment-0001.htm>