All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] qcom-scm: Include <linux/err.h> header
@ 2018-12-26 12:06 Fabio Estevam
  2018-12-28 19:31 ` Bjorn Andersson
  2019-01-04 21:15 ` Guenter Roeck
  0 siblings, 2 replies; 16+ messages in thread
From: Fabio Estevam @ 2018-12-26 12:06 UTC (permalink / raw)
  To: robdclark
  Cc: jonathan, dri-devel, bjorn.andersson, andy.gross, shawnguo, cphealy

Since commit e6f6d63ed14c ("drm/msm: add headless gpu device for imx5")
the DRM_MSM symbol can be selected by SOC_IMX5 causing the following
error when building imx_v6_v7_defconfig:

In file included from ../drivers/gpu/drm/msm/adreno/a5xx_gpu.c:17:0:
../include/linux/qcom_scm.h: In function 'qcom_scm_set_cold_boot_addr':
../include/linux/qcom_scm.h:73:10: error: 'ENODEV' undeclared (first use in this function)
  return -ENODEV;

Include the <linux/err.h> header file to fix this problem.

Reported-by: kernelci.org bot <bot@kernelci.org>
Fixes: e6f6d63ed14c ("drm/msm: add headless gpu device for imx5")
Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
 include/linux/qcom_scm.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/include/linux/qcom_scm.h b/include/linux/qcom_scm.h
index 06996ad4f2bc..ce5a476fd733 100644
--- a/include/linux/qcom_scm.h
+++ b/include/linux/qcom_scm.h
@@ -13,6 +13,7 @@
 #ifndef __QCOM_SCM_H
 #define __QCOM_SCM_H
 
+#include <linux/err.h>
 #include <linux/types.h>
 #include <linux/cpumask.h>
 
-- 
2.17.1

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 16+ messages in thread

* Re: [PATCH] qcom-scm: Include <linux/err.h> header
  2018-12-28 19:31 ` Bjorn Andersson
@ 2018-12-28 15:39   ` Jonathan marek
  2018-12-29  0:12     ` Bjorn Andersson
  2018-12-28 19:52   ` Fabio Estevam
  1 sibling, 1 reply; 16+ messages in thread
From: Jonathan marek @ 2018-12-28 15:39 UTC (permalink / raw)
  To: Bjorn Andersson, Fabio Estevam, andy.gross
  Cc: linux-arm-msm, shawnguo, cphealy, dri-devel

FYI, I already had a patch fixing this error (it is in linux-next: 
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/log/include/linux/qcom_scm.h). 
This one is probably better though.

On 12/28/2018 02:31 PM, Bjorn Andersson wrote:
> On Wed 26 Dec 04:06 PST 2018, Fabio Estevam wrote:
> 
>> Since commit e6f6d63ed14c ("drm/msm: add headless gpu device for imx5")
>> the DRM_MSM symbol can be selected by SOC_IMX5 causing the following
>> error when building imx_v6_v7_defconfig:
>>
>> In file included from ../drivers/gpu/drm/msm/adreno/a5xx_gpu.c:17:0:
>> ../include/linux/qcom_scm.h: In function 'qcom_scm_set_cold_boot_addr':
>> ../include/linux/qcom_scm.h:73:10: error: 'ENODEV' undeclared (first use in this function)
>>    return -ENODEV;
>>
>> Include the <linux/err.h> header file to fix this problem.
>>
> 
> Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org>
> 
> Andy, please pick up for inclusion in -rc
> 
> Fabio, please use get_maintainers, so your patches hits the appropriate
> mailing lists (linux-arm-msm@ in this case)
> 
> Regards,
> Bjorn
> 
>> Reported-by: kernelci.org bot <bot@kernelci.org>
>> Fixes: e6f6d63ed14c ("drm/msm: add headless gpu device for imx5")
>> Signed-off-by: Fabio Estevam <festevam@gmail.com>
>> ---
>>   include/linux/qcom_scm.h | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/include/linux/qcom_scm.h b/include/linux/qcom_scm.h
>> index 06996ad4f2bc..ce5a476fd733 100644
>> --- a/include/linux/qcom_scm.h
>> +++ b/include/linux/qcom_scm.h
>> @@ -13,6 +13,7 @@
>>   #ifndef __QCOM_SCM_H
>>   #define __QCOM_SCM_H
>>   
>> +#include <linux/err.h>
>>   #include <linux/types.h>
>>   #include <linux/cpumask.h>
>>   
>> -- 
>> 2.17.1
>>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH] qcom-scm: Include <linux/err.h> header
  2018-12-26 12:06 [PATCH] qcom-scm: Include <linux/err.h> header Fabio Estevam
@ 2018-12-28 19:31 ` Bjorn Andersson
  2018-12-28 15:39   ` Jonathan marek
  2018-12-28 19:52   ` Fabio Estevam
  2019-01-04 21:15 ` Guenter Roeck
  1 sibling, 2 replies; 16+ messages in thread
From: Bjorn Andersson @ 2018-12-28 19:31 UTC (permalink / raw)
  To: Fabio Estevam, andy.gross
  Cc: jonathan, linux-arm-msm, dri-devel, shawnguo, cphealy

On Wed 26 Dec 04:06 PST 2018, Fabio Estevam wrote:

> Since commit e6f6d63ed14c ("drm/msm: add headless gpu device for imx5")
> the DRM_MSM symbol can be selected by SOC_IMX5 causing the following
> error when building imx_v6_v7_defconfig:
> 
> In file included from ../drivers/gpu/drm/msm/adreno/a5xx_gpu.c:17:0:
> ../include/linux/qcom_scm.h: In function 'qcom_scm_set_cold_boot_addr':
> ../include/linux/qcom_scm.h:73:10: error: 'ENODEV' undeclared (first use in this function)
>   return -ENODEV;
> 
> Include the <linux/err.h> header file to fix this problem.
> 

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

Andy, please pick up for inclusion in -rc

Fabio, please use get_maintainers, so your patches hits the appropriate
mailing lists (linux-arm-msm@ in this case)

Regards,
Bjorn

> Reported-by: kernelci.org bot <bot@kernelci.org>
> Fixes: e6f6d63ed14c ("drm/msm: add headless gpu device for imx5")
> Signed-off-by: Fabio Estevam <festevam@gmail.com>
> ---
>  include/linux/qcom_scm.h | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/include/linux/qcom_scm.h b/include/linux/qcom_scm.h
> index 06996ad4f2bc..ce5a476fd733 100644
> --- a/include/linux/qcom_scm.h
> +++ b/include/linux/qcom_scm.h
> @@ -13,6 +13,7 @@
>  #ifndef __QCOM_SCM_H
>  #define __QCOM_SCM_H
>  
> +#include <linux/err.h>
>  #include <linux/types.h>
>  #include <linux/cpumask.h>
>  
> -- 
> 2.17.1
> 
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH] qcom-scm: Include <linux/err.h> header
  2018-12-28 19:31 ` Bjorn Andersson
  2018-12-28 15:39   ` Jonathan marek
@ 2018-12-28 19:52   ` Fabio Estevam
  2018-12-28 19:56     ` Fabio Estevam
                       ` (2 more replies)
  1 sibling, 3 replies; 16+ messages in thread
From: Fabio Estevam @ 2018-12-28 19:52 UTC (permalink / raw)
  To: Bjorn Andersson
  Cc: Jonathan Marek, linux-arm-msm, DRI mailing list, Andy Gross,
	Shawn Guo, Chris Healy

Hi Bjorn,

On Fri, Dec 28, 2018 at 5:31 PM Bjorn Andersson
<bjorn.andersson@linaro.org> wrote:
>
> On Wed 26 Dec 04:06 PST 2018, Fabio Estevam wrote:
>
> > Since commit e6f6d63ed14c ("drm/msm: add headless gpu device for imx5")
> > the DRM_MSM symbol can be selected by SOC_IMX5 causing the following
> > error when building imx_v6_v7_defconfig:
> >
> > In file included from ../drivers/gpu/drm/msm/adreno/a5xx_gpu.c:17:0:
> > ../include/linux/qcom_scm.h: In function 'qcom_scm_set_cold_boot_addr':
> > ../include/linux/qcom_scm.h:73:10: error: 'ENODEV' undeclared (first use in this function)
> >   return -ENODEV;
> >
> > Include the <linux/err.h> header file to fix this problem.
> >
>
> Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org>
>
> Andy, please pick up for inclusion in -rc

Yes, it would be really nice if we could get this fix into 4.20-rc1 so
that imx_v6_v7_defconfig could be built.

> Fabio, please use get_maintainers, so your patches hits the appropriate
> mailing lists (linux-arm-msm@ in this case)

Sorry, I copied the folks involved in the offending commit:
e6f6d63ed14c ("drm/msm: add headless gpu device for imx5")

By the way, I just ran get_maintainers in this patch and linux-arm-msm
is not listed.

Thanks
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH] qcom-scm: Include <linux/err.h> header
  2018-12-28 19:52   ` Fabio Estevam
@ 2018-12-28 19:56     ` Fabio Estevam
  2018-12-28 20:08       ` Fabio Estevam
  2018-12-28 20:00     ` Fabio Estevam
  2018-12-29  0:27     ` Bjorn Andersson
  2 siblings, 1 reply; 16+ messages in thread
From: Fabio Estevam @ 2018-12-28 19:56 UTC (permalink / raw)
  To: Bjorn Andersson
  Cc: Jonathan Marek, linux-arm-msm, DRI mailing list, Andy Gross,
	Shawn Guo, Chris Healy

Hi Bjorn,

On Fri, Dec 28, 2018 at 5:52 PM Fabio Estevam <festevam@gmail.com> wrote:

> By the way, I just ran get_maintainers in this patch and linux-arm-msm
> is not listed.

Would you like to me to submit a patch like this to fix this problem?

diff --git a/MAINTAINERS b/MAINTAINERS
index 7a9804a891fd..e014de05b046 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -4797,6 +4797,7 @@ L:        freedreno@lists.freedesktop.org
 T:     git git://people.freedesktop.org/~robclark/linux
 S:     Maintained
 F:     drivers/gpu/drm/msm/
+F:     include/linux/qcom_scm.h
 F:     include/uapi/drm/msm_drm.h
 F:     Documentation/devicetree/bindings/display/msm/
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 16+ messages in thread

* Re: [PATCH] qcom-scm: Include <linux/err.h> header
  2018-12-28 19:52   ` Fabio Estevam
  2018-12-28 19:56     ` Fabio Estevam
@ 2018-12-28 20:00     ` Fabio Estevam
  2018-12-29  0:27     ` Bjorn Andersson
  2 siblings, 0 replies; 16+ messages in thread
From: Fabio Estevam @ 2018-12-28 20:00 UTC (permalink / raw)
  To: Bjorn Andersson
  Cc: Jonathan Marek, linux-arm-msm, DRI mailing list, Andy Gross,
	Shawn Guo, Chris Healy

On Fri, Dec 28, 2018 at 5:52 PM Fabio Estevam <festevam@gmail.com> wrote:

> > Andy, please pick up for inclusion in -rc
>
> Yes, it would be really nice if we could get this fix into 4.20-rc1 so
> that imx_v6_v7_defconfig could be built.

Ops, I meant 4.21-rc1 :-)
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH] qcom-scm: Include <linux/err.h> header
  2018-12-28 19:56     ` Fabio Estevam
@ 2018-12-28 20:08       ` Fabio Estevam
  0 siblings, 0 replies; 16+ messages in thread
From: Fabio Estevam @ 2018-12-28 20:08 UTC (permalink / raw)
  To: Bjorn Andersson
  Cc: Jonathan Marek, linux-arm-msm, DRI mailing list, Andy Gross,
	Shawn Guo, Chris Healy

On Fri, Dec 28, 2018 at 5:56 PM Fabio Estevam <festevam@gmail.com> wrote:
>
> Hi Bjorn,
>
> On Fri, Dec 28, 2018 at 5:52 PM Fabio Estevam <festevam@gmail.com> wrote:
>
> > By the way, I just ran get_maintainers in this patch and linux-arm-msm
> > is not listed.
>
> Would you like to me to submit a patch like this to fix this problem?

Just realized that include/linux/qcom_scm.h is used in several
subsystems, so maybe a better location would be under ARM/QUALCOMM
SUPPORT?

diff --git a/MAINTAINERS b/MAINTAINERS
index 7a9804a891fd..77836b9a8ffd 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1950,6 +1950,7 @@ F:        drivers/tty/serial/msm_serial.c
 F:     drivers/*/pm8???-*
 F:     drivers/mfd/ssbi.c
 F:     drivers/firmware/qcom_scm*
+F:     include/linux/qcom_scm.h
 T:     git git://git.kernel.org/pub/scm/linux/kernel/git/agross/linux.git

 ARM/RADISYS ENP2611 MACHINE SUPPORT
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 16+ messages in thread

* Re: [PATCH] qcom-scm: Include <linux/err.h> header
  2018-12-28 15:39   ` Jonathan marek
@ 2018-12-29  0:12     ` Bjorn Andersson
  0 siblings, 0 replies; 16+ messages in thread
From: Bjorn Andersson @ 2018-12-29  0:12 UTC (permalink / raw)
  To: Jonathan marek; +Cc: dri-devel, linux-arm-msm, andy.gross, shawnguo, cphealy

On Fri 28 Dec 07:39 PST 2018, Jonathan marek wrote:

> FYI, I already had a patch fixing this error (it is in linux-next: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/log/include/linux/qcom_scm.h).
> This one is probably better though.
> 

I thought I had seen this before, but I was unable to find your patch
when searching my inbox.

That patch has been sent to and picked up by arm-soc for v4.21, so we're
good.

Thanks,
Bjorn

> On 12/28/2018 02:31 PM, Bjorn Andersson wrote:
> > On Wed 26 Dec 04:06 PST 2018, Fabio Estevam wrote:
> > 
> > > Since commit e6f6d63ed14c ("drm/msm: add headless gpu device for imx5")
> > > the DRM_MSM symbol can be selected by SOC_IMX5 causing the following
> > > error when building imx_v6_v7_defconfig:
> > > 
> > > In file included from ../drivers/gpu/drm/msm/adreno/a5xx_gpu.c:17:0:
> > > ../include/linux/qcom_scm.h: In function 'qcom_scm_set_cold_boot_addr':
> > > ../include/linux/qcom_scm.h:73:10: error: 'ENODEV' undeclared (first use in this function)
> > >    return -ENODEV;
> > > 
> > > Include the <linux/err.h> header file to fix this problem.
> > > 
> > 
> > Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org>
> > 
> > Andy, please pick up for inclusion in -rc
> > 
> > Fabio, please use get_maintainers, so your patches hits the appropriate
> > mailing lists (linux-arm-msm@ in this case)
> > 
> > Regards,
> > Bjorn
> > 
> > > Reported-by: kernelci.org bot <bot@kernelci.org>
> > > Fixes: e6f6d63ed14c ("drm/msm: add headless gpu device for imx5")
> > > Signed-off-by: Fabio Estevam <festevam@gmail.com>
> > > ---
> > >   include/linux/qcom_scm.h | 1 +
> > >   1 file changed, 1 insertion(+)
> > > 
> > > diff --git a/include/linux/qcom_scm.h b/include/linux/qcom_scm.h
> > > index 06996ad4f2bc..ce5a476fd733 100644
> > > --- a/include/linux/qcom_scm.h
> > > +++ b/include/linux/qcom_scm.h
> > > @@ -13,6 +13,7 @@
> > >   #ifndef __QCOM_SCM_H
> > >   #define __QCOM_SCM_H
> > > +#include <linux/err.h>
> > >   #include <linux/types.h>
> > >   #include <linux/cpumask.h>
> > > -- 
> > > 2.17.1
> > > 
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH] qcom-scm: Include <linux/err.h> header
  2018-12-28 19:52   ` Fabio Estevam
  2018-12-28 19:56     ` Fabio Estevam
  2018-12-28 20:00     ` Fabio Estevam
@ 2018-12-29  0:27     ` Bjorn Andersson
  2018-12-29 12:19       ` Fabio Estevam
  2 siblings, 1 reply; 16+ messages in thread
From: Bjorn Andersson @ 2018-12-29  0:27 UTC (permalink / raw)
  To: Fabio Estevam
  Cc: Jonathan Marek, linux-arm-msm, DRI mailing list, Andy Gross,
	Shawn Guo, Chris Healy

On Fri 28 Dec 11:52 PST 2018, Fabio Estevam wrote:

> Hi Bjorn,
> 
> On Fri, Dec 28, 2018 at 5:31 PM Bjorn Andersson
> <bjorn.andersson@linaro.org> wrote:
[..]
> > Fabio, please use get_maintainers, so your patches hits the appropriate
> > mailing lists (linux-arm-msm@ in this case)
[..]
> By the way, I just ran get_maintainers in this patch and linux-arm-msm
> is not listed.
> 

Sorry about that, I forgot that the header file is not covered by the
MAINTAINERS file. 

Your second patch looks good, but I'm hoping we can merge the upcoming
v3 of Amit's patch right after the merge window. It fixes this and a lot
of other pieces where we would like to include linux-arm-msm@:

https://lore.kernel.org/lkml/d153a86748f99526e7790bfc4ef8781a2016fd51.1545126964.git.amit.kucheria@linaro.org/

Regards,
Bjorn
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH] qcom-scm: Include <linux/err.h> header
  2018-12-29  0:27     ` Bjorn Andersson
@ 2018-12-29 12:19       ` Fabio Estevam
  2018-12-29 19:51         ` Andy Gross
  0 siblings, 1 reply; 16+ messages in thread
From: Fabio Estevam @ 2018-12-29 12:19 UTC (permalink / raw)
  To: Bjorn Andersson
  Cc: Jonathan Marek, linux-arm-msm, DRI mailing list, Andy Gross,
	Shawn Guo, Chris Healy

Hi Bjorn,

On Fri, Dec 28, 2018 at 10:27 PM Bjorn Andersson
<bjorn.andersson@linaro.org> wrote:

> Sorry about that, I forgot that the header file is not covered by the
> MAINTAINERS file.
>
> Your second patch looks good, but I'm hoping we can merge the upcoming
> v3 of Amit's patch right after the merge window. It fixes this and a lot
> of other pieces where we would like to include linux-arm-msm@:
>
> https://lore.kernel.org/lkml/d153a86748f99526e7790bfc4ef8781a2016fd51.1545126964.git.amit.kucheria@linaro.org/

Amit's patch adds the following entry:

+F: include/linux/*/qcom*

but it does not catch include/linux/qcom_scm.h

It also needs

+F: include/linux/qcom*

in order to catch include/linux/qcom-geni-se.h  and include/linux/qcom_scm.h

I can add that entry after Amit's patch gets applied.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH] qcom-scm: Include <linux/err.h> header
  2018-12-29 12:19       ` Fabio Estevam
@ 2018-12-29 19:51         ` Andy Gross
  2019-01-14 19:50           ` Amit Kucheria
  0 siblings, 1 reply; 16+ messages in thread
From: Andy Gross @ 2018-12-29 19:51 UTC (permalink / raw)
  To: Fabio Estevam
  Cc: Jonathan Marek, linux-arm-msm, DRI mailing list, Bjorn Andersson,
	Shawn Guo, Chris Healy

On Sat, Dec 29, 2018 at 10:19:32AM -0200, Fabio Estevam wrote:
> Hi Bjorn,
> 
> On Fri, Dec 28, 2018 at 10:27 PM Bjorn Andersson
> <bjorn.andersson@linaro.org> wrote:
> 
> > Sorry about that, I forgot that the header file is not covered by the
> > MAINTAINERS file.
> >
> > Your second patch looks good, but I'm hoping we can merge the upcoming
> > v3 of Amit's patch right after the merge window. It fixes this and a lot
> > of other pieces where we would like to include linux-arm-msm@:
> >
> > https://lore.kernel.org/lkml/d153a86748f99526e7790bfc4ef8781a2016fd51.1545126964.git.amit.kucheria@linaro.org/
> 
> Amit's patch adds the following entry:
> 
> +F: include/linux/*/qcom*
> 
> but it does not catch include/linux/qcom_scm.h
> 
> It also needs
> 
> +F: include/linux/qcom*
> 
> in order to catch include/linux/qcom-geni-se.h  and include/linux/qcom_scm.h
> 
> I can add that entry after Amit's patch gets applied.

Or I can add it to Amit's.  I'll ping him to make sure that's ok.

Thanks,

Andy
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH] qcom-scm: Include <linux/err.h> header
  2018-12-26 12:06 [PATCH] qcom-scm: Include <linux/err.h> header Fabio Estevam
  2018-12-28 19:31 ` Bjorn Andersson
@ 2019-01-04 21:15 ` Guenter Roeck
  1 sibling, 0 replies; 16+ messages in thread
From: Guenter Roeck @ 2019-01-04 21:15 UTC (permalink / raw)
  To: Fabio Estevam
  Cc: robdclark, jonathan, dri-devel, bjorn.andersson, andy.gross,
	shawnguo, cphealy, linux-kernel, freedreno, linux-arm-msm

On Wed, Dec 26, 2018 at 10:06:19AM -0200, Fabio Estevam wrote:
> Since commit e6f6d63ed14c ("drm/msm: add headless gpu device for imx5")
> the DRM_MSM symbol can be selected by SOC_IMX5 causing the following
> error when building imx_v6_v7_defconfig:
> 
> In file included from ../drivers/gpu/drm/msm/adreno/a5xx_gpu.c:17:0:
> ../include/linux/qcom_scm.h: In function 'qcom_scm_set_cold_boot_addr':
> ../include/linux/qcom_scm.h:73:10: error: 'ENODEV' undeclared (first use in this function)
>   return -ENODEV;
> 
> Include the <linux/err.h> header file to fix this problem.
> 
> Reported-by: kernelci.org bot <bot@kernelci.org>
> Fixes: e6f6d63ed14c ("drm/msm: add headless gpu device for imx5")
> Signed-off-by: Fabio Estevam <festevam@gmail.com>
> Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org>

Tested-by: Guenter Roeck <linux@roeck-us.net>

Still broken upstream, and the commit window is about to close.
Any chance to send this patch upstream anytime soon ?

Guenter

> ---
>  include/linux/qcom_scm.h | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/include/linux/qcom_scm.h b/include/linux/qcom_scm.h
> index 06996ad4f2bc..ce5a476fd733 100644
> --- a/include/linux/qcom_scm.h
> +++ b/include/linux/qcom_scm.h
> @@ -13,6 +13,7 @@
>  #ifndef __QCOM_SCM_H
>  #define __QCOM_SCM_H
>  
> +#include <linux/err.h>
>  #include <linux/types.h>
>  #include <linux/cpumask.h>
>  

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH] qcom-scm: Include <linux/err.h> header
  2018-12-29 19:51         ` Andy Gross
@ 2019-01-14 19:50           ` Amit Kucheria
  2019-01-14 19:53             ` Chris Healy
  0 siblings, 1 reply; 16+ messages in thread
From: Amit Kucheria @ 2019-01-14 19:50 UTC (permalink / raw)
  To: Andy Gross
  Cc: Jonathan Marek, DRI mailing list, Bjorn Andersson, linux-arm-msm,
	Shawn Guo, Chris Healy

On Sun, Dec 30, 2018 at 1:21 AM Andy Gross <andy.gross@linaro.org> wrote:
>
> On Sat, Dec 29, 2018 at 10:19:32AM -0200, Fabio Estevam wrote:
> > Hi Bjorn,
> >
> > On Fri, Dec 28, 2018 at 10:27 PM Bjorn Andersson
> > <bjorn.andersson@linaro.org> wrote:
> >
> > > Sorry about that, I forgot that the header file is not covered by the
> > > MAINTAINERS file.
> > >
> > > Your second patch looks good, but I'm hoping we can merge the upcoming
> > > v3 of Amit's patch right after the merge window. It fixes this and a lot
> > > of other pieces where we would like to include linux-arm-msm@:
> > >
> > > https://lore.kernel.org/lkml/d153a86748f99526e7790bfc4ef8781a2016fd51.1545126964.git.amit.kucheria@linaro.org/
> >
> > Amit's patch adds the following entry:
> >
> > +F: include/linux/*/qcom*
> >
> > but it does not catch include/linux/qcom_scm.h
> >
> > It also needs
> >
> > +F: include/linux/qcom*
> >
> > in order to catch include/linux/qcom-geni-se.h  and include/linux/qcom_scm.h
> >
> > I can add that entry after Amit's patch gets applied.
>
> Or I can add it to Amit's.  I'll ping him to make sure that's ok.
>

I'd forgotten about this patch! Just sent out v3 which is still
missing "F: include/linux/qcom*".

Let me know if you want me to send out v4 with this added.

Regards,
Amit
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH] qcom-scm: Include <linux/err.h> header
  2019-01-14 19:50           ` Amit Kucheria
@ 2019-01-14 19:53             ` Chris Healy
  2019-01-14 20:02               ` Fabio Estevam
  0 siblings, 1 reply; 16+ messages in thread
From: Chris Healy @ 2019-01-14 19:53 UTC (permalink / raw)
  To: Amit Kucheria
  Cc: Jonathan Marek, Shawn Guo, DRI mailing list, Bjorn Andersson,
	linux-arm-msm, Andy Gross

Perhaps I am confused but it appears that this patch has already
landed upstream and got included in 5.0-rc2:

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/include/linux/qcom_scm.h?h=v5.0-rc2&id=2076607a20bd4dfba699185616cbbbce06d3fa59

On Mon, Jan 14, 2019 at 11:51 AM Amit Kucheria
<amit.kucheria@verdurent.com> wrote:
>
> On Sun, Dec 30, 2018 at 1:21 AM Andy Gross <andy.gross@linaro.org> wrote:
> >
> > On Sat, Dec 29, 2018 at 10:19:32AM -0200, Fabio Estevam wrote:
> > > Hi Bjorn,
> > >
> > > On Fri, Dec 28, 2018 at 10:27 PM Bjorn Andersson
> > > <bjorn.andersson@linaro.org> wrote:
> > >
> > > > Sorry about that, I forgot that the header file is not covered by the
> > > > MAINTAINERS file.
> > > >
> > > > Your second patch looks good, but I'm hoping we can merge the upcoming
> > > > v3 of Amit's patch right after the merge window. It fixes this and a lot
> > > > of other pieces where we would like to include linux-arm-msm@:
> > > >
> > > > https://lore.kernel.org/lkml/d153a86748f99526e7790bfc4ef8781a2016fd51.1545126964.git.amit.kucheria@linaro.org/
> > >
> > > Amit's patch adds the following entry:
> > >
> > > +F: include/linux/*/qcom*
> > >
> > > but it does not catch include/linux/qcom_scm.h
> > >
> > > It also needs
> > >
> > > +F: include/linux/qcom*
> > >
> > > in order to catch include/linux/qcom-geni-se.h  and include/linux/qcom_scm.h
> > >
> > > I can add that entry after Amit's patch gets applied.
> >
> > Or I can add it to Amit's.  I'll ping him to make sure that's ok.
> >
>
> I'd forgotten about this patch! Just sent out v3 which is still
> missing "F: include/linux/qcom*".
>
> Let me know if you want me to send out v4 with this added.
>
> Regards,
> Amit
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH] qcom-scm: Include <linux/err.h> header
  2019-01-14 19:53             ` Chris Healy
@ 2019-01-14 20:02               ` Fabio Estevam
  2019-01-14 20:09                 ` Chris Healy
  0 siblings, 1 reply; 16+ messages in thread
From: Fabio Estevam @ 2019-01-14 20:02 UTC (permalink / raw)
  To: Chris Healy
  Cc: Amit Kucheria, Jonathan Marek, linux-arm-msm, DRI mailing list,
	Bjorn Andersson, Andy Gross, Shawn Guo

Hi Chris,

On Mon, Jan 14, 2019 at 5:54 PM Chris Healy <cphealy@gmail.com> wrote:
>
> Perhaps I am confused but it appears that this patch has already
> landed upstream and got included in 5.0-rc2:

The patch that Amit is referring is the following entry in MAINTAINERS file:

+F: include/linux/qcom*

so that the proper lists can be put on Cc on future changes of this file.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH] qcom-scm: Include <linux/err.h> header
  2019-01-14 20:02               ` Fabio Estevam
@ 2019-01-14 20:09                 ` Chris Healy
  0 siblings, 0 replies; 16+ messages in thread
From: Chris Healy @ 2019-01-14 20:09 UTC (permalink / raw)
  To: Fabio Estevam
  Cc: Amit Kucheria, Jonathan Marek, linux-arm-msm, DRI mailing list,
	Bjorn Andersson, Andy Gross, Shawn Guo

There we go, I was definitely confused...  Tnx

On Mon, Jan 14, 2019 at 12:02 PM Fabio Estevam <festevam@gmail.com> wrote:
>
> Hi Chris,
>
> On Mon, Jan 14, 2019 at 5:54 PM Chris Healy <cphealy@gmail.com> wrote:
> >
> > Perhaps I am confused but it appears that this patch has already
> > landed upstream and got included in 5.0-rc2:
>
> The patch that Amit is referring is the following entry in MAINTAINERS file:
>
> +F: include/linux/qcom*
>
> so that the proper lists can be put on Cc on future changes of this file.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 16+ messages in thread

end of thread, other threads:[~2019-01-14 20:09 UTC | newest]

Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-12-26 12:06 [PATCH] qcom-scm: Include <linux/err.h> header Fabio Estevam
2018-12-28 19:31 ` Bjorn Andersson
2018-12-28 15:39   ` Jonathan marek
2018-12-29  0:12     ` Bjorn Andersson
2018-12-28 19:52   ` Fabio Estevam
2018-12-28 19:56     ` Fabio Estevam
2018-12-28 20:08       ` Fabio Estevam
2018-12-28 20:00     ` Fabio Estevam
2018-12-29  0:27     ` Bjorn Andersson
2018-12-29 12:19       ` Fabio Estevam
2018-12-29 19:51         ` Andy Gross
2019-01-14 19:50           ` Amit Kucheria
2019-01-14 19:53             ` Chris Healy
2019-01-14 20:02               ` Fabio Estevam
2019-01-14 20:09                 ` Chris Healy
2019-01-04 21:15 ` Guenter Roeck

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.