From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jani Nikula Subject: Re: [PATCH] drm/i915: BDW: Adding Reserved PCI IDs. Date: Tue, 24 Jun 2014 16:19:38 +0300 Message-ID: <87oaxi8mxh.fsf@intel.com> References: <20140611001700.GA15475@bwidawsk.net> <1402422067-5717-1-git-send-email-rodrigo.vivi@intel.com> <20140611174743.GA950@bwidawsk.net> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mga03.intel.com (mga03.intel.com [143.182.124.21]) by gabe.freedesktop.org (Postfix) with ESMTP id A4E566E12D for ; Tue, 24 Jun 2014 06:19:43 -0700 (PDT) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Rodrigo Vivi , Ben Widawsky Cc: intel-gfx , Rodrigo Vivi List-Id: intel-gfx@lists.freedesktop.org On Fri, 13 Jun 2014, Rodrigo Vivi wrote: > Hi Daniel, > > please consider to merge the first version. So we can move fwd with ddx > patche followed by proper marketing names. Pushed v1 to -fixes, thanks for the patch and review. BR, Jani. > > Thanks, > Rodrigo. > > > On Wed, Jun 11, 2014 at 10:47 AM, Ben Widawsky wrote: > >> On Tue, Jun 10, 2014 at 10:41:07AM -0700, Rodrigo Vivi wrote: >> > These PCI IDs are reserved on BSpec and can be used at any time in the >> future. >> > So let's add this now in order to avoid issues that we already faced on >> previous >> > platforms, like finding out about new ids when user reported >> accelaration weren't >> > enabled. >> > >> > v2: Reserved IDs doesn't have GT defined. So, creating a separated list. >> (Ben) >> > >> > Cc: Ben Widawsky >> > Signed-off-by: Rodrigo Vivi >> > --- >> > include/drm/i915_pciids.h | 16 ++++++++++++++-- >> > 1 file changed, 14 insertions(+), 2 deletions(-) >> > >> > diff --git a/include/drm/i915_pciids.h b/include/drm/i915_pciids.h >> > index 0572035..0968478 100644 >> > --- a/include/drm/i915_pciids.h >> > +++ b/include/drm/i915_pciids.h >> > @@ -237,13 +237,25 @@ >> > #define INTEL_BDW_GT3D_IDS(info) \ >> > _INTEL_BDW_D_IDS(3, info) >> > >> > +#define INTEL_BDW_RSVDM_IDS(info) \ >> > + INTEL_VGA_DEVICE(0x1632, info), \ >> > + INTEL_VGA_DEVICE(0x1636, info), \ >> > + INTEL_VGA_DEVICE(0x163B, info), \ >> > + INTEL_VGA_DEVICE(0x163A, info) >> > + >> > +#define INTEL_BDW_RSVDD_IDS(info) \ >> > + INTEL_VGA_DEVICE(0x163D, info), \ >> > + INTEL_VGA_DEVICE(0x163E, info) >> > + >> > #define INTEL_BDW_M_IDS(info) \ >> > INTEL_BDW_GT12M_IDS(info), \ >> > - INTEL_BDW_GT3M_IDS(info) >> > + INTEL_BDW_GT3M_IDS(info), \ >> > + INTEL_BDW_RSVDM_IDS(info) >> > >> > #define INTEL_BDW_D_IDS(info) \ >> > INTEL_BDW_GT12D_IDS(info), \ >> > - INTEL_BDW_GT3D_IDS(info) >> > + INTEL_BDW_GT3D_IDS(info), \ >> > + INTEL_BDW_RSVDD_IDS(info) >> > >> > #define INTEL_CHV_IDS(info) \ >> > INTEL_VGA_DEVICE(0x22b0, info), \ >> >> I thought we saved off the GT info, but now that I actually look at the >> code, we do not. Therefore, I actually think v1 is a better patch. >> >> In either case, both v1 and v2 are: >> Reviewed-by: Ben Widawsky >> >> I apologize for the extra work. >> >> -- >> Ben Widawsky, Intel Open Source Technology Center >> _______________________________________________ >> Intel-gfx mailing list >> Intel-gfx@lists.freedesktop.org >> http://lists.freedesktop.org/mailman/listinfo/intel-gfx >> > > > > -- > Rodrigo Vivi > Blog: http://blog.vivi.eng.br > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Jani Nikula, Intel Open Source Technology Center