? 12/03/2014 01:27 PM, Doug Anderson ??: > Jianqun, > > On Tue, Dec 2, 2014 at 5:51 PM, Jianqun Xu <jay.xu at rock-chips.com> wrote: >> From: Sonny Rao <sonnyrao at chromium.org> >> >> We need to claim the clock which is driving the codec so that when we enable >> clock gating, we continue to clock the codec when needed. I make this an >> optional clock since there might be some applications where we don't need it >> but can still use the I2S block. >> >> Signed-off-by: Sonny Rao <sonnyrao at chromium.org> >> Signed-off-by: Jianqun Xu <jay.xu at rock-chips.com> >> --- >> sound/soc/rockchip/rockchip_i2s.c | 12 ++++++++++++ >> 1 file changed, 12 insertions(+) > > This looks fine to me now and you can add: > > Reviewed-by: Doug Anderson <dianders at chromium.org> > > ...but you lost the whole CC list and this definitely needs to include > broonie in the "To" list, so you need to repost. When you repost it > would be super helpful if you could include a version number and a > list of changes between versions. > Dear Doug: I try to send patches to linux-rockchip at lists.infradead.org first, and if no problem, I will re-send it to more reviewers and thanks for your review I will do it today, but may a little later > > -- Jianqun Xu **************************************************************************** *IMPORTANT NOTICE:*This email is from Fuzhou Rockchip Electronics Co., Ltd .The contents of this email and any attachments may contain information that is privileged, confidential and/or exempt from disclosure under applicable law and relevant NDA. If you are not the intended recipient, you are hereby notified that any disclosure, copying, distribution, or use of the information is STRICTLY PROHIBITED. Please immediately contact the sender as soon as possible and destroy the material in its entirety in any format. Thank you. ****************************************************************************
