On Thu, Dec 01, 2011 at 01:49:20PM -0700, Stephen Warren wrote: > + /* Default platform data, for use if none is supplied */ > + defpdata.irq_active_low = false; > + defpdata.micdet_cfg = 0; > + defpdata.micdet_delay = 0; No need to set things to zero (or just memset the structure). > + defpdata.gpio_base = -1; > + for (i = 0; i < ARRAY_SIZE(defpdata.gpio_cfg); i++) > + defpdata.gpio_cfg[i] = 0xffffffff; These should come after we've taken the copy, checking to see if there'a anything there before overwriting. -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html