[PATCH 1/3] powerpc/mpc85xx: remove the unneeded pci init functions for corenet ds board

[Kevin Hao]

On Sat, Jun 01, 2013 at 09:47:16PM +1000, Benjamin Herrenschmidt wrote:
> On Sat, 2013-06-01 at 18:59 +0800, Kevin Hao wrote:
> 
> > The effect of this change is that the isa_io_base will be 0 and the IO
> > resource are equal to the virtual address of the IO space. But the IO
> > functions such as outx/inx should work as well. This is why I ask the
> > above question. What do you think about this? Are there any subtle bugs
> > that will be triggered by this?
> 
> I don't see any obvious reason why that wouldn't work but like anything
> in that area, it needs a bit of testing & hammering to be sure ;-)

Agreed.

> 
> In fact it would work on pmac32 as well since those generally don't have
> legacy crap either.
> 
> So I have no fundamental objection, it just needs testing. My worry is
> that we need to make sure we don't break old chrp and I don't have any
> to test with.

Don't worry, my patch just drop the picking of primary bus for several
fsl boards. All these changes are in board specific file, so it should
have no affect to other boards at all.

For the other boards which have the same issue, since this is not emergency,
we don't want handle them in a rush.  So I would like to merge this patch
first and then wait one or two kernel release cycle before handling the others
to make sure that we don't break anything.

Thanks,
Kevin

> I'm happy to drop support for stuff nobody uses anymore
> (we did drop PReP after all and I'm *that* close to drop power3) but as
> long as somebody is still using a CHRP RS6K or a Pegasos I can't quite
> drop those... And they do have legacy ISA crap to deal with.
> 
> Cheers,
> Ben.
> 
> 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://lists.ozlabs.org/pipermail/linuxppc-dev/attachments/20130602/918d72e9/attachment.sig>