On Wed, 21 Mar 2012 17:44:12 -0300 Eugeni Dodonov <[email protected]> wrote:
> 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; > > > Maybe but I think most callers just check for the EDID later anyway. Really this patch can be dropped, but I was wondering exactly where the GPIO side of things was failing and added some debug output... -- Jesse Barnes, Intel Open Source Technology Center
signature.asc
Description: PGP signature
_______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
