From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pankaj Dubey Subject: Re: [v2,3/3] ARM: EXYNOS: Remove left over 'extra_save' Date: Thu, 05 Feb 2015 10:19:10 +0530 Message-ID: <54D2F646.3090306@samsung.com> References: <1422961601-7087-3-git-send-email-k.kozlowski@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from mailout3.samsung.com ([203.254.224.33]:65274 "EHLO mailout3.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750923AbbBEEsf (ORCPT ); Wed, 4 Feb 2015 23:48:35 -0500 In-reply-to: <1422961601-7087-3-git-send-email-k.kozlowski@samsung.com> Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: Krzysztof Kozlowski , Russell King , Kukjin Kim , linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org Hi Krzysztof, On Tuesday 03 February 2015 04:36 PM, Krzysztof Kozlowski wrote: > Since 32b0aa9aaeb4 ("ARM: EXYNOS: Remove i2c sys configuration related > code") the Exynos 5250 no longer saves additional registers under > 'exynos_pm_data.extra_save' field. > > No one else uses this code so get rid of it making also 'exynos_pm_data' > const everywhere. > > Signed-off-by: Krzysztof Kozlowski > > --- > Changes since v1: > 1. New patch. > --- > arch/arm/mach-exynos/suspend.c | 13 ++----------- > 1 file changed, 2 insertions(+), 11 deletions(-) > > diff --git a/arch/arm/mach-exynos/suspend.c b/arch/arm/mach-exynos/suspend.c > index e6c961126087..d56bd6ca5e86 100644 > --- a/arch/arm/mach-exynos/suspend.c > +++ b/arch/arm/mach-exynos/suspend.c > @@ -63,7 +63,6 @@ static struct sleep_save exynos_core_save[] = { > > struct exynos_pm_data { > const struct exynos_wkup_irq *wkup_irq; > - struct sleep_save *extra_save; > int num_extra_save; 'num_extra_save' can be removed at the same time. Rest changes looks ok. Thanks, Pankaj Dubey From mboxrd@z Thu Jan 1 00:00:00 1970 From: pankaj.dubey@samsung.com (Pankaj Dubey) Date: Thu, 05 Feb 2015 10:19:10 +0530 Subject: [v2,3/3] ARM: EXYNOS: Remove left over 'extra_save' In-Reply-To: <1422961601-7087-3-git-send-email-k.kozlowski@samsung.com> References: <1422961601-7087-3-git-send-email-k.kozlowski@samsung.com> Message-ID: <54D2F646.3090306@samsung.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Krzysztof, On Tuesday 03 February 2015 04:36 PM, Krzysztof Kozlowski wrote: > Since 32b0aa9aaeb4 ("ARM: EXYNOS: Remove i2c sys configuration related > code") the Exynos 5250 no longer saves additional registers under > 'exynos_pm_data.extra_save' field. > > No one else uses this code so get rid of it making also 'exynos_pm_data' > const everywhere. > > Signed-off-by: Krzysztof Kozlowski > > --- > Changes since v1: > 1. New patch. > --- > arch/arm/mach-exynos/suspend.c | 13 ++----------- > 1 file changed, 2 insertions(+), 11 deletions(-) > > diff --git a/arch/arm/mach-exynos/suspend.c b/arch/arm/mach-exynos/suspend.c > index e6c961126087..d56bd6ca5e86 100644 > --- a/arch/arm/mach-exynos/suspend.c > +++ b/arch/arm/mach-exynos/suspend.c > @@ -63,7 +63,6 @@ static struct sleep_save exynos_core_save[] = { > > struct exynos_pm_data { > const struct exynos_wkup_irq *wkup_irq; > - struct sleep_save *extra_save; > int num_extra_save; 'num_extra_save' can be removed at the same time. Rest changes looks ok. Thanks, Pankaj Dubey