On Wed, 11 Apr 2012, Mark A. Greer wrote: > On Wed, Apr 11, 2012 at 03:44:38PM -0600, Paul Walmsley wrote: > > > Looks like there are an extra set of commas here, maybe just use: > > > > static struct clkdm_dep gfx_sgx_am35x_wkdeps[] = { > > { .clkdm_name = "mpu_clkdm" }, > > { .clkdm_name = "wkup_clkdm" }, > > { NULL }, > > Yes, there are extra commas but they come from the code I'm copying > and removing iva2_clkdm from. I was keeping things consistent so I > kept the commas. Given that, do you still want them removed? > (I'm indifferent.) Yes please. I realize it's a minor issue, but I'd like to keep the format of these files internally consistent. - Paul -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html