Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1235415

Re: [PATCH v4 0/9] Making the generic ACPI GSI layer irqdomain aware

From Marc Zyngier <marc.zyngier@arm.com>
Newsgroups linux.kernel
Subject Re: [PATCH v4 0/9] Making the generic ACPI GSI layer irqdomain aware
Date 2015-09-29 19:20 +0200
Message-ID <qe6Jk-4tk-3@gated-at.bofh.it> (permalink)
References <qdJMJ-54p-3@gated-at.bofh.it> <qe6Jk-4tk-5@gated-at.bofh.it>
Organization ARM Ltd

Show all headers | View raw


On 29/09/15 18:11, Rob Herring wrote:
> On Mon, Sep 28, 2015 at 11:42 AM, Marc Zyngier <marc.zyngier@arm.com> wrote:
>> The irqdomain code is not entierely ACPI friendly, as it has some
>> built-in knowledge of the device-tree. Nothing too harmful, but enough
>> to scare the ARM ACPI developpers which end up with their own version
>> of the square wheel.
>>
>> This small patch series adapts the irqdomain code to remove the
>> hurdles that prevent the full blown irqdomain subsystem to be used on
>> ACPI, creates an interface between the GSI layer and the irqdomain,
>> and as an example, convert the ARM GIC ACPI support to use irqdomains
>> as originally intended.
>>
>> Overall, this gives us a way to use irqdomains on both DT and ACPI
>> enabled platforms, having very little changes made to the actual
>> drivers (other than the probing infrastructure). Because we keep the
>> flow of information between the various layers identical between ACPI
>> and DT, we immediately benefit from the existing infrastructure. The
>> "convert the GSI information to be DT friendly" is admitedly not very
>> pretty, but I see it as a stepping stone towards unifying the two
>> structures.
> 
> Did I miss v1-v3 or did the DT change just show up? I probably tuned
> out with ACPI in the subject.

No, that's new. It was far worse before.

> Needing fake DT nodes for ACPI is just wrong. There's got to be
> another, better way.

Well, the alternative is to rewrite the whole of irqdomain and
everything that gets called from there in terms of fwnode_handle. Which
will trickle in the whole of OF as well.

Note that the fake DT node doesn't represent an ACPI object at all. It
is an identifier for an irqdomain, and that's where it stops.

	M.
-- 
Jazz is not dead. It just smells funny...
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH v4 0/9] Making the generic ACPI GSI layer irqdomain aware Marc Zyngier <marc.zyngier@arm.com> - 2015-09-28 18:50 +0200
  [PATCH v4 2/9] genirq/irqdomain: Add irq_create_acpi_mapping Marc Zyngier <marc.zyngier@arm.com> - 2015-09-28 18:50 +0200
  [PATCH v4 7/9] irqchip/GIC: Get rid of gic_init_bases() Marc Zyngier <marc.zyngier@arm.com> - 2015-09-28 18:50 +0200
  [PATCH v4 5/9] acpi/gsi: Add acpi_set_irq_model to initialize the GSI layer Marc Zyngier <marc.zyngier@arm.com> - 2015-09-28 18:50 +0200
  Re: [PATCH v4 0/9] Making the generic ACPI GSI layer irqdomain aware Marc Zyngier <marc.zyngier@arm.com> - 2015-09-29 19:20 +0200
  Re: [PATCH v4 0/9] Making the generic ACPI GSI layer irqdomain aware Rob Herring <robherring2@gmail.com> - 2015-09-29 19:20 +0200

csiph-web