Messages in this thread | | | From | David Brown <> | Subject | Re: [PATCH 4/4] msm: clock: Add support for more proc_comm clocks | Date | Wed, 26 Jan 2011 14:03:35 -0800 |
| |
On Tue, Jan 25 2011, Daniel Walker wrote:
> 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..
Or it is just an argument to not use the word also in the commit text. I don't think it's really necessary to split up a patch that adds a few clocks to the clock table.
David
-- Sent by an employee of the Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.
| |