On 18/03/14 14:59, Jonathan Gray wrote:
> Signed-off-by: Jonathan Gray <j...@jsg.id.au>
> ---
Hi Jonathan

While the summary covers what the patch does, the *ahem* commit message
fails to explain why it's needed. AFAICS this will cause some very nasty
breakage in some cases, which we want to avoid without a valid reason.

-Emil

>  src/loader/loader.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git src/loader/loader.c src/loader/loader.c
> index caeeebf..1744acb 100644
> --- src/loader/loader.c
> +++ src/loader/loader.c
> @@ -202,7 +202,7 @@ out:
>     return (*chip_id >= 0);
>  }
>  
> -#elif defined(ANDROID) && !defined(__NOT_HAVE_DRM_H)
> +#elif !defined(__NOT_HAVE_DRM_H)
>  
>  /* for i915 */
>  #include <i915_drm.h>
> 

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to