| To: | David Daney <ddaney.cavm@gmail.com> |
|---|---|
| Subject: | Re: [PATCH] OF: PCI: const usage needed by MIPS |
| From: | John Crispin <blogic@openwrt.org> |
| Date: | Tue, 01 May 2012 15:28:22 +0200 |
| Cc: | Ralf Baechle <ralf@linux-mips.org>, Grant Likely <grant.likely@secretlab.ca>, linux-pci@vger.kernel.org, devicetree-discuss@lists.ozlabs.org, linux-mips@linux-mips.org |
| In-reply-to: | <4F9ED1DC.3050007@gmail.com> |
| References: | <1335808019-24502-1-git-send-email-blogic@openwrt.org> <4F9ED1DC.3050007@gmail.com> |
| Sender: | linux-mips-bounce@linux-mips.org |
| User-agent: | Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.24) Gecko/20111114 Icedove/3.1.16 |
On 30/04/12 19:54, David Daney wrote: > On 04/30/2012 10:46 AM, John Crispin wrote: >> On MIPS we want to call of_irq_map_pci from inside >> >> arch/mips/include/asm/pci.h:extern int pcibios_map_irq( >> const struct pci_dev *dev, u8 slot, u8 pin); >> >> For this to work we need to change several functions to const usage. > > I think there is a mismatch on this throughout the kernel. > > Properly fixing it requires touching many more places than these. > Although I haven't tried it, I wouldn't be surprised if doing this > caused warnings to appear in non-MIPS code. > > Ralf had a patch at one point that tried to make this consistent > tree-wide, but it is not yet applied. > > David Daney Hi, Ok, lets see what Ralf has to say. I just tested the patch on x86 with OF enabled and drivers turned on that use the API. I did not see any errors appear. Thanks, John |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH 02/14] MIPS: pci: parse memory ranges from devicetree, John Crispin |
|---|---|
| Next by Date: | [RFC][PATCH 5/5] sched: Rewrite the CONFIG_NUMA sched domain support, Peter Zijlstra |
| Previous by Thread: | Re: [PATCH 02/14] MIPS: pci: parse memory ranges from devicetree, John Crispin |
| Next by Thread: | Re: [PATCH] OF: PCI: const usage needed by MIPS, Bjorn Helgaas |
| Indexes: | [Date] [Thread] [Top] [All Lists] |