Re: [PATCH] soc: qcom: ocmem: add missing includes

[Bjorn Andersson]

On Sun 01 Sep 14:30 PDT 2019, Brian Masney wrote:

> The kbuild bot reported the following compiler errors when compiling on
> MIPS with CONFIG_QCOM_OCMEM disabled:
> 
>   In file included from <command-line>:0:0:
>   >> include/soc/qcom/ocmem.h:43:49: warning: 'struct device' declared
>      inside parameter list will not be visible outside of this
>      definition or declaration
>       static inline struct ocmem *of_get_ocmem(struct device *dev)
>                                                       ^~~~~~
>      include/soc/qcom/ocmem.h: In function 'of_get_ocmem':
>   >> include/soc/qcom/ocmem.h:45:9: error: implicit declaration of
>      function 'ERR_PTR' [-Werror=implicit-function-declaration]
>        return ERR_PTR(-ENODEV);
>               ^~~~~~~
>   >> include/soc/qcom/ocmem.h:45:18: error: 'ENODEV' undeclared (first
>      use in this function)
>        return ERR_PTR(-ENODEV);
> 
> Add the proper includes to fix the compiler errors.
> 
> Signed-off-by: Brian Masney <masneyb@xxxxxxxxxxxxx>
> Reported-by: kbuild test robot <lkp@xxxxxxxxx>

Reviewed-by: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx>

> ---
> My OCMEM series [1] hasn't landed upstream yet so let me know if you
> want me to squash this into the existing patch set. I made this a
> separate patch so that the Reported-by could be included. The kbuild
> report is at [2].
> 
> [1] https://lore.kernel.org/lkml/20190823121637.5861-1-masneyb@xxxxxxxxxxxxx/
> [2] https://lists.01.org/pipermail/kbuild-all/2019-August/063530.html
> 
>  include/soc/qcom/ocmem.h | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/include/soc/qcom/ocmem.h b/include/soc/qcom/ocmem.h
> index a0ae336ba78b..02a8bc2677b1 100644
> --- a/include/soc/qcom/ocmem.h
> +++ b/include/soc/qcom/ocmem.h
> @@ -9,6 +9,9 @@
>   * Copyright (C) 2015 Red Hat. Author: Rob Clark <robdclark@xxxxxxxxx>
>   */
>  
> +#include <linux/device.h>
> +#include <linux/err.h>
> +
>  #ifndef __OCMEM_H__
>  #define __OCMEM_H__
>  
> -- 
> 2.21.0
>