* [PATCH v2] ARM: pxa: ssp: Fix build error by removing originally incorrect DT binding
@ 2015-09-07 7:23 Jarkko Nikula
2015-09-12 13:43 ` Robert Jarzmik
0 siblings, 1 reply; 6+ messages in thread
From: Jarkko Nikula @ 2015-09-07 7:23 UTC (permalink / raw)
To: linux-arm-kernel
Commit 03fbf488cece ("spi: pxa2xx: Differentiate Intel LPSS types") caused
build error here because it removed the type LPSS_SSP and I didn't notice
the type was used here too.
I believe commit a6e56c28a178 ("ARM: pxa: ssp: add DT bindings") added it
accidentally by copying all enum pxa_ssp_type types from
include/linux/pxa2xx_ssp.h even LPSS_SSP was for Intel LPSS SPI devices.
Fix the build error by removing this incorrect binding.
Fixes: 03fbf488cece ("spi: pxa2xx: Differentiate Intel LPSS types")
Signed-off-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
Reported-by: Axel Lin <axel.lin@ingics.com>
Cc: <stable@vger.kernel.org> # 4.2
---
v2: Added Fixes tag suggested by Robert Jarzmik
---
arch/arm/plat-pxa/ssp.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm/plat-pxa/ssp.c b/arch/arm/plat-pxa/ssp.c
index ad9529cc4203..daa1a65f2eb7 100644
--- a/arch/arm/plat-pxa/ssp.c
+++ b/arch/arm/plat-pxa/ssp.c
@@ -107,7 +107,6 @@ static const struct of_device_id pxa_ssp_of_ids[] = {
{ .compatible = "mvrl,pxa168-ssp", .data = (void *) PXA168_SSP },
{ .compatible = "mrvl,pxa910-ssp", .data = (void *) PXA910_SSP },
{ .compatible = "mrvl,ce4100-ssp", .data = (void *) CE4100_SSP },
- { .compatible = "mrvl,lpss-ssp", .data = (void *) LPSS_SSP },
{ },
};
MODULE_DEVICE_TABLE(of, pxa_ssp_of_ids);
--
2.5.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH v2] ARM: pxa: ssp: Fix build error by removing originally incorrect DT binding
2015-09-07 7:23 [PATCH v2] ARM: pxa: ssp: Fix build error by removing originally incorrect DT binding Jarkko Nikula
@ 2015-09-12 13:43 ` Robert Jarzmik
2015-09-21 7:48 ` Jarkko Nikula
0 siblings, 1 reply; 6+ messages in thread
From: Robert Jarzmik @ 2015-09-12 13:43 UTC (permalink / raw)
To: linux-arm-kernel
Jarkko Nikula <jarkko.nikula@linux.intel.com> writes:
> Commit 03fbf488cece ("spi: pxa2xx: Differentiate Intel LPSS types") caused
> build error here because it removed the type LPSS_SSP and I didn't notice
> the type was used here too.
>
> I believe commit a6e56c28a178 ("ARM: pxa: ssp: add DT bindings") added it
> accidentally by copying all enum pxa_ssp_type types from
> include/linux/pxa2xx_ssp.h even LPSS_SSP was for Intel LPSS SPI devices.
>
> Fix the build error by removing this incorrect binding.
>
> Fixes: 03fbf488cece ("spi: pxa2xx: Differentiate Intel LPSS types")
> Signed-off-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
> Reported-by: Axel Lin <axel.lin@ingics.com>
> Cc: <stable@vger.kernel.org> # 4.2
> ---
> v2: Added Fixes tag suggested by Robert Jarzmik
Queued into pxa/fixes, thanks.
Cheers.
--
Robert
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2] ARM: pxa: ssp: Fix build error by removing originally incorrect DT binding
2015-09-12 13:43 ` Robert Jarzmik
@ 2015-09-21 7:48 ` Jarkko Nikula
2015-09-21 9:03 ` Robert Jarzmik
0 siblings, 1 reply; 6+ messages in thread
From: Jarkko Nikula @ 2015-09-21 7:48 UTC (permalink / raw)
To: linux-arm-kernel
Hi
On 09/12/2015 04:43 PM, Robert Jarzmik wrote:
> Jarkko Nikula <jarkko.nikula@linux.intel.com> writes:
>
>> Commit 03fbf488cece ("spi: pxa2xx: Differentiate Intel LPSS types") caused
>> build error here because it removed the type LPSS_SSP and I didn't notice
>> the type was used here too.
>>
>> I believe commit a6e56c28a178 ("ARM: pxa: ssp: add DT bindings") added it
>> accidentally by copying all enum pxa_ssp_type types from
>> include/linux/pxa2xx_ssp.h even LPSS_SSP was for Intel LPSS SPI devices.
>>
>> Fix the build error by removing this incorrect binding.
>>
>> Fixes: 03fbf488cece ("spi: pxa2xx: Differentiate Intel LPSS types")
>> Signed-off-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
>> Reported-by: Axel Lin <axel.lin@ingics.com>
>> Cc: <stable@vger.kernel.org> # 4.2
>> ---
>> v2: Added Fixes tag suggested by Robert Jarzmik
>
> Queued into pxa/fixes, thanks.
>
Gentle reminder, is this going to 4.3-rc3?
--
Jarkko
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2] ARM: pxa: ssp: Fix build error by removing originally incorrect DT binding
2015-09-21 7:48 ` Jarkko Nikula
@ 2015-09-21 9:03 ` Robert Jarzmik
2015-09-21 15:15 ` Vinod Koul
0 siblings, 1 reply; 6+ messages in thread
From: Robert Jarzmik @ 2015-09-21 9:03 UTC (permalink / raw)
To: linux-arm-kernel
Jarkko Nikula <jarkko.nikula@linux.intel.com> writes:
> Hi
>
> On 09/12/2015 04:43 PM, Robert Jarzmik wrote:
>> Jarkko Nikula <jarkko.nikula@linux.intel.com> writes:
>>
>>> Commit 03fbf488cece ("spi: pxa2xx: Differentiate Intel LPSS types") caused
>>> build error here because it removed the type LPSS_SSP and I didn't notice
>>> the type was used here too.
>> Queued into pxa/fixes, thanks.
>>
> Gentle reminder, is this going to 4.3-rc3?
It is going into v4.3.
Now knowing if it is v4.3-rc3 is another story, as currently the pxa platform is
broken because of a dmaengine regression. And if the necessary fix is not
applied through dmaengine tree (by lack of time from Vinod), I'll want to carry
it through my tree, which is the reason why I have not made the pull request
yet.
Cheers.
--
Robert
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2] ARM: pxa: ssp: Fix build error by removing originally incorrect DT binding
2015-09-21 9:03 ` Robert Jarzmik
@ 2015-09-21 15:15 ` Vinod Koul
2015-09-21 18:06 ` Robert Jarzmik
0 siblings, 1 reply; 6+ messages in thread
From: Vinod Koul @ 2015-09-21 15:15 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Sep 21, 2015 at 11:03:18AM +0200, Robert Jarzmik wrote:
> Jarkko Nikula <jarkko.nikula@linux.intel.com> writes:
>
> > Hi
> >
> > On 09/12/2015 04:43 PM, Robert Jarzmik wrote:
> >> Jarkko Nikula <jarkko.nikula@linux.intel.com> writes:
> >>
> >>> Commit 03fbf488cece ("spi: pxa2xx: Differentiate Intel LPSS types") caused
> >>> build error here because it removed the type LPSS_SSP and I didn't notice
> >>> the type was used here too.
> >> Queued into pxa/fixes, thanks.
> >>
> > Gentle reminder, is this going to 4.3-rc3?
> It is going into v4.3.
>
> Now knowing if it is v4.3-rc3 is another story, as currently the pxa platform is
> broken because of a dmaengine regression. And if the necessary fix is not
> applied through dmaengine tree (by lack of time from Vinod), I'll want to carry
> it through my tree, which is the reason why I have not made the pull request
> yet.
Which ones are required for fixing, I will priortize those...
--
~Vinod
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2] ARM: pxa: ssp: Fix build error by removing originally incorrect DT binding
2015-09-21 15:15 ` Vinod Koul
@ 2015-09-21 18:06 ` Robert Jarzmik
0 siblings, 0 replies; 6+ messages in thread
From: Robert Jarzmik @ 2015-09-21 18:06 UTC (permalink / raw)
To: linux-arm-kernel
Vinod Koul <vinod.koul@intel.com> writes:
>> Now knowing if it is v4.3-rc3 is another story, as currently the pxa platform is
>> broken because of a dmaengine regression. And if the necessary fix is not
>> applied through dmaengine tree (by lack of time from Vinod), I'll want to carry
>> it through my tree, which is the reason why I have not made the pull request
>> yet.
>
> Which ones are required for fixing, I will priortize those...
Hi Vinod,
My favorite (which fixes it fully) is :
- https://lkml.org/lkml/2015/9/6/112 (ie. only 1/3)
As a fallback plan, if the above patch is too risky for a -rc serie, my fallback
plan is :
- https://lkml.org/lkml/2015/9/21/128
PXA can sustain only the fallback for v4.3, but the favorite has to be in v4.4,
with its 2 other brothers, in a form or another. Fallback plan is compatible
with the favorite, ie. fallback should not be reverted after favorite is applied.
Cheers.
--
Robert
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-09-21 18:06 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-09-07 7:23 [PATCH v2] ARM: pxa: ssp: Fix build error by removing originally incorrect DT binding Jarkko Nikula
2015-09-12 13:43 ` Robert Jarzmik
2015-09-21 7:48 ` Jarkko Nikula
2015-09-21 9:03 ` Robert Jarzmik
2015-09-21 15:15 ` Vinod Koul
2015-09-21 18:06 ` Robert Jarzmik
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).