mbox series

[v3,0/3] Add driver Support for MMCSD on AM654x

Message ID 20181210183507.9617-1-faiz_abbas@ti.com (mailing list archive)
Headers show
Series Add driver Support for MMCSD on AM654x | expand

Message

Faiz Abbas Dec. 10, 2018, 6:35 p.m. UTC
The following patches add driver support for MMCSD on TI's AM654
platforms.

Previously I had added the support to sdhci-of-arasan driver with
a separate phy driver[1]. Since then it has turned out that tuning
operation (for HS200, HS400 and SDR104 speed modes) will require
configuration of phy registers. This completely breaks the model of
the sdhci-of-arasan driver which relies on a separate driver to
configure the phy register space.

Because of this, I am creating a new driver with both the sdhci and
phy register spaces. This helps me use the phy registers in a future
patch that adds tuning support.

DT patches will be posted in a separate series.

Changes in v3:
 1. Removed reference to clk_ahb as its not being used.
 2. Added comment for pm_runtime being used for enabling
    clocks.

Changes in v2:
 1. Reset OTAP delay back to default value in init.
 2. Improved patch description for patch 2.
 3. Minor style fixes.

[1] driver and phy patches posted before
    https://patchwork.kernel.org/project/linux-mmc/list/?series=26623

Faiz Abbas (3):
  dt-bindings: mmc: sdhci-am654: Document bindings for the host
    controllers on TI's AM654 SOCs
  dt-bindings: mmc: sdhci-of-arasan: Add deprecated message for am65
  mmc: sdhci_am654: Add Initial Support for AM654 SDHCI driver

 .../devicetree/bindings/mmc/arasan,sdhci.txt  |   4 +
 .../devicetree/bindings/mmc/sdhci-am654.txt   |  37 ++
 drivers/mmc/host/Kconfig                      |  12 +
 drivers/mmc/host/Makefile                     |   1 +
 drivers/mmc/host/sdhci-of-arasan.c            |  46 ---
 drivers/mmc/host/sdhci_am654.c                | 374 ++++++++++++++++++
 6 files changed, 428 insertions(+), 46 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/mmc/sdhci-am654.txt
 create mode 100644 drivers/mmc/host/sdhci_am654.c

Comments

Faiz Abbas Dec. 17, 2018, 8:15 a.m. UTC | #1
Hi,

On 11/12/18 12:05 AM, Faiz Abbas wrote:
> The following patches add driver support for MMCSD on TI's AM654
> platforms.
> 
> Previously I had added the support to sdhci-of-arasan driver with
> a separate phy driver[1]. Since then it has turned out that tuning
> operation (for HS200, HS400 and SDR104 speed modes) will require
> configuration of phy registers. This completely breaks the model of
> the sdhci-of-arasan driver which relies on a separate driver to
> configure the phy register space.
> 
> Because of this, I am creating a new driver with both the sdhci and
> phy register spaces. This helps me use the phy registers in a future
> patch that adds tuning support.
> 
> DT patches will be posted in a separate series.
> 
> Changes in v3:
>  1. Removed reference to clk_ahb as its not being used.
>  2. Added comment for pm_runtime being used for enabling
>     clocks.
> 
> Changes in v2:
>  1. Reset OTAP delay back to default value in init.
>  2. Improved patch description for patch 2.
>  3. Minor style fixes.
> 
> [1] driver and phy patches posted before
>     https://patchwork.kernel.org/project/linux-mmc/list/?series=26623
> 
> Faiz Abbas (3):
>   dt-bindings: mmc: sdhci-am654: Document bindings for the host
>     controllers on TI's AM654 SOCs
>   dt-bindings: mmc: sdhci-of-arasan: Add deprecated message for am65
>   mmc: sdhci_am654: Add Initial Support for AM654 SDHCI driver
> 
>  .../devicetree/bindings/mmc/arasan,sdhci.txt  |   4 +
>  .../devicetree/bindings/mmc/sdhci-am654.txt   |  37 ++
>  drivers/mmc/host/Kconfig                      |  12 +
>  drivers/mmc/host/Makefile                     |   1 +
>  drivers/mmc/host/sdhci-of-arasan.c            |  46 ---
>  drivers/mmc/host/sdhci_am654.c                | 374 ++++++++++++++++++
>  6 files changed, 428 insertions(+), 46 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/mmc/sdhci-am654.txt
>  create mode 100644 drivers/mmc/host/sdhci_am654.c


Gentle ping.

Thanks,
Faiz
Sekhar Nori Dec. 17, 2018, 8:24 a.m. UTC | #2
On 17/12/18 1:45 PM, Faiz Abbas wrote:
> Hi,
> 
> On 11/12/18 12:05 AM, Faiz Abbas wrote:
>> The following patches add driver support for MMCSD on TI's AM654
>> platforms.
>>
>> Previously I had added the support to sdhci-of-arasan driver with
>> a separate phy driver[1]. Since then it has turned out that tuning
>> operation (for HS200, HS400 and SDR104 speed modes) will require
>> configuration of phy registers. This completely breaks the model of
>> the sdhci-of-arasan driver which relies on a separate driver to
>> configure the phy register space.
>>
>> Because of this, I am creating a new driver with both the sdhci and
>> phy register spaces. This helps me use the phy registers in a future
>> patch that adds tuning support.
>>
>> DT patches will be posted in a separate series.
>>
>> Changes in v3:
>>  1. Removed reference to clk_ahb as its not being used.
>>  2. Added comment for pm_runtime being used for enabling
>>     clocks.
>>
>> Changes in v2:
>>  1. Reset OTAP delay back to default value in init.
>>  2. Improved patch description for patch 2.
>>  3. Minor style fixes.
>>
>> [1] driver and phy patches posted before
>>     https://patchwork.kernel.org/project/linux-mmc/list/?series=26623
>>
>> Faiz Abbas (3):
>>   dt-bindings: mmc: sdhci-am654: Document bindings for the host
>>     controllers on TI's AM654 SOCs
>>   dt-bindings: mmc: sdhci-of-arasan: Add deprecated message for am65
>>   mmc: sdhci_am654: Add Initial Support for AM654 SDHCI driver
>>
>>  .../devicetree/bindings/mmc/arasan,sdhci.txt  |   4 +
>>  .../devicetree/bindings/mmc/sdhci-am654.txt   |  37 ++
>>  drivers/mmc/host/Kconfig                      |  12 +
>>  drivers/mmc/host/Makefile                     |   1 +
>>  drivers/mmc/host/sdhci-of-arasan.c            |  46 ---
>>  drivers/mmc/host/sdhci_am654.c                | 374 ++++++++++++++++++
>>  6 files changed, 428 insertions(+), 46 deletions(-)
>>  create mode 100644 Documentation/devicetree/bindings/mmc/sdhci-am654.txt
>>  create mode 100644 drivers/mmc/host/sdhci_am654.c
> 
> 
> Gentle ping.

I see these in linux-next. 

aff88ff23512 (Faiz Abbas) 6 days ago  mmc: sdhci_am654: Add Initial Support for AM654 SDHCI driver
7f6e4cc6727d (Faiz Abbas) 6 days ago  dt-bindings: mmc: sdhci-of-arasan: Add deprecated message for AM65
351b766ef2c7 (Faiz Abbas) 6 days ago  dt-bindings: mmc: sdhci-am654: Document bindings for the host controllers on TI's AM654 SOCs

Thanks,
Sekhar
Ulf Hansson Dec. 17, 2018, 8:26 a.m. UTC | #3
On Mon, 17 Dec 2018 at 09:24, Sekhar Nori <nsekhar@ti.com> wrote:
>
> On 17/12/18 1:45 PM, Faiz Abbas wrote:
> > Hi,
> >
> > On 11/12/18 12:05 AM, Faiz Abbas wrote:
> >> The following patches add driver support for MMCSD on TI's AM654
> >> platforms.
> >>
> >> Previously I had added the support to sdhci-of-arasan driver with
> >> a separate phy driver[1]. Since then it has turned out that tuning
> >> operation (for HS200, HS400 and SDR104 speed modes) will require
> >> configuration of phy registers. This completely breaks the model of
> >> the sdhci-of-arasan driver which relies on a separate driver to
> >> configure the phy register space.
> >>
> >> Because of this, I am creating a new driver with both the sdhci and
> >> phy register spaces. This helps me use the phy registers in a future
> >> patch that adds tuning support.
> >>
> >> DT patches will be posted in a separate series.
> >>
> >> Changes in v3:
> >>  1. Removed reference to clk_ahb as its not being used.
> >>  2. Added comment for pm_runtime being used for enabling
> >>     clocks.
> >>
> >> Changes in v2:
> >>  1. Reset OTAP delay back to default value in init.
> >>  2. Improved patch description for patch 2.
> >>  3. Minor style fixes.
> >>
> >> [1] driver and phy patches posted before
> >>     https://patchwork.kernel.org/project/linux-mmc/list/?series=26623
> >>
> >> Faiz Abbas (3):
> >>   dt-bindings: mmc: sdhci-am654: Document bindings for the host
> >>     controllers on TI's AM654 SOCs
> >>   dt-bindings: mmc: sdhci-of-arasan: Add deprecated message for am65
> >>   mmc: sdhci_am654: Add Initial Support for AM654 SDHCI driver
> >>
> >>  .../devicetree/bindings/mmc/arasan,sdhci.txt  |   4 +
> >>  .../devicetree/bindings/mmc/sdhci-am654.txt   |  37 ++
> >>  drivers/mmc/host/Kconfig                      |  12 +
> >>  drivers/mmc/host/Makefile                     |   1 +
> >>  drivers/mmc/host/sdhci-of-arasan.c            |  46 ---
> >>  drivers/mmc/host/sdhci_am654.c                | 374 ++++++++++++++++++
> >>  6 files changed, 428 insertions(+), 46 deletions(-)
> >>  create mode 100644 Documentation/devicetree/bindings/mmc/sdhci-am654.txt
> >>  create mode 100644 drivers/mmc/host/sdhci_am654.c
> >
> >
> > Gentle ping.
>
> I see these in linux-next.
>
> aff88ff23512 (Faiz Abbas) 6 days ago  mmc: sdhci_am654: Add Initial Support for AM654 SDHCI driver
> 7f6e4cc6727d (Faiz Abbas) 6 days ago  dt-bindings: mmc: sdhci-of-arasan: Add deprecated message for AM65
> 351b766ef2c7 (Faiz Abbas) 6 days ago  dt-bindings: mmc: sdhci-am654: Document bindings for the host controllers on TI's AM654 SOCs

Correct!

Apologize for not confirming that I have applied them. I don't know
why I missed that for this series.

Kind regards
Uffe