On 05/30/2013 09:18 AM, Mark Langsdorf wrote: > - const unsigned long *timing = sata_ehc_deb_timing(&link->eh_context); > + const unsigned long timing[] = { 5, 100, 500}; You'll save space and time if you also make this array "static", otherwise the compiler will build the array every time this function is called. -- Timur Tabi Software Developer at Calxeda -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html