Date: Mon, 18 Mar 2013 09:05:14 -0700
From: Greg KH <gregkh@linuxfoundation.org>
To: Li Fei <fei.li@intel.com>
Cc: rjw@sisk.pl, akpm@linux-foundation.org, ldewangan@nvidia.com,
        linux-kernel@vger.kernel.org, chuansheng.liu@intel.com
Subject: Re: [PATCH] pm: print the name of failed suspend function for
 platform device
Message-ID: <20130318160514.GE3887@kroah.com>
References: <1363586303.9792.1.camel@fli24-HP-Compaq-8100-Elite-CMT-PC>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1363586303.9792.1.camel@fli24-HP-Compaq-8100-Elite-CMT-PC>
User-Agent: Mutt/1.5.21 (2010-09-15)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 880
Lines: 24

On Mon, Mar 18, 2013 at 01:58:23PM +0800, Li Fei wrote:
> 
> Print more info when platform device suspend function failed.
> Without this patch, we can not get the real platform device suspend
> API info.
> 
> Example without this patch:
> pm_op(): platform_pm_suspend+0x0/0x50 returns -11
> PM: Device power.0 failed to suspend: error -11
> 
> And with this patch:
> platform_pm_suspend(): power_suspend+0x0/0x30 returns -11
> pm_op(): platform_pm_suspend+0x0/0x80 returns -11
> PM: Device power.0 failed to suspend: error -11

How does that help someone out?  One extra line?  What is that going to
allow a user to now do?

greg k-h
--
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/