Received-SPF: pass (google.com: domain of linux-kernel+bounces-86551-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
From: Xu Yang <xu.yang_2@nxp.com>
To: Will Deacon <will@kernel.org>
CC: Frank Li <frank.li@nxp.com>, "mark.rutland@arm.com"
	<mark.rutland@arm.com>, "robh+dt@kernel.org" <robh+dt@kernel.org>,
	"krzysztof.kozlowski+dt@linaro.org" <krzysztof.kozlowski+dt@linaro.org>,
	"conor+dt@kernel.org" <conor+dt@kernel.org>, "shawnguo@kernel.org"
	<shawnguo@kernel.org>, "s.hauer@pengutronix.de" <s.hauer@pengutronix.de>,
	"kernel@pengutronix.de" <kernel@pengutronix.de>, "festevam@gmail.com"
	<festevam@gmail.com>, "john.g.garry@oracle.com" <john.g.garry@oracle.com>,
	"jolsa@kernel.org" <jolsa@kernel.org>, "namhyung@kernel.org"
	<namhyung@kernel.org>, "irogers@google.com" <irogers@google.com>,
	dl-linux-imx <linux-imx@nxp.com>, "mike.leach@linaro.org"
	<mike.leach@linaro.org>, "leo.yan@linaro.org" <leo.yan@linaro.org>,
	"peterz@infradead.org" <peterz@infradead.org>, "mingo@redhat.com"
	<mingo@redhat.com>, "acme@kernel.org" <acme@kernel.org>,
	"alexander.shishkin@linux.intel.com" <alexander.shishkin@linux.intel.com>,
	"adrian.hunter@intel.com" <adrian.hunter@intel.com>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>, "devicetree@vger.kernel.org"
	<devicetree@vger.kernel.org>, "linux-kernel@vger.kernel.org"
	<linux-kernel@vger.kernel.org>, "linux-perf-users@vger.kernel.org"
	<linux-perf-users@vger.kernel.org>, "imx@lists.linux.dev"
	<imx@lists.linux.dev>
Subject: RE: [EXT] Re: [PATCH v4 4/6] perf: imx_perf: add macro definitions
 for parsing config attr
Thread-Topic: [EXT] Re: [PATCH v4 4/6] perf: imx_perf: add macro definitions
 for parsing config attr
Thread-Index: AQHaVAmgyIcNdifNHUmteT26AlgXorEWa92AgArgGCA=
Date: Thu, 29 Feb 2024 10:29:09 +0000
Message-ID:
 <DU2PR04MB882282882D1D5824ED8DC90B8C5F2@DU2PR04MB8822.eurprd04.prod.outlook.com>
References: <20240131055811.3035741-1-xu.yang_2@nxp.com>
 <20240131055811.3035741-4-xu.yang_2@nxp.com>
 <20240222122341.GC8308@willie-the-truck>
In-Reply-To: <20240222122341.GC8308@willie-the-truck>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0:
 =?us-ascii?Q?TNRySU+c/6PFL42ZfJx0x79MwspAk0rN2OqIsulUnYvKRW7oRV/gVNbQ9qZf?=
 =?us-ascii?Q?jOIijlc6/QlN+8qus28fGKOpQRR/g9/EJewT59uwqngYE7+C0oV82eeSBC1G?=
 =?us-ascii?Q?AoLyQ3D+tMvdJPvDbDXaUF7tvoILDXFdc2a9hioS9Nkb+QB/soZKSi9f/dDa?=
 =?us-ascii?Q?28Pc3vkjimTAQM+HbZaL2VeCRHT8vfe+ilAIlHRiHnufVQ8D2f22rmcSwQUg?=
 =?us-ascii?Q?jEgiSnqCVq91uN05C/dCpqZR90NZglnd/DpOeMkD1dKRRxdaFDD280AMkTb4?=
 =?us-ascii?Q?F8OTM5LpA0rry/ykiPyEuo6TQPkFRUMqlZQZ6DRdd0nNVYS+U8Kbj5641bQK?=
 =?us-ascii?Q?URjwaAdalqtlm0FjNGkcPIEsM+q5O1/KyTqhO7jsNFO6mquDR+hyX3Y0vJjo?=
 =?us-ascii?Q?gl6UZLtLPIblEyF4W6XoiI22nI/SNEAMQ+NX6JdQGmc+vXAOC2GKK0f6uXXY?=
 =?us-ascii?Q?mUlXhBAGVdLlw9doVexjdAT2SNong+AKAeKXfEDJGRBm3dnLnVBJsM7RLzIt?=
 =?us-ascii?Q?wRPKQZcxkoWu5ksXpSbqbeeeyPv5GaZGWRA+jVcDf10SpIHhJq5CiqXWwnuf?=
 =?us-ascii?Q?w0mXJ2pNHF497CXxdtVmV5kl0G2Ukitd75q5uy00MP++esA5qZeW+tTS3I7B?=
 =?us-ascii?Q?bqOWeEvU6qC4k7IFUvUCr8ZYmNvUGuA+8U0+/ktTUkoyzgYM+qgd0LvRVZde?=
 =?us-ascii?Q?gRZSfAyzf3Hyal58HYDnxYmGm42VdVoS+soRN0u8w8ffT9Nc158cqGJNntQf?=
 =?us-ascii?Q?kkwox+ZkfB6Ats0jhxhq/RuYVtIWdZ1qRn0dmuvC07DcPIA0dhHUAcGSNMHh?=
 =?us-ascii?Q?JwKBCkYrd1NRizYWWe+75FYm449nhLlIVieJni1A/98eRxKX3krDdE/C4e6V?=
 =?us-ascii?Q?JJ8sZH/S8JudSCK+uafL4SSteZQRINnLHPXrOdW/OzTW09iMRfw/xIrWYo6P?=
 =?us-ascii?Q?8URH0YjlSq3f6v5rCI+39TGwL1jydXON9QIXHWIS4SADbupSiRyg6A7skz2F?=
 =?us-ascii?Q?oTqvOv9GXQz5SXCIdoWmUTPO/96dfoETR5XSd8ag9GFTrugYRdkbsdFJnn9X?=
 =?us-ascii?Q?QCDk8b9ijt0CPSDuK5MhcHlrLbS7vmaEHpjFWyLNaeyXYb5U8sC+LXvgUMEs?=
 =?us-ascii?Q?6KKh6jfnTqqV8P1DaZl9crk5AGiUmM/DBD6tVCOCPr1XCT9itnRxW9PpNUfs?=
 =?us-ascii?Q?HaueoS0YiZhZajEtUt8ukcPE0SXHLNUIPTxrskhVwuzDUjWQyqowBv3XK+YU?=
 =?us-ascii?Q?IkiA2u7K1zAziNKz6s5Cx4zpQbH92K2oc4uT85ge2QiktiExOVyZ8FNcinT1?=
 =?us-ascii?Q?M1As/ZZ5jkRVUHJ6nuGYLvUY14REit3oryjiOAMHlvP6SCTlszmf+2sdYr/2?=
 =?us-ascii?Q?KlUsS3mHzkI7qwT3y3TnZKb6rQsSy0+c8PGRoDhLUKb6GNnCOBqUmUxly7oM?=
 =?us-ascii?Q?LK8CsLd3KDGChS3za8bLRC9N2YqRPdQbYxMfnQjlIxVzZlYVmdC/9QOUzMl+?=
 =?us-ascii?Q?crfpiNSzPy51Y3AWaqoxWFwRgfqujoPNiz3IxBqIzS4OlqsNuN3ovTUtwCak?=
 =?us-ascii?Q?UyIc++b7ZQcPIbZzlUA=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
X-Mailing-List: linux-kernel@vger.kernel.org
List-Id: <linux-kernel.vger.kernel.org>
List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org>
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DU2PR04MB8822.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 2d588d7b-1443-4d6e-2770-08dc391143e2
X-MS-Exchange-CrossTenant-originalarrivaltime: 29 Feb 2024 10:29:09.5850
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: llxWKcLiOIxRpR3VqQZoArRHwRdRwAfrQU8+jbUsIIrcOs+qU7c0G5ax7pQKAuFgFQQdJW+X6dp2sW41x5d/ag==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR04MB8360

Hi Will,

>=20
> On Wed, Jan 31, 2024 at 01:58:09PM +0800, Xu Yang wrote:
> > The user can set event and counter in cmdline and the driver need to pa=
rse
> > it using 'config' attr value. This will add macro definitions to avoid
> > hard-code in driver.
> >
> > Signed-off-by: Xu Yang <xu.yang_2@nxp.com>
> >
> > ---
> > Changes in v4:
> >  - new patch
> > ---
> >  drivers/perf/fsl_imx9_ddr_perf.c | 19 +++++++++++++------
> >  1 file changed, 13 insertions(+), 6 deletions(-)
> >
> > diff --git a/drivers/perf/fsl_imx9_ddr_perf.c b/drivers/perf/fsl_imx9_d=
dr_perf.c
> > index 85aaaef7212f..94041f06c152 100644
> > --- a/drivers/perf/fsl_imx9_ddr_perf.c
> > +++ b/drivers/perf/fsl_imx9_ddr_perf.c
> > @@ -52,6 +52,11 @@
> >  #define NUM_COUNTERS         11
> >  #define CYCLES_COUNTER               0
> >
> > +#define CONFIG_EVENT_MASK    0x00FF
> > +#define CONFIG_EVENT_OFFSET  0
> > +#define CONFIG_COUNTER_MASK  0xFF00
> > +#define CONFIG_COUNTER_OFFSET        8
>=20
> Can you move this patch earlier, please, so that the imx95 support can us=
e
> these new definitions immediately?

Yes, I will try it in next version.

Thanks,
Xu Yang

>=20
> Will