[U-Boot] [PATCH] video: ipu_disp: squash clang warning
Troy Kisky
troy.kisky at boundarydevices.com
Mon Jun 16 21:52:22 CEST 2014
On 6/16/2014 12:41 PM, Jeroen Hofstee wrote:
> Since &rgb2ycbcr_coeff and friends are declared const, but
> assigned to a void pointer, clang will warn that the const
> is implicity casted away. Add const to indicate the pointer
> points to const data to get rid of these warnings.
>
> Cc: Stefano Babic <sbabic at denx.de>
> Signed-off-by: Jeroen Hofstee <jeroen at myspectrum.nl>
> ---
> drivers/video/ipu_disp.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/video/ipu_disp.c b/drivers/video/ipu_disp.c
> index cefd2dc..31b5a10 100644
> --- a/drivers/video/ipu_disp.c
> +++ b/drivers/video/ipu_disp.c
> @@ -33,7 +33,7 @@ enum csc_type_t {
>
> struct dp_csc_param_t {
> int mode;
> - void *coeff;
> + void * const coeff;
> };
>
> #define SYNC_WAVE 0
>
Doesn't this says the pointer is constant, not that it points to constant data?
So, commit log needs updated or
const void *coeff;
Troy
More information about the U-Boot
mailing list