* [PATCH] fpga: allow to compile-test Altera FPGA bridge drivers @ 2017-04-11 9:22 Tobias Klauser 2017-04-11 14:27 ` Alan Tull 2017-04-12 5:47 ` kbuild test robot 0 siblings, 2 replies; 8+ messages in thread From: Tobias Klauser @ 2017-04-11 9:22 UTC (permalink / raw) To: Alan Tull, Moritz Fischer; +Cc: linux-fpga, linux-kernel Add COMPILE_TEST to the Kconfig entries for the Altera SoCFPGA FPGA Bridge and the Altera FPGA Freeze Bridge drivers to allow test-compiling them on other architectures. Signed-off-by: Tobias Klauser <tklauser@distanz.ch> --- drivers/fpga/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig index 161ba9dccede..38ab05d261bf 100644 --- a/drivers/fpga/Kconfig +++ b/drivers/fpga/Kconfig @@ -69,14 +69,14 @@ config FPGA_BRIDGE config SOCFPGA_FPGA_BRIDGE tristate "Altera SoCFPGA FPGA Bridges" - depends on ARCH_SOCFPGA && FPGA_BRIDGE + depends on (ARCH_SOCFPGA || COMPILE_TEST) && FPGA_BRIDGE help Say Y to enable drivers for FPGA bridges for Altera SOCFPGA devices. config ALTERA_FREEZE_BRIDGE tristate "Altera FPGA Freeze Bridge" - depends on ARCH_SOCFPGA && FPGA_BRIDGE + depends on (ARCH_SOCFPGA || COMPILE_TEST) && FPGA_BRIDGE help Say Y to enable drivers for Altera FPGA Freeze bridges. A freeze bridge is a bridge that exists in the FPGA fabric to -- 2.12.2 ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] fpga: allow to compile-test Altera FPGA bridge drivers 2017-04-11 9:22 [PATCH] fpga: allow to compile-test Altera FPGA bridge drivers Tobias Klauser @ 2017-04-11 14:27 ` Alan Tull 2017-04-11 14:35 ` Tobias Klauser 2017-04-12 5:47 ` kbuild test robot 1 sibling, 1 reply; 8+ messages in thread From: Alan Tull @ 2017-04-11 14:27 UTC (permalink / raw) To: Tobias Klauser; +Cc: Moritz Fischer, linux-fpga, linux-kernel, matthew.gerlach On Tue, Apr 11, 2017 at 4:22 AM, Tobias Klauser <tklauser@distanz.ch> wrote: Hi Tobias, Thanks for your fixes that you have been sending. I disagree with this particular one (see below), but I appreciate your efforts. > Add COMPILE_TEST to the Kconfig entries for the Altera SoCFPGA FPGA > Bridge and the Altera FPGA Freeze Bridge drivers to allow test-compiling > them on other architectures. > > Signed-off-by: Tobias Klauser <tklauser@distanz.ch> > --- > drivers/fpga/Kconfig | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig > index 161ba9dccede..38ab05d261bf 100644 > --- a/drivers/fpga/Kconfig > +++ b/drivers/fpga/Kconfig > @@ -69,14 +69,14 @@ config FPGA_BRIDGE > > config SOCFPGA_FPGA_BRIDGE > tristate "Altera SoCFPGA FPGA Bridges" > - depends on ARCH_SOCFPGA && FPGA_BRIDGE > + depends on (ARCH_SOCFPGA || COMPILE_TEST) && FPGA_BRIDGE It doesn't make sense to compile test these on other arch's. The bridges enabled by this only appear on Cyclone 5 SoCFPGA parts. > help > Say Y to enable drivers for FPGA bridges for Altera SOCFPGA > devices. > > config ALTERA_FREEZE_BRIDGE > tristate "Altera FPGA Freeze Bridge" > - depends on ARCH_SOCFPGA && FPGA_BRIDGE > + depends on (ARCH_SOCFPGA || COMPILE_TEST) && FPGA_BRIDGE These bridges can be used on Altera PCIe FPGA's. So this shouldn't have been dependent on ARCH_SOCFPGA in the first place. Alan > help > Say Y to enable drivers for Altera FPGA Freeze bridges. A > freeze bridge is a bridge that exists in the FPGA fabric to > -- > 2.12.2 > > ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] fpga: allow to compile-test Altera FPGA bridge drivers 2017-04-11 14:27 ` Alan Tull @ 2017-04-11 14:35 ` Tobias Klauser 2017-04-11 15:21 ` Alan Tull 0 siblings, 1 reply; 8+ messages in thread From: Tobias Klauser @ 2017-04-11 14:35 UTC (permalink / raw) To: Alan Tull; +Cc: Moritz Fischer, linux-fpga, linux-kernel, matthew.gerlach Hi Alan, On 2017-04-11 at 16:27:03 +0200, Alan Tull <atull@kernel.org> wrote: > On Tue, Apr 11, 2017 at 4:22 AM, Tobias Klauser <tklauser@distanz.ch> wrote: > > Hi Tobias, > > Thanks for your fixes that you have been sending. I disagree with > this particular one (see below), but I appreciate your efforts. Thanks for your feedback on the patch. > > Add COMPILE_TEST to the Kconfig entries for the Altera SoCFPGA FPGA > > Bridge and the Altera FPGA Freeze Bridge drivers to allow test-compiling > > them on other architectures. > > > > Signed-off-by: Tobias Klauser <tklauser@distanz.ch> > > --- > > drivers/fpga/Kconfig | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig > > index 161ba9dccede..38ab05d261bf 100644 > > --- a/drivers/fpga/Kconfig > > +++ b/drivers/fpga/Kconfig > > @@ -69,14 +69,14 @@ config FPGA_BRIDGE > > > > config SOCFPGA_FPGA_BRIDGE > > tristate "Altera SoCFPGA FPGA Bridges" > > - depends on ARCH_SOCFPGA && FPGA_BRIDGE > > + depends on (ARCH_SOCFPGA || COMPILE_TEST) && FPGA_BRIDGE > > It doesn't make sense to compile test these on other arch's. > The bridges enabled by this only appear on Cyclone 5 SoCFPGA parts. There are many drivers in the tree where this is the case (i.e. the driver is compiled on an arch where the corresponding hardware is not available). Still it could be benefitial IMO in order to catch compiler errors and warnings, e.g. due to API/header changes earlier on. I occasionally found this quite handy for janitorial/cleanup patches where I didn't have the corresponding cross-compiler/hardware. In any case, I don't feel strongly about this. So feel free to drop at your discretion. > > help > > Say Y to enable drivers for FPGA bridges for Altera SOCFPGA > > devices. > > > > config ALTERA_FREEZE_BRIDGE > > tristate "Altera FPGA Freeze Bridge" > > - depends on ARCH_SOCFPGA && FPGA_BRIDGE > > + depends on (ARCH_SOCFPGA || COMPILE_TEST) && FPGA_BRIDGE > > These bridges can be used on Altera PCIe FPGA's. So this shouldn't > have been dependent on ARCH_SOCFPGA in the first place. Ok. So the dependency on ARCH_SOCFPGA should be dropped altogether? I can send an updated patch if you want. > Alan > > > help > > Say Y to enable drivers for Altera FPGA Freeze bridges. A > > freeze bridge is a bridge that exists in the FPGA fabric to > > -- > > 2.12.2 > > > > > ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] fpga: allow to compile-test Altera FPGA bridge drivers 2017-04-11 14:35 ` Tobias Klauser @ 2017-04-11 15:21 ` Alan Tull 2017-04-12 6:51 ` Tobias Klauser 0 siblings, 1 reply; 8+ messages in thread From: Alan Tull @ 2017-04-11 15:21 UTC (permalink / raw) To: Tobias Klauser; +Cc: Moritz Fischer, linux-fpga, linux-kernel, matthew.gerlach On Tue, Apr 11, 2017 at 9:35 AM, Tobias Klauser <tklauser@distanz.ch> wrote: > Hi Alan, > > On 2017-04-11 at 16:27:03 +0200, Alan Tull <atull@kernel.org> wrote: >> On Tue, Apr 11, 2017 at 4:22 AM, Tobias Klauser <tklauser@distanz.ch> wrote: >> >> Hi Tobias, >> >> Thanks for your fixes that you have been sending. I disagree with >> this particular one (see below), but I appreciate your efforts. > > Thanks for your feedback on the patch. > >> > Add COMPILE_TEST to the Kconfig entries for the Altera SoCFPGA FPGA >> > Bridge and the Altera FPGA Freeze Bridge drivers to allow test-compiling >> > them on other architectures. >> > >> > Signed-off-by: Tobias Klauser <tklauser@distanz.ch> >> > --- >> > drivers/fpga/Kconfig | 4 ++-- >> > 1 file changed, 2 insertions(+), 2 deletions(-) >> > >> > diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig >> > index 161ba9dccede..38ab05d261bf 100644 >> > --- a/drivers/fpga/Kconfig >> > +++ b/drivers/fpga/Kconfig >> > @@ -69,14 +69,14 @@ config FPGA_BRIDGE >> > >> > config SOCFPGA_FPGA_BRIDGE >> > tristate "Altera SoCFPGA FPGA Bridges" >> > - depends on ARCH_SOCFPGA && FPGA_BRIDGE >> > + depends on (ARCH_SOCFPGA || COMPILE_TEST) && FPGA_BRIDGE >> >> It doesn't make sense to compile test these on other arch's. >> The bridges enabled by this only appear on Cyclone 5 SoCFPGA parts. > > There are many drivers in the tree where this is the case (i.e. the > driver is compiled on an arch where the corresponding hardware is not > available). Still it could be benefitial IMO in order to catch compiler > errors and warnings, e.g. due to API/header changes earlier on. I > occasionally found this quite handy for janitorial/cleanup patches where > I didn't have the corresponding cross-compiler/hardware. That sounds good and useful. > > In any case, I don't feel strongly about this. So feel free to drop at > your discretion. > >> > help >> > Say Y to enable drivers for FPGA bridges for Altera SOCFPGA >> > devices. >> > >> > config ALTERA_FREEZE_BRIDGE >> > tristate "Altera FPGA Freeze Bridge" >> > - depends on ARCH_SOCFPGA && FPGA_BRIDGE >> > + depends on (ARCH_SOCFPGA || COMPILE_TEST) && FPGA_BRIDGE >> >> These bridges can be used on Altera PCIe FPGA's. So this shouldn't >> have been dependent on ARCH_SOCFPGA in the first place. > > Ok. So the dependency on ARCH_SOCFPGA should be dropped altogether? I > can send an updated patch if you want. Yes please. Thanks! ALan > >> Alan >> >> > help >> > Say Y to enable drivers for Altera FPGA Freeze bridges. A >> > freeze bridge is a bridge that exists in the FPGA fabric to >> > -- >> > 2.12.2 >> > >> > >> ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] fpga: allow to compile-test Altera FPGA bridge drivers 2017-04-11 15:21 ` Alan Tull @ 2017-04-12 6:51 ` Tobias Klauser 0 siblings, 0 replies; 8+ messages in thread From: Tobias Klauser @ 2017-04-12 6:51 UTC (permalink / raw) To: Alan Tull; +Cc: Moritz Fischer, linux-fpga, linux-kernel, matthew.gerlach On 2017-04-11 at 17:21:19 +0200, Alan Tull <atull@kernel.org> wrote: > On Tue, Apr 11, 2017 at 9:35 AM, Tobias Klauser <tklauser@distanz.ch> wrote: > > Hi Alan, > > > > On 2017-04-11 at 16:27:03 +0200, Alan Tull <atull@kernel.org> wrote: > >> On Tue, Apr 11, 2017 at 4:22 AM, Tobias Klauser <tklauser@distanz.ch> wrote: > >> > >> Hi Tobias, > >> > >> Thanks for your fixes that you have been sending. I disagree with > >> this particular one (see below), but I appreciate your efforts. > > > > Thanks for your feedback on the patch. > > > >> > Add COMPILE_TEST to the Kconfig entries for the Altera SoCFPGA FPGA > >> > Bridge and the Altera FPGA Freeze Bridge drivers to allow test-compiling > >> > them on other architectures. > >> > > >> > Signed-off-by: Tobias Klauser <tklauser@distanz.ch> > >> > --- > >> > drivers/fpga/Kconfig | 4 ++-- > >> > 1 file changed, 2 insertions(+), 2 deletions(-) > >> > > >> > diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig > >> > index 161ba9dccede..38ab05d261bf 100644 > >> > --- a/drivers/fpga/Kconfig > >> > +++ b/drivers/fpga/Kconfig > >> > @@ -69,14 +69,14 @@ config FPGA_BRIDGE > >> > > >> > config SOCFPGA_FPGA_BRIDGE > >> > tristate "Altera SoCFPGA FPGA Bridges" > >> > - depends on ARCH_SOCFPGA && FPGA_BRIDGE > >> > + depends on (ARCH_SOCFPGA || COMPILE_TEST) && FPGA_BRIDGE > >> > >> It doesn't make sense to compile test these on other arch's. > >> The bridges enabled by this only appear on Cyclone 5 SoCFPGA parts. > > > > There are many drivers in the tree where this is the case (i.e. the > > driver is compiled on an arch where the corresponding hardware is not > > available). Still it could be benefitial IMO in order to catch compiler > > errors and warnings, e.g. due to API/header changes earlier on. I > > occasionally found this quite handy for janitorial/cleanup patches where > > I didn't have the corresponding cross-compiler/hardware. > > That sounds good and useful. > > > > > In any case, I don't feel strongly about this. So feel free to drop at > > your discretion. > > > >> > help > >> > Say Y to enable drivers for FPGA bridges for Altera SOCFPGA > >> > devices. > >> > > >> > config ALTERA_FREEZE_BRIDGE > >> > tristate "Altera FPGA Freeze Bridge" > >> > - depends on ARCH_SOCFPGA && FPGA_BRIDGE > >> > + depends on (ARCH_SOCFPGA || COMPILE_TEST) && FPGA_BRIDGE > >> > >> These bridges can be used on Altera PCIe FPGA's. So this shouldn't > >> have been dependent on ARCH_SOCFPGA in the first place. > > > > Ok. So the dependency on ARCH_SOCFPGA should be dropped altogether? I > > can send an updated patch if you want. > > Yes please. Thanks, I'll send a v2. ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] fpga: allow to compile-test Altera FPGA bridge drivers 2017-04-11 9:22 [PATCH] fpga: allow to compile-test Altera FPGA bridge drivers Tobias Klauser 2017-04-11 14:27 ` Alan Tull @ 2017-04-12 5:47 ` kbuild test robot 2017-04-18 15:27 ` matthew.gerlach 1 sibling, 1 reply; 8+ messages in thread From: kbuild test robot @ 2017-04-12 5:47 UTC (permalink / raw) To: Tobias Klauser Cc: kbuild-all, Alan Tull, Moritz Fischer, linux-fpga, linux-kernel [-- Attachment #1: Type: text/plain, Size: 9528 bytes --] Hi Tobias, [auto build test WARNING on linus/master] [also build test WARNING on v4.11-rc6 next-20170411] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Tobias-Klauser/fpga-allow-to-compile-test-Altera-FPGA-bridge-drivers/20170411-181401 config: m32r-allmodconfig (attached as .config) compiler: m32r-linux-gcc (GCC) 6.2.0 reproduce: wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config to linux build tree make.cross ARCH=m32r All warnings (new ones prefixed by >>): In file included from include/linux/printk.h:329:0, from include/linux/kernel.h:13, from include/linux/delay.h:21, from drivers/fpga/altera-freeze-bridge.c:18: drivers/fpga/altera-freeze-bridge.c: In function 'altera_freeze_br_do_freeze': >> drivers/fpga/altera-freeze-bridge.c:107:15: warning: format '%d' expects argument of type 'int', but argument 6 has type 'long unsigned int' [-Wformat=] dev_dbg(dev, "%s %d %d\n", __func__, status, readl(csr_ctrl_addr)); ^ include/linux/dynamic_debug.h:134:39: note: in definition of macro 'dynamic_dev_dbg' __dynamic_dev_dbg(&descriptor, dev, fmt, \ ^~~ >> drivers/fpga/altera-freeze-bridge.c:107:2: note: in expansion of macro 'dev_dbg' dev_dbg(dev, "%s %d %d\n", __func__, status, readl(csr_ctrl_addr)); ^~~~~~~ drivers/fpga/altera-freeze-bridge.c: In function 'altera_freeze_br_do_unfreeze': drivers/fpga/altera-freeze-bridge.c:144:15: warning: format '%d' expects argument of type 'int', but argument 6 has type 'long unsigned int' [-Wformat=] dev_dbg(dev, "%s %d %d\n", __func__, status, readl(csr_ctrl_addr)); ^ include/linux/dynamic_debug.h:134:39: note: in definition of macro 'dynamic_dev_dbg' __dynamic_dev_dbg(&descriptor, dev, fmt, \ ^~~ drivers/fpga/altera-freeze-bridge.c:144:2: note: in expansion of macro 'dev_dbg' dev_dbg(dev, "%s %d %d\n", __func__, status, readl(csr_ctrl_addr)); ^~~~~~~ drivers/fpga/altera-freeze-bridge.c:162:15: warning: format '%d' expects argument of type 'int', but argument 6 has type 'long unsigned int' [-Wformat=] dev_dbg(dev, "%s %d %d\n", __func__, status, readl(csr_ctrl_addr)); ^ include/linux/dynamic_debug.h:134:39: note: in definition of macro 'dynamic_dev_dbg' __dynamic_dev_dbg(&descriptor, dev, fmt, \ ^~~ drivers/fpga/altera-freeze-bridge.c:162:2: note: in expansion of macro 'dev_dbg' dev_dbg(dev, "%s %d %d\n", __func__, status, readl(csr_ctrl_addr)); ^~~~~~~ vim +107 drivers/fpga/altera-freeze-bridge.c ca24a648 Alan Tull 2016-11-01 12 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for ca24a648 Alan Tull 2016-11-01 13 * more details. ca24a648 Alan Tull 2016-11-01 14 * ca24a648 Alan Tull 2016-11-01 15 * You should have received a copy of the GNU General Public License along with ca24a648 Alan Tull 2016-11-01 16 * this program. If not, see <http://www.gnu.org/licenses/>. ca24a648 Alan Tull 2016-11-01 17 */ ca24a648 Alan Tull 2016-11-01 @18 #include <linux/delay.h> ca24a648 Alan Tull 2016-11-01 19 #include <linux/io.h> ca24a648 Alan Tull 2016-11-01 20 #include <linux/kernel.h> ca24a648 Alan Tull 2016-11-01 21 #include <linux/of_device.h> ca24a648 Alan Tull 2016-11-01 22 #include <linux/module.h> ca24a648 Alan Tull 2016-11-01 23 #include <linux/fpga/fpga-bridge.h> ca24a648 Alan Tull 2016-11-01 24 ca24a648 Alan Tull 2016-11-01 25 #define FREEZE_CSR_STATUS_OFFSET 0 ca24a648 Alan Tull 2016-11-01 26 #define FREEZE_CSR_CTRL_OFFSET 4 ca24a648 Alan Tull 2016-11-01 27 #define FREEZE_CSR_ILLEGAL_REQ_OFFSET 8 ca24a648 Alan Tull 2016-11-01 28 #define FREEZE_CSR_REG_VERSION 12 ca24a648 Alan Tull 2016-11-01 29 ca24a648 Alan Tull 2016-11-01 30 #define FREEZE_CSR_SUPPORTED_VERSION 2 ca24a648 Alan Tull 2016-11-01 31 ca24a648 Alan Tull 2016-11-01 32 #define FREEZE_CSR_STATUS_FREEZE_REQ_DONE BIT(0) ca24a648 Alan Tull 2016-11-01 33 #define FREEZE_CSR_STATUS_UNFREEZE_REQ_DONE BIT(1) ca24a648 Alan Tull 2016-11-01 34 ca24a648 Alan Tull 2016-11-01 35 #define FREEZE_CSR_CTRL_FREEZE_REQ BIT(0) ca24a648 Alan Tull 2016-11-01 36 #define FREEZE_CSR_CTRL_RESET_REQ BIT(1) ca24a648 Alan Tull 2016-11-01 37 #define FREEZE_CSR_CTRL_UNFREEZE_REQ BIT(2) ca24a648 Alan Tull 2016-11-01 38 ca24a648 Alan Tull 2016-11-01 39 #define FREEZE_BRIDGE_NAME "freeze" ca24a648 Alan Tull 2016-11-01 40 ca24a648 Alan Tull 2016-11-01 41 struct altera_freeze_br_data { ca24a648 Alan Tull 2016-11-01 42 struct device *dev; ca24a648 Alan Tull 2016-11-01 43 void __iomem *base_addr; ca24a648 Alan Tull 2016-11-01 44 bool enable; ca24a648 Alan Tull 2016-11-01 45 }; ca24a648 Alan Tull 2016-11-01 46 ca24a648 Alan Tull 2016-11-01 47 /* ca24a648 Alan Tull 2016-11-01 48 * Poll status until status bit is set or we have a timeout. ca24a648 Alan Tull 2016-11-01 49 */ ca24a648 Alan Tull 2016-11-01 50 static int altera_freeze_br_req_ack(struct altera_freeze_br_data *priv, ca24a648 Alan Tull 2016-11-01 51 u32 timeout, u32 req_ack) ca24a648 Alan Tull 2016-11-01 52 { ca24a648 Alan Tull 2016-11-01 53 struct device *dev = priv->dev; ca24a648 Alan Tull 2016-11-01 54 void __iomem *csr_illegal_req_addr = priv->base_addr + ca24a648 Alan Tull 2016-11-01 55 FREEZE_CSR_ILLEGAL_REQ_OFFSET; ca24a648 Alan Tull 2016-11-01 56 u32 status, illegal, ctrl; ca24a648 Alan Tull 2016-11-01 57 int ret = -ETIMEDOUT; ca24a648 Alan Tull 2016-11-01 58 ca24a648 Alan Tull 2016-11-01 59 do { ca24a648 Alan Tull 2016-11-01 60 illegal = readl(csr_illegal_req_addr); ca24a648 Alan Tull 2016-11-01 61 if (illegal) { ca24a648 Alan Tull 2016-11-01 62 dev_err(dev, "illegal request detected 0x%x", illegal); ca24a648 Alan Tull 2016-11-01 63 ca24a648 Alan Tull 2016-11-01 64 writel(1, csr_illegal_req_addr); ca24a648 Alan Tull 2016-11-01 65 ca24a648 Alan Tull 2016-11-01 66 illegal = readl(csr_illegal_req_addr); ca24a648 Alan Tull 2016-11-01 67 if (illegal) ca24a648 Alan Tull 2016-11-01 68 dev_err(dev, "illegal request not cleared 0x%x", ca24a648 Alan Tull 2016-11-01 69 illegal); ca24a648 Alan Tull 2016-11-01 70 ca24a648 Alan Tull 2016-11-01 71 ret = -EINVAL; ca24a648 Alan Tull 2016-11-01 72 break; ca24a648 Alan Tull 2016-11-01 73 } ca24a648 Alan Tull 2016-11-01 74 ca24a648 Alan Tull 2016-11-01 75 status = readl(priv->base_addr + FREEZE_CSR_STATUS_OFFSET); ca24a648 Alan Tull 2016-11-01 76 dev_dbg(dev, "%s %x %x\n", __func__, status, req_ack); ca24a648 Alan Tull 2016-11-01 77 status &= req_ack; ca24a648 Alan Tull 2016-11-01 78 if (status) { ca24a648 Alan Tull 2016-11-01 79 ctrl = readl(priv->base_addr + FREEZE_CSR_CTRL_OFFSET); ca24a648 Alan Tull 2016-11-01 80 dev_dbg(dev, "%s request %x acknowledged %x %x\n", ca24a648 Alan Tull 2016-11-01 81 __func__, req_ack, status, ctrl); ca24a648 Alan Tull 2016-11-01 82 ret = 0; ca24a648 Alan Tull 2016-11-01 83 break; ca24a648 Alan Tull 2016-11-01 84 } ca24a648 Alan Tull 2016-11-01 85 ca24a648 Alan Tull 2016-11-01 86 udelay(1); ca24a648 Alan Tull 2016-11-01 87 } while (timeout--); ca24a648 Alan Tull 2016-11-01 88 ca24a648 Alan Tull 2016-11-01 89 if (ret == -ETIMEDOUT) ca24a648 Alan Tull 2016-11-01 90 dev_err(dev, "%s timeout waiting for 0x%x\n", ca24a648 Alan Tull 2016-11-01 91 __func__, req_ack); ca24a648 Alan Tull 2016-11-01 92 ca24a648 Alan Tull 2016-11-01 93 return ret; ca24a648 Alan Tull 2016-11-01 94 } ca24a648 Alan Tull 2016-11-01 95 ca24a648 Alan Tull 2016-11-01 96 static int altera_freeze_br_do_freeze(struct altera_freeze_br_data *priv, ca24a648 Alan Tull 2016-11-01 97 u32 timeout) ca24a648 Alan Tull 2016-11-01 98 { ca24a648 Alan Tull 2016-11-01 99 struct device *dev = priv->dev; ca24a648 Alan Tull 2016-11-01 100 void __iomem *csr_ctrl_addr = priv->base_addr + ca24a648 Alan Tull 2016-11-01 101 FREEZE_CSR_CTRL_OFFSET; ca24a648 Alan Tull 2016-11-01 102 u32 status; ca24a648 Alan Tull 2016-11-01 103 int ret; ca24a648 Alan Tull 2016-11-01 104 ca24a648 Alan Tull 2016-11-01 105 status = readl(priv->base_addr + FREEZE_CSR_STATUS_OFFSET); ca24a648 Alan Tull 2016-11-01 106 ca24a648 Alan Tull 2016-11-01 @107 dev_dbg(dev, "%s %d %d\n", __func__, status, readl(csr_ctrl_addr)); ca24a648 Alan Tull 2016-11-01 108 ca24a648 Alan Tull 2016-11-01 109 if (status & FREEZE_CSR_STATUS_FREEZE_REQ_DONE) { ca24a648 Alan Tull 2016-11-01 110 dev_dbg(dev, "%s bridge already disabled %d\n", :::::: The code at line 107 was first introduced by commit :::::: ca24a648f535a02b4163ca4f4d2e51869f155a3a fpga: add altera freeze bridge support :::::: TO: Alan Tull <atull@opensource.altera.com> :::::: CC: Greg Kroah-Hartman <gregkh@linuxfoundation.org> --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation [-- Attachment #2: .config.gz --] [-- Type: application/gzip, Size: 41284 bytes --] ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] fpga: allow to compile-test Altera FPGA bridge drivers 2017-04-12 5:47 ` kbuild test robot @ 2017-04-18 15:27 ` matthew.gerlach 2017-04-19 12:07 ` Tobias Klauser 0 siblings, 1 reply; 8+ messages in thread From: matthew.gerlach @ 2017-04-18 15:27 UTC (permalink / raw) To: kbuild test robot Cc: Tobias Klauser, kbuild-all, Alan Tull, Moritz Fischer, linux-fpga, linux-kernel On Wed, 12 Apr 2017, kbuild test robot wrote: > Hi Tobias, Hi Tobias, This is very interesting issue brought up by your patch that turns on COMPILE_TEST in drivers/fpga/Kconfig. See my comment below. Matthew Gerlach > > [auto build test WARNING on linus/master] > [also build test WARNING on v4.11-rc6 next-20170411] > [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] > > url: https://github.com/0day-ci/linux/commits/Tobias-Klauser/fpga-allow-to-compile-test-Altera-FPGA-bridge-drivers/20170411-181401 > config: m32r-allmodconfig (attached as .config) > compiler: m32r-linux-gcc (GCC) 6.2.0 > reproduce: > wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > chmod +x ~/bin/make.cross > # save the attached .config to linux build tree > make.cross ARCH=m32r > > All warnings (new ones prefixed by >>): > > In file included from include/linux/printk.h:329:0, > from include/linux/kernel.h:13, > from include/linux/delay.h:21, > from drivers/fpga/altera-freeze-bridge.c:18: > drivers/fpga/altera-freeze-bridge.c: In function 'altera_freeze_br_do_freeze': >>> drivers/fpga/altera-freeze-bridge.c:107:15: warning: format '%d' expects argument of type 'int', but argument 6 has type 'long unsigned int' [-Wformat=] > dev_dbg(dev, "%s %d %d\n", __func__, status, readl(csr_ctrl_addr)); > ^ The line above compiles cleanly for x86 and 32-bit ARM. Interestingly, the variable, status, is declared locally as an u32 and is assigned by a call to readl(), and there is no warning. On x86 and 32-bit arm, readl(), is defined as returning u32, but on this m32r processor, readl() returns unsigned long, thus the warning. A possilbe fix to this warning is to declare another local u32, ctrl, and assign a value with a call to readl(). In other words, mimic what is done with the variable, status. While this would get rid of the warning, I can't help but think there is either a compiler problem or a problem with m32r. > include/linux/dynamic_debug.h:134:39: note: in definition of macro 'dynamic_dev_dbg' > __dynamic_dev_dbg(&descriptor, dev, fmt, \ > ^~~ >>> drivers/fpga/altera-freeze-bridge.c:107:2: note: in expansion of macro 'dev_dbg' > dev_dbg(dev, "%s %d %d\n", __func__, status, readl(csr_ctrl_addr)); > ^~~~~~~ > drivers/fpga/altera-freeze-bridge.c: In function 'altera_freeze_br_do_unfreeze': > drivers/fpga/altera-freeze-bridge.c:144:15: warning: format '%d' expects argument of type 'int', but argument 6 has type 'long unsigned int' [-Wformat=] > dev_dbg(dev, "%s %d %d\n", __func__, status, readl(csr_ctrl_addr)); > ^ > include/linux/dynamic_debug.h:134:39: note: in definition of macro 'dynamic_dev_dbg' > __dynamic_dev_dbg(&descriptor, dev, fmt, \ > ^~~ > drivers/fpga/altera-freeze-bridge.c:144:2: note: in expansion of macro 'dev_dbg' > dev_dbg(dev, "%s %d %d\n", __func__, status, readl(csr_ctrl_addr)); > ^~~~~~~ > drivers/fpga/altera-freeze-bridge.c:162:15: warning: format '%d' expects argument of type 'int', but argument 6 has type 'long unsigned int' [-Wformat=] > dev_dbg(dev, "%s %d %d\n", __func__, status, readl(csr_ctrl_addr)); > ^ > include/linux/dynamic_debug.h:134:39: note: in definition of macro 'dynamic_dev_dbg' > __dynamic_dev_dbg(&descriptor, dev, fmt, \ > ^~~ > drivers/fpga/altera-freeze-bridge.c:162:2: note: in expansion of macro 'dev_dbg' > dev_dbg(dev, "%s %d %d\n", __func__, status, readl(csr_ctrl_addr)); > ^~~~~~~ > > vim +107 drivers/fpga/altera-freeze-bridge.c > > ca24a648 Alan Tull 2016-11-01 12 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for > ca24a648 Alan Tull 2016-11-01 13 * more details. > ca24a648 Alan Tull 2016-11-01 14 * > ca24a648 Alan Tull 2016-11-01 15 * You should have received a copy of the GNU General Public License along with > ca24a648 Alan Tull 2016-11-01 16 * this program. If not, see <http://www.gnu.org/licenses/>. > ca24a648 Alan Tull 2016-11-01 17 */ > ca24a648 Alan Tull 2016-11-01 @18 #include <linux/delay.h> > ca24a648 Alan Tull 2016-11-01 19 #include <linux/io.h> > ca24a648 Alan Tull 2016-11-01 20 #include <linux/kernel.h> > ca24a648 Alan Tull 2016-11-01 21 #include <linux/of_device.h> > ca24a648 Alan Tull 2016-11-01 22 #include <linux/module.h> > ca24a648 Alan Tull 2016-11-01 23 #include <linux/fpga/fpga-bridge.h> > ca24a648 Alan Tull 2016-11-01 24 > ca24a648 Alan Tull 2016-11-01 25 #define FREEZE_CSR_STATUS_OFFSET 0 > ca24a648 Alan Tull 2016-11-01 26 #define FREEZE_CSR_CTRL_OFFSET 4 > ca24a648 Alan Tull 2016-11-01 27 #define FREEZE_CSR_ILLEGAL_REQ_OFFSET 8 > ca24a648 Alan Tull 2016-11-01 28 #define FREEZE_CSR_REG_VERSION 12 > ca24a648 Alan Tull 2016-11-01 29 > ca24a648 Alan Tull 2016-11-01 30 #define FREEZE_CSR_SUPPORTED_VERSION 2 > ca24a648 Alan Tull 2016-11-01 31 > ca24a648 Alan Tull 2016-11-01 32 #define FREEZE_CSR_STATUS_FREEZE_REQ_DONE BIT(0) > ca24a648 Alan Tull 2016-11-01 33 #define FREEZE_CSR_STATUS_UNFREEZE_REQ_DONE BIT(1) > ca24a648 Alan Tull 2016-11-01 34 > ca24a648 Alan Tull 2016-11-01 35 #define FREEZE_CSR_CTRL_FREEZE_REQ BIT(0) > ca24a648 Alan Tull 2016-11-01 36 #define FREEZE_CSR_CTRL_RESET_REQ BIT(1) > ca24a648 Alan Tull 2016-11-01 37 #define FREEZE_CSR_CTRL_UNFREEZE_REQ BIT(2) > ca24a648 Alan Tull 2016-11-01 38 > ca24a648 Alan Tull 2016-11-01 39 #define FREEZE_BRIDGE_NAME "freeze" > ca24a648 Alan Tull 2016-11-01 40 > ca24a648 Alan Tull 2016-11-01 41 struct altera_freeze_br_data { > ca24a648 Alan Tull 2016-11-01 42 struct device *dev; > ca24a648 Alan Tull 2016-11-01 43 void __iomem *base_addr; > ca24a648 Alan Tull 2016-11-01 44 bool enable; > ca24a648 Alan Tull 2016-11-01 45 }; > ca24a648 Alan Tull 2016-11-01 46 > ca24a648 Alan Tull 2016-11-01 47 /* > ca24a648 Alan Tull 2016-11-01 48 * Poll status until status bit is set or we have a timeout. > ca24a648 Alan Tull 2016-11-01 49 */ > ca24a648 Alan Tull 2016-11-01 50 static int altera_freeze_br_req_ack(struct altera_freeze_br_data *priv, > ca24a648 Alan Tull 2016-11-01 51 u32 timeout, u32 req_ack) > ca24a648 Alan Tull 2016-11-01 52 { > ca24a648 Alan Tull 2016-11-01 53 struct device *dev = priv->dev; > ca24a648 Alan Tull 2016-11-01 54 void __iomem *csr_illegal_req_addr = priv->base_addr + > ca24a648 Alan Tull 2016-11-01 55 FREEZE_CSR_ILLEGAL_REQ_OFFSET; > ca24a648 Alan Tull 2016-11-01 56 u32 status, illegal, ctrl; > ca24a648 Alan Tull 2016-11-01 57 int ret = -ETIMEDOUT; > ca24a648 Alan Tull 2016-11-01 58 > ca24a648 Alan Tull 2016-11-01 59 do { > ca24a648 Alan Tull 2016-11-01 60 illegal = readl(csr_illegal_req_addr); > ca24a648 Alan Tull 2016-11-01 61 if (illegal) { > ca24a648 Alan Tull 2016-11-01 62 dev_err(dev, "illegal request detected 0x%x", illegal); > ca24a648 Alan Tull 2016-11-01 63 > ca24a648 Alan Tull 2016-11-01 64 writel(1, csr_illegal_req_addr); > ca24a648 Alan Tull 2016-11-01 65 > ca24a648 Alan Tull 2016-11-01 66 illegal = readl(csr_illegal_req_addr); > ca24a648 Alan Tull 2016-11-01 67 if (illegal) > ca24a648 Alan Tull 2016-11-01 68 dev_err(dev, "illegal request not cleared 0x%x", > ca24a648 Alan Tull 2016-11-01 69 illegal); > ca24a648 Alan Tull 2016-11-01 70 > ca24a648 Alan Tull 2016-11-01 71 ret = -EINVAL; > ca24a648 Alan Tull 2016-11-01 72 break; > ca24a648 Alan Tull 2016-11-01 73 } > ca24a648 Alan Tull 2016-11-01 74 > ca24a648 Alan Tull 2016-11-01 75 status = readl(priv->base_addr + FREEZE_CSR_STATUS_OFFSET); > ca24a648 Alan Tull 2016-11-01 76 dev_dbg(dev, "%s %x %x\n", __func__, status, req_ack); > ca24a648 Alan Tull 2016-11-01 77 status &= req_ack; > ca24a648 Alan Tull 2016-11-01 78 if (status) { > ca24a648 Alan Tull 2016-11-01 79 ctrl = readl(priv->base_addr + FREEZE_CSR_CTRL_OFFSET); > ca24a648 Alan Tull 2016-11-01 80 dev_dbg(dev, "%s request %x acknowledged %x %x\n", > ca24a648 Alan Tull 2016-11-01 81 __func__, req_ack, status, ctrl); > ca24a648 Alan Tull 2016-11-01 82 ret = 0; > ca24a648 Alan Tull 2016-11-01 83 break; > ca24a648 Alan Tull 2016-11-01 84 } > ca24a648 Alan Tull 2016-11-01 85 > ca24a648 Alan Tull 2016-11-01 86 udelay(1); > ca24a648 Alan Tull 2016-11-01 87 } while (timeout--); > ca24a648 Alan Tull 2016-11-01 88 > ca24a648 Alan Tull 2016-11-01 89 if (ret == -ETIMEDOUT) > ca24a648 Alan Tull 2016-11-01 90 dev_err(dev, "%s timeout waiting for 0x%x\n", > ca24a648 Alan Tull 2016-11-01 91 __func__, req_ack); > ca24a648 Alan Tull 2016-11-01 92 > ca24a648 Alan Tull 2016-11-01 93 return ret; > ca24a648 Alan Tull 2016-11-01 94 } > ca24a648 Alan Tull 2016-11-01 95 > ca24a648 Alan Tull 2016-11-01 96 static int altera_freeze_br_do_freeze(struct altera_freeze_br_data *priv, > ca24a648 Alan Tull 2016-11-01 97 u32 timeout) > ca24a648 Alan Tull 2016-11-01 98 { > ca24a648 Alan Tull 2016-11-01 99 struct device *dev = priv->dev; > ca24a648 Alan Tull 2016-11-01 100 void __iomem *csr_ctrl_addr = priv->base_addr + > ca24a648 Alan Tull 2016-11-01 101 FREEZE_CSR_CTRL_OFFSET; > ca24a648 Alan Tull 2016-11-01 102 u32 status; > ca24a648 Alan Tull 2016-11-01 103 int ret; > ca24a648 Alan Tull 2016-11-01 104 > ca24a648 Alan Tull 2016-11-01 105 status = readl(priv->base_addr + FREEZE_CSR_STATUS_OFFSET); > ca24a648 Alan Tull 2016-11-01 106 > ca24a648 Alan Tull 2016-11-01 @107 dev_dbg(dev, "%s %d %d\n", __func__, status, readl(csr_ctrl_addr)); > ca24a648 Alan Tull 2016-11-01 108 > ca24a648 Alan Tull 2016-11-01 109 if (status & FREEZE_CSR_STATUS_FREEZE_REQ_DONE) { > ca24a648 Alan Tull 2016-11-01 110 dev_dbg(dev, "%s bridge already disabled %d\n", > > :::::: The code at line 107 was first introduced by commit > :::::: ca24a648f535a02b4163ca4f4d2e51869f155a3a fpga: add altera freeze bridge support > > :::::: TO: Alan Tull <atull@opensource.altera.com> > :::::: CC: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > > --- > 0-DAY kernel test infrastructure Open Source Technology Center > https://lists.01.org/pipermail/kbuild-all Intel Corporation > ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] fpga: allow to compile-test Altera FPGA bridge drivers 2017-04-18 15:27 ` matthew.gerlach @ 2017-04-19 12:07 ` Tobias Klauser 0 siblings, 0 replies; 8+ messages in thread From: Tobias Klauser @ 2017-04-19 12:07 UTC (permalink / raw) To: matthew.gerlach Cc: kbuild test robot, kbuild-all, Alan Tull, Moritz Fischer, linux-fpga, linux-kernel Hi Matthew Thanks for the detailed analysis! On 2017-04-18 at 17:27:15 +0200, matthew.gerlach@linux.intel.com <matthew.gerlach@linux.intel.com> wrote: [...] > >[auto build test WARNING on linus/master] > >[also build test WARNING on v4.11-rc6 next-20170411] > >[if your patch is applied to the wrong git tree, please drop us a note to help improve the system] > > > >url: https://github.com/0day-ci/linux/commits/Tobias-Klauser/fpga-allow-to-compile-test-Altera-FPGA-bridge-drivers/20170411-181401 > >config: m32r-allmodconfig (attached as .config) > >compiler: m32r-linux-gcc (GCC) 6.2.0 > >reproduce: > > wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > > chmod +x ~/bin/make.cross > > # save the attached .config to linux build tree > > make.cross ARCH=m32r > > > >All warnings (new ones prefixed by >>): > > > > In file included from include/linux/printk.h:329:0, > > from include/linux/kernel.h:13, > > from include/linux/delay.h:21, > > from drivers/fpga/altera-freeze-bridge.c:18: > > drivers/fpga/altera-freeze-bridge.c: In function 'altera_freeze_br_do_freeze': > >>>drivers/fpga/altera-freeze-bridge.c:107:15: warning: format '%d' expects argument of type 'int', but argument 6 has type 'long unsigned int' [-Wformat=] > > dev_dbg(dev, "%s %d %d\n", __func__, status, readl(csr_ctrl_addr)); > > ^ > > The line above compiles cleanly for x86 and 32-bit ARM. > Interestingly, the variable, status, is declared locally as an u32 > and is assigned by a > call to readl(), and there is no warning. On x86 and 32-bit arm, > readl(), is defined as returning u32, but on this m32r processor, > readl() returns unsigned long, thus the warning. > > A possilbe fix to this warning is to declare another local u32, ctrl, and > assign a value with a call to readl(). In other words, mimic what > is done with the variable, status. While this would get rid of the > warning, I can't help but think there is either a compiler problem > or a problem with m32r. I'd say this is a problem with how readl() is declared on m32r. On almost all architectures readl() returns a u32 (which seems to be unsigned int on all architectures). Judging from a quick m32r allmodconfig build, it seems the warning occurs at quite a few places in the kernel already and a quick Google search also turns up quite some patches, where the same warning occured [1], [2], [3]. [1] https://www.spinics.net/lists/linux-ide/msg53956.html [2] https://lists.01.org/pipermail/kbuild-all/2016-September/025251.html [3] https://www.spinics.net/lists/linux-bluetooth/msg70076.html IMO the proper fix would probably be to fix the readl() implementation on m32r. > > > include/linux/dynamic_debug.h:134:39: note: in definition of macro 'dynamic_dev_dbg' > > __dynamic_dev_dbg(&descriptor, dev, fmt, \ > > ^~~ > >>>drivers/fpga/altera-freeze-bridge.c:107:2: note: in expansion of macro 'dev_dbg' > > dev_dbg(dev, "%s %d %d\n", __func__, status, readl(csr_ctrl_addr)); > > ^~~~~~~ > > drivers/fpga/altera-freeze-bridge.c: In function 'altera_freeze_br_do_unfreeze': > > drivers/fpga/altera-freeze-bridge.c:144:15: warning: format '%d' expects argument of type 'int', but argument 6 has type 'long unsigned int' [-Wformat=] > > dev_dbg(dev, "%s %d %d\n", __func__, status, readl(csr_ctrl_addr)); > > ^ > > include/linux/dynamic_debug.h:134:39: note: in definition of macro 'dynamic_dev_dbg' > > __dynamic_dev_dbg(&descriptor, dev, fmt, \ > > ^~~ > > drivers/fpga/altera-freeze-bridge.c:144:2: note: in expansion of macro 'dev_dbg' > > dev_dbg(dev, "%s %d %d\n", __func__, status, readl(csr_ctrl_addr)); > > ^~~~~~~ > > drivers/fpga/altera-freeze-bridge.c:162:15: warning: format '%d' expects argument of type 'int', but argument 6 has type 'long unsigned int' [-Wformat=] > > dev_dbg(dev, "%s %d %d\n", __func__, status, readl(csr_ctrl_addr)); > > ^ > > include/linux/dynamic_debug.h:134:39: note: in definition of macro 'dynamic_dev_dbg' > > __dynamic_dev_dbg(&descriptor, dev, fmt, \ > > ^~~ > > drivers/fpga/altera-freeze-bridge.c:162:2: note: in expansion of macro 'dev_dbg' > > dev_dbg(dev, "%s %d %d\n", __func__, status, readl(csr_ctrl_addr)); > > ^~~~~~~ > > > >vim +107 drivers/fpga/altera-freeze-bridge.c > > > >ca24a648 Alan Tull 2016-11-01 12 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for > >ca24a648 Alan Tull 2016-11-01 13 * more details. > >ca24a648 Alan Tull 2016-11-01 14 * > >ca24a648 Alan Tull 2016-11-01 15 * You should have received a copy of the GNU General Public License along with > >ca24a648 Alan Tull 2016-11-01 16 * this program. If not, see <http://www.gnu.org/licenses/>. > >ca24a648 Alan Tull 2016-11-01 17 */ > >ca24a648 Alan Tull 2016-11-01 @18 #include <linux/delay.h> > >ca24a648 Alan Tull 2016-11-01 19 #include <linux/io.h> > >ca24a648 Alan Tull 2016-11-01 20 #include <linux/kernel.h> > >ca24a648 Alan Tull 2016-11-01 21 #include <linux/of_device.h> > >ca24a648 Alan Tull 2016-11-01 22 #include <linux/module.h> > >ca24a648 Alan Tull 2016-11-01 23 #include <linux/fpga/fpga-bridge.h> > >ca24a648 Alan Tull 2016-11-01 24 > >ca24a648 Alan Tull 2016-11-01 25 #define FREEZE_CSR_STATUS_OFFSET 0 > >ca24a648 Alan Tull 2016-11-01 26 #define FREEZE_CSR_CTRL_OFFSET 4 > >ca24a648 Alan Tull 2016-11-01 27 #define FREEZE_CSR_ILLEGAL_REQ_OFFSET 8 > >ca24a648 Alan Tull 2016-11-01 28 #define FREEZE_CSR_REG_VERSION 12 > >ca24a648 Alan Tull 2016-11-01 29 > >ca24a648 Alan Tull 2016-11-01 30 #define FREEZE_CSR_SUPPORTED_VERSION 2 > >ca24a648 Alan Tull 2016-11-01 31 > >ca24a648 Alan Tull 2016-11-01 32 #define FREEZE_CSR_STATUS_FREEZE_REQ_DONE BIT(0) > >ca24a648 Alan Tull 2016-11-01 33 #define FREEZE_CSR_STATUS_UNFREEZE_REQ_DONE BIT(1) > >ca24a648 Alan Tull 2016-11-01 34 > >ca24a648 Alan Tull 2016-11-01 35 #define FREEZE_CSR_CTRL_FREEZE_REQ BIT(0) > >ca24a648 Alan Tull 2016-11-01 36 #define FREEZE_CSR_CTRL_RESET_REQ BIT(1) > >ca24a648 Alan Tull 2016-11-01 37 #define FREEZE_CSR_CTRL_UNFREEZE_REQ BIT(2) > >ca24a648 Alan Tull 2016-11-01 38 > >ca24a648 Alan Tull 2016-11-01 39 #define FREEZE_BRIDGE_NAME "freeze" > >ca24a648 Alan Tull 2016-11-01 40 > >ca24a648 Alan Tull 2016-11-01 41 struct altera_freeze_br_data { > >ca24a648 Alan Tull 2016-11-01 42 struct device *dev; > >ca24a648 Alan Tull 2016-11-01 43 void __iomem *base_addr; > >ca24a648 Alan Tull 2016-11-01 44 bool enable; > >ca24a648 Alan Tull 2016-11-01 45 }; > >ca24a648 Alan Tull 2016-11-01 46 > >ca24a648 Alan Tull 2016-11-01 47 /* > >ca24a648 Alan Tull 2016-11-01 48 * Poll status until status bit is set or we have a timeout. > >ca24a648 Alan Tull 2016-11-01 49 */ > >ca24a648 Alan Tull 2016-11-01 50 static int altera_freeze_br_req_ack(struct altera_freeze_br_data *priv, > >ca24a648 Alan Tull 2016-11-01 51 u32 timeout, u32 req_ack) > >ca24a648 Alan Tull 2016-11-01 52 { > >ca24a648 Alan Tull 2016-11-01 53 struct device *dev = priv->dev; > >ca24a648 Alan Tull 2016-11-01 54 void __iomem *csr_illegal_req_addr = priv->base_addr + > >ca24a648 Alan Tull 2016-11-01 55 FREEZE_CSR_ILLEGAL_REQ_OFFSET; > >ca24a648 Alan Tull 2016-11-01 56 u32 status, illegal, ctrl; > >ca24a648 Alan Tull 2016-11-01 57 int ret = -ETIMEDOUT; > >ca24a648 Alan Tull 2016-11-01 58 > >ca24a648 Alan Tull 2016-11-01 59 do { > >ca24a648 Alan Tull 2016-11-01 60 illegal = readl(csr_illegal_req_addr); > >ca24a648 Alan Tull 2016-11-01 61 if (illegal) { > >ca24a648 Alan Tull 2016-11-01 62 dev_err(dev, "illegal request detected 0x%x", illegal); > >ca24a648 Alan Tull 2016-11-01 63 > >ca24a648 Alan Tull 2016-11-01 64 writel(1, csr_illegal_req_addr); > >ca24a648 Alan Tull 2016-11-01 65 > >ca24a648 Alan Tull 2016-11-01 66 illegal = readl(csr_illegal_req_addr); > >ca24a648 Alan Tull 2016-11-01 67 if (illegal) > >ca24a648 Alan Tull 2016-11-01 68 dev_err(dev, "illegal request not cleared 0x%x", > >ca24a648 Alan Tull 2016-11-01 69 illegal); > >ca24a648 Alan Tull 2016-11-01 70 > >ca24a648 Alan Tull 2016-11-01 71 ret = -EINVAL; > >ca24a648 Alan Tull 2016-11-01 72 break; > >ca24a648 Alan Tull 2016-11-01 73 } > >ca24a648 Alan Tull 2016-11-01 74 > >ca24a648 Alan Tull 2016-11-01 75 status = readl(priv->base_addr + FREEZE_CSR_STATUS_OFFSET); > >ca24a648 Alan Tull 2016-11-01 76 dev_dbg(dev, "%s %x %x\n", __func__, status, req_ack); > >ca24a648 Alan Tull 2016-11-01 77 status &= req_ack; > >ca24a648 Alan Tull 2016-11-01 78 if (status) { > >ca24a648 Alan Tull 2016-11-01 79 ctrl = readl(priv->base_addr + FREEZE_CSR_CTRL_OFFSET); > >ca24a648 Alan Tull 2016-11-01 80 dev_dbg(dev, "%s request %x acknowledged %x %x\n", > >ca24a648 Alan Tull 2016-11-01 81 __func__, req_ack, status, ctrl); > >ca24a648 Alan Tull 2016-11-01 82 ret = 0; > >ca24a648 Alan Tull 2016-11-01 83 break; > >ca24a648 Alan Tull 2016-11-01 84 } > >ca24a648 Alan Tull 2016-11-01 85 > >ca24a648 Alan Tull 2016-11-01 86 udelay(1); > >ca24a648 Alan Tull 2016-11-01 87 } while (timeout--); > >ca24a648 Alan Tull 2016-11-01 88 > >ca24a648 Alan Tull 2016-11-01 89 if (ret == -ETIMEDOUT) > >ca24a648 Alan Tull 2016-11-01 90 dev_err(dev, "%s timeout waiting for 0x%x\n", > >ca24a648 Alan Tull 2016-11-01 91 __func__, req_ack); > >ca24a648 Alan Tull 2016-11-01 92 > >ca24a648 Alan Tull 2016-11-01 93 return ret; > >ca24a648 Alan Tull 2016-11-01 94 } > >ca24a648 Alan Tull 2016-11-01 95 > >ca24a648 Alan Tull 2016-11-01 96 static int altera_freeze_br_do_freeze(struct altera_freeze_br_data *priv, > >ca24a648 Alan Tull 2016-11-01 97 u32 timeout) > >ca24a648 Alan Tull 2016-11-01 98 { > >ca24a648 Alan Tull 2016-11-01 99 struct device *dev = priv->dev; > >ca24a648 Alan Tull 2016-11-01 100 void __iomem *csr_ctrl_addr = priv->base_addr + > >ca24a648 Alan Tull 2016-11-01 101 FREEZE_CSR_CTRL_OFFSET; > >ca24a648 Alan Tull 2016-11-01 102 u32 status; > >ca24a648 Alan Tull 2016-11-01 103 int ret; > >ca24a648 Alan Tull 2016-11-01 104 > >ca24a648 Alan Tull 2016-11-01 105 status = readl(priv->base_addr + FREEZE_CSR_STATUS_OFFSET); > >ca24a648 Alan Tull 2016-11-01 106 > >ca24a648 Alan Tull 2016-11-01 @107 dev_dbg(dev, "%s %d %d\n", __func__, status, readl(csr_ctrl_addr)); > >ca24a648 Alan Tull 2016-11-01 108 > >ca24a648 Alan Tull 2016-11-01 109 if (status & FREEZE_CSR_STATUS_FREEZE_REQ_DONE) { > >ca24a648 Alan Tull 2016-11-01 110 dev_dbg(dev, "%s bridge already disabled %d\n", > > > >:::::: The code at line 107 was first introduced by commit > >:::::: ca24a648f535a02b4163ca4f4d2e51869f155a3a fpga: add altera freeze bridge support > > > >:::::: TO: Alan Tull <atull@opensource.altera.com> > >:::::: CC: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > > > >--- > >0-DAY kernel test infrastructure Open Source Technology Center > >https://lists.01.org/pipermail/kbuild-all Intel Corporation > > > ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2017-04-19 12:07 UTC | newest] Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2017-04-11 9:22 [PATCH] fpga: allow to compile-test Altera FPGA bridge drivers Tobias Klauser 2017-04-11 14:27 ` Alan Tull 2017-04-11 14:35 ` Tobias Klauser 2017-04-11 15:21 ` Alan Tull 2017-04-12 6:51 ` Tobias Klauser 2017-04-12 5:47 ` kbuild test robot 2017-04-18 15:27 ` matthew.gerlach 2017-04-19 12:07 ` Tobias Klauser
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®