* [PATCH] mxs/spi: Fix misuse of init_completion
@ 2012-08-22 20:57 Marek Vasut
[not found] ` <1345669065-17750-1-git-send-email-marex-ynQEQJNshbs@public.gmane.org>
0 siblings, 1 reply; 4+ messages in thread
From: Marek Vasut @ 2012-08-22 20:57 UTC (permalink / raw)
To: spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f
Cc: Marek Vasut, Fabio Estevam, Shawn Guo, Mark Brown, Chris Ball,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
Signed-off-by: Marek Vasut <marex-ynQEQJNshbs@public.gmane.org>
Cc: Chris Ball <cjb-2X9k7bc8m7Mdnm+yROfE0A@public.gmane.org>
Cc: Shawn Guo <shawn.guo-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
Cc: Mark Brown <broonie-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E@public.gmane.org>
Cc: Fabio Estevam <fabio.estevam-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
---
drivers/spi/spi-mxs.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/spi/spi-mxs.c b/drivers/spi/spi-mxs.c
index 331f600..c965cc6 100644
--- a/drivers/spi/spi-mxs.c
+++ b/drivers/spi/spi-mxs.c
@@ -230,7 +230,7 @@ static int mxs_spi_txrx_dma(struct mxs_spi *spi, int cs,
return -EINVAL;
}
- init_completion(&spi->c);
+ INIT_COMPLETION(spi->c);
if (*first)
pio |= BM_SSP_CTRL0_LOCK_CS;
@@ -560,6 +560,8 @@ static int __devinit mxs_spi_probe(struct platform_device *pdev)
ssp->devid = devid;
ssp->dma_channel = dma_channel;
+ init_completion(&spi->c);
+
ret = devm_request_irq(&pdev->dev, irq_err, mxs_ssp_irq_handler, 0,
DRIVER_NAME, ssp);
if (ret)
--
1.7.10.4
------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and
threat landscape has changed and how IT managers can respond. Discussions
will include endpoint security, mobile security and the latest in malware
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] mxs/spi: Fix misuse of init_completion
[not found] ` <1345669065-17750-1-git-send-email-marex-ynQEQJNshbs@public.gmane.org>
@ 2012-08-22 21:07 ` Chris Ball
[not found] ` <87ehmyzlds.fsf-DGHOrqG7t0YzNDMTQreKSUB+6BGkLq7r@public.gmane.org>
2012-09-05 23:46 ` Mark Brown
1 sibling, 1 reply; 4+ messages in thread
From: Chris Ball @ 2012-08-22 21:07 UTC (permalink / raw)
To: Marek Vasut
Cc: Fabio Estevam,
spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f, Shawn Guo,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Mark Brown
Hi,
On Wed, Aug 22 2012, Marek Vasut wrote:
> Signed-off-by: Marek Vasut <marex-ynQEQJNshbs@public.gmane.org>
> Cc: Chris Ball <cjb-2X9k7bc8m7Mdnm+yROfE0A@public.gmane.org>
> Cc: Shawn Guo <shawn.guo-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
> Cc: Mark Brown <broonie-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E@public.gmane.org>
> Cc: Fabio Estevam <fabio.estevam-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
> ---
> drivers/spi/spi-mxs.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/spi/spi-mxs.c b/drivers/spi/spi-mxs.c
> index 331f600..c965cc6 100644
> --- a/drivers/spi/spi-mxs.c
> +++ b/drivers/spi/spi-mxs.c
> @@ -230,7 +230,7 @@ static int mxs_spi_txrx_dma(struct mxs_spi *spi, int cs,
> return -EINVAL;
> }
>
> - init_completion(&spi->c);
> + INIT_COMPLETION(spi->c);
>
> if (*first)
> pio |= BM_SSP_CTRL0_LOCK_CS;
> @@ -560,6 +560,8 @@ static int __devinit mxs_spi_probe(struct platform_device *pdev)
> ssp->devid = devid;
> ssp->dma_channel = dma_channel;
>
> + init_completion(&spi->c);
> +
> ret = devm_request_irq(&pdev->dev, irq_err, mxs_ssp_irq_handler, 0,
> DRIVER_NAME, ssp);
> if (ret)
Please be more specific with commit messages -- is there an observed
symptom that this fixes? Does the patch need to go to stable@, too?
Thanks,
- Chris.
--
Chris Ball <cjb-2X9k7bc8m7Mdnm+yROfE0A@public.gmane.org> <http://printf.net/>
One Laptop Per Child
------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and
threat landscape has changed and how IT managers can respond. Discussions
will include endpoint security, mobile security and the latest in malware
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] mxs/spi: Fix misuse of init_completion
[not found] ` <87ehmyzlds.fsf-DGHOrqG7t0YzNDMTQreKSUB+6BGkLq7r@public.gmane.org>
@ 2012-08-22 21:30 ` Marek Vasut
0 siblings, 0 replies; 4+ messages in thread
From: Marek Vasut @ 2012-08-22 21:30 UTC (permalink / raw)
To: Chris Ball
Cc: Fabio Estevam,
spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f, Shawn Guo,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Mark Brown
Dear Chris Ball,
> Hi,
>
> On Wed, Aug 22 2012, Marek Vasut wrote:
> > Signed-off-by: Marek Vasut <marex-ynQEQJNshbs@public.gmane.org>
> > Cc: Chris Ball <cjb-2X9k7bc8m7Mdnm+yROfE0A@public.gmane.org>
> > Cc: Shawn Guo <shawn.guo-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
> > Cc: Mark Brown <broonie-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E@public.gmane.org>
> > Cc: Fabio Estevam <fabio.estevam-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
> > ---
> >
> > drivers/spi/spi-mxs.c | 4 +++-
> > 1 file changed, 3 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/spi/spi-mxs.c b/drivers/spi/spi-mxs.c
> > index 331f600..c965cc6 100644
> > --- a/drivers/spi/spi-mxs.c
> > +++ b/drivers/spi/spi-mxs.c
> > @@ -230,7 +230,7 @@ static int mxs_spi_txrx_dma(struct mxs_spi *spi, int
> > cs,
> >
> > return -EINVAL;
> >
> > }
> >
> > - init_completion(&spi->c);
> > + INIT_COMPLETION(spi->c);
> >
> > if (*first)
> >
> > pio |= BM_SSP_CTRL0_LOCK_CS;
> >
> > @@ -560,6 +560,8 @@ static int __devinit mxs_spi_probe(struct
> > platform_device *pdev)
> >
> > ssp->devid = devid;
> > ssp->dma_channel = dma_channel;
> >
> > + init_completion(&spi->c);
> > +
> >
> > ret = devm_request_irq(&pdev->dev, irq_err, mxs_ssp_irq_handler, 0,
> >
> > DRIVER_NAME, ssp);
> >
> > if (ret)
>
> Please be more specific with commit messages
I kind of expected this change was pretty obvious.
> is there an observed symptom that this fixes?
None. I noticed this misuse, so I fixed it.
> Does the patch need to go to stable@, too?
No, this driver is in -next currently.
> Thanks,
>
> - Chris.
Best regards,
Marek Vasut
------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and
threat landscape has changed and how IT managers can respond. Discussions
will include endpoint security, mobile security and the latest in malware
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] mxs/spi: Fix misuse of init_completion
[not found] ` <1345669065-17750-1-git-send-email-marex-ynQEQJNshbs@public.gmane.org>
2012-08-22 21:07 ` Chris Ball
@ 2012-09-05 23:46 ` Mark Brown
1 sibling, 0 replies; 4+ messages in thread
From: Mark Brown @ 2012-09-05 23:46 UTC (permalink / raw)
To: Marek Vasut
Cc: Fabio Estevam,
spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f, Chris Ball,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Shawn Guo
On Wed, Aug 22, 2012 at 10:57:45PM +0200, Marek Vasut wrote:
> Signed-off-by: Marek Vasut <marex-ynQEQJNshbs@public.gmane.org>
Applied, thanks.
------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and
threat landscape has changed and how IT managers can respond. Discussions
will include endpoint security, mobile security and the latest in malware
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2012-09-05 23:46 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-08-22 20:57 [PATCH] mxs/spi: Fix misuse of init_completion Marek Vasut
[not found] ` <1345669065-17750-1-git-send-email-marex-ynQEQJNshbs@public.gmane.org>
2012-08-22 21:07 ` Chris Ball
[not found] ` <87ehmyzlds.fsf-DGHOrqG7t0YzNDMTQreKSUB+6BGkLq7r@public.gmane.org>
2012-08-22 21:30 ` Marek Vasut
2012-09-05 23:46 ` Mark 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).