* [U-Boot] [RFC PATCHv1 0/3] misc: pl310: add a dm cache driver
@ 2019-03-05 19:03 Dinh Nguyen
2019-03-05 19:03 ` [U-Boot] [RFC PATCHv1 1/3] misc: pl310: add a misc driver for the pl310 cache controller Dinh Nguyen
` (2 more replies)
0 siblings, 3 replies; 13+ messages in thread
From: Dinh Nguyen @ 2019-03-05 19:03 UTC (permalink / raw)
To: u-boot
Hello,
I'm soliciting comments of this patchset. The patchset adds a driver
for the cache controller(specifically the PL310). This cache controller
can be found on many ARMv7 SoCs. I don't think it's used on ARMv8
platforms.
This driver will retrieve the cache properties from the board DTS files
and set the corresponding bits in cache controller.
I think we can do something similar without make this driver and placing
the file in /arch/arm/cpu/armv7, but wanted to get feedback.
To-dos:
- Add error checking
- Add more cache properties
Thanks,
Dinh Nguyen (3):
misc: pl310: add a misc driver for the pl310 cache controller
defconfig: socfpga_sockit_defconfig: enable L2X0_CACHE driver
ARM: socfpga: let the pl310 driver configure the cache settings
arch/arm/include/asm/pl310.h | 3 ++
arch/arm/mach-socfpga/misc.c | 15 +-----
configs/socfpga_sockit_defconfig | 2 +
drivers/misc/Kconfig | 7 +++
drivers/misc/Makefile | 1 +
drivers/misc/cache-l2x0.c | 84 ++++++++++++++++++++++++++++++++
6 files changed, 99 insertions(+), 13 deletions(-)
create mode 100644 drivers/misc/cache-l2x0.c
--
2.20.0
^ permalink raw reply [flat|nested] 13+ messages in thread
* [U-Boot] [RFC PATCHv1 1/3] misc: pl310: add a misc driver for the pl310 cache controller
2019-03-05 19:03 [U-Boot] [RFC PATCHv1 0/3] misc: pl310: add a dm cache driver Dinh Nguyen
@ 2019-03-05 19:03 ` Dinh Nguyen
2019-03-05 19:19 ` Marek Vasut
2019-03-05 19:03 ` [U-Boot] [RFC PATCHv1 2/3] defconfig: socfpga_sockit_defconfig: enable L2X0_CACHE driver Dinh Nguyen
2019-03-05 19:03 ` [U-Boot] [RFC PATCHv1 3/3] ARM: socfpga: let the pl310 driver configure the cache settings Dinh Nguyen
2 siblings, 1 reply; 13+ messages in thread
From: Dinh Nguyen @ 2019-03-05 19:03 UTC (permalink / raw)
To: u-boot
The driver will read the cache properties from the device tree file and
set it up.
Signed-off-by: Dinh Nguyen <dinguyen@kernel.org>
---
arch/arm/include/asm/pl310.h | 3 ++
drivers/misc/Kconfig | 7 +++
drivers/misc/Makefile | 1 +
drivers/misc/cache-l2x0.c | 84 ++++++++++++++++++++++++++++++++++++
4 files changed, 95 insertions(+)
create mode 100644 drivers/misc/cache-l2x0.c
diff --git a/arch/arm/include/asm/pl310.h b/arch/arm/include/asm/pl310.h
index b83978b1cc..f69e9e45f8 100644
--- a/arch/arm/include/asm/pl310.h
+++ b/arch/arm/include/asm/pl310.h
@@ -18,6 +18,9 @@
#define L310_SHARED_ATT_OVERRIDE_ENABLE (1 << 22)
#define L310_AUX_CTRL_DATA_PREFETCH_MASK (1 << 28)
#define L310_AUX_CTRL_INST_PREFETCH_MASK (1 << 29)
+#define L310_LATENCY_CTRL_SETUP(n) ((n) << 0)
+#define L310_LATENCY_CTRL_RD(n) ((n) << 4)
+#define L310_LATENCY_CTRL_WR(n) ((n) << 8)
#define L2X0_CACHE_ID_PART_MASK (0xf << 6)
#define L2X0_CACHE_ID_PART_L310 (3 << 6)
diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig
index d6e677fba8..c5c34b4dbb 100644
--- a/drivers/misc/Kconfig
+++ b/drivers/misc/Kconfig
@@ -13,6 +13,13 @@ config MISC
set of generic read, write and ioctl methods may be used to
access the device.
+config L2X0_CACHE
+ bool "L2x0 Cache support"
+ depends on MISC
+ help
+ Select this to enable a PL310 L2 Cache driver. The driver will
+ configure the L2 Cache settings found in the device tree.
+
config ALTERA_SYSID
bool "Altera Sysid support"
depends on MISC
diff --git a/drivers/misc/Makefile b/drivers/misc/Makefile
index 6bdf5054f4..ea726f4668 100644
--- a/drivers/misc/Makefile
+++ b/drivers/misc/Makefile
@@ -23,6 +23,7 @@ obj-$(CONFIG_SANDBOX) += spltest_sandbox.o
endif
endif
obj-$(CONFIG_ALI152X) += ali512x.o
+obj-$(CONFIG_L2X0_CACHE) += cache-l2x0.o
obj-$(CONFIG_ALTERA_SYSID) += altera_sysid.o
obj-$(CONFIG_ATSHA204A) += atsha204a-i2c.o
obj-$(CONFIG_CBMEM_CONSOLE) += cbmem_console.o
diff --git a/drivers/misc/cache-l2x0.c b/drivers/misc/cache-l2x0.c
new file mode 100644
index 0000000000..b31598b1cd
--- /dev/null
+++ b/drivers/misc/cache-l2x0.c
@@ -0,0 +1,84 @@
+// SPDX-License-Identifier: GPL-2.0
+/*
+ * Copyright (C) 2007 ARM Limited
+ *
+ * copied from Linux(arch/arm/mm/cache-l2x0.c
+ */
+#include <common.h>
+#include <command.h>
+#include <dm.h>
+
+#include <asm/io.h>
+#include <asm/pl310.h>
+
+static void l2c310_of_parse(struct udevice *dev)
+{
+ u32 tag[3] = { 0, 0, 0 };
+ u32 saved_reg, prefetch_i, prefetch_d;
+ bool shared_override;
+ struct pl310_regs *regs = (struct pl310_regs *)devfdt_get_addr(dev);
+
+ /*Disable the L2 Cache */
+ clrbits_le32(®s->pl310_ctrl, L2X0_CTRL_EN);
+
+ dev_read_u32(dev, "prefetch-data", &prefetch_d);
+ dev_read_u32(dev, "prefetch-instr", &prefetch_i);
+ shared_override = dev_read_bool(dev, "arm,shared-override");
+
+ saved_reg = readl(®s->pl310_aux_ctrl);
+ if (prefetch_d)
+ saved_reg |= L310_AUX_CTRL_DATA_PREFETCH_MASK;
+ if (prefetch_i)
+ saved_reg |= L310_AUX_CTRL_INST_PREFETCH_MASK;
+ if (shared_override)
+ saved_reg |= L310_SHARED_ATT_OVERRIDE_ENABLE;
+
+ writel(saved_reg, ®s->pl310_aux_ctrl);
+
+ saved_reg = readl(®s->pl310_tag_latency_ctrl);
+ dev_read_u32_array(dev, "arm,tag-latency", tag, 3);
+
+ saved_reg |= L310_LATENCY_CTRL_RD(tag[0] - 1) |
+ L310_LATENCY_CTRL_WR(tag[1] - 1) |
+ L310_LATENCY_CTRL_SETUP(tag[2] - 1);
+
+ writel(saved_reg, ®s->pl310_tag_latency_ctrl);
+
+ saved_reg = readl(®s->pl310_data_latency_ctrl);
+ dev_read_u32_array(dev, "arm,data-latency", tag, 3);
+
+ saved_reg |= L310_LATENCY_CTRL_RD(tag[0] - 1) |
+ L310_LATENCY_CTRL_WR(tag[1] - 1) |
+ L310_LATENCY_CTRL_SETUP(tag[2] - 1);
+
+ writel(saved_reg, ®s->pl310_data_latency_ctrl);
+
+ /* Enable the L2 cache */
+ setbits_le32(®s->pl310_ctrl, L2X0_CTRL_EN);
+}
+
+static int l2x0_ofdata_to_platdata(struct udevice *dev)
+{
+ return 0;
+}
+
+static int l2x0_probe(struct udevice *dev)
+{
+ l2c310_of_parse(dev);
+ return 0;
+}
+
+
+static const struct udevice_id l2x0_ids[] = {
+ { .compatible = "arm,pl310-cache" },
+ {}
+};
+
+U_BOOT_DRIVER(pl310_cache) = {
+ .name = "pl310_cache",
+ .id = UCLASS_MISC,
+ .of_match = l2x0_ids,
+ .probe = l2x0_probe,
+ .ofdata_to_platdata = l2x0_ofdata_to_platdata,
+ .flags = DM_FLAG_PRE_RELOC,
+};
--
2.20.0
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [U-Boot] [RFC PATCHv1 2/3] defconfig: socfpga_sockit_defconfig: enable L2X0_CACHE driver
2019-03-05 19:03 [U-Boot] [RFC PATCHv1 0/3] misc: pl310: add a dm cache driver Dinh Nguyen
2019-03-05 19:03 ` [U-Boot] [RFC PATCHv1 1/3] misc: pl310: add a misc driver for the pl310 cache controller Dinh Nguyen
@ 2019-03-05 19:03 ` Dinh Nguyen
2019-03-05 19:20 ` Marek Vasut
2019-03-05 19:03 ` [U-Boot] [RFC PATCHv1 3/3] ARM: socfpga: let the pl310 driver configure the cache settings Dinh Nguyen
2 siblings, 1 reply; 13+ messages in thread
From: Dinh Nguyen @ 2019-03-05 19:03 UTC (permalink / raw)
To: u-boot
Enable CONFIG_MISC and CONFIG_L2X0_CACHE config options.
Signed-off-by: Dinh Nguyen <dinguyen@kernel.org>
---
configs/socfpga_sockit_defconfig | 2 ++
1 file changed, 2 insertions(+)
diff --git a/configs/socfpga_sockit_defconfig b/configs/socfpga_sockit_defconfig
index 4c17d1a9e4..0009b0ebc3 100644
--- a/configs/socfpga_sockit_defconfig
+++ b/configs/socfpga_sockit_defconfig
@@ -47,6 +47,8 @@ CONFIG_DM_GPIO=y
CONFIG_DWAPB_GPIO=y
CONFIG_DM_I2C=y
CONFIG_SYS_I2C_DW=y
+CONFIG_MISC=y
+CONFIG_L2X0_CACHE=y
CONFIG_DM_MMC=y
CONFIG_MMC_DW=y
CONFIG_MTD_DEVICE=y
--
2.20.0
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [U-Boot] [RFC PATCHv1 3/3] ARM: socfpga: let the pl310 driver configure the cache settings
2019-03-05 19:03 [U-Boot] [RFC PATCHv1 0/3] misc: pl310: add a dm cache driver Dinh Nguyen
2019-03-05 19:03 ` [U-Boot] [RFC PATCHv1 1/3] misc: pl310: add a misc driver for the pl310 cache controller Dinh Nguyen
2019-03-05 19:03 ` [U-Boot] [RFC PATCHv1 2/3] defconfig: socfpga_sockit_defconfig: enable L2X0_CACHE driver Dinh Nguyen
@ 2019-03-05 19:03 ` Dinh Nguyen
2019-03-05 19:20 ` Marek Vasut
2 siblings, 1 reply; 13+ messages in thread
From: Dinh Nguyen @ 2019-03-05 19:03 UTC (permalink / raw)
To: u-boot
Load the PL310 L2 cache driver and allow it to setup the cache settings
Signed-off-by: Dinh Nguyen <dinguyen@kernel.org>
---
arch/arm/mach-socfpga/misc.c | 15 ++-------------
1 file changed, 2 insertions(+), 13 deletions(-)
diff --git a/arch/arm/mach-socfpga/misc.c b/arch/arm/mach-socfpga/misc.c
index fcf211d62b..fb0cfd3c1a 100644
--- a/arch/arm/mach-socfpga/misc.c
+++ b/arch/arm/mach-socfpga/misc.c
@@ -59,20 +59,9 @@ void enable_caches(void)
#ifdef CONFIG_SYS_L2_PL310
void v7_outer_cache_enable(void)
{
- /* Disable the L2 cache */
- clrbits_le32(&pl310->pl310_ctrl, L2X0_CTRL_EN);
-
- writel(0x111, &pl310->pl310_tag_latency_ctrl);
- writel(0x121, &pl310->pl310_data_latency_ctrl);
-
- /* enable BRESP, instruction and data prefetch, full line of zeroes */
- setbits_le32(&pl310->pl310_aux_ctrl,
- L310_AUX_CTRL_DATA_PREFETCH_MASK |
- L310_AUX_CTRL_INST_PREFETCH_MASK |
- L310_SHARED_ATT_OVERRIDE_ENABLE);
+ struct udevice *dev;
- /* Enable the L2 cache */
- setbits_le32(&pl310->pl310_ctrl, L2X0_CTRL_EN);
+ uclass_first_device(UCLASS_MISC, &dev);
}
void v7_outer_cache_disable(void)
--
2.20.0
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [U-Boot] [RFC PATCHv1 1/3] misc: pl310: add a misc driver for the pl310 cache controller
2019-03-05 19:03 ` [U-Boot] [RFC PATCHv1 1/3] misc: pl310: add a misc driver for the pl310 cache controller Dinh Nguyen
@ 2019-03-05 19:19 ` Marek Vasut
2019-03-05 20:07 ` Dinh Nguyen
0 siblings, 1 reply; 13+ messages in thread
From: Marek Vasut @ 2019-03-05 19:19 UTC (permalink / raw)
To: u-boot
On 3/5/19 8:03 PM, Dinh Nguyen wrote:
> The driver will read the cache properties from the device tree file and
> set it up.
>
> Signed-off-by: Dinh Nguyen <dinguyen@kernel.org>
> ---
> arch/arm/include/asm/pl310.h | 3 ++
> drivers/misc/Kconfig | 7 +++
> drivers/misc/Makefile | 1 +
> drivers/misc/cache-l2x0.c | 84 ++++++++++++++++++++++++++++++++++++
> 4 files changed, 95 insertions(+)
> create mode 100644 drivers/misc/cache-l2x0.c
>
> diff --git a/arch/arm/include/asm/pl310.h b/arch/arm/include/asm/pl310.h
> index b83978b1cc..f69e9e45f8 100644
> --- a/arch/arm/include/asm/pl310.h
> +++ b/arch/arm/include/asm/pl310.h
> @@ -18,6 +18,9 @@
> #define L310_SHARED_ATT_OVERRIDE_ENABLE (1 << 22)
> #define L310_AUX_CTRL_DATA_PREFETCH_MASK (1 << 28)
> #define L310_AUX_CTRL_INST_PREFETCH_MASK (1 << 29)
> +#define L310_LATENCY_CTRL_SETUP(n) ((n) << 0)
> +#define L310_LATENCY_CTRL_RD(n) ((n) << 4)
> +#define L310_LATENCY_CTRL_WR(n) ((n) << 8)
>
> #define L2X0_CACHE_ID_PART_MASK (0xf << 6)
> #define L2X0_CACHE_ID_PART_L310 (3 << 6)
> diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig
> index d6e677fba8..c5c34b4dbb 100644
> --- a/drivers/misc/Kconfig
> +++ b/drivers/misc/Kconfig
> @@ -13,6 +13,13 @@ config MISC
> set of generic read, write and ioctl methods may be used to
> access the device.
>
> +config L2X0_CACHE
> + bool "L2x0 Cache support"
> + depends on MISC
> + help
> + Select this to enable a PL310 L2 Cache driver. The driver will
> + configure the L2 Cache settings found in the device tree.
I wonder whether we don't need some drivers/plat or drivers/soc for this ?
> config ALTERA_SYSID
> bool "Altera Sysid support"
> depends on MISC
> diff --git a/drivers/misc/Makefile b/drivers/misc/Makefile
> index 6bdf5054f4..ea726f4668 100644
> --- a/drivers/misc/Makefile
> +++ b/drivers/misc/Makefile
> @@ -23,6 +23,7 @@ obj-$(CONFIG_SANDBOX) += spltest_sandbox.o
> endif
> endif
> obj-$(CONFIG_ALI152X) += ali512x.o
> +obj-$(CONFIG_L2X0_CACHE) += cache-l2x0.o
> obj-$(CONFIG_ALTERA_SYSID) += altera_sysid.o
> obj-$(CONFIG_ATSHA204A) += atsha204a-i2c.o
> obj-$(CONFIG_CBMEM_CONSOLE) += cbmem_console.o
> diff --git a/drivers/misc/cache-l2x0.c b/drivers/misc/cache-l2x0.c
> new file mode 100644
> index 0000000000..b31598b1cd
> --- /dev/null
> +++ b/drivers/misc/cache-l2x0.c
> @@ -0,0 +1,84 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/*
> + * Copyright (C) 2007 ARM Limited
2007 ? :)
> + * copied from Linux(arch/arm/mm/cache-l2x0.c
> + */
> +#include <common.h>
> +#include <command.h>
> +#include <dm.h>
> +
> +#include <asm/io.h>
> +#include <asm/pl310.h>
> +
> +static void l2c310_of_parse(struct udevice *dev)
> +{
> + u32 tag[3] = { 0, 0, 0 };
> + u32 saved_reg, prefetch_i, prefetch_d;
> + bool shared_override;
> + struct pl310_regs *regs = (struct pl310_regs *)devfdt_get_addr(dev);
> +
> + /*Disable the L2 Cache */
/*<SPACE>D
> + clrbits_le32(®s->pl310_ctrl, L2X0_CTRL_EN);
> +
> + dev_read_u32(dev, "prefetch-data", &prefetch_d);
> + dev_read_u32(dev, "prefetch-instr", &prefetch_i);
> + shared_override = dev_read_bool(dev, "arm,shared-override");
> +
> + saved_reg = readl(®s->pl310_aux_ctrl);
> + if (prefetch_d)
> + saved_reg |= L310_AUX_CTRL_DATA_PREFETCH_MASK;
> + if (prefetch_i)
> + saved_reg |= L310_AUX_CTRL_INST_PREFETCH_MASK;
> + if (shared_override)
> + saved_reg |= L310_SHARED_ATT_OVERRIDE_ENABLE;
> +
> + writel(saved_reg, ®s->pl310_aux_ctrl);
> +
> + saved_reg = readl(®s->pl310_tag_latency_ctrl);
> + dev_read_u32_array(dev, "arm,tag-latency", tag, 3);
> +
> + saved_reg |= L310_LATENCY_CTRL_RD(tag[0] - 1) |
> + L310_LATENCY_CTRL_WR(tag[1] - 1) |
> + L310_LATENCY_CTRL_SETUP(tag[2] - 1);
> +
> + writel(saved_reg, ®s->pl310_tag_latency_ctrl);
> +
> + saved_reg = readl(®s->pl310_data_latency_ctrl);
> + dev_read_u32_array(dev, "arm,data-latency", tag, 3);
What happens if the array isn't present ?
Should we _not_ configure the latencies in such case ?
> + saved_reg |= L310_LATENCY_CTRL_RD(tag[0] - 1) |
> + L310_LATENCY_CTRL_WR(tag[1] - 1) |
> + L310_LATENCY_CTRL_SETUP(tag[2] - 1);
> +
> + writel(saved_reg, ®s->pl310_data_latency_ctrl);
> +
> + /* Enable the L2 cache */
> + setbits_le32(®s->pl310_ctrl, L2X0_CTRL_EN);
> +}
> +
> +static int l2x0_ofdata_to_platdata(struct udevice *dev)
> +{
> + return 0;
> +}
> +
> +static int l2x0_probe(struct udevice *dev)
> +{
> + l2c310_of_parse(dev);
> + return 0;
> +}
> +
> +
Two newlines here ^ drop one
> +static const struct udevice_id l2x0_ids[] = {
> + { .compatible = "arm,pl310-cache" },
> + {}
> +};
> +
> +U_BOOT_DRIVER(pl310_cache) = {
> + .name = "pl310_cache",
> + .id = UCLASS_MISC,
> + .of_match = l2x0_ids,
> + .probe = l2x0_probe,
> + .ofdata_to_platdata = l2x0_ofdata_to_platdata,
> + .flags = DM_FLAG_PRE_RELOC,
> +};
>
--
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 13+ messages in thread
* [U-Boot] [RFC PATCHv1 2/3] defconfig: socfpga_sockit_defconfig: enable L2X0_CACHE driver
2019-03-05 19:03 ` [U-Boot] [RFC PATCHv1 2/3] defconfig: socfpga_sockit_defconfig: enable L2X0_CACHE driver Dinh Nguyen
@ 2019-03-05 19:20 ` Marek Vasut
2019-03-05 20:17 ` Dinh Nguyen
0 siblings, 1 reply; 13+ messages in thread
From: Marek Vasut @ 2019-03-05 19:20 UTC (permalink / raw)
To: u-boot
On 3/5/19 8:03 PM, Dinh Nguyen wrote:
> Enable CONFIG_MISC and CONFIG_L2X0_CACHE config options.
>
> Signed-off-by: Dinh Nguyen <dinguyen@kernel.org>
> ---
> configs/socfpga_sockit_defconfig | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/configs/socfpga_sockit_defconfig b/configs/socfpga_sockit_defconfig
> index 4c17d1a9e4..0009b0ebc3 100644
> --- a/configs/socfpga_sockit_defconfig
> +++ b/configs/socfpga_sockit_defconfig
> @@ -47,6 +47,8 @@ CONFIG_DM_GPIO=y
> CONFIG_DWAPB_GPIO=y
> CONFIG_DM_I2C=y
> CONFIG_SYS_I2C_DW=y
> +CONFIG_MISC=y
> +CONFIG_L2X0_CACHE=y
> CONFIG_DM_MMC=y
> CONFIG_MMC_DW=y
> CONFIG_MTD_DEVICE=y
I think you can just add it as a default into arch/arm/socfpga/Kconfig
with 'imply CONFIG_...' instead.
--
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 13+ messages in thread
* [U-Boot] [RFC PATCHv1 3/3] ARM: socfpga: let the pl310 driver configure the cache settings
2019-03-05 19:03 ` [U-Boot] [RFC PATCHv1 3/3] ARM: socfpga: let the pl310 driver configure the cache settings Dinh Nguyen
@ 2019-03-05 19:20 ` Marek Vasut
2019-03-05 20:13 ` Dinh Nguyen
0 siblings, 1 reply; 13+ messages in thread
From: Marek Vasut @ 2019-03-05 19:20 UTC (permalink / raw)
To: u-boot
On 3/5/19 8:03 PM, Dinh Nguyen wrote:
> Load the PL310 L2 cache driver and allow it to setup the cache settings
>
> Signed-off-by: Dinh Nguyen <dinguyen@kernel.org>
> ---
> arch/arm/mach-socfpga/misc.c | 15 ++-------------
> 1 file changed, 2 insertions(+), 13 deletions(-)
>
> diff --git a/arch/arm/mach-socfpga/misc.c b/arch/arm/mach-socfpga/misc.c
> index fcf211d62b..fb0cfd3c1a 100644
> --- a/arch/arm/mach-socfpga/misc.c
> +++ b/arch/arm/mach-socfpga/misc.c
> @@ -59,20 +59,9 @@ void enable_caches(void)
> #ifdef CONFIG_SYS_L2_PL310
> void v7_outer_cache_enable(void)
> {
> - /* Disable the L2 cache */
> - clrbits_le32(&pl310->pl310_ctrl, L2X0_CTRL_EN);
> -
> - writel(0x111, &pl310->pl310_tag_latency_ctrl);
> - writel(0x121, &pl310->pl310_data_latency_ctrl);
> -
> - /* enable BRESP, instruction and data prefetch, full line of zeroes */
> - setbits_le32(&pl310->pl310_aux_ctrl,
> - L310_AUX_CTRL_DATA_PREFETCH_MASK |
> - L310_AUX_CTRL_INST_PREFETCH_MASK |
> - L310_SHARED_ATT_OVERRIDE_ENABLE);
> + struct udevice *dev;
>
> - /* Enable the L2 cache */
> - setbits_le32(&pl310->pl310_ctrl, L2X0_CTRL_EN);
> + uclass_first_device(UCLASS_MISC, &dev);
Error handling might help here
> }
>
> void v7_outer_cache_disable(void)
>
--
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 13+ messages in thread
* [U-Boot] [RFC PATCHv1 1/3] misc: pl310: add a misc driver for the pl310 cache controller
2019-03-05 19:19 ` Marek Vasut
@ 2019-03-05 20:07 ` Dinh Nguyen
2019-03-05 20:57 ` Marek Vasut
0 siblings, 1 reply; 13+ messages in thread
From: Dinh Nguyen @ 2019-03-05 20:07 UTC (permalink / raw)
To: u-boot
On 3/5/19 1:19 PM, Marek Vasut wrote:
> On 3/5/19 8:03 PM, Dinh Nguyen wrote:
>> The driver will read the cache properties from the device tree file and
>> set it up.
>>
>>
>> +config L2X0_CACHE
>> + bool "L2x0 Cache support"
>> + depends on MISC
>> + help
>> + Select this to enable a PL310 L2 Cache driver. The driver will
>> + configure the L2 Cache settings found in the device tree.
>
> I wonder whether we don't need some drivers/plat or drivers/soc for this ?
You mean for plat specific implementations?
>
>> config ALTERA_SYSID
>> bool "Altera Sysid support"
>> depends on MISC
>> diff --git a/drivers/misc/Makefile b/drivers/misc/Makefile
>> index 6bdf5054f4..ea726f4668 100644
>> --- a/drivers/misc/Makefile
>> +++ b/drivers/misc/Makefile
>> @@ -23,6 +23,7 @@ obj-$(CONFIG_SANDBOX) += spltest_sandbox.o
>> endif
>> endif
>> obj-$(CONFIG_ALI152X) += ali512x.o
>> +obj-$(CONFIG_L2X0_CACHE) += cache-l2x0.o
>> obj-$(CONFIG_ALTERA_SYSID) += altera_sysid.o
>> obj-$(CONFIG_ATSHA204A) += atsha204a-i2c.o
>> obj-$(CONFIG_CBMEM_CONSOLE) += cbmem_console.o
>> diff --git a/drivers/misc/cache-l2x0.c b/drivers/misc/cache-l2x0.c
>> new file mode 100644
>> index 0000000000..b31598b1cd
>> --- /dev/null
>> +++ b/drivers/misc/cache-l2x0.c
>> @@ -0,0 +1,84 @@
>> +// SPDX-License-Identifier: GPL-2.0
>> +/*
>> + * Copyright (C) 2007 ARM Limited
>
> 2007 ? :)
I think I can remove this. I started with the Linux's cache-l2x0, but
removed almost all of the code. Just a left-over artifact.
>> +
>> + saved_reg |= L310_LATENCY_CTRL_RD(tag[0] - 1) |
>> + L310_LATENCY_CTRL_WR(tag[1] - 1) |
>> + L310_LATENCY_CTRL_SETUP(tag[2] - 1);
>> +
>> + writel(saved_reg, ®s->pl310_tag_latency_ctrl);
>> +
>> + saved_reg = readl(®s->pl310_data_latency_ctrl);
>> + dev_read_u32_array(dev, "arm,data-latency", tag, 3);
>
> What happens if the array isn't present ?
> Should we _not_ configure the latencies in such case ?
Right, I have a to-do list that need to handle error conditions like this.
^ permalink raw reply [flat|nested] 13+ messages in thread
* [U-Boot] [RFC PATCHv1 3/3] ARM: socfpga: let the pl310 driver configure the cache settings
2019-03-05 19:20 ` Marek Vasut
@ 2019-03-05 20:13 ` Dinh Nguyen
2019-03-05 20:57 ` Marek Vasut
0 siblings, 1 reply; 13+ messages in thread
From: Dinh Nguyen @ 2019-03-05 20:13 UTC (permalink / raw)
To: u-boot
On 3/5/19 1:20 PM, Marek Vasut wrote:
> On 3/5/19 8:03 PM, Dinh Nguyen wrote:
>> Load the PL310 L2 cache driver and allow it to setup the cache settings
>>
>> Signed-off-by: Dinh Nguyen <dinguyen@kernel.org>
>> ---
>> arch/arm/mach-socfpga/misc.c | 15 ++-------------
>> 1 file changed, 2 insertions(+), 13 deletions(-)
>>
>> diff --git a/arch/arm/mach-socfpga/misc.c b/arch/arm/mach-socfpga/misc.c
>> index fcf211d62b..fb0cfd3c1a 100644
>> --- a/arch/arm/mach-socfpga/misc.c
>> +++ b/arch/arm/mach-socfpga/misc.c
>> @@ -59,20 +59,9 @@ void enable_caches(void)
>> #ifdef CONFIG_SYS_L2_PL310
>> void v7_outer_cache_enable(void)
>> {
>> - /* Disable the L2 cache */
>> - clrbits_le32(&pl310->pl310_ctrl, L2X0_CTRL_EN);
>> -
>> - writel(0x111, &pl310->pl310_tag_latency_ctrl);
>> - writel(0x121, &pl310->pl310_data_latency_ctrl);
>> -
>> - /* enable BRESP, instruction and data prefetch, full line of zeroes */
>> - setbits_le32(&pl310->pl310_aux_ctrl,
>> - L310_AUX_CTRL_DATA_PREFETCH_MASK |
>> - L310_AUX_CTRL_INST_PREFETCH_MASK |
>> - L310_SHARED_ATT_OVERRIDE_ENABLE);
>> + struct udevice *dev;
>>
>> - /* Enable the L2 cache */
>> - setbits_le32(&pl310->pl310_ctrl, L2X0_CTRL_EN);
>> + uclass_first_device(UCLASS_MISC, &dev);
>
> Error handling might help here
>
Agreed..Just trying to solicit comments on whether this is the right
approach? Will add error handling..
Thanks for reviewing...
Dinh
^ permalink raw reply [flat|nested] 13+ messages in thread
* [U-Boot] [RFC PATCHv1 2/3] defconfig: socfpga_sockit_defconfig: enable L2X0_CACHE driver
2019-03-05 19:20 ` Marek Vasut
@ 2019-03-05 20:17 ` Dinh Nguyen
2019-03-05 20:57 ` Marek Vasut
0 siblings, 1 reply; 13+ messages in thread
From: Dinh Nguyen @ 2019-03-05 20:17 UTC (permalink / raw)
To: u-boot
On 3/5/19 1:20 PM, Marek Vasut wrote:
> On 3/5/19 8:03 PM, Dinh Nguyen wrote:
>> Enable CONFIG_MISC and CONFIG_L2X0_CACHE config options.
>>
>> Signed-off-by: Dinh Nguyen <dinguyen@kernel.org>
>> ---
>> configs/socfpga_sockit_defconfig | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/configs/socfpga_sockit_defconfig b/configs/socfpga_sockit_defconfig
>> index 4c17d1a9e4..0009b0ebc3 100644
>> --- a/configs/socfpga_sockit_defconfig
>> +++ b/configs/socfpga_sockit_defconfig
>> @@ -47,6 +47,8 @@ CONFIG_DM_GPIO=y
>> CONFIG_DWAPB_GPIO=y
>> CONFIG_DM_I2C=y
>> CONFIG_SYS_I2C_DW=y
>> +CONFIG_MISC=y
>> +CONFIG_L2X0_CACHE=y
>> CONFIG_DM_MMC=y
>> CONFIG_MMC_DW=y
>> CONFIG_MTD_DEVICE=y
>
> I think you can just add it as a default into arch/arm/socfpga/Kconfig
> with 'imply CONFIG_...' instead.
>
Ah, yes...thanks. This would prevent the need make the same edits for
all the other socfpga_*_defconfigs.
Dinh
^ permalink raw reply [flat|nested] 13+ messages in thread
* [U-Boot] [RFC PATCHv1 1/3] misc: pl310: add a misc driver for the pl310 cache controller
2019-03-05 20:07 ` Dinh Nguyen
@ 2019-03-05 20:57 ` Marek Vasut
0 siblings, 0 replies; 13+ messages in thread
From: Marek Vasut @ 2019-03-05 20:57 UTC (permalink / raw)
To: u-boot
On 3/5/19 9:07 PM, Dinh Nguyen wrote:
>
>
> On 3/5/19 1:19 PM, Marek Vasut wrote:
>> On 3/5/19 8:03 PM, Dinh Nguyen wrote:
>>> The driver will read the cache properties from the device tree file and
>>> set it up.
>>>
>
>>>
>>> +config L2X0_CACHE
>>> + bool "L2x0 Cache support"
>>> + depends on MISC
>>> + help
>>> + Select this to enable a PL310 L2 Cache driver. The driver will
>>> + configure the L2 Cache settings found in the device tree.
>>
>> I wonder whether we don't need some drivers/plat or drivers/soc for this ?
>
> You mean for plat specific implementations?
Rather, since this is ARM-specific driver.
>>> config ALTERA_SYSID
>>> bool "Altera Sysid support"
>>> depends on MISC
>>> diff --git a/drivers/misc/Makefile b/drivers/misc/Makefile
>>> index 6bdf5054f4..ea726f4668 100644
>>> --- a/drivers/misc/Makefile
>>> +++ b/drivers/misc/Makefile
>>> @@ -23,6 +23,7 @@ obj-$(CONFIG_SANDBOX) += spltest_sandbox.o
>>> endif
>>> endif
>>> obj-$(CONFIG_ALI152X) += ali512x.o
>>> +obj-$(CONFIG_L2X0_CACHE) += cache-l2x0.o
>>> obj-$(CONFIG_ALTERA_SYSID) += altera_sysid.o
>>> obj-$(CONFIG_ATSHA204A) += atsha204a-i2c.o
>>> obj-$(CONFIG_CBMEM_CONSOLE) += cbmem_console.o
>>> diff --git a/drivers/misc/cache-l2x0.c b/drivers/misc/cache-l2x0.c
>>> new file mode 100644
>>> index 0000000000..b31598b1cd
>>> --- /dev/null
>>> +++ b/drivers/misc/cache-l2x0.c
>>> @@ -0,0 +1,84 @@
>>> +// SPDX-License-Identifier: GPL-2.0
>>> +/*
>>> + * Copyright (C) 2007 ARM Limited
>>
>> 2007 ? :)
>
> I think I can remove this. I started with the Linux's cache-l2x0, but
> removed almost all of the code. Just a left-over artifact.
2007-2019 then ?
>>> +
>>> + saved_reg |= L310_LATENCY_CTRL_RD(tag[0] - 1) |
>>> + L310_LATENCY_CTRL_WR(tag[1] - 1) |
>>> + L310_LATENCY_CTRL_SETUP(tag[2] - 1);
>>> +
>>> + writel(saved_reg, ®s->pl310_tag_latency_ctrl);
>>> +
>>> + saved_reg = readl(®s->pl310_data_latency_ctrl);
>>> + dev_read_u32_array(dev, "arm,data-latency", tag, 3);
>>
>> What happens if the array isn't present ?
>> Should we _not_ configure the latencies in such case ?
>
> Right, I have a to-do list that need to handle error conditions like this.
Please fix.
--
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 13+ messages in thread
* [U-Boot] [RFC PATCHv1 2/3] defconfig: socfpga_sockit_defconfig: enable L2X0_CACHE driver
2019-03-05 20:17 ` Dinh Nguyen
@ 2019-03-05 20:57 ` Marek Vasut
0 siblings, 0 replies; 13+ messages in thread
From: Marek Vasut @ 2019-03-05 20:57 UTC (permalink / raw)
To: u-boot
On 3/5/19 9:17 PM, Dinh Nguyen wrote:
>
>
> On 3/5/19 1:20 PM, Marek Vasut wrote:
>> On 3/5/19 8:03 PM, Dinh Nguyen wrote:
>>> Enable CONFIG_MISC and CONFIG_L2X0_CACHE config options.
>>>
>>> Signed-off-by: Dinh Nguyen <dinguyen@kernel.org>
>>> ---
>>> configs/socfpga_sockit_defconfig | 2 ++
>>> 1 file changed, 2 insertions(+)
>>>
>>> diff --git a/configs/socfpga_sockit_defconfig b/configs/socfpga_sockit_defconfig
>>> index 4c17d1a9e4..0009b0ebc3 100644
>>> --- a/configs/socfpga_sockit_defconfig
>>> +++ b/configs/socfpga_sockit_defconfig
>>> @@ -47,6 +47,8 @@ CONFIG_DM_GPIO=y
>>> CONFIG_DWAPB_GPIO=y
>>> CONFIG_DM_I2C=y
>>> CONFIG_SYS_I2C_DW=y
>>> +CONFIG_MISC=y
>>> +CONFIG_L2X0_CACHE=y
>>> CONFIG_DM_MMC=y
>>> CONFIG_MMC_DW=y
>>> CONFIG_MTD_DEVICE=y
>>
>> I think you can just add it as a default into arch/arm/socfpga/Kconfig
>> with 'imply CONFIG_...' instead.
>>
>
> Ah, yes...thanks. This would prevent the need make the same edits for
> all the other socfpga_*_defconfigs.
Right
--
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 13+ messages in thread
* [U-Boot] [RFC PATCHv1 3/3] ARM: socfpga: let the pl310 driver configure the cache settings
2019-03-05 20:13 ` Dinh Nguyen
@ 2019-03-05 20:57 ` Marek Vasut
0 siblings, 0 replies; 13+ messages in thread
From: Marek Vasut @ 2019-03-05 20:57 UTC (permalink / raw)
To: u-boot
On 3/5/19 9:13 PM, Dinh Nguyen wrote:
>
>
> On 3/5/19 1:20 PM, Marek Vasut wrote:
>> On 3/5/19 8:03 PM, Dinh Nguyen wrote:
>>> Load the PL310 L2 cache driver and allow it to setup the cache settings
>>>
>>> Signed-off-by: Dinh Nguyen <dinguyen@kernel.org>
>>> ---
>>> arch/arm/mach-socfpga/misc.c | 15 ++-------------
>>> 1 file changed, 2 insertions(+), 13 deletions(-)
>>>
>>> diff --git a/arch/arm/mach-socfpga/misc.c b/arch/arm/mach-socfpga/misc.c
>>> index fcf211d62b..fb0cfd3c1a 100644
>>> --- a/arch/arm/mach-socfpga/misc.c
>>> +++ b/arch/arm/mach-socfpga/misc.c
>>> @@ -59,20 +59,9 @@ void enable_caches(void)
>>> #ifdef CONFIG_SYS_L2_PL310
>>> void v7_outer_cache_enable(void)
>>> {
>>> - /* Disable the L2 cache */
>>> - clrbits_le32(&pl310->pl310_ctrl, L2X0_CTRL_EN);
>>> -
>>> - writel(0x111, &pl310->pl310_tag_latency_ctrl);
>>> - writel(0x121, &pl310->pl310_data_latency_ctrl);
>>> -
>>> - /* enable BRESP, instruction and data prefetch, full line of zeroes */
>>> - setbits_le32(&pl310->pl310_aux_ctrl,
>>> - L310_AUX_CTRL_DATA_PREFETCH_MASK |
>>> - L310_AUX_CTRL_INST_PREFETCH_MASK |
>>> - L310_SHARED_ATT_OVERRIDE_ENABLE);
>>> + struct udevice *dev;
>>>
>>> - /* Enable the L2 cache */
>>> - setbits_le32(&pl310->pl310_ctrl, L2X0_CTRL_EN);
>>> + uclass_first_device(UCLASS_MISC, &dev);
>>
>> Error handling might help here
>>
>
> Agreed..Just trying to solicit comments on whether this is the right
> approach? Will add error handling..
I think so.
--
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2019-03-05 20:57 UTC | newest]
Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-05 19:03 [U-Boot] [RFC PATCHv1 0/3] misc: pl310: add a dm cache driver Dinh Nguyen
2019-03-05 19:03 ` [U-Boot] [RFC PATCHv1 1/3] misc: pl310: add a misc driver for the pl310 cache controller Dinh Nguyen
2019-03-05 19:19 ` Marek Vasut
2019-03-05 20:07 ` Dinh Nguyen
2019-03-05 20:57 ` Marek Vasut
2019-03-05 19:03 ` [U-Boot] [RFC PATCHv1 2/3] defconfig: socfpga_sockit_defconfig: enable L2X0_CACHE driver Dinh Nguyen
2019-03-05 19:20 ` Marek Vasut
2019-03-05 20:17 ` Dinh Nguyen
2019-03-05 20:57 ` Marek Vasut
2019-03-05 19:03 ` [U-Boot] [RFC PATCHv1 3/3] ARM: socfpga: let the pl310 driver configure the cache settings Dinh Nguyen
2019-03-05 19:20 ` Marek Vasut
2019-03-05 20:13 ` Dinh Nguyen
2019-03-05 20:57 ` Marek Vasut
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.