On Fri, Oct 23, 2015 at 3:13 PM, Ben Widawsky <b...@bwidawsk.net> wrote:
> On Fri, Oct 23, 2015 at 02:38:39PM -0700, Ben Widawsky wrote:
>> I left the function to obtain the revision because it is, and will continue
>> to
>> be useful in the future. I'd rather not have to dig it up every time we need
>> it.
Seems reasonable.
Reviewed-by: Jason Ekstrand <jason.ekstr...@intel.com>
>> Comments left at the implementation to say as much.
>>
>
> Added locally I've also referenced the SHA where I originally missed this:
> commit 28ed1e08e8ba98ebd4ff0b56326372f0df9c73ad
> Author: Ben Widawsky <benjamin.widaw...@intel.com>
> Date: Fri Aug 7 13:58:37 2015 -0700
>
> i965/skl: Remove early platform support
>
>
>> Signed-off-by: Ben Widawsky <b...@bwidawsk.net>
>> ---
>> src/mesa/drivers/dri/i965/brw_device_info.c | 2 +-
>> src/mesa/drivers/dri/i965/brw_device_info.h | 2 +-
>> src/mesa/drivers/dri/i965/intel_screen.c | 14 +++++++++++---
>> 3 files changed, 13 insertions(+), 5 deletions(-)
>>
>> diff --git a/src/mesa/drivers/dri/i965/brw_device_info.c
>> b/src/mesa/drivers/dri/i965/brw_device_info.c
>> index e7a016c..789b93f 100644
>> --- a/src/mesa/drivers/dri/i965/brw_device_info.c
>> +++ b/src/mesa/drivers/dri/i965/brw_device_info.c
>> @@ -363,7 +363,7 @@ static const struct brw_device_info brw_device_info_bxt
>> = {
>> };
>>
>> const struct brw_device_info *
>> -brw_get_device_info(int devid, int revision)
>> +brw_get_device_info(int devid)
>> {
>> const struct brw_device_info *devinfo;
>> switch (devid) {
>> diff --git a/src/mesa/drivers/dri/i965/brw_device_info.h
>> b/src/mesa/drivers/dri/i965/brw_device_info.h
>> index 2a73e93..4911c23 100644
>> --- a/src/mesa/drivers/dri/i965/brw_device_info.h
>> +++ b/src/mesa/drivers/dri/i965/brw_device_info.h
>> @@ -86,4 +86,4 @@ struct brw_device_info
>> /** @} */
>> };
>>
>> -const struct brw_device_info *brw_get_device_info(int devid, int revision);
>> +const struct brw_device_info *brw_get_device_info(int devid);
>> diff --git a/src/mesa/drivers/dri/i965/intel_screen.c
>> b/src/mesa/drivers/dri/i965/intel_screen.c
>> index 590c45d..fb95fb6 100644
>> --- a/src/mesa/drivers/dri/i965/intel_screen.c
>> +++ b/src/mesa/drivers/dri/i965/intel_screen.c
>> @@ -1357,7 +1357,16 @@ set_max_gl_versions(struct intel_screen *screen)
>> }
>> }
>>
>> -static int
>> +/**
>> + * Return the revision (generally the revid field of the PCI header) of the
>> + * graphics device.
>> + *
>> + * XXX: This function is useful to keep around even if it is not currently
>> in
>> + * use. It is necessary for new platforms and revision specific workarounds
>> or
>> + * features. Please don't remove it so that we know it at least continues to
>> + * build.
>> + */
>> +static __attribute__((__unused__)) int
>> brw_get_revision(int fd)
>> {
>> struct drm_i915_getparam gp;
>> @@ -1416,8 +1425,7 @@ __DRIconfig **intelInitScreen2(__DRIscreen *psp)
>> return false;
>>
>> intelScreen->deviceID =
>> drm_intel_bufmgr_gem_get_devid(intelScreen->bufmgr);
>> - intelScreen->devinfo = brw_get_device_info(intelScreen->deviceID,
>> - brw_get_revision(psp->fd));
>> + intelScreen->devinfo = brw_get_device_info(intelScreen->deviceID);
>> if (!intelScreen->devinfo)
>> return false;
>>
>> --
>> 2.6.1
>>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
