On Thu, 2011-12-01 at 18:39 +1100, Tony Breeds wrote: > + if (ppc4xx_pciex_hwops->want_sdr == true) { if (ppc4xx_pciex_hwops->want_sdr) { Please ! This is a boolean after all :-) Cheers, Ben.