On Mon, 2011-01-24 at 19:45 -0800, Stephen Boyd wrote: > Support the CE_CLK and CODEC_SSBI_CLK. Also add support for uart, > and i2c clocks on targets which support proc_comm clocks. Basically when you catch yourself writing "also" in the commit text you need to re-evaluate making two patches.. In this case it looks appropriate.. I'd do CE_CLK and CODEC_SSBI_CLK in one, with the increment for the number of clocks, and adding those two clocks. Then the rest of it I'm not sure. If it was me I'd break out uart and i2c in different patches but that might be overkill. Daniel -- Sent by a consultant of the Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum. -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html