Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.34 as permitted sender) client-ip=23.128.96.34;
From:   Linu Cherian <lcherian@marvell.com>
To:     Conor Dooley <conor@kernel.org>
CC:     Mike Leach <mike.leach@linaro.org>,
        "suzuki.poulose@arm.com" <suzuki.poulose@arm.com>,
        "james.clark@arm.com" <james.clark@arm.com>,
        "leo.yan@linaro.org" <leo.yan@linaro.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "coresight@lists.linaro.org" <coresight@lists.linaro.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "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>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        Sunil Kovvuri Goutham <sgoutham@marvell.com>,
        George Cherian <gcherian@marvell.com>
Subject: RE: [EXT] Re: [PATCH 1/7] dt-bindings: arm: coresight-tmc: Add
 "memory-region" property
Thread-Topic: [EXT] Re: [PATCH 1/7] dt-bindings: arm: coresight-tmc: Add
 "memory-region" property
Thread-Index: AQHZ8to0uuH+gntbaUWsA3manUggY7A8o7aAgAZwxCCABkm3AIAHDoqg
Date:   Thu, 19 Oct 2023 01:12:22 +0000
Message-ID: <PH0PR18MB5002D6EE41A5C285A2CC4E38CED4A@PH0PR18MB5002.namprd18.prod.outlook.com>
References: <20230929133754.857678-1-lcherian@marvell.com>
 <20230929133754.857678-2-lcherian@marvell.com>
 <CAJ9a7VhzARGmywQFPNCZ27D5UKEEPSR9_hmL5fo3daFWpB26Vg@mail.gmail.com>
 <PH0PR18MB500266F8C82877450DC9DDD2CED1A@PH0PR18MB5002.namprd18.prod.outlook.com>
 <20231014-carbon-slept-9464499a4156@spud>
In-Reply-To: <20231014-carbon-slept-9464499a4156@spud>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?L6dX0RpdE+aWDnUoMwRQ7tVxNLu7H4cajdWXka50HkT/ieVcHzUKvPMrU+Q1?=
 =?us-ascii?Q?lum5ePldq0zfUG91al1Ap1IHxGZyeCY585oIZVwivBJ3GAEbH5wKpE8bvsJZ?=
 =?us-ascii?Q?bej0/FhuwFYZJkVt2UEJW4wseBSO161xiK/37T9g8VzJFXPwzCL3mGnQp+U4?=
 =?us-ascii?Q?1+UZaGVjx4kWuWpQlFeJw3UiaSJDRDfD0yatxEGN7x+poBcQKDCrOIRUl4R3?=
 =?us-ascii?Q?Khn+ZqkQ2z7HDgFBwSlPrt3dQ0+b2+DQ1Ub3MV5IA/JPqSQq4Azbt+ejDY+M?=
 =?us-ascii?Q?Pw8reerfuRv+XxDzP6kZKDFQMoBZm+9cd+1bDE2ApHjpC7+VBTTXMVgRcYCR?=
 =?us-ascii?Q?kts4gBB9o6LmwyfoZSbHXp6uOVUsQ221ghaZLJLcSipD8r1wwNueIFQCrLiY?=
 =?us-ascii?Q?0qFU5z6vN/vtkS+G3HS5aS8XWDIzSK7E7DQH35tRjemK4D0iMFp6wkrIM1k7?=
 =?us-ascii?Q?2Y5CLTfeJWNK2d3pUH266nU9+8ZFAvjBR+dp8oR7a4AEbEk+cfXh5+LES3Sk?=
 =?us-ascii?Q?8sSwrnwhZnApm9OQNn+/NjbYfbW6QtS60qZ/QfO4w6rQqA0K6RIi7ZtA2C0g?=
 =?us-ascii?Q?f6tzOQrgmOP9VZY5fF3xmakuvPQI6HybXdrEkIBgd45JXrNQOz3s8+1zLUQz?=
 =?us-ascii?Q?cHJc9LPqPWk0HhgqwcVhLzxf+QVU+voIz6Ic8eH8chys2VeOoXLyDICPZTUx?=
 =?us-ascii?Q?0vT3vMnWdlSesIiBJmUSAenzF00PT5v8A3ceLueNnIqoFXmKWJMtaf3OeIi/?=
 =?us-ascii?Q?oDkCkCR6FdsG82XUYZx8B3BdRYD3mUy5pZyPQ7yh2A8p19OhCzmj0ZdWFxd2?=
 =?us-ascii?Q?MtAKsAdcoNy5M/YBMH3dD5NmlVJtyTQtdFdeSdkRNSg54tmMlaaz7m0O0ko3?=
 =?us-ascii?Q?KpmeeUB83tqmd3LoHc8Qk3G70i4fEbkEZOZlEY9+Wf8NhDfAsNmTuPgkbSyq?=
 =?us-ascii?Q?ObSiytNqfpdqbd/KDewQYETVs0qoxJAwdfmBFZrwAAvRp0oY5psivCXFmnEF?=
 =?us-ascii?Q?glAW2fz4rFviNrVAi3tcUmLspLSV+LiLEBQ62vgwTTrkO6qTGKLoc5f9S+rc?=
 =?us-ascii?Q?a7hs1eIF9k37yg22MbUjAHdahw8bU2verog+/f0ixZ5yAh2mYjXBMqrzBY5Q?=
 =?us-ascii?Q?Spc96VlBsJMU6E5TPjvXkLtTZjj269bu4dN9u0eZlfO/BIMCEKFhK9Lx8oBP?=
 =?us-ascii?Q?mQIn3pIp+R1bQPAyV5d7Cbj6v90BOWk05CVZ/XHnkIoTw798fxHsMsJjUxoT?=
 =?us-ascii?Q?v8DZzM3hxoFpuzil2yEbmfspid/4MQX7jj9nkjQemmiZS0j5ncvYv6s8C0eY?=
 =?us-ascii?Q?ReknWCjMaPVSXZcaVXI6H0K6tdnjXJJ4ZMi1rEM/1othte5pLlyjjcYzEPCz?=
 =?us-ascii?Q?wEAKXnHSqFkdgnL1rTScH6gdvkO6wvTh8rQgfafh989RCkaVYsrBzFCUwq9R?=
 =?us-ascii?Q?jwdPPw6eB4lacJ5AUVVeV6tNLrEke/yvV+GBj5whqRluD8++CmIT6s2eNUqN?=
 =?us-ascii?Q?9eleuX8OoR8iXvy+QqcOYrtlZrZ1L30x/XkYXtb/23BWMPfYvaqQ5YTKvpqm?=
 =?us-ascii?Q?zBuz0+GZercWSkTw3Cu6oCaGJ9Pg85H5OjCmSCPa?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: marvell.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: PH0PR18MB5002.namprd18.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 75233eef-4e88-4abc-ca3c-08dbd04072c1
X-MS-Exchange-CrossTenant-originalarrivaltime: 19 Oct 2023 01:12:22.4507
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 70e1fb47-1155-421d-87fc-2e58f638b6e0
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: XNPsryIr3pyVuvqdx0vGb3yy87mFo1v3XYNAbKljrXSAKzBsGbH2Vm7ERugSrFgtVczEIMWnWXKLnIvT9EVEgg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN7PR18MB5387
X-Proofpoint-ORIG-GUID: ojJoZ2KrEqXL4udZuOeWQIDSWbyBQksf
X-Proofpoint-GUID: ojJoZ2KrEqXL4udZuOeWQIDSWbyBQksf
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.272,Aquarius:18.0.980,Hydra:6.0.619,FMLib:17.11.176.26
 definitions=2023-10-19_01,2023-10-18_01,2023-05-22_02
X-Spam-Status: No, score=-0.8 required=5.0 tests=DKIM_SIGNED,DKIM_VALID,
	HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,
	SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on howler.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (howler.vger.email [0.0.0.0]); Wed, 18 Oct 2023 18:13:13 -0700 (PDT)

Hi Conor,

> -----Original Message-----
> From: Conor Dooley <conor@kernel.org>
> Sent: Saturday, October 14, 2023 6:56 PM
> To: Linu Cherian <lcherian@marvell.com>
> Cc: Mike Leach <mike.leach@linaro.org>; suzuki.poulose@arm.com;
> james.clark@arm.com; leo.yan@linaro.org; linux-arm-
> kernel@lists.infradead.org; coresight@lists.linaro.org; linux-
> kernel@vger.kernel.org; robh+dt@kernel.org;
> krzysztof.kozlowski+dt@linaro.org; conor+dt@kernel.org;
> devicetree@vger.kernel.org; Sunil Kovvuri Goutham
> <sgoutham@marvell.com>; George Cherian <gcherian@marvell.com>
> Subject: Re: [EXT] Re: [PATCH 1/7] dt-bindings: arm: coresight-tmc: Add
> "memory-region" property
>=20
> On Sat, Oct 14, 2023 at 11:36:37AM +0000, Linu Cherian wrote:
> > > > +      - description: Reserved meta data memory. Used for ETR and E=
TF
> sinks.
> > > > +
> > > > +  memory-region-names:
> > > > +    items:
> > > > +      - const: trace-mem
> > > > +      - const: metadata-mem
> > > > +
> > >
> > > Is there a constraint required here? If we are using the memory area
> > > for trace in a panic situation, then we must have the meta data
> > > memory area defined?
> > > Perhaps a set of names such as "etr-trace-mem", "panic-trace-mem" ,
> > > "panic-metadata-mem", were the first is for general ETR trace in
> > > non-panic situation and then constrain the "panic-" areas to appear
> together.
> > > The "etr-trace-mem", "panic-trace-mem" could easily point to the
> > > same area.
> > >
> > As noted above, we do not have other generic use case for these reserve=
d
> regions now.
> > Hence two regions/names, panic-trace-mem and panic-metadata-mem
> with
> > constraints kept as
> >  minItems: 2 and maxItems: 2 would suffice ?
>=20
> Whatever you do, please delete the -mem suffix.

Ack.