* [PATCH v3 0/3] add inversion signal presence support
@ 2019-11-18 10:46 Ivan Mikhaylov
2019-11-18 10:46 ` [PATCH v3 1/3] aspeed: dts: add sd card for vesnin Ivan Mikhaylov
` (2 more replies)
0 siblings, 3 replies; 12+ messages in thread
From: Ivan Mikhaylov @ 2019-11-18 10:46 UTC (permalink / raw)
Cc: Ivan Mikhaylov, Joel Stanley, Andrew Jeffery, Rob Herring,
Mark Rutland, Adrian Hunter, Ulf Hansson, devicetree,
linux-arm-kernel, linux-aspeed, linux-kernel, linux-mmc, openbmc
Vesnin BMC uses microSD with card presence signal inversion in the
schematics. Add the .read_l callback to detect 'cd-inverted' option
in dts. There is no WP switch, due to this 'disable-wp' also was added
into vesnin dts for sdhci.
Ivan Mikhaylov (3):
aspeed: dts: add sd card for vesnin
mmc: sdhci-of-aspeed: enable CONFIG_MMC_SDHCI_IO_ACCESSORS
mmc: sdhci-of-aspeed: add inversion signal presence
arch/arm/boot/dts/aspeed-bmc-opp-vesnin.dts | 13 +++++++++++++
drivers/mmc/host/Kconfig | 1 +
drivers/mmc/host/sdhci-of-aspeed.c | 12 ++++++++++++
3 files changed, 26 insertions(+)
--
2.20.1
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH v3 1/3] aspeed: dts: add sd card for vesnin
2019-11-18 10:46 [PATCH v3 0/3] add inversion signal presence support Ivan Mikhaylov
@ 2019-11-18 10:46 ` Ivan Mikhaylov
2019-11-20 6:09 ` Andrew Jeffery
2019-11-18 10:46 ` [PATCH v3 2/3] mmc: sdhci-of-aspeed: enable CONFIG_MMC_SDHCI_IO_ACCESSORS Ivan Mikhaylov
2019-11-18 10:46 ` [PATCH v3 3/3] mmc: sdhci-of-aspeed: add inversion signal presence Ivan Mikhaylov
2 siblings, 1 reply; 12+ messages in thread
From: Ivan Mikhaylov @ 2019-11-18 10:46 UTC (permalink / raw)
Cc: Ivan Mikhaylov, Joel Stanley, Andrew Jeffery, Rob Herring,
Mark Rutland, Adrian Hunter, Ulf Hansson, devicetree,
linux-arm-kernel, linux-aspeed, linux-kernel, linux-mmc, openbmc
Presence signal is inverted for vesnin boards, 'cd-inverted' added
for invertion signal enablement. Vesnin BMC uses microSD, there is
no WP switch, 'disable-wp' is used for this purpose.
Signed-off-by: Ivan Mikhaylov <i.mikhaylov@yadro.com>
diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-vesnin.dts b/arch/arm/boot/dts/aspeed-bmc-opp-vesnin.dts
index a27c88d23056..7ae3436e0d99 100644
--- a/arch/arm/boot/dts/aspeed-bmc-opp-vesnin.dts
+++ b/arch/arm/boot/dts/aspeed-bmc-opp-vesnin.dts
@@ -232,3 +232,16 @@
&wdt2 {
aspeed,alt-boot;
};
+
+&sdmmc {
+ status = "okay";
+};
+
+&sdhci1 {
+ status = "okay";
+
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinctrl_sd2_default>;
+ cd-inverted;
+ disable-wp;
+};
--
2.20.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH v3 2/3] mmc: sdhci-of-aspeed: enable CONFIG_MMC_SDHCI_IO_ACCESSORS
2019-11-18 10:46 [PATCH v3 0/3] add inversion signal presence support Ivan Mikhaylov
2019-11-18 10:46 ` [PATCH v3 1/3] aspeed: dts: add sd card for vesnin Ivan Mikhaylov
@ 2019-11-18 10:46 ` Ivan Mikhaylov
2019-11-20 6:08 ` Andrew Jeffery
` (2 more replies)
2019-11-18 10:46 ` [PATCH v3 3/3] mmc: sdhci-of-aspeed: add inversion signal presence Ivan Mikhaylov
2 siblings, 3 replies; 12+ messages in thread
From: Ivan Mikhaylov @ 2019-11-18 10:46 UTC (permalink / raw)
Cc: Ivan Mikhaylov, Joel Stanley, Andrew Jeffery, Rob Herring,
Mark Rutland, Adrian Hunter, Ulf Hansson, devicetree,
linux-arm-kernel, linux-aspeed, linux-kernel, linux-mmc, openbmc
Enable CONFIG_MMC_SDHCI_IO_ACCESSORS on the aspeed board. The read_l
callback is used for inverted card detection.
Signed-off-by: Ivan Mikhaylov <i.mikhaylov@yadro.com>
diff --git a/drivers/mmc/host/Kconfig b/drivers/mmc/host/Kconfig
index 49ea02c467bf..c9c1bb722368 100644
--- a/drivers/mmc/host/Kconfig
+++ b/drivers/mmc/host/Kconfig
@@ -159,6 +159,7 @@ config MMC_SDHCI_OF_ASPEED
tristate "SDHCI OF support for the ASPEED SDHCI controller"
depends on MMC_SDHCI_PLTFM
depends on OF && OF_ADDRESS
+ select MMC_SDHCI_IO_ACCESSORS
help
This selects the ASPEED Secure Digital Host Controller Interface.
--
2.20.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH v3 3/3] mmc: sdhci-of-aspeed: add inversion signal presence
2019-11-18 10:46 [PATCH v3 0/3] add inversion signal presence support Ivan Mikhaylov
2019-11-18 10:46 ` [PATCH v3 1/3] aspeed: dts: add sd card for vesnin Ivan Mikhaylov
2019-11-18 10:46 ` [PATCH v3 2/3] mmc: sdhci-of-aspeed: enable CONFIG_MMC_SDHCI_IO_ACCESSORS Ivan Mikhaylov
@ 2019-11-18 10:46 ` Ivan Mikhaylov
2019-11-20 6:06 ` Andrew Jeffery
` (2 more replies)
2 siblings, 3 replies; 12+ messages in thread
From: Ivan Mikhaylov @ 2019-11-18 10:46 UTC (permalink / raw)
Cc: Ivan Mikhaylov, Joel Stanley, Andrew Jeffery, Rob Herring,
Mark Rutland, Adrian Hunter, Ulf Hansson, devicetree,
linux-arm-kernel, linux-aspeed, linux-kernel, linux-mmc, openbmc
Add read_l callback in sdhci_ops with flipping of SDHCI_CARD_PRESENT
bit in case of inverted card detection signal.
Signed-off-by: Ivan Mikhaylov <i.mikhaylov@yadro.com>
diff --git a/drivers/mmc/host/sdhci-of-aspeed.c b/drivers/mmc/host/sdhci-of-aspeed.c
index 8962f6664381..56912e30c47e 100644
--- a/drivers/mmc/host/sdhci-of-aspeed.c
+++ b/drivers/mmc/host/sdhci-of-aspeed.c
@@ -111,7 +111,19 @@ static void aspeed_sdhci_set_bus_width(struct sdhci_host *host, int width)
sdhci_writeb(host, ctrl, SDHCI_HOST_CONTROL);
}
+static u32 aspeed_sdhci_readl(struct sdhci_host *host, int reg)
+{
+ u32 val = readl(host->ioaddr + reg);
+
+ if (unlikely(reg == SDHCI_PRESENT_STATE) &&
+ (host->mmc->caps2 & MMC_CAP2_CD_ACTIVE_HIGH))
+ val ^= SDHCI_CARD_PRESENT;
+
+ return val;
+}
+
static const struct sdhci_ops aspeed_sdhci_ops = {
+ .read_l = aspeed_sdhci_readl,
.set_clock = aspeed_sdhci_set_clock,
.get_max_clock = aspeed_sdhci_get_max_clock,
.set_bus_width = aspeed_sdhci_set_bus_width,
--
2.20.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH v3 3/3] mmc: sdhci-of-aspeed: add inversion signal presence
2019-11-18 10:46 ` [PATCH v3 3/3] mmc: sdhci-of-aspeed: add inversion signal presence Ivan Mikhaylov
@ 2019-11-20 6:06 ` Andrew Jeffery
2019-11-20 7:45 ` Adrian Hunter
2019-11-20 12:59 ` Ulf Hansson
2 siblings, 0 replies; 12+ messages in thread
From: Andrew Jeffery @ 2019-11-20 6:06 UTC (permalink / raw)
To: Ivan Mikhaylov
Cc: Joel Stanley, Rob Herring, Mark Rutland, Adrian Hunter,
Ulf Hansson, devicetree, linux-arm-kernel, linux-aspeed,
linux-kernel, linux-mmc, openbmc
On Mon, 18 Nov 2019, at 21:16, Ivan Mikhaylov wrote:
> Add read_l callback in sdhci_ops with flipping of SDHCI_CARD_PRESENT
> bit in case of inverted card detection signal.
>
> Signed-off-by: Ivan Mikhaylov <i.mikhaylov@yadro.com>
Reviewed-by: Andrew Jeffery <andrew@aj.id.au>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v3 2/3] mmc: sdhci-of-aspeed: enable CONFIG_MMC_SDHCI_IO_ACCESSORS
2019-11-18 10:46 ` [PATCH v3 2/3] mmc: sdhci-of-aspeed: enable CONFIG_MMC_SDHCI_IO_ACCESSORS Ivan Mikhaylov
@ 2019-11-20 6:08 ` Andrew Jeffery
2019-11-20 7:44 ` Adrian Hunter
2019-11-20 12:58 ` Ulf Hansson
2 siblings, 0 replies; 12+ messages in thread
From: Andrew Jeffery @ 2019-11-20 6:08 UTC (permalink / raw)
To: Ivan Mikhaylov
Cc: Joel Stanley, Rob Herring, Mark Rutland, Adrian Hunter,
Ulf Hansson, devicetree, linux-arm-kernel, linux-aspeed,
linux-kernel, linux-mmc, openbmc
On Mon, 18 Nov 2019, at 21:16, Ivan Mikhaylov wrote:
> Enable CONFIG_MMC_SDHCI_IO_ACCESSORS on the aspeed board.
s/on the aspeed board/for the ASPEED MMC driver/
but otherwise:
Reviewed-by: Andrew Jeffery <andrew@aj.id.au>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v3 1/3] aspeed: dts: add sd card for vesnin
2019-11-18 10:46 ` [PATCH v3 1/3] aspeed: dts: add sd card for vesnin Ivan Mikhaylov
@ 2019-11-20 6:09 ` Andrew Jeffery
0 siblings, 0 replies; 12+ messages in thread
From: Andrew Jeffery @ 2019-11-20 6:09 UTC (permalink / raw)
To: Ivan Mikhaylov
Cc: Joel Stanley, Rob Herring, Mark Rutland, Adrian Hunter,
Ulf Hansson, devicetree, linux-arm-kernel, linux-aspeed,
linux-kernel, linux-mmc, openbmc
On Mon, 18 Nov 2019, at 21:16, Ivan Mikhaylov wrote:
> Presence signal is inverted for vesnin boards, 'cd-inverted' added
> for invertion signal enablement. Vesnin BMC uses microSD, there is
> no WP switch, 'disable-wp' is used for this purpose.
>
> Signed-off-by: Ivan Mikhaylov <i.mikhaylov@yadro.com>
Acked-by: Andrew Jeffery <andrew@aj.id.au>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v3 2/3] mmc: sdhci-of-aspeed: enable CONFIG_MMC_SDHCI_IO_ACCESSORS
2019-11-18 10:46 ` [PATCH v3 2/3] mmc: sdhci-of-aspeed: enable CONFIG_MMC_SDHCI_IO_ACCESSORS Ivan Mikhaylov
2019-11-20 6:08 ` Andrew Jeffery
@ 2019-11-20 7:44 ` Adrian Hunter
2019-11-20 12:58 ` Ulf Hansson
2 siblings, 0 replies; 12+ messages in thread
From: Adrian Hunter @ 2019-11-20 7:44 UTC (permalink / raw)
To: Ivan Mikhaylov
Cc: Joel Stanley, Andrew Jeffery, Rob Herring, Mark Rutland,
Ulf Hansson, devicetree, linux-arm-kernel, linux-aspeed,
linux-kernel, linux-mmc, openbmc
On 18/11/19 12:46 PM, Ivan Mikhaylov wrote:
> Enable CONFIG_MMC_SDHCI_IO_ACCESSORS on the aspeed board. The read_l
> callback is used for inverted card detection.
>
> Signed-off-by: Ivan Mikhaylov <i.mikhaylov@yadro.com>
Acked-by: Adrian Hunter <adrian.hunter@intel.com>
>
> diff --git a/drivers/mmc/host/Kconfig b/drivers/mmc/host/Kconfig
> index 49ea02c467bf..c9c1bb722368 100644
> --- a/drivers/mmc/host/Kconfig
> +++ b/drivers/mmc/host/Kconfig
> @@ -159,6 +159,7 @@ config MMC_SDHCI_OF_ASPEED
> tristate "SDHCI OF support for the ASPEED SDHCI controller"
> depends on MMC_SDHCI_PLTFM
> depends on OF && OF_ADDRESS
> + select MMC_SDHCI_IO_ACCESSORS
> help
> This selects the ASPEED Secure Digital Host Controller Interface.
>
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v3 3/3] mmc: sdhci-of-aspeed: add inversion signal presence
2019-11-18 10:46 ` [PATCH v3 3/3] mmc: sdhci-of-aspeed: add inversion signal presence Ivan Mikhaylov
2019-11-20 6:06 ` Andrew Jeffery
@ 2019-11-20 7:45 ` Adrian Hunter
2019-11-20 12:59 ` Ulf Hansson
2 siblings, 0 replies; 12+ messages in thread
From: Adrian Hunter @ 2019-11-20 7:45 UTC (permalink / raw)
To: Ivan Mikhaylov
Cc: Joel Stanley, Andrew Jeffery, Rob Herring, Mark Rutland,
Ulf Hansson, devicetree, linux-arm-kernel, linux-aspeed,
linux-kernel, linux-mmc, openbmc
On 18/11/19 12:46 PM, Ivan Mikhaylov wrote:
> Add read_l callback in sdhci_ops with flipping of SDHCI_CARD_PRESENT
> bit in case of inverted card detection signal.
>
> Signed-off-by: Ivan Mikhaylov <i.mikhaylov@yadro.com>
Acked-by: Adrian Hunter <adrian.hunter@intel.com>
>
> diff --git a/drivers/mmc/host/sdhci-of-aspeed.c b/drivers/mmc/host/sdhci-of-aspeed.c
> index 8962f6664381..56912e30c47e 100644
> --- a/drivers/mmc/host/sdhci-of-aspeed.c
> +++ b/drivers/mmc/host/sdhci-of-aspeed.c
> @@ -111,7 +111,19 @@ static void aspeed_sdhci_set_bus_width(struct sdhci_host *host, int width)
> sdhci_writeb(host, ctrl, SDHCI_HOST_CONTROL);
> }
>
> +static u32 aspeed_sdhci_readl(struct sdhci_host *host, int reg)
> +{
> + u32 val = readl(host->ioaddr + reg);
> +
> + if (unlikely(reg == SDHCI_PRESENT_STATE) &&
> + (host->mmc->caps2 & MMC_CAP2_CD_ACTIVE_HIGH))
> + val ^= SDHCI_CARD_PRESENT;
> +
> + return val;
> +}
> +
> static const struct sdhci_ops aspeed_sdhci_ops = {
> + .read_l = aspeed_sdhci_readl,
> .set_clock = aspeed_sdhci_set_clock,
> .get_max_clock = aspeed_sdhci_get_max_clock,
> .set_bus_width = aspeed_sdhci_set_bus_width,
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v3 2/3] mmc: sdhci-of-aspeed: enable CONFIG_MMC_SDHCI_IO_ACCESSORS
2019-11-18 10:46 ` [PATCH v3 2/3] mmc: sdhci-of-aspeed: enable CONFIG_MMC_SDHCI_IO_ACCESSORS Ivan Mikhaylov
2019-11-20 6:08 ` Andrew Jeffery
2019-11-20 7:44 ` Adrian Hunter
@ 2019-11-20 12:58 ` Ulf Hansson
2 siblings, 0 replies; 12+ messages in thread
From: Ulf Hansson @ 2019-11-20 12:58 UTC (permalink / raw)
To: Ivan Mikhaylov
Cc: Joel Stanley, Andrew Jeffery, Rob Herring, Mark Rutland,
Adrian Hunter, DTML, Linux ARM, linux-aspeed,
Linux Kernel Mailing List, linux-mmc, OpenBMC Maillist
On Mon, 18 Nov 2019 at 11:47, Ivan Mikhaylov <i.mikhaylov@yadro.com> wrote:
>
> Enable CONFIG_MMC_SDHCI_IO_ACCESSORS on the aspeed board. The read_l
> callback is used for inverted card detection.
>
> Signed-off-by: Ivan Mikhaylov <i.mikhaylov@yadro.com>
Applied for next, updating the changelog according to Andrew's comments, thanks!
Kind regards
Uffe
>
> diff --git a/drivers/mmc/host/Kconfig b/drivers/mmc/host/Kconfig
> index 49ea02c467bf..c9c1bb722368 100644
> --- a/drivers/mmc/host/Kconfig
> +++ b/drivers/mmc/host/Kconfig
> @@ -159,6 +159,7 @@ config MMC_SDHCI_OF_ASPEED
> tristate "SDHCI OF support for the ASPEED SDHCI controller"
> depends on MMC_SDHCI_PLTFM
> depends on OF && OF_ADDRESS
> + select MMC_SDHCI_IO_ACCESSORS
> help
> This selects the ASPEED Secure Digital Host Controller Interface.
>
> --
> 2.20.1
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v3 3/3] mmc: sdhci-of-aspeed: add inversion signal presence
2019-11-18 10:46 ` [PATCH v3 3/3] mmc: sdhci-of-aspeed: add inversion signal presence Ivan Mikhaylov
2019-11-20 6:06 ` Andrew Jeffery
2019-11-20 7:45 ` Adrian Hunter
@ 2019-11-20 12:59 ` Ulf Hansson
2019-12-02 21:57 ` Joel Stanley
2 siblings, 1 reply; 12+ messages in thread
From: Ulf Hansson @ 2019-11-20 12:59 UTC (permalink / raw)
To: Ivan Mikhaylov
Cc: Joel Stanley, Andrew Jeffery, Rob Herring, Mark Rutland,
Adrian Hunter, DTML, Linux ARM, linux-aspeed,
Linux Kernel Mailing List, linux-mmc, OpenBMC Maillist
On Mon, 18 Nov 2019 at 11:47, Ivan Mikhaylov <i.mikhaylov@yadro.com> wrote:
>
> Add read_l callback in sdhci_ops with flipping of SDHCI_CARD_PRESENT
> bit in case of inverted card detection signal.
>
> Signed-off-by: Ivan Mikhaylov <i.mikhaylov@yadro.com>
Applied for next, thanks!
For clarity, I am leaving patch 1 for arm-soc.
Kind regards
Uffe
>
> diff --git a/drivers/mmc/host/sdhci-of-aspeed.c b/drivers/mmc/host/sdhci-of-aspeed.c
> index 8962f6664381..56912e30c47e 100644
> --- a/drivers/mmc/host/sdhci-of-aspeed.c
> +++ b/drivers/mmc/host/sdhci-of-aspeed.c
> @@ -111,7 +111,19 @@ static void aspeed_sdhci_set_bus_width(struct sdhci_host *host, int width)
> sdhci_writeb(host, ctrl, SDHCI_HOST_CONTROL);
> }
>
> +static u32 aspeed_sdhci_readl(struct sdhci_host *host, int reg)
> +{
> + u32 val = readl(host->ioaddr + reg);
> +
> + if (unlikely(reg == SDHCI_PRESENT_STATE) &&
> + (host->mmc->caps2 & MMC_CAP2_CD_ACTIVE_HIGH))
> + val ^= SDHCI_CARD_PRESENT;
> +
> + return val;
> +}
> +
> static const struct sdhci_ops aspeed_sdhci_ops = {
> + .read_l = aspeed_sdhci_readl,
> .set_clock = aspeed_sdhci_set_clock,
> .get_max_clock = aspeed_sdhci_get_max_clock,
> .set_bus_width = aspeed_sdhci_set_bus_width,
> --
> 2.20.1
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v3 3/3] mmc: sdhci-of-aspeed: add inversion signal presence
2019-11-20 12:59 ` Ulf Hansson
@ 2019-12-02 21:57 ` Joel Stanley
0 siblings, 0 replies; 12+ messages in thread
From: Joel Stanley @ 2019-12-02 21:57 UTC (permalink / raw)
To: Ulf Hansson
Cc: Ivan Mikhaylov, Andrew Jeffery, Rob Herring, Mark Rutland,
Adrian Hunter, DTML, Linux ARM, linux-aspeed,
Linux Kernel Mailing List, linux-mmc, OpenBMC Maillist
On Wed, 20 Nov 2019 at 12:59, Ulf Hansson <ulf.hansson@linaro.org> wrote:
>
> On Mon, 18 Nov 2019 at 11:47, Ivan Mikhaylov <i.mikhaylov@yadro.com> wrote:
> >
> > Add read_l callback in sdhci_ops with flipping of SDHCI_CARD_PRESENT
> > bit in case of inverted card detection signal.
> >
> > Signed-off-by: Ivan Mikhaylov <i.mikhaylov@yadro.com>
>
> Applied for next, thanks!
>
> For clarity, I am leaving patch 1 for arm-soc.
Thanks. I'd already sent the aspeed pull request for 5.5, so I'll send
the device tree patch next merge window.
Cheers,
Joel
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2019-12-02 21:58 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-11-18 10:46 [PATCH v3 0/3] add inversion signal presence support Ivan Mikhaylov
2019-11-18 10:46 ` [PATCH v3 1/3] aspeed: dts: add sd card for vesnin Ivan Mikhaylov
2019-11-20 6:09 ` Andrew Jeffery
2019-11-18 10:46 ` [PATCH v3 2/3] mmc: sdhci-of-aspeed: enable CONFIG_MMC_SDHCI_IO_ACCESSORS Ivan Mikhaylov
2019-11-20 6:08 ` Andrew Jeffery
2019-11-20 7:44 ` Adrian Hunter
2019-11-20 12:58 ` Ulf Hansson
2019-11-18 10:46 ` [PATCH v3 3/3] mmc: sdhci-of-aspeed: add inversion signal presence Ivan Mikhaylov
2019-11-20 6:06 ` Andrew Jeffery
2019-11-20 7:45 ` Adrian Hunter
2019-11-20 12:59 ` Ulf Hansson
2019-12-02 21:57 ` Joel Stanley
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).