* [PATCH v2 0/2] SPI ThunderX driver
@ 2016-07-28 8:31 ` Jan Glauber
0 siblings, 0 replies; 17+ messages in thread
From: Jan Glauber @ 2016-07-28 8:31 UTC (permalink / raw)
To: Mark Brown
Cc: linux-kernel, linux-spi, David Daney, Steven J . Hill, Jan Glauber
This series adds support for SPI on Cavium's ThunderX (arm64). The SPI
hardware is the same as on MIPS Octeon, the only difference is that the
device appears as a PCI device. To avoid copy and paste of the Octeon
driver I've moved the common parts into a shared file.
The series was tested on MIPS (Edge Router PRO and cn71xx) and ThunderX.
Changes to v1:
- Changed Kconfig depencency for ThunderX
- Merged clock setup functions with main probe/remove
- Fail if SPI DT entry misses clock reference
- Removed debug prints at end of probe
- Removed unneeded includes from spi-cavium.c
- Dropped merged patches
thanks,
Jan
--------------
Jan Glauber (2):
spi: octeon: Split driver into Octeon specific and common parts
spi: octeon: Add thunderx driver
drivers/spi/Kconfig | 7 ++
drivers/spi/Makefile | 3 +
drivers/spi/spi-cavium-octeon.c | 102 +++++++++++++++++++++
drivers/spi/spi-cavium-thunderx.c | 140 +++++++++++++++++++++++++++++
drivers/spi/{spi-octeon.c => spi-cavium.c} | 122 +------------------------
drivers/spi/spi-cavium.h | 34 +++++++
6 files changed, 288 insertions(+), 120 deletions(-)
create mode 100644 drivers/spi/spi-cavium-octeon.c
create mode 100644 drivers/spi/spi-cavium-thunderx.c
rename drivers/spi/{spi-octeon.c => spi-cavium.c} (54%)
--
2.9.0.rc0.21.g7777322
^ permalink raw reply [flat|nested] 17+ messages in thread* [PATCH v2 0/2] SPI ThunderX driver @ 2016-07-28 8:31 ` Jan Glauber 0 siblings, 0 replies; 17+ messages in thread From: Jan Glauber @ 2016-07-28 8:31 UTC (permalink / raw) To: Mark Brown Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA, linux-spi-u79uwXL29TY76Z2rM5mHXA, David Daney, Steven J . Hill, Jan Glauber This series adds support for SPI on Cavium's ThunderX (arm64). The SPI hardware is the same as on MIPS Octeon, the only difference is that the device appears as a PCI device. To avoid copy and paste of the Octeon driver I've moved the common parts into a shared file. The series was tested on MIPS (Edge Router PRO and cn71xx) and ThunderX. Changes to v1: - Changed Kconfig depencency for ThunderX - Merged clock setup functions with main probe/remove - Fail if SPI DT entry misses clock reference - Removed debug prints at end of probe - Removed unneeded includes from spi-cavium.c - Dropped merged patches thanks, Jan -------------- Jan Glauber (2): spi: octeon: Split driver into Octeon specific and common parts spi: octeon: Add thunderx driver drivers/spi/Kconfig | 7 ++ drivers/spi/Makefile | 3 + drivers/spi/spi-cavium-octeon.c | 102 +++++++++++++++++++++ drivers/spi/spi-cavium-thunderx.c | 140 +++++++++++++++++++++++++++++ drivers/spi/{spi-octeon.c => spi-cavium.c} | 122 +------------------------ drivers/spi/spi-cavium.h | 34 +++++++ 6 files changed, 288 insertions(+), 120 deletions(-) create mode 100644 drivers/spi/spi-cavium-octeon.c create mode 100644 drivers/spi/spi-cavium-thunderx.c rename drivers/spi/{spi-octeon.c => spi-cavium.c} (54%) -- 2.9.0.rc0.21.g7777322 -- To unsubscribe from this list: send the line "unsubscribe linux-spi" 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] 17+ messages in thread
* [PATCH v2 1/2] spi: octeon: Split driver into Octeon specific and common parts 2016-07-28 8:31 ` Jan Glauber (?) @ 2016-07-28 8:31 ` Jan Glauber 2016-08-01 17:36 ` Mark Brown -1 siblings, 1 reply; 17+ messages in thread From: Jan Glauber @ 2016-07-28 8:31 UTC (permalink / raw) To: Mark Brown Cc: linux-kernel, linux-spi, David Daney, Steven J . Hill, Jan Glauber Separate driver probing from SPI transfer functions. Signed-off-by: Jan Glauber <jglauber@cavium.com> --- drivers/spi/Makefile | 1 + drivers/spi/spi-cavium-octeon.c | 102 ++++++++++++++++++++++++ drivers/spi/{spi-octeon.c => spi-cavium.c} | 122 +---------------------------- drivers/spi/spi-cavium.h | 31 ++++++++ 4 files changed, 136 insertions(+), 120 deletions(-) create mode 100644 drivers/spi/spi-cavium-octeon.c rename drivers/spi/{spi-octeon.c => spi-cavium.c} (54%) diff --git a/drivers/spi/Makefile b/drivers/spi/Makefile index 3c74d00..185367e 100644 --- a/drivers/spi/Makefile +++ b/drivers/spi/Makefile @@ -56,6 +56,7 @@ obj-$(CONFIG_SPI_MT65XX) += spi-mt65xx.o obj-$(CONFIG_SPI_MXS) += spi-mxs.o obj-$(CONFIG_SPI_NUC900) += spi-nuc900.o obj-$(CONFIG_SPI_OC_TINY) += spi-oc-tiny.o +spi-octeon-objs := spi-cavium.o spi-cavium-octeon.o obj-$(CONFIG_SPI_OCTEON) += spi-octeon.o obj-$(CONFIG_SPI_OMAP_UWIRE) += spi-omap-uwire.o obj-$(CONFIG_SPI_OMAP_100K) += spi-omap-100k.o diff --git a/drivers/spi/spi-cavium-octeon.c b/drivers/spi/spi-cavium-octeon.c new file mode 100644 index 0000000..97310c1 --- /dev/null +++ b/drivers/spi/spi-cavium-octeon.c @@ -0,0 +1,102 @@ +/* + * This file is subject to the terms and conditions of the GNU General Public + * License. See the file "COPYING" in the main directory of this archive + * for more details. + * + * Copyright (C) 2011, 2012 Cavium, Inc. + */ + +#include <linux/platform_device.h> +#include <linux/spi/spi.h> +#include <linux/module.h> +#include <linux/io.h> +#include <linux/of.h> + +#include <asm/octeon/octeon.h> + +#include "spi-cavium.h" + +static int octeon_spi_probe(struct platform_device *pdev) +{ + struct resource *res_mem; + void __iomem *reg_base; + struct spi_master *master; + struct octeon_spi *p; + int err = -ENOENT; + + master = spi_alloc_master(&pdev->dev, sizeof(struct octeon_spi)); + if (!master) + return -ENOMEM; + p = spi_master_get_devdata(master); + platform_set_drvdata(pdev, master); + + res_mem = platform_get_resource(pdev, IORESOURCE_MEM, 0); + reg_base = devm_ioremap_resource(&pdev->dev, res_mem); + if (IS_ERR(reg_base)) { + err = PTR_ERR(reg_base); + goto fail; + } + + p->register_base = reg_base; + p->sys_freq = octeon_get_io_clock_rate(); + + p->regs.config = 0; + p->regs.status = 0x08; + p->regs.tx = 0x10; + p->regs.data = 0x80; + + master->num_chipselect = 4; + master->mode_bits = SPI_CPHA | + SPI_CPOL | + SPI_CS_HIGH | + SPI_LSB_FIRST | + SPI_3WIRE; + + master->transfer_one_message = octeon_spi_transfer_one_message; + master->bits_per_word_mask = SPI_BPW_MASK(8); + master->max_speed_hz = OCTEON_SPI_MAX_CLOCK_HZ; + + master->dev.of_node = pdev->dev.of_node; + err = devm_spi_register_master(&pdev->dev, master); + if (err) { + dev_err(&pdev->dev, "register master failed: %d\n", err); + goto fail; + } + + return 0; +fail: + spi_master_put(master); + return err; +} + +static int octeon_spi_remove(struct platform_device *pdev) +{ + struct spi_master *master = platform_get_drvdata(pdev); + struct octeon_spi *p = spi_master_get_devdata(master); + + /* Clear the CSENA* and put everything in a known state. */ + writeq(0, p->register_base + OCTEON_SPI_CFG(p)); + + return 0; +} + +static const struct of_device_id octeon_spi_match[] = { + { .compatible = "cavium,octeon-3010-spi", }, + {}, +}; +MODULE_DEVICE_TABLE(of, octeon_spi_match); + +static struct platform_driver octeon_spi_driver = { + .driver = { + .name = "spi-octeon", + .of_match_table = octeon_spi_match, + }, + .probe = octeon_spi_probe, + .remove = octeon_spi_remove, +}; + +module_platform_driver(octeon_spi_driver); + +MODULE_DESCRIPTION("Cavium, Inc. OCTEON SPI bus driver"); +MODULE_AUTHOR("David Daney"); +MODULE_LICENSE("GPL"); diff --git a/drivers/spi/spi-octeon.c b/drivers/spi/spi-cavium.c similarity index 54% rename from drivers/spi/spi-octeon.c rename to drivers/spi/spi-cavium.c index 2180176..8857e7d 100644 --- a/drivers/spi/spi-octeon.c +++ b/drivers/spi/spi-cavium.c @@ -6,42 +6,11 @@ * Copyright (C) 2011, 2012 Cavium, Inc. */ -#include <linux/platform_device.h> -#include <linux/interrupt.h> #include <linux/spi/spi.h> -#include <linux/module.h> #include <linux/delay.h> -#include <linux/io.h> -#include <linux/of.h> - -#include <asm/octeon/octeon.h> #include "spi-cavium.h" -#define OCTEON_SPI_MAX_BYTES 9 - -#define OCTEON_SPI_MAX_CLOCK_HZ 16000000 - -struct octeon_spi_regs { - int config; - int status; - int tx; - int data; -}; - -struct octeon_spi { - void __iomem *register_base; - u64 last_cfg; - u64 cs_enax; - int sys_freq; - struct octeon_spi_regs regs; -}; - -#define OCTEON_SPI_CFG(x) (x->regs.config) -#define OCTEON_SPI_STS(x) (x->regs.status) -#define OCTEON_SPI_TX(x) (x->regs.tx) -#define OCTEON_SPI_DAT0(x) (x->regs.data) - static void octeon_spi_wait_ready(struct octeon_spi *p) { union cvmx_mpi_sts mpi_sts; @@ -154,8 +123,8 @@ static int octeon_spi_do_transfer(struct octeon_spi *p, return xfer->len; } -static int octeon_spi_transfer_one_message(struct spi_master *master, - struct spi_message *msg) +int octeon_spi_transfer_one_message(struct spi_master *master, + struct spi_message *msg) { struct octeon_spi *p = spi_master_get_devdata(master); unsigned int total_len = 0; @@ -178,90 +147,3 @@ err: spi_finalize_current_message(master); return status; } - -static int octeon_spi_probe(struct platform_device *pdev) -{ - struct resource *res_mem; - void __iomem *reg_base; - struct spi_master *master; - struct octeon_spi *p; - int err = -ENOENT; - - master = spi_alloc_master(&pdev->dev, sizeof(struct octeon_spi)); - if (!master) - return -ENOMEM; - p = spi_master_get_devdata(master); - platform_set_drvdata(pdev, master); - - res_mem = platform_get_resource(pdev, IORESOURCE_MEM, 0); - reg_base = devm_ioremap_resource(&pdev->dev, res_mem); - if (IS_ERR(reg_base)) { - err = PTR_ERR(reg_base); - goto fail; - } - - p->register_base = reg_base; - p->sys_freq = octeon_get_io_clock_rate(); - - p->regs.config = 0; - p->regs.status = 0x08; - p->regs.tx = 0x10; - p->regs.data = 0x80; - - master->num_chipselect = 4; - master->mode_bits = SPI_CPHA | - SPI_CPOL | - SPI_CS_HIGH | - SPI_LSB_FIRST | - SPI_3WIRE; - - master->transfer_one_message = octeon_spi_transfer_one_message; - master->bits_per_word_mask = SPI_BPW_MASK(8); - master->max_speed_hz = OCTEON_SPI_MAX_CLOCK_HZ; - - master->dev.of_node = pdev->dev.of_node; - err = devm_spi_register_master(&pdev->dev, master); - if (err) { - dev_err(&pdev->dev, "register master failed: %d\n", err); - goto fail; - } - - dev_info(&pdev->dev, "OCTEON SPI bus driver\n"); - - return 0; -fail: - spi_master_put(master); - return err; -} - -static int octeon_spi_remove(struct platform_device *pdev) -{ - struct spi_master *master = platform_get_drvdata(pdev); - struct octeon_spi *p = spi_master_get_devdata(master); - - /* Clear the CSENA* and put everything in a known state. */ - writeq(0, p->register_base + OCTEON_SPI_CFG(p)); - - return 0; -} - -static const struct of_device_id octeon_spi_match[] = { - { .compatible = "cavium,octeon-3010-spi", }, - {}, -}; -MODULE_DEVICE_TABLE(of, octeon_spi_match); - -static struct platform_driver octeon_spi_driver = { - .driver = { - .name = "spi-octeon", - .of_match_table = octeon_spi_match, - }, - .probe = octeon_spi_probe, - .remove = octeon_spi_remove, -}; - -module_platform_driver(octeon_spi_driver); - -MODULE_DESCRIPTION("Cavium, Inc. OCTEON SPI bus driver"); -MODULE_AUTHOR("David Daney"); -MODULE_LICENSE("GPL"); diff --git a/drivers/spi/spi-cavium.h b/drivers/spi/spi-cavium.h index d41dba5..88c5f36 100644 --- a/drivers/spi/spi-cavium.h +++ b/drivers/spi/spi-cavium.h @@ -1,3 +1,32 @@ +#ifndef __SPI_CAVIUM_H +#define __SPI_CAVIUM_H + +#define OCTEON_SPI_MAX_BYTES 9 +#define OCTEON_SPI_MAX_CLOCK_HZ 16000000 + +struct octeon_spi_regs { + int config; + int status; + int tx; + int data; +}; + +struct octeon_spi { + void __iomem *register_base; + u64 last_cfg; + u64 cs_enax; + int sys_freq; + struct octeon_spi_regs regs; +}; + +#define OCTEON_SPI_CFG(x) (x->regs.config) +#define OCTEON_SPI_STS(x) (x->regs.status) +#define OCTEON_SPI_TX(x) (x->regs.tx) +#define OCTEON_SPI_DAT0(x) (x->regs.data) + +int octeon_spi_transfer_one_message(struct spi_master *master, + struct spi_message *msg); + /* MPI register descriptions */ #define CVMX_MPI_CFG (CVMX_ADD_IO_SEG(0x0001070000001000ull)) @@ -296,3 +325,5 @@ union cvmx_mpi_tx { struct cvmx_mpi_tx_s cn66xx; struct cvmx_mpi_tx_cn61xx cnf71xx; }; + +#endif /* __SPI_CAVIUM_H */ -- 2.9.0.rc0.21.g7777322 ^ permalink raw reply related [flat|nested] 17+ messages in thread
* Re: [PATCH v2 1/2] spi: octeon: Split driver into Octeon specific and common parts @ 2016-08-01 17:36 ` Mark Brown 0 siblings, 0 replies; 17+ messages in thread From: Mark Brown @ 2016-08-01 17:36 UTC (permalink / raw) To: Jan Glauber; +Cc: linux-kernel, linux-spi, David Daney, Steven J . Hill [-- Attachment #1: Type: text/plain, Size: 337 bytes --] On Thu, Jul 28, 2016 at 10:31:43AM +0200, Jan Glauber wrote: > Separate driver probing from SPI transfer functions. This doesn't apply against current code as an earlier version of it was already applied. Please don't resubmit already applied code, please send incremental updates against current code for any changes that are needed. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 473 bytes --] ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH v2 1/2] spi: octeon: Split driver into Octeon specific and common parts @ 2016-08-01 17:36 ` Mark Brown 0 siblings, 0 replies; 17+ messages in thread From: Mark Brown @ 2016-08-01 17:36 UTC (permalink / raw) To: Jan Glauber Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA, linux-spi-u79uwXL29TY76Z2rM5mHXA, David Daney, Steven J . Hill [-- Attachment #1: Type: text/plain, Size: 337 bytes --] On Thu, Jul 28, 2016 at 10:31:43AM +0200, Jan Glauber wrote: > Separate driver probing from SPI transfer functions. This doesn't apply against current code as an earlier version of it was already applied. Please don't resubmit already applied code, please send incremental updates against current code for any changes that are needed. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 473 bytes --] ^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH v2 2/2] spi: octeon: Add thunderx driver @ 2016-07-28 8:31 ` Jan Glauber 0 siblings, 0 replies; 17+ messages in thread From: Jan Glauber @ 2016-07-28 8:31 UTC (permalink / raw) To: Mark Brown Cc: linux-kernel, linux-spi, David Daney, Steven J . Hill, Jan Glauber Add ThunderX SPI driver using the shared part from the Octeon driver. The main difference of the ThunderX driver is that it is a PCI device so probing is different. The system clock settings can be specified in device tree. Signed-off-by: Jan Glauber <jglauber@cavium.com> --- drivers/spi/Kconfig | 7 ++ drivers/spi/Makefile | 2 + drivers/spi/spi-cavium-thunderx.c | 140 ++++++++++++++++++++++++++++++++++++++ drivers/spi/spi-cavium.h | 3 + 4 files changed, 152 insertions(+) create mode 100644 drivers/spi/spi-cavium-thunderx.c diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig index 4b931ec..e0ee112 100644 --- a/drivers/spi/Kconfig +++ b/drivers/spi/Kconfig @@ -630,6 +630,13 @@ config SPI_TEGRA20_SLINK help SPI driver for Nvidia Tegra20/Tegra30 SLINK Controller interface. +config SPI_THUNDERX + tristate "Cavium ThunderX SPI controller" + depends on (ARM64 || CONFIG_TEST) && 64BIT && PCI + help + SPI host driver for the hardware found on Cavium ThunderX + SOCs. + config SPI_TOPCLIFF_PCH tristate "Intel EG20T PCH/LAPIS Semicon IOH(ML7213/ML7223/ML7831) SPI" depends on PCI && (X86_32 || MIPS || COMPILE_TEST) diff --git a/drivers/spi/Makefile b/drivers/spi/Makefile index 185367e..133364b 100644 --- a/drivers/spi/Makefile +++ b/drivers/spi/Makefile @@ -91,6 +91,8 @@ obj-$(CONFIG_SPI_TEGRA114) += spi-tegra114.o obj-$(CONFIG_SPI_TEGRA20_SFLASH) += spi-tegra20-sflash.o obj-$(CONFIG_SPI_TEGRA20_SLINK) += spi-tegra20-slink.o obj-$(CONFIG_SPI_TLE62X0) += spi-tle62x0.o +spi-thunderx-objs := spi-cavium.o spi-cavium-thunderx.o +obj-$(CONFIG_SPI_THUNDERX) += spi-thunderx.o obj-$(CONFIG_SPI_TOPCLIFF_PCH) += spi-topcliff-pch.o obj-$(CONFIG_SPI_TXX9) += spi-txx9.o obj-$(CONFIG_SPI_XCOMM) += spi-xcomm.o diff --git a/drivers/spi/spi-cavium-thunderx.c b/drivers/spi/spi-cavium-thunderx.c new file mode 100644 index 0000000..28c3dcc --- /dev/null +++ b/drivers/spi/spi-cavium-thunderx.c @@ -0,0 +1,140 @@ +/* + * Cavium ThunderX SPI driver. + * + * Copyright (C) 2016 Cavium Inc. + * Authors: Jan Glauber <jglauber@cavium.com> + */ + +#include <linux/delay.h> +#include <linux/io.h> +#include <linux/module.h> +#include <linux/pci.h> +#include <linux/spi/spi.h> + +#include "spi-cavium.h" + +#define DRV_NAME "spi-thunderx" + +#define SYS_FREQ_DEFAULT 700000000 /* 700 Mhz */ + +static int thunderx_spi_probe(struct pci_dev *pdev, + const struct pci_device_id *ent) +{ + struct device *dev = &pdev->dev; + struct spi_master *master; + struct octeon_spi *p; + int ret = -ENOENT; + + master = spi_alloc_master(dev, sizeof(struct octeon_spi)); + if (!master) + return -ENOMEM; + p = spi_master_get_devdata(master); + + ret = pci_enable_device(pdev); + if (ret) { + dev_err(dev, "Failed to enable PCI device\n"); + goto out_free; + } + + ret = pci_request_regions(pdev, DRV_NAME); + if (ret) { + dev_err(dev, "PCI request regions failed 0x%x\n", ret); + goto out_disable; + } + + p->register_base = pci_ioremap_bar(pdev, 0); + if (!p->register_base) { + dev_err(dev, "Cannot map reg base\n"); + ret = -EINVAL; + goto out_region; + } + + p->regs.config = 0x1000; + p->regs.status = 0x1008; + p->regs.tx = 0x1010; + p->regs.data = 0x1080; + + p->clk = devm_clk_get(dev, NULL); + if (IS_ERR(p->clk)) + goto out_unmap; + + ret = clk_prepare_enable(p->clk); + if (ret) + goto out_clock_devm; + + p->sys_freq = clk_get_rate(p->clk); + if (!p->sys_freq) + p->sys_freq = SYS_FREQ_DEFAULT; + dev_info(dev, "Set system clock to %u\n", p->sys_freq); + + master->num_chipselect = 4; + master->mode_bits = SPI_CPHA | SPI_CPOL | SPI_CS_HIGH | + SPI_LSB_FIRST | SPI_3WIRE; + master->transfer_one_message = octeon_spi_transfer_one_message; + master->bits_per_word_mask = SPI_BPW_MASK(8); + master->max_speed_hz = OCTEON_SPI_MAX_CLOCK_HZ; + master->dev.of_node = pdev->dev.of_node; + + pci_set_drvdata(pdev, master); + ret = devm_spi_register_master(dev, master); + if (ret) { + dev_err(&pdev->dev, "Register master failed: %d\n", ret); + goto out_clock; + } + + return 0; + +out_clock: + clk_disable_unprepare(p->clk); +out_clock_devm: + devm_clk_put(dev, p->clk); +out_unmap: + iounmap(p->register_base); +out_region: + pci_release_regions(pdev); +out_disable: + pci_disable_device(pdev); +out_free: + spi_master_put(master); + return ret; +} + +static void thunderx_spi_remove(struct pci_dev *pdev) +{ + struct spi_master *master = pci_get_drvdata(pdev); + struct octeon_spi *p; + + p = spi_master_get_devdata(master); + if (!p) + return; + + /* Put everything in a known state. */ + writeq(0, p->register_base + OCTEON_SPI_CFG(p)); + + clk_disable_unprepare(p->clk); + devm_clk_put(&pdev->dev, p->clk); + iounmap(p->register_base); + pci_release_regions(pdev); + pci_disable_device(pdev); + pci_set_drvdata(pdev, NULL); +} + +static const struct pci_device_id thunderx_spi_pci_id_table[] = { + { PCI_DEVICE(PCI_VENDOR_ID_CAVIUM, 0xa00b) }, + { 0, } +}; + +MODULE_DEVICE_TABLE(pci, thunderx_spi_pci_id_table); + +static struct pci_driver thunderx_spi_driver = { + .name = DRV_NAME, + .id_table = thunderx_spi_pci_id_table, + .probe = thunderx_spi_probe, + .remove = thunderx_spi_remove, +}; + +module_pci_driver(thunderx_spi_driver); + +MODULE_DESCRIPTION("Cavium, Inc. ThunderX SPI bus driver"); +MODULE_AUTHOR("Jan Glauber"); +MODULE_LICENSE("GPL"); diff --git a/drivers/spi/spi-cavium.h b/drivers/spi/spi-cavium.h index 88c5f36..1f91d61 100644 --- a/drivers/spi/spi-cavium.h +++ b/drivers/spi/spi-cavium.h @@ -1,6 +1,8 @@ #ifndef __SPI_CAVIUM_H #define __SPI_CAVIUM_H +#include <linux/clk.h> + #define OCTEON_SPI_MAX_BYTES 9 #define OCTEON_SPI_MAX_CLOCK_HZ 16000000 @@ -17,6 +19,7 @@ struct octeon_spi { u64 cs_enax; int sys_freq; struct octeon_spi_regs regs; + struct clk *clk; }; #define OCTEON_SPI_CFG(x) (x->regs.config) -- 2.9.0.rc0.21.g7777322 ^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH v2 2/2] spi: octeon: Add thunderx driver @ 2016-07-28 8:31 ` Jan Glauber 0 siblings, 0 replies; 17+ messages in thread From: Jan Glauber @ 2016-07-28 8:31 UTC (permalink / raw) To: Mark Brown Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA, linux-spi-u79uwXL29TY76Z2rM5mHXA, David Daney, Steven J . Hill, Jan Glauber Add ThunderX SPI driver using the shared part from the Octeon driver. The main difference of the ThunderX driver is that it is a PCI device so probing is different. The system clock settings can be specified in device tree. Signed-off-by: Jan Glauber <jglauber-YGCgFSpz5w/QT0dZR+AlfA@public.gmane.org> --- drivers/spi/Kconfig | 7 ++ drivers/spi/Makefile | 2 + drivers/spi/spi-cavium-thunderx.c | 140 ++++++++++++++++++++++++++++++++++++++ drivers/spi/spi-cavium.h | 3 + 4 files changed, 152 insertions(+) create mode 100644 drivers/spi/spi-cavium-thunderx.c diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig index 4b931ec..e0ee112 100644 --- a/drivers/spi/Kconfig +++ b/drivers/spi/Kconfig @@ -630,6 +630,13 @@ config SPI_TEGRA20_SLINK help SPI driver for Nvidia Tegra20/Tegra30 SLINK Controller interface. +config SPI_THUNDERX + tristate "Cavium ThunderX SPI controller" + depends on (ARM64 || CONFIG_TEST) && 64BIT && PCI + help + SPI host driver for the hardware found on Cavium ThunderX + SOCs. + config SPI_TOPCLIFF_PCH tristate "Intel EG20T PCH/LAPIS Semicon IOH(ML7213/ML7223/ML7831) SPI" depends on PCI && (X86_32 || MIPS || COMPILE_TEST) diff --git a/drivers/spi/Makefile b/drivers/spi/Makefile index 185367e..133364b 100644 --- a/drivers/spi/Makefile +++ b/drivers/spi/Makefile @@ -91,6 +91,8 @@ obj-$(CONFIG_SPI_TEGRA114) += spi-tegra114.o obj-$(CONFIG_SPI_TEGRA20_SFLASH) += spi-tegra20-sflash.o obj-$(CONFIG_SPI_TEGRA20_SLINK) += spi-tegra20-slink.o obj-$(CONFIG_SPI_TLE62X0) += spi-tle62x0.o +spi-thunderx-objs := spi-cavium.o spi-cavium-thunderx.o +obj-$(CONFIG_SPI_THUNDERX) += spi-thunderx.o obj-$(CONFIG_SPI_TOPCLIFF_PCH) += spi-topcliff-pch.o obj-$(CONFIG_SPI_TXX9) += spi-txx9.o obj-$(CONFIG_SPI_XCOMM) += spi-xcomm.o diff --git a/drivers/spi/spi-cavium-thunderx.c b/drivers/spi/spi-cavium-thunderx.c new file mode 100644 index 0000000..28c3dcc --- /dev/null +++ b/drivers/spi/spi-cavium-thunderx.c @@ -0,0 +1,140 @@ +/* + * Cavium ThunderX SPI driver. + * + * Copyright (C) 2016 Cavium Inc. + * Authors: Jan Glauber <jglauber-YGCgFSpz5w/QT0dZR+AlfA@public.gmane.org> + */ + +#include <linux/delay.h> +#include <linux/io.h> +#include <linux/module.h> +#include <linux/pci.h> +#include <linux/spi/spi.h> + +#include "spi-cavium.h" + +#define DRV_NAME "spi-thunderx" + +#define SYS_FREQ_DEFAULT 700000000 /* 700 Mhz */ + +static int thunderx_spi_probe(struct pci_dev *pdev, + const struct pci_device_id *ent) +{ + struct device *dev = &pdev->dev; + struct spi_master *master; + struct octeon_spi *p; + int ret = -ENOENT; + + master = spi_alloc_master(dev, sizeof(struct octeon_spi)); + if (!master) + return -ENOMEM; + p = spi_master_get_devdata(master); + + ret = pci_enable_device(pdev); + if (ret) { + dev_err(dev, "Failed to enable PCI device\n"); + goto out_free; + } + + ret = pci_request_regions(pdev, DRV_NAME); + if (ret) { + dev_err(dev, "PCI request regions failed 0x%x\n", ret); + goto out_disable; + } + + p->register_base = pci_ioremap_bar(pdev, 0); + if (!p->register_base) { + dev_err(dev, "Cannot map reg base\n"); + ret = -EINVAL; + goto out_region; + } + + p->regs.config = 0x1000; + p->regs.status = 0x1008; + p->regs.tx = 0x1010; + p->regs.data = 0x1080; + + p->clk = devm_clk_get(dev, NULL); + if (IS_ERR(p->clk)) + goto out_unmap; + + ret = clk_prepare_enable(p->clk); + if (ret) + goto out_clock_devm; + + p->sys_freq = clk_get_rate(p->clk); + if (!p->sys_freq) + p->sys_freq = SYS_FREQ_DEFAULT; + dev_info(dev, "Set system clock to %u\n", p->sys_freq); + + master->num_chipselect = 4; + master->mode_bits = SPI_CPHA | SPI_CPOL | SPI_CS_HIGH | + SPI_LSB_FIRST | SPI_3WIRE; + master->transfer_one_message = octeon_spi_transfer_one_message; + master->bits_per_word_mask = SPI_BPW_MASK(8); + master->max_speed_hz = OCTEON_SPI_MAX_CLOCK_HZ; + master->dev.of_node = pdev->dev.of_node; + + pci_set_drvdata(pdev, master); + ret = devm_spi_register_master(dev, master); + if (ret) { + dev_err(&pdev->dev, "Register master failed: %d\n", ret); + goto out_clock; + } + + return 0; + +out_clock: + clk_disable_unprepare(p->clk); +out_clock_devm: + devm_clk_put(dev, p->clk); +out_unmap: + iounmap(p->register_base); +out_region: + pci_release_regions(pdev); +out_disable: + pci_disable_device(pdev); +out_free: + spi_master_put(master); + return ret; +} + +static void thunderx_spi_remove(struct pci_dev *pdev) +{ + struct spi_master *master = pci_get_drvdata(pdev); + struct octeon_spi *p; + + p = spi_master_get_devdata(master); + if (!p) + return; + + /* Put everything in a known state. */ + writeq(0, p->register_base + OCTEON_SPI_CFG(p)); + + clk_disable_unprepare(p->clk); + devm_clk_put(&pdev->dev, p->clk); + iounmap(p->register_base); + pci_release_regions(pdev); + pci_disable_device(pdev); + pci_set_drvdata(pdev, NULL); +} + +static const struct pci_device_id thunderx_spi_pci_id_table[] = { + { PCI_DEVICE(PCI_VENDOR_ID_CAVIUM, 0xa00b) }, + { 0, } +}; + +MODULE_DEVICE_TABLE(pci, thunderx_spi_pci_id_table); + +static struct pci_driver thunderx_spi_driver = { + .name = DRV_NAME, + .id_table = thunderx_spi_pci_id_table, + .probe = thunderx_spi_probe, + .remove = thunderx_spi_remove, +}; + +module_pci_driver(thunderx_spi_driver); + +MODULE_DESCRIPTION("Cavium, Inc. ThunderX SPI bus driver"); +MODULE_AUTHOR("Jan Glauber"); +MODULE_LICENSE("GPL"); diff --git a/drivers/spi/spi-cavium.h b/drivers/spi/spi-cavium.h index 88c5f36..1f91d61 100644 --- a/drivers/spi/spi-cavium.h +++ b/drivers/spi/spi-cavium.h @@ -1,6 +1,8 @@ #ifndef __SPI_CAVIUM_H #define __SPI_CAVIUM_H +#include <linux/clk.h> + #define OCTEON_SPI_MAX_BYTES 9 #define OCTEON_SPI_MAX_CLOCK_HZ 16000000 @@ -17,6 +19,7 @@ struct octeon_spi { u64 cs_enax; int sys_freq; struct octeon_spi_regs regs; + struct clk *clk; }; #define OCTEON_SPI_CFG(x) (x->regs.config) -- 2.9.0.rc0.21.g7777322 -- To unsubscribe from this list: send the line "unsubscribe linux-spi" 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] 17+ messages in thread
* Re: [PATCH v2 2/2] spi: octeon: Add thunderx driver @ 2016-08-01 17:28 ` Mark Brown 0 siblings, 0 replies; 17+ messages in thread From: Mark Brown @ 2016-08-01 17:28 UTC (permalink / raw) To: Jan Glauber; +Cc: linux-kernel, linux-spi, David Daney, Steven J . Hill [-- Attachment #1: Type: text/plain, Size: 771 bytes --] On Thu, Jul 28, 2016 at 10:31:44AM +0200, Jan Glauber wrote: > +config SPI_THUNDERX > + tristate "Cavium ThunderX SPI controller" > + depends on (ARM64 || CONFIG_TEST) && 64BIT && PCI You mean COMPILE_TEST. > + p->clk = devm_clk_get(dev, NULL); > + if (IS_ERR(p->clk)) > + goto out_unmap; We're now just using the normal clock API which is good but I'm now unclear what is going to ensure that the clock is there - is there some other change elsewhere that I'm not aware of? We're also not passing the resulting error code back to the caller which will break deferred probe. > +out_clock: > + clk_disable_unprepare(p->clk); > +out_clock_devm: > + devm_clk_put(dev, p->clk); There's no point in using managed allocations if you're going to manually free things... [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 473 bytes --] ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH v2 2/2] spi: octeon: Add thunderx driver @ 2016-08-01 17:28 ` Mark Brown 0 siblings, 0 replies; 17+ messages in thread From: Mark Brown @ 2016-08-01 17:28 UTC (permalink / raw) To: Jan Glauber Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA, linux-spi-u79uwXL29TY76Z2rM5mHXA, David Daney, Steven J . Hill [-- Attachment #1: Type: text/plain, Size: 771 bytes --] On Thu, Jul 28, 2016 at 10:31:44AM +0200, Jan Glauber wrote: > +config SPI_THUNDERX > + tristate "Cavium ThunderX SPI controller" > + depends on (ARM64 || CONFIG_TEST) && 64BIT && PCI You mean COMPILE_TEST. > + p->clk = devm_clk_get(dev, NULL); > + if (IS_ERR(p->clk)) > + goto out_unmap; We're now just using the normal clock API which is good but I'm now unclear what is going to ensure that the clock is there - is there some other change elsewhere that I'm not aware of? We're also not passing the resulting error code back to the caller which will break deferred probe. > +out_clock: > + clk_disable_unprepare(p->clk); > +out_clock_devm: > + devm_clk_put(dev, p->clk); There's no point in using managed allocations if you're going to manually free things... [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 473 bytes --] ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH v2 2/2] spi: octeon: Add thunderx driver 2016-08-01 17:28 ` Mark Brown (?) @ 2016-08-01 18:31 ` David Daney 2016-08-01 18:49 ` Mark Brown -1 siblings, 1 reply; 17+ messages in thread From: David Daney @ 2016-08-01 18:31 UTC (permalink / raw) To: Mark Brown Cc: Jan Glauber, linux-kernel, linux-spi, David Daney, Steven J . Hill On 08/01/2016 10:28 AM, Mark Brown wrote: > On Thu, Jul 28, 2016 at 10:31:44AM +0200, Jan Glauber wrote: > >> +config SPI_THUNDERX >> + tristate "Cavium ThunderX SPI controller" >> + depends on (ARM64 || CONFIG_TEST) && 64BIT && PCI > > You mean COMPILE_TEST. Yes, we will fix that typo. > >> + p->clk = devm_clk_get(dev, NULL); >> + if (IS_ERR(p->clk)) >> + goto out_unmap; > > We're now just using the normal clock API which is good but I'm now > unclear what is going to ensure that the clock is there - is there some > other change elsewhere that I'm not aware of? The clock is an integral part of the SoC and is always running, so it will always be there. All we want to know is the frequency, which is supplied by the device tree clock-bindings framework > We're also not passing > the resulting error code back to the caller which will break deferred > probe. > Yes, we should do that. >> +out_clock: >> + clk_disable_unprepare(p->clk); >> +out_clock_devm: >> + devm_clk_put(dev, p->clk); > > There's no point in using managed allocations if you're going to manually > free things... Yes, we should let the automatic cleanup do its work here. Probably we should consider using pcim_iomap(...) as well > ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH v2 2/2] spi: octeon: Add thunderx driver @ 2016-08-01 18:49 ` Mark Brown 0 siblings, 0 replies; 17+ messages in thread From: Mark Brown @ 2016-08-01 18:49 UTC (permalink / raw) To: David Daney Cc: Jan Glauber, linux-kernel, linux-spi, David Daney, Steven J . Hill [-- Attachment #1: Type: text/plain, Size: 756 bytes --] On Mon, Aug 01, 2016 at 11:31:43AM -0700, David Daney wrote: > On 08/01/2016 10:28 AM, Mark Brown wrote: > > On Thu, Jul 28, 2016 at 10:31:44AM +0200, Jan Glauber wrote: > > > + p->clk = devm_clk_get(dev, NULL); > > > + if (IS_ERR(p->clk)) > > > + goto out_unmap; > > We're now just using the normal clock API which is good but I'm now > > unclear what is going to ensure that the clock is there - is there some > > other change elsewhere that I'm not aware of? > The clock is an integral part of the SoC and is always running, so it will > always be there. All we want to know is the frequency, which is supplied by > the device tree clock-bindings framework So there's something there that registers the clock? What is that thing on ACPI systems? [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 473 bytes --] ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH v2 2/2] spi: octeon: Add thunderx driver @ 2016-08-01 18:49 ` Mark Brown 0 siblings, 0 replies; 17+ messages in thread From: Mark Brown @ 2016-08-01 18:49 UTC (permalink / raw) To: David Daney Cc: Jan Glauber, linux-kernel-u79uwXL29TY76Z2rM5mHXA, linux-spi-u79uwXL29TY76Z2rM5mHXA, David Daney, Steven J . Hill [-- Attachment #1: Type: text/plain, Size: 756 bytes --] On Mon, Aug 01, 2016 at 11:31:43AM -0700, David Daney wrote: > On 08/01/2016 10:28 AM, Mark Brown wrote: > > On Thu, Jul 28, 2016 at 10:31:44AM +0200, Jan Glauber wrote: > > > + p->clk = devm_clk_get(dev, NULL); > > > + if (IS_ERR(p->clk)) > > > + goto out_unmap; > > We're now just using the normal clock API which is good but I'm now > > unclear what is going to ensure that the clock is there - is there some > > other change elsewhere that I'm not aware of? > The clock is an integral part of the SoC and is always running, so it will > always be there. All we want to know is the frequency, which is supplied by > the device tree clock-bindings framework So there's something there that registers the clock? What is that thing on ACPI systems? [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 473 bytes --] ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH v2 2/2] spi: octeon: Add thunderx driver @ 2016-08-01 19:02 ` David Daney 0 siblings, 0 replies; 17+ messages in thread From: David Daney @ 2016-08-01 19:02 UTC (permalink / raw) To: Mark Brown Cc: Jan Glauber, linux-kernel, linux-spi, David Daney, Steven J . Hill On 08/01/2016 11:49 AM, Mark Brown wrote: > On Mon, Aug 01, 2016 at 11:31:43AM -0700, David Daney wrote: >> On 08/01/2016 10:28 AM, Mark Brown wrote: >>> On Thu, Jul 28, 2016 at 10:31:44AM +0200, Jan Glauber wrote: > >>>> + p->clk = devm_clk_get(dev, NULL); >>>> + if (IS_ERR(p->clk)) >>>> + goto out_unmap; > >>> We're now just using the normal clock API which is good but I'm now >>> unclear what is going to ensure that the clock is there - is there some >>> other change elsewhere that I'm not aware of? > >> The clock is an integral part of the SoC and is always running, so it will >> always be there. All we want to know is the frequency, which is supplied by >> the device tree clock-bindings framework > > So there's something there that registers the clock? Yes, when using OF device tree, standard device probing registers the clock. > What is that thing on ACPI systems? I don't know if it works ACPI, or if ACPI even has support for the clock framework. But does it matter? We are not currently using ACPI on systems where this driver is used. In the future, if we ever need ACPI support, we will add support for it. David. ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH v2 2/2] spi: octeon: Add thunderx driver @ 2016-08-01 19:02 ` David Daney 0 siblings, 0 replies; 17+ messages in thread From: David Daney @ 2016-08-01 19:02 UTC (permalink / raw) To: Mark Brown Cc: Jan Glauber, linux-kernel-u79uwXL29TY76Z2rM5mHXA, linux-spi-u79uwXL29TY76Z2rM5mHXA, David Daney, Steven J . Hill On 08/01/2016 11:49 AM, Mark Brown wrote: > On Mon, Aug 01, 2016 at 11:31:43AM -0700, David Daney wrote: >> On 08/01/2016 10:28 AM, Mark Brown wrote: >>> On Thu, Jul 28, 2016 at 10:31:44AM +0200, Jan Glauber wrote: > >>>> + p->clk = devm_clk_get(dev, NULL); >>>> + if (IS_ERR(p->clk)) >>>> + goto out_unmap; > >>> We're now just using the normal clock API which is good but I'm now >>> unclear what is going to ensure that the clock is there - is there some >>> other change elsewhere that I'm not aware of? > >> The clock is an integral part of the SoC and is always running, so it will >> always be there. All we want to know is the frequency, which is supplied by >> the device tree clock-bindings framework > > So there's something there that registers the clock? Yes, when using OF device tree, standard device probing registers the clock. > What is that thing on ACPI systems? I don't know if it works ACPI, or if ACPI even has support for the clock framework. But does it matter? We are not currently using ACPI on systems where this driver is used. In the future, if we ever need ACPI support, we will add support for it. David. -- To unsubscribe from this list: send the line "unsubscribe linux-spi" 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] 17+ messages in thread
* Re: [PATCH v2 2/2] spi: octeon: Add thunderx driver @ 2016-08-02 21:30 ` Mark Brown 0 siblings, 0 replies; 17+ messages in thread From: Mark Brown @ 2016-08-02 21:30 UTC (permalink / raw) To: David Daney Cc: Jan Glauber, linux-kernel, linux-spi, David Daney, Steven J . Hill [-- Attachment #1: Type: text/plain, Size: 599 bytes --] On Mon, Aug 01, 2016 at 12:02:54PM -0700, David Daney wrote: > On 08/01/2016 11:49 AM, Mark Brown wrote: > > What is that thing on ACPI systems? > I don't know if it works ACPI, or if ACPI even has support for the clock > framework. But does it matter? We are not currently using ACPI on systems > where this driver is used. > In the future, if we ever need ACPI support, we will add support for it. Oh, that's surprising - I thought these were server systems. With ACPI you need to use DMI data or something to instantiate the clock. In any case if you've got it working that should be OK. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 473 bytes --] ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH v2 2/2] spi: octeon: Add thunderx driver @ 2016-08-02 21:30 ` Mark Brown 0 siblings, 0 replies; 17+ messages in thread From: Mark Brown @ 2016-08-02 21:30 UTC (permalink / raw) To: David Daney Cc: Jan Glauber, linux-kernel-u79uwXL29TY76Z2rM5mHXA, linux-spi-u79uwXL29TY76Z2rM5mHXA, David Daney, Steven J . Hill [-- Attachment #1: Type: text/plain, Size: 599 bytes --] On Mon, Aug 01, 2016 at 12:02:54PM -0700, David Daney wrote: > On 08/01/2016 11:49 AM, Mark Brown wrote: > > What is that thing on ACPI systems? > I don't know if it works ACPI, or if ACPI even has support for the clock > framework. But does it matter? We are not currently using ACPI on systems > where this driver is used. > In the future, if we ever need ACPI support, we will add support for it. Oh, that's surprising - I thought these were server systems. With ACPI you need to use DMI data or something to instantiate the clock. In any case if you've got it working that should be OK. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 473 bytes --] ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH v2 2/2] spi: octeon: Add thunderx driver 2016-08-02 21:30 ` Mark Brown (?) @ 2016-08-02 21:49 ` David Daney -1 siblings, 0 replies; 17+ messages in thread From: David Daney @ 2016-08-02 21:49 UTC (permalink / raw) To: Mark Brown Cc: Jan Glauber, linux-kernel, linux-spi, David Daney, Steven J . Hill On 08/02/2016 02:30 PM, Mark Brown wrote: > On Mon, Aug 01, 2016 at 12:02:54PM -0700, David Daney wrote: >> On 08/01/2016 11:49 AM, Mark Brown wrote: > >>> What is that thing on ACPI systems? > >> I don't know if it works ACPI, or if ACPI even has support for the clock >> framework. But does it matter? We are not currently using ACPI on systems >> where this driver is used. > >> In the future, if we ever need ACPI support, we will add support for it. > > Oh, that's surprising - I thought these were server systems. There are two broad classes of systems targeted by Cavium's arm64 based SoCs: 1) 2-node NUMA servers with 96 CPUs running UEFI firmware and ACPI. For these, any SPI buses are managed by the firmware and we don't need Kernel support. 2) Embedded controllers with 4 - 32 CPUs, typically running u-boot and OF device tree firmware descriptions. For these configurations, SPI buses managed by the kernel must be supported. > With ACPI > you need to use DMI data or something to instantiate the clock. In any > case if you've got it working that should be OK. > ^ permalink raw reply [flat|nested] 17+ messages in thread
end of thread, other threads:[~2016-08-02 22:07 UTC | newest] Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2016-07-28 8:31 [PATCH v2 0/2] SPI ThunderX driver Jan Glauber 2016-07-28 8:31 ` Jan Glauber 2016-07-28 8:31 ` [PATCH v2 1/2] spi: octeon: Split driver into Octeon specific and common parts Jan Glauber 2016-08-01 17:36 ` Mark Brown 2016-08-01 17:36 ` Mark Brown 2016-07-28 8:31 ` [PATCH v2 2/2] spi: octeon: Add thunderx driver Jan Glauber 2016-07-28 8:31 ` Jan Glauber 2016-08-01 17:28 ` Mark Brown 2016-08-01 17:28 ` Mark Brown 2016-08-01 18:31 ` David Daney 2016-08-01 18:49 ` Mark Brown 2016-08-01 18:49 ` Mark Brown 2016-08-01 19:02 ` David Daney 2016-08-01 19:02 ` David Daney 2016-08-02 21:30 ` Mark Brown 2016-08-02 21:30 ` Mark Brown 2016-08-02 21:49 ` David Daney
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.