Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Mark Tomlinson <mark.tomlinson@alliedtelesis.co.nz>
To:     bhelgaas@google.com, rjui@broadcom.com, sbranden@broadcom.com,
        f.fainelli@gmail.com
Cc:     linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org,
        Mark Tomlinson <mark.tomlinson@alliedtelesis.co.nz>
Subject: [PATCH 2/3] PCI: iproc: Stop using generic config read/write functions
Date:   Thu, 30 Jul 2020 15:37:46 +1200
Message-Id: <20200730033747.18931-2-mark.tomlinson@alliedtelesis.co.nz>
In-Reply-To: <20200730033747.18931-1-mark.tomlinson@alliedtelesis.co.nz>
References: <20200730033747.18931-1-mark.tomlinson@alliedtelesis.co.nz>
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

The pci_generic_config_write32() function will give warning messages
whenever writing less than 4 bytes at a time. As there is nothing we can
do about this without changing the hardware, the message is just a
nuisance. So instead of using the generic functions, use the functions
that have already been written for reading/writing the config registers.

Signed-off-by: Mark Tomlinson <mark.tomlinson@alliedtelesis.co.nz>
---
 drivers/pci/controller/pcie-iproc.c | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/drivers/pci/controller/pcie-iproc.c b/drivers/pci/controller=
/pcie-iproc.c
index 2c836eede42c..68ecd3050529 100644
--- a/drivers/pci/controller/pcie-iproc.c
+++ b/drivers/pci/controller/pcie-iproc.c
@@ -709,12 +709,13 @@ static int iproc_pcie_config_read32(struct pci_bus =
*bus, unsigned int devfn,
 {
 	int ret;
 	struct iproc_pcie *pcie =3D iproc_data(bus);
+	int busno =3D bus->number;
=20
 	iproc_pcie_apb_err_disable(bus, true);
 	if (pcie->iproc_cfg_read)
 		ret =3D iproc_pcie_config_read(bus, devfn, where, size, val);
 	else
-		ret =3D pci_generic_config_read32(bus, devfn, where, size, val);
+		ret =3D iproc_pci_raw_config_read32(pcie, busno, devfn, where, size, v=
al);
 	iproc_pcie_apb_err_disable(bus, false);
=20
 	return ret;
@@ -724,9 +725,11 @@ static int iproc_pcie_config_write32(struct pci_bus =
*bus, unsigned int devfn,
 				     int where, int size, u32 val)
 {
 	int ret;
+	struct iproc_pcie *pcie =3D iproc_data(bus);
+	int busno =3D bus->number;
=20
 	iproc_pcie_apb_err_disable(bus, true);
-	ret =3D pci_generic_config_write32(bus, devfn, where, size, val);
+	ret =3D iproc_pci_raw_config_write32(pcie, busno, devfn, where, size, v=
al);
 	iproc_pcie_apb_err_disable(bus, false);
=20
 	return ret;
--=20
2.28.0