From: Randy Dunlap <rdunlap@infradead.org>
To: sam@ravnborg.org, "Maciej W. Rozycki" <macro@orcam.me.uk>,
sparclinux@vger.kernel.org, Andreas Larsson <andreas@gaisler.com>
Cc: Miquel Raynal <miquel.raynal@bootlin.com>,
linux-parport@lists.infradead.org,
"David S. Miller" <davem@davemloft.net>,
Arnd Bergmann <arnd@arndb.de>,
linux-kernel@vger.kernel.org, stable@vger.kernel.org
Subject: Re: [PATCH v2 6/7] sparc32: Fix parport build with sparc32
Date: Sat, 2 Mar 2024 15:52:02 -0800 [thread overview]
Message-ID: <482c88c9-1598-4f51-b2ee-3a8e224eeefe@infradead.org> (raw)
In-Reply-To: <20240224-sam-fix-sparc32-all-builds-v2-6-1f186603c5c4@ravnborg.org>
On 2/24/24 09:42, Sam Ravnborg via B4 Relay wrote:
> From: Sam Ravnborg <sam@ravnborg.org>
>
> include/asm/parport.h is sparc64 specific.
> Rename it to parport_64.h and use the generic version for sparc32.
>
> This fixed all{mod,yes}config build errors like:
>
> parport_pc.c:(.text):undefined-reference-to-ebus_dma_enable
> parport_pc.c:(.text):undefined-reference-to-ebus_dma_irq_enable
> parport_pc.c:(.text):undefined-reference-to-ebus_dma_register
>
> The errors occur as the sparc32 build references sparc64 symbols.
>
> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> Cc: "David S. Miller" <davem@davemloft.net>
> Cc: Andreas Larsson <andreas@gaisler.com>
> Cc: Randy Dunlap <rdunlap@infradead.org>
> Cc: Maciej W. Rozycki <macro@orcam.me.uk>
> Closes: https://lore.kernel.org/r/20230406160548.25721-1-rdunlap@infradead.org/
> Fixes: 66bcd06099bb ("parport_pc: Also enable driver for PCI systems")
> Cc: stable@vger.kernel.org # v5.18+
Tested-by: Randy Dunlap <rdunlap@infradead.org> # build-tested
Thanks.
> ---
> arch/sparc/include/asm/parport.h | 259 +-----------------------------------
> arch/sparc/include/asm/parport_64.h | 256 +++++++++++++++++++++++++++++++++++
> 2 files changed, 263 insertions(+), 252 deletions(-)
>
> diff --git a/arch/sparc/include/asm/parport.h b/arch/sparc/include/asm/parport.h
> index 0a7ffcfd59cd..e2eed8f97665 100644
> --- a/arch/sparc/include/asm/parport.h
> +++ b/arch/sparc/include/asm/parport.h
> @@ -1,256 +1,11 @@
> /* SPDX-License-Identifier: GPL-2.0 */
> -/* parport.h: sparc64 specific parport initialization and dma.
> - *
> - * Copyright (C) 1999 Eddie C. Dost (ecd@skynet.be)
> - */
> +#ifndef ___ASM_SPARC_PARPORT_H
> +#define ___ASM_SPARC_PARPORT_H
>
> -#ifndef _ASM_SPARC64_PARPORT_H
> -#define _ASM_SPARC64_PARPORT_H 1
> -
> -#include <linux/of.h>
> -#include <linux/platform_device.h>
> -
> -#include <asm/ebus_dma.h>
> -#include <asm/ns87303.h>
> -#include <asm/prom.h>
> -
> -#define PARPORT_PC_MAX_PORTS PARPORT_MAX
> -
> -/*
> - * While sparc64 doesn't have an ISA DMA API, we provide something that looks
> - * close enough to make parport_pc happy
> - */
> -#define HAS_DMA
> -
> -#ifdef CONFIG_PARPORT_PC_FIFO
> -static DEFINE_SPINLOCK(dma_spin_lock);
> -
> -#define claim_dma_lock() \
> -({ unsigned long flags; \
> - spin_lock_irqsave(&dma_spin_lock, flags); \
> - flags; \
> -})
> -
> -#define release_dma_lock(__flags) \
> - spin_unlock_irqrestore(&dma_spin_lock, __flags);
> +#if defined(__sparc__) && defined(__arch64__)
> +#include <asm/parport_64.h>
> +#else
> +#include <asm-generic/parport.h>
> +#endif
> #endif
>
> -static struct sparc_ebus_info {
> - struct ebus_dma_info info;
> - unsigned int addr;
> - unsigned int count;
> - int lock;
> -
> - struct parport *port;
> -} sparc_ebus_dmas[PARPORT_PC_MAX_PORTS];
> -
> -static DECLARE_BITMAP(dma_slot_map, PARPORT_PC_MAX_PORTS);
> -
> -static inline int request_dma(unsigned int dmanr, const char *device_id)
> -{
> - if (dmanr >= PARPORT_PC_MAX_PORTS)
> - return -EINVAL;
> - if (xchg(&sparc_ebus_dmas[dmanr].lock, 1) != 0)
> - return -EBUSY;
> - return 0;
> -}
> -
> -static inline void free_dma(unsigned int dmanr)
> -{
> - if (dmanr >= PARPORT_PC_MAX_PORTS) {
> - printk(KERN_WARNING "Trying to free DMA%d\n", dmanr);
> - return;
> - }
> - if (xchg(&sparc_ebus_dmas[dmanr].lock, 0) == 0) {
> - printk(KERN_WARNING "Trying to free free DMA%d\n", dmanr);
> - return;
> - }
> -}
> -
> -static inline void enable_dma(unsigned int dmanr)
> -{
> - ebus_dma_enable(&sparc_ebus_dmas[dmanr].info, 1);
> -
> - if (ebus_dma_request(&sparc_ebus_dmas[dmanr].info,
> - sparc_ebus_dmas[dmanr].addr,
> - sparc_ebus_dmas[dmanr].count))
> - BUG();
> -}
> -
> -static inline void disable_dma(unsigned int dmanr)
> -{
> - ebus_dma_enable(&sparc_ebus_dmas[dmanr].info, 0);
> -}
> -
> -static inline void clear_dma_ff(unsigned int dmanr)
> -{
> - /* nothing */
> -}
> -
> -static inline void set_dma_mode(unsigned int dmanr, char mode)
> -{
> - ebus_dma_prepare(&sparc_ebus_dmas[dmanr].info, (mode != DMA_MODE_WRITE));
> -}
> -
> -static inline void set_dma_addr(unsigned int dmanr, unsigned int addr)
> -{
> - sparc_ebus_dmas[dmanr].addr = addr;
> -}
> -
> -static inline void set_dma_count(unsigned int dmanr, unsigned int count)
> -{
> - sparc_ebus_dmas[dmanr].count = count;
> -}
> -
> -static inline unsigned int get_dma_residue(unsigned int dmanr)
> -{
> - return ebus_dma_residue(&sparc_ebus_dmas[dmanr].info);
> -}
> -
> -static int ecpp_probe(struct platform_device *op)
> -{
> - unsigned long base = op->resource[0].start;
> - unsigned long config = op->resource[1].start;
> - unsigned long d_base = op->resource[2].start;
> - unsigned long d_len;
> - struct device_node *parent;
> - struct parport *p;
> - int slot, err;
> -
> - parent = op->dev.of_node->parent;
> - if (of_node_name_eq(parent, "dma")) {
> - p = parport_pc_probe_port(base, base + 0x400,
> - op->archdata.irqs[0], PARPORT_DMA_NOFIFO,
> - op->dev.parent->parent, 0);
> - if (!p)
> - return -ENOMEM;
> - dev_set_drvdata(&op->dev, p);
> - return 0;
> - }
> -
> - for (slot = 0; slot < PARPORT_PC_MAX_PORTS; slot++) {
> - if (!test_and_set_bit(slot, dma_slot_map))
> - break;
> - }
> - err = -ENODEV;
> - if (slot >= PARPORT_PC_MAX_PORTS)
> - goto out_err;
> -
> - spin_lock_init(&sparc_ebus_dmas[slot].info.lock);
> -
> - d_len = (op->resource[2].end - d_base) + 1UL;
> - sparc_ebus_dmas[slot].info.regs =
> - of_ioremap(&op->resource[2], 0, d_len, "ECPP DMA");
> -
> - if (!sparc_ebus_dmas[slot].info.regs)
> - goto out_clear_map;
> -
> - sparc_ebus_dmas[slot].info.flags = 0;
> - sparc_ebus_dmas[slot].info.callback = NULL;
> - sparc_ebus_dmas[slot].info.client_cookie = NULL;
> - sparc_ebus_dmas[slot].info.irq = 0xdeadbeef;
> - strcpy(sparc_ebus_dmas[slot].info.name, "parport");
> - if (ebus_dma_register(&sparc_ebus_dmas[slot].info))
> - goto out_unmap_regs;
> -
> - ebus_dma_irq_enable(&sparc_ebus_dmas[slot].info, 1);
> -
> - /* Configure IRQ to Push Pull, Level Low */
> - /* Enable ECP, set bit 2 of the CTR first */
> - outb(0x04, base + 0x02);
> - ns87303_modify(config, PCR,
> - PCR_EPP_ENABLE |
> - PCR_IRQ_ODRAIN,
> - PCR_ECP_ENABLE |
> - PCR_ECP_CLK_ENA |
> - PCR_IRQ_POLAR);
> -
> - /* CTR bit 5 controls direction of port */
> - ns87303_modify(config, PTR,
> - 0, PTR_LPT_REG_DIR);
> -
> - p = parport_pc_probe_port(base, base + 0x400,
> - op->archdata.irqs[0],
> - slot,
> - op->dev.parent,
> - 0);
> - err = -ENOMEM;
> - if (!p)
> - goto out_disable_irq;
> -
> - dev_set_drvdata(&op->dev, p);
> -
> - return 0;
> -
> -out_disable_irq:
> - ebus_dma_irq_enable(&sparc_ebus_dmas[slot].info, 0);
> - ebus_dma_unregister(&sparc_ebus_dmas[slot].info);
> -
> -out_unmap_regs:
> - of_iounmap(&op->resource[2], sparc_ebus_dmas[slot].info.regs, d_len);
> -
> -out_clear_map:
> - clear_bit(slot, dma_slot_map);
> -
> -out_err:
> - return err;
> -}
> -
> -static int ecpp_remove(struct platform_device *op)
> -{
> - struct parport *p = dev_get_drvdata(&op->dev);
> - int slot = p->dma;
> -
> - parport_pc_unregister_port(p);
> -
> - if (slot != PARPORT_DMA_NOFIFO) {
> - unsigned long d_base = op->resource[2].start;
> - unsigned long d_len;
> -
> - d_len = (op->resource[2].end - d_base) + 1UL;
> -
> - ebus_dma_irq_enable(&sparc_ebus_dmas[slot].info, 0);
> - ebus_dma_unregister(&sparc_ebus_dmas[slot].info);
> - of_iounmap(&op->resource[2],
> - sparc_ebus_dmas[slot].info.regs,
> - d_len);
> - clear_bit(slot, dma_slot_map);
> - }
> -
> - return 0;
> -}
> -
> -static const struct of_device_id ecpp_match[] = {
> - {
> - .name = "ecpp",
> - },
> - {
> - .name = "parallel",
> - .compatible = "ecpp",
> - },
> - {
> - .name = "parallel",
> - .compatible = "ns87317-ecpp",
> - },
> - {
> - .name = "parallel",
> - .compatible = "pnpALI,1533,3",
> - },
> - {},
> -};
> -
> -static struct platform_driver ecpp_driver = {
> - .driver = {
> - .name = "ecpp",
> - .of_match_table = ecpp_match,
> - },
> - .probe = ecpp_probe,
> - .remove = ecpp_remove,
> -};
> -
> -static int parport_pc_find_nonpci_ports(int autoirq, int autodma)
> -{
> - return platform_driver_register(&ecpp_driver);
> -}
> -
> -#endif /* !(_ASM_SPARC64_PARPORT_H */
> diff --git a/arch/sparc/include/asm/parport_64.h b/arch/sparc/include/asm/parport_64.h
> new file mode 100644
> index 000000000000..0a7ffcfd59cd
> --- /dev/null
> +++ b/arch/sparc/include/asm/parport_64.h
> @@ -0,0 +1,256 @@
> +/* SPDX-License-Identifier: GPL-2.0 */
> +/* parport.h: sparc64 specific parport initialization and dma.
> + *
> + * Copyright (C) 1999 Eddie C. Dost (ecd@skynet.be)
> + */
> +
> +#ifndef _ASM_SPARC64_PARPORT_H
> +#define _ASM_SPARC64_PARPORT_H 1
> +
> +#include <linux/of.h>
> +#include <linux/platform_device.h>
> +
> +#include <asm/ebus_dma.h>
> +#include <asm/ns87303.h>
> +#include <asm/prom.h>
> +
> +#define PARPORT_PC_MAX_PORTS PARPORT_MAX
> +
> +/*
> + * While sparc64 doesn't have an ISA DMA API, we provide something that looks
> + * close enough to make parport_pc happy
> + */
> +#define HAS_DMA
> +
> +#ifdef CONFIG_PARPORT_PC_FIFO
> +static DEFINE_SPINLOCK(dma_spin_lock);
> +
> +#define claim_dma_lock() \
> +({ unsigned long flags; \
> + spin_lock_irqsave(&dma_spin_lock, flags); \
> + flags; \
> +})
> +
> +#define release_dma_lock(__flags) \
> + spin_unlock_irqrestore(&dma_spin_lock, __flags);
> +#endif
> +
> +static struct sparc_ebus_info {
> + struct ebus_dma_info info;
> + unsigned int addr;
> + unsigned int count;
> + int lock;
> +
> + struct parport *port;
> +} sparc_ebus_dmas[PARPORT_PC_MAX_PORTS];
> +
> +static DECLARE_BITMAP(dma_slot_map, PARPORT_PC_MAX_PORTS);
> +
> +static inline int request_dma(unsigned int dmanr, const char *device_id)
> +{
> + if (dmanr >= PARPORT_PC_MAX_PORTS)
> + return -EINVAL;
> + if (xchg(&sparc_ebus_dmas[dmanr].lock, 1) != 0)
> + return -EBUSY;
> + return 0;
> +}
> +
> +static inline void free_dma(unsigned int dmanr)
> +{
> + if (dmanr >= PARPORT_PC_MAX_PORTS) {
> + printk(KERN_WARNING "Trying to free DMA%d\n", dmanr);
> + return;
> + }
> + if (xchg(&sparc_ebus_dmas[dmanr].lock, 0) == 0) {
> + printk(KERN_WARNING "Trying to free free DMA%d\n", dmanr);
> + return;
> + }
> +}
> +
> +static inline void enable_dma(unsigned int dmanr)
> +{
> + ebus_dma_enable(&sparc_ebus_dmas[dmanr].info, 1);
> +
> + if (ebus_dma_request(&sparc_ebus_dmas[dmanr].info,
> + sparc_ebus_dmas[dmanr].addr,
> + sparc_ebus_dmas[dmanr].count))
> + BUG();
> +}
> +
> +static inline void disable_dma(unsigned int dmanr)
> +{
> + ebus_dma_enable(&sparc_ebus_dmas[dmanr].info, 0);
> +}
> +
> +static inline void clear_dma_ff(unsigned int dmanr)
> +{
> + /* nothing */
> +}
> +
> +static inline void set_dma_mode(unsigned int dmanr, char mode)
> +{
> + ebus_dma_prepare(&sparc_ebus_dmas[dmanr].info, (mode != DMA_MODE_WRITE));
> +}
> +
> +static inline void set_dma_addr(unsigned int dmanr, unsigned int addr)
> +{
> + sparc_ebus_dmas[dmanr].addr = addr;
> +}
> +
> +static inline void set_dma_count(unsigned int dmanr, unsigned int count)
> +{
> + sparc_ebus_dmas[dmanr].count = count;
> +}
> +
> +static inline unsigned int get_dma_residue(unsigned int dmanr)
> +{
> + return ebus_dma_residue(&sparc_ebus_dmas[dmanr].info);
> +}
> +
> +static int ecpp_probe(struct platform_device *op)
> +{
> + unsigned long base = op->resource[0].start;
> + unsigned long config = op->resource[1].start;
> + unsigned long d_base = op->resource[2].start;
> + unsigned long d_len;
> + struct device_node *parent;
> + struct parport *p;
> + int slot, err;
> +
> + parent = op->dev.of_node->parent;
> + if (of_node_name_eq(parent, "dma")) {
> + p = parport_pc_probe_port(base, base + 0x400,
> + op->archdata.irqs[0], PARPORT_DMA_NOFIFO,
> + op->dev.parent->parent, 0);
> + if (!p)
> + return -ENOMEM;
> + dev_set_drvdata(&op->dev, p);
> + return 0;
> + }
> +
> + for (slot = 0; slot < PARPORT_PC_MAX_PORTS; slot++) {
> + if (!test_and_set_bit(slot, dma_slot_map))
> + break;
> + }
> + err = -ENODEV;
> + if (slot >= PARPORT_PC_MAX_PORTS)
> + goto out_err;
> +
> + spin_lock_init(&sparc_ebus_dmas[slot].info.lock);
> +
> + d_len = (op->resource[2].end - d_base) + 1UL;
> + sparc_ebus_dmas[slot].info.regs =
> + of_ioremap(&op->resource[2], 0, d_len, "ECPP DMA");
> +
> + if (!sparc_ebus_dmas[slot].info.regs)
> + goto out_clear_map;
> +
> + sparc_ebus_dmas[slot].info.flags = 0;
> + sparc_ebus_dmas[slot].info.callback = NULL;
> + sparc_ebus_dmas[slot].info.client_cookie = NULL;
> + sparc_ebus_dmas[slot].info.irq = 0xdeadbeef;
> + strcpy(sparc_ebus_dmas[slot].info.name, "parport");
> + if (ebus_dma_register(&sparc_ebus_dmas[slot].info))
> + goto out_unmap_regs;
> +
> + ebus_dma_irq_enable(&sparc_ebus_dmas[slot].info, 1);
> +
> + /* Configure IRQ to Push Pull, Level Low */
> + /* Enable ECP, set bit 2 of the CTR first */
> + outb(0x04, base + 0x02);
> + ns87303_modify(config, PCR,
> + PCR_EPP_ENABLE |
> + PCR_IRQ_ODRAIN,
> + PCR_ECP_ENABLE |
> + PCR_ECP_CLK_ENA |
> + PCR_IRQ_POLAR);
> +
> + /* CTR bit 5 controls direction of port */
> + ns87303_modify(config, PTR,
> + 0, PTR_LPT_REG_DIR);
> +
> + p = parport_pc_probe_port(base, base + 0x400,
> + op->archdata.irqs[0],
> + slot,
> + op->dev.parent,
> + 0);
> + err = -ENOMEM;
> + if (!p)
> + goto out_disable_irq;
> +
> + dev_set_drvdata(&op->dev, p);
> +
> + return 0;
> +
> +out_disable_irq:
> + ebus_dma_irq_enable(&sparc_ebus_dmas[slot].info, 0);
> + ebus_dma_unregister(&sparc_ebus_dmas[slot].info);
> +
> +out_unmap_regs:
> + of_iounmap(&op->resource[2], sparc_ebus_dmas[slot].info.regs, d_len);
> +
> +out_clear_map:
> + clear_bit(slot, dma_slot_map);
> +
> +out_err:
> + return err;
> +}
> +
> +static int ecpp_remove(struct platform_device *op)
> +{
> + struct parport *p = dev_get_drvdata(&op->dev);
> + int slot = p->dma;
> +
> + parport_pc_unregister_port(p);
> +
> + if (slot != PARPORT_DMA_NOFIFO) {
> + unsigned long d_base = op->resource[2].start;
> + unsigned long d_len;
> +
> + d_len = (op->resource[2].end - d_base) + 1UL;
> +
> + ebus_dma_irq_enable(&sparc_ebus_dmas[slot].info, 0);
> + ebus_dma_unregister(&sparc_ebus_dmas[slot].info);
> + of_iounmap(&op->resource[2],
> + sparc_ebus_dmas[slot].info.regs,
> + d_len);
> + clear_bit(slot, dma_slot_map);
> + }
> +
> + return 0;
> +}
> +
> +static const struct of_device_id ecpp_match[] = {
> + {
> + .name = "ecpp",
> + },
> + {
> + .name = "parallel",
> + .compatible = "ecpp",
> + },
> + {
> + .name = "parallel",
> + .compatible = "ns87317-ecpp",
> + },
> + {
> + .name = "parallel",
> + .compatible = "pnpALI,1533,3",
> + },
> + {},
> +};
> +
> +static struct platform_driver ecpp_driver = {
> + .driver = {
> + .name = "ecpp",
> + .of_match_table = ecpp_match,
> + },
> + .probe = ecpp_probe,
> + .remove = ecpp_remove,
> +};
> +
> +static int parport_pc_find_nonpci_ports(int autoirq, int autodma)
> +{
> + return platform_driver_register(&ecpp_driver);
> +}
> +
> +#endif /* !(_ASM_SPARC64_PARPORT_H */
>
--
#Randy
next prev parent reply other threads:[~2024-03-02 23:52 UTC|newest]
Thread overview: 47+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-02-24 17:42 [PATCH v2 0/7] sparc32: build fixes for all{yes,mod}config builds Sam Ravnborg via B4 Relay
2024-02-24 17:42 ` Sam Ravnborg
2024-02-24 17:42 ` [PATCH v2 1/7] sparc32: Use generic cmpdi2/ucmpdi2 variants Sam Ravnborg via B4 Relay
2024-02-24 17:42 ` Sam Ravnborg
2024-03-08 20:00 ` Andreas Larsson
2024-02-24 17:42 ` [PATCH v2 2/7] sparc32: Fix build with trapbase Sam Ravnborg via B4 Relay
2024-02-24 17:42 ` Sam Ravnborg
2024-03-08 20:01 ` Andreas Larsson
2024-02-24 17:42 ` [PATCH v2 3/7] mtd: maps: sun_uflash: Declare uflash_devinit static Sam Ravnborg via B4 Relay
2024-02-24 17:42 ` Sam Ravnborg
2024-02-26 10:23 ` Miquel Raynal
2024-03-08 20:02 ` Andreas Larsson
2024-02-24 17:42 ` [PATCH v2 4/7] sparc32: Do not select ZONE_DMA Sam Ravnborg via B4 Relay
2024-02-24 17:42 ` Sam Ravnborg
2024-02-25 0:57 ` Maciej W. Rozycki
2024-02-25 7:08 ` Sam Ravnborg
2024-03-02 23:51 ` Randy Dunlap
2024-03-05 15:06 ` Andreas Larsson
2024-03-05 15:26 ` Arnd Bergmann
2024-03-06 14:19 ` Andreas Larsson
2024-03-06 14:45 ` Arnd Bergmann
2024-03-06 15:31 ` Andreas Larsson
2024-03-06 16:22 ` Arnd Bergmann
2024-03-06 18:19 ` Arnd Bergmann
2024-03-07 12:42 ` Andreas Larsson
2024-03-07 13:06 ` Arnd Bergmann
2024-03-06 15:04 ` Christoph Hellwig
2024-03-06 15:24 ` Arnd Bergmann
2024-02-24 17:42 ` [PATCH v2 5/7] sparc32: Do not select GENERIC_ISA_DMA Sam Ravnborg via B4 Relay
2024-02-24 17:42 ` Sam Ravnborg
2024-03-08 20:03 ` Andreas Larsson
2024-03-09 9:04 ` Maciej W. Rozycki
2024-02-24 17:42 ` [PATCH v2 6/7] sparc32: Fix parport build with sparc32 Sam Ravnborg via B4 Relay
2024-02-24 17:42 ` Sam Ravnborg
2024-03-02 23:52 ` Randy Dunlap [this message]
2024-03-08 20:04 ` Andreas Larsson
2024-03-09 9:06 ` Maciej W. Rozycki
2024-02-24 17:42 ` [PATCH v2 7/7] sparc32: Fix section mismatch in leon_pci_grpci Sam Ravnborg via B4 Relay
2024-02-24 17:42 ` Sam Ravnborg
2024-03-02 23:54 ` Randy Dunlap
2024-03-05 15:06 ` Andreas Larsson
2024-03-07 18:20 ` Andreas Larsson
2024-03-07 20:16 ` Maciej W. Rozycki
2024-03-08 7:06 ` Andreas Larsson
2024-03-08 12:09 ` Maciej W. Rozycki
2024-03-08 22:08 ` Sam Ravnborg
2024-03-08 20:07 ` Andreas Larsson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=482c88c9-1598-4f51-b2ee-3a8e224eeefe@infradead.org \
--to=rdunlap@infradead.org \
--cc=andreas@gaisler.com \
--cc=arnd@arndb.de \
--cc=davem@davemloft.net \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-parport@lists.infradead.org \
--cc=macro@orcam.me.uk \
--cc=miquel.raynal@bootlin.com \
--cc=sam@ravnborg.org \
--cc=sparclinux@vger.kernel.org \
--cc=stable@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.