Re: [PATCH V2 0/3] of/irq: fix guards and dummies for OF=y and OF_IRQ=n

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On Mon, Oct 12, 2015 at 7:43 AM, Jonas Gorski <jogo@xxxxxxxxxxx> wrote:
> Historically OF was only selectable by arch code, which usually also used
> IRQ_DOMAIN, thus making OF_IRQ and OF interchangeable for !SPARC.
>
> Since OF is now a user selectable symbol, it is now easy to create a
> situation where OF=y and OF_IRQ=n, by not enabling IRQ_DOMAIN on e.g.
> x86.
>
> This causes linking errors as for a few functions the prototype guard
> was OF, which is too broad and does not catch all combinations. Recently
> added of_msi_configure is even missing a dummy implementation.
>
> The following three patches fix these issues by moving of_msi_configure
> to under the right guard and add a dummy implementation, make
> of_irq_find_parent static as there are no external users, and make the
> guard for irq_of_parse_and_map more specific.
>
> Based on robh/linux.git#for-next, as the tree from MAINTAINERS haven't
> been updated for a while. I hope that is the right git to use.
>
> Changes v1 -> v2:
>  * merge the of_msi_configure into one patch
>  * make of_irq_find_parent static and remove the prototype/dummy
>  * rebased onto robh/linux.git#for-next
>
> Jonas Gorski (3):
>   of/irq: move of_msi_configure to the right guard and add a dummy
>   of/irq: make of_irq_find_parent static
>   of/irq: fix guards for irq_of_parse_and_map prototype
>
>  drivers/of/irq.c       |  2 +-
>  include/linux/of_irq.h | 15 ++++++---------
>  2 files changed, 7 insertions(+), 10 deletions(-)

Series applied, thanks.

Rob

>
> --
> 2.1.4
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux