* [PATCH] PCI: rcar: Change macro name of PCIEPARL and PCIEPARH
@ 2015-02-04 9:02 Nobuhiro Iwamatsu
2015-02-04 9:14 ` Phil Edworthy
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Nobuhiro Iwamatsu @ 2015-02-04 9:02 UTC (permalink / raw)
To: bhelgaas; +Cc: linux-pci, phil.edworthy, horms+renesas, Nobuhiro Iwamatsu
PCIEPARL and PCIEPARH are macro that calculates address of the register.
However, register name is incorrect, PCIEPALR and PCIEPAUR is correct.
This change macro name to PCIEPALR and PCIEPAUR.
Signed-off-by: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com>
---
drivers/pci/host/pcie-rcar.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/pci/host/pcie-rcar.c b/drivers/pci/host/pcie-rcar.c
index eb12659..183c32d 100644
--- a/drivers/pci/host/pcie-rcar.c
+++ b/drivers/pci/host/pcie-rcar.c
@@ -64,8 +64,8 @@
#define LAR_ENABLE (1 << 1)
/* PCIe address reg & mask */
-#define PCIEPARL(x) (0x03400 + ((x) * 0x20))
-#define PCIEPARH(x) (0x03404 + ((x) * 0x20))
+#define PCIEPALR(x) (0x03400 + ((x) * 0x20))
+#define PCIEPAUR(x) (0x03404 + ((x) * 0x20))
#define PCIEPAMR(x) (0x03408 + ((x) * 0x20))
#define PCIEPTCTLR(x) (0x0340c + ((x) * 0x20))
#define PAR_ENABLE (1 << 31)
@@ -341,9 +341,9 @@ static void rcar_pcie_setup_window(int win, struct rcar_pcie *pcie)
else
res_start = res->start;
- rcar_pci_write_reg(pcie, upper_32_bits(res_start), PCIEPARH(win));
+ rcar_pci_write_reg(pcie, upper_32_bits(res_start), PCIEPAUR(win));
rcar_pci_write_reg(pcie, lower_32_bits(res_start) & ~0x7F,
- PCIEPARL(win));
+ PCIEPALR(win));
/* First resource is for IO */
mask = PAR_ENABLE;
--
2.1.3
^ permalink raw reply related [flat|nested] 5+ messages in thread
* RE: [PATCH] PCI: rcar: Change macro name of PCIEPARL and PCIEPARH
2015-02-04 9:02 [PATCH] PCI: rcar: Change macro name of PCIEPARL and PCIEPARH Nobuhiro Iwamatsu
@ 2015-02-04 9:14 ` Phil Edworthy
2015-02-05 0:35 ` Simon Horman
2015-02-24 2:03 ` Simon Horman
2015-02-24 6:26 ` Bjorn Helgaas
2 siblings, 1 reply; 5+ messages in thread
From: Phil Edworthy @ 2015-02-04 9:14 UTC (permalink / raw)
To: Nobuhiro Iwamatsu; +Cc: linux-pci, horms+renesas, Nobuhiro Iwamatsu, bhelgaas
Hi Iwamatsu-san,
On: 04 February 2015 09:03, Nobuhiro Iwamatsu wrote:
>
> PCIEPARL and PCIEPARH are macro that calculates address of the register.
> However, register name is incorrect, PCIEPALR and PCIEPAUR is correct.
> This change macro name to PCIEPALR and PCIEPAUR.
>
> Signed-off-by: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com>
Acked-by: Phil Edworthy <phil.edworthy@renesas.com>
Thanks
Phil
> ---
> drivers/pci/host/pcie-rcar.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/pci/host/pcie-rcar.c b/drivers/pci/host/pcie-rcar.c
> index eb12659..183c32d 100644
> --- a/drivers/pci/host/pcie-rcar.c
> +++ b/drivers/pci/host/pcie-rcar.c
> @@ -64,8 +64,8 @@
> #define LAR_ENABLE (1 << 1)
>
> /* PCIe address reg & mask */
> -#define PCIEPARL(x) (0x03400 + ((x) * 0x20))
> -#define PCIEPARH(x) (0x03404 + ((x) * 0x20))
> +#define PCIEPALR(x) (0x03400 + ((x) * 0x20))
> +#define PCIEPAUR(x) (0x03404 + ((x) * 0x20))
> #define PCIEPAMR(x) (0x03408 + ((x) * 0x20))
> #define PCIEPTCTLR(x) (0x0340c + ((x) * 0x20))
> #define PAR_ENABLE (1 << 31)
> @@ -341,9 +341,9 @@ static void rcar_pcie_setup_window(int win, struct
> rcar_pcie *pcie)
> else
> res_start = res->start;
>
> - rcar_pci_write_reg(pcie, upper_32_bits(res_start), PCIEPARH(win));
> + rcar_pci_write_reg(pcie, upper_32_bits(res_start), PCIEPAUR(win));
> rcar_pci_write_reg(pcie, lower_32_bits(res_start) & ~0x7F,
> - PCIEPARL(win));
> + PCIEPALR(win));
>
> /* First resource is for IO */
> mask = PAR_ENABLE;
> --
> 2.1.3
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] PCI: rcar: Change macro name of PCIEPARL and PCIEPARH
2015-02-04 9:14 ` Phil Edworthy
@ 2015-02-05 0:35 ` Simon Horman
0 siblings, 0 replies; 5+ messages in thread
From: Simon Horman @ 2015-02-05 0:35 UTC (permalink / raw)
To: Phil Edworthy; +Cc: Nobuhiro Iwamatsu, linux-pci, bhelgaas
On Wed, Feb 04, 2015 at 09:14:40AM +0000, Phil Edworthy wrote:
> Hi Iwamatsu-san,
>
> On: 04 February 2015 09:03, Nobuhiro Iwamatsu wrote:
> >
> > PCIEPARL and PCIEPARH are macro that calculates address of the register.
> > However, register name is incorrect, PCIEPALR and PCIEPAUR is correct.
> > This change macro name to PCIEPALR and PCIEPAUR.
> >
> > Signed-off-by: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com>
> Acked-by: Phil Edworthy <phil.edworthy@renesas.com>
Acked-by: Simon Horman <horms+renesas@verge.net.au>
>
> Thanks
> Phil
>
> > ---
> > drivers/pci/host/pcie-rcar.c | 8 ++++----
> > 1 file changed, 4 insertions(+), 4 deletions(-)
> >
> > diff --git a/drivers/pci/host/pcie-rcar.c b/drivers/pci/host/pcie-rcar.c
> > index eb12659..183c32d 100644
> > --- a/drivers/pci/host/pcie-rcar.c
> > +++ b/drivers/pci/host/pcie-rcar.c
> > @@ -64,8 +64,8 @@
> > #define LAR_ENABLE (1 << 1)
> >
> > /* PCIe address reg & mask */
> > -#define PCIEPARL(x) (0x03400 + ((x) * 0x20))
> > -#define PCIEPARH(x) (0x03404 + ((x) * 0x20))
> > +#define PCIEPALR(x) (0x03400 + ((x) * 0x20))
> > +#define PCIEPAUR(x) (0x03404 + ((x) * 0x20))
> > #define PCIEPAMR(x) (0x03408 + ((x) * 0x20))
> > #define PCIEPTCTLR(x) (0x0340c + ((x) * 0x20))
> > #define PAR_ENABLE (1 << 31)
> > @@ -341,9 +341,9 @@ static void rcar_pcie_setup_window(int win, struct
> > rcar_pcie *pcie)
> > else
> > res_start = res->start;
> >
> > - rcar_pci_write_reg(pcie, upper_32_bits(res_start), PCIEPARH(win));
> > + rcar_pci_write_reg(pcie, upper_32_bits(res_start), PCIEPAUR(win));
> > rcar_pci_write_reg(pcie, lower_32_bits(res_start) & ~0x7F,
> > - PCIEPARL(win));
> > + PCIEPALR(win));
> >
> > /* First resource is for IO */
> > mask = PAR_ENABLE;
> > --
> > 2.1.3
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] PCI: rcar: Change macro name of PCIEPARL and PCIEPARH
2015-02-04 9:02 [PATCH] PCI: rcar: Change macro name of PCIEPARL and PCIEPARH Nobuhiro Iwamatsu
2015-02-04 9:14 ` Phil Edworthy
@ 2015-02-24 2:03 ` Simon Horman
2015-02-24 6:26 ` Bjorn Helgaas
2 siblings, 0 replies; 5+ messages in thread
From: Simon Horman @ 2015-02-24 2:03 UTC (permalink / raw)
To: Nobuhiro Iwamatsu; +Cc: bhelgaas, linux-pci, phil.edworthy
On Wed, Feb 04, 2015 at 06:02:55PM +0900, Nobuhiro Iwamatsu wrote:
> PCIEPARL and PCIEPARH are macro that calculates address of the register.
> However, register name is incorrect, PCIEPALR and PCIEPAUR is correct.
> This change macro name to PCIEPALR and PCIEPAUR.
>
> Signed-off-by: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com>
Acked-by: Simon Horman <horms+renesas@verge.net.au>
> ---
> drivers/pci/host/pcie-rcar.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/pci/host/pcie-rcar.c b/drivers/pci/host/pcie-rcar.c
> index eb12659..183c32d 100644
> --- a/drivers/pci/host/pcie-rcar.c
> +++ b/drivers/pci/host/pcie-rcar.c
> @@ -64,8 +64,8 @@
> #define LAR_ENABLE (1 << 1)
>
> /* PCIe address reg & mask */
> -#define PCIEPARL(x) (0x03400 + ((x) * 0x20))
> -#define PCIEPARH(x) (0x03404 + ((x) * 0x20))
> +#define PCIEPALR(x) (0x03400 + ((x) * 0x20))
> +#define PCIEPAUR(x) (0x03404 + ((x) * 0x20))
> #define PCIEPAMR(x) (0x03408 + ((x) * 0x20))
> #define PCIEPTCTLR(x) (0x0340c + ((x) * 0x20))
> #define PAR_ENABLE (1 << 31)
> @@ -341,9 +341,9 @@ static void rcar_pcie_setup_window(int win, struct rcar_pcie *pcie)
> else
> res_start = res->start;
>
> - rcar_pci_write_reg(pcie, upper_32_bits(res_start), PCIEPARH(win));
> + rcar_pci_write_reg(pcie, upper_32_bits(res_start), PCIEPAUR(win));
> rcar_pci_write_reg(pcie, lower_32_bits(res_start) & ~0x7F,
> - PCIEPARL(win));
> + PCIEPALR(win));
>
> /* First resource is for IO */
> mask = PAR_ENABLE;
> --
> 2.1.3
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] PCI: rcar: Change macro name of PCIEPARL and PCIEPARH
2015-02-04 9:02 [PATCH] PCI: rcar: Change macro name of PCIEPARL and PCIEPARH Nobuhiro Iwamatsu
2015-02-04 9:14 ` Phil Edworthy
2015-02-24 2:03 ` Simon Horman
@ 2015-02-24 6:26 ` Bjorn Helgaas
2 siblings, 0 replies; 5+ messages in thread
From: Bjorn Helgaas @ 2015-02-24 6:26 UTC (permalink / raw)
To: Nobuhiro Iwamatsu; +Cc: linux-pci, phil.edworthy, horms+renesas
On Wed, Feb 04, 2015 at 06:02:55PM +0900, Nobuhiro Iwamatsu wrote:
> PCIEPARL and PCIEPARH are macro that calculates address of the register.
> However, register name is incorrect, PCIEPALR and PCIEPAUR is correct.
> This change macro name to PCIEPALR and PCIEPAUR.
>
> Signed-off-by: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com>
Applied to pci/host-rcar for v4.1, thanks!
> ---
> drivers/pci/host/pcie-rcar.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/pci/host/pcie-rcar.c b/drivers/pci/host/pcie-rcar.c
> index eb12659..183c32d 100644
> --- a/drivers/pci/host/pcie-rcar.c
> +++ b/drivers/pci/host/pcie-rcar.c
> @@ -64,8 +64,8 @@
> #define LAR_ENABLE (1 << 1)
>
> /* PCIe address reg & mask */
> -#define PCIEPARL(x) (0x03400 + ((x) * 0x20))
> -#define PCIEPARH(x) (0x03404 + ((x) * 0x20))
> +#define PCIEPALR(x) (0x03400 + ((x) * 0x20))
> +#define PCIEPAUR(x) (0x03404 + ((x) * 0x20))
> #define PCIEPAMR(x) (0x03408 + ((x) * 0x20))
> #define PCIEPTCTLR(x) (0x0340c + ((x) * 0x20))
> #define PAR_ENABLE (1 << 31)
> @@ -341,9 +341,9 @@ static void rcar_pcie_setup_window(int win, struct rcar_pcie *pcie)
> else
> res_start = res->start;
>
> - rcar_pci_write_reg(pcie, upper_32_bits(res_start), PCIEPARH(win));
> + rcar_pci_write_reg(pcie, upper_32_bits(res_start), PCIEPAUR(win));
> rcar_pci_write_reg(pcie, lower_32_bits(res_start) & ~0x7F,
> - PCIEPARL(win));
> + PCIEPALR(win));
>
> /* First resource is for IO */
> mask = PAR_ENABLE;
> --
> 2.1.3
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-02-24 6:26 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-02-04 9:02 [PATCH] PCI: rcar: Change macro name of PCIEPARL and PCIEPARH Nobuhiro Iwamatsu
2015-02-04 9:14 ` Phil Edworthy
2015-02-05 0:35 ` Simon Horman
2015-02-24 2:03 ` Simon Horman
2015-02-24 6:26 ` Bjorn Helgaas
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.