On Tue, Jun 3, 2008 at 8:22 AM, Carlos O'Donell <carlos@xxxxxxxxxxxxxxxx> wrote: >> +#ifdef CONFIG_64BIT >> + asm volatile( >> + " ldq 0(%0), %1\n" >> + : : "r"(addr), "r"(q) : "memory"); >> +#else >> + unsigned int q_lo, q_hi; >> + q_hi = __raw_readl(addr); >> + q_lo = __raw_readl(addr+4); >> + q = (unsigned long long)(q_hi << 32) | (q_lo); >> +#endif >> + >> + return q; > > Is there any reason the 32-bit version is uses plain C? I see, you optimized this for the 64-bit case, that makes sense. I should drink more coffee. Cheers, Carlos. -- To unsubscribe from this list: send the line "unsubscribe linux-parisc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
