diff mbox

[libdrm,10/10] drmdevice: print the correct host1x information

Message ID 20180625173626.8046-10-emil.l.velikov@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Emil Velikov June 25, 2018, 5:36 p.m. UTC
From: Emil Velikov <emil.velikov@collabora.com>

While fairly close, the host1x and platform are two separate things.

Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
---
 tests/drmdevice.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Robert Foss June 28, 2018, 4:09 p.m. UTC | #1
Nice catch!

Feel free to add my r-b to this patch.

On 2018-06-25 19:36, Emil Velikov wrote:
> From: Emil Velikov <emil.velikov@collabora.com>
> 
> While fairly close, the host1x and platform are two separate things.
> 
> Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
> ---
>   tests/drmdevice.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/drmdevice.c b/tests/drmdevice.c
> index e9e9d7f1..97ce8ff9 100644
> --- a/tests/drmdevice.c
> +++ b/tests/drmdevice.c
> @@ -86,13 +86,13 @@ print_device_info(drmDevicePtr device, int i, bool print_revision)
>               compatible++;
>           }
>       } else if (device->bustype == DRM_BUS_HOST1X) {
> -        char **compatible = device->deviceinfo.platform->compatible;
> +        char **compatible = device->deviceinfo.host1x->compatible;
>   
>           printf("|  +->host1x\n");
>           printf("|     +->fullname\t%s\n", device->businfo.host1x->fullname);
>   
>           printf("+->deviceinfo\n");
> -        printf("   +->platform\n");
> +        printf("   +->host1x\n");
>           printf("      +->compatible\n");
>   
>           while (*compatible) {
>
diff mbox

Patch

diff --git a/tests/drmdevice.c b/tests/drmdevice.c
index e9e9d7f1..97ce8ff9 100644
--- a/tests/drmdevice.c
+++ b/tests/drmdevice.c
@@ -86,13 +86,13 @@  print_device_info(drmDevicePtr device, int i, bool print_revision)
             compatible++;
         }
     } else if (device->bustype == DRM_BUS_HOST1X) {
-        char **compatible = device->deviceinfo.platform->compatible;
+        char **compatible = device->deviceinfo.host1x->compatible;
 
         printf("|  +->host1x\n");
         printf("|     +->fullname\t%s\n", device->businfo.host1x->fullname);
 
         printf("+->deviceinfo\n");
-        printf("   +->platform\n");
+        printf("   +->host1x\n");
         printf("      +->compatible\n");
 
         while (*compatible) {