* [PATCH 0/2] Add efuse driver for Ingenic JZ4780 SoC
@ 2017-12-27 12:27 Mathieu Malaterre
[not found] ` <20171227122722.5219-1-malat-8fiUuRrzOP0dnm+yROfE0A@public.gmane.org>
0 siblings, 1 reply; 8+ messages in thread
From: Mathieu Malaterre @ 2017-12-27 12:27 UTC (permalink / raw)
To: Zubair.Kakakhel-1AXoQHu6uovQT0dZR+AlfA
Cc: Mathieu Malaterre, Srinivas Kandagatla, Rob Herring, Mark Rutland,
Ralf Baechle, PrasannaKumar Muralidharan, David S. Miller,
Greg Kroah-Hartman, Mauro Carvalho Chehab, Randy Dunlap,
Linus Walleij, Philippe Ombredanne, Paul Cercueil, Harvey Hunt,
Krzysztof Kozlowski, James Hogan,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-mips-6z/3iImG2C8G8FEW9MqTrA
This patchset bring support for read-only access to the JZ4780 efuse as found
on MIPS Creator CI20.
To keep the driver as simple as possible, it was not possible to re-use most of
the nvmem core functionalities. This driver is not compatible with the original
efuse driver as found in the custom linux kernel from upstream (1), in
particular it does not expose to the users neither:
`/sys/devices/platform/*/chip_id` nor `/sys/devices/platform/*/user_id`.
The goal of this driver is to provide access to the MAC address to the dm9000
driver.
(1) https://github.com/ZubairLK/CI20_linux/commit/6efd4ffca7dcfaff0794ab60cd6922ce96c60419
Mathieu Malaterre (1):
dts: Probe efuse for CI20
PrasannaKumar Muralidharan (1):
nvmem: add driver for JZ4780 efuse
.../ABI/testing/sysfs-driver-jz4780-efuse | 16 ++
.../bindings/nvmem/ingenic,jz4780-efuse.txt | 17 ++
MAINTAINERS | 5 +
arch/mips/boot/dts/ingenic/jz4780.dtsi | 40 ++-
arch/mips/configs/ci20_defconfig | 2 +
drivers/nvmem/Kconfig | 10 +
drivers/nvmem/Makefile | 2 +
drivers/nvmem/jz4780-efuse.c | 274 +++++++++++++++++++++
8 files changed, 354 insertions(+), 12 deletions(-)
create mode 100644 Documentation/ABI/testing/sysfs-driver-jz4780-efuse
create mode 100644 Documentation/devicetree/bindings/nvmem/ingenic,jz4780-efuse.txt
create mode 100644 drivers/nvmem/jz4780-efuse.c
--
2.11.0
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 8+ messages in thread[parent not found: <20171227122722.5219-1-malat-8fiUuRrzOP0dnm+yROfE0A@public.gmane.org>]
* [PATCH 1/2] nvmem: add driver for JZ4780 efuse [not found] ` <20171227122722.5219-1-malat-8fiUuRrzOP0dnm+yROfE0A@public.gmane.org> @ 2017-12-27 12:27 ` Mathieu Malaterre 2017-12-28 7:13 ` Marcin Nowakowski 2017-12-27 12:27 ` [PATCH 2/2] dts: Probe efuse for CI20 Mathieu Malaterre 1 sibling, 1 reply; 8+ messages in thread From: Mathieu Malaterre @ 2017-12-27 12:27 UTC (permalink / raw) To: Zubair.Kakakhel-1AXoQHu6uovQT0dZR+AlfA Cc: PrasannaKumar Muralidharan, Srinivas Kandagatla, Rob Herring, Mark Rutland, Ralf Baechle, David S. Miller, Mauro Carvalho Chehab, Greg Kroah-Hartman, Randy Dunlap, Mathieu Malaterre, Thomas Gleixner, Paul Cercueil, Linus Walleij, Harvey Hunt, James Hogan, Krzysztof Kozlowski, linux-kernel-u79uwXL29TY76Z2rM5mHXA, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-mips-6z/3iImG2C8G8FEW9MqTrA From: PrasannaKumar Muralidharan <prasannatsmkumar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> This patch brings support for the JZ4780 efuse. Currently it only expose a read only access to the entire 8K bits efuse memory. Tested-by: Mathieu Malaterre <malat-8fiUuRrzOP0dnm+yROfE0A@public.gmane.org> Signed-off-by: PrasannaKumar Muralidharan <prasannatsmkumar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> --- .../ABI/testing/sysfs-driver-jz4780-efuse | 16 ++ .../bindings/nvmem/ingenic,jz4780-efuse.txt | 17 ++ MAINTAINERS | 5 + arch/mips/boot/dts/ingenic/jz4780.dtsi | 40 ++- drivers/nvmem/Kconfig | 10 + drivers/nvmem/Makefile | 2 + drivers/nvmem/jz4780-efuse.c | 274 +++++++++++++++++++++ 7 files changed, 352 insertions(+), 12 deletions(-) create mode 100644 Documentation/ABI/testing/sysfs-driver-jz4780-efuse create mode 100644 Documentation/devicetree/bindings/nvmem/ingenic,jz4780-efuse.txt create mode 100644 drivers/nvmem/jz4780-efuse.c diff --git a/Documentation/ABI/testing/sysfs-driver-jz4780-efuse b/Documentation/ABI/testing/sysfs-driver-jz4780-efuse new file mode 100644 index 000000000000..bb6f5d6ceea0 --- /dev/null +++ b/Documentation/ABI/testing/sysfs-driver-jz4780-efuse @@ -0,0 +1,16 @@ +What: /sys/devices/*/<our-device>/nvmem +Date: December 2017 +Contact: PrasannaKumar Muralidharan <prasannatsmkumar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> +Description: read-only access to the efuse on the Ingenic JZ4780 SoC + The SoC has a one time programmable 8K efuse that is + split into segments. The driver supports read only. + The segments are + 0x000 64 bit Random Number + 0x008 128 bit Ingenic Chip ID + 0x018 128 bit Customer ID + 0x028 3520 bit Reserved + 0x1E0 8 bit Protect Segment + 0x1E1 2296 bit HDMI Key + 0x300 2048 bit Security boot key +Users: any user space application which wants to read the Chip + and Customer ID diff --git a/Documentation/devicetree/bindings/nvmem/ingenic,jz4780-efuse.txt b/Documentation/devicetree/bindings/nvmem/ingenic,jz4780-efuse.txt new file mode 100644 index 000000000000..cd6d67ec22fc --- /dev/null +++ b/Documentation/devicetree/bindings/nvmem/ingenic,jz4780-efuse.txt @@ -0,0 +1,17 @@ +Ingenic JZ EFUSE driver bindings + +Required properties: +- "compatible" Must be set to "ingenic,jz4780-efuse" +- "reg" Register location and length +- "clocks" Handle for the ahb clock for the efuse. +- "clock-names" Must be "bus_clk" + +Example: + +efuse: efuse@134100d0 { + compatible = "ingenic,jz4780-efuse"; + reg = <0x134100D0 0xFF>; + + clocks = <&cgu JZ4780_CLK_AHB2>; + clock-names = "bus_clk"; +}; diff --git a/MAINTAINERS b/MAINTAINERS index a6e86e20761e..7a050c20c533 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -6902,6 +6902,11 @@ M: Zubair Lutfullah Kakakhel <Zubair.Kakakhel-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org> S: Maintained F: drivers/dma/dma-jz4780.c +INGENIC JZ4780 EFUSE Driver +M: PrasannaKumar Muralidharan <prasannatsmkumar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> +S: Maintained +F: drivers/nvmem/jz4780-efuse.c + INGENIC JZ4780 NAND DRIVER M: Harvey Hunt <harveyhuntnexus-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> L: linux-mtd-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org diff --git a/arch/mips/boot/dts/ingenic/jz4780.dtsi b/arch/mips/boot/dts/ingenic/jz4780.dtsi index 9b5794667aee..3fb9d916a2ea 100644 --- a/arch/mips/boot/dts/ingenic/jz4780.dtsi +++ b/arch/mips/boot/dts/ingenic/jz4780.dtsi @@ -224,21 +224,37 @@ reg = <0x10002000 0x100>; }; - nemc: nemc@13410000 { - compatible = "ingenic,jz4780-nemc"; - reg = <0x13410000 0x10000>; - #address-cells = <2>; + + ahb2: ahb2 { + compatible = "simple-bus"; + #address-cells = <1>; #size-cells = <1>; - ranges = <1 0 0x1b000000 0x1000000 - 2 0 0x1a000000 0x1000000 - 3 0 0x19000000 0x1000000 - 4 0 0x18000000 0x1000000 - 5 0 0x17000000 0x1000000 - 6 0 0x16000000 0x1000000>; + ranges = <>; + + nemc: nemc@13410000 { + compatible = "ingenic,jz4780-nemc"; + reg = <0x13410000 0x10000>; + #address-cells = <2>; + #size-cells = <1>; + ranges = <1 0 0x1b000000 0x1000000 + 2 0 0x1a000000 0x1000000 + 3 0 0x19000000 0x1000000 + 4 0 0x18000000 0x1000000 + 5 0 0x17000000 0x1000000 + 6 0 0x16000000 0x1000000>; + + clocks = <&cgu JZ4780_CLK_NEMC>; + + status = "disabled"; + }; - clocks = <&cgu JZ4780_CLK_NEMC>; + efuse: efuse@134100d0 { + compatible = "ingenic,jz4780-efuse"; + reg = <0x134100d0 0xff>; - status = "disabled"; + clocks = <&cgu JZ4780_CLK_AHB2>; + clock-names = "bus_clk"; + }; }; bch: bch@134d0000 { diff --git a/drivers/nvmem/Kconfig b/drivers/nvmem/Kconfig index ff505af064ba..75400982abac 100644 --- a/drivers/nvmem/Kconfig +++ b/drivers/nvmem/Kconfig @@ -36,6 +36,16 @@ config NVMEM_IMX_OCOTP This driver can also be built as a module. If so, the module will be called nvmem-imx-ocotp. +config JZ4780_EFUSE + tristate "JZ4780 EFUSE Memory Support" + depends on MACH_JZ4780 || COMPILE_TEST + depends on HAS_IOMEM + help + Say Y here to include support for JZ4780 efuse memory found on + all JZ4780 SoC based devices. + To compile this driver as a module, choose M here: the module + will be called nvmem_jz4780_efuse. + config NVMEM_LPC18XX_EEPROM tristate "NXP LPC18XX EEPROM Memory Support" depends on ARCH_LPC18XX || COMPILE_TEST diff --git a/drivers/nvmem/Makefile b/drivers/nvmem/Makefile index e54dcfa6565a..3b7c18df6771 100644 --- a/drivers/nvmem/Makefile +++ b/drivers/nvmem/Makefile @@ -13,6 +13,8 @@ obj-$(CONFIG_NVMEM_IMX_IIM) += nvmem-imx-iim.o nvmem-imx-iim-y := imx-iim.o obj-$(CONFIG_NVMEM_IMX_OCOTP) += nvmem-imx-ocotp.o nvmem-imx-ocotp-y := imx-ocotp.o +obj-$(CONFIG_JZ4780_EFUSE) += nvmem_jz4780_efuse.o +nvmem_jz4780_efuse-y := jz4780-efuse.o obj-$(CONFIG_NVMEM_LPC18XX_EEPROM) += nvmem_lpc18xx_eeprom.o nvmem_lpc18xx_eeprom-y := lpc18xx_eeprom.o obj-$(CONFIG_NVMEM_LPC18XX_OTP) += nvmem_lpc18xx_otp.o diff --git a/drivers/nvmem/jz4780-efuse.c b/drivers/nvmem/jz4780-efuse.c new file mode 100644 index 000000000000..4860716ed25d --- /dev/null +++ b/drivers/nvmem/jz4780-efuse.c @@ -0,0 +1,274 @@ +/* + * JZ4780 EFUSE Memory Support driver + * + * Copyright (c) 2017 PrasannaKumar Muralidharan <prasannatsmkumar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 + * as published by the Free Software Foundation. + */ + +/* + * Currently supports JZ4780 efuse which has 8K programmable bit. + * Efuse is separated into seven segments as below: + * + * ----------------------------------------------------------------------- + * | 64 bit | 128 bit | 128 bit | 3520 bit | 8 bit | 2296 bit | 2048 bit | + * ----------------------------------------------------------------------- + * + * The rom itself is accessed using a 9 bit address line and an 8 word wide bus + * which reads/writes based on strobes. The strobe is configured in the config + * register and is based on number of cycles of the bus clock. + * + * Driver supports read only as the writes are done in the Factory. + */ +#include <linux/bitops.h> +#include <linux/clk.h> +#include <linux/delay.h> +#include <linux/device.h> +#include <linux/io.h> +#include <linux/module.h> +#include <linux/nvmem-provider.h> +#include <linux/of.h> +#include <linux/of_device.h> +#include <linux/platform_device.h> +#include <linux/slab.h> +#include <linux/timer.h> + +#define JZ_EFUCTRL (0x0) /* Control Register */ +#define JZ_EFUCFG (0x4) /* Configure Register*/ +#define JZ_EFUSTATE (0x8) /* Status Register */ +#define JZ_EFUDATA(n) (0xC + (n)*4) + +#define JZ_EFUSE_START_ADDR 0x200 +#define JZ_EFUSE_SEG1_OFF 0x00 /* 64 bit Random Number */ +#define JZ_EFUSE_SEG2_OFF 0x08 /* 128 bit Ingenic Chip ID */ +#define JZ_EFUSE_SEG3_OFF 0x18 /* 128 bit Customer ID */ +#define JZ_EFUSE_SEG4_OFF 0x28 /* 3520 bit Reserved */ +#define JZ_EFUSE_SEG5_OFF 0x1E0 /* 8 bit Protect Segment */ +#define JZ_EFUSE_SEG6_OFF 0x1E1 /* 2296 bit HDMI Key */ +#define JZ_EFUSE_SEG7_OFF 0x300 /* 2048 bit Security boot key */ +#define JZ_EFUSE_END_ADDR 0x5FF + +#define JZ_EFUSE_EFUCTRL_CS BIT(30) +#define JZ_EFUSE_EFUCTRL_ADDR_MASK 0x1FF +#define JZ_EFUSE_EFUCTRL_ADDR_SHIFT 21 +#define JZ_EFUSE_EFUCTRL_LEN_MASK 0x1F +#define JZ_EFUSE_EFUCTRL_LEN_SHIFT 16 +#define JZ_EFUSE_EFUCTRL_PG_EN BIT(15) +#define JZ_EFUSE_EFUCTRL_WR_EN BIT(1) +#define JZ_EFUSE_EFUCTRL_RD_EN BIT(0) + +#define JZ_EFUSE_EFUCFG_INT_EN BIT(31) +#define JZ_EFUSE_EFUCFG_RD_ADJ_MASK 0xF +#define JZ_EFUSE_EFUCFG_RD_ADJ_SHIFT 20 +#define JZ_EFUSE_EFUCFG_RD_STR_MASK 0xF +#define JZ_EFUSE_EFUCFG_RD_STR_SHIFT 16 +#define JZ_EFUSE_EFUCFG_WR_ADJ_MASK 0xF +#define JZ_EFUSE_EFUCFG_WR_ADJ_SHIFT 12 +#define JZ_EFUSE_EFUCFG_WR_STR_MASK 0xFFF +#define JZ_EFUSE_EFUCFG_WR_STR_SHIFT 0 + +#define JZ_EFUSE_EFUSTATE_WR_DONE BIT(1) +#define JZ_EFUSE_EFUSTATE_RD_DONE BIT(0) + +#define JZ_EFUSE_WORD_SIZE 16 +#define JZ_EFUSE_STRIDE 8 + +struct jz4780_efuse { + struct device *dev; + void __iomem *iomem; + struct clk *clk; + unsigned int rd_adj; + unsigned int rd_strobe; +}; + +/* We read 32 byte chunks to avoid complexity in the driver. */ +static ssize_t jz4780_efuse_read_32bytes(struct jz4780_efuse *efuse, char *buf, + unsigned int addr) +{ + unsigned int tmp = 0; + int i = 0; + int timeout = 1000; + int size = 32; + + /* 1. Set config register */ + tmp = readl(efuse->iomem + JZ_EFUCFG); + tmp &= ~((JZ_EFUSE_EFUCFG_RD_ADJ_MASK << JZ_EFUSE_EFUCFG_RD_ADJ_SHIFT) + | (JZ_EFUSE_EFUCFG_RD_STR_MASK << JZ_EFUSE_EFUCFG_RD_STR_SHIFT)); + tmp |= (efuse->rd_adj << JZ_EFUSE_EFUCFG_RD_ADJ_SHIFT) + | (efuse->rd_strobe << JZ_EFUSE_EFUCFG_RD_STR_SHIFT); + writel(tmp, efuse->iomem + JZ_EFUCFG); + + /* + * 2. Set control register to indicate what to read data address, + * read data numbers and read enable. + */ + tmp = readl(efuse->iomem + JZ_EFUCTRL); + tmp &= ~(JZ_EFUSE_EFUCFG_RD_STR_SHIFT + | (JZ_EFUSE_EFUCTRL_ADDR_MASK << JZ_EFUSE_EFUCTRL_ADDR_SHIFT) + | JZ_EFUSE_EFUCTRL_PG_EN | JZ_EFUSE_EFUCTRL_WR_EN + | JZ_EFUSE_EFUCTRL_WR_EN); + + /* Need to select CS bit if address accesses upper 4Kbits memory */ + if (addr >= (JZ_EFUSE_START_ADDR + 512)) + tmp |= JZ_EFUSE_EFUCTRL_CS; + + tmp |= (addr << JZ_EFUSE_EFUCTRL_ADDR_SHIFT) + | ((size - 1) << JZ_EFUSE_EFUCTRL_LEN_SHIFT) + | JZ_EFUSE_EFUCTRL_RD_EN; + writel(tmp, efuse->iomem + JZ_EFUCTRL); + + /* + * 3. Wait status register RD_DONE set to 1 or EFUSE interrupted, + * software can read EFUSE data buffer 0 - 8 registers. + */ + do { + tmp = readl(efuse->iomem + JZ_EFUSTATE); + usleep_range(1000, 2000); + if (timeout--) + break; + } while (!(tmp & JZ_EFUSE_EFUSTATE_RD_DONE)); + + if (timeout <= 0) { + dev_err(efuse->dev, "Timed out while reading\n"); + return -EAGAIN; + } + + for (i = 0; i < (size / 4); i++) + *((unsigned int *)(buf + i * 4)) + = readl(efuse->iomem + JZ_EFUDATA(i)); + + return 0; +} + +static unsigned int segments[][2] = { + /* offset , size in bits */ + { JZ_EFUSE_SEG1_OFF, 64 }, /* bit Random Number */ + { JZ_EFUSE_SEG2_OFF, 128 }, /* bit Ingenic Chip ID */ + { JZ_EFUSE_SEG3_OFF, 128 }, /* bit Customer ID */ + { JZ_EFUSE_SEG4_OFF, 3520 }, /* bit Reserved */ + { JZ_EFUSE_SEG5_OFF, 8 }, /* bit Protect Segment */ + { JZ_EFUSE_SEG6_OFF, 2296 }, /* bit HDMI Key */ + { JZ_EFUSE_SEG7_OFF, 2048 } /* bit Security boot key */ +}; + +/* main entry point */ +static int jz4780_efuse_read(void *context, unsigned int offset, + void *val, size_t bytes) +{ + static const int nsegments = sizeof(segments) / sizeof(*segments); + struct jz4780_efuse *efuse = context; + char buf[32]; + char *cur = val; + int i; + /* PM recommends read/write each segment separately */ + for (i = 0; i < nsegments; ++i) { + unsigned int *segment = segments[i]; + unsigned int lpos = segment[0]; + unsigned int buflen = segment[1] / 8; + unsigned int ncount = buflen / 32; + unsigned int remain = buflen % 32; + int j; + /* EFUSE can read or write maximum 256bit in each time */ + for (j = 0; j < ncount ; ++j) { + jz4780_efuse_read_32bytes(efuse, buf, lpos); + memcpy(cur, buf, sizeof(buf)); + cur += sizeof(buf); + lpos += sizeof(buf); + } + if (remain) { + jz4780_efuse_read_32bytes(efuse, buf, lpos); + memcpy(cur, buf, remain); + cur += remain; + } + } + + return 0; +} + +static struct nvmem_config jz4780_efuse_nvmem_config = { + .name = "jz4780-efuse", + .read_only = true, + .word_size = JZ_EFUSE_WORD_SIZE, + .stride = JZ_EFUSE_STRIDE, + .owner = THIS_MODULE, + .reg_read = jz4780_efuse_read, +}; + +static int jz4780_efuse_probe(struct platform_device *pdev) +{ + struct nvmem_device *nvmem; + struct jz4780_efuse *efuse; + struct resource *res; + unsigned long clk_rate; + struct device *dev = &pdev->dev; + + efuse = devm_kzalloc(&pdev->dev, sizeof(*efuse), GFP_KERNEL); + if (!efuse) + return -ENOMEM; + + res = platform_get_resource(pdev, IORESOURCE_MEM, 0); + efuse->iomem = devm_ioremap(&pdev->dev, res->start, resource_size(res)); + if (IS_ERR(efuse->iomem)) + return PTR_ERR(efuse->iomem); + + efuse->clk = devm_clk_get(&pdev->dev, "bus_clk"); + if (IS_ERR(efuse->clk)) + return PTR_ERR(efuse->clk); + + clk_rate = clk_get_rate(efuse->clk); + /* + * rd_adj and rd_strobe are 4 bit values + * bus clk period * (rd_adj + 1) > 6.5ns + * bus clk period * (rd_adj + 5 + rd_strobe) > 35ns + */ + efuse->rd_adj = (((6500 * (clk_rate / 1000000)) / 1000000) + 1) - 1; + efuse->rd_strobe = ((((35000 * (clk_rate / 1000000)) / 1000000) + 1) + - 5 - efuse->rd_adj); + + if ((efuse->rd_adj > 0x1F) || (efuse->rd_strobe > 0x1F)) { + dev_err(&pdev->dev, "Cannot set clock configuration\n"); + return -EINVAL; + } + efuse->dev = dev; + + jz4780_efuse_nvmem_config.size = 1024; + jz4780_efuse_nvmem_config.dev = &pdev->dev; + jz4780_efuse_nvmem_config.priv = efuse; + + nvmem = nvmem_register(&jz4780_efuse_nvmem_config); + if (IS_ERR(nvmem)) + return PTR_ERR(nvmem); + + platform_set_drvdata(pdev, nvmem); + + return 0; +} + +static int jz4780_efuse_remove(struct platform_device *pdev) +{ + struct nvmem_device *nvmem = platform_get_drvdata(pdev); + + return nvmem_unregister(nvmem); +} + +static const struct of_device_id jz4780_efuse_match[] = { + { .compatible = "ingenic,jz4780-efuse" }, + { /* sentinel */ }, +}; +MODULE_DEVICE_TABLE(of, jz4780_efuse_match); + +static struct platform_driver jz4780_efuse_driver = { + .probe = jz4780_efuse_probe, + .remove = jz4780_efuse_remove, + .driver = { + .name = "jz4780-efuse", + .of_match_table = jz4780_efuse_match, + }, +}; +module_platform_driver(jz4780_efuse_driver); + +MODULE_AUTHOR("PrasannaKumar Muralidharan <prasannatsmkumar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>"); +MODULE_DESCRIPTION("Ingenic JZ4780 efuse driver"); +MODULE_LICENSE("GPL v2"); -- 2.11.0 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH 1/2] nvmem: add driver for JZ4780 efuse 2017-12-27 12:27 ` [PATCH 1/2] nvmem: add driver for JZ4780 efuse Mathieu Malaterre @ 2017-12-28 7:13 ` Marcin Nowakowski 2017-12-28 7:26 ` Mathieu Malaterre 0 siblings, 1 reply; 8+ messages in thread From: Marcin Nowakowski @ 2017-12-28 7:13 UTC (permalink / raw) To: Mathieu Malaterre, Zubair.Kakakhel Cc: PrasannaKumar Muralidharan, Srinivas Kandagatla, Rob Herring, Mark Rutland, Ralf Baechle, David S. Miller, Mauro Carvalho Chehab, Greg Kroah-Hartman, Randy Dunlap, Thomas Gleixner, Paul Cercueil, Linus Walleij, Harvey Hunt, James Hogan, Krzysztof Kozlowski, linux-kernel, devicetree, linux-mips Hi Mathieu, PrasannaKumar, On 27.12.2017 13:27, Mathieu Malaterre wrote: > From: PrasannaKumar Muralidharan <prasannatsmkumar@gmail.com> > > This patch brings support for the JZ4780 efuse. Currently it only expose > a read only access to the entire 8K bits efuse memory. > > Tested-by: Mathieu Malaterre <malat@debian.org> > Signed-off-by: PrasannaKumar Muralidharan <prasannatsmkumar@gmail.com> > --- > + > +/* main entry point */ > +static int jz4780_efuse_read(void *context, unsigned int offset, > + void *val, size_t bytes) > +{ > + static const int nsegments = sizeof(segments) / sizeof(*segments); > + struct jz4780_efuse *efuse = context; > + char buf[32]; > + char *cur = val; > + int i; > + /* PM recommends read/write each segment separately */ > + for (i = 0; i < nsegments; ++i) { > + unsigned int *segment = segments[i]; > + unsigned int lpos = segment[0]; > + unsigned int buflen = segment[1] / 8; > + unsigned int ncount = buflen / 32; > + unsigned int remain = buflen % 32; > + int j; This doesn't look right, as offset & bytes are completely ignored. This means it will return data from an offset other than requested and may also overrun the provided output buffer? > + /* EFUSE can read or write maximum 256bit in each time */ > + for (j = 0; j < ncount ; ++j) { > + jz4780_efuse_read_32bytes(efuse, buf, lpos); > + memcpy(cur, buf, sizeof(buf)); > + cur += sizeof(buf); > + lpos += sizeof(buf); > + } > + if (remain) { > + jz4780_efuse_read_32bytes(efuse, buf, lpos); > + memcpy(cur, buf, remain); > + cur += remain; > + } > + } > + > + return 0; > +} Marcin ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/2] nvmem: add driver for JZ4780 efuse 2017-12-28 7:13 ` Marcin Nowakowski @ 2017-12-28 7:26 ` Mathieu Malaterre [not found] ` <CA+7wUsxM4Cq-K6ONSO-WzmYYvq8PmT92Jfrf7M-MqY-ntObi-g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 0 siblings, 1 reply; 8+ messages in thread From: Mathieu Malaterre @ 2017-12-28 7:26 UTC (permalink / raw) To: Marcin Nowakowski Cc: Zubair.Kakakhel, PrasannaKumar Muralidharan, Srinivas Kandagatla, Rob Herring, Mark Rutland, Ralf Baechle, David S. Miller, Mauro Carvalho Chehab, Greg Kroah-Hartman, Randy Dunlap, Thomas Gleixner, Paul Cercueil, Linus Walleij, Harvey Hunt, James Hogan, Krzysztof Kozlowski, linux-kernel, devicetree, linux-mips [-- Attachment #1: Type: text/plain, Size: 2878 bytes --] Hi Marcin, On Thu, Dec 28, 2017 at 8:13 AM, Marcin Nowakowski < marcin.nowakowski@mips.com> wrote: > Hi Mathieu, PrasannaKumar, > > On 27.12.2017 13:27, Mathieu Malaterre wrote: >> >> From: PrasannaKumar Muralidharan <prasannatsmkumar@gmail.com> >> >> This patch brings support for the JZ4780 efuse. Currently it only expose >> a read only access to the entire 8K bits efuse memory. >> >> Tested-by: Mathieu Malaterre <malat@debian.org> >> Signed-off-by: PrasannaKumar Muralidharan <prasannatsmkumar@gmail.com> >> --- > > >> + >> +/* main entry point */ >> +static int jz4780_efuse_read(void *context, unsigned int offset, >> + void *val, size_t bytes) >> +{ >> + static const int nsegments = sizeof(segments) / sizeof(*segments); >> + struct jz4780_efuse *efuse = context; >> + char buf[32]; >> + char *cur = val; >> + int i; >> + /* PM recommends read/write each segment separately */ >> + for (i = 0; i < nsegments; ++i) { >> + unsigned int *segment = segments[i]; >> + unsigned int lpos = segment[0]; >> + unsigned int buflen = segment[1] / 8; >> + unsigned int ncount = buflen / 32; >> + unsigned int remain = buflen % 32; >> + int j; > > > This doesn't look right, as offset & bytes are completely ignored. This > means it will return data from an offset other than requested and may also > overrun the provided output buffer? Thanks for the review ! That was the part of nvmem framework I was not totally clear. Let say I want to expose only a portion of efuse space, eg: diff --git a/arch/mips/boot/dts/ingenic/jz4780.dtsi b/arch/mips/boot/dts/ingenic/jz4780.dtsi index 2f26922718559..44d97c06a6d15 100644 --- a/arch/mips/boot/dts/ingenic/jz4780.dtsi +++ b/arch/mips/boot/dts/ingenic/jz4780.dtsi @@ -299,6 +299,15 @@ clocks = <&cgu JZ4780_CLK_AHB2>; clock-names = "bus_clk"; + + #address-cells = <1>; + #size-cells = <1>; + + eth_mac: eth_mac@12 { + /* six byte/48bit MAC address stored as 8-bit integers */ + reg = <0x12 0x6>; + }; + }; }; What should I do to expose that chunk only in the user space ? > >> + /* EFUSE can read or write maximum 256bit in each time */ >> + for (j = 0; j < ncount ; ++j) { >> + jz4780_efuse_read_32bytes(efuse, buf, lpos); >> + memcpy(cur, buf, sizeof(buf)); >> + cur += sizeof(buf); >> + lpos += sizeof(buf); >> + } >> + if (remain) { >> + jz4780_efuse_read_32bytes(efuse, buf, lpos); >> + memcpy(cur, buf, remain); >> + cur += remain; >> + } >> + } >> + >> + return 0; >> +} > > > > Marcin > -M [-- Attachment #2: Type: text/html, Size: 4501 bytes --] ^ permalink raw reply related [flat|nested] 8+ messages in thread
[parent not found: <CA+7wUsxM4Cq-K6ONSO-WzmYYvq8PmT92Jfrf7M-MqY-ntObi-g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH 1/2] nvmem: add driver for JZ4780 efuse [not found] ` <CA+7wUsxM4Cq-K6ONSO-WzmYYvq8PmT92Jfrf7M-MqY-ntObi-g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> @ 2017-12-28 8:05 ` Marcin Nowakowski [not found] ` <0a3b0b12-80c0-9d6d-bfdb-8d2541947750-8NJIiSa5LzA@public.gmane.org> 0 siblings, 1 reply; 8+ messages in thread From: Marcin Nowakowski @ 2017-12-28 8:05 UTC (permalink / raw) To: Mathieu Malaterre Cc: Zubair.Kakakhel-8NJIiSa5LzA, PrasannaKumar Muralidharan, Srinivas Kandagatla, Rob Herring, Mark Rutland, Ralf Baechle, David S. Miller, Mauro Carvalho Chehab, Greg Kroah-Hartman, Randy Dunlap, Thomas Gleixner, Paul Cercueil, Linus Walleij, Harvey Hunt, James Hogan, Krzysztof Kozlowski, linux-kernel-u79uwXL29TY76Z2rM5mHXA, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-mips-6z/3iImG2C8G8FEW9MqTrA Hi Mathieu, On 28.12.2017 08:26, Mathieu Malaterre wrote: > Hi Marcin, > > On Thu, Dec 28, 2017 at 8:13 AM, Marcin Nowakowski > <marcin.nowakowski-8NJIiSa5LzA@public.gmane.org <mailto:marcin.nowakowski-8NJIiSa5LzA@public.gmane.org>> wrote: > > Hi Mathieu, PrasannaKumar, > > > > On 27.12.2017 13:27, Mathieu Malaterre wrote: > >> > >> From: PrasannaKumar Muralidharan <prasannatsmkumar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org > <mailto:prasannatsmkumar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>> > >> > >> This patch brings support for the JZ4780 efuse. Currently it only expose > >> a read only access to the entire 8K bits efuse memory. > >> > >> Tested-by: Mathieu Malaterre <malat-8fiUuRrzOP0dnm+yROfE0A@public.gmane.org > <mailto:malat-8fiUuRrzOP0dnm+yROfE0A@public.gmane.org>> > >> Signed-off-by: PrasannaKumar Muralidharan > <prasannatsmkumar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org <mailto:prasannatsmkumar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>> > >> --- > > > > > >> + > >> +/* main entry point */ > >> +static int jz4780_efuse_read(void *context, unsigned int offset, > >> + void *val, size_t bytes) > >> +{ > >> + static const int nsegments = sizeof(segments) / > sizeof(*segments); > >> + struct jz4780_efuse *efuse = context; > >> + char buf[32]; > >> + char *cur = val; > >> + int i; > >> + /* PM recommends read/write each segment separately */ > >> + for (i = 0; i < nsegments; ++i) { > >> + unsigned int *segment = segments[i]; > >> + unsigned int lpos = segment[0]; > >> + unsigned int buflen = segment[1] / 8; > >> + unsigned int ncount = buflen / 32; > >> + unsigned int remain = buflen % 32; > >> + int j; > > > > > > This doesn't look right, as offset & bytes are completely ignored. This > > means it will return data from an offset other than requested and may > also > > overrun the provided output buffer? > > > Thanks for the review ! That was the part of nvmem framework I was not > totally clear. Let say I want to expose only a portion of efuse space, eg: Do you need to expose this to the userspace or to other drivers only? For the second case have a look at the description of nvmem cell interface. > diff --git a/arch/mips/boot/dts/ingenic/jz4780.dtsi > b/arch/mips/boot/dts/ingenic/jz4780.dtsi > index 2f26922718559..44d97c06a6d15 100644 > --- a/arch/mips/boot/dts/ingenic/jz4780.dtsi > +++ b/arch/mips/boot/dts/ingenic/jz4780.dtsi > @@ -299,6 +299,15 @@ > clocks = <&cgu JZ4780_CLK_AHB2>; > clock-names = "bus_clk"; > + > +#address-cells = <1>; > +#size-cells = <1>; > + > +eth_mac: eth_mac@12 { > +/* six byte/48bit MAC address stored as 8-bit integers */ > +reg = <0x12 0x6>; > +}; > + > }; > }; > What should I do to expose that chunk only in the user space ? The nvmem interface's userspace interface (via /sys/.../nvmem) provides access to the complete device raw memory so the only way to achieve that would be to parse the devicetree description in your driver and only register part of the memory with the nvmem driver - but that would be a slight abuse of the interface. The nvmem devicetree binding document shows clearly how to define the cell interface that can later be used by any consumer - that way you could have the ethernet driver access the cell directly. However, as the dm9000 driver isn't designed to do that and this is a SoC-specific extention, I don't know how it fits with the general eth driver design ... Potentially a good and useful compromise would be to have all of the cell regs exposed via /sys/.../nvmem-cellname file (or something similar), but this is not currently supported and I don't know what the view of nvmem maintainers on adding such extension would be. > > > >> + /* EFUSE can read or write maximum 256bit in each > time */ > >> + for (j = 0; j < ncount ; ++j) { > >> + jz4780_efuse_read_32bytes(efuse, buf, lpos); > >> + memcpy(cur, buf, sizeof(buf)); > >> + cur += sizeof(buf); > >> + lpos += sizeof(buf); > >> + } > >> + if (remain) { > >> + jz4780_efuse_read_32bytes(efuse, buf, lpos); > >> + memcpy(cur, buf, remain); > >> + cur += remain; > >> + } > >> + } > >> + > >> + return 0; > >> +} Regardless of the choices above, you still always have to make sure in your reg_read method that you only read from the offset specified in method arguments and never return more than 'bytes' of data requested. Marcin -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 8+ messages in thread
[parent not found: <0a3b0b12-80c0-9d6d-bfdb-8d2541947750-8NJIiSa5LzA@public.gmane.org>]
* Re: [PATCH 1/2] nvmem: add driver for JZ4780 efuse [not found] ` <0a3b0b12-80c0-9d6d-bfdb-8d2541947750-8NJIiSa5LzA@public.gmane.org> @ 2018-01-06 11:52 ` PrasannaKumar Muralidharan 0 siblings, 0 replies; 8+ messages in thread From: PrasannaKumar Muralidharan @ 2018-01-06 11:52 UTC (permalink / raw) To: Marcin Nowakowski Cc: Mathieu Malaterre, Zubair.Kakakhel-8NJIiSa5LzA, Srinivas Kandagatla, Rob Herring, Mark Rutland, Ralf Baechle, David S. Miller, Mauro Carvalho Chehab, Greg Kroah-Hartman, Randy Dunlap, Thomas Gleixner, Paul Cercueil, Linus Walleij, Harvey Hunt, James Hogan, Krzysztof Kozlowski, open list, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-mips-6z/3iImG2C8G8FEW9MqTrA Hi Marcin, On 28 December 2017 at 13:35, Marcin Nowakowski <marcin.nowakowski-8NJIiSa5LzA@public.gmane.org> wrote: > Hi Mathieu, > > On 28.12.2017 08:26, Mathieu Malaterre wrote: >> >> Hi Marcin, >> >> On Thu, Dec 28, 2017 at 8:13 AM, Marcin Nowakowski >> <marcin.nowakowski-8NJIiSa5LzA@public.gmane.org <mailto:marcin.nowakowski-8NJIiSa5LzA@public.gmane.org>> wrote: >> > Hi Mathieu, PrasannaKumar, >> > >> > On 27.12.2017 13:27, Mathieu Malaterre wrote: >> >> >> >> From: PrasannaKumar Muralidharan <prasannatsmkumar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org >> <mailto:prasannatsmkumar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>> >> >> >> >> This patch brings support for the JZ4780 efuse. Currently it only >> expose >> >> a read only access to the entire 8K bits efuse memory. >> >> >> >> Tested-by: Mathieu Malaterre <malat-8fiUuRrzOP0dnm+yROfE0A@public.gmane.org >> <mailto:malat-8fiUuRrzOP0dnm+yROfE0A@public.gmane.org>> >> >> Signed-off-by: PrasannaKumar Muralidharan <prasannatsmkumar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org >> <mailto:prasannatsmkumar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>> >> >> --- >> > >> > >> >> + >> >> +/* main entry point */ >> >> +static int jz4780_efuse_read(void *context, unsigned int offset, >> >> + void *val, size_t bytes) >> >> +{ >> >> + static const int nsegments = sizeof(segments) / >> sizeof(*segments); >> >> + struct jz4780_efuse *efuse = context; >> >> + char buf[32]; >> >> + char *cur = val; >> >> + int i; >> >> + /* PM recommends read/write each segment separately */ >> >> + for (i = 0; i < nsegments; ++i) { >> >> + unsigned int *segment = segments[i]; >> >> + unsigned int lpos = segment[0]; >> >> + unsigned int buflen = segment[1] / 8; >> >> + unsigned int ncount = buflen / 32; >> >> + unsigned int remain = buflen % 32; >> >> + int j; >> > >> > >> > This doesn't look right, as offset & bytes are completely ignored. This >> > means it will return data from an offset other than requested and may >> also >> > overrun the provided output buffer? >> >> >> Thanks for the review ! That was the part of nvmem framework I was not >> totally clear. Let say I want to expose only a portion of efuse space, eg: > > > Do you need to expose this to the userspace or to other drivers only? > For the second case have a look at the description of nvmem cell interface. > > >> diff --git a/arch/mips/boot/dts/ingenic/jz4780.dtsi >> b/arch/mips/boot/dts/ingenic/jz4780.dtsi >> index 2f26922718559..44d97c06a6d15 100644 >> --- a/arch/mips/boot/dts/ingenic/jz4780.dtsi >> +++ b/arch/mips/boot/dts/ingenic/jz4780.dtsi >> @@ -299,6 +299,15 @@ >> clocks = <&cgu JZ4780_CLK_AHB2>; >> clock-names = "bus_clk"; >> + >> +#address-cells = <1>; >> +#size-cells = <1>; >> + >> +eth_mac: eth_mac@12 { >> +/* six byte/48bit MAC address stored as 8-bit integers */ >> +reg = <0x12 0x6>; >> +}; >> + >> }; >> }; >> What should I do to expose that chunk only in the user space ? > > > The nvmem interface's userspace interface (via /sys/.../nvmem) provides > access to the complete device raw memory so the only way to achieve that > would be to parse the devicetree description in your driver and only > register part of the memory with the nvmem driver - but that would be a > slight abuse of the interface. > The nvmem devicetree binding document shows clearly how to define the cell > interface that can later be used by any consumer - that way you could have > the ethernet driver access the cell directly. However, as the dm9000 driver > isn't designed to do that and this is a SoC-specific extention, I don't know > how it fits with the general eth driver design ... > > Potentially a good and useful compromise would be to have all of the cell > regs exposed via /sys/.../nvmem-cellname file (or something similar), but > this is not currently supported and I don't know what the view of nvmem > maintainers on adding such extension would be. Currently exposing MAC address is necessary. No need to worry about user space stuff for now. >> > >> >> + /* EFUSE can read or write maximum 256bit in each time >> */ >> >> + for (j = 0; j < ncount ; ++j) { >> >> + jz4780_efuse_read_32bytes(efuse, buf, lpos); >> >> + memcpy(cur, buf, sizeof(buf)); >> >> + cur += sizeof(buf); >> >> + lpos += sizeof(buf); >> >> + } >> >> + if (remain) { >> >> + jz4780_efuse_read_32bytes(efuse, buf, lpos); >> >> + memcpy(cur, buf, remain); >> >> + cur += remain; >> >> + } >> >> + } >> >> + >> >> + return 0; >> >> +} > > > Regardless of the choices above, you still always have to make sure in your > reg_read method that you only read from the offset specified in method > arguments and never return more than 'bytes' of data requested. Sure, will do that. Regards, PrasannaKumar -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 2/2] dts: Probe efuse for CI20 [not found] ` <20171227122722.5219-1-malat-8fiUuRrzOP0dnm+yROfE0A@public.gmane.org> 2017-12-27 12:27 ` [PATCH 1/2] nvmem: add driver for JZ4780 efuse Mathieu Malaterre @ 2017-12-27 12:27 ` Mathieu Malaterre [not found] ` <20171227122722.5219-3-malat-8fiUuRrzOP0dnm+yROfE0A@public.gmane.org> 1 sibling, 1 reply; 8+ messages in thread From: Mathieu Malaterre @ 2017-12-27 12:27 UTC (permalink / raw) To: Zubair.Kakakhel-1AXoQHu6uovQT0dZR+AlfA Cc: Mathieu Malaterre, Srinivas Kandagatla, Rob Herring, Mark Rutland, Ralf Baechle, PrasannaKumar Muralidharan, David S. Miller, Mauro Carvalho Chehab, Greg Kroah-Hartman, Randy Dunlap, Philippe Ombredanne, Kate Stewart, Thomas Gleixner, Paul Cercueil, Harvey Hunt, Krzysztof Kozlowski, James Hogan, linux-kernel-u79uwXL29TY76Z2rM5mHXA, devicetree-u79uwXL29TY76Z2rM5mHXA Signed-off-by: Mathieu Malaterre <malat-8fiUuRrzOP0dnm+yROfE0A@public.gmane.org> --- arch/mips/configs/ci20_defconfig | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/mips/configs/ci20_defconfig b/arch/mips/configs/ci20_defconfig index b5f4ad8f2c45..62c63617e97a 100644 --- a/arch/mips/configs/ci20_defconfig +++ b/arch/mips/configs/ci20_defconfig @@ -171,3 +171,5 @@ CONFIG_STACKTRACE=y # CONFIG_FTRACE is not set CONFIG_CMDLINE_BOOL=y CONFIG_CMDLINE="earlycon console=ttyS4,115200 clk_ignore_unused" +CONFIG_NVMEM=y +CONFIG_JZ4780_EFUSE=y -- 2.11.0 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 8+ messages in thread
[parent not found: <20171227122722.5219-3-malat-8fiUuRrzOP0dnm+yROfE0A@public.gmane.org>]
* Re: [PATCH 2/2] dts: Probe efuse for CI20 [not found] ` <20171227122722.5219-3-malat-8fiUuRrzOP0dnm+yROfE0A@public.gmane.org> @ 2017-12-27 12:46 ` Greg Kroah-Hartman 0 siblings, 0 replies; 8+ messages in thread From: Greg Kroah-Hartman @ 2017-12-27 12:46 UTC (permalink / raw) To: Mathieu Malaterre Cc: Zubair.Kakakhel-1AXoQHu6uovQT0dZR+AlfA, Srinivas Kandagatla, Rob Herring, Mark Rutland, Ralf Baechle, PrasannaKumar Muralidharan, David S. Miller, Mauro Carvalho Chehab, Randy Dunlap, Philippe Ombredanne, Kate Stewart, Thomas Gleixner, Paul Cercueil, Harvey Hunt, Krzysztof Kozlowski, James Hogan, linux-kernel-u79uwXL29TY76Z2rM5mHXA, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-mips-6z/3iImG2C8G8FEW9MqTrA On Wed, Dec 27, 2017 at 01:27:02PM +0100, Mathieu Malaterre wrote: > Signed-off-by: Mathieu Malaterre <malat-8fiUuRrzOP0dnm+yROfE0A@public.gmane.org> I know i can't take patches without any changelog text at all, and really, you shouldn't ever create such a thing :) thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2018-01-06 11:52 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-12-27 12:27 [PATCH 0/2] Add efuse driver for Ingenic JZ4780 SoC Mathieu Malaterre
[not found] ` <20171227122722.5219-1-malat-8fiUuRrzOP0dnm+yROfE0A@public.gmane.org>
2017-12-27 12:27 ` [PATCH 1/2] nvmem: add driver for JZ4780 efuse Mathieu Malaterre
2017-12-28 7:13 ` Marcin Nowakowski
2017-12-28 7:26 ` Mathieu Malaterre
[not found] ` <CA+7wUsxM4Cq-K6ONSO-WzmYYvq8PmT92Jfrf7M-MqY-ntObi-g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-12-28 8:05 ` Marcin Nowakowski
[not found] ` <0a3b0b12-80c0-9d6d-bfdb-8d2541947750-8NJIiSa5LzA@public.gmane.org>
2018-01-06 11:52 ` PrasannaKumar Muralidharan
2017-12-27 12:27 ` [PATCH 2/2] dts: Probe efuse for CI20 Mathieu Malaterre
[not found] ` <20171227122722.5219-3-malat-8fiUuRrzOP0dnm+yROfE0A@public.gmane.org>
2017-12-27 12:46 ` Greg Kroah-Hartman
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).