Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755274Ab3JDPH2 (ORCPT ); Fri, 4 Oct 2013 11:07:28 -0400 Received: from arroyo.ext.ti.com ([192.94.94.40]:54569 "EHLO arroyo.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755138Ab3JDPH0 (ORCPT ); Fri, 4 Oct 2013 11:07:26 -0400 Message-ID: <524ED994.7020600@ti.com> Date: Fri, 4 Oct 2013 11:07:00 -0400 From: Eduardo Valentin User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130510 Thunderbird/17.0.6 MIME-Version: 1.0 To: Bartlomiej Zolnierkiewicz CC: Lukasz Majewski , Eduardo Valentin , Zhang Rui , Amit Daniel Kachhap , "Rafael J. Wysocki" , Linux PM list , Jonghwa Lee , Lukasz Majewski , linux-kernel , Tomasz Figa , Myungjoo Ham , , Subject: Re: [PATCH 4/6] thermal: exynos: Replace SOC_ARCH_EXYNOS with SOC_ARCH_EXYNOS5250 References: <1380010102-25817-1-git-send-email-l.majewski@samsung.com> <524DEAED.4030905@ti.com> <20131004122330.47e21d24@amdc308.digital.local> <2265935.YcHunKd7UC@amdc1032> In-Reply-To: <2265935.YcHunKd7UC@amdc1032> X-Enigmail-Version: 1.5.2 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="VxriV23ILdJdStvRGSMKosPtajFoJ5uWu" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4391 Lines: 126 --VxriV23ILdJdStvRGSMKosPtajFoJ5uWu Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable On 04-10-2013 06:47, Bartlomiej Zolnierkiewicz wrote: > On Friday, October 04, 2013 12:23:30 PM Lukasz Majewski wrote: >> Hi Eduardo, >> >>> On 24-09-2013 04:08, Lukasz Majewski wrote: >>>> The TMU data definition is now separated to Exynos4412 and >>>> Exynos5250. >>>> >>>> Now SOC_ARCH_EXYNOS only refers to Exynos5250. Hence the name >>>> SOC_ARCH_EXYNOS has been changed to SOC_ARCH_EXYNOS5250. >>>> >>> >>> How about merging this one with patch 02? >> >> I wanted to show how the work was logically split - patch 02 >> extracts Exynos4412 from Exynos5250. Then this patch changes the name.= >> >> I will merge those two patches. No problem. >=20 > Oh, well. Looking at the patch #2 again the merge can be done without > sacrificing backportability. Please, merge them and describe that you are reworking the type and the enum definition. Also add your reasonings, specially if you found bugs. The way the descriptions are being presented, the bugs you found do not get properly explained. >=20 > Best regards, > -- > Bartlomiej Zolnierkiewicz > Samsung R&D Institute Poland > Samsung Electronics >=20 >>> >>>> Signed-off-by: Lukasz Majewski >>>> --- >>>> drivers/thermal/samsung/exynos_tmu.c | 4 ++-- >>>> drivers/thermal/samsung/exynos_tmu.h | 2 +- >>>> drivers/thermal/samsung/exynos_tmu_data.c | 2 +- >>>> 3 files changed, 4 insertions(+), 4 deletions(-) >>>> >>>> diff --git a/drivers/thermal/samsung/exynos_tmu.c >>>> b/drivers/thermal/samsung/exynos_tmu.c index 21b89e4..037cd46 100644= >>>> --- a/drivers/thermal/samsung/exynos_tmu.c >>>> +++ b/drivers/thermal/samsung/exynos_tmu.c >>>> @@ -632,9 +632,9 @@ static int exynos_tmu_probe(struct >>>> platform_device *pdev) if (ret) >>>> return ret; >>>> =20 >>>> - if (pdata->type =3D=3D SOC_ARCH_EXYNOS || >>>> - pdata->type =3D=3D SOC_ARCH_EXYNOS4210 || >>>> + if (pdata->type =3D=3D SOC_ARCH_EXYNOS4210 || >>>> pdata->type =3D=3D SOC_ARCH_EXYNOS4412 || >>>> + pdata->type =3D=3D SOC_ARCH_EXYNOS5250 || >>>> pdata->type =3D=3D SOC_ARCH_EXYNOS5440) >>>> data->soc =3D pdata->type; >>>> else { >>>> diff --git a/drivers/thermal/samsung/exynos_tmu.h >>>> b/drivers/thermal/samsung/exynos_tmu.h index db37df0..b42ece4 100644= >>>> --- a/drivers/thermal/samsung/exynos_tmu.h >>>> +++ b/drivers/thermal/samsung/exynos_tmu.h >>>> @@ -42,7 +42,7 @@ enum calibration_mode { >>>> enum soc_type { >>>> SOC_ARCH_EXYNOS4210 =3D 1, >>>> SOC_ARCH_EXYNOS4412, >>>> - SOC_ARCH_EXYNOS, >>>> + SOC_ARCH_EXYNOS5250, >>>> SOC_ARCH_EXYNOS5440, >>>> }; >>>> =20 >>>> diff --git a/drivers/thermal/samsung/exynos_tmu_data.c >>>> b/drivers/thermal/samsung/exynos_tmu_data.c index bd08093..188223f >>>> 100644 --- a/drivers/thermal/samsung/exynos_tmu_data.c >>>> +++ b/drivers/thermal/samsung/exynos_tmu_data.c >>>> @@ -183,7 +183,7 @@ struct exynos_tmu_init_data const >>>> exynos4412_default_tmu_data =3D { struct exynos_tmu_init_data const >>>> exynos5250_default_tmu_data =3D { .tmu_data =3D { >>>> { EXYNOS4412_TMU_DATA, >>>> - .type =3D SOC_ARCH_EXYNOS, >>>> + .type =3D SOC_ARCH_EXYNOS5250, >>>> }, >>>> }, >>>> .tmu_count =3D 1, >=20 > -- > To unsubscribe from this list: send the line "unsubscribe linux-pm" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html >=20 >=20 --=20 You have got to be excited about what you are doing. (L. Lamport) Eduardo Valentin --VxriV23ILdJdStvRGSMKosPtajFoJ5uWu Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.12 (GNU/Linux) Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iF4EAREIAAYFAlJO2ZoACgkQCXcVR3XQvP0sDQD7B52MD2xoWPbCYvVisLgvmobk MuRwecQ+owTvny7WD/0A/1T6FMBw/dgmv5ouYybMEElHRaNuD3WYksqvtdweaUDR =UoWf -----END PGP SIGNATURE----- --VxriV23ILdJdStvRGSMKosPtajFoJ5uWu-- -- 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/