From: Guenter Roeck <linux@roeck-us.net> To: Stafford Horne <shorne@gmail.com> Cc: LKML <linux-kernel@vger.kernel.org>, Bjorn Helgaas <helgaas@kernel.org>, Arnd Bergmann <arnd@arndb.de>, Jonas Bonn <jonas@southpole.se>, Stefan Kristiansson <stefan.kristiansson@saunalahti.fi>, Palmer Dabbelt <palmer@rivosinc.com>, Peter Zijlstra <peterz@infradead.org>, openrisc@lists.librecores.org Subject: Re: [PATCH v3 2/3] openrisc: Add pci bus support Date: Thu, 28 Jul 2022 23:10:37 -0700 [thread overview] Message-ID: <ceb732ee-0ea1-b471-0b57-3cc3bcb80a2d@roeck-us.net> (raw) In-Reply-To: <YuN1PWcDukEPdtQy@antec> On 7/28/22 22:50, Stafford Horne wrote: > On Thu, Jul 28, 2022 at 08:37:28PM -0700, Guenter Roeck wrote: >> On Mon, Jul 25, 2022 at 11:07:36AM +0900, Stafford Horne wrote: >>> This patch adds required definitions to allow for PCI buses on OpenRISC. >>> This is being tested on the OpenRISC QEMU virt platform which is in >>> development. >>> >>> OpenRISC does not have IO ports so we keep the definition of >>> IO_SPACE_LIMIT and PIO_RESERVED to be 0. >>> >>> Note, since commit 66bcd06099bb ("parport_pc: Also enable driver for PCI >>> systems") all platforms that support PCI also need to support parallel >>> port. We add a generic header to support compiling parallel port >>> drivers, though they generally will not work as they require IO ports. >>> >>> Signed-off-by: Stafford Horne <shorne@gmail.com> >> >> This patch results in >> >> Building openrisc:allmodconfig ... failed >> -------------- >> Error log: >> drivers/video/fbdev/riva/fbdev.c: In function 'rivafb_probe': >> drivers/video/fbdev/riva/fbdev.c:2062:42: error: >> passing argument 1 of 'iounmap' discards 'volatile' qualifier from pointer target type >> >> drivers/video/fbdev/nvidia/nvidia.c: In function 'nvidiafb_probe': >> drivers/video/fbdev/nvidia/nvidia.c:1414:20: error: >> passing argument 1 of 'iounmap' discards 'volatile' qualifier from pointer target type >> >> drivers/scsi/aic7xxx/aic7xxx_osm.c: In function 'ahc_platform_free': >> drivers/scsi/aic7xxx/aic7xxx_osm.c:1231:41: error: >> passing argument 1 of 'iounmap' discards 'volatile' qualifier from pointer target type >> >> ... and so on. >> >> Prior to this patch, the code was not enabled because it depends on PCI. > > Hi Guenter, > > Thanks for reporting this. > > It's interesting, I don't get this on the openrisc/for-next branch. > Hmm, weird. I see it all over the place. Complete log is at https://kerneltests.org/builders/next-openrisc-next/builds/1880/steps/buildcommand/logs/stdio if you are interested. > BTW, do you turn off WERROR on the allmodconfig config? I get many warnings > such as the below, but I haven't looked into it much yet: > No, I don't. Disabling it would defeat its purpose. > fs/exec.c: In function 'shift_arg_pages': > fs/exec.c:687:27: error: 'tlb' is used uninitialized [-Werror=uninitialized] > 687 | struct mmu_gather tlb; > | ^~~ > I don't see that in next-20220728. I tried with gcc-11.2 and 11.3. Which compiler do you use ? Thanks, Guenter > I will try to see what is going on in linux-next. > > -Stafford >
WARNING: multiple messages have this Message-ID (diff)
From: Guenter Roeck <linux@roeck-us.net> To: Stafford Horne <shorne@gmail.com> Cc: Jonas Bonn <jonas@southpole.se>, Arnd Bergmann <arnd@arndb.de>, Peter Zijlstra <peterz@infradead.org>, LKML <linux-kernel@vger.kernel.org>, Palmer Dabbelt <palmer@rivosinc.com>, openrisc@lists.librecores.org, Bjorn Helgaas <helgaas@kernel.org> Subject: Re: [PATCH v3 2/3] openrisc: Add pci bus support Date: Thu, 28 Jul 2022 23:10:37 -0700 [thread overview] Message-ID: <ceb732ee-0ea1-b471-0b57-3cc3bcb80a2d@roeck-us.net> (raw) In-Reply-To: <YuN1PWcDukEPdtQy@antec> On 7/28/22 22:50, Stafford Horne wrote: > On Thu, Jul 28, 2022 at 08:37:28PM -0700, Guenter Roeck wrote: >> On Mon, Jul 25, 2022 at 11:07:36AM +0900, Stafford Horne wrote: >>> This patch adds required definitions to allow for PCI buses on OpenRISC. >>> This is being tested on the OpenRISC QEMU virt platform which is in >>> development. >>> >>> OpenRISC does not have IO ports so we keep the definition of >>> IO_SPACE_LIMIT and PIO_RESERVED to be 0. >>> >>> Note, since commit 66bcd06099bb ("parport_pc: Also enable driver for PCI >>> systems") all platforms that support PCI also need to support parallel >>> port. We add a generic header to support compiling parallel port >>> drivers, though they generally will not work as they require IO ports. >>> >>> Signed-off-by: Stafford Horne <shorne@gmail.com> >> >> This patch results in >> >> Building openrisc:allmodconfig ... failed >> -------------- >> Error log: >> drivers/video/fbdev/riva/fbdev.c: In function 'rivafb_probe': >> drivers/video/fbdev/riva/fbdev.c:2062:42: error: >> passing argument 1 of 'iounmap' discards 'volatile' qualifier from pointer target type >> >> drivers/video/fbdev/nvidia/nvidia.c: In function 'nvidiafb_probe': >> drivers/video/fbdev/nvidia/nvidia.c:1414:20: error: >> passing argument 1 of 'iounmap' discards 'volatile' qualifier from pointer target type >> >> drivers/scsi/aic7xxx/aic7xxx_osm.c: In function 'ahc_platform_free': >> drivers/scsi/aic7xxx/aic7xxx_osm.c:1231:41: error: >> passing argument 1 of 'iounmap' discards 'volatile' qualifier from pointer target type >> >> ... and so on. >> >> Prior to this patch, the code was not enabled because it depends on PCI. > > Hi Guenter, > > Thanks for reporting this. > > It's interesting, I don't get this on the openrisc/for-next branch. > Hmm, weird. I see it all over the place. Complete log is at https://kerneltests.org/builders/next-openrisc-next/builds/1880/steps/buildcommand/logs/stdio if you are interested. > BTW, do you turn off WERROR on the allmodconfig config? I get many warnings > such as the below, but I haven't looked into it much yet: > No, I don't. Disabling it would defeat its purpose. > fs/exec.c: In function 'shift_arg_pages': > fs/exec.c:687:27: error: 'tlb' is used uninitialized [-Werror=uninitialized] > 687 | struct mmu_gather tlb; > | ^~~ > I don't see that in next-20220728. I tried with gcc-11.2 and 11.3. Which compiler do you use ? Thanks, Guenter > I will try to see what is going on in linux-next. > > -Stafford >
next prev parent reply other threads:[~2022-07-29 6:10 UTC|newest] Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-07-25 2:07 [PATCH v3 0/3] OpenRISC support for virt platform with PCI Stafford Horne 2022-07-25 2:07 ` [PATCH v3 1/3] asm-generic: Support NO_IOPORT_MAP in pci_iomap.h Stafford Horne 2022-07-25 17:10 ` Bjorn Helgaas 2022-07-25 21:25 ` Stafford Horne 2022-07-25 2:07 ` [PATCH v3 2/3] openrisc: Add pci bus support Stafford Horne 2022-07-25 2:07 ` Stafford Horne 2022-07-29 3:37 ` Guenter Roeck 2022-07-29 3:37 ` Guenter Roeck 2022-07-29 5:50 ` Stafford Horne 2022-07-29 5:50 ` Stafford Horne 2022-07-29 6:10 ` Guenter Roeck [this message] 2022-07-29 6:10 ` Guenter Roeck 2022-07-29 7:54 ` Stafford Horne 2022-07-29 7:54 ` Stafford Horne 2022-07-25 2:07 ` [PATCH v3 3/3] openrisc: Add virt defconfig Stafford Horne 2022-07-25 2:07 ` Stafford Horne
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=ceb732ee-0ea1-b471-0b57-3cc3bcb80a2d@roeck-us.net \ --to=linux@roeck-us.net \ --cc=arnd@arndb.de \ --cc=helgaas@kernel.org \ --cc=jonas@southpole.se \ --cc=linux-kernel@vger.kernel.org \ --cc=openrisc@lists.librecores.org \ --cc=palmer@rivosinc.com \ --cc=peterz@infradead.org \ --cc=shorne@gmail.com \ --cc=stefan.kristiansson@saunalahti.fi \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.