Re: [PATCH i-g-t] i915_pciids: Change a KBL pci id to GT2 from GT1.5

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Sep 22, 2017 at 11:23:34AM +0000, Jani Nikula wrote:
> On Fri, 22 Sep 2017, Jani Nikula <jani.nikula@xxxxxxxxxxxxxxx> wrote:
> > On Fri, 22 Sep 2017, Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> wrote:
> >> On Thu, Sep 21, 2017 at 10:03:48PM +0000, Anuj Phogat wrote:
> >>> On Thu, Sep 21, 2017 at 2:58 PM, Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> wrote:
> >>> > In sync with 41693fd52373 ("drm/i915/kbl: Change a KBL pci id
> >>> > to GT2 from GT1.5")
> >>> >
> >>> > "See Mesa commit 9c588ff"
> >>> >
> >>> > Cc: Anuj Phogat <anuj.phogat@xxxxxxxxx>
> >>> > Signed-off-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>
> >>> > ---
> >>> >  lib/i915_pciids.h | 2 +-
> >>> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >>> >
> >>> > diff --git a/lib/i915_pciids.h b/lib/i915_pciids.h
> >>> > index 3e4f9614..0d2b125c 100644
> >>> > --- a/lib/i915_pciids.h
> >>> > +++ b/lib/i915_pciids.h
> >>> > @@ -303,7 +303,6 @@
> >>> >  #define INTEL_KBL_GT1_IDS(info)        \
> >>> >         INTEL_VGA_DEVICE(0x5913, info), /* ULT GT1.5 */ \
> >>> >         INTEL_VGA_DEVICE(0x5915, info), /* ULX GT1.5 */ \
> >>> > -       INTEL_VGA_DEVICE(0x5917, info), /* DT  GT1.5 */ \
> >>> >         INTEL_VGA_DEVICE(0x5906, info), /* ULT GT1 */ \
> >>> >         INTEL_VGA_DEVICE(0x590E, info), /* ULX GT1 */ \
> >>> >         INTEL_VGA_DEVICE(0x5902, info), /* DT  GT1 */ \
> >>> > @@ -313,6 +312,7 @@
> >>> >
> >>> >  #define INTEL_KBL_GT2_IDS(info)        \
> >>> >         INTEL_VGA_DEVICE(0x5916, info), /* ULT GT2 */ \
> >>> > +       INTEL_VGA_DEVICE(0x5917, info), /* DT  GT2 */   \
> >>> Mobile GT2
> >>> >         INTEL_VGA_DEVICE(0x5921, info), /* ULT GT2F */ \
> >>> >         INTEL_VGA_DEVICE(0x591E, info), /* ULX GT2 */ \
> >>> >         INTEL_VGA_DEVICE(0x5912, info), /* DT  GT2 */ \
> >>> > --
> >>> > 2.13.5
> >>> >
> >>> With above change:
> >>> Reviewed-by: Anuj Phogat <anuj.phogat@xxxxxxxxx>
> >>
> >> Fixed and merged.
> >> Thanks for starting this and for the review.
> >
> > Should it have been Cc: stable? Fixes:?
> 
> ...was the kernel patch...?

Hmm... It din't occur to me to add since it was merely informative
and the commit that added here was right by that time.

But for consistency and coherency that should've been :(

> 
> J.
> 
> >
> > BR,
> > Jani.
> >
> >> _______________________________________________
> >> Intel-gfx mailing list
> >> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> >> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
> 
> -- 
> Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux