On Wed, May 1, 2013 at 4:34 PM, Mark Langsdorf <mark.langsdorf@xxxxxxxxxxx> wrote: > > - const unsigned long *timing = sata_ehc_deb_timing(&link->eh_context); > + unsigned long timing[] = { 5, 100, 500}; Why are you dropping the 'const'? Assuming it works, this should be more efficient: static const unsigned long timing[] = {5, 100, 500}; -- 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