Re: [Intel-gfx] [PATCH] drm/i915/tgl: Fix REVID macros for TGL to fetch correct stepping

2020-11-25 Thread Chris Wilson
Quoting Aditya Swarup (2020-11-25 17:51:04) > On 11/25/20 7:33 AM, Chris Wilson wrote: > > Quoting Jani Nikula (2020-11-25 11:45:56) > >> On Tue, 24 Nov 2020, Aditya Swarup wrote: > >>> static inline const struct i915_rev_steppings * > >>> tgl_revids_get(struct drm_i915_private *dev_priv) > >>>

Re: [Intel-gfx] [PATCH] drm/i915/tgl: Fix REVID macros for TGL to fetch correct stepping

2020-11-25 Thread Lucas De Marchi
On Wed, Nov 25, 2020 at 11:30:44AM -0800, Aditya Swarup wrote: As I said in the other reply, sizeof does actually work here: The question is not about sizeof() not working but rather the usage of ARRAY_SIZE() macro in i915_drv.h with just extern declaration without size specified. ARRAY_SIZE

Re: [Intel-gfx] [PATCH] drm/i915/tgl: Fix REVID macros for TGL to fetch correct stepping

2020-11-25 Thread Aditya Swarup
On 11/25/20 11:29 AM, Lucas De Marchi wrote: > On Wed, Nov 25, 2020 at 09:51:04AM -0800, Aditya Swarup wrote: >> On 11/25/20 7:33 AM, Chris Wilson wrote: >>> Quoting Jani Nikula (2020-11-25 11:45:56) On Tue, 24 Nov 2020, Aditya Swarup wrote: > + if (IS_TGL_U(dev_priv) || IS_TGL_Y(dev_

Re: [Intel-gfx] [PATCH] drm/i915/tgl: Fix REVID macros for TGL to fetch correct stepping

2020-11-25 Thread Aditya Swarup
On 11/25/20 11:18 AM, Lucas De Marchi wrote: > On Wed, Nov 25, 2020 at 09:51:04AM -0800, Aditya Swarup wrote: >> On 11/25/20 7:33 AM, Chris Wilson wrote: >>> Quoting Jani Nikula (2020-11-25 11:45:56) On Tue, 24 Nov 2020, Aditya Swarup wrote: > Fix TGL REVID macros to fetch correct display

Re: [Intel-gfx] [PATCH] drm/i915/tgl: Fix REVID macros for TGL to fetch correct stepping

2020-11-25 Thread Lucas De Marchi
On Wed, Nov 25, 2020 at 09:51:04AM -0800, Aditya Swarup wrote: On 11/25/20 7:33 AM, Chris Wilson wrote: Quoting Jani Nikula (2020-11-25 11:45:56) On Tue, 24 Nov 2020, Aditya Swarup wrote: + if (IS_TGL_U(dev_priv) || IS_TGL_Y(dev_priv)) { + if (TGL_UY_REVID_RANGE(revid)) { +

Re: [Intel-gfx] [PATCH] drm/i915/tgl: Fix REVID macros for TGL to fetch correct stepping

2020-11-25 Thread Lucas De Marchi
On Wed, Nov 25, 2020 at 09:51:04AM -0800, Aditya Swarup wrote: On 11/25/20 7:33 AM, Chris Wilson wrote: Quoting Jani Nikula (2020-11-25 11:45:56) On Tue, 24 Nov 2020, Aditya Swarup wrote: Fix TGL REVID macros to fetch correct display/gt stepping based on SOC rev id from INTEL_REVID() macro. P

Re: [Intel-gfx] [PATCH] drm/i915/tgl: Fix REVID macros for TGL to fetch correct stepping

2020-11-25 Thread Lucas De Marchi
On Wed, Nov 25, 2020 at 03:33:23PM +, Chris Wilson wrote: Quoting Jani Nikula (2020-11-25 11:45:56) On Tue, 24 Nov 2020, Aditya Swarup wrote: > Fix TGL REVID macros to fetch correct display/gt stepping based > on SOC rev id from INTEL_REVID() macro. Previously, we were just > returning the

Re: [Intel-gfx] [PATCH] drm/i915/tgl: Fix REVID macros for TGL to fetch correct stepping

2020-11-25 Thread Ville Syrjälä
On Wed, Nov 25, 2020 at 09:51:04AM -0800, Aditya Swarup wrote: > On 11/25/20 7:33 AM, Chris Wilson wrote: > > Quoting Jani Nikula (2020-11-25 11:45:56) > >> On Tue, 24 Nov 2020, Aditya Swarup wrote: > >>> Fix TGL REVID macros to fetch correct display/gt stepping based > >>> on SOC rev id from INTE

Re: [Intel-gfx] [PATCH] drm/i915/tgl: Fix REVID macros for TGL to fetch correct stepping

2020-11-25 Thread Souza, Jose
On Wed, 2020-11-25 at 10:03 -0800, Aditya Swarup wrote: > On 11/25/20 5:21 AM, Souza, Jose wrote: > > On Tue, 2020-11-24 at 16:31 -0800, Aditya Swarup wrote: > > > Fix TGL REVID macros to fetch correct display/gt stepping based > > > on SOC rev id from INTEL_REVID() macro. Previously, we were just

Re: [Intel-gfx] [PATCH] drm/i915/tgl: Fix REVID macros for TGL to fetch correct stepping

2020-11-25 Thread Aditya Swarup
On 11/25/20 5:21 AM, Souza, Jose wrote: > On Tue, 2020-11-24 at 16:31 -0800, Aditya Swarup wrote: >> Fix TGL REVID macros to fetch correct display/gt stepping based >> on SOC rev id from INTEL_REVID() macro. Previously, we were just >> returning the first element of the revid array instead of using

Re: [Intel-gfx] [PATCH] drm/i915/tgl: Fix REVID macros for TGL to fetch correct stepping

2020-11-25 Thread Aditya Swarup
On 11/25/20 7:33 AM, Chris Wilson wrote: > Quoting Jani Nikula (2020-11-25 11:45:56) >> On Tue, 24 Nov 2020, Aditya Swarup wrote: >>> Fix TGL REVID macros to fetch correct display/gt stepping based >>> on SOC rev id from INTEL_REVID() macro. Previously, we were just >>> returning the first element

Re: [Intel-gfx] [PATCH] drm/i915/tgl: Fix REVID macros for TGL to fetch correct stepping

2020-11-25 Thread Chris Wilson
Quoting Jani Nikula (2020-11-25 11:45:56) > On Tue, 24 Nov 2020, Aditya Swarup wrote: > > Fix TGL REVID macros to fetch correct display/gt stepping based > > on SOC rev id from INTEL_REVID() macro. Previously, we were just > > returning the first element of the revid array instead of using > > the

Re: [Intel-gfx] [PATCH] drm/i915/tgl: Fix REVID macros for TGL to fetch correct stepping

2020-11-25 Thread Souza, Jose
On Tue, 2020-11-24 at 16:31 -0800, Aditya Swarup wrote: > Fix TGL REVID macros to fetch correct display/gt stepping based > on SOC rev id from INTEL_REVID() macro. Previously, we were just > returning the first element of the revid array instead of using > the correct index based on SOC rev id. >

Re: [Intel-gfx] [PATCH] drm/i915/tgl: Fix REVID macros for TGL to fetch correct stepping

2020-11-25 Thread Jani Nikula
On Tue, 24 Nov 2020, Aditya Swarup wrote: > Fix TGL REVID macros to fetch correct display/gt stepping based > on SOC rev id from INTEL_REVID() macro. Previously, we were just > returning the first element of the revid array instead of using > the correct index based on SOC rev id. > > Also, add ar

Re: [Intel-gfx] [PATCH] drm/i915/tgl: Fix REVID macros for TGL to fetch correct stepping

2020-11-24 Thread kernel test robot
Hi Aditya, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on drm-intel/for-linux-next] [also build test WARNING on drm-tip/drm-tip v5.10-rc5 next-20201124] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we sugges

Re: [Intel-gfx] [PATCH] drm/i915/tgl: Fix REVID macros for TGL to fetch correct stepping

2020-11-24 Thread kernel test robot
Hi Aditya, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on drm-intel/for-linux-next] [also build test WARNING on drm-tip/drm-tip v5.10-rc5 next-20201124] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we sugges