It looks good to me. Acked-by: Jaehoon Chung <jh80.chung@xxxxxxxxxxx> On 12/04/2012 08:33 PM, Sachin Kamat wrote: > 'sc' is used only when CONFIG_PM_RUNTIME is defined. Hence define it > conditionally. > > Silences the following warning: > drivers/mmc/host/sdhci-s3c.c: In function ‘sdhci_s3c_notify_change’: > drivers/mmc/host/sdhci-s3c.c:378:20: warning: unused variable ‘sc’ [-Wunused-variable] > > Signed-off-by: Sachin Kamat <sachin.kamat@xxxxxxxxxx> > --- > drivers/mmc/host/sdhci-s3c.c | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/drivers/mmc/host/sdhci-s3c.c b/drivers/mmc/host/sdhci-s3c.c > index 7ea3aea..82a8de1 100644 > --- a/drivers/mmc/host/sdhci-s3c.c > +++ b/drivers/mmc/host/sdhci-s3c.c > @@ -375,7 +375,9 @@ static struct sdhci_ops sdhci_s3c_ops = { > static void sdhci_s3c_notify_change(struct platform_device *dev, int state) > { > struct sdhci_host *host = platform_get_drvdata(dev); > +#ifdef CONFIG_PM_RUNTIME > struct sdhci_s3c *sc = sdhci_priv(host); > +#endif > unsigned long flags; > > if (host) { > -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html