Thread (6 messages) flat view 6 messages, 3 authors, 2014-07-21

Re: [RFC PATCH 4/7] ARM: OMAP4+: PRM: register interrupt information from DT

From: Nishanth Menon <nm@ti.com>
Date: 2014-07-21 11:22:24
Also in: linux-arm-kernel, linux-omap

On Mon, Jul 21, 2014 at 5:51 AM, Tony Lindgren [off-list ref] wrote:
quoted
+static struct of_device_id omap_prm_dt_match_table[] = {
+     { .compatible = "ti,omap4-prm" },
+     { .compatible = "ti,omap5-prm" },
+     { .compatible = "ti,dra7-prm" },
+     { }
+};
+
I'd like to avoid adding more driver like stuff to mach-omap2
and parsing compatible flags and dealing with interupts sounds
very driver like.. But maybe just the handling can be moved
out?
I understand your view, but, Handling of interrupts is already in
place even now in mach-omap2. Currently the prm devices are handled by
mach-omap2 and all this does it to prevent hardcoding of irq numbers
within the current code.
Would a simple driver be doable that parses the compatible
flags, takes care of the IRQ chaining, and gets some SoC specific
function pointers as auxdata?
Tero has been trying to move PRM/CM stuff to a separate drivers of
thier own. With that there wont be a need for auxdata even. - this
current logic will get merged with that driver - if and when that is
ready. I am not actually adding any driver logic here - just reusing
the logic and providing glue for using dt description instead of
hardcoded logic that the current mach-omap2 driver does.
That would allow further work later on to remove the auxdata
dependencies possibly.



-- 
---
Regards,
Nishanth Menon
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help