[RFC 1/3] xfree86: Store kernel driver name in platform device attribute

Thierry Reding thierry.reding at gmail.com
Thu Feb 13 12:07:44 PST 2014


On Thu, Feb 13, 2014 at 08:41:15AM -0800, Aaron Plattner wrote:
> On 02/13/14 05:22, Thierry Reding wrote:
[...]
> >diff --git a/hw/xfree86/os-support/linux/lnx_platform.c b/hw/xfree86/os-support/linux/lnx_platform.c
[...]
> >@@ -52,6 +53,17 @@ get_drm_info(struct OdevAttributes *attribs, char *path, int delayed_index)
> >      xf86_add_platform_device_attrib(delayed_index,
> >                                      ODEV_ATTRIB_BUSID, buf);
> >      drmFreeBusid(buf);
> >+
> >+    v = drmGetVersion(fd);
> >+    if (!v) {
> >+        ErrorF("getversion failed\n");
> 
> I know there's precedent in this function, but this error message could use
> some more info.  Maybe something like this?
> 
>   xf86Msg(X_ERROR, "%s: failed to query DRM version\n", path);

Yes, that reads a lot better.

> Other than that,
> 
> Reviewed-by: Aaron Plattner <aplattner at nvidia.com>

Thanks!
Thierry
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20140213/cbd163c0/attachment.pgp>


More information about the xorg-devel mailing list