On 10/10/2014 12:08, Sergei Shtylyov wrote: >> + if (bga) { name = "MT7620A"; soc_info->compatible = >> "ralink,mt7620a-soc"; } else { - panic("mt7620: unknown >> SoC, n0:%08x n1:%08x", n0, n1); + name = "MT7620N"; + >> soc_info->compatible = "ralink,mt7620n-soc"; +#ifdef CONFIG_PCI > > I suggest: > > if (IS_ENABLED(CONFIG_PCI)) > > in order to avoid this #ifdef. > > [...] > > WBR, Sergei > yep, holds true fo the irq-gic.c patch where i also use ifdef. I'll fix and resend the 2 patches. John