* [PATCH v4 0/6] Add MMC/SD support for S700/S900
@ 2021-11-28 11:32 Amit Singh Tomar
2021-11-28 11:32 ` [PATCH v4 1/6] clk: actions: Introduce dummy get/set_rate callbacks Amit Singh Tomar
` (5 more replies)
0 siblings, 6 replies; 18+ messages in thread
From: Amit Singh Tomar @ 2021-11-28 11:32 UTC (permalink / raw)
To: u-boot
Cc: andre.przywara, manivannan.sadhasivam, joe.hershberger,
jh80.chung, Amit Singh Tomar
Hi,
Sorry, it took me some time to come up with v4. For v3, all the
comments are from MMC driver(patch 5/6), and tried to address in
this series(v4) with changelog mentioned in the patch (5/6)
itself.
--------------------------------------------------------
This series(v3) addresses review comments provided by Andre and Jaehoon. Earlier
in MMC driver, we were *not* checking the command response errors(for instance,
the CRC error) based on STATU register. This is now addressed in patch 5/6.
Also, MMC driver(patch 5/6) now has replaced the switch/case with if/else to
correlate MMC_RSP_Rx bitmask values with register bits as suggested by Andre.
Apart from it, there are changes in patch 2/6 to fix weird assignments to div
variable.
----------------------------------------------------------------------------------
Series(v2) has few important updates, while loading large files we found
that MMC framework puts "0x1fffe00" into DMA Frame Length(DMA_FRAME_LEN 0x000C)
but register itself is limited to 24 bits and hence it was failing.
It is due to the wrong Block count(cfg->b_max) used in driver, that should be
just 512. This is now fixed in patch 5/6.
Apart from it, as Andre pointed that we might be just overclocking the MMC/SD
clock, and to confirm this we run following test:
$ md5sum clang
349eac46cbbe28f8e44da2dce07fa7b7 clang
U-Boot => ext4load mmc 0:2 0x0 clang
503316480 bytes read in 19516 ms (24.6 MiB/s)
U-Boot => md5sum 0x0 0x1e000000
md5 for 00000000 ... 1dffffff ==> d793bb51c4a1cf83c96d1980927461ff
Even though file gets loaded but md5sum doesn't match. This is now fixed in
patch 2/6
U-Boot => ext4load mmc 0:2 0x0 clang
503316480 bytes read in 41524 ms (11.6 MiB/s)
U-Boot => md5sum 0x0 0x1e000000
md5 for 00000000 ... 1dffffff ==> 349eac46cbbe28f8e44da2dce07fa7b7
-----------------------------------------------------------------------------
At the moment on S700 based platforms, only way to load/boot the Kernel
is from Ethernet, and with these patches one can now load/boot the
Kernel from uSD card.
Patches(1/6 and 2/6) adds changes needed for MMC/SD clock. It introduces
set/get callback routine and get/set MMC/SD clock rate based on device id.
Patch 4/6 adds MMC/SD node in U-boot specific dtsi file, which is used by MMC/SD
driver to read controller base address later on.
Patch 5/6 adds driver for MMC/SD controller present on S700 SoC, and its based
on Mainline Linux driver and DMA related bits is picked and simpilified from
vendor source.
Final patch 6/6 enables the driver support along with MMC commands in
Cubieboard7 config file.
Also, while at it just took the opportunity to synchronize the S700 SoC DT with
Linux in patch 3/6.
This patch-set is tested on Cubieboard7-lite board with following results:
U-Boot 2021.01-rc1-04434-g6589149-dirty (Dec 13 2020 - 13:51:07 +0530)
cubieboard7
DRAM: 1 GiB
PSCI: v0.2
MMC: mmc@e0210000: 0
In: serial@e0126000
Out: serial@e0126000
Err: serial@e0126000
Net: eth0: ethernet@e0220000
Hit any key to stop autoboot: 0
U-Boot =>
U-Boot =>
U-Boot =>
U-Boot => mmc info
Device: mmc@e0210000
Manufacturer ID: 3
OEM: 5344
Name: SC16G
Bus Speed: 50000000
Mode: SD High Speed (50MHz)
Rd Block Len: 512
SD version 3.0
High Capacity: Yes
Capacity: 14.8 GiB
Bus Width: 4-bit
Erase Group Size: 512 Bytes
U-Boot => setenv bootargs console=ttyOWL3,115200n8 earlycon=owl,0xe0126000 init=/sbin/init root=/dev/mmcblk0p2 rw rootwait
U-Boot => setenv kernel_addr_r 0x80000;setenv fdt_addr_r 0x10000000;
U-Boot => fatload mmc 0:1 ${kernel_addr_r} image ;fatload mmc 0:1 ${fdt_addr_r} s700-cubieboard7.dtb
27480576 bytes read in 1041 ms (25.2 MiB/s)
7056 bytes read in 2 ms (3.4 MiB/s)
U-Boot => booti $kernel_addr_r - $fdt_addr_r
## Flattened Device Tree blob at 10000000
Booting using the fdt blob at 0x10000000
Loading Device Tree to 000000003df56000, end 000000003df5ab8f ... OK
Starting kernel ...
[ 0.000000] Booting Linux on physical CPU 0x0000000000 [0x410fd034]
[ 0.000000] Linux version 5.7.0-rc6-00022-g99f1c330923b-dirty (amit@amit-ThinkPad-X230) (gcc version 7.3.1 20180425 [linaro-7.3-2018.05 revision d29120a424ecfbc167ef90065c0eeb7f91977701] (Linaro GCC 7.3-2018.05), GNU ld (GNU Binutils for Ubuntu) 2.26.1) #474 SMP PREEMPT Thu Aug 13 15:27:10 IST 2020
[ 0.000000] Machine model: CubieBoard7
[ 0.000000] earlycon: owl0 at MMIO 0x00000000e0126000 (options '')
[ 0.000000] printk: bootconsole [owl0] enabled
[ 0.000000] efi: UEFI not found.
[ 0.000000] cma: Reserved 32 MiB at 0x000000003e000000
Amit Singh Tomar (6):
clk: actions: Introduce dummy get/set_rate callbacks
clk: actions: Add SD/MMC clocks
ARM: dts: sync Actions Semi S700 DT from Linux 5.10-rc7
ARM: dts: s700: add MMC/SD controller node
mmc: actions: add MMC driver for Actions OWL S700/S900
configs: Enable mmc support
arch/arm/dts/s700-u-boot.dtsi | 10 +
arch/arm/dts/s700.dtsi | 17 +-
configs/cubieboard7_defconfig | 3 +
drivers/clk/owl/clk_owl.c | 99 +++++
drivers/mmc/Kconfig | 7 +
drivers/mmc/Makefile | 1 +
drivers/mmc/owl_mmc.c | 408 ++++++++++++++++++
.../dt-bindings/power/owl-s700-powergate.h | 19 +
8 files changed, 563 insertions(+), 1 deletion(-)
create mode 100644 drivers/mmc/owl_mmc.c
create mode 100644 include/dt-bindings/power/owl-s700-powergate.h
--
2.27.0
^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH v4 1/6] clk: actions: Introduce dummy get/set_rate callbacks
2021-11-28 11:32 [PATCH v4 0/6] Add MMC/SD support for S700/S900 Amit Singh Tomar
@ 2021-11-28 11:32 ` Amit Singh Tomar
2022-01-15 17:44 ` Sean Anderson
2022-01-18 21:09 ` Tom Rini
2021-11-28 11:32 ` [PATCH v4 2/6] clk: actions: Add SD/MMC clocks Amit Singh Tomar
` (4 subsequent siblings)
5 siblings, 2 replies; 18+ messages in thread
From: Amit Singh Tomar @ 2021-11-28 11:32 UTC (permalink / raw)
To: u-boot
Cc: andre.przywara, manivannan.sadhasivam, joe.hershberger,
jh80.chung, Amit Singh Tomar
From: Amit Singh Tomar <amittomer25@gmail.com>
This commit introduces get/set_rate callbacks, these are dummy at
the moment, and can be used to get/set clock for various devices
based on the clk id.
Signed-off-by: Amit Singh Tomar <amittomer25@gmail.com>
---
Changes since v3:
* No changes.
Changes since v2:
* No changes.
Changes since previous version:
* Removed premature initialization to avoid
compiler warnings.
---
drivers/clk/owl/clk_owl.c | 26 ++++++++++++++++++++++++++
1 file changed, 26 insertions(+)
diff --git a/drivers/clk/owl/clk_owl.c b/drivers/clk/owl/clk_owl.c
index 96ab7fed1f37..f78e5fa3f08d 100644
--- a/drivers/clk/owl/clk_owl.c
+++ b/drivers/clk/owl/clk_owl.c
@@ -128,6 +128,30 @@ int owl_clk_disable(struct clk *clk)
return 0;
}
+static ulong owl_clk_get_rate(struct clk *clk)
+{
+ ulong rate;
+
+ switch (clk->id) {
+ default:
+ return -ENOENT;
+ }
+
+ return rate;
+}
+
+static ulong owl_clk_set_rate(struct clk *clk, ulong rate)
+{
+ ulong new_rate;
+
+ switch (clk->id) {
+ default:
+ return -ENOENT;
+ }
+
+ return new_rate;
+}
+
static int owl_clk_probe(struct udevice *dev)
{
struct owl_clk_priv *priv = dev_get_priv(dev);
@@ -145,6 +169,8 @@ static int owl_clk_probe(struct udevice *dev)
static const struct clk_ops owl_clk_ops = {
.enable = owl_clk_enable,
.disable = owl_clk_disable,
+ .get_rate = owl_clk_get_rate,
+ .set_rate = owl_clk_set_rate,
};
static const struct udevice_id owl_clk_ids[] = {
--
2.27.0
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH v4 2/6] clk: actions: Add SD/MMC clocks
2021-11-28 11:32 [PATCH v4 0/6] Add MMC/SD support for S700/S900 Amit Singh Tomar
2021-11-28 11:32 ` [PATCH v4 1/6] clk: actions: Introduce dummy get/set_rate callbacks Amit Singh Tomar
@ 2021-11-28 11:32 ` Amit Singh Tomar
2022-01-15 17:44 ` Sean Anderson
2022-01-18 21:09 ` Tom Rini
2021-11-28 11:32 ` [PATCH v4 3/6] ARM: dts: sync Actions Semi S700 DT from Linux 5.10-rc7 Amit Singh Tomar
` (3 subsequent siblings)
5 siblings, 2 replies; 18+ messages in thread
From: Amit Singh Tomar @ 2021-11-28 11:32 UTC (permalink / raw)
To: u-boot
Cc: andre.przywara, manivannan.sadhasivam, joe.hershberger,
jh80.chung, Amit Singh Tomar
From: Amit Singh Tomar <amittomer25@gmail.com>
This commit adds SD/MMC clocks, and provides .set/get_rate callbacks
for SD/MMC device present on Actions OWL S700 SoCs.
Signed-off-by: Amit Singh Tomar <amittomer25@gmail.com>
---
Changes since v3:
* No change.
Changes since v2:
* Fixed the weird div assignment.
* Moved the clock bit for SD from header file
to driver file.
* Removed "< 0" part while comparing unsigned.
Changes since previous version:
* Removed rate *= 2 as this just overclocks.
* Separated the divide by 128 bit from divider value.
* Provided the separate routine to get sd parent rate
based on bit 9.
* Removed unnecessary initialization.
---
drivers/clk/owl/clk_owl.c | 73 +++++++++++++++++++++++++++++++++++++++
1 file changed, 73 insertions(+)
diff --git a/drivers/clk/owl/clk_owl.c b/drivers/clk/owl/clk_owl.c
index f78e5fa3f08d..678fdd5a4540 100644
--- a/drivers/clk/owl/clk_owl.c
+++ b/drivers/clk/owl/clk_owl.c
@@ -20,6 +20,8 @@
#include <linux/bitops.h>
#include <linux/delay.h>
+#define CMU_DEVCLKEN0_SD0 BIT(22)
+
void owl_clk_init(struct owl_clk_priv *priv)
{
u32 bus_clk = 0, core_pll, dev_pll;
@@ -92,6 +94,9 @@ int owl_clk_enable(struct clk *clk)
setbits_le32(priv->base + CMU_DEVCLKEN1, CMU_DEVCLKEN1_ETH);
setbits_le32(priv->base + CMU_ETHERNETPLL, 5);
break;
+ case CLK_SD0:
+ setbits_le32(priv->base + CMU_DEVCLKEN0, CMU_DEVCLKEN0_SD0);
+ break;
default:
return -EINVAL;
}
@@ -121,6 +126,9 @@ int owl_clk_disable(struct clk *clk)
case CLK_ETHERNET:
clrbits_le32(priv->base + CMU_DEVCLKEN1, CMU_DEVCLKEN1_ETH);
break;
+ case CLK_SD0:
+ clrbits_le32(priv->base + CMU_DEVCLKEN0, CMU_DEVCLKEN0_SD0);
+ break;
default:
return -EINVAL;
}
@@ -128,11 +136,72 @@ int owl_clk_disable(struct clk *clk)
return 0;
}
+static ulong get_sd_parent_rate(struct owl_clk_priv *priv, u32 dev_index)
+{
+ ulong rate;
+ u32 reg;
+
+ reg = readl(priv->base + (CMU_SD0CLK + dev_index * 0x4));
+ /* Clock output of DEV/NAND_PLL
+ * Range: 48M ~ 756M
+ * Frequency= PLLCLK * 6
+ */
+ if (reg & 0x200)
+ rate = readl(priv->base + CMU_NANDPLL) & 0x7f;
+ else
+ rate = readl(priv->base + CMU_DEVPLL) & 0x7f;
+
+ rate *= 6000000;
+
+ return rate;
+}
+
+static ulong owl_get_sd_clk_rate(struct owl_clk_priv *priv, int sd_index)
+{
+ uint div, val;
+ ulong parent_rate = get_sd_parent_rate(priv, sd_index);
+
+ val = readl(priv->base + (CMU_SD0CLK + sd_index * 0x4));
+ div = (val & 0x1f) + 1;
+
+ return (parent_rate / div);
+}
+
+static ulong owl_set_sd_clk_rate(struct owl_clk_priv *priv, ulong rate,
+ int sd_index)
+{
+ uint div, val;
+ ulong parent_rate = get_sd_parent_rate(priv, sd_index);
+
+ if (rate == 0)
+ return rate;
+
+ div = (parent_rate / rate);
+
+ val = readl(priv->base + (CMU_SD0CLK + sd_index * 0x4));
+ /* Bits 4..0 is used to program div value and bit 8 to enable
+ * divide by 128 circuit
+ */
+ val &= ~0x11f;
+ if (div >= 128) {
+ div = div / 128;
+ val |= 0x100; /* enable divide by 128 circuit */
+ }
+ val |= ((div - 1) & 0x1f);
+ writel(val, priv->base + (CMU_SD0CLK + sd_index * 0x4));
+
+ return owl_get_sd_clk_rate(priv, 0);
+}
+
static ulong owl_clk_get_rate(struct clk *clk)
{
+ struct owl_clk_priv *priv = dev_get_priv(clk->dev);
ulong rate;
switch (clk->id) {
+ case CLK_SD0:
+ rate = owl_get_sd_clk_rate(priv, 0);
+ break;
default:
return -ENOENT;
}
@@ -142,9 +211,13 @@ static ulong owl_clk_get_rate(struct clk *clk)
static ulong owl_clk_set_rate(struct clk *clk, ulong rate)
{
+ struct owl_clk_priv *priv = dev_get_priv(clk->dev);
ulong new_rate;
switch (clk->id) {
+ case CLK_SD0:
+ new_rate = owl_set_sd_clk_rate(priv, rate, 0);
+ break;
default:
return -ENOENT;
}
--
2.27.0
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH v4 3/6] ARM: dts: sync Actions Semi S700 DT from Linux 5.10-rc7
2021-11-28 11:32 [PATCH v4 0/6] Add MMC/SD support for S700/S900 Amit Singh Tomar
2021-11-28 11:32 ` [PATCH v4 1/6] clk: actions: Introduce dummy get/set_rate callbacks Amit Singh Tomar
2021-11-28 11:32 ` [PATCH v4 2/6] clk: actions: Add SD/MMC clocks Amit Singh Tomar
@ 2021-11-28 11:32 ` Amit Singh Tomar
2021-11-29 16:13 ` Tom Rini
2021-12-02 18:23 ` [PATCH v4 3/6] ARM: dts: sync Actions Semi S700 DT from Linux v5.16-rc3 Amit Singh Tomar
2021-11-28 11:32 ` [PATCH v4 4/6] ARM: dts: s700: add MMC/SD controller node Amit Singh Tomar
` (2 subsequent siblings)
5 siblings, 2 replies; 18+ messages in thread
From: Amit Singh Tomar @ 2021-11-28 11:32 UTC (permalink / raw)
To: u-boot
Cc: andre.przywara, manivannan.sadhasivam, joe.hershberger,
jh80.chung, Amit Singh Tomar
From: Amit Singh Tomar <amittomer25@gmail.com>
This Synchronizes the Actions Semi S700 SoC DT changes from
commit "0477e9288185" ("Linux 5.10-rc7").
Signed-off-by: Amit Singh Tomar <amittomer25@gmail.com>
---
Changes since previous versions
* No change.
---
arch/arm/dts/s700.dtsi | 17 ++++++++++++++++-
.../dt-bindings/power/owl-s700-powergate.h | 19 +++++++++++++++++++
2 files changed, 35 insertions(+), 1 deletion(-)
create mode 100644 include/dt-bindings/power/owl-s700-powergate.h
diff --git a/arch/arm/dts/s700.dtsi b/arch/arm/dts/s700.dtsi
index 2006ad5424fa..2c78caebf515 100644
--- a/arch/arm/dts/s700.dtsi
+++ b/arch/arm/dts/s700.dtsi
@@ -5,6 +5,7 @@
#include <dt-bindings/clock/actions,s700-cmu.h>
#include <dt-bindings/interrupt-controller/arm-gic.h>
+#include <dt-bindings/power/owl-s700-powergate.h>
#include <dt-bindings/reset/actions,s700-reset.h>
/ {
@@ -231,7 +232,7 @@
pinctrl: pinctrl@e01b0000 {
compatible = "actions,s700-pinctrl";
- reg = <0x0 0xe01b0000 0x0 0x1000>;
+ reg = <0x0 0xe01b0000 0x0 0x100>;
clocks = <&cmu CLK_GPIO>;
gpio-controller;
gpio-ranges = <&pinctrl 0 0 136>;
@@ -244,5 +245,19 @@
<GIC_SPI 39 IRQ_TYPE_LEVEL_HIGH>,
<GIC_SPI 40 IRQ_TYPE_LEVEL_HIGH>;
};
+
+ dma: dma-controller@e0230000 {
+ compatible = "actions,s700-dma";
+ reg = <0x0 0xe0230000 0x0 0x1000>;
+ interrupts = <GIC_SPI 57 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 58 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 59 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 60 IRQ_TYPE_LEVEL_HIGH>;
+ #dma-cells = <1>;
+ dma-channels = <10>;
+ dma-requests = <44>;
+ clocks = <&cmu CLK_DMAC>;
+ power-domains = <&sps S700_PD_DMA>;
+ };
};
};
diff --git a/include/dt-bindings/power/owl-s700-powergate.h b/include/dt-bindings/power/owl-s700-powergate.h
new file mode 100644
index 000000000000..4cf1aefbf09c
--- /dev/null
+++ b/include/dt-bindings/power/owl-s700-powergate.h
@@ -0,0 +1,19 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+/*
+ * Actions Semi S700 SPS
+ *
+ * Copyright (c) 2017 Andreas Färber
+ */
+#ifndef DT_BINDINGS_POWER_OWL_S700_POWERGATE_H
+#define DT_BINDINGS_POWER_OWL_S700_POWERGATE_H
+
+#define S700_PD_VDE 0
+#define S700_PD_VCE_SI 1
+#define S700_PD_USB2_1 2
+#define S700_PD_HDE 3
+#define S700_PD_DMA 4
+#define S700_PD_DS 5
+#define S700_PD_USB3 6
+#define S700_PD_USB2_0 7
+
+#endif
--
2.27.0
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH v4 4/6] ARM: dts: s700: add MMC/SD controller node
2021-11-28 11:32 [PATCH v4 0/6] Add MMC/SD support for S700/S900 Amit Singh Tomar
` (2 preceding siblings ...)
2021-11-28 11:32 ` [PATCH v4 3/6] ARM: dts: sync Actions Semi S700 DT from Linux 5.10-rc7 Amit Singh Tomar
@ 2021-11-28 11:32 ` Amit Singh Tomar
2022-01-05 17:43 ` Tom Rini
2022-01-18 21:09 ` Tom Rini
2021-11-28 11:32 ` [PATCH v4 5/6] mmc: actions: add MMC driver for Actions OWL S700/S900 Amit Singh Tomar
2021-11-28 11:32 ` [PATCH v4 6/6] configs: Enable mmc support Amit Singh Tomar
5 siblings, 2 replies; 18+ messages in thread
From: Amit Singh Tomar @ 2021-11-28 11:32 UTC (permalink / raw)
To: u-boot
Cc: andre.przywara, manivannan.sadhasivam, joe.hershberger,
jh80.chung, Amit Singh Tomar
From: Amit Singh Tomar <amittomer25@gmail.com>
This patch adds node for mmc/sd controller found on Action Semi OWL
S700 SoC.
Since, upstream Linux binding has not been merged for S700 MMC/SD
controller, Changes are put in u-boot specific dtsi file.
Signed-off-by: Amit Singh Tomar <amittomer25@gmail.com>
---
Changes since previous versions
* No change.
---
arch/arm/dts/s700-u-boot.dtsi | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/arch/arm/dts/s700-u-boot.dtsi b/arch/arm/dts/s700-u-boot.dtsi
index 1b2768272c62..3c3396bccf11 100644
--- a/arch/arm/dts/s700-u-boot.dtsi
+++ b/arch/arm/dts/s700-u-boot.dtsi
@@ -19,6 +19,16 @@
status = "okay";
};
+ mmc0: mmc@e0210000 {
+ compatible = "actions,s700-mmc", "actions,owl-mmc";
+ reg = <0x0 0xe0210000 0x0 0x4000>;
+ interrupts = <GIC_SPI 42 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&cmu CLK_SD0>;
+ dmas = <&dma 2>;
+ dma-names = "mmc";
+ bus-width = <4>;
+ status = "okay";
+ };
};
};
--
2.27.0
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH v4 5/6] mmc: actions: add MMC driver for Actions OWL S700/S900
2021-11-28 11:32 [PATCH v4 0/6] Add MMC/SD support for S700/S900 Amit Singh Tomar
` (3 preceding siblings ...)
2021-11-28 11:32 ` [PATCH v4 4/6] ARM: dts: s700: add MMC/SD controller node Amit Singh Tomar
@ 2021-11-28 11:32 ` Amit Singh Tomar
2022-01-18 21:09 ` Tom Rini
2021-11-28 11:32 ` [PATCH v4 6/6] configs: Enable mmc support Amit Singh Tomar
5 siblings, 1 reply; 18+ messages in thread
From: Amit Singh Tomar @ 2021-11-28 11:32 UTC (permalink / raw)
To: u-boot
Cc: andre.przywara, manivannan.sadhasivam, joe.hershberger,
jh80.chung, Amit Singh Tomar
From: Amit Singh Tomar <amittomer25@gmail.com>
This commit adds support for MMC controllers found on Actions OWL
SoC platform(S700/S900).
Signed-off-by: Amit Singh Tomar <amittomer25@gmail.com>
---
Changes since v3:
* Re-based to latest master (2ad8d0cb950da)
There are some "dm: treewide" changes have been
made since v3, for instance:
dm: treewide: Rename ofdata_to_platdata() to of_to_plat()
It is addressed in v4 for this MMC driver.
* Used setbits_le32( wherever possible.
* Properly write the OWL_REG_SD_BUF_SIZE register.
* Updated the mode handling as suggested by Ander:
mode = 0;
if (cmd->resp_type != MMC_RSP_NONE)
cmd_rsp_mask = OWL_SD_STATE_CLNR | OWL_SD_STATE_CRC7ER;
if (md->resp_type == MMC_RSP_R1) {
.... (always assign to "mode")
* Properly handled the unsupported SD clock rate.
* Wait for DMA transfer to complete before stoping the DMA.
Changes since v2:
* Progammed read/write delays as suggested by
Jaehoon, its more readable now.
* Updated commit message and source file to reflect
it supports S900 as well.
* Updated SD_DMA_CHANNEL macro to include channel offset.
* Checked the command response Error based on SDC STATU register
bits.
* Replaced switch/case with if/else to correlate with MMC_RSP_Rx bitmask
values, as suggested by Andre.
* Introduced clk_disable based on mmc->clk_disable flag.
* Guarded the clk_set_rate under if statment to avoid reprogramming the
rate.
* Fetched the DRQ number from using args.args[0].
Changes since previous version
* Corrected block count to 512.
* Changed the command timeout value to 30ms.
* Used readl_poll_timeout.
* Read DMA parameters from DT instead of hardcoding it.
* Reduced number of arguments passed to own_dma_cofig.
* Removed debug leftover.
* Used mmc_of_parse().
---
drivers/mmc/Kconfig | 7 +
drivers/mmc/Makefile | 1 +
drivers/mmc/owl_mmc.c | 408 ++++++++++++++++++++++++++++++++++++++++++
3 files changed, 416 insertions(+)
create mode 100644 drivers/mmc/owl_mmc.c
diff --git a/drivers/mmc/Kconfig b/drivers/mmc/Kconfig
index e0927ce1c9bd..f04cc44e1973 100644
--- a/drivers/mmc/Kconfig
+++ b/drivers/mmc/Kconfig
@@ -295,6 +295,13 @@ config MMC_MXC
If unsure, say N.
+config MMC_OWL
+ bool "Actions OWL Multimedia Card Interface support"
+ depends on ARCH_OWL && DM_MMC && BLK
+ help
+ This selects the OWL SD/MMC host controller found on board
+ based on Actions S700/S900 SoC.
+
config MMC_MXS
bool "Freescale MXS Multimedia Card Interface support"
depends on MX23 || MX28 || MX6 || MX7
diff --git a/drivers/mmc/Makefile b/drivers/mmc/Makefile
index 7e819d2b3a25..17ebc04203ed 100644
--- a/drivers/mmc/Makefile
+++ b/drivers/mmc/Makefile
@@ -39,6 +39,7 @@ obj-$(CONFIG_MMC_OMAP_HS) += omap_hsmmc.o
obj-$(CONFIG_MMC_MXC) += mxcmmc.o
obj-$(CONFIG_MMC_MXS) += mxsmmc.o
obj-$(CONFIG_MMC_OCTEONTX) += octeontx_hsmmc.o
+obj-$(CONFIG_MMC_OWL) += owl_mmc.o
obj-$(CONFIG_MMC_PCI) += pci_mmc.o
obj-$(CONFIG_PXA_MMC_GENERIC) += pxa_mmc_gen.o
obj-$(CONFIG_$(SPL_TPL_)SUPPORT_EMMC_RPMB) += rpmb.o
diff --git a/drivers/mmc/owl_mmc.c b/drivers/mmc/owl_mmc.c
new file mode 100644
index 000000000000..e84171a661a7
--- /dev/null
+++ b/drivers/mmc/owl_mmc.c
@@ -0,0 +1,408 @@
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * Copyright (C) 2020 Amit Singh Tomar <amittomer25@gmail.com>
+ *
+ * Driver for SD/MMC controller present on Actions Semi S700/S900 SoC, based
+ * on Linux Driver "drivers/mmc/host/owl-mmc.c".
+ *
+ * Though, there is a bit (BSEL, BUS or DMA Special Channel Selection) that
+ * controls the data transfer from SDx_DAT register either using CPU AHB Bus
+ * or DMA channel, but seems like, it only works correctly using external DMA
+ * channel, and those special bits used in this driver is picked from vendor
+ * source exclusively for MMC/SD.
+ */
+#include <common.h>
+#include <clk.h>
+#include <cpu_func.h>
+#include <dm.h>
+#include <errno.h>
+#include <log.h>
+#include <mmc.h>
+#include <asm/io.h>
+#include <linux/bitops.h>
+#include <linux/delay.h>
+#include <linux/err.h>
+#include <linux/iopoll.h>
+
+/*
+ * SDC registers
+ */
+#define OWL_REG_SD_EN 0x0000
+#define OWL_REG_SD_CTL 0x0004
+#define OWL_REG_SD_STATE 0x0008
+#define OWL_REG_SD_CMD 0x000c
+#define OWL_REG_SD_ARG 0x0010
+#define OWL_REG_SD_RSPBUF0 0x0014
+#define OWL_REG_SD_RSPBUF1 0x0018
+#define OWL_REG_SD_RSPBUF2 0x001c
+#define OWL_REG_SD_RSPBUF3 0x0020
+#define OWL_REG_SD_RSPBUF4 0x0024
+#define OWL_REG_SD_DAT 0x0028
+#define OWL_REG_SD_BLK_SIZE 0x002c
+#define OWL_REG_SD_BLK_NUM 0x0030
+#define OWL_REG_SD_BUF_SIZE 0x0034
+
+/* SD_EN Bits */
+#define OWL_SD_EN_RANE BIT(31)
+#define OWL_SD_EN_RESE BIT(10)
+#define OWL_SD_ENABLE BIT(7)
+#define OWL_SD_EN_BSEL BIT(6)
+#define OWL_SD_EN_DATAWID(x) (((x) & 0x3) << 0)
+#define OWL_SD_EN_DATAWID_MASK 0x03
+
+/* SD_CTL Bits */
+#define OWL_SD_CTL_TOUTEN BIT(31)
+#define OWL_SD_CTL_DELAY_MSK GENMASK(23, 16)
+#define OWL_SD_CTL_RDELAY(x) (((x) & 0xf) << 20)
+#define OWL_SD_CTL_WDELAY(x) (((x) & 0xf) << 16)
+#define OWL_SD_CTL_TS BIT(7)
+#define OWL_SD_CTL_LBE BIT(6)
+#define OWL_SD_CTL_TM(x) (((x) & 0xf) << 0)
+
+#define OWL_SD_DELAY_LOW_CLK 0x0f
+#define OWL_SD_DELAY_MID_CLK 0x0a
+#define OWL_SD_RDELAY_HIGH 0x08
+#define OWL_SD_WDELAY_HIGH 0x09
+
+/* SD_STATE Bits */
+#define OWL_SD_STATE_DAT0S BIT(7)
+#define OWL_SD_STATE_CLNR BIT(4)
+#define OWL_SD_STATE_CRC7ER BIT(0)
+
+#define OWL_MMC_OCR (MMC_VDD_32_33 | MMC_VDD_33_34 | \
+ MMC_VDD_165_195)
+
+#define DATA_TRANSFER_TIMEOUT 3000000
+#define DMA_TRANSFER_TIMEOUT 5000000
+
+/*
+ * Simple DMA transfer operations defines for MMC/SD card
+ */
+#define SD_DMA_CHANNEL(base, channel) ((base) + 0x100 + 0x100 * (channel))
+
+#define DMA_MODE 0x0000
+#define DMA_SOURCE 0x0004
+#define DMA_DESTINATION 0x0008
+#define DMA_FRAME_LEN 0x000C
+#define DMA_FRAME_CNT 0x0010
+#define DMA_START 0x0024
+
+/* DMAx_MODE */
+#define DMA_MODE_ST(x) (((x) & 0x3) << 8)
+#define DMA_MODE_ST_DEV DMA_MODE_ST(0)
+#define DMA_MODE_DT(x) (((x) & 0x3) << 10)
+#define DMA_MODE_DT_DCU DMA_MODE_DT(2)
+#define DMA_MODE_SAM(x) (((x) & 0x3) << 16)
+#define DMA_MODE_SAM_CONST DMA_MODE_SAM(0)
+#define DMA_MODE_DAM(x) (((x) & 0x3) << 18)
+#define DMA_MODE_DAM_INC DMA_MODE_DAM(1)
+
+#define DMA_ENABLE 0x1
+
+struct owl_mmc_plat {
+ struct mmc_config cfg;
+ struct mmc mmc;
+};
+
+struct owl_mmc_priv {
+ void *reg_base;
+ void *dma_channel;
+ struct clk clk;
+ unsigned int clock; /* Current clock */
+ unsigned int dma_drq; /* Trigger Source */
+};
+
+static void owl_dma_config(struct owl_mmc_priv *priv, unsigned int src,
+ unsigned int dst, unsigned int len)
+{
+ unsigned int mode = priv->dma_drq;
+
+ /* Set Source and Destination adderess mode */
+ mode |= (DMA_MODE_ST_DEV | DMA_MODE_SAM_CONST | DMA_MODE_DT_DCU |
+ DMA_MODE_DAM_INC);
+
+ writel(mode, SD_DMA_CHANNEL(priv->dma_channel, 0) + DMA_MODE);
+ writel(src, SD_DMA_CHANNEL(priv->dma_channel, 0) + DMA_SOURCE);
+ writel(dst, SD_DMA_CHANNEL(priv->dma_channel, 0) + DMA_DESTINATION);
+ writel(len, SD_DMA_CHANNEL(priv->dma_channel, 0) + DMA_FRAME_LEN);
+ writel(0x1, SD_DMA_CHANNEL(priv->dma_channel, 0) + DMA_FRAME_CNT);
+}
+
+static void owl_mmc_prepare_data(struct owl_mmc_priv *priv,
+ struct mmc_data *data)
+{
+ unsigned int total;
+ u32 buf = 0;
+
+ setbits_le32(priv->reg_base + OWL_REG_SD_EN, OWL_SD_EN_BSEL);
+
+ writel(data->blocks, priv->reg_base + OWL_REG_SD_BLK_NUM);
+ writel(data->blocksize, priv->reg_base + OWL_REG_SD_BLK_SIZE);
+ total = data->blocksize * data->blocks;
+
+ if (total < 512)
+ writel(total, priv->reg_base + OWL_REG_SD_BUF_SIZE);
+ else
+ writel(512, priv->reg_base + OWL_REG_SD_BUF_SIZE);
+
+ /* DMA STOP */
+ writel(0x0, SD_DMA_CHANNEL(priv->dma_channel, 0) + DMA_START);
+
+ if (data) {
+ if (data->flags == MMC_DATA_READ) {
+ buf = (ulong) (data->dest);
+ owl_dma_config(priv, (ulong) priv->reg_base +
+ OWL_REG_SD_DAT, buf, total);
+ invalidate_dcache_range(buf, buf + total);
+ } else {
+ buf = (ulong) (data->src);
+ owl_dma_config(priv, buf, (ulong) priv->reg_base +
+ OWL_REG_SD_DAT, total);
+ flush_dcache_range(buf, buf + total);
+ }
+ /* DMA START */
+ writel(0x1, SD_DMA_CHANNEL(priv->dma_channel, 0) + DMA_START);
+ }
+}
+
+static int owl_mmc_send_cmd(struct udevice *dev, struct mmc_cmd *cmd,
+ struct mmc_data *data)
+{
+ struct owl_mmc_priv *priv = dev_get_priv(dev);
+ unsigned int cmd_rsp_mask, mode, reg;
+ int ret;
+
+ setbits_le32(priv->reg_base + OWL_REG_SD_EN, OWL_SD_ENABLE);
+
+ /* setup response */
+ mode = 0;
+ if (cmd->resp_type != MMC_RSP_NONE)
+ cmd_rsp_mask = OWL_SD_STATE_CLNR | OWL_SD_STATE_CRC7ER;
+ if (cmd->resp_type == MMC_RSP_R1) {
+ if (data) {
+ if (data->flags == MMC_DATA_READ)
+ mode |= OWL_SD_CTL_TM(4);
+ else
+ mode |= OWL_SD_CTL_TM(5);
+ } else
+ mode |= OWL_SD_CTL_TM(1);
+ } else if (cmd->resp_type == MMC_RSP_R2) {
+ mode = OWL_SD_CTL_TM(2);
+ } else if (cmd->resp_type == MMC_RSP_R1b) {
+ mode = OWL_SD_CTL_TM(3);
+ } else if (cmd->resp_type == MMC_RSP_R3) {
+ cmd_rsp_mask = OWL_SD_STATE_CLNR;
+ mode = OWL_SD_CTL_TM(1);
+ }
+
+ mode |= (readl(priv->reg_base + OWL_REG_SD_CTL) & (0xff << 16));
+
+ /* setup command */
+ writel(cmd->cmdidx, priv->reg_base + OWL_REG_SD_CMD);
+ writel(cmd->cmdarg, priv->reg_base + OWL_REG_SD_ARG);
+
+ /* Set LBE to send clk at the end of last read block */
+ if (data)
+ mode |= (OWL_SD_CTL_TS | OWL_SD_CTL_LBE | 0xE4000000);
+ else
+ mode |= OWL_SD_CTL_TS;
+
+ if (data)
+ owl_mmc_prepare_data(priv, data);
+
+ /* Start transfer */
+ writel(mode, priv->reg_base + OWL_REG_SD_CTL);
+
+ ret = readl_poll_timeout(priv->reg_base + OWL_REG_SD_CTL, reg,
+ !(reg & OWL_SD_CTL_TS), DATA_TRANSFER_TIMEOUT);
+
+ if (ret == -ETIMEDOUT) {
+ debug("error: transferred data timeout\n");
+ return ret;
+ }
+
+ reg = readl(priv->reg_base + OWL_REG_SD_STATE) & cmd_rsp_mask;
+ if (cmd->resp_type & MMC_RSP_PRESENT) {
+ if (reg & OWL_SD_STATE_CLNR) {
+ printf("Error CMD_NO_RSP\n");
+ return -1;
+ }
+
+ if (reg & OWL_SD_STATE_CRC7ER) {
+ printf("Error CMD_RSP_CRC\n");
+ return -1;
+ }
+
+ if (cmd->resp_type & MMC_RSP_136) {
+ cmd->response[3] = readl(priv->reg_base + OWL_REG_SD_RSPBUF0);
+ cmd->response[2] = readl(priv->reg_base + OWL_REG_SD_RSPBUF1);
+ cmd->response[1] = readl(priv->reg_base + OWL_REG_SD_RSPBUF2);
+ cmd->response[0] = readl(priv->reg_base + OWL_REG_SD_RSPBUF3);
+ } else {
+ u32 rsp[2];
+
+ rsp[0] = readl(priv->reg_base + OWL_REG_SD_RSPBUF0);
+ rsp[1] = readl(priv->reg_base + OWL_REG_SD_RSPBUF1);
+ cmd->response[0] = rsp[1] << 24 | rsp[0] >> 8;
+ cmd->response[1] = rsp[1] >> 8;
+ }
+ }
+
+ if (data) {
+ ret = readl_poll_timeout(SD_DMA_CHANNEL(priv->dma_channel, 0) + DMA_START,
+ reg, !(reg & DMA_ENABLE), DMA_TRANSFER_TIMEOUT);
+
+ if (ret == -ETIMEDOUT) {
+ debug("error: DMA transfer timeout\n");
+ return ret;
+ }
+
+ /* DMA STOP */
+ writel(0x0, SD_DMA_CHANNEL(priv->dma_channel, 0) + DMA_START);
+ /* Transmission STOP */
+ while (readl(priv->reg_base + OWL_REG_SD_CTL) & OWL_SD_CTL_TS)
+ clrbits_le32(priv->reg_base + OWL_REG_SD_CTL,
+ OWL_SD_CTL_TS);
+ }
+
+ return 0;
+}
+
+static int owl_mmc_clk_set(struct owl_mmc_priv *priv, int rate)
+{
+ u32 reg, wdelay, rdelay;
+
+ reg = readl(priv->reg_base + OWL_REG_SD_CTL);
+ reg &= ~OWL_SD_CTL_DELAY_MSK;
+
+ /* Set RDELAY and WDELAY based on the clock */
+ if (rate <= 1000000)
+ rdelay = wdelay = OWL_SD_DELAY_LOW_CLK;
+ else if ((rate > 1000000) && (rate <= 26000000))
+ rdelay = wdelay = OWL_SD_DELAY_MID_CLK;
+ else if ((rate > 26000000) && (rate <= 52000000)) {
+ rdelay = OWL_SD_RDELAY_HIGH;
+ wdelay = OWL_SD_WDELAY_HIGH;
+ } else {
+ debug("SD clock rate not supported\n");
+ return -EINVAL;
+ }
+
+ writel(reg | OWL_SD_CTL_RDELAY(rdelay) | OWL_SD_CTL_WDELAY(wdelay),
+ priv->reg_base + OWL_REG_SD_CTL);
+
+ return 0;
+}
+
+static int owl_mmc_set_ios(struct udevice *dev)
+{
+ struct owl_mmc_priv *priv = dev_get_priv(dev);
+ struct owl_mmc_plat *plat = dev_get_plat(dev);
+ struct mmc *mmc = &plat->mmc;
+ u32 reg, ret;
+
+ if (mmc->clock != priv->clock) {
+ priv->clock = mmc->clock;
+ ret = owl_mmc_clk_set(priv, mmc->clock);
+ if (IS_ERR_VALUE(ret))
+ return ret;
+
+ ret = clk_set_rate(&priv->clk, mmc->clock);
+ if (IS_ERR_VALUE(ret))
+ return ret;
+ }
+
+ if (mmc->clk_disable)
+ ret = clk_disable(&priv->clk);
+ else
+ ret = clk_enable(&priv->clk);
+ if (ret)
+ return ret;
+
+ /* Set the Bus width */
+ reg = readl(priv->reg_base + OWL_REG_SD_EN);
+ reg &= ~OWL_SD_EN_DATAWID_MASK;
+ if (mmc->bus_width == 8)
+ reg |= OWL_SD_EN_DATAWID(2);
+ else if (mmc->bus_width == 4)
+ reg |= OWL_SD_EN_DATAWID(1);
+
+ writel(reg, priv->reg_base + OWL_REG_SD_EN);
+
+ return 0;
+}
+
+static const struct dm_mmc_ops owl_mmc_ops = {
+ .send_cmd = owl_mmc_send_cmd,
+ .set_ios = owl_mmc_set_ios,
+};
+
+static int owl_mmc_probe(struct udevice *dev)
+{
+ struct mmc_uclass_priv *upriv = dev_get_uclass_priv(dev);
+ struct owl_mmc_plat *plat = dev_get_plat(dev);
+ struct owl_mmc_priv *priv = dev_get_priv(dev);
+ struct mmc_config *cfg = &plat->cfg;
+ struct ofnode_phandle_args args;
+ int ret;
+ fdt_addr_t addr;
+
+ cfg->name = dev->name;
+ cfg->voltages = OWL_MMC_OCR;
+ cfg->f_min = 400000;
+ cfg->f_max = 52000000;
+ cfg->b_max = 512;
+ cfg->host_caps = MMC_MODE_HS | MMC_MODE_HS_52MHz;
+
+ ret = mmc_of_parse(dev, cfg);
+ if (ret)
+ return ret;
+
+ addr = dev_read_addr(dev);
+ if (addr == FDT_ADDR_T_NONE)
+ return -EINVAL;
+
+ priv->reg_base = (void *)addr;
+
+ ret = dev_read_phandle_with_args(dev, "dmas", "#dma-cells", 0, 0,
+ &args);
+ if (ret)
+ return ret;
+
+ priv->dma_channel = (void *)ofnode_get_addr(args.node);
+ priv->dma_drq = args.args[0];
+
+ ret = clk_get_by_index(dev, 0, &priv->clk);
+ if (ret) {
+ debug("clk_get_by_index() failed: %d\n", ret);
+ return ret;
+ }
+
+ upriv->mmc = &plat->mmc;
+
+ return 0;
+}
+
+static int owl_mmc_bind(struct udevice *dev)
+{
+ struct owl_mmc_plat *plat = dev_get_plat(dev);
+
+ return mmc_bind(dev, &plat->mmc, &plat->cfg);
+}
+
+static const struct udevice_id owl_mmc_ids[] = {
+ { .compatible = "actions,s700-mmc" },
+ { .compatible = "actions,owl-mmc" },
+ { }
+};
+
+U_BOOT_DRIVER(owl_mmc_drv) = {
+ .name = "owl_mmc",
+ .id = UCLASS_MMC,
+ .of_match = owl_mmc_ids,
+ .bind = owl_mmc_bind,
+ .probe = owl_mmc_probe,
+ .ops = &owl_mmc_ops,
+ .plat_auto = sizeof(struct owl_mmc_plat),
+ .priv_auto = sizeof(struct owl_mmc_priv),
+};
--
2.27.0
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH v4 6/6] configs: Enable mmc support
2021-11-28 11:32 [PATCH v4 0/6] Add MMC/SD support for S700/S900 Amit Singh Tomar
` (4 preceding siblings ...)
2021-11-28 11:32 ` [PATCH v4 5/6] mmc: actions: add MMC driver for Actions OWL S700/S900 Amit Singh Tomar
@ 2021-11-28 11:32 ` Amit Singh Tomar
2022-01-18 21:09 ` Tom Rini
5 siblings, 1 reply; 18+ messages in thread
From: Amit Singh Tomar @ 2021-11-28 11:32 UTC (permalink / raw)
To: u-boot
Cc: andre.przywara, manivannan.sadhasivam, joe.hershberger,
jh80.chung, Amit Singh Tomar
From: Amit Singh Tomar <amittomer25@gmail.com>
This commits enables mmc on the Actions Cubieboard7 board.
Signed-off-by: Amit Singh Tomar <amittomer25@gmail.com>
---
Changes since previous versions
* No change.
---
configs/cubieboard7_defconfig | 3 +++
1 file changed, 3 insertions(+)
diff --git a/configs/cubieboard7_defconfig b/configs/cubieboard7_defconfig
index 2937a54d1195..536b023a3eec 100644
--- a/configs/cubieboard7_defconfig
+++ b/configs/cubieboard7_defconfig
@@ -16,3 +16,6 @@ CONFIG_PHY_REALTEK=y
CONFIG_RTL8201F_PHY_S700_RMII_TIMINGS=y
CONFIG_ETH_DESIGNWARE=y
CONFIG_ETH_DESIGNWARE_S700=y
+CONFIG_DM_MMC=y
+CONFIG_MMC_OWL=y
+CONFIG_CMD_MMC=y
--
2.27.0
^ permalink raw reply related [flat|nested] 18+ messages in thread
* Re: [PATCH v4 3/6] ARM: dts: sync Actions Semi S700 DT from Linux 5.10-rc7
2021-11-28 11:32 ` [PATCH v4 3/6] ARM: dts: sync Actions Semi S700 DT from Linux 5.10-rc7 Amit Singh Tomar
@ 2021-11-29 16:13 ` Tom Rini
2021-12-02 18:23 ` [PATCH v4 3/6] ARM: dts: sync Actions Semi S700 DT from Linux v5.16-rc3 Amit Singh Tomar
1 sibling, 0 replies; 18+ messages in thread
From: Tom Rini @ 2021-11-29 16:13 UTC (permalink / raw)
To: Amit Singh Tomar
Cc: u-boot, andre.przywara, manivannan.sadhasivam, joe.hershberger,
jh80.chung, Amit Singh Tomar
[-- Attachment #1: Type: text/plain, Size: 765 bytes --]
On Sun, Nov 28, 2021 at 05:02:22PM +0530, Amit Singh Tomar wrote:
> From: Amit Singh Tomar <amittomer25@gmail.com>
>
> This Synchronizes the Actions Semi S700 SoC DT changes from
> commit "0477e9288185" ("Linux 5.10-rc7").
>
> Signed-off-by: Amit Singh Tomar <amittomer25@gmail.com>
> ---
> Changes since previous versions
> * No change.
> ---
> arch/arm/dts/s700.dtsi | 17 ++++++++++++++++-
> .../dt-bindings/power/owl-s700-powergate.h | 19 +++++++++++++++++++
> 2 files changed, 35 insertions(+), 1 deletion(-)
> create mode 100644 include/dt-bindings/power/owl-s700-powergate.h
That's a fairly old tag to sync to, can we please pick something newer
(either latest release or latest rc)? Thanks!
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH v4 3/6] ARM: dts: sync Actions Semi S700 DT from Linux v5.16-rc3
2021-11-28 11:32 ` [PATCH v4 3/6] ARM: dts: sync Actions Semi S700 DT from Linux 5.10-rc7 Amit Singh Tomar
2021-11-29 16:13 ` Tom Rini
@ 2021-12-02 18:23 ` Amit Singh Tomar
2022-01-18 21:09 ` Tom Rini
1 sibling, 1 reply; 18+ messages in thread
From: Amit Singh Tomar @ 2021-12-02 18:23 UTC (permalink / raw)
To: u-boot
Cc: andre.przywara, manivannan.sadhasivam, trini, joe.hershberger,
jh80.chung, Amit Singh Tomar
From: Amit Singh Tomar <amittomer25@gmail.com>
This Synchronizes the Actions Semi S700 SoC DT changes from
commit "g58e1100fdc59" ("Linux v5.16-rc3").
Signed-off-by: Amit Singh Tomar <amittomer25@gmail.com>
---
arch/arm/dts/s700.dtsi | 17 ++++++++++++++++-
.../dt-bindings/power/owl-s700-powergate.h | 19 +++++++++++++++++++
2 files changed, 35 insertions(+), 1 deletion(-)
create mode 100644 include/dt-bindings/power/owl-s700-powergate.h
diff --git a/arch/arm/dts/s700.dtsi b/arch/arm/dts/s700.dtsi
index 2006ad5424fa..2c78caebf515 100644
--- a/arch/arm/dts/s700.dtsi
+++ b/arch/arm/dts/s700.dtsi
@@ -5,6 +5,7 @@
#include <dt-bindings/clock/actions,s700-cmu.h>
#include <dt-bindings/interrupt-controller/arm-gic.h>
+#include <dt-bindings/power/owl-s700-powergate.h>
#include <dt-bindings/reset/actions,s700-reset.h>
/ {
@@ -231,7 +232,7 @@
pinctrl: pinctrl@e01b0000 {
compatible = "actions,s700-pinctrl";
- reg = <0x0 0xe01b0000 0x0 0x1000>;
+ reg = <0x0 0xe01b0000 0x0 0x100>;
clocks = <&cmu CLK_GPIO>;
gpio-controller;
gpio-ranges = <&pinctrl 0 0 136>;
@@ -244,5 +245,19 @@
<GIC_SPI 39 IRQ_TYPE_LEVEL_HIGH>,
<GIC_SPI 40 IRQ_TYPE_LEVEL_HIGH>;
};
+
+ dma: dma-controller@e0230000 {
+ compatible = "actions,s700-dma";
+ reg = <0x0 0xe0230000 0x0 0x1000>;
+ interrupts = <GIC_SPI 57 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 58 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 59 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 60 IRQ_TYPE_LEVEL_HIGH>;
+ #dma-cells = <1>;
+ dma-channels = <10>;
+ dma-requests = <44>;
+ clocks = <&cmu CLK_DMAC>;
+ power-domains = <&sps S700_PD_DMA>;
+ };
};
};
diff --git a/include/dt-bindings/power/owl-s700-powergate.h b/include/dt-bindings/power/owl-s700-powergate.h
new file mode 100644
index 000000000000..4cf1aefbf09c
--- /dev/null
+++ b/include/dt-bindings/power/owl-s700-powergate.h
@@ -0,0 +1,19 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+/*
+ * Actions Semi S700 SPS
+ *
+ * Copyright (c) 2017 Andreas Färber
+ */
+#ifndef DT_BINDINGS_POWER_OWL_S700_POWERGATE_H
+#define DT_BINDINGS_POWER_OWL_S700_POWERGATE_H
+
+#define S700_PD_VDE 0
+#define S700_PD_VCE_SI 1
+#define S700_PD_USB2_1 2
+#define S700_PD_HDE 3
+#define S700_PD_DMA 4
+#define S700_PD_DS 5
+#define S700_PD_USB3 6
+#define S700_PD_USB2_0 7
+
+#endif
--
2.27.0
^ permalink raw reply related [flat|nested] 18+ messages in thread
* Re: [PATCH v4 4/6] ARM: dts: s700: add MMC/SD controller node
2021-11-28 11:32 ` [PATCH v4 4/6] ARM: dts: s700: add MMC/SD controller node Amit Singh Tomar
@ 2022-01-05 17:43 ` Tom Rini
2022-01-18 21:09 ` Tom Rini
1 sibling, 0 replies; 18+ messages in thread
From: Tom Rini @ 2022-01-05 17:43 UTC (permalink / raw)
To: Amit Singh Tomar
Cc: u-boot, andre.przywara, manivannan.sadhasivam, joe.hershberger,
jh80.chung, Amit Singh Tomar
[-- Attachment #1: Type: text/plain, Size: 614 bytes --]
On Sun, Nov 28, 2021 at 05:02:23PM +0530, Amit Singh Tomar wrote:
> From: Amit Singh Tomar <amittomer25@gmail.com>
>
> This patch adds node for mmc/sd controller found on Action Semi OWL
> S700 SoC.
>
> Since, upstream Linux binding has not been merged for S700 MMC/SD
> controller, Changes are put in u-boot specific dtsi file.
>
> Signed-off-by: Amit Singh Tomar <amittomer25@gmail.com>
This leads to:
arch/arm/dts/s700-cubieboard7.dtb: ERROR (phandle_references):
/soc/mmc@e0210000 : Reference to non-existent node or label "dma"
Please rebase on top of current next, thanks.
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v4 1/6] clk: actions: Introduce dummy get/set_rate callbacks
2021-11-28 11:32 ` [PATCH v4 1/6] clk: actions: Introduce dummy get/set_rate callbacks Amit Singh Tomar
@ 2022-01-15 17:44 ` Sean Anderson
2022-01-18 21:09 ` Tom Rini
1 sibling, 0 replies; 18+ messages in thread
From: Sean Anderson @ 2022-01-15 17:44 UTC (permalink / raw)
To: Amit Singh Tomar, u-boot
Cc: andre.przywara, manivannan.sadhasivam, joe.hershberger,
jh80.chung, Amit Singh Tomar
On 11/28/21 6:32 AM, Amit Singh Tomar wrote:
> From: Amit Singh Tomar <amittomer25@gmail.com>
>
> This commit introduces get/set_rate callbacks, these are dummy at
> the moment, and can be used to get/set clock for various devices
> based on the clk id.
>
> Signed-off-by: Amit Singh Tomar <amittomer25@gmail.com>
> ---
> Changes since v3:
> * No changes.
> Changes since v2:
> * No changes.
> Changes since previous version:
> * Removed premature initialization to avoid
> compiler warnings.
> ---
> drivers/clk/owl/clk_owl.c | 26 ++++++++++++++++++++++++++
> 1 file changed, 26 insertions(+)
>
> diff --git a/drivers/clk/owl/clk_owl.c b/drivers/clk/owl/clk_owl.c
> index 96ab7fed1f37..f78e5fa3f08d 100644
> --- a/drivers/clk/owl/clk_owl.c
> +++ b/drivers/clk/owl/clk_owl.c
> @@ -128,6 +128,30 @@ int owl_clk_disable(struct clk *clk)
> return 0;
> }
>
> +static ulong owl_clk_get_rate(struct clk *clk)
> +{
> + ulong rate;
> +
> + switch (clk->id) {
> + default:
> + return -ENOENT;
-ENOSYS please
> + }
> +
> + return rate;
> +}
> +
> +static ulong owl_clk_set_rate(struct clk *clk, ulong rate)
> +{
> + ulong new_rate;
> +
> + switch (clk->id) {
> + default:
> + return -ENOENT;
ditto
> + }
> +
> + return new_rate;
> +}
> +
> static int owl_clk_probe(struct udevice *dev)
> {
> struct owl_clk_priv *priv = dev_get_priv(dev);
> @@ -145,6 +169,8 @@ static int owl_clk_probe(struct udevice *dev)
> static const struct clk_ops owl_clk_ops = {
> .enable = owl_clk_enable,
> .disable = owl_clk_disable,
> + .get_rate = owl_clk_get_rate,
> + .set_rate = owl_clk_set_rate,
> };
>
> static const struct udevice_id owl_clk_ids[] = {
>
With that fixed,
Reviewed-by: Sean Anderson <seanga2@gmail.com>
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v4 2/6] clk: actions: Add SD/MMC clocks
2021-11-28 11:32 ` [PATCH v4 2/6] clk: actions: Add SD/MMC clocks Amit Singh Tomar
@ 2022-01-15 17:44 ` Sean Anderson
2022-01-18 21:09 ` Tom Rini
1 sibling, 0 replies; 18+ messages in thread
From: Sean Anderson @ 2022-01-15 17:44 UTC (permalink / raw)
To: Amit Singh Tomar, u-boot
Cc: andre.przywara, manivannan.sadhasivam, joe.hershberger,
jh80.chung, Amit Singh Tomar
On 11/28/21 6:32 AM, Amit Singh Tomar wrote:
> From: Amit Singh Tomar <amittomer25@gmail.com>
>
> This commit adds SD/MMC clocks, and provides .set/get_rate callbacks
> for SD/MMC device present on Actions OWL S700 SoCs.
>
> Signed-off-by: Amit Singh Tomar <amittomer25@gmail.com>
> ---
> Changes since v3:
> * No change.
> Changes since v2:
> * Fixed the weird div assignment.
> * Moved the clock bit for SD from header file
> to driver file.
> * Removed "< 0" part while comparing unsigned.
> Changes since previous version:
> * Removed rate *= 2 as this just overclocks.
> * Separated the divide by 128 bit from divider value.
> * Provided the separate routine to get sd parent rate
> based on bit 9.
> * Removed unnecessary initialization.
> ---
> drivers/clk/owl/clk_owl.c | 73 +++++++++++++++++++++++++++++++++++++++
> 1 file changed, 73 insertions(+)
>
> diff --git a/drivers/clk/owl/clk_owl.c b/drivers/clk/owl/clk_owl.c
> index f78e5fa3f08d..678fdd5a4540 100644
> --- a/drivers/clk/owl/clk_owl.c
> +++ b/drivers/clk/owl/clk_owl.c
> @@ -20,6 +20,8 @@
> #include <linux/bitops.h>
> #include <linux/delay.h>
>
> +#define CMU_DEVCLKEN0_SD0 BIT(22)
Please define this in arch/arm/include/asm/arch-owl/regs_s[79]00.h like CMU_DEVCLKEN1_ETH.
> void owl_clk_init(struct owl_clk_priv *priv)
> {
> u32 bus_clk = 0, core_pll, dev_pll;
> @@ -92,6 +94,9 @@ int owl_clk_enable(struct clk *clk)
> setbits_le32(priv->base + CMU_DEVCLKEN1, CMU_DEVCLKEN1_ETH);
> setbits_le32(priv->base + CMU_ETHERNETPLL, 5);
> break;
> + case CLK_SD0:
> + setbits_le32(priv->base + CMU_DEVCLKEN0, CMU_DEVCLKEN0_SD0);
> + break;
> default:
> return -EINVAL;
> }
> @@ -121,6 +126,9 @@ int owl_clk_disable(struct clk *clk)
> case CLK_ETHERNET:
> clrbits_le32(priv->base + CMU_DEVCLKEN1, CMU_DEVCLKEN1_ETH);
> break;
> + case CLK_SD0:
> + clrbits_le32(priv->base + CMU_DEVCLKEN0, CMU_DEVCLKEN0_SD0);
> + break;
> default:
> return -EINVAL;
> }
> @@ -128,11 +136,72 @@ int owl_clk_disable(struct clk *clk)
> return 0;
> }
>
> +static ulong get_sd_parent_rate(struct owl_clk_priv *priv, u32 dev_index)
> +{
> + ulong rate;
> + u32 reg;
> +
> + reg = readl(priv->base + (CMU_SD0CLK + dev_index * 0x4));
> + /* Clock output of DEV/NAND_PLL
> + * Range: 48M ~ 756M
> + * Frequency= PLLCLK * 6
> + */
> + if (reg & 0x200)
Please add a define for this bit
> + rate = readl(priv->base + CMU_NANDPLL) & 0x7f;
ditto for this mask
> + else
> + rate = readl(priv->base + CMU_DEVPLL) & 0x7f;
> +
> + rate *= 6000000;
> +
> + return rate;
> +}
> +
> +static ulong owl_get_sd_clk_rate(struct owl_clk_priv *priv, int sd_index)
> +{
> + uint div, val;
> + ulong parent_rate = get_sd_parent_rate(priv, sd_index);
> +
> + val = readl(priv->base + (CMU_SD0CLK + sd_index * 0x4));
> + div = (val & 0x1f) + 1;
ditto
> +
> + return (parent_rate / div);
> +}
> +
> +static ulong owl_set_sd_clk_rate(struct owl_clk_priv *priv, ulong rate,
> + int sd_index)
> +{
> + uint div, val;
> + ulong parent_rate = get_sd_parent_rate(priv, sd_index);
> +
> + if (rate == 0)
> + return rate;
> +
> + div = (parent_rate / rate);
no parentheses needed here
> +
> + val = readl(priv->base + (CMU_SD0CLK + sd_index * 0x4));
> + /* Bits 4..0 is used to program div value and bit 8 to enable
> + * divide by 128 circuit
> + */
The comment is good, but please use defines. Additionally, we are not
in the net subsystem, so it should be formatted like
/*
* Bits 4..0 is used to program div value and bit 8 to enable
* divide by 128 circuit
*/
> + val &= ~0x11f;
> + if (div >= 128) {
> + div = div / 128;
> + val |= 0x100; /* enable divide by 128 circuit */
> + }
> + val |= ((div - 1) & 0x1f);
> + writel(val, priv->base + (CMU_SD0CLK + sd_index * 0x4));
> +
> + return owl_get_sd_clk_rate(priv, 0);
> +}
> +
> static ulong owl_clk_get_rate(struct clk *clk)
> {
> + struct owl_clk_priv *priv = dev_get_priv(clk->dev);
> ulong rate;
>
> switch (clk->id) {
> + case CLK_SD0:
> + rate = owl_get_sd_clk_rate(priv, 0);
> + break;
> default:
> return -ENOENT;
> }
> @@ -142,9 +211,13 @@ static ulong owl_clk_get_rate(struct clk *clk)
>
> static ulong owl_clk_set_rate(struct clk *clk, ulong rate)
> {
> + struct owl_clk_priv *priv = dev_get_priv(clk->dev);
> ulong new_rate;
>
> switch (clk->id) {
> + case CLK_SD0:
> + new_rate = owl_set_sd_clk_rate(priv, rate, 0);
> + break;
> default:
> return -ENOENT;
> }
>
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v4 1/6] clk: actions: Introduce dummy get/set_rate callbacks
2021-11-28 11:32 ` [PATCH v4 1/6] clk: actions: Introduce dummy get/set_rate callbacks Amit Singh Tomar
2022-01-15 17:44 ` Sean Anderson
@ 2022-01-18 21:09 ` Tom Rini
1 sibling, 0 replies; 18+ messages in thread
From: Tom Rini @ 2022-01-18 21:09 UTC (permalink / raw)
To: Amit Singh Tomar
Cc: u-boot, andre.przywara, manivannan.sadhasivam, joe.hershberger,
jh80.chung, Amit Singh Tomar
[-- Attachment #1: Type: text/plain, Size: 446 bytes --]
On Sun, Nov 28, 2021 at 05:02:20PM +0530, Amit Singh Tomar wrote:
> From: Amit Singh Tomar <amittomer25@gmail.com>
>
> This commit introduces get/set_rate callbacks, these are dummy at
> the moment, and can be used to get/set clock for various devices
> based on the clk id.
>
> Signed-off-by: Amit Singh Tomar <amittomer25@gmail.com>
> Reviewed-by: Sean Anderson <seanga2@gmail.com>
Applied to u-boot/master, thanks!
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v4 2/6] clk: actions: Add SD/MMC clocks
2021-11-28 11:32 ` [PATCH v4 2/6] clk: actions: Add SD/MMC clocks Amit Singh Tomar
2022-01-15 17:44 ` Sean Anderson
@ 2022-01-18 21:09 ` Tom Rini
1 sibling, 0 replies; 18+ messages in thread
From: Tom Rini @ 2022-01-18 21:09 UTC (permalink / raw)
To: Amit Singh Tomar
Cc: u-boot, andre.przywara, manivannan.sadhasivam, joe.hershberger,
jh80.chung, Amit Singh Tomar
[-- Attachment #1: Type: text/plain, Size: 362 bytes --]
On Sun, Nov 28, 2021 at 05:02:21PM +0530, Amit Singh Tomar wrote:
> From: Amit Singh Tomar <amittomer25@gmail.com>
>
> This commit adds SD/MMC clocks, and provides .set/get_rate callbacks
> for SD/MMC device present on Actions OWL S700 SoCs.
>
> Signed-off-by: Amit Singh Tomar <amittomer25@gmail.com>
Applied to u-boot/master, thanks!
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v4 3/6] ARM: dts: sync Actions Semi S700 DT from Linux v5.16-rc3
2021-12-02 18:23 ` [PATCH v4 3/6] ARM: dts: sync Actions Semi S700 DT from Linux v5.16-rc3 Amit Singh Tomar
@ 2022-01-18 21:09 ` Tom Rini
0 siblings, 0 replies; 18+ messages in thread
From: Tom Rini @ 2022-01-18 21:09 UTC (permalink / raw)
To: Amit Singh Tomar
Cc: u-boot, andre.przywara, manivannan.sadhasivam, joe.hershberger,
jh80.chung, Amit Singh Tomar
[-- Attachment #1: Type: text/plain, Size: 345 bytes --]
On Thu, Dec 02, 2021 at 11:53:28PM +0530, Amit Singh Tomar wrote:
> From: Amit Singh Tomar <amittomer25@gmail.com>
>
> This Synchronizes the Actions Semi S700 SoC DT changes from
> commit "g58e1100fdc59" ("Linux v5.16-rc3").
>
> Signed-off-by: Amit Singh Tomar <amittomer25@gmail.com>
Applied to u-boot/master, thanks!
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v4 4/6] ARM: dts: s700: add MMC/SD controller node
2021-11-28 11:32 ` [PATCH v4 4/6] ARM: dts: s700: add MMC/SD controller node Amit Singh Tomar
2022-01-05 17:43 ` Tom Rini
@ 2022-01-18 21:09 ` Tom Rini
1 sibling, 0 replies; 18+ messages in thread
From: Tom Rini @ 2022-01-18 21:09 UTC (permalink / raw)
To: Amit Singh Tomar
Cc: u-boot, andre.przywara, manivannan.sadhasivam, joe.hershberger,
jh80.chung, Amit Singh Tomar
[-- Attachment #1: Type: text/plain, Size: 453 bytes --]
On Sun, Nov 28, 2021 at 05:02:23PM +0530, Amit Singh Tomar wrote:
> From: Amit Singh Tomar <amittomer25@gmail.com>
>
> This patch adds node for mmc/sd controller found on Action Semi OWL
> S700 SoC.
>
> Since, upstream Linux binding has not been merged for S700 MMC/SD
> controller, Changes are put in u-boot specific dtsi file.
>
> Signed-off-by: Amit Singh Tomar <amittomer25@gmail.com>
Applied to u-boot/master, thanks!
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v4 5/6] mmc: actions: add MMC driver for Actions OWL S700/S900
2021-11-28 11:32 ` [PATCH v4 5/6] mmc: actions: add MMC driver for Actions OWL S700/S900 Amit Singh Tomar
@ 2022-01-18 21:09 ` Tom Rini
0 siblings, 0 replies; 18+ messages in thread
From: Tom Rini @ 2022-01-18 21:09 UTC (permalink / raw)
To: Amit Singh Tomar
Cc: u-boot, andre.przywara, manivannan.sadhasivam, joe.hershberger,
jh80.chung, Amit Singh Tomar
[-- Attachment #1: Type: text/plain, Size: 332 bytes --]
On Sun, Nov 28, 2021 at 05:02:24PM +0530, Amit Singh Tomar wrote:
> From: Amit Singh Tomar <amittomer25@gmail.com>
>
> This commit adds support for MMC controllers found on Actions OWL
> SoC platform(S700/S900).
>
> Signed-off-by: Amit Singh Tomar <amittomer25@gmail.com>
Applied to u-boot/master, thanks!
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v4 6/6] configs: Enable mmc support
2021-11-28 11:32 ` [PATCH v4 6/6] configs: Enable mmc support Amit Singh Tomar
@ 2022-01-18 21:09 ` Tom Rini
0 siblings, 0 replies; 18+ messages in thread
From: Tom Rini @ 2022-01-18 21:09 UTC (permalink / raw)
To: Amit Singh Tomar
Cc: u-boot, andre.przywara, manivannan.sadhasivam, joe.hershberger,
jh80.chung, Amit Singh Tomar
[-- Attachment #1: Type: text/plain, Size: 297 bytes --]
On Sun, Nov 28, 2021 at 05:02:25PM +0530, Amit Singh Tomar wrote:
> From: Amit Singh Tomar <amittomer25@gmail.com>
>
> This commits enables mmc on the Actions Cubieboard7 board.
>
> Signed-off-by: Amit Singh Tomar <amittomer25@gmail.com>
Applied to u-boot/master, thanks!
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread
end of thread, other threads:[~2022-01-18 21:10 UTC | newest]
Thread overview: 18+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-28 11:32 [PATCH v4 0/6] Add MMC/SD support for S700/S900 Amit Singh Tomar
2021-11-28 11:32 ` [PATCH v4 1/6] clk: actions: Introduce dummy get/set_rate callbacks Amit Singh Tomar
2022-01-15 17:44 ` Sean Anderson
2022-01-18 21:09 ` Tom Rini
2021-11-28 11:32 ` [PATCH v4 2/6] clk: actions: Add SD/MMC clocks Amit Singh Tomar
2022-01-15 17:44 ` Sean Anderson
2022-01-18 21:09 ` Tom Rini
2021-11-28 11:32 ` [PATCH v4 3/6] ARM: dts: sync Actions Semi S700 DT from Linux 5.10-rc7 Amit Singh Tomar
2021-11-29 16:13 ` Tom Rini
2021-12-02 18:23 ` [PATCH v4 3/6] ARM: dts: sync Actions Semi S700 DT from Linux v5.16-rc3 Amit Singh Tomar
2022-01-18 21:09 ` Tom Rini
2021-11-28 11:32 ` [PATCH v4 4/6] ARM: dts: s700: add MMC/SD controller node Amit Singh Tomar
2022-01-05 17:43 ` Tom Rini
2022-01-18 21:09 ` Tom Rini
2021-11-28 11:32 ` [PATCH v4 5/6] mmc: actions: add MMC driver for Actions OWL S700/S900 Amit Singh Tomar
2022-01-18 21:09 ` Tom Rini
2021-11-28 11:32 ` [PATCH v4 6/6] configs: Enable mmc support Amit Singh Tomar
2022-01-18 21:09 ` Tom Rini
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.