Thread (9 messages) 9 messages, 4 authors, 2011-10-07

[PATCH 0/2] OMAP: Update nr_irqs field in machine descriptors

flat view
STALE5482d

From: nico@fluxnic.net (Nicolas Pitre)
Date: 2011-10-07 20:15:54
Also in: linux-omap

Revision v1 of 2 in this series.

Revisions (2)
  1. v1 current
  2. v1
On Thu, 6 Oct 2011, Tony Lindgren wrote:
* S, Venkatraman [off-list ref] [110825 07:23]:
quoted
On Thu, Aug 25, 2011 at 5:19 PM, Cousson, Benoit [off-list ref] wrote:
quoted
Hi Venkat,

On 8/24/2011 9:46 PM, S, Venkatraman wrote:
quoted
As part of an effort to get single ARM kernel binary [1],
multiple ?definitions of NR_IRQS under various platforms
have to be reconciled and abstracted away from common code.

This patch series takes the small step of populating the
machine descriptors with the pre-existing nr_irqs field.
Eventually, the common irq handler code will only look at this
field and not the compile time constant.
Not related to this patch, but still on that topic. The current NR_IRQS
depends as well on board stuff, like for example : the Phoenix
IRQs:TWL6030_IRQ_BASE, TWL6040_CODEC_IRQ_BASE.
Is there a plan to get rid of this static defines?
Currently, the goal is to get rid of the singleton nature
of NR_IRQS. Then it just becomes a property of the
platform, and the arm common code should not see this define.
This cleanup has to be done across multiple SoCs, not just OMAP.

After I get to complete some meaningful cleanup of NR_IRQS,
I can look into the static defines that you mention.
I suggest we wait on this patch as the NR_IRQS should be the
board specific true number of interrupts including chained
interrupts from external devices like twl. So just setting
it to NR_IRQS does not help much. Also, the board-*.c files
will be going aways with device tree at some point.
This is prerequisite to some other cleanup orthogonal to DT being worked 
in parallel.  I would prefer if DT wasn't a serialization point for 
this.


Nicolas
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help