From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757390AbcAaMIY (ORCPT ); Sun, 31 Jan 2016 07:08:24 -0500 Received: from down.free-electrons.com ([37.187.137.238]:43646 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1757303AbcAaMIX (ORCPT ); Sun, 31 Jan 2016 07:08:23 -0500 Date: Sun, 31 Jan 2016 13:08:22 +0100 From: Alexandre Belloni To: Krzysztof Kozlowski Cc: Chanwoo Choi , Alessandro Zummo , linux-kernel@vger.kernel.org, rtc-linux@googlegroups.com, Javier Martinez Canillas , Andi Shyti , Laxman Dewangan Subject: Re: [PATCH] rtc: max77686: Cleanup and reduce dmesg output Message-ID: <20160131120822.GN20165@piout.net> References: <1453877171-30510-1-git-send-email-k.kozlowski@samsung.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1453877171-30510-1-git-send-email-k.kozlowski@samsung.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 27/01/2016 at 15:46:11 +0900, Krzysztof Kozlowski wrote : > Cleanup of entire driver of its dmesg output: > 1. Remove printing of the function name, because printing device name is > sufficient. This also makes the dev_err()-like functions more compact > and readable (not need of line break). > 2. Lower from info to debug printing of each RTC interrupt (no need to > make noise on each alarm). > 3. Remove dev_info() at beginning of probe because a message is already > always printed by either probe failure or from registering the RTC > device as /dev/rtcX. > > Signed-off-by: Krzysztof Kozlowski > > --- > > Patch rebased on top of Javier's v4: > [PATCH v4 00/10] rtc: max77686: Extend driver and add max77802 support > http://www.spinics.net/lists/arm-kernel/msg477194.html > > Feel free to join with these series. > --- > drivers/rtc/rtc-max77686.c | 39 ++++++++++++++------------------------- > 1 file changed, 14 insertions(+), 25 deletions(-) > Applied, thanks. -- Alexandre Belloni, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com