[Libva] [PATCH Intel-driver 3/9] Dump chipset information in the vendor string

[Xiang, Haihao]

On Mon, 2014-04-21 at 18:29 +0200, Gwenole Beauchesne wrote: 
> 2014-04-18 18:21 GMT+02:00 Xiang, Haihao <haihao.xiang at intel.com>:
> > From: "Xiang, Haihao" <haihao.xiang at intel.com>
> >
> > Signed-off-by: Xiang, Haihao <haihao.xiang at intel.com>
> > ---
> >  src/i965_drv_video.c | 13 ++++++++++++-
> >  1 file changed, 12 insertions(+), 1 deletion(-)
> >
> > diff --git a/src/i965_drv_video.c b/src/i965_drv_video.c
> > index 27d6d38..c2b2416 100755
> > --- a/src/i965_drv_video.c
> > +++ b/src/i965_drv_video.c
> > @@ -5168,6 +5168,7 @@ i965_Init(VADriverContextP ctx)
> >  {
> >      struct i965_driver_data *i965 = i965_driver_data(ctx);
> >      int i;
> > +    const char *chipset;
> >
> >      for (i = 0; i < ARRAY_ELEMS(i965_sub_ops); i++) {
> >          if ((i965_sub_ops[i].display_type == 0 ||
> > @@ -5177,9 +5178,19 @@ i965_Init(VADriverContextP ctx)
> >      }
> >
> >      if (i == ARRAY_ELEMS(i965_sub_ops)) {
> > -        sprintf(i965->va_vendor, "%s %s driver - %d.%d.%d",
> > +        switch (i965->intel.device_id) {
> > +#undef CHIPSET
> > +#define CHIPSET(id, family, dev, str) case id: chipset = str; break;
> > +#include "i965_pciids.h"
> > +        default:
> > +            chipset = "Unknown Intel Chipset";
> > +            break;
> > +        }
> > +
> > +        sprintf(i965->va_vendor, "%s %s driver %s - %d.%d.%d",
> 
> "driver for %s" would probably look better than "driver %s".

Thanks for your comment. I will follow your suggestion.

> 
> >                  INTEL_STR_DRIVER_VENDOR,
> >                  INTEL_STR_DRIVER_NAME,
> > +                chipset,
> >                  INTEL_DRIVER_MAJOR_VERSION,
> >                  INTEL_DRIVER_MINOR_VERSION,
> >                  INTEL_DRIVER_MICRO_VERSION);
> > --
> > 1.8.3.2
> >
> > _______________________________________________
> > Libva mailing list
> > Libva at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/libva