On Fri, 2016-03-18 at 00:42 +0000, Arun Siluvery wrote:
> On 17/03/2016 21:39, Rodrigo Vivi wrote:
> > These IDs were already part of the kernel since:
> > 
> > kernel commit 985dd4360fdf2533fe48a33a4a2094f2e4718dc0
> > Author: Imre Deak <imre.d...@intel.com>
> > Date:   Thu Jan 28 16:04:12 2016 +0200
> > 
> >       drm/i915/bxt: update list of PCIIDs
> > 
> > Signed-off-by: Rodrigo Vivi <rodrigo.v...@intel.com>
> > ---
> >   lib/intel_chipset.h | 6 +++++-
> >   1 file changed, 5 insertions(+), 1 deletion(-)
> > 
> > diff --git a/lib/intel_chipset.h b/lib/intel_chipset.h
> > index f3bd04f..4b40612 100644
> > --- a/lib/intel_chipset.h
> > +++ b/lib/intel_chipset.h
> > @@ -236,6 +236,8 @@ void intel_check_pch(void);
> >   #define PCI_CHIP_BROXTON_0                0x0A84
> >   #define PCI_CHIP_BROXTON_1                0x1A84
> >   #define PCI_CHIP_BROXTON_2                0x5A84
> > +#define PCI_CHIP_BROXTON_3         0x1A85
> > +#define PCI_CHIP_BROXTON_4         0x5A85
> > 
> >   #endif /* __GTK_DOC_IGNORE__ */
> > 
> > @@ -460,7 +462,9 @@ void intel_check_pch(void);
> > 
> >   #define IS_BROXTON(devid) ((devid) == PCI_CHIP_BROXTON_0
> > || \
> >                              (devid) == PCI_CHIP_BROXTON_1 ||
> > \
> > -                            (devid) == PCI_CHIP_BROXTON_2)
> > +                            (devid) == PCI_CHIP_BROXTON_2 ||
> > \
> > +                            (devid) == PCI_CHIP_BROXTON_3 ||
> > \
> > +                            (devid) == PCI_CHIP_BROXTON_4)
> > 
> >   #define IS_GEN9(devid)            (IS_KABYLAKE(devid) || \
> >                              IS_SKYLAKE(devid) || \
> > 
> I have recently added them to libdrm in android, would you also be 
> adding them to libdrm?

I did libdrm one 2 weeks ago. It was reviewed and merged already.

> 
> looks good to me,
> Reviewed-by: Arun Siluvery <arun.siluv...@linux.intel.com>

Thanks for the review. Patch merged.

> 
> regards
> Arun
> 
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to