Re: [PATCH] qcom-scm: Include header

2019-01-14 Thread Chris Healy
There we go, I was definitely confused... Tnx On Mon, Jan 14, 2019 at 12:02 PM Fabio Estevam wrote: > > Hi Chris, > > On Mon, Jan 14, 2019 at 5:54 PM Chris Healy wrote: > > > > Perhaps I am confused but it appears that this patch has already > > landed upstream and got included in 5.0-rc2: > >

Re: [PATCH] qcom-scm: Include header

2019-01-14 Thread Amit Kucheria
On Sun, Dec 30, 2018 at 1:21 AM Andy Gross 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 > > wrote: > > > > > Sorry about that, I forgot that the header file is not covered by the > > > MAINTAINERS

Re: [PATCH] qcom-scm: Include header

2019-01-14 Thread Chris Healy
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

Re: [PATCH] qcom-scm: Include header

2019-01-14 Thread Fabio Estevam
Hi Chris, On Mon, Jan 14, 2019 at 5:54 PM Chris Healy 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 prop

Re: [PATCH] qcom-scm: Include header

2019-01-07 Thread Guenter Roeck
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/ms

Re: [PATCH] qcom-scm: Include header

2019-01-01 Thread Bjorn Andersson
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/a5

Re: [PATCH] qcom-scm: Include header

2019-01-01 Thread Jonathan marek
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 w

Re: [PATCH] qcom-scm: Include header

2019-01-01 Thread Bjorn Andersson
On Fri 28 Dec 11:52 PST 2018, Fabio Estevam wrote: > Hi Bjorn, > > On Fri, Dec 28, 2018 at 5:31 PM Bjorn Andersson > 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_maintaine

Re: [PATCH] qcom-scm: Include header

2019-01-01 Thread Bjorn Andersson
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,

Re: [PATCH] qcom-scm: Include header

2019-01-01 Thread Andy Gross
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 > 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 m

Re: [PATCH] qcom-scm: Include header

2018-12-29 Thread Fabio Estevam
Hi Bjorn, On Fri, Dec 28, 2018 at 10:27 PM Bjorn Andersson 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 thi

Re: [PATCH] qcom-scm: Include header

2018-12-28 Thread Fabio Estevam
On Fri, Dec 28, 2018 at 5:56 PM Fabio Estevam wrote: > > Hi Bjorn, > > On Fri, Dec 28, 2018 at 5:52 PM Fabio Estevam 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? Ju

Re: [PATCH] qcom-scm: Include header

2018-12-28 Thread Fabio Estevam
On Fri, Dec 28, 2018 at 5:52 PM Fabio Estevam 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-d

Re: [PATCH] qcom-scm: Include header

2018-12-28 Thread Fabio Estevam
Hi Bjorn, On Fri, Dec 28, 2018 at 5:52 PM Fabio Estevam 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

Re: [PATCH] qcom-scm: Include header

2018-12-28 Thread Fabio Estevam
Hi Bjorn, On Fri, Dec 28, 2018 at 5: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 im

[PATCH] qcom-scm: Include header

2018-12-26 Thread Fabio Estevam
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_