mbox series

[0/4] mmc: core: Preparations to support SD UHS-II cards

Message ID 20210225170352.76872-1-ulf.hansson@linaro.org
Headers show
Series mmc: core: Preparations to support SD UHS-II cards | expand

Message

Ulf Hansson Feb. 25, 2021, 5:03 p.m. UTC
A series [1] that has been collaborative worked upon by Takahiro Akashi
(Linaro) and Ben Chuang (Genesys Logic) is targeting to add SD UHS-II support
to the mmc subsystem.

Throughout the reviews, we realized that the changes affecting the mmc core to
support the UHS-II interface/protocol might not be entirely straightforward to
implement. Especially, I expressed some concerns about the code that manages
power on/off, initialization and power management of a SD UHS-II card.

Therefore, I have posted this small series to try to help to put some of the
foundation in the mmc core in place. Hopefully this can provide some guidance
and an overall structure, of how I think the code could evolve.

More details are available in the commit messages and through comments in the
code, for each path.

Kind regards
Uffe

[1]
https://lkml.org/lkml/2020/11/5/1472


Ulf Hansson (4):
  mmc: core: Cleanup printing of speed mode at card insertion
  mmc: core: Prepare to support SD UHS-II cards
  mmc: core: Announce successful insertion of an SD UHS-II card
  mmc: core: Extend support for mmc regulators with a vqmmc2

 drivers/mmc/core/Makefile    |   2 +-
 drivers/mmc/core/bus.c       |  38 +++--
 drivers/mmc/core/core.c      |  17 ++-
 drivers/mmc/core/core.h      |   1 +
 drivers/mmc/core/host.h      |   5 +
 drivers/mmc/core/regulator.c |  34 +++++
 drivers/mmc/core/sd_uhs2.c   | 289 +++++++++++++++++++++++++++++++++++
 include/linux/mmc/card.h     |   6 +
 include/linux/mmc/host.h     |  30 ++++
 9 files changed, 404 insertions(+), 18 deletions(-)
 create mode 100644 drivers/mmc/core/sd_uhs2.c

-- 
2.25.1

Comments

Ulf Hansson March 31, 2021, 9:57 a.m. UTC | #1
Hi Ben, Renius,

On Thu, 25 Feb 2021 at 18:04, Ulf Hansson <ulf.hansson@linaro.org> wrote:
>

> A series [1] that has been collaborative worked upon by Takahiro Akashi

> (Linaro) and Ben Chuang (Genesys Logic) is targeting to add SD UHS-II support

> to the mmc subsystem.

>

> Throughout the reviews, we realized that the changes affecting the mmc core to

> support the UHS-II interface/protocol might not be entirely straightforward to

> implement. Especially, I expressed some concerns about the code that manages

> power on/off, initialization and power management of a SD UHS-II card.

>

> Therefore, I have posted this small series to try to help to put some of the

> foundation in the mmc core in place. Hopefully this can provide some guidance

> and an overall structure, of how I think the code could evolve.

>

> More details are available in the commit messages and through comments in the

> code, for each path.

>

> Kind regards

> Uffe


I just wanted to get an update from your side. Did $subject series
help to move things forward?

If there is anything else I can help with, then please ping me.

Kind regards
Uffe

>

> [1]

> https://lkml.org/lkml/2020/11/5/1472

>

>

> Ulf Hansson (4):

>   mmc: core: Cleanup printing of speed mode at card insertion

>   mmc: core: Prepare to support SD UHS-II cards

>   mmc: core: Announce successful insertion of an SD UHS-II card

>   mmc: core: Extend support for mmc regulators with a vqmmc2

>

>  drivers/mmc/core/Makefile    |   2 +-

>  drivers/mmc/core/bus.c       |  38 +++--

>  drivers/mmc/core/core.c      |  17 ++-

>  drivers/mmc/core/core.h      |   1 +

>  drivers/mmc/core/host.h      |   5 +

>  drivers/mmc/core/regulator.c |  34 +++++

>  drivers/mmc/core/sd_uhs2.c   | 289 +++++++++++++++++++++++++++++++++++

>  include/linux/mmc/card.h     |   6 +

>  include/linux/mmc/host.h     |  30 ++++

>  9 files changed, 404 insertions(+), 18 deletions(-)

>  create mode 100644 drivers/mmc/core/sd_uhs2.c

>

> --

> 2.25.1

>
ReniusChen[陳建宏] March 31, 2021, 10:26 a.m. UTC | #2
Hi Ulf,


I'm currently trying to take over and refactor the task refer to your series.
And there hasn't been much progress recently because I'm busy with other work.


Regards,

Renius

>-----Original Message-----

>From: Ulf Hansson [mailto:ulf.hansson@linaro.org]

>Sent: Wednesday, March 31, 2021 5:57 PM

>To: BenChuang[莊智量]; Renius Chen

>Cc: AKASHI Takahiro; GregTu[杜啟軒]; Masami Hiramatsu; Takao Orito; Linux

>Kernel Mailing List; Ulf Hansson; Adrian Hunter; linux-mmc

>Subject: Re: [PATCH 0/4] mmc: core: Preparations to support SD UHS-II cards

>

>Hi Ben, Renius,

>

>On Thu, 25 Feb 2021 at 18:04, Ulf Hansson <ulf.hansson@linaro.org> wrote:

>>

>> A series [1] that has been collaborative worked upon by Takahiro Akashi

>> (Linaro) and Ben Chuang (Genesys Logic) is targeting to add SD UHS-II support

>> to the mmc subsystem.

>>

>> Throughout the reviews, we realized that the changes affecting the mmc core

>to

>> support the UHS-II interface/protocol might not be entirely straightforward to

>> implement. Especially, I expressed some concerns about the code that

>manages

>> power on/off, initialization and power management of a SD UHS-II card.

>>

>> Therefore, I have posted this small series to try to help to put some of the

>> foundation in the mmc core in place. Hopefully this can provide some

>guidance

>> and an overall structure, of how I think the code could evolve.

>>

>> More details are available in the commit messages and through comments in

>the

>> code, for each path.

>>

>> Kind regards

>> Uffe

>

>I just wanted to get an update from your side. Did $subject series

>help to move things forward?

>

>If there is anything else I can help with, then please ping me.

>

>Kind regards

>Uffe

>

>>

>> [1]

>> https://lkml.org/lkml/2020/11/5/1472

>>

>>

>> Ulf Hansson (4):

>>   mmc: core: Cleanup printing of speed mode at card insertion

>>   mmc: core: Prepare to support SD UHS-II cards

>>   mmc: core: Announce successful insertion of an SD UHS-II card

>>   mmc: core: Extend support for mmc regulators with a vqmmc2

>>

>>  drivers/mmc/core/Makefile    |   2 +-

>>  drivers/mmc/core/bus.c       |  38 +++--

>>  drivers/mmc/core/core.c      |  17 ++-

>>  drivers/mmc/core/core.h      |   1 +

>>  drivers/mmc/core/host.h      |   5 +

>>  drivers/mmc/core/regulator.c |  34 +++++

>>  drivers/mmc/core/sd_uhs2.c   | 289

>+++++++++++++++++++++++++++++++++++

>>  include/linux/mmc/card.h     |   6 +

>>  include/linux/mmc/host.h     |  30 ++++

>>  9 files changed, 404 insertions(+), 18 deletions(-)

>>  create mode 100644 drivers/mmc/core/sd_uhs2.c

>>

>> --

>> 2.25.1

>>

________________________________

Genesys Logic Email Confidentiality Notice:
This mail and any attachments may contain information that is confidential, proprietary, privileged or otherwise protected by law. The mail is intended solely for the named addressee (or a person responsible for delivering it to the addressee). If you are not the intended recipient of this mail, you are not authorized to read, print, copy or disseminate this mail.

If you have received this email in error, please notify us immediately by reply email and immediately delete this message and any attachments from your system. Please be noted that any unauthorized use, dissemination, distribution or copying of this email is strictly prohibited.
Ulf Hansson March 31, 2021, 10:31 a.m. UTC | #3
On Wed, 31 Mar 2021 at 12:26, ReniusChen[陳建宏]
<Renius.Chen@genesyslogic.com.tw> wrote:
>

> Hi Ulf,

>

>

> I'm currently trying to take over and refactor the task refer to your series.

> And there hasn't been much progress recently because I'm busy with other work.


Alright, thanks for sharing the update.

Don't hesitate to ping me if there is anything you need my help on
around the UHS-II support.

[...]

Kind regards
Uffe