[RESEND,07/12] pci: sh: SH7751 PCI host bridge driver

Signed-off-by: Yoshinori Sato <ysato@users.sourceforge.jp>
---
 .../devicetree/bindings/pci/sh7751-pci.txt         |  51 ++++
 drivers/pci/host/Kconfig                           |   7 +
 drivers/pci/host/Makefile                          |   1 +
 drivers/pci/host/pci-sh7751.c                      | 321 +++++++++++++++++++++
 drivers/pci/host/pci-sh7751.h                      | 269 +++++++++++++++++
 5 files changed, 649 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/pci/sh7751-pci.txt
 create mode 100644 drivers/pci/host/pci-sh7751.c
 create mode 100644 drivers/pci/host/pci-sh7751.h

On Sun, May 01, 2016 at 02:08:31PM +0900, Yoshinori Sato wrote:
> Signed-off-by: Yoshinori Sato <ysato@users.sourceforge.jp>
> ---
>  .../devicetree/bindings/pci/sh7751-pci.txt         |  51 ++++
>  drivers/pci/host/Kconfig                           |   7 +
>  drivers/pci/host/Makefile                          |   1 +
>  drivers/pci/host/pci-sh7751.c                      | 321 +++++++++++++++++++++
>  drivers/pci/host/pci-sh7751.h                      | 269 +++++++++++++++++
>  5 files changed, 649 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/pci/sh7751-pci.txt
>  create mode 100644 drivers/pci/host/pci-sh7751.c
>  create mode 100644 drivers/pci/host/pci-sh7751.h
> 
> diff --git a/Documentation/devicetree/bindings/pci/sh7751-pci.txt b/Documentation/devicetree/bindings/pci/sh7751-pci.txt
> new file mode 100644
> index 0000000..45b5152
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/pci/sh7751-pci.txt
> @@ -0,0 +1,51 @@
> +* Renesas SH7751 PCI host interfaces
> +
> +Required properties:
> +  - compatible: "renesas,sh7751-pci" is required.
> +    And board specific compatible if fixup required.
> +
> +  - reg: base address and length of the pcie controller registers.

s/pcie/PCIe/

> +  - #address-cells: set to <2>
> +  - #size-cells: set to <1>
> +  - bus-range: PCI bus numbers covered
> +  - device_type: set to "pci"
> +  - ranges: ranges for the PCI memory and I/O regions.
> +  - interrupt-map-mask and interrupt-map: standard PCI properties
> +  	to define the mapping of the PCIe interface to interrupt

Whitespace error (space before tab).

> +	numbers.
> +
> +Example:
> +	pci: pci-controller@fe200000 {
> +		compatible = "renesas,sh7751-pci", "iodata,landisk";
> +		device_type = "pci";
> +		bus-range = <0 0>;
> +		#address-cells = <2>;
> +		#size-cells = <1>;
> +		ranges = <0x02000000 0x00000000 0xfd000000 0xfd000000 0x00000000 0x01000000>,
> +		         <0x01000000 0x00000000 0xfe240000 0x00000000 0x00000000 0x00040000>;
> +		reg = <0xfe200000 0x0400>,
> +		      <0x0c000000 0x04000000>,
> +		      <0xff800000 0x0030>;
> +		#interrupt-cells = <1>;
> +		interrupt-map-mask = <0x1800 0 7>;
> +		interrupt-map = <0x0000 0 1 &cpldintc evt2irq(0x2a0) 0
> +		                 0x0000 0 2 &cpldintc evt2irq(0x2c0) 0
> +		                 0x0000 0 3 &cpldintc evt2irq(0x2e0) 0
> +		                 0x0000 0 4 &cpldintc evt2irq(0x300) 0
> +
> +		                 0x0800 0 1 &cpldintc evt2irq(0x2c0) 0
> +		                 0x0800 0 2 &cpldintc evt2irq(0x2e0) 0
> +		                 0x0800 0 3 &cpldintc evt2irq(0x300) 0
> +		                 0x0800 0 4 &cpldintc evt2irq(0x2a0) 0
> +
> +		                 0x1000 0 1 &cpldintc evt2irq(0x2e0) 0
> +		                 0x1000 0 2 &cpldintc evt2irq(0x300) 0
> +		                 0x1000 0 3 &cpldintc evt2irq(0x2a0) 0
> +		                 0x1000 0 4 &cpldintc evt2irq(0x2c0) 0
> +
> +		                 0x1800 0 1 &cpldintc evt2irq(0x300) 0
> +		                 0x1800 0 2 &cpldintc evt2irq(0x2a0) 0
> +		                 0x1800 0 3 &cpldintc evt2irq(0x2c0) 0
> +		                 0x1800 0 4 &cpldintc evt2irq(0x2e0) 0>;
> +	};
> +};
> diff --git a/drivers/pci/host/Kconfig b/drivers/pci/host/Kconfig
> index 7a0780d..a8596db 100644
> --- a/drivers/pci/host/Kconfig
> +++ b/drivers/pci/host/Kconfig
> @@ -231,4 +231,11 @@ config PCI_HOST_THUNDER_ECAM
>  	help
>  	  Say Y here if you want ECAM support for CN88XX-Pass-1.x Cavium Thunder SoCs.
>  
> +config PCI_SH7751
> +	bool "Renesas SH7751 On-Chip PCI controller"
> +	depends on OF && SUPERH
> +	select PCI_HOST_COMMON
> +	help
> +	  Say Y here if you want PCI support on SH7751.
> +
>  endmenu
> diff --git a/drivers/pci/host/Makefile b/drivers/pci/host/Makefile
> index d85b5fa..91268cb 100644
> --- a/drivers/pci/host/Makefile
> +++ b/drivers/pci/host/Makefile
> @@ -28,3 +28,4 @@ obj-$(CONFIG_PCI_HISI) += pcie-hisi.o
>  obj-$(CONFIG_PCIE_QCOM) += pcie-qcom.o
>  obj-$(CONFIG_PCI_HOST_THUNDER_ECAM) += pci-thunder-ecam.o
>  obj-$(CONFIG_PCI_HOST_THUNDER_PEM) += pci-thunder-pem.o
> +obj-$(CONFIG_PCI_SH7751) += pci-sh7751.o
> diff --git a/drivers/pci/host/pci-sh7751.c b/drivers/pci/host/pci-sh7751.c
> new file mode 100644
> index 0000000..fd6548b
> --- /dev/null
> +++ b/drivers/pci/host/pci-sh7751.c
> @@ -0,0 +1,321 @@
> +/*
> + * SH7751 PCI driver
> + * Copyright (C) 2016 Yoshinori Sato
> + *
> + */
> +
> +#include <linux/kernel.h>
> +#include <linux/module.h>
> +#include <linux/of_address.h>
> +#include <linux/of_pci.h>
> +#include <linux/of_platform.h>
> +#include <linux/pci.h>
> +#include <linux/platform_device.h>
> +#include <linux/io.h>
> +#include "pci-host-common.h"
> +#include "pci-sh7751.h"
> +
> +#define pcic_writel(val, reg) __raw_writel(val, pci_reg_base + (reg))
> +#define pcic_readl(reg) __raw_readl(pci_reg_base + (reg))
> +
> +unsigned long PCIBIOS_MIN_IO;
> +unsigned long PCIBIOS_MIN_MEM;
> +DEFINE_RAW_SPINLOCK(pci_config_lock);

I think this (pci_config_lock) can be static.

> +/*
> + * PCIC fixups
> + */
> +
> +static __initconst const struct fixups {
> +	char *compatible;
> +	void (*fixup)(void __iomem *, void __iomem *);
> +} fixup_list[] = {
> +};
> +
> +static __init void pcic_fixups(struct device_node *np,
> +		       void __iomem *pcic, void __iomem *bcr)
> +{
> +	int i;
> +	const struct fixups *f = fixup_list;
> +
> +	for (i = 0; i < ARRAY_SIZE(fixup_list); i++) {
> +		if (of_device_is_compatible(np, f->compatible)) {
> +			f->fixup(pcic, bcr);
> +			break;
> +		}
> +	}
> +}

This fixup feature looks unused (fixup_list[] is empty).  Please
remove it unless it's needed.

> +
> +/*
> + * Direct access to PCI hardware...
> + */
> +#define CONFIG_CMD(bus, devfn, where) \
> +	(0x80000000 | (bus->number << 16) | (devfn << 8) | (where & ~3))
> +
> +/*
> + * Functions for accessing PCI configuration space with type 1 accesses
> + */
> +static int sh4_pci_read(struct pci_bus *bus, unsigned int devfn,
> +			   int where, int size, u32 *val)
> +{
> +	struct gen_pci *pci = bus->sysdata;
> +	void __iomem *pci_reg_base = (void __iomem *)pci->cfg.res.start;

pci->cfg.res is a struct resource, which contains CPU physical
addresses.  You would need to ioremap() those addresses before
accessing them.

> +	unsigned long flags;
> +	u32 data;
> +
> +	/*
> +	 * PCIPDR may only be accessed as 32 bit words,
> +	 * so we must do byte alignment by hand
> +	 */
> +	raw_spin_lock_irqsave(&pci_config_lock, flags);
> +	pcic_writel(CONFIG_CMD(bus, devfn, where), SH4_PCIPAR);
> +	data = pcic_readl(SH4_PCIPDR);
> +	raw_spin_unlock_irqrestore(&pci_config_lock, flags);
> +
> +	switch (size) {
> +	case 1:
> +		*val = (data >> ((where & 3) << 3)) & 0xff;
> +		break;
> +	case 2:
> +		*val = (data >> ((where & 2) << 3)) & 0xffff;
> +		break;
> +	case 4:
> +		*val = data;
> +		break;
> +	default:
> +		return PCIBIOS_FUNC_NOT_SUPPORTED;
> +	}
> +
> +	return PCIBIOS_SUCCESSFUL;
> +}
> +
> +/*
> + * Since SH4 only does 32bit access we'll have to do a read,
> + * mask,write operation.
> + * We'll allow an odd byte offset, though it should be illegal.

PCI config space allows byte accesses at any alignment.  Your hardware
is not fully PCI-compliant if it can only do 32-bit accesses.  The
read/mask/write scheme works most of the time, but it will sometimes
corrupt RW1C bits in nearby registers.

> + */
> +static int sh4_pci_write(struct pci_bus *bus, unsigned int devfn,
> +			 int where, int size, u32 val)
> +{
> +	struct gen_pci *pci = bus->sysdata;
> +	void __iomem *pci_reg_base = (void __iomem *)pci->cfg.res.start;
> +	unsigned long flags;
> +	int shift;
> +	u32 data;
> +
> +	raw_spin_lock_irqsave(&pci_config_lock, flags);
> +	pcic_writel(CONFIG_CMD(bus, devfn, where), SH4_PCIPAR);
> +	data = pcic_readl(SH4_PCIPDR);
> +	raw_spin_unlock_irqrestore(&pci_config_lock, flags);
> +
> +	switch (size) {
> +	case 1:
> +		shift = (where & 3) << 3;
> +		data &= ~(0xff << shift);
> +		data |= ((val & 0xff) << shift);
> +		break;
> +	case 2:
> +		shift = (where & 2) << 3;
> +		data &= ~(0xffff << shift);
> +		data |= ((val & 0xffff) << shift);
> +		break;
> +	case 4:
> +		data = val;
> +		break;
> +	default:
> +		return PCIBIOS_FUNC_NOT_SUPPORTED;
> +	}
> +
> +	pcic_writel(data, SH4_PCIPDR);
> +
> +	return PCIBIOS_SUCCESSFUL;
> +}
> +
> +static struct gen_pci_cfg_bus_ops pci_sh7751_ops = {
> +	.ops = {
> +		.read	= sh4_pci_read,
> +		.write	= sh4_pci_write,
> +	},
> +};
> +
> +/*
> + *  Called after each bus is probed, but before its children
> + *  are examined.

Extra spaces at the beginnings of these lines.

> + */
> +void pcibios_fixup_bus(struct pci_bus *bus)
> +{
> +}
> +
> +/*
> + * We need to avoid collisions with `mirrored' VGA ports
> + * and other strange ISA hardware, so we always want the
> + * addresses to be allocated in the 0x000-0x0ff region
> + * modulo 0x400.
> + */
> +resource_size_t pcibios_align_resource(void *data, const struct resource *res,
> +					      resource_size_t size, resource_size_t align)
> +{
> +	resource_size_t start = res->start;
> +
> +	return start;
> +}
> +
> +int pci_mmap_page_range(struct pci_dev *dev, struct vm_area_struct *vma,
> +			enum pci_mmap_state mmap_state, int write_combine)
> +{
> +	/*
> +	 * I/O space can be accessed via normal processor loads and stores on
> +	 * this platform but for now we elect not to do this and portable
> +	 * drivers should not do this anyway.
> +	 */
> +	if (mmap_state == pci_mmap_io)
> +		return -EINVAL;
> +
> +	/*
> +	 * Ignore write-combine; for now only return uncached mappings.
> +	 */
> +	vma->vm_page_prot = pgprot_noncached(vma->vm_page_prot);
> +
> +	return remap_pfn_range(vma, vma->vm_start, vma->vm_pgoff,
> +			       vma->vm_end - vma->vm_start,
> +			       vma->vm_page_prot);
> +}
> +
> +static const struct of_device_id sh7751_pci_of_match[] = {
> +	{ .compatible = "renesas,sh7751-pci", },
> +	{ },
> +};
> +MODULE_DEVICE_TABLE(of, sh7751_pci_of_match);
> +
> +static int __init area_sdram_check(void __iomem *pci_reg_base,
> +				   void __iomem *bcr,
> +				   unsigned int area)

Nobody checks the return value, so this could be void.  "check" in the
name doesn't really tell us what it does.  In fact, it looks like it
programs something.

> +{
> +	unsigned long word;
> +
> +	word = __raw_readl(bcr + SH7751_BCR1);
> +	/* check BCR for SDRAM in area */
> +	if (((word >> area) & 1) == 0) {
> +		printk("PCI: Area %d is not configured for SDRAM. BCR1=0x%lx\n",
> +		       area, word);
> +		return 0;
> +	}
> +	pcic_writel(word, SH4_PCIBCR1);
> +
> +	word = __raw_readw(bcr + SH7751_BCR2);
> +	/* check BCR2 for 32bit SDRAM interface*/
> +	if (((word >> (area << 1)) & 0x3) != 0x3) {
> +		printk("PCI: Area %d is not 32 bit SDRAM. BCR2=0x%lx\n",
> +		       area, word);
> +		return 0;
> +	}
> +	pcic_writel(word, SH4_PCIBCR2);
> +
> +	return 1;
> +}
> +
> +static __init int sh7751_pci_probe(struct platform_device *pdev)
> +{
> +	struct resource *res, *wres;
> +	u32 id;
> +	u32 reg, word;
> +	void __iomem *pci_reg_base;
> +	void __iomem *bcr;
> +	struct gen_pci *pci;
> +
> +	pci = devm_kzalloc(&pdev->dev, sizeof(*pci), GFP_KERNEL);
> +	if (!pci)
> +		return -ENOMEM;
> +
> +	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> +	pci_reg_base = (void __iomem *)res->start;

res->start is a CPU physical address.  You need to ioremap() it if you
want a virtual address.

> +	if (IS_ERR(pci_reg_base))

platform_get_resource() returns a valid struct resource * or NULL for
error.  This should check "!res" for error.  Same below for "wres".

> +		return PTR_ERR(pci_reg_base);
> +
> +	wres = platform_get_resource(pdev, IORESOURCE_MEM, 1);
> +	if (IS_ERR(wres))
> +		return PTR_ERR(wres);

Shouldn't we be requesting these two regions (0 and 1)?  Even if you
don't need to ioremap them, if the device consumes those two regions
of address space, they should be in iomem_resource so they appear in
/proc/iomem and so we don't allocate that space to anything else.

> +	res = platform_get_resource(pdev, IORESOURCE_MEM, 2);
> +	bcr = devm_ioremap_resource(&pdev->dev, res);
> +	if (IS_ERR(pci_reg_base))

This looks like a typo for "bcr"?  (s/pci_reg_base/bcr/ ?)

> +		return PTR_ERR(bcr);
> +
> +	/* check for SH7751/SH7751R hardware */
> +	id = pcic_readl(SH7751_PCICONF0);
> +	if (id != ((SH7751_DEVICE_ID << 16) | SH7751_VENDOR_ID) &&
> +	    id != ((SH7751R_DEVICE_ID << 16) | SH7751_VENDOR_ID)) {
> +		pr_warn("PCI: This is not an SH7751(R)\n");
> +		return -ENODEV;
> +	}
> +	dev_info(&pdev->dev, "PCI core found at %p\n",
> +		pci_reg_base);

Save the struct resource pointer and use %pR instead of %p.

Please add a printk like:

  dev_warn(&pdev->dev, "only 32-bit config accesses supported; smaller writes may corrupt adjacent RW1C fields\n")

> +	/* Set the BCR's to enable PCI access */

s/BCR's/BCRs/  (apostrophe is for possessives or omitted letters, not
making things plural)

> +	reg = __raw_readl(bcr);
> +	reg |= 0x80000;
> +	__raw_writel(reg, bcr);
> +
> +	/* Turn the clocks back on (not done in reset)*/
> +	pcic_writel(0, SH4_PCICLKR);
> +	/* Clear Powerdown IRQ's (not done in reset) */

s/IRQ's/IRQs/

> +	word = SH4_PCIPINT_D3 | SH4_PCIPINT_D0;
> +	pcic_writel(word, SH4_PCIPINT);
> +
> +	/* set the command/status bits to:
> +	 * Wait Cycle Control + Parity Enable + Bus Master +
> +	 * Mem space enable
> +	 */
> +	word = SH7751_PCICONF1_WCC | SH7751_PCICONF1_PER |
> +	       SH7751_PCICONF1_BUM | SH7751_PCICONF1_MES;
> +	pcic_writel(word, SH7751_PCICONF1);
> +
> +	/* define this host as the host bridge */
> +	word = PCI_BASE_CLASS_BRIDGE << 24;
> +	pcic_writel(word, SH7751_PCICONF2);
> +
> +	/* Set IO and Mem windows to local address
> +	 * Make PCI and local address the same for easy 1 to 1 mapping
> +	 */
> +	word = wres->end - wres->start - 1;
> +	pcic_writel(word, SH4_PCILSR0);
> +	/* Set the values on window 0 PCI config registers */
> +	word = P2SEGADDR(wres->start);
> +	pcic_writel(word, SH4_PCILAR0);
> +	pcic_writel(word, SH7751_PCICONF5);
> +
> +	/* check BCR for SDRAM in specified area */
> +	area_sdram_check(pci_reg_base, bcr, (wres->start >> 27) & 0x07);
> +
> +	/* configure the wait control registers */
> +	word = __raw_readl(bcr + SH7751_WCR1);
> +	pcic_writel(word, SH4_PCIWCR1);
> +	word = __raw_readl(bcr + SH7751_WCR2);
> +	pcic_writel(word, SH4_PCIWCR2);
> +	word = __raw_readl(bcr + SH7751_WCR3);
> +	pcic_writel(word, SH4_PCIWCR3);
> +	word = __raw_readl(bcr + SH7751_MCR);
> +	pcic_writel(word, SH4_PCIMCR);
> +
> +	pcic_fixups(pdev->dev.of_node, pci_reg_base, bcr);
> +
> +	/* SH7751 init done, set central function init complete */
> +	/* use round robin mode to stop a device starving/overruning */

Please use normal block comment style here and elsewhere, e.g.,

  /*
   * text...
   * more text...
   */

> +	word = SH4_PCICR_PREFIX | SH4_PCICR_CFIN | SH4_PCICR_ARBM;
> +	pcic_writel(word, SH4_PCICR);
> +
> +	pci->cfg.ops = &pci_sh7751_ops;
> +	return pci_host_common_probe(pdev, pci);
> +}
> +
> +static __refdata struct platform_driver sh7751_pci_driver = {
> +	.driver = {
> +		.name = "sh7751-pci",
> +		.of_match_table = sh7751_pci_of_match,
> +	},
> +	.probe = sh7751_pci_probe,
> +};
> +module_platform_driver(sh7751_pci_driver);
> +
> +MODULE_DESCRIPTION("SH7751 PCI driver");
> +MODULE_LICENSE("GPL v2");
> diff --git a/drivers/pci/host/pci-sh7751.h b/drivers/pci/host/pci-sh7751.h
> new file mode 100644
> index 0000000..4f8230e
> --- /dev/null
> +++ b/drivers/pci/host/pci-sh7751.h
> @@ -0,0 +1,269 @@
> +/*
> + *	Low-Level PCI Support for SH7751 targets
> + *
> + *  Dustin McIntire (dustin@sensoria.com) (c) 2001
> + *  Paul Mundt (lethal@linux-sh.org) (c) 2003
> + *
> + *  May be copied or modified under the terms of the GNU General Public
> + *  License.  See linux/COPYING for more information.
> + *
> + */

Since these definitions are only used in pci-sh7751.c, please move
them there so there's only one source file instead of two.

Many of the bitfield definitions are unused; you can also remove them.

> +#ifndef _PCI_SH7751_H_
> +#define _PCI_SH7751_H_
> +
> +#define SH4_PCICR		0x100		/* PCI Control Register */
> +  #define SH4_PCICR_PREFIX	  0xA5000000	/* CR prefix for write */
> +  #define SH4_PCICR_FTO		  0x00000400	/* TRDY/IRDY Enable */
> +  #define SH4_PCICR_TRSB	  0x00000200	/* Target Read Single */
> +  #define SH4_PCICR_BSWP	  0x00000100	/* Target Byte Swap */
> +  #define SH4_PCICR_PLUP	  0x00000080	/* Enable PCI Pullup */
> +  #define SH4_PCICR_ARBM	  0x00000040	/* PCI Arbitration Mode */
> +  #define SH4_PCICR_MD		  0x00000030	/* MD9 and MD10 status */
> +  #define SH4_PCICR_SERR	  0x00000008	/* SERR output assert */
> +  #define SH4_PCICR_INTA	  0x00000004	/* INTA output assert */
> +  #define SH4_PCICR_PRST	  0x00000002	/* PCI Reset Assert */
> +  #define SH4_PCICR_CFIN	  0x00000001	/* Central Fun. Init Done */
> +#define SH4_PCILSR0		0x104		/* PCI Local Space Register0 */
> +#define SH4_PCILSR1		0x108		/* PCI Local Space Register1 */
> +#define SH4_PCILAR0		0x10C		/* PCI Local Addr Register1 */
> +#define SH4_PCILAR1		0x110		/* PCI Local Addr Register1 */
> +#define SH4_PCIINT		0x114		/* PCI Interrupt Register */
> +  #define SH4_PCIINT_MLCK	  0x00008000	/* Master Lock Error */
> +  #define SH4_PCIINT_TABT	  0x00004000	/* Target Abort Error */
> +  #define SH4_PCIINT_TRET	  0x00000200	/* Target Retry Error */
> +  #define SH4_PCIINT_MFDE	  0x00000100	/* Master Func. Disable Error */
> +  #define SH4_PCIINT_PRTY	  0x00000080	/* Address Parity Error */
> +  #define SH4_PCIINT_SERR	  0x00000040	/* SERR Detection Error */
> +  #define SH4_PCIINT_TWDP	  0x00000020	/* Tgt. Write Parity Error */
> +  #define SH4_PCIINT_TRDP	  0x00000010	/* Tgt. Read Parity Err Det. */
> +  #define SH4_PCIINT_MTABT	  0x00000008	/* Master-Tgt. Abort Error */
> +  #define SH4_PCIINT_MMABT	  0x00000004	/* Master-Master Abort Error */
> +  #define SH4_PCIINT_MWPD	  0x00000002	/* Master Write PERR Detect */
> +  #define SH4_PCIINT_MRPD	  0x00000001	/* Master Read PERR Detect */
> +#define SH4_PCIINTM		0x118		/* PCI Interrupt Mask */
> +  #define SH4_PCIINTM_TTADIM	  BIT(14)	/* Target-target abort interrupt */
> +  #define SH4_PCIINTM_TMTOIM	  BIT(9)	/* Target retry timeout */
> +  #define SH4_PCIINTM_MDEIM	  BIT(8)	/* Master function disable error */
> +  #define SH4_PCIINTM_APEDIM	  BIT(7)	/* Address parity error detection */
> +  #define SH4_PCIINTM_SDIM	  BIT(6)	/* SERR detection */
> +  #define SH4_PCIINTM_DPEITWM	  BIT(5)	/* Data parity error for target write */
> +  #define SH4_PCIINTM_PEDITRM	  BIT(4)	/* PERR detection for target read */
> +  #define SH4_PCIINTM_TADIMM	  BIT(3)	/* Target abort for master */
> +  #define SH4_PCIINTM_MADIMM	  BIT(2)	/* Master abort for master */
> +  #define SH4_PCIINTM_MWPDIM	  BIT(1)	/* Master write data parity error */
> +  #define SH4_PCIINTM_MRDPEIM	  BIT(0)	/* Master read data parity error */
> +#define SH4_PCIALR		0x11C		/* Error Address Register */
> +#define SH4_PCICLR		0x120		/* Error Command/Data */
> +  #define SH4_PCICLR_MPIO	  0x80000000
> +  #define SH4_PCICLR_MDMA0	  0x40000000	/* DMA0 Transfer Error */
> +  #define SH4_PCICLR_MDMA1	  0x20000000	/* DMA1 Transfer Error */
> +  #define SH4_PCICLR_MDMA2	  0x10000000	/* DMA2 Transfer Error */
> +  #define SH4_PCICLR_MDMA3	  0x08000000	/* DMA3 Transfer Error */
> +  #define SH4_PCICLR_TGT	  0x04000000	/* Target Transfer Error */
> +  #define SH4_PCICLR_CMDL	  0x0000000F	/* PCI Command at Error */
> +#define SH4_PCIAINT		0x130		/* Arbiter Interrupt Register */
> +  #define SH4_PCIAINT_MBKN	  0x00002000	/* Master Broken Interrupt */
> +  #define SH4_PCIAINT_TBTO	  0x00001000	/* Target Bus Time Out */
> +  #define SH4_PCIAINT_MBTO	  0x00000800	/* Master Bus Time Out */
> +  #define SH4_PCIAINT_TABT	  0x00000008	/* Target Abort */
> +  #define SH4_PCIAINT_MABT	  0x00000004	/* Master Abort */
> +  #define SH4_PCIAINT_RDPE	  0x00000002	/* Read Data Parity Error */
> +  #define SH4_PCIAINT_WDPE	  0x00000001	/* Write Data Parity Error */
> +#define SH4_PCIAINTM            0x134		/* Arbiter Int. Mask Register */
> +#define SH4_PCIBMLR		0x138		/* Error Bus Master Register */
> +  #define SH4_PCIBMLR_REQ4	  0x00000010	/* REQ4 bus master at error */
> +  #define SH4_PCIBMLR_REQ3	  0x00000008	/* REQ3 bus master at error */
> +  #define SH4_PCIBMLR_REQ2	  0x00000004	/* REQ2 bus master at error */
> +  #define SH4_PCIBMLR_REQ1	  0x00000002	/* REQ1 bus master at error */
> +  #define SH4_PCIBMLR_REQ0	  0x00000001	/* REQ0 bus master at error */
> +#define SH4_PCIDMABT		0x140		/* DMA Transfer Arb. Register */
> +  #define SH4_PCIDMABT_RRBN	  0x00000001	/* DMA Arbitor Round-Robin */
> +#define SH4_PCIDPA0		0x180		/* DMA0 Transfer Addr. */
> +#define SH4_PCIDLA0		0x184		/* DMA0 Local Addr. */
> +#define SH4_PCIDTC0		0x188		/* DMA0 Transfer Cnt. */
> +#define SH4_PCIDCR0		0x18C		/* DMA0 Control Register */
> +  #define SH4_PCIDCR_ALGN	  0x00000600	/* DMA Alignment Mode */
> +  #define SH4_PCIDCR_MAST	  0x00000100	/* DMA Termination Type */
> +  #define SH4_PCIDCR_INTM	  0x00000080	/* DMA Interrupt Done Mask*/
> +  #define SH4_PCIDCR_INTS	  0x00000040	/* DMA Interrupt Done Status */
> +  #define SH4_PCIDCR_LHLD	  0x00000020	/* Local Address Control */
> +  #define SH4_PCIDCR_PHLD	  0x00000010	/* PCI Address Control*/
> +  #define SH4_PCIDCR_IOSEL	  0x00000008	/* PCI Address Space Type */
> +  #define SH4_PCIDCR_DIR	  0x00000004	/* DMA Transfer Direction */
> +  #define SH4_PCIDCR_STOP	  0x00000002	/* Force DMA Stop */
> +  #define SH4_PCIDCR_STRT	  0x00000001	/* DMA Start */
> +#define SH4_PCIDPA1		0x190		/* DMA1 Transfer Addr. */
> +#define SH4_PCIDLA1		0x194		/* DMA1 Local Addr. */
> +#define SH4_PCIDTC1		0x198		/* DMA1 Transfer Cnt. */
> +#define SH4_PCIDCR1		0x19C		/* DMA1 Control Register */
> +#define SH4_PCIDPA2		0x1A0		/* DMA2 Transfer Addr. */
> +#define SH4_PCIDLA2		0x1A4		/* DMA2 Local Addr. */
> +#define SH4_PCIDTC2		0x1A8		/* DMA2 Transfer Cnt. */
> +#define SH4_PCIDCR2		0x1AC		/* DMA2 Control Register */
> +#define SH4_PCIDPA3		0x1B0		/* DMA3 Transfer Addr. */
> +#define SH4_PCIDLA3		0x1B4		/* DMA3 Local Addr. */
> +#define SH4_PCIDTC3		0x1B8		/* DMA3 Transfer Cnt. */
> +#define SH4_PCIDCR3		0x1BC		/* DMA3 Control Register */
> +#define SH4_PCIPAR		0x1C0		/* PIO Address Register */
> +  #define SH4_PCIPAR_CFGEN	  0x80000000	/* Configuration Enable */
> +  #define SH4_PCIPAR_BUSNO	  0x00FF0000	/* Config. Bus Number */
> +  #define SH4_PCIPAR_DEVNO	  0x0000FF00	/* Config. Device Number */
> +  #define SH4_PCIPAR_REGAD	  0x000000FC	/* Register Address Number */
> +#define SH4_PCIMBR		0x1C4		/* Memory Base Address */
> +  #define SH4_PCIMBR_MASK	  0xFF000000	/* Memory Space Mask */
> +  #define SH4_PCIMBR_LOCK	  0x00000001	/* Lock Memory Space */
> +#define SH4_PCIIOBR		0x1C8		/* I/O Base Address Register */
> +  #define SH4_PCIIOBR_MASK	  0xFFFC0000	/* IO Space Mask */
> +  #define SH4_PCIIOBR_LOCK	  0x00000001	/* Lock IO Space */
> +#define SH4_PCIPINT		0x1CC		/* Power Mgmnt Int. Register */
> +  #define SH4_PCIPINT_D3	  0x00000002	/* D3 Pwr Mgmt. Interrupt */
> +  #define SH4_PCIPINT_D0	  0x00000001	/* D0 Pwr Mgmt. Interrupt */
> +#define SH4_PCIPINTM		0x1D0		/* Power Mgmnt Mask Register */
> +#define SH4_PCICLKR		0x1D4		/* Clock Ctrl. Register */
> +  #define SH4_PCICLKR_PCSTP	  0x00000002	/* PCI Clock Stop */
> +  #define SH4_PCICLKR_BCSTP	  0x00000001	/* BCLK Clock Stop */
> +/* For definitions of BCR, MCR see ... */
> +#define SH4_PCIBCR1		0x1E0		/* Memory BCR1 Register */
> +  #define SH4_PCIMBR0		SH4_PCIBCR1
> +#define SH4_PCIBCR2		0x1E4		/* Memory BCR2 Register */
> +  #define SH4_PCIMBMR0		SH4_PCIBCR2
> +#define SH4_PCIWCR1		0x1E8		/* Wait Control 1 Register */
> +#define SH4_PCIWCR2		0x1EC		/* Wait Control 2 Register */
> +#define SH4_PCIWCR3		0x1F0		/* Wait Control 3 Register */
> +  #define SH4_PCIMBR2		SH4_PCIWCR3
> +#define SH4_PCIMCR		0x1F4		/* Memory Control Register */
> +#define SH4_PCIBCR3		0x1f8		/* Memory BCR3 Register */
> +#define SH4_PCIPCTR             0x200		/* Port Control Register */
> +  #define SH4_PCIPCTR_P2EN	  0x000400000	/* Port 2 Enable */
> +  #define SH4_PCIPCTR_P1EN	  0x000200000	/* Port 1 Enable */
> +  #define SH4_PCIPCTR_P0EN	  0x000100000	/* Port 0 Enable */
> +  #define SH4_PCIPCTR_P2UP	  0x000000020	/* Port2 Pull Up Enable */
> +  #define SH4_PCIPCTR_P2IO	  0x000000010	/* Port2 Output Enable */
> +  #define SH4_PCIPCTR_P1UP	  0x000000008	/* Port1 Pull Up Enable */
> +  #define SH4_PCIPCTR_P1IO	  0x000000004	/* Port1 Output Enable */
> +  #define SH4_PCIPCTR_P0UP	  0x000000002	/* Port0 Pull Up Enable */
> +  #define SH4_PCIPCTR_P0IO	  0x000000001	/* Port0 Output Enable */
> +#define SH4_PCIPDTR		0x204		/* Port Data Register */
> +  #define SH4_PCIPDTR_PB5	  0x000000020	/* Port 5 Enable */
> +  #define SH4_PCIPDTR_PB4	  0x000000010	/* Port 4 Enable */
> +  #define SH4_PCIPDTR_PB3	  0x000000008	/* Port 3 Enable */
> +  #define SH4_PCIPDTR_PB2	  0x000000004	/* Port 2 Enable */
> +  #define SH4_PCIPDTR_PB1	  0x000000002	/* Port 1 Enable */
> +  #define SH4_PCIPDTR_PB0	  0x000000001	/* Port 0 Enable */
> +#define SH4_PCIPDR		0x220		/* Port IO Data Register */
> +
> +/* Platform Specific Values */
> +#define SH7751_VENDOR_ID             0x1054
> +#define SH7751_DEVICE_ID             0x3505
> +#define SH7751R_DEVICE_ID            0x350e
> +
> +/* SH7751 Specific Values */
> +#define SH7751_PCI_CONFIG_BASE	     0xFD000000  /* Config space base addr */
> +#define SH7751_PCI_CONFIG_SIZE       0x1000000   /* Config space size */
> +#define SH7751_PCI_MEMORY_BASE	     0xFD000000  /* Memory space base addr */
> +#define SH7751_PCI_MEM_SIZE          0x01000000  /* Size of Memory window */
> +#define SH7751_PCI_IO_BASE           0xFE240000  /* IO space base address */
> +#define SH7751_PCI_IO_SIZE           0x40000     /* Size of IO window */
> +
> +#define SH7751_PCIREG_BASE           0xFE200000  /* PCI regs base address */
> +
> +#define SH7751_PCICONF0            0x0           /* PCI Config Reg 0 */
> +  #define SH7751_PCICONF0_DEVID      0xFFFF0000  /* Device ID */
> +  #define SH7751_PCICONF0_VNDID      0x0000FFFF  /* Vendor ID */
> +#define SH7751_PCICONF1            0x4           /* PCI Config Reg 1 */
> +  #define SH7751_PCICONF1_DPE        0x80000000  /* Data Parity Error */
> +  #define SH7751_PCICONF1_SSE        0x40000000  /* System Error Status */
> +  #define SH7751_PCICONF1_RMA        0x20000000  /* Master Abort */
> +  #define SH7751_PCICONF1_RTA        0x10000000  /* Target Abort Rx Status */
> +  #define SH7751_PCICONF1_STA        0x08000000  /* Target Abort Exec Status */
> +  #define SH7751_PCICONF1_DEV        0x06000000  /* Timing Status */
> +  #define SH7751_PCICONF1_DPD        0x01000000  /* Data Parity Status */
> +  #define SH7751_PCICONF1_FBBC       0x00800000  /* Back 2 Back Status */
> +  #define SH7751_PCICONF1_UDF        0x00400000  /* User Defined Status */
> +  #define SH7751_PCICONF1_66M        0x00200000  /* 66Mhz Operation Status */
> +  #define SH7751_PCICONF1_PM         0x00100000  /* Power Management Status */
> +  #define SH7751_PCICONF1_PBBE       0x00000200  /* Back 2 Back Control */
> +  #define SH7751_PCICONF1_SER        0x00000100  /* SERR Output Control */
> +  #define SH7751_PCICONF1_WCC        0x00000080  /* Wait Cycle Control */
> +  #define SH7751_PCICONF1_PER        0x00000040  /* Parity Error Response */
> +  #define SH7751_PCICONF1_VPS        0x00000020  /* VGA Pallet Snoop */
> +  #define SH7751_PCICONF1_MWIE       0x00000010  /* Memory Write+Invalidate */
> +  #define SH7751_PCICONF1_SPC        0x00000008  /* Special Cycle Control */
> +  #define SH7751_PCICONF1_BUM        0x00000004  /* Bus Master Control */
> +  #define SH7751_PCICONF1_MES        0x00000002  /* Memory Space Control */
> +  #define SH7751_PCICONF1_IOS        0x00000001  /* I/O Space Control */
> +#define SH7751_PCICONF2            0x8           /* PCI Config Reg 2 */
> +  #define SH7751_PCICONF2_BCC        0xFF000000  /* Base Class Code */
> +  #define SH7751_PCICONF2_SCC        0x00FF0000  /* Sub-Class Code */
> +  #define SH7751_PCICONF2_RLPI       0x0000FF00  /* Programming Interface */
> +  #define SH7751_PCICONF2_REV        0x000000FF  /* Revision ID */
> +#define SH7751_PCICONF3            0xC           /* PCI Config Reg 3 */
> +  #define SH7751_PCICONF3_BIST7      0x80000000  /* Bist Supported */
> +  #define SH7751_PCICONF3_BIST6      0x40000000  /* Bist Executing */
> +  #define SH7751_PCICONF3_BIST3_0    0x0F000000  /* Bist Passed */
> +  #define SH7751_PCICONF3_HD7        0x00800000  /* Single Function device */
> +  #define SH7751_PCICONF3_HD6_0      0x007F0000  /* Configuration Layout */
> +  #define SH7751_PCICONF3_LAT        0x0000FF00  /* Latency Timer */
> +  #define SH7751_PCICONF3_CLS        0x000000FF  /* Cache Line Size */
> +#define SH7751_PCICONF4            0x10          /* PCI Config Reg 4 */
> +  #define SH7751_PCICONF4_BASE       0xFFFFFFFC  /* I/O Space Base Addr */
> +  #define SH7751_PCICONF4_ASI        0x00000001  /* Address Space Type */
> +#define SH7751_PCICONF5            0x14          /* PCI Config Reg 5 */
> +  #define SH7751_PCICONF5_BASE       0xFFFFFFF0  /* Mem Space Base Addr */
> +  #define SH7751_PCICONF5_LAP        0x00000008  /* Prefetch Enabled */
> +  #define SH7751_PCICONF5_LAT        0x00000006  /* Local Memory type */
> +  #define SH7751_PCICONF5_ASI        0x00000001  /* Address Space Type */
> +#define SH7751_PCICONF6            0x18          /* PCI Config Reg 6 */
> +  #define SH7751_PCICONF6_BASE       0xFFFFFFF0  /* Mem Space Base Addr */
> +  #define SH7751_PCICONF6_LAP        0x00000008  /* Prefetch Enabled */
> +  #define SH7751_PCICONF6_LAT        0x00000006  /* Local Memory type */
> +  #define SH7751_PCICONF6_ASI        0x00000001  /* Address Space Type */
> +/* PCICONF7 - PCICONF10 are undefined */
> +#define SH7751_PCICONF11           0x2C          /* PCI Config Reg 11 */
> +  #define SH7751_PCICONF11_SSID      0xFFFF0000  /* Subsystem ID */
> +  #define SH7751_PCICONF11_SVID      0x0000FFFF  /* Subsystem Vendor ID */
> +/* PCICONF12 is undefined */
> +#define SH7751_PCICONF13           0x34          /* PCI Config Reg 13 */
> +  #define SH7751_PCICONF13_CPTR      0x000000FF  /* PM function pointer */
> +/* PCICONF14 is undefined */
> +#define SH7751_PCICONF15           0x3C          /* PCI Config Reg 15 */
> +  #define SH7751_PCICONF15_IPIN      0x000000FF  /* Interrupt Pin */
> +#define SH7751_PCICONF16           0x40          /* PCI Config Reg 16 */
> +  #define SH7751_PCICONF16_PMES      0xF8000000  /* PME Support */
> +  #define SH7751_PCICONF16_D2S       0x04000000  /* D2 Support */
> +  #define SH7751_PCICONF16_D1S       0x02000000  /* D1 Support */
> +  #define SH7751_PCICONF16_DSI       0x00200000  /* Bit Device Init. */
> +  #define SH7751_PCICONF16_PMCK      0x00080000  /* Clock for PME req. */
> +  #define SH7751_PCICONF16_VER       0x00070000  /* PM Version */
> +  #define SH7751_PCICONF16_NIP       0x0000FF00  /* Next Item Pointer */
> +  #define SH7751_PCICONF16_CID       0x000000FF  /* Capability Identifier */
> +#define SH7751_PCICONF17           0x44          /* PCI Config Reg 17 */
> +  #define SH7751_PCICONF17_DATA      0xFF000000  /* Data field for PM */
> +  #define SH7751_PCICONF17_PMES      0x00800000  /* PME Status */
> +  #define SH7751_PCICONF17_DSCL      0x00600000  /* Data Scaling Value */
> +  #define SH7751_PCICONF17_DSEL      0x001E0000  /* Data Select */
> +  #define SH7751_PCICONF17_PMEN      0x00010000  /* PME Enable */
> +  #define SH7751_PCICONF17_PWST      0x00000003  /* Power State */
> +/* SH7751 Internal PCI Registers */
> +
> +/* Memory Control Registers */
> +#define SH7751_BCR1                0x0000    /* Memory BCR1 Register */
> +#define SH7751_BCR2                0x0004    /* Memory BCR2 Register */
> +#define SH7751_BCR3                0x0050    /* Memory BCR3 Register */
> +#define SH7751_WCR1                0x0008    /* Wait Control 1 Register */
> +#define SH7751_WCR2                0x000C    /* Wait Control 2 Register */
> +#define SH7751_WCR3                0x0010    /* Wait Control 3 Register */
> +#define SH7751_MCR                 0x0014    /* Memory Control Register */
> +
> +/* General Memory Config Addresses */
> +#define SH7751_CS0_BASE_ADDR       0x0
> +#define SH7751_MEM_REGION_SIZE     0x04000000
> +#define SH7751_CS1_BASE_ADDR       (SH7751_CS0_BASE_ADDR + SH7751_MEM_REGION_SIZE)
> +#define SH7751_CS2_BASE_ADDR       (SH7751_CS1_BASE_ADDR + SH7751_MEM_REGION_SIZE)
> +#define SH7751_CS3_BASE_ADDR       (SH7751_CS2_BASE_ADDR + SH7751_MEM_REGION_SIZE)
> +#define SH7751_CS4_BASE_ADDR       (SH7751_CS3_BASE_ADDR + SH7751_MEM_REGION_SIZE)
> +#define SH7751_CS5_BASE_ADDR       (SH7751_CS4_BASE_ADDR + SH7751_MEM_REGION_SIZE)
> +#define SH7751_CS6_BASE_ADDR       (SH7751_CS5_BASE_ADDR + SH7751_MEM_REGION_SIZE)
> +
> +#endif /* _PCI_SH7751_H_ */
> -- 
> 2.7.0
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

On Sun, May 01, 2016 at 02:08:31PM +0900, Yoshinori Sato wrote:

> +static __init int sh7751_pci_probe(struct platform_device *pdev)
> +{
> ...

> +	pci->cfg.ops = &pci_sh7751_ops;
> +	return pci_host_common_probe(pdev, pci);
> +}
> +
> +static __refdata struct platform_driver sh7751_pci_driver = {
> +	.driver = {
> +		.name = "sh7751-pci",
> +		.of_match_table = sh7751_pci_of_match,
> +	},
> +	.probe = sh7751_pci_probe,
> +};
> +module_platform_driver(sh7751_pci_driver);

I *think* that if you use builtin_platform_driver_probe() or
module_platform_driver_probe() here, you can get rid of the __refdata
on sh7751_pci_driver while keeping sh7751_pci_probe() as __init.
--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html