On Wednesday 13 November 2013 09:02 PM, Nishanth Menon wrote: > On 11/11/2013 11:47 PM, Sricharan R wrote: >> Hi Rajendra, >> >> On Tuesday 12 November 2013 11:11 AM, Rajendra Nayak wrote: >>> On Tuesday 05 November 2013 06:44 PM, Sricharan R wrote: >>>> Enable the crossbar IP support for DRA7xx soc. >>>> >>>> Cc: Santosh Shilimkar <santosh.shilimkar@xxxxxx> >>>> Cc: Rajendra Nayak <rnayak@xxxxxx> >>>> Cc: Tony Lindgren <tony@xxxxxxxxxxx> >>>> Signed-off-by: Sricharan R <r.sricharan@xxxxxx> >>>> --- >>>> arch/arm/mach-omap2/Kconfig | 1 + >>>> arch/arm/mach-omap2/omap4-common.c | 4 ++++ >>>> 2 files changed, 5 insertions(+) >>>> >>>> diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig >>>> index b5fb5f7..2086c65 100644 >>>> --- a/arch/arm/mach-omap2/Kconfig >>>> +++ b/arch/arm/mach-omap2/Kconfig >>>> @@ -141,6 +141,7 @@ config SOC_DRA7XX >>>> select ARM_GIC >>>> select HAVE_SMP >>>> select COMMON_CLK >>>> + select IRQ_CROSSBAR >>>> >>>> comment "OMAP Core Type" >>>> depends on ARCH_OMAP2 >>>> diff --git a/arch/arm/mach-omap2/omap4-common.c b/arch/arm/mach-omap2/omap4-common.c >>>> index 5791143..274cbfa 100644 >>>> --- a/arch/arm/mach-omap2/omap4-common.c >>>> +++ b/arch/arm/mach-omap2/omap4-common.c >>>> @@ -22,6 +22,7 @@ >>>> #include <linux/of_platform.h> >>>> #include <linux/export.h> >>>> #include <linux/irqchip/arm-gic.h> >>>> +#include <linux/irqchip/irq-crossbar.h> >>>> #include <linux/of_address.h> >>>> #include <linux/reboot.h> >>>> >>>> @@ -282,9 +283,12 @@ void __init omap_gic_of_init(void) >>>> >>>> skip_errata_init: >>>> omap_wakeupgen_init(); >>>> + if (soc_is_dra7xx()) >>>> + crossbar_init(); >>> Its good if this is called irqcrossbar_init() to avoid confusion >>> with the dma crossbar which also exists in dra7 devices. >>> >> Ya makes sense. Will change this. > Also why not call it without explicit soc check? there is already > parse for of_find_matching_node(NULL, crossbar_match); to see if we > have a match in the crossbar_init. OR call it from dra7 init without > SoC check dependency? > > Rationale - with more TI SoCs trending towards crossbar, we dont need > to keep adding to the soc_is checks, further, we intend to remove > soc_is checks in it's entirety.. > > Oh yes. I missed this. Thanks for pointing. Will correct. Regards, Sricharan -- 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