[PATCH] fix for firewire patch added in that breaks things on ppc

danny at mailmij.org danny at mailmij.org
Wed Aug 9 02:50:23 EST 2006

On Tue, Aug 08, 2006 at 03:05:15PM +1000, Paul Mackerras wrote:
> danny at mailmij.org writes:
> > -#endif
> > -
> > +#else
> >  	pci_save_state(pdev);
> > -
> > +#endif /* CONFIG_PPC_PMAC */
> Shouldn't we just move the pci_save_state call up before the
> pmac_call_feature() call?
> Paul.
I tested this. It works, but it has no effect (pci_restore_state does not report that anything changed,
thus this means that the config space already contained the correct values).

I discovered one additional problem with leaving a device running through a sleep/resume cycle:
ohci1394: fw-host0: SelfID received outside of bus reset sequence
ohci1394: fw-host0: SelfID is inconsistent [0x7e80773f/0x7e80773f]
The last leads to the node not being able to resume when the device is actually plugged.

Anyway, it's a separate issue, I will investigate.


More information about the Linuxppc-dev mailing list