On Thu, Apr 16, 2009 at 10:57 PM, Anton Vorontsov <avorontsov@xxxxxxxxxxxxx> wrote: > On Thu, Apr 16, 2009 at 09:30:00PM +0200, Bartlomiej Zolnierkiewicz wrote: >> >> Hi, >> >> On Wednesday 15 April 2009 16:34:22 Roderick Colenbrander wrote: >> > Hi, >> > >> > I'm using a Xilinx ML510 it features a PowerPC 440 cpu inside a >> > Virtex-5 FPGA. The board also contains a ALI M1533 south bridge >> > for IDE, USB and Audio. I did a lot of work to get the pci bus working >> > on this board and it works correctly but the default init code >> > of the alim15x3 driver doesn't work for me. The driver explicitly >> > disabled some initialization code for powerpc after uncommenting this >> > code it works properly. Benjamin Herrenschmidt and I think this >> > !CONFIG_PPC check should be removed because the system behaves >> > like a real 'x86' system (also the i8259 interrupt controller is used). >> >> Ben, I guess you are OK with the change and there are no longer other >> platforms requiring CONFIG_PPC check below? [I don't see your ACK here] > > I'd like to nak this patch. It'll break MPC8610HPCD boards. > > See commit 6d1cee44361b8d06ccd1812e80448d86ae60dfe3 > > I'm not sure how should we handle this for Xilinx PPC, MPC8610 > PPC and Sparc all at the same time, but it's obvious that #ifdef > isn't enough any longer... > > > If it is really a board specific issue (in case of powerpc it appears to be one) we could also use the default_irq override in ppc_md. What do you think? Roderick -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html