[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAPDyKFp+3E35KXVCACO3-6JcRyqdsNGJ-5mAEKmUK0nggGs9nQ@mail.gmail.com>
Date: Thu, 24 Jan 2019 16:12:35 +0100
From: Ulf Hansson <ulf.hansson@...aro.org>
To: Ludovic BARRE <ludovic.barre@...com>
Cc: DTML <devicetree@...r.kernel.org>,
"linux-mmc@...r.kernel.org" <linux-mmc@...r.kernel.org>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Rob Herring <robh+dt@...nel.org>,
Srinivas Kandagatla <srinivas.kandagatla@...aro.org>,
Maxime Coquelin <mcoquelin.stm32@...il.com>,
linux-stm32@...md-mailman.stormreply.com,
Linux ARM <linux-arm-kernel@...ts.infradead.org>
Subject: Re: [Linux-stm32] [PATCH V3 1/2] mmc: mmci: add variant property to
set command stop bit
On Thu, 24 Jan 2019 at 16:03, Ludovic BARRE <ludovic.barre@...com> wrote:
>
> hi Ulf
>
> I don't think you've seen my previous mail :-(
> what is your feeling about "mmc: mmci: send stop command to clear the dpsm"
Apologize for the delay. I wanted to check this in detail so I applied
your patch locally and started to play/test it.
However, a couple of other regressions was reported for v5.0 rcs, so I
got sidetracked.
Back on track by now, so I will have look asap. Thanks for pinging me!
Kind regards
Uffe
>
> Regards
> Ludo
>
> On 1/3/19 11:35 AM, Ludovic BARRE wrote:
> > hi Ulf
> >
> > happy new years.
> >
> > Just a gentleman ping about patch2 of this series
> > "mmc: mmci: send stop command to clear the dpsm."
> >
> > Regards
> > Ludo
> >
> > On 12/11/18 10:53 AM, Ludovic BARRE wrote:
> >>
> >>
> >> On 12/11/18 10:47 AM, Ulf Hansson wrote:
> >>> On Thu, 6 Dec 2018 at 16:13, Ludovic Barre <ludovic.Barre@...com> wrote:
> >>>>
> >>>> From: Ludovic Barre <ludovic.barre@...com>
> >>>>
> >>>> On cmd12 (STOP_TRANSMISSION), STM32 sdmmc variant needs to set
> >>>> cmdstop bit in command register. The CPSM ("Command Path State
> >>>> Machine")
> >>>> treats the command as a Stop Transmission command and signals
> >>>> abort to the DPSM ("Data Path State Machine").
> >>>>
> >>>> Signed-off-by: Ludovic Barre <ludovic.barre@...com>
> >>>
> >>> Applied for next, thanks!
> >>
> >> thanks
> >>
> >>>
> >>> Withholding patch2 for a while, as I need some more time to review it.
> >>
> >> No problem,
> >>
> >> Regards
> >> Ludo
> >>
> >>>
> >>> Kind regards
> >>> Uffe
> >>>
> >>>> ---
> >>>> drivers/mmc/host/mmci.c | 6 ++++++
> >>>> drivers/mmc/host/mmci.h | 2 ++
> >>>> 2 files changed, 8 insertions(+)
> >>>>
> >>>> diff --git a/drivers/mmc/host/mmci.c b/drivers/mmc/host/mmci.c
> >>>> index 13fa640..e352f5a 100644
> >>>> --- a/drivers/mmc/host/mmci.c
> >>>> +++ b/drivers/mmc/host/mmci.c
> >>>> @@ -21,6 +21,7 @@
> >>>> #include <linux/err.h>
> >>>> #include <linux/highmem.h>
> >>>> #include <linux/log2.h>
> >>>> +#include <linux/mmc/mmc.h>
> >>>> #include <linux/mmc/pm.h>
> >>>> #include <linux/mmc/host.h>
> >>>> #include <linux/mmc/card.h>
> >>>> @@ -274,6 +275,7 @@ static struct variant_data variant_stm32_sdmmc = {
> >>>> .cmdreg_lrsp_crc = MCI_CPSM_STM32_LRSP_CRC,
> >>>> .cmdreg_srsp_crc = MCI_CPSM_STM32_SRSP_CRC,
> >>>> .cmdreg_srsp = MCI_CPSM_STM32_SRSP,
> >>>> + .cmdreg_stop = MCI_CPSM_STM32_CMDSTOP,
> >>>> .data_cmd_enable = MCI_CPSM_STM32_CMDTRANS,
> >>>> .irq_pio_mask = MCI_IRQ_PIO_STM32_MASK,
> >>>> .datactrl_first = true,
> >>>> @@ -1100,6 +1102,10 @@ mmci_start_command(struct mmci_host *host,
> >>>> struct mmc_command *cmd, u32 c)
> >>>> mmci_reg_delay(host);
> >>>> }
> >>>>
> >>>> + if (host->variant->cmdreg_stop &&
> >>>> + cmd->opcode == MMC_STOP_TRANSMISSION)
> >>>> + c |= host->variant->cmdreg_stop;
> >>>> +
> >>>> c |= cmd->opcode | host->variant->cmdreg_cpsm_enable;
> >>>> if (cmd->flags & MMC_RSP_PRESENT) {
> >>>> if (cmd->flags & MMC_RSP_136)
> >>>> diff --git a/drivers/mmc/host/mmci.h b/drivers/mmc/host/mmci.h
> >>>> index 550dd39..2422909 100644
> >>>> --- a/drivers/mmc/host/mmci.h
> >>>> +++ b/drivers/mmc/host/mmci.h
> >>>> @@ -264,6 +264,7 @@ struct mmci_host;
> >>>> * @cmdreg_lrsp_crc: enable value for long response with crc
> >>>> * @cmdreg_srsp_crc: enable value for short response with crc
> >>>> * @cmdreg_srsp: enable value for short response without crc
> >>>> + * @cmdreg_stop: enable value for stop and abort transmission
> >>>> * @datalength_bits: number of bits in the MMCIDATALENGTH register
> >>>> * @fifosize: number of bytes that can be written when
> >>>> MMCI_TXFIFOEMPTY
> >>>> * is asserted (likewise for RX)
> >>>> @@ -316,6 +317,7 @@ struct variant_data {
> >>>> unsigned int cmdreg_lrsp_crc;
> >>>> unsigned int cmdreg_srsp_crc;
> >>>> unsigned int cmdreg_srsp;
> >>>> + unsigned int cmdreg_stop;
> >>>> unsigned int datalength_bits;
> >>>> unsigned int fifosize;
> >>>> unsigned int fifohalfsize;
> >>>> --
> >>>> 2.7.4
> >>>>
> > _______________________________________________
> > Linux-stm32 mailing list
> > Linux-stm32@...md-mailman.stormreply.com
> > https://st-md-mailman.stormreply.com/mailman/listinfo/linux-stm32
Powered by blists - more mailing lists