[Freedreno] [PATCH 01/13] include: Move ascii85 functions from i915 to linux/ascii85.h

Chris Wilson chris at chris-wilson.co.uk
Thu Jul 12 19:34:25 UTC 2018


Quoting Jordan Crouse (2018-07-12 19:59:18)
> The i915 DRM driver very cleverly used ascii85 encoding for their
> GPU state file. Move the encode functions to a general header file to
> support other drivers that might be interested in the same
> functionality.
> 
> v3: Fix error_puts -> err_puts pointed out by the 01.org bot
> v2: Update API to be cleaner for the caller as suggested by Chris Wilson
> 
> Signed-off-by: Jordan Crouse <jcrouse at codeaurora.org>
> ---
> +static inline long
> +ascii85_encode_len(long len)
> +{
> +       return DIV_ROUND_UP(len, 4);
> +}
> +
> +static inline char *

const char * to avoid a compiler warning with return "z".

Looks like that will be ok with the callers.

With that,
Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
-Chris


More information about the Freedreno mailing list