Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758250AbaDIGRY (ORCPT ); Wed, 9 Apr 2014 02:17:24 -0400 Received: from rtits2.realtek.com ([60.250.210.242]:53152 "EHLO rtits2.realtek.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750889AbaDIGRU (ORCPT ); Wed, 9 Apr 2014 02:17:20 -0400 X-SpamFilter-By: BOX Solutions SpamTrap 5.39 with qID s396Ga3T001790, This message is accepted by code: ctloc85258 From: To: Samuel Ortiz , Lee Jones , Chris Ball , Greg Kroah-Hartman , Maxim Levitsky , Alex Dubov , Dan Carpenter , Ulf Hansson , Andrew Morton , Oliver Neukum CC: , , , , , Subject: [PATCH v6 2/4] [v6 fix] mfd/rtsx_usb: add comment in rtsx_usb_suspend Date: Wed, 9 Apr 2014 14:16:01 +0800 Message-ID: <1397024163-6643-3-git-send-email-rogerable@realtek.com> X-Mailer: git-send-email 1.8.2 In-Reply-To: <1397024163-6643-1-git-send-email-rogerable@realtek.com> References: <1397024163-6643-1-git-send-email-rogerable@realtek.com> MIME-Version: 1.0 Content-Type: text/plain X-Originating-IP: [172.21.81.189] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Roger Tseng Explain why there is no need to have a symmetric LED turn-on in resume handler while calling rtsx_usb_turn_off_led() in suspend handler. Signed-off-by: Roger Tseng --- drivers/mfd/rtsx_usb.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/drivers/mfd/rtsx_usb.c b/drivers/mfd/rtsx_usb.c index 0ca7973..141ea52 100644 --- a/drivers/mfd/rtsx_usb.c +++ b/drivers/mfd/rtsx_usb.c @@ -687,9 +687,15 @@ static int rtsx_usb_suspend(struct usb_interface *intf, pm_message_t message) dev_dbg(&intf->dev, "%s called with pm message 0x%04u\n", __func__, message.event); + /* + * Call to make sure LED is off during suspend to save more power. + * It is NOT a permanent state and could be turned on anytime later. + * Thus no need to call turn_on when resunming. + */ mutex_lock(&ucr->dev_mutex); rtsx_usb_turn_off_led(ucr); mutex_unlock(&ucr->dev_mutex); + return 0; } -- 1.8.2 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/