* [PATCH] [TRIVIAL] ARM: msm: fix compilation error in gpiomux
@ 2013-08-07 6:34 Krzysztof Kozlowski
2013-08-07 17:40 ` David Brown
0 siblings, 1 reply; 6+ messages in thread
From: Krzysztof Kozlowski @ 2013-08-07 6:34 UTC (permalink / raw)
To: linux-arm-kernel
Fix compilation error in gpiomux (CONFIG_MSM_GPIOMUX=y):
arch/arm/mach-msm/gpiomux.c:24:13: error: static declaration of
?__msm_gpiomux_write? follows non-static declaration
arch/arm/mach-msm/gpiomux.h:85:6: note: previous declaration of
?__msm_gpiomux_write? was here
Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
---
arch/arm/mach-msm/gpiomux.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/mach-msm/gpiomux.c b/arch/arm/mach-msm/gpiomux.c
index 2b8e2d2..4110b82 100644
--- a/arch/arm/mach-msm/gpiomux.c
+++ b/arch/arm/mach-msm/gpiomux.c
@@ -21,7 +21,7 @@
static DEFINE_SPINLOCK(gpiomux_lock);
-static void __msm_gpiomux_write(unsigned gpio, gpiomux_config_t val)
+void __msm_gpiomux_write(unsigned gpio, gpiomux_config_t val)
{
unsigned tlmm_config = (val & ~GPIOMUX_CTL_MASK) |
((gpio & 0x3ff) << 4);
--
1.7.9.5
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH] [TRIVIAL] ARM: msm: fix compilation error in gpiomux
2013-08-07 6:34 [PATCH] [TRIVIAL] ARM: msm: fix compilation error in gpiomux Krzysztof Kozlowski
@ 2013-08-07 17:40 ` David Brown
2013-08-07 21:08 ` Stephen Boyd
0 siblings, 1 reply; 6+ messages in thread
From: David Brown @ 2013-08-07 17:40 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Aug 07, 2013 at 08:34:39AM +0200, Krzysztof Kozlowski wrote:
>Fix compilation error in gpiomux (CONFIG_MSM_GPIOMUX=y):
>arch/arm/mach-msm/gpiomux.c:24:13: error: static declaration of
> ?__msm_gpiomux_write? follows non-static declaration
>arch/arm/mach-msm/gpiomux.h:85:6: note: previous declaration of
> ?__msm_gpiomux_write? was here
>
>Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
Thanks, I'll pull this in.
<https://patchwork.kernel.org/patch/2839921/>
Daivd
--
sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
hosted by The Linux Foundation
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] [TRIVIAL] ARM: msm: fix compilation error in gpiomux
2013-08-07 17:40 ` David Brown
@ 2013-08-07 21:08 ` Stephen Boyd
2013-08-08 6:32 ` Krzysztof Kozlowski
0 siblings, 1 reply; 6+ messages in thread
From: Stephen Boyd @ 2013-08-07 21:08 UTC (permalink / raw)
To: linux-arm-kernel
On 08/07, David Brown wrote:
> On Wed, Aug 07, 2013 at 08:34:39AM +0200, Krzysztof Kozlowski wrote:
> >Fix compilation error in gpiomux (CONFIG_MSM_GPIOMUX=y):
> >arch/arm/mach-msm/gpiomux.c:24:13: error: static declaration of
> > ???__msm_gpiomux_write??? follows non-static declaration
> >arch/arm/mach-msm/gpiomux.h:85:6: note: previous declaration of
> > ???__msm_gpiomux_write??? was here
> >
> >Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
>
> Thanks, I'll pull this in.
>
> <https://patchwork.kernel.org/patch/2839921/>
>
I believe this is fixed by Rohit's patch that has already been
taken in a pull request?
https://lkml.org/lkml/2013/7/26/608
--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
hosted by The Linux Foundation
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] [TRIVIAL] ARM: msm: fix compilation error in gpiomux
2013-08-07 21:08 ` Stephen Boyd
@ 2013-08-08 6:32 ` Krzysztof Kozlowski
2013-08-08 22:17 ` Rohit Vaswani
0 siblings, 1 reply; 6+ messages in thread
From: Krzysztof Kozlowski @ 2013-08-08 6:32 UTC (permalink / raw)
To: linux-arm-kernel
On ?ro, 2013-08-07 at 14:08 -0700, Stephen Boyd wrote:
> On 08/07, David Brown wrote:
> > On Wed, Aug 07, 2013 at 08:34:39AM +0200, Krzysztof Kozlowski wrote:
> > >Fix compilation error in gpiomux (CONFIG_MSM_GPIOMUX=y):
> > >arch/arm/mach-msm/gpiomux.c:24:13: error: static declaration of
> > > ???__msm_gpiomux_write??? follows non-static declaration
> > >arch/arm/mach-msm/gpiomux.h:85:6: note: previous declaration of
> > > ???__msm_gpiomux_write??? was here
> > >
> > >Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
> >
> > Thanks, I'll pull this in.
> >
> > <https://patchwork.kernel.org/patch/2839921/>
> >
>
> I believe this is fixed by Rohit's patch that has already been
> taken in a pull request?
>
> https://lkml.org/lkml/2013/7/26/608
Yes, Rohit's patch fixes the problem so this is already in linux-msm
tree.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] [TRIVIAL] ARM: msm: fix compilation error in gpiomux
2013-08-08 6:32 ` Krzysztof Kozlowski
@ 2013-08-08 22:17 ` Rohit Vaswani
2013-08-09 18:00 ` David Brown
0 siblings, 1 reply; 6+ messages in thread
From: Rohit Vaswani @ 2013-08-08 22:17 UTC (permalink / raw)
To: linux-arm-kernel
On 8/7/2013 11:32 PM, Krzysztof Kozlowski wrote:
> On ?ro, 2013-08-07 at 14:08 -0700, Stephen Boyd wrote:
>> On 08/07, David Brown wrote:
>>> On Wed, Aug 07, 2013 at 08:34:39AM +0200, Krzysztof Kozlowski wrote:
>>>> Fix compilation error in gpiomux (CONFIG_MSM_GPIOMUX=y):
>>>> arch/arm/mach-msm/gpiomux.c:24:13: error: static declaration of
>>>> ???__msm_gpiomux_write??? follows non-static declaration
>>>> arch/arm/mach-msm/gpiomux.h:85:6: note: previous declaration of
>>>> ???__msm_gpiomux_write??? was here
>>>>
>>>> Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
>>> Thanks, I'll pull this in.
>>>
>>> <https://patchwork.kernel.org/patch/2839921/>
>>>
>> I believe this is fixed by Rohit's patch that has already been
>> taken in a pull request?
>>
>> https://lkml.org/lkml/2013/7/26/608
> Yes, Rohit's patch fixes the problem so this is already in linux-msm
> tree.
>
David, please can you confirm which pull request has this patch ?
> Best regards,
> Krzysztof
>
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Thanks,
Rohit Vaswani
--
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, hosted by The Linux Foundation
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] [TRIVIAL] ARM: msm: fix compilation error in gpiomux
2013-08-08 22:17 ` Rohit Vaswani
@ 2013-08-09 18:00 ` David Brown
0 siblings, 0 replies; 6+ messages in thread
From: David Brown @ 2013-08-09 18:00 UTC (permalink / raw)
To: linux-arm-kernel
On Thu, Aug 08, 2013 at 03:17:25PM -0700, Rohit Vaswani wrote:
>On 8/7/2013 11:32 PM, Krzysztof Kozlowski wrote:
>> On ?ro, 2013-08-07 at 14:08 -0700, Stephen Boyd wrote:
>>> On 08/07, David Brown wrote:
>>>> On Wed, Aug 07, 2013 at 08:34:39AM +0200, Krzysztof Kozlowski wrote:
>>>>> Fix compilation error in gpiomux (CONFIG_MSM_GPIOMUX=y):
>>>>> arch/arm/mach-msm/gpiomux.c:24:13: error: static declaration of
>>>>> ???__msm_gpiomux_write??? follows non-static declaration
>>>>> arch/arm/mach-msm/gpiomux.h:85:6: note: previous declaration of
>>>>> ???__msm_gpiomux_write??? was here
>>>>>
>>>>> Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
>>>> Thanks, I'll pull this in.
>>>>
>>>> <https://patchwork.kernel.org/patch/2839921/>
>>>>
>>> I believe this is fixed by Rohit's patch that has already been
>>> taken in a pull request?
>>>
>>> https://lkml.org/lkml/2013/7/26/608
>> Yes, Rohit's patch fixes the problem so this is already in linux-msm
>> tree.
>
>David, please can you confirm which pull request has this patch ?
It is in Linus' current tree. Olof pulled it into the ARM tree on
Sunday, and Linus pulled those fixes yesterday morning.
David
--
sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
hosted by The Linux Foundation
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-08-09 18:00 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-08-07 6:34 [PATCH] [TRIVIAL] ARM: msm: fix compilation error in gpiomux Krzysztof Kozlowski
2013-08-07 17:40 ` David Brown
2013-08-07 21:08 ` Stephen Boyd
2013-08-08 6:32 ` Krzysztof Kozlowski
2013-08-08 22:17 ` Rohit Vaswani
2013-08-09 18:00 ` David Brown
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).