All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/3] drm/i915: fix indentation on skl stepping info
@ 2015-10-20 12:38 Jani Nikula
  2015-10-20 12:38 ` [PATCH 2/3] drm/i915: constify bxt " Jani Nikula
  2015-10-20 12:38 ` [PATCH 3/3] drm/i915: refactor stepping info retrieval Jani Nikula
  0 siblings, 2 replies; 6+ messages in thread
From: Jani Nikula @ 2015-10-20 12:38 UTC (permalink / raw)
  To: intel-gfx; +Cc: jani.nikula

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/i915/intel_csr.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/i915/intel_csr.c b/drivers/gpu/drm/i915/intel_csr.c
index 9e530a739354..7d9c2d42fc1a 100644
--- a/drivers/gpu/drm/i915/intel_csr.c
+++ b/drivers/gpu/drm/i915/intel_csr.c
@@ -178,9 +178,9 @@ struct stepping_info {
 };
 
 static const struct stepping_info skl_stepping_info[] = {
-		{'A', '0'}, {'B', '0'}, {'C', '0'},
-		{'D', '0'}, {'E', '0'}, {'F', '0'},
-		{'G', '0'}, {'H', '0'}, {'I', '0'}
+	{'A', '0'}, {'B', '0'}, {'C', '0'},
+	{'D', '0'}, {'E', '0'}, {'F', '0'},
+	{'G', '0'}, {'H', '0'}, {'I', '0'}
 };
 
 static struct stepping_info bxt_stepping_info[] = {
-- 
2.1.4

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply related	[flat|nested] 6+ messages in thread
* [PATCH 2/3] drm/i915: constify bxt stepping info
  2015-10-20 12:38 [PATCH 1/3] drm/i915: fix indentation on skl stepping info Jani Nikula
@ 2015-10-20 12:38 ` Jani Nikula
  2015-10-20 12:38 ` [PATCH 3/3] drm/i915: refactor stepping info retrieval Jani Nikula
  1 sibling, 0 replies; 6+ messages in thread
From: Jani Nikula @ 2015-10-20 12:38 UTC (permalink / raw)
  To: intel-gfx; +Cc: jani.nikula

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/i915/intel_csr.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/i915/intel_csr.c b/drivers/gpu/drm/i915/intel_csr.c
index 7d9c2d42fc1a..fe9a55cf8212 100644
--- a/drivers/gpu/drm/i915/intel_csr.c
+++ b/drivers/gpu/drm/i915/intel_csr.c
@@ -183,7 +183,7 @@ static const struct stepping_info skl_stepping_info[] = {
 	{'G', '0'}, {'H', '0'}, {'I', '0'}
 };
 
-static struct stepping_info bxt_stepping_info[] = {
+static const struct stepping_info bxt_stepping_info[] = {
 	{'A', '0'}, {'A', '1'}, {'A', '2'},
 	{'B', '0'}, {'B', '1'}, {'B', '2'}
 };
-- 
2.1.4

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply related	[flat|nested] 6+ messages in thread
* [PATCH 3/3] drm/i915: refactor stepping info retrieval
  2015-10-20 12:38 [PATCH 1/3] drm/i915: fix indentation on skl stepping info Jani Nikula
  2015-10-20 12:38 ` [PATCH 2/3] drm/i915: constify bxt " Jani Nikula
@ 2015-10-20 12:38 ` Jani Nikula
  2015-10-21  6:44   ` Daniel Vetter
  1 sibling, 1 reply; 6+ messages in thread
From: Jani Nikula @ 2015-10-20 12:38 UTC (permalink / raw)
  To: intel-gfx; +Cc: jani.nikula

Have only one if ladder for platforms and only one range check for
size. Makes it easier to handle new platforms. Remove the use of
negative return values in char, which might underflow to be positive for
some negative error codes.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/i915/intel_csr.c | 46 ++++++++++++++++++++--------------------
 1 file changed, 23 insertions(+), 23 deletions(-)

diff --git a/drivers/gpu/drm/i915/intel_csr.c b/drivers/gpu/drm/i915/intel_csr.c
index fe9a55cf8212..3e7953254803 100644
--- a/drivers/gpu/drm/i915/intel_csr.c
+++ b/drivers/gpu/drm/i915/intel_csr.c
@@ -188,28 +188,25 @@ static const struct stepping_info bxt_stepping_info[] = {
 	{'B', '0'}, {'B', '1'}, {'B', '2'}
 };
 
-static char intel_get_stepping(struct drm_device *dev)
+static const struct stepping_info *intel_get_stepping_info(struct drm_device *dev)
 {
-	if (IS_SKYLAKE(dev) && (dev->pdev->revision <
-			ARRAY_SIZE(skl_stepping_info)))
-		return skl_stepping_info[dev->pdev->revision].stepping;
-	else if (IS_BROXTON(dev) && (dev->pdev->revision <
-				ARRAY_SIZE(bxt_stepping_info)))
-		return bxt_stepping_info[dev->pdev->revision].stepping;
-	else
-		return -ENODATA;
-}
+	const struct stepping_info *si;
+	unsigned int size;
+
+	if (IS_SKYLAKE(dev)) {
+		size = ARRAY_SIZE(skl_stepping_info);
+		si = skl_stepping_info;
+	} else if (IS_BROXTON(dev)) {
+		size = ARRAY_SIZE(bxt_stepping_info);
+		si = bxt_stepping_info;
+	} else {
+		return NULL;
+	}
 
-static char intel_get_substepping(struct drm_device *dev)
-{
-	if (IS_SKYLAKE(dev) && (dev->pdev->revision <
-			ARRAY_SIZE(skl_stepping_info)))
-		return skl_stepping_info[dev->pdev->revision].substepping;
-	else if (IS_BROXTON(dev) && (dev->pdev->revision <
-			ARRAY_SIZE(bxt_stepping_info)))
-		return bxt_stepping_info[dev->pdev->revision].substepping;
-	else
-		return -ENODATA;
+	if (INTEL_REVID(dev) < size)
+		return si + INTEL_REVID(dev);
+
+	return NULL;
 }
 
 /**
@@ -296,8 +293,8 @@ static void finish_csr_load(const struct firmware *fw, void *context)
 	struct intel_package_header *package_header;
 	struct intel_dmc_header *dmc_header;
 	struct intel_csr *csr = &dev_priv->csr;
-	char stepping = intel_get_stepping(dev);
-	char substepping = intel_get_substepping(dev);
+	const struct stepping_info *stepping_info = intel_get_stepping_info(dev);
+	char stepping, substepping;
 	uint32_t dmc_offset = CSR_DEFAULT_FW_OFFSET, readcount = 0, nbytes;
 	uint32_t i;
 	uint32_t *dmc_payload;
@@ -308,11 +305,14 @@ static void finish_csr_load(const struct firmware *fw, void *context)
 		goto out;
 	}
 
-	if ((stepping == -ENODATA) || (substepping == -ENODATA)) {
+	if (!stepping_info) {
 		DRM_ERROR("Unknown stepping info, firmware loading failed\n");
 		goto out;
 	}
 
+	stepping = stepping_info->stepping;
+	substepping = stepping_info->substepping;
+
 	/* Extract CSS Header information*/
 	css_header = (struct intel_css_header *)fw->data;
 	if (sizeof(struct intel_css_header) !=
-- 
2.1.4

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply related	[flat|nested] 6+ messages in thread
* Re: [PATCH 3/3] drm/i915: refactor stepping info retrieval
  2015-10-20 12:38 ` [PATCH 3/3] drm/i915: refactor stepping info retrieval Jani Nikula
@ 2015-10-21  6:44   ` Daniel Vetter
  2015-10-21  8:12     ` Jani Nikula
  0 siblings, 1 reply; 6+ messages in thread
From: Daniel Vetter @ 2015-10-21  6:44 UTC (permalink / raw)
  To: Jani Nikula; +Cc: intel-gfx

On Tue, Oct 20, 2015 at 03:38:33PM +0300, Jani Nikula wrote:
> Have only one if ladder for platforms and only one range check for
> size. Makes it easier to handle new platforms. Remove the use of
> negative return values in char, which might underflow to be positive for
> some negative error codes.
> 
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>

On the series: Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>

> ---
>  drivers/gpu/drm/i915/intel_csr.c | 46 ++++++++++++++++++++--------------------
>  1 file changed, 23 insertions(+), 23 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/intel_csr.c b/drivers/gpu/drm/i915/intel_csr.c
> index fe9a55cf8212..3e7953254803 100644
> --- a/drivers/gpu/drm/i915/intel_csr.c
> +++ b/drivers/gpu/drm/i915/intel_csr.c
> @@ -188,28 +188,25 @@ static const struct stepping_info bxt_stepping_info[] = {
>  	{'B', '0'}, {'B', '1'}, {'B', '2'}
>  };
>  
> -static char intel_get_stepping(struct drm_device *dev)
> +static const struct stepping_info *intel_get_stepping_info(struct drm_device *dev)
>  {
> -	if (IS_SKYLAKE(dev) && (dev->pdev->revision <
> -			ARRAY_SIZE(skl_stepping_info)))
> -		return skl_stepping_info[dev->pdev->revision].stepping;
> -	else if (IS_BROXTON(dev) && (dev->pdev->revision <
> -				ARRAY_SIZE(bxt_stepping_info)))
> -		return bxt_stepping_info[dev->pdev->revision].stepping;
> -	else
> -		return -ENODATA;
> -}
> +	const struct stepping_info *si;
> +	unsigned int size;
> +
> +	if (IS_SKYLAKE(dev)) {
> +		size = ARRAY_SIZE(skl_stepping_info);
> +		si = skl_stepping_info;
> +	} else if (IS_BROXTON(dev)) {
> +		size = ARRAY_SIZE(bxt_stepping_info);
> +		si = bxt_stepping_info;
> +	} else {
> +		return NULL;
> +	}
>  
> -static char intel_get_substepping(struct drm_device *dev)
> -{
> -	if (IS_SKYLAKE(dev) && (dev->pdev->revision <
> -			ARRAY_SIZE(skl_stepping_info)))
> -		return skl_stepping_info[dev->pdev->revision].substepping;
> -	else if (IS_BROXTON(dev) && (dev->pdev->revision <
> -			ARRAY_SIZE(bxt_stepping_info)))
> -		return bxt_stepping_info[dev->pdev->revision].substepping;
> -	else
> -		return -ENODATA;
> +	if (INTEL_REVID(dev) < size)
> +		return si + INTEL_REVID(dev);
> +
> +	return NULL;
>  }
>  
>  /**
> @@ -296,8 +293,8 @@ static void finish_csr_load(const struct firmware *fw, void *context)
>  	struct intel_package_header *package_header;
>  	struct intel_dmc_header *dmc_header;
>  	struct intel_csr *csr = &dev_priv->csr;
> -	char stepping = intel_get_stepping(dev);
> -	char substepping = intel_get_substepping(dev);
> +	const struct stepping_info *stepping_info = intel_get_stepping_info(dev);
> +	char stepping, substepping;
>  	uint32_t dmc_offset = CSR_DEFAULT_FW_OFFSET, readcount = 0, nbytes;
>  	uint32_t i;
>  	uint32_t *dmc_payload;
> @@ -308,11 +305,14 @@ static void finish_csr_load(const struct firmware *fw, void *context)
>  		goto out;
>  	}
>  
> -	if ((stepping == -ENODATA) || (substepping == -ENODATA)) {
> +	if (!stepping_info) {
>  		DRM_ERROR("Unknown stepping info, firmware loading failed\n");
>  		goto out;
>  	}
>  
> +	stepping = stepping_info->stepping;
> +	substepping = stepping_info->substepping;
> +
>  	/* Extract CSS Header information*/
>  	css_header = (struct intel_css_header *)fw->data;
>  	if (sizeof(struct intel_css_header) !=
> -- 
> 2.1.4
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [PATCH 3/3] drm/i915: refactor stepping info retrieval
  2015-10-21  6:44   ` Daniel Vetter
@ 2015-10-21  8:12     ` Jani Nikula
  2015-11-12 14:33       ` Jani Nikula
  0 siblings, 1 reply; 6+ messages in thread
From: Jani Nikula @ 2015-10-21  8:12 UTC (permalink / raw)
  To: Daniel Vetter; +Cc: intel-gfx

On Wed, 21 Oct 2015, Daniel Vetter <daniel@ffwll.ch> wrote:
> On Tue, Oct 20, 2015 at 03:38:33PM +0300, Jani Nikula wrote:
>> Have only one if ladder for platforms and only one range check for
>> size. Makes it easier to handle new platforms. Remove the use of
>> negative return values in char, which might underflow to be positive for
>> some negative error codes.
>> 
>> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
>
> On the series: Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>

Apparently this conflicts with some other firmware patches. I'll let
Imre decide if we should give them priority, or just merge these.

BR,
Jani.


>
>> ---
>>  drivers/gpu/drm/i915/intel_csr.c | 46 ++++++++++++++++++++--------------------
>>  1 file changed, 23 insertions(+), 23 deletions(-)
>> 
>> diff --git a/drivers/gpu/drm/i915/intel_csr.c b/drivers/gpu/drm/i915/intel_csr.c
>> index fe9a55cf8212..3e7953254803 100644
>> --- a/drivers/gpu/drm/i915/intel_csr.c
>> +++ b/drivers/gpu/drm/i915/intel_csr.c
>> @@ -188,28 +188,25 @@ static const struct stepping_info bxt_stepping_info[] = {
>>  	{'B', '0'}, {'B', '1'}, {'B', '2'}
>>  };
>>  
>> -static char intel_get_stepping(struct drm_device *dev)
>> +static const struct stepping_info *intel_get_stepping_info(struct drm_device *dev)
>>  {
>> -	if (IS_SKYLAKE(dev) && (dev->pdev->revision <
>> -			ARRAY_SIZE(skl_stepping_info)))
>> -		return skl_stepping_info[dev->pdev->revision].stepping;
>> -	else if (IS_BROXTON(dev) && (dev->pdev->revision <
>> -				ARRAY_SIZE(bxt_stepping_info)))
>> -		return bxt_stepping_info[dev->pdev->revision].stepping;
>> -	else
>> -		return -ENODATA;
>> -}
>> +	const struct stepping_info *si;
>> +	unsigned int size;
>> +
>> +	if (IS_SKYLAKE(dev)) {
>> +		size = ARRAY_SIZE(skl_stepping_info);
>> +		si = skl_stepping_info;
>> +	} else if (IS_BROXTON(dev)) {
>> +		size = ARRAY_SIZE(bxt_stepping_info);
>> +		si = bxt_stepping_info;
>> +	} else {
>> +		return NULL;
>> +	}
>>  
>> -static char intel_get_substepping(struct drm_device *dev)
>> -{
>> -	if (IS_SKYLAKE(dev) && (dev->pdev->revision <
>> -			ARRAY_SIZE(skl_stepping_info)))
>> -		return skl_stepping_info[dev->pdev->revision].substepping;
>> -	else if (IS_BROXTON(dev) && (dev->pdev->revision <
>> -			ARRAY_SIZE(bxt_stepping_info)))
>> -		return bxt_stepping_info[dev->pdev->revision].substepping;
>> -	else
>> -		return -ENODATA;
>> +	if (INTEL_REVID(dev) < size)
>> +		return si + INTEL_REVID(dev);
>> +
>> +	return NULL;
>>  }
>>  
>>  /**
>> @@ -296,8 +293,8 @@ static void finish_csr_load(const struct firmware *fw, void *context)
>>  	struct intel_package_header *package_header;
>>  	struct intel_dmc_header *dmc_header;
>>  	struct intel_csr *csr = &dev_priv->csr;
>> -	char stepping = intel_get_stepping(dev);
>> -	char substepping = intel_get_substepping(dev);
>> +	const struct stepping_info *stepping_info = intel_get_stepping_info(dev);
>> +	char stepping, substepping;
>>  	uint32_t dmc_offset = CSR_DEFAULT_FW_OFFSET, readcount = 0, nbytes;
>>  	uint32_t i;
>>  	uint32_t *dmc_payload;
>> @@ -308,11 +305,14 @@ static void finish_csr_load(const struct firmware *fw, void *context)
>>  		goto out;
>>  	}
>>  
>> -	if ((stepping == -ENODATA) || (substepping == -ENODATA)) {
>> +	if (!stepping_info) {
>>  		DRM_ERROR("Unknown stepping info, firmware loading failed\n");
>>  		goto out;
>>  	}
>>  
>> +	stepping = stepping_info->stepping;
>> +	substepping = stepping_info->substepping;
>> +
>>  	/* Extract CSS Header information*/
>>  	css_header = (struct intel_css_header *)fw->data;
>>  	if (sizeof(struct intel_css_header) !=
>> -- 
>> 2.1.4
>> 
>> _______________________________________________
>> Intel-gfx mailing list
>> Intel-gfx@lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
>
> -- 
> Daniel Vetter
> Software Engineer, Intel Corporation
> http://blog.ffwll.ch

-- 
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [PATCH 3/3] drm/i915: refactor stepping info retrieval
  2015-10-21  8:12     ` Jani Nikula
@ 2015-11-12 14:33       ` Jani Nikula
  0 siblings, 0 replies; 6+ messages in thread
From: Jani Nikula @ 2015-11-12 14:33 UTC (permalink / raw)
  To: Daniel Vetter; +Cc: intel-gfx

On Wed, 21 Oct 2015, Jani Nikula <jani.nikula@intel.com> wrote:
> On Wed, 21 Oct 2015, Daniel Vetter <daniel@ffwll.ch> wrote:
>> On Tue, Oct 20, 2015 at 03:38:33PM +0300, Jani Nikula wrote:
>>> Have only one if ladder for platforms and only one range check for
>>> size. Makes it easier to handle new platforms. Remove the use of
>>> negative return values in char, which might underflow to be positive for
>>> some negative error codes.
>>> 
>>> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
>>
>> On the series: Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
>
> Apparently this conflicts with some other firmware patches. I'll let
> Imre decide if we should give them priority, or just merge these.

Imre gave the green light, series pushed to drm-intel-next-queued,
thanks for the review.

BR,
Jani.


>
> BR,
> Jani.
>
>
>>
>>> ---
>>>  drivers/gpu/drm/i915/intel_csr.c | 46 ++++++++++++++++++++--------------------
>>>  1 file changed, 23 insertions(+), 23 deletions(-)
>>> 
>>> diff --git a/drivers/gpu/drm/i915/intel_csr.c b/drivers/gpu/drm/i915/intel_csr.c
>>> index fe9a55cf8212..3e7953254803 100644
>>> --- a/drivers/gpu/drm/i915/intel_csr.c
>>> +++ b/drivers/gpu/drm/i915/intel_csr.c
>>> @@ -188,28 +188,25 @@ static const struct stepping_info bxt_stepping_info[] = {
>>>  	{'B', '0'}, {'B', '1'}, {'B', '2'}
>>>  };
>>>  
>>> -static char intel_get_stepping(struct drm_device *dev)
>>> +static const struct stepping_info *intel_get_stepping_info(struct drm_device *dev)
>>>  {
>>> -	if (IS_SKYLAKE(dev) && (dev->pdev->revision <
>>> -			ARRAY_SIZE(skl_stepping_info)))
>>> -		return skl_stepping_info[dev->pdev->revision].stepping;
>>> -	else if (IS_BROXTON(dev) && (dev->pdev->revision <
>>> -				ARRAY_SIZE(bxt_stepping_info)))
>>> -		return bxt_stepping_info[dev->pdev->revision].stepping;
>>> -	else
>>> -		return -ENODATA;
>>> -}
>>> +	const struct stepping_info *si;
>>> +	unsigned int size;
>>> +
>>> +	if (IS_SKYLAKE(dev)) {
>>> +		size = ARRAY_SIZE(skl_stepping_info);
>>> +		si = skl_stepping_info;
>>> +	} else if (IS_BROXTON(dev)) {
>>> +		size = ARRAY_SIZE(bxt_stepping_info);
>>> +		si = bxt_stepping_info;
>>> +	} else {
>>> +		return NULL;
>>> +	}
>>>  
>>> -static char intel_get_substepping(struct drm_device *dev)
>>> -{
>>> -	if (IS_SKYLAKE(dev) && (dev->pdev->revision <
>>> -			ARRAY_SIZE(skl_stepping_info)))
>>> -		return skl_stepping_info[dev->pdev->revision].substepping;
>>> -	else if (IS_BROXTON(dev) && (dev->pdev->revision <
>>> -			ARRAY_SIZE(bxt_stepping_info)))
>>> -		return bxt_stepping_info[dev->pdev->revision].substepping;
>>> -	else
>>> -		return -ENODATA;
>>> +	if (INTEL_REVID(dev) < size)
>>> +		return si + INTEL_REVID(dev);
>>> +
>>> +	return NULL;
>>>  }
>>>  
>>>  /**
>>> @@ -296,8 +293,8 @@ static void finish_csr_load(const struct firmware *fw, void *context)
>>>  	struct intel_package_header *package_header;
>>>  	struct intel_dmc_header *dmc_header;
>>>  	struct intel_csr *csr = &dev_priv->csr;
>>> -	char stepping = intel_get_stepping(dev);
>>> -	char substepping = intel_get_substepping(dev);
>>> +	const struct stepping_info *stepping_info = intel_get_stepping_info(dev);
>>> +	char stepping, substepping;
>>>  	uint32_t dmc_offset = CSR_DEFAULT_FW_OFFSET, readcount = 0, nbytes;
>>>  	uint32_t i;
>>>  	uint32_t *dmc_payload;
>>> @@ -308,11 +305,14 @@ static void finish_csr_load(const struct firmware *fw, void *context)
>>>  		goto out;
>>>  	}
>>>  
>>> -	if ((stepping == -ENODATA) || (substepping == -ENODATA)) {
>>> +	if (!stepping_info) {
>>>  		DRM_ERROR("Unknown stepping info, firmware loading failed\n");
>>>  		goto out;
>>>  	}
>>>  
>>> +	stepping = stepping_info->stepping;
>>> +	substepping = stepping_info->substepping;
>>> +
>>>  	/* Extract CSS Header information*/
>>>  	css_header = (struct intel_css_header *)fw->data;
>>>  	if (sizeof(struct intel_css_header) !=
>>> -- 
>>> 2.1.4
>>> 
>>> _______________________________________________
>>> Intel-gfx mailing list
>>> Intel-gfx@lists.freedesktop.org
>>> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
>>
>> -- 
>> Daniel Vetter
>> Software Engineer, Intel Corporation
>> http://blog.ffwll.ch

-- 
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-11-12 14:29 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-10-20 12:38 [PATCH 1/3] drm/i915: fix indentation on skl stepping info Jani Nikula
2015-10-20 12:38 ` [PATCH 2/3] drm/i915: constify bxt " Jani Nikula
2015-10-20 12:38 ` [PATCH 3/3] drm/i915: refactor stepping info retrieval Jani Nikula
2015-10-21  6:44   ` Daniel Vetter
2015-10-21  8:12     ` Jani Nikula
2015-11-12 14:33       ` Jani Nikula

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.