Re: [PATCH and RFC] Remove request_8xxirq

[Dan Malek <dan@embeddededge.com>]

Tom Rini wrote:

> The first benefit of all of these changes is that it gets PCI working on
> MBX (and any other 8xx system with PCI).

No, it doesn't.  It just enables the 8259 to kinda work.  The PCI on MBX
does not work well, and this does nothing to improve it.  Does anyone have
a working MBX anymore?

> Comments?

There is more to these patches than just the interrupt change.  Why is
the CPM microcode, and so much of commproc.c changed?

Of course, I don't like it, but oh well.......It's just another hack that
doesn't do anything different.  Most importantly, I hate shit like this:

	request_irq(CPM_IRQ_OFFSET + CPMVEC_SMC2, ......

because you have to program the CPM with only the CPM vector number, not the
vector plus offset.  I want a real cascaded interrupt design, not just another
hack that opens the possibility to calling a generic PC function with the
wrong parameters.  The OpenPIC/EPIC are hacked just as badly, and there are
new integrated controllers coming that would benefit from a real design improvement
instead of just another hack.

I'm most concerned with the updates outside of just the interrupt stuff, and
I hope someone tests this.

Whatever you want......Have fun........


	-- Dan


** Sent via the linuxppc-embedded mail list. See http://lists.linuxppc.org/