diff for duplicates of <016d01d3c07b$34fa84e0$9eef8ea0$@codeaurora.org> diff --git a/a/1.txt b/N1/1.txt index ff282ca..2053638 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -3,16 +3,19 @@ > -----Original Message----- > From: Stephen Boyd <sboyd@kernel.org> > Sent: Monday, March 19, 2018 19:45 -> To: Ilia Lin <ilialin@codeaurora.org>; linux-arm-kernel@lists.infradead.org; +> To: Ilia Lin <ilialin@codeaurora.org>; = +linux-arm-kernel@lists.infradead.org; > linux-arm-msm@vger.kernel.org; linux-clk@vger.kernel.org; > sboyd@codeaurora.org > Cc: mark.rutland@arm.com; devicetree@vger.kernel.org; > rnayak@codeaurora.org; robh@kernel.org; will.deacon@arm.com; -> amit.kucheria@linaro.org; tfinkel@codeaurora.org; ilialin@codeaurora.org; +> amit.kucheria@linaro.org; tfinkel@codeaurora.org; = +ilialin@codeaurora.org; > nicolas.dechesne@linaro.org; celster@codeaurora.org -> Subject: Re: [PATCH v3 01/10] soc: qcom: Separate kryo l2 accessors from +> Subject: Re: [PATCH v3 01/10] soc: qcom: Separate kryo l2 accessors = +from > PMU driver -> +>=20 > Quoting Ilia Lin (2018-02-14 05:59:43) > > diff --git a/arch/arm64/Kconfig.platforms > > b/arch/arm64/Kconfig.platforms index fbedbd8..78a103b 100644 @@ -25,17 +28,19 @@ > > +config ARCH_MSM8996 > > + bool "Enable Support for Qualcomm Technologies, Inc. MSM8996" > > + depends on ARCH_QCOM -> +>=20 > Is this really the proposed approach? Doesn't the PMU code use these -> accessors? I would think the accessors are compiled if ARCH_QCOM=y so +> accessors? I would think the accessors are compiled if ARCH_QCOM=3Dy = +so > that the PMU stuff keeps working. Drop this Kconfig addition? I'll check whether the kryo-accessors code is SOC specific or not. -> +>=20 > > + help > > + This enables support for the MSM8996 chipset. If you do not -> > + wish to build a kernel that runs on this chipset, say 'N' here. +> > + wish to build a kernel that runs on this chipset, say 'N' = +here. > > + > > config ARCH_ROCKCHIP > > bool "Rockchip Platforms" @@ -45,28 +50,33 @@ I'll check whether the kryo-accessors code is SOC specific or not. > > --- a/drivers/perf/qcom_l2_pmu.c > > +++ b/drivers/perf/qcom_l2_pmu.c > > @@ -1,4 +1,4 @@ -> > -/* Copyright (c) 2015-2017 The Linux Foundation. All rights reserved. -> > +/* Copyright (c) 2015-2018 The Linux Foundation. All rights reserved. -> -> Sorry, this makes little sense. First off, code is only being deleted and more +> > -/* Copyright (c) 2015-2017 The Linux Foundation. All rights = +reserved. +> > +/* Copyright (c) 2015-2018 The Linux Foundation. All rights = +reserved. +>=20 +> Sorry, this makes little sense. First off, code is only being deleted = +and more > than half the file isn't being modified so why update copyright dates? Agree. Will roll back. -> +>=20 > > * -> > * This program is free software; you can redistribute it and/or modify -> > * it under the terms of the GNU General Public License version 2 and +> > * This program is free software; you can redistribute it and/or = +modify +> > * it under the terms of the GNU General Public License version 2 = +and > > diff --git a/drivers/soc/qcom/Makefile b/drivers/soc/qcom/Makefile > > index dcebf28..4e91e2a 100644 > > --- a/drivers/soc/qcom/Makefile > > +++ b/drivers/soc/qcom/Makefile -> > @@ -12,3 +12,4 @@ obj-$(CONFIG_QCOM_SMEM_STATE) += +> > @@ -12,3 +12,4 @@ obj-$(CONFIG_QCOM_SMEM_STATE) +=3D > smem_state.o -> > obj-$(CONFIG_QCOM_SMP2P) += smp2p.o -> > obj-$(CONFIG_QCOM_SMSM) += smsm.o -> > obj-$(CONFIG_QCOM_WCNSS_CTRL) += wcnss_ctrl.o -> > +obj-$(CONFIG_ARCH_MSM8996) += kryo-l2-accessors.o +> > obj-$(CONFIG_QCOM_SMP2P) +=3D smp2p.o +> > obj-$(CONFIG_QCOM_SMSM) +=3D smsm.o +> > obj-$(CONFIG_QCOM_WCNSS_CTRL) +=3D wcnss_ctrl.o +> > +obj-$(CONFIG_ARCH_MSM8996) +=3D kryo-l2-accessors.o > > diff --git a/drivers/soc/qcom/kryo-l2-accessors.c > > b/drivers/soc/qcom/kryo-l2-accessors.c > > new file mode 100644 @@ -75,11 +85,13 @@ Agree. Will roll back. > > +++ b/drivers/soc/qcom/kryo-l2-accessors.c > > @@ -0,0 +1,64 @@ > > +/* -> > + * Copyright (c) 2014-2015, 2018, The Linux Foundation. All rights reserved. +> > + * Copyright (c) 2014-2015, 2018, The Linux Foundation. All rights = +reserved. > > + * > > + * This program is free software; you can redistribute it and/or > > +modify -> > + * it under the terms of the GNU General Public License version 2 and +> > + * it under the terms of the GNU General Public License version 2 = +and > > + * only version 2 as published by the Free Software Foundation. > > + * > > + * This program is distributed in the hope that it will be useful, @@ -87,12 +99,12 @@ Agree. Will roll back. > > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > > + * GNU General Public License for more details. > > + */ -> +>=20 > SPDX tags? Will do. -> +>=20 > > diff --git a/include/soc/qcom/kryo-l2-accessors.h > > b/include/soc/qcom/kryo-l2-accessors.h > > new file mode 100644 @@ -105,7 +117,8 @@ Will do. > > + * > > + * This program is free software; you can redistribute it and/or > > +modify -> > + * it under the terms of the GNU General Public License version 2 and +> > + * it under the terms of the GNU General Public License version 2 = +and > > + * only version 2 as published by the Free Software Foundation. > > + * > > + * This program is distributed in the hope that it will be useful, @@ -116,17 +129,18 @@ Will do. > > + > > +#ifndef __ASM_ARCH_MSM_MSM_KRYO_L2_ACCESSORS_H > > +#define __ASM_ARCH_MSM_MSM_KRYO_L2_ACCESSORS_H -> +>=20 > Maybe change this tO __SOC_ARCH_QCOM_KRYO_L2_ACCESSORS_H No problem. Will change. -> +>=20 > > + > > +#ifdef CONFIG_ARCH_QCOM > > +void set_l2_indirect_reg(u64 reg_addr, u64 val); > > +u64 get_l2_indirect_reg(u64 reg_addr); #endif -> -> This ifdef isn't doing much. Drop it? Or provide the inline nop alternatives. +>=20 +> This ifdef isn't doing much. Drop it? Or provide the inline nop = +alternatives. Right. I'll drop it. diff --git a/a/content_digest b/N1/content_digest index a47bb14..a2eeb8f 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -5,19 +5,19 @@ "Subject\0RE: [PATCH v3 01/10] soc: qcom: Separate kryo l2 accessors from PMU driver\0" "Date\0Tue, 20 Mar 2018 20:42:23 +0200\0" "To\0'Stephen Boyd' <sboyd@kernel.org>" - linux-arm-kernel@lists.infradead.org - linux-arm-msm@vger.kernel.org - linux-clk@vger.kernel.org - " sboyd@codeaurora.org\0" - "Cc\0mark.rutland@arm.com" - devicetree@vger.kernel.org - rnayak@codeaurora.org - robh@kernel.org - will.deacon@arm.com - amit.kucheria@linaro.org - tfinkel@codeaurora.org - nicolas.dechesne@linaro.org - " celster@codeaurora.org\0" + <linux-arm-kernel@lists.infradead.org> + <linux-arm-msm@vger.kernel.org> + <linux-clk@vger.kernel.org> + " <sboyd@codeaurora.org>\0" + "Cc\0<mark.rutland@arm.com>" + <devicetree@vger.kernel.org> + <rnayak@codeaurora.org> + <robh@kernel.org> + <will.deacon@arm.com> + <amit.kucheria@linaro.org> + <tfinkel@codeaurora.org> + <nicolas.dechesne@linaro.org> + " <celster@codeaurora.org>\0" "\00:1\0" "b\0" "\n" @@ -25,16 +25,19 @@ "> -----Original Message-----\n" "> From: Stephen Boyd <sboyd@kernel.org>\n" "> Sent: Monday, March 19, 2018 19:45\n" - "> To: Ilia Lin <ilialin@codeaurora.org>; linux-arm-kernel@lists.infradead.org;\n" + "> To: Ilia Lin <ilialin@codeaurora.org>; =\n" + "linux-arm-kernel@lists.infradead.org;\n" "> linux-arm-msm@vger.kernel.org; linux-clk@vger.kernel.org;\n" "> sboyd@codeaurora.org\n" "> Cc: mark.rutland@arm.com; devicetree@vger.kernel.org;\n" "> rnayak@codeaurora.org; robh@kernel.org; will.deacon@arm.com;\n" - "> amit.kucheria@linaro.org; tfinkel@codeaurora.org; ilialin@codeaurora.org;\n" + "> amit.kucheria@linaro.org; tfinkel@codeaurora.org; =\n" + "ilialin@codeaurora.org;\n" "> nicolas.dechesne@linaro.org; celster@codeaurora.org\n" - "> Subject: Re: [PATCH v3 01/10] soc: qcom: Separate kryo l2 accessors from\n" + "> Subject: Re: [PATCH v3 01/10] soc: qcom: Separate kryo l2 accessors =\n" + "from\n" "> PMU driver\n" - "> \n" + ">=20\n" "> Quoting Ilia Lin (2018-02-14 05:59:43)\n" "> > diff --git a/arch/arm64/Kconfig.platforms\n" "> > b/arch/arm64/Kconfig.platforms index fbedbd8..78a103b 100644\n" @@ -47,17 +50,19 @@ "> > +config ARCH_MSM8996\n" "> > + bool \"Enable Support for Qualcomm Technologies, Inc. MSM8996\"\n" "> > + depends on ARCH_QCOM\n" - "> \n" + ">=20\n" "> Is this really the proposed approach? Doesn't the PMU code use these\n" - "> accessors? I would think the accessors are compiled if ARCH_QCOM=y so\n" + "> accessors? I would think the accessors are compiled if ARCH_QCOM=3Dy =\n" + "so\n" "> that the PMU stuff keeps working. Drop this Kconfig addition?\n" "\n" "I'll check whether the kryo-accessors code is SOC specific or not.\n" "\n" - "> \n" + ">=20\n" "> > + help\n" "> > + This enables support for the MSM8996 chipset. If you do not\n" - "> > + wish to build a kernel that runs on this chipset, say 'N' here.\n" + "> > + wish to build a kernel that runs on this chipset, say 'N' =\n" + "here.\n" "> > +\n" "> > config ARCH_ROCKCHIP\n" "> > bool \"Rockchip Platforms\"\n" @@ -67,28 +72,33 @@ "> > --- a/drivers/perf/qcom_l2_pmu.c\n" "> > +++ b/drivers/perf/qcom_l2_pmu.c\n" "> > @@ -1,4 +1,4 @@\n" - "> > -/* Copyright (c) 2015-2017 The Linux Foundation. All rights reserved.\n" - "> > +/* Copyright (c) 2015-2018 The Linux Foundation. All rights reserved.\n" - "> \n" - "> Sorry, this makes little sense. First off, code is only being deleted and more\n" + "> > -/* Copyright (c) 2015-2017 The Linux Foundation. All rights =\n" + "reserved.\n" + "> > +/* Copyright (c) 2015-2018 The Linux Foundation. All rights =\n" + "reserved.\n" + ">=20\n" + "> Sorry, this makes little sense. First off, code is only being deleted =\n" + "and more\n" "> than half the file isn't being modified so why update copyright dates?\n" "\n" "Agree. Will roll back.\n" "\n" - "> \n" + ">=20\n" "> > *\n" - "> > * This program is free software; you can redistribute it and/or modify\n" - "> > * it under the terms of the GNU General Public License version 2 and\n" + "> > * This program is free software; you can redistribute it and/or =\n" + "modify\n" + "> > * it under the terms of the GNU General Public License version 2 =\n" + "and\n" "> > diff --git a/drivers/soc/qcom/Makefile b/drivers/soc/qcom/Makefile\n" "> > index dcebf28..4e91e2a 100644\n" "> > --- a/drivers/soc/qcom/Makefile\n" "> > +++ b/drivers/soc/qcom/Makefile\n" - "> > @@ -12,3 +12,4 @@ obj-$(CONFIG_QCOM_SMEM_STATE) +=\n" + "> > @@ -12,3 +12,4 @@ obj-$(CONFIG_QCOM_SMEM_STATE) +=3D\n" "> smem_state.o\n" - "> > obj-$(CONFIG_QCOM_SMP2P) += smp2p.o\n" - "> > obj-$(CONFIG_QCOM_SMSM) += smsm.o\n" - "> > obj-$(CONFIG_QCOM_WCNSS_CTRL) += wcnss_ctrl.o\n" - "> > +obj-$(CONFIG_ARCH_MSM8996) += kryo-l2-accessors.o\n" + "> > obj-$(CONFIG_QCOM_SMP2P) +=3D smp2p.o\n" + "> > obj-$(CONFIG_QCOM_SMSM) +=3D smsm.o\n" + "> > obj-$(CONFIG_QCOM_WCNSS_CTRL) +=3D wcnss_ctrl.o\n" + "> > +obj-$(CONFIG_ARCH_MSM8996) +=3D kryo-l2-accessors.o\n" "> > diff --git a/drivers/soc/qcom/kryo-l2-accessors.c\n" "> > b/drivers/soc/qcom/kryo-l2-accessors.c\n" "> > new file mode 100644\n" @@ -97,11 +107,13 @@ "> > +++ b/drivers/soc/qcom/kryo-l2-accessors.c\n" "> > @@ -0,0 +1,64 @@\n" "> > +/*\n" - "> > + * Copyright (c) 2014-2015, 2018, The Linux Foundation. All rights reserved.\n" + "> > + * Copyright (c) 2014-2015, 2018, The Linux Foundation. All rights =\n" + "reserved.\n" "> > + *\n" "> > + * This program is free software; you can redistribute it and/or\n" "> > +modify\n" - "> > + * it under the terms of the GNU General Public License version 2 and\n" + "> > + * it under the terms of the GNU General Public License version 2 =\n" + "and\n" "> > + * only version 2 as published by the Free Software Foundation.\n" "> > + *\n" "> > + * This program is distributed in the hope that it will be useful,\n" @@ -109,12 +121,12 @@ "> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the\n" "> > + * GNU General Public License for more details.\n" "> > + */\n" - "> \n" + ">=20\n" "> SPDX tags?\n" "\n" "Will do.\n" "\n" - "> \n" + ">=20\n" "> > diff --git a/include/soc/qcom/kryo-l2-accessors.h\n" "> > b/include/soc/qcom/kryo-l2-accessors.h\n" "> > new file mode 100644\n" @@ -127,7 +139,8 @@ "> > + *\n" "> > + * This program is free software; you can redistribute it and/or\n" "> > +modify\n" - "> > + * it under the terms of the GNU General Public License version 2 and\n" + "> > + * it under the terms of the GNU General Public License version 2 =\n" + "and\n" "> > + * only version 2 as published by the Free Software Foundation.\n" "> > + *\n" "> > + * This program is distributed in the hope that it will be useful,\n" @@ -138,19 +151,20 @@ "> > +\n" "> > +#ifndef __ASM_ARCH_MSM_MSM_KRYO_L2_ACCESSORS_H\n" "> > +#define __ASM_ARCH_MSM_MSM_KRYO_L2_ACCESSORS_H\n" - "> \n" + ">=20\n" "> Maybe change this tO __SOC_ARCH_QCOM_KRYO_L2_ACCESSORS_H\n" "\n" "No problem. Will change.\n" "\n" - "> \n" + ">=20\n" "> > +\n" "> > +#ifdef CONFIG_ARCH_QCOM\n" "> > +void set_l2_indirect_reg(u64 reg_addr, u64 val);\n" "> > +u64 get_l2_indirect_reg(u64 reg_addr); #endif\n" - "> \n" - "> This ifdef isn't doing much. Drop it? Or provide the inline nop alternatives.\n" + ">=20\n" + "> This ifdef isn't doing much. Drop it? Or provide the inline nop =\n" + "alternatives.\n" "\n" Right. I'll drop it. -05e45524e77174f58879a9dbf9d2a2b23db85af5b2d766ea48ad68b0c132cfff +969cef1e3e218de4773b45b90d3fc08aaade29efb2fd19ae4e3fa52f36786c0b
diff --git a/a/1.txt b/N2/1.txt index ff282ca..89d61fe 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -3,13 +3,13 @@ > -----Original Message----- > From: Stephen Boyd <sboyd@kernel.org> > Sent: Monday, March 19, 2018 19:45 -> To: Ilia Lin <ilialin@codeaurora.org>; linux-arm-kernel@lists.infradead.org; -> linux-arm-msm@vger.kernel.org; linux-clk@vger.kernel.org; -> sboyd@codeaurora.org -> Cc: mark.rutland@arm.com; devicetree@vger.kernel.org; -> rnayak@codeaurora.org; robh@kernel.org; will.deacon@arm.com; -> amit.kucheria@linaro.org; tfinkel@codeaurora.org; ilialin@codeaurora.org; -> nicolas.dechesne@linaro.org; celster@codeaurora.org +> To: Ilia Lin <ilialin@codeaurora.org>; linux-arm-kernel at lists.infradead.org; +> linux-arm-msm at vger.kernel.org; linux-clk at vger.kernel.org; +> sboyd at codeaurora.org +> Cc: mark.rutland at arm.com; devicetree at vger.kernel.org; +> rnayak at codeaurora.org; robh at kernel.org; will.deacon at arm.com; +> amit.kucheria at linaro.org; tfinkel at codeaurora.org; ilialin at codeaurora.org; +> nicolas.dechesne at linaro.org; celster at codeaurora.org > Subject: Re: [PATCH v3 01/10] soc: qcom: Separate kryo l2 accessors from > PMU driver > diff --git a/a/content_digest b/N2/content_digest index a47bb14..062c321 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -1,23 +1,10 @@ "ref\01518616792-29028-1-git-send-email-ilialin@codeaurora.org\0" "ref\01518616792-29028-2-git-send-email-ilialin@codeaurora.org\0" "ref\0152148150406.242365.14021083340615345047@swboyd.mtv.corp.google.com\0" - "From\0<ilialin@codeaurora.org>\0" - "Subject\0RE: [PATCH v3 01/10] soc: qcom: Separate kryo l2 accessors from PMU driver\0" + "From\0ilialin@codeaurora.org (ilialin at codeaurora.org)\0" + "Subject\0[PATCH v3 01/10] soc: qcom: Separate kryo l2 accessors from PMU driver\0" "Date\0Tue, 20 Mar 2018 20:42:23 +0200\0" - "To\0'Stephen Boyd' <sboyd@kernel.org>" - linux-arm-kernel@lists.infradead.org - linux-arm-msm@vger.kernel.org - linux-clk@vger.kernel.org - " sboyd@codeaurora.org\0" - "Cc\0mark.rutland@arm.com" - devicetree@vger.kernel.org - rnayak@codeaurora.org - robh@kernel.org - will.deacon@arm.com - amit.kucheria@linaro.org - tfinkel@codeaurora.org - nicolas.dechesne@linaro.org - " celster@codeaurora.org\0" + "To\0linux-arm-kernel@lists.infradead.org\0" "\00:1\0" "b\0" "\n" @@ -25,13 +12,13 @@ "> -----Original Message-----\n" "> From: Stephen Boyd <sboyd@kernel.org>\n" "> Sent: Monday, March 19, 2018 19:45\n" - "> To: Ilia Lin <ilialin@codeaurora.org>; linux-arm-kernel@lists.infradead.org;\n" - "> linux-arm-msm@vger.kernel.org; linux-clk@vger.kernel.org;\n" - "> sboyd@codeaurora.org\n" - "> Cc: mark.rutland@arm.com; devicetree@vger.kernel.org;\n" - "> rnayak@codeaurora.org; robh@kernel.org; will.deacon@arm.com;\n" - "> amit.kucheria@linaro.org; tfinkel@codeaurora.org; ilialin@codeaurora.org;\n" - "> nicolas.dechesne@linaro.org; celster@codeaurora.org\n" + "> To: Ilia Lin <ilialin@codeaurora.org>; linux-arm-kernel at lists.infradead.org;\n" + "> linux-arm-msm at vger.kernel.org; linux-clk at vger.kernel.org;\n" + "> sboyd at codeaurora.org\n" + "> Cc: mark.rutland at arm.com; devicetree at vger.kernel.org;\n" + "> rnayak at codeaurora.org; robh at kernel.org; will.deacon at arm.com;\n" + "> amit.kucheria at linaro.org; tfinkel at codeaurora.org; ilialin at codeaurora.org;\n" + "> nicolas.dechesne at linaro.org; celster at codeaurora.org\n" "> Subject: Re: [PATCH v3 01/10] soc: qcom: Separate kryo l2 accessors from\n" "> PMU driver\n" "> \n" @@ -153,4 +140,4 @@ "\n" Right. I'll drop it. -05e45524e77174f58879a9dbf9d2a2b23db85af5b2d766ea48ad68b0c132cfff +2893533adc2bf19948d5f88418c7149e36f62aaed1ca8c0fc923009eacdcad35
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.