Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   "N, Pandith" <pandith.n@intel.com>
To:     Greg KH <gregkh@linuxfoundation.org>
CC:     "arnd@arndb.de" <arnd@arndb.de>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "Zhou, Furong" <furong.zhou@intel.com>,
        "mgross@linux.intel.com" <mgross@linux.intel.com>,
        "Sangannavar, Mallikarjunappa" 
        <mallikarjunappa.sangannavar@intel.com>,
        "andriy.shevchenko@linux.intel.com" 
        <andriy.shevchenko@linux.intel.com>,
        "Raja Subramanian, Lakshmi Bai" 
        <lakshmi.bai.raja.subramanian@intel.com>
Subject: RE: [PATCH V5 1/1] misc: NOC Probe Counters
Thread-Topic: [PATCH V5 1/1] misc: NOC Probe Counters
Thread-Index: AQHXd7PdZivYur/1kEWpwYZUnXcrZ6tAeXqAgAMVBiA=
Date:   Thu, 15 Jul 2021 07:00:13 +0000
Message-ID: <BYAPR11MB3528AA1CD11585D2BC94DEB9E1129@BYAPR11MB3528.namprd11.prod.outlook.com>
References: <20210713065347.21554-1-pandith.n@intel.com>
 <YO05g5JiOcF/bFza@kroah.com>
In-Reply-To: <YO05g5JiOcF/bFza@kroah.com>
Accept-Language: en-US
Content-Language: en-US
dlp-product: dlpe-windows
dlp-reaction: no-action
dlp-version: 11.5.1.3
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?RYy7JUoHb4Is7+ou3k8+q1dfKTR28QxdgFZdvQS7K3OGXKqTNRkAtpXJjmAI?=
 =?us-ascii?Q?j26dHWTBzbNP/bkvpWKuwpS38C2/xV1yR09aCR7t5L+/cK8bqSIGrviJDqXn?=
 =?us-ascii?Q?LvCpLtwxJcs8isIDKjvGdtHMZbYl0TezDGgbnADW3/4hM1FRz4Nxb9DYXsQd?=
 =?us-ascii?Q?HS+unBvh7g7rBQ2rXv6luEAOVcjiORt2PDCyA6KHewsH6mejPNBe1b7Xw+5h?=
 =?us-ascii?Q?GNNCfSEEj0yE0CoiemNP91Sl9D2sjRdKsGCUkjlcDjrMBFsgW6J3mMJhSG2t?=
 =?us-ascii?Q?WhkDOqms3aGe69aiOzlv8raRXu42Zz889D/8E4qU8VJF7+7/XNL1zFrKYzce?=
 =?us-ascii?Q?+jNgPwzAPDuA9OxemMQjwArW1hkl8fhrbdgnjTY5N01/GFpEbiysgrRsnaQW?=
 =?us-ascii?Q?RbPqdyxyQBel7VHItziqVOAoOJq3sL32tXjIhXbhCrsd4ecdADDV8cqpTFtI?=
 =?us-ascii?Q?fua73OchLfKSdnPd2Gm0vXrQ4IUXTy0mQyc/T8Q5WRZtWnXT+mryt/JquHvn?=
 =?us-ascii?Q?jXlxAzhxl4tmtAC2TIWENgFFUewIxA9aJiFHauVGH9cDGZcR1A2WnCp+ham0?=
 =?us-ascii?Q?NO84jEQ67IQmdrBuGJ8tbbwxna7e89altWxPaIVhvJpc1IECKMm/89tWChEG?=
 =?us-ascii?Q?zFaHSmx2ahUNBsHixr98rlscyEiP/7HV1J68RY4E82nTMoWTGvxJbD0FGqxD?=
 =?us-ascii?Q?z9xsxznmW4z/t0jQOyqFNL/aVJrvL8nCHyG6JcLzu6WFsmZt9UDdVWGkjAB5?=
 =?us-ascii?Q?5KJr5cvY5Nmib5cP+1KUGKrL8Y/IYSypAOX7W09zfA8VrvQV0bzUKOzP8mpd?=
 =?us-ascii?Q?6mutZS1BIBGJ6Pbum465f+EiKfptyVV0YEQmHj8vMQNj0d+QYzb/JJi9+6EL?=
 =?us-ascii?Q?NCtMAVup44QRbJhOIzrBaHH+zmqhqagm+Oc5Ayh6Ww5eIupTDXgxOuTej8as?=
 =?us-ascii?Q?ybe75bXTuI7rXVVSB26SHMioO9jluoJ95OHAmNuhIqsLsXwPGFu93YWwyVez?=
 =?us-ascii?Q?XfeyATTQ33S66NeAecwVDy+QvmX57fNmItzytpdVWQ9u9+KYwhT5s6fCgq/4?=
 =?us-ascii?Q?3emjr8rSpm21L8iHDpFPjTDnDQ8FmgfMh3RmnZSuoGzF4trs6HCKIdIWnkie?=
 =?us-ascii?Q?+wko09zqPsJDpSx5OK3QoqmAUij+7jvIwjT2Z3C7JWGOGwGF3biRZcsfp5RV?=
 =?us-ascii?Q?Y3Y+U+N5hmTnapQknQnofglE8/Z6Qlwzh8T1f1o1s/EuEnxMIyFcAWE6/UbU?=
 =?us-ascii?Q?kpxaBHfcW9Utm37E9l89x/uDqvX5JY/FCpaL0fhE6y2QxYzdDonHweQ4k8Dh?=
 =?us-ascii?Q?chw=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: BYAPR11MB3528.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 2be38b06-9d49-4053-a09c-08d9475e31c3
X-MS-Exchange-CrossTenant-originalarrivaltime: 15 Jul 2021 07:00:13.5508
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: yZ50zVXNs1jytgcxQEJExKEOa0IeMbnbLFscX74PyRE0dkpseubwONdMzCf1/VFp5ctyCIyrp40Ps1nVs/FJEw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR11MB4893
X-OriginatorOrg: intel.com
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Greg KH,

> -----Original Message-----
> From: Greg KH <gregkh@linuxfoundation.org>
> Sent: Tuesday, July 13, 2021 12:28 PM
> To: N, Pandith <pandith.n@intel.com>
> Cc: arnd@arndb.de; linux-kernel@vger.kernel.org; Zhou, Furong
> <furong.zhou@intel.com>; mgross@linux.intel.com; Sangannavar,
> Mallikarjunappa <mallikarjunappa.sangannavar@intel.com>;
> andriy.shevchenko@linux.intel.com; Raja Subramanian, Lakshmi Bai
> <lakshmi.bai.raja.subramanian@intel.com>
> Subject: Re: [PATCH V5 1/1] misc: NOC Probe Counters
>=20
> On Tue, Jul 13, 2021 at 12:23:47PM +0530, pandith.n@intel.com wrote:
> > --- /dev/null
> > +++ b/drivers/misc/noc/Kconfig
> > @@ -0,0 +1,16 @@
> > +# Copyright (C) 2019 Intel Corporation # SPDX-License-Identifier:
> > +GPL-2.0-only
> > +
> > +
> > +config NOC_DSS
> > +	tristate "Enable DSS NOC probing"
>=20
> Please be more descriptive of what exactly this is, it does not make sens=
e to
> anyone who does not know what DSS or NOC means.
>=20
Will rephrase and add more description about DSS, NOC in help section

         tristate "Enable DDR profiling using NOC for Intel Keem Bay"
        depends on ARCH_KEEMBAY || COMPILE_TEST
        help
          Enable this option for DDR bandwidth measurements using NOC

          Add support for Network on chip (NOC) in DDR Subsystem(DSS).
          DSS NOC has capabilities to enable and get statistics profiling.
          NOC driver enables features to configure and capture NOC probe
          counters, needed for DSS bandwidth measurement.
          Say Y if using a processor that includes the Intel VPU such as
          Keem Bay.  If unsure, say N.

> > +	depends on ARCH_KEEMBAY
> > +	default y
>=20
> You only use 'y' if the machine can not boot without the option.
> Otherwise it is left alone.
>=20
Have removed default 'y' =20

> And what about CONFIG_TEST ability to test build this code?  Without that=
, you
> will get almost no one every building this code to see if anything breaks=
 over
> time.
>=20
Have added COMPILE_TEST option as below=20
depends on ARCH_KEEMBAY || COMPILE_TEST

> thanks,
>=20
> greg k-h