Re: [PATCH AUTOSEL 4.18 069/131] PCI: mobiveil: Fix struct mobiveil_pcie.pcie_reg_base address type
Acked-by: Subrahmanya Lingappa Thanks. On Sun, Sep 2, 2018 at 6:34 PM Sasha Levin wrote: > > From: Lorenzo Pieralisi > > [ Upstream commit af3f606e0bbb6d811c50b7b90fe324b07fb7cab8 ] > > The field pcie_reg_base in struct mobiveil_pcie represents a physical > address so it should be of phys_addr_t type rather than void __iomem*; > this results in the following compilation warnings: > > drivers/pci/controller/pcie-mobiveil.c: In function > 'mobiveil_pcie_parse_dt': > drivers/pci/controller/pcie-mobiveil.c:326:22: warning: assignment makes > pointer from integer without a cast [-Wint-conversion] > pcie->pcie_reg_base = res->start; > ^ > drivers/pci/controller/pcie-mobiveil.c: In function > 'mobiveil_pcie_enable_msi': > drivers/pci/controller/pcie-mobiveil.c:485:25: warning: initialization > makes integer from pointer without a cast [-Wint-conversion] > phys_addr_t msg_addr = pcie->pcie_reg_base; > ^~~~ > drivers/pci/controller/pcie-mobiveil.c: In function > 'mobiveil_compose_msi_msg': > drivers/pci/controller/pcie-mobiveil.c:640:21: warning: initialization > makes integer from pointer without a cast [-Wint-conversion] > phys_addr_t addr = pcie->pcie_reg_base + (data->hwirq * sizeof(int)); > > Fix the type and with it the compilation warnings. > > Fixes: 9af6bcb11e12 ("PCI: mobiveil: Add Mobiveil PCIe Host Bridge IP > driver") > Signed-off-by: Lorenzo Pieralisi > Cc: Bjorn Helgaas > Cc: Subrahmanya Lingappa > Signed-off-by: Sasha Levin > --- > drivers/pci/controller/pcie-mobiveil.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/pci/controller/pcie-mobiveil.c > b/drivers/pci/controller/pcie-mobiveil.c > index 2c81be37b010..a939e8d31735 100644 > --- a/drivers/pci/controller/pcie-mobiveil.c > +++ b/drivers/pci/controller/pcie-mobiveil.c > @@ -132,7 +132,7 @@ struct mobiveil_pcie { > void __iomem *config_axi_slave_base;/* endpoint config base */ > void __iomem *csr_axi_slave_base; /* root port config base */ > void __iomem *apb_csr_base; /* MSI register base */ > - void __iomem *pcie_reg_base;/* Physical PCIe Controller Base */ > + phys_addr_t pcie_reg_base; /* Physical PCIe Controller Base */ > struct irq_domain *intx_domain; > raw_spinlock_t intx_mask_lock; > int irq; > -- > 2.17.1
Re: [PATCH AUTOSEL 4.18 069/131] PCI: mobiveil: Fix struct mobiveil_pcie.pcie_reg_base address type
Acked-by: Subrahmanya Lingappa Thanks. On Sun, Sep 2, 2018 at 6:34 PM Sasha Levin wrote: > > From: Lorenzo Pieralisi > > [ Upstream commit af3f606e0bbb6d811c50b7b90fe324b07fb7cab8 ] > > The field pcie_reg_base in struct mobiveil_pcie represents a physical > address so it should be of phys_addr_t type rather than void __iomem*; > this results in the following compilation warnings: > > drivers/pci/controller/pcie-mobiveil.c: In function > 'mobiveil_pcie_parse_dt': > drivers/pci/controller/pcie-mobiveil.c:326:22: warning: assignment makes > pointer from integer without a cast [-Wint-conversion] > pcie->pcie_reg_base = res->start; > ^ > drivers/pci/controller/pcie-mobiveil.c: In function > 'mobiveil_pcie_enable_msi': > drivers/pci/controller/pcie-mobiveil.c:485:25: warning: initialization > makes integer from pointer without a cast [-Wint-conversion] > phys_addr_t msg_addr = pcie->pcie_reg_base; > ^~~~ > drivers/pci/controller/pcie-mobiveil.c: In function > 'mobiveil_compose_msi_msg': > drivers/pci/controller/pcie-mobiveil.c:640:21: warning: initialization > makes integer from pointer without a cast [-Wint-conversion] > phys_addr_t addr = pcie->pcie_reg_base + (data->hwirq * sizeof(int)); > > Fix the type and with it the compilation warnings. > > Fixes: 9af6bcb11e12 ("PCI: mobiveil: Add Mobiveil PCIe Host Bridge IP > driver") > Signed-off-by: Lorenzo Pieralisi > Cc: Bjorn Helgaas > Cc: Subrahmanya Lingappa > Signed-off-by: Sasha Levin > --- > drivers/pci/controller/pcie-mobiveil.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/pci/controller/pcie-mobiveil.c > b/drivers/pci/controller/pcie-mobiveil.c > index 2c81be37b010..a939e8d31735 100644 > --- a/drivers/pci/controller/pcie-mobiveil.c > +++ b/drivers/pci/controller/pcie-mobiveil.c > @@ -132,7 +132,7 @@ struct mobiveil_pcie { > void __iomem *config_axi_slave_base;/* endpoint config base */ > void __iomem *csr_axi_slave_base; /* root port config base */ > void __iomem *apb_csr_base; /* MSI register base */ > - void __iomem *pcie_reg_base;/* Physical PCIe Controller Base */ > + phys_addr_t pcie_reg_base; /* Physical PCIe Controller Base */ > struct irq_domain *intx_domain; > raw_spinlock_t intx_mask_lock; > int irq; > -- > 2.17.1
[PATCH AUTOSEL 4.18 069/131] PCI: mobiveil: Fix struct mobiveil_pcie.pcie_reg_base address type
From: Lorenzo Pieralisi [ Upstream commit af3f606e0bbb6d811c50b7b90fe324b07fb7cab8 ] The field pcie_reg_base in struct mobiveil_pcie represents a physical address so it should be of phys_addr_t type rather than void __iomem*; this results in the following compilation warnings: drivers/pci/controller/pcie-mobiveil.c: In function 'mobiveil_pcie_parse_dt': drivers/pci/controller/pcie-mobiveil.c:326:22: warning: assignment makes pointer from integer without a cast [-Wint-conversion] pcie->pcie_reg_base = res->start; ^ drivers/pci/controller/pcie-mobiveil.c: In function 'mobiveil_pcie_enable_msi': drivers/pci/controller/pcie-mobiveil.c:485:25: warning: initialization makes integer from pointer without a cast [-Wint-conversion] phys_addr_t msg_addr = pcie->pcie_reg_base; ^~~~ drivers/pci/controller/pcie-mobiveil.c: In function 'mobiveil_compose_msi_msg': drivers/pci/controller/pcie-mobiveil.c:640:21: warning: initialization makes integer from pointer without a cast [-Wint-conversion] phys_addr_t addr = pcie->pcie_reg_base + (data->hwirq * sizeof(int)); Fix the type and with it the compilation warnings. Fixes: 9af6bcb11e12 ("PCI: mobiveil: Add Mobiveil PCIe Host Bridge IP driver") Signed-off-by: Lorenzo Pieralisi Cc: Bjorn Helgaas Cc: Subrahmanya Lingappa Signed-off-by: Sasha Levin --- drivers/pci/controller/pcie-mobiveil.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pci/controller/pcie-mobiveil.c b/drivers/pci/controller/pcie-mobiveil.c index 2c81be37b010..a939e8d31735 100644 --- a/drivers/pci/controller/pcie-mobiveil.c +++ b/drivers/pci/controller/pcie-mobiveil.c @@ -132,7 +132,7 @@ struct mobiveil_pcie { void __iomem *config_axi_slave_base;/* endpoint config base */ void __iomem *csr_axi_slave_base; /* root port config base */ void __iomem *apb_csr_base; /* MSI register base */ - void __iomem *pcie_reg_base;/* Physical PCIe Controller Base */ + phys_addr_t pcie_reg_base; /* Physical PCIe Controller Base */ struct irq_domain *intx_domain; raw_spinlock_t intx_mask_lock; int irq; -- 2.17.1
[PATCH AUTOSEL 4.18 069/131] PCI: mobiveil: Fix struct mobiveil_pcie.pcie_reg_base address type
From: Lorenzo Pieralisi [ Upstream commit af3f606e0bbb6d811c50b7b90fe324b07fb7cab8 ] The field pcie_reg_base in struct mobiveil_pcie represents a physical address so it should be of phys_addr_t type rather than void __iomem*; this results in the following compilation warnings: drivers/pci/controller/pcie-mobiveil.c: In function 'mobiveil_pcie_parse_dt': drivers/pci/controller/pcie-mobiveil.c:326:22: warning: assignment makes pointer from integer without a cast [-Wint-conversion] pcie->pcie_reg_base = res->start; ^ drivers/pci/controller/pcie-mobiveil.c: In function 'mobiveil_pcie_enable_msi': drivers/pci/controller/pcie-mobiveil.c:485:25: warning: initialization makes integer from pointer without a cast [-Wint-conversion] phys_addr_t msg_addr = pcie->pcie_reg_base; ^~~~ drivers/pci/controller/pcie-mobiveil.c: In function 'mobiveil_compose_msi_msg': drivers/pci/controller/pcie-mobiveil.c:640:21: warning: initialization makes integer from pointer without a cast [-Wint-conversion] phys_addr_t addr = pcie->pcie_reg_base + (data->hwirq * sizeof(int)); Fix the type and with it the compilation warnings. Fixes: 9af6bcb11e12 ("PCI: mobiveil: Add Mobiveil PCIe Host Bridge IP driver") Signed-off-by: Lorenzo Pieralisi Cc: Bjorn Helgaas Cc: Subrahmanya Lingappa Signed-off-by: Sasha Levin --- drivers/pci/controller/pcie-mobiveil.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pci/controller/pcie-mobiveil.c b/drivers/pci/controller/pcie-mobiveil.c index 2c81be37b010..a939e8d31735 100644 --- a/drivers/pci/controller/pcie-mobiveil.c +++ b/drivers/pci/controller/pcie-mobiveil.c @@ -132,7 +132,7 @@ struct mobiveil_pcie { void __iomem *config_axi_slave_base;/* endpoint config base */ void __iomem *csr_axi_slave_base; /* root port config base */ void __iomem *apb_csr_base; /* MSI register base */ - void __iomem *pcie_reg_base;/* Physical PCIe Controller Base */ + phys_addr_t pcie_reg_base; /* Physical PCIe Controller Base */ struct irq_domain *intx_domain; raw_spinlock_t intx_mask_lock; int irq; -- 2.17.1