Re: [PATCH] mmc: mxcmmc: replace spin_lock_irqsave with spin_lock in ISR
On 11 September 2018 at 16:47, jun qian wrote: > As you are already in ISR, it is unnecessary to call spin_lock_irqsave. > > Signed-off-by: jun qian Applied for next, thanks! Kind regards Uffe > --- > drivers/mmc/host/mxcmmc.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/drivers/mmc/host/mxcmmc.c b/drivers/mmc/host/mxcmmc.c > index de4e6e5bf304..4d17032d15ee 100644 > --- a/drivers/mmc/host/mxcmmc.c > +++ b/drivers/mmc/host/mxcmmc.c > @@ -728,7 +728,6 @@ static void mxcmci_cmd_done(struct mxcmci_host *host, > unsigned int stat) > static irqreturn_t mxcmci_irq(int irq, void *devid) > { > struct mxcmci_host *host = devid; > - unsigned long flags; > bool sdio_irq; > u32 stat; > > @@ -740,9 +739,9 @@ static irqreturn_t mxcmci_irq(int irq, void *devid) > > dev_dbg(mmc_dev(host->mmc), "%s: 0x%08x\n", __func__, stat); > > - spin_lock_irqsave(>lock, flags); > + spin_lock(>lock); > sdio_irq = (stat & STATUS_SDIO_INT_ACTIVE) && host->use_sdio; > - spin_unlock_irqrestore(>lock, flags); > + spin_unlock(>lock); > > if (mxcmci_use_dma(host) && (stat & (STATUS_WRITE_OP_DONE))) > mxcmci_writel(host, STATUS_WRITE_OP_DONE, MMC_REG_STATUS); > -- > 2.17.1 > > >
Re: [PATCH] mmc: mxcmmc: replace spin_lock_irqsave with spin_lock in ISR
On 11 September 2018 at 16:47, jun qian wrote: > As you are already in ISR, it is unnecessary to call spin_lock_irqsave. > > Signed-off-by: jun qian Applied for next, thanks! Kind regards Uffe > --- > drivers/mmc/host/mxcmmc.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/drivers/mmc/host/mxcmmc.c b/drivers/mmc/host/mxcmmc.c > index de4e6e5bf304..4d17032d15ee 100644 > --- a/drivers/mmc/host/mxcmmc.c > +++ b/drivers/mmc/host/mxcmmc.c > @@ -728,7 +728,6 @@ static void mxcmci_cmd_done(struct mxcmci_host *host, > unsigned int stat) > static irqreturn_t mxcmci_irq(int irq, void *devid) > { > struct mxcmci_host *host = devid; > - unsigned long flags; > bool sdio_irq; > u32 stat; > > @@ -740,9 +739,9 @@ static irqreturn_t mxcmci_irq(int irq, void *devid) > > dev_dbg(mmc_dev(host->mmc), "%s: 0x%08x\n", __func__, stat); > > - spin_lock_irqsave(>lock, flags); > + spin_lock(>lock); > sdio_irq = (stat & STATUS_SDIO_INT_ACTIVE) && host->use_sdio; > - spin_unlock_irqrestore(>lock, flags); > + spin_unlock(>lock); > > if (mxcmci_use_dma(host) && (stat & (STATUS_WRITE_OP_DONE))) > mxcmci_writel(host, STATUS_WRITE_OP_DONE, MMC_REG_STATUS); > -- > 2.17.1 > > >
Re: [PATCH] mmc: mxcmmc: replace spin_lock_irqsave with spin_lock in ISR
On 09/11/2018 05:47 PM, jun qian wrote: > As you are already in ISR, it is unnecessary to call spin_lock_irqsave. > > Signed-off-by: jun qian Reviewed-by: Vladimir Zapolskiy --- Best wishes, Vladimir
Re: [PATCH] mmc: mxcmmc: replace spin_lock_irqsave with spin_lock in ISR
On 09/11/2018 05:47 PM, jun qian wrote: > As you are already in ISR, it is unnecessary to call spin_lock_irqsave. > > Signed-off-by: jun qian Reviewed-by: Vladimir Zapolskiy --- Best wishes, Vladimir
[PATCH] mmc: mxcmmc: replace spin_lock_irqsave with spin_lock in ISR
As you are already in ISR, it is unnecessary to call spin_lock_irqsave. Signed-off-by: jun qian --- drivers/mmc/host/mxcmmc.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/mmc/host/mxcmmc.c b/drivers/mmc/host/mxcmmc.c index de4e6e5bf304..4d17032d15ee 100644 --- a/drivers/mmc/host/mxcmmc.c +++ b/drivers/mmc/host/mxcmmc.c @@ -728,7 +728,6 @@ static void mxcmci_cmd_done(struct mxcmci_host *host, unsigned int stat) static irqreturn_t mxcmci_irq(int irq, void *devid) { struct mxcmci_host *host = devid; - unsigned long flags; bool sdio_irq; u32 stat; @@ -740,9 +739,9 @@ static irqreturn_t mxcmci_irq(int irq, void *devid) dev_dbg(mmc_dev(host->mmc), "%s: 0x%08x\n", __func__, stat); - spin_lock_irqsave(>lock, flags); + spin_lock(>lock); sdio_irq = (stat & STATUS_SDIO_INT_ACTIVE) && host->use_sdio; - spin_unlock_irqrestore(>lock, flags); + spin_unlock(>lock); if (mxcmci_use_dma(host) && (stat & (STATUS_WRITE_OP_DONE))) mxcmci_writel(host, STATUS_WRITE_OP_DONE, MMC_REG_STATUS); -- 2.17.1
[PATCH] mmc: mxcmmc: replace spin_lock_irqsave with spin_lock in ISR
As you are already in ISR, it is unnecessary to call spin_lock_irqsave. Signed-off-by: jun qian --- drivers/mmc/host/mxcmmc.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/mmc/host/mxcmmc.c b/drivers/mmc/host/mxcmmc.c index de4e6e5bf304..4d17032d15ee 100644 --- a/drivers/mmc/host/mxcmmc.c +++ b/drivers/mmc/host/mxcmmc.c @@ -728,7 +728,6 @@ static void mxcmci_cmd_done(struct mxcmci_host *host, unsigned int stat) static irqreturn_t mxcmci_irq(int irq, void *devid) { struct mxcmci_host *host = devid; - unsigned long flags; bool sdio_irq; u32 stat; @@ -740,9 +739,9 @@ static irqreturn_t mxcmci_irq(int irq, void *devid) dev_dbg(mmc_dev(host->mmc), "%s: 0x%08x\n", __func__, stat); - spin_lock_irqsave(>lock, flags); + spin_lock(>lock); sdio_irq = (stat & STATUS_SDIO_INT_ACTIVE) && host->use_sdio; - spin_unlock_irqrestore(>lock, flags); + spin_unlock(>lock); if (mxcmci_use_dma(host) && (stat & (STATUS_WRITE_OP_DONE))) mxcmci_writel(host, STATUS_WRITE_OP_DONE, MMC_REG_STATUS); -- 2.17.1