Received-SPF: pass (google.com: domain of linux-kernel+bounces-106868-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Message-ID: <fe7f271c-1e16-4aff-8d72-4c9e9b2bc2b2@amd.com>
Date: Mon, 18 Mar 2024 19:39:56 -0500
User-Agent: Mozilla Thunderbird Beta
Subject: Re: [PATCH 1/3] dt-bindings: remoteproc: add Versal platform support
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
 andersson@kernel.org, mathieu.poirier@linaro.org, robh+dt@kernel.org,
 krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org,
 michal.simek@amd.com, ben.levinsky@amd.com
Cc: linux-remoteproc@vger.kernel.org, devicetree@vger.kernel.org,
 linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org
References: <20240315211533.1996543-1-tanmay.shah@amd.com>
 <20240315211533.1996543-2-tanmay.shah@amd.com>
 <b81d86b4-eb3c-4996-bbe6-32a8bc1de028@linaro.org>
Content-Language: en-US
From: Tanmay Shah <tanmay.shah@amd.com>
In-Reply-To: <b81d86b4-eb3c-4996-bbe6-32a8bc1de028@linaro.org>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?UWFVeFBmZ0VteUt6VnphZmtxTElsbE5DOHZTVHVpa2ozeEZ0R0RFbzBSMXYr?=
 =?utf-8?B?VGhDS2lCOVpQZWYyWEx2QjBwVDVHMXZIbTJ1RjgxMEpMTWc5elB2ZitSb3JO?=
 =?utf-8?B?cUEwcHp3Z2hPSUVWUGF6UzFDanJVZHZBMXgzQyt1Z0srQkc2TFdQdHpSTDdN?=
 =?utf-8?B?VWVwdTN0QjRmcDBHZzFBUzdGSVRscHpRL0wrNmVjb3ZXc2NTNGlkYTZOWXUy?=
 =?utf-8?B?amhjNGlWVHJ6S3lOblg5amRaL0RjWWRucE0zMXMvNUlxUEY5YzBMbnNjM0JI?=
 =?utf-8?B?V1FRcVAxclU4WnUwQ01wenJ6dXRUeW5KTzg3SW9FZjljNFQ1RkJFSm8zNm1y?=
 =?utf-8?B?M2drWk5pS3p4RlVrNlFvMzM1ZG9CVnphdm02azdYQTI4djZJT3FxWHpUckJa?=
 =?utf-8?B?YWJEbmNPZ29VU0JTZ09MNHNhUVhxY013OEFZS25aZ3FOeHhQS3g2UCtCWHBS?=
 =?utf-8?B?L3hFY3MxT0VMOGJuUUdjR2YwUVV1Rjk1V2ZvTjVuZlc2ZzN0aFY2TFpjc1oz?=
 =?utf-8?B?L3JyanZIZ1lRZXJBM2VEZWhXSmxkaHMwUzZLbkpSR1dOaEtlVFdwOE5ZYzdm?=
 =?utf-8?B?R0YyMzVBSENuWS81V25DODBPVlpURXA3TTNIeGo5K1ZadmMySVBnNzVRZGdF?=
 =?utf-8?B?VTFPYnEyY0ZzTFEzeThQL09XY3h4U1YyY2lBQlY5bndjS0k4WWpBWVg0eHRu?=
 =?utf-8?B?MXVXNDBtdzJUUTRVU2hOTjR3b3FMT0VtWXZTbWsyWWp2UE9zY3piTTFqZUp2?=
 =?utf-8?B?MjRxZENJc0UwSzJTRnpER1VzS3hQQ0FvRHFBa1ZHSmtxcm9PaytqVnBTZW9T?=
 =?utf-8?B?REo1cnhQeW1ZcFc5ODZUSWFpWjFndTRuRHRVUVorcjlubEphZ01mbXoxYWlY?=
 =?utf-8?B?VUNzY0N4ZVd6K0FoeW03b0YwWE93RjkrdjFPQ3JnTnBVZ2tHdUxyM3h5YU5k?=
 =?utf-8?B?Mlk1MG1LN29OUFV3S01oOEQyNTFoSDdLS1M5enExWCtVc213K2p3NXppNEJF?=
 =?utf-8?B?L0tvQVp2aU5rNk1paXhVQmw0VHFSSWVhTFcvYTFxVmIrMGxsaXgvT3ZjTVRm?=
 =?utf-8?B?bkRHZFFyQm5VV0ZQYmNvS0MzTTR2TjJDQlVpeFQxdTVsZzZ5S3JlMFZFQ0VZ?=
 =?utf-8?B?NHRsYXBrQklLN1h6UTlJRGErV3hHOXhVVjFSU3RQemFYbmxaTEZDRmVQT0k3?=
 =?utf-8?B?bk9QMFZBVWpRZ2ZQS1FVSWg1bFlCR2hJZ1Zobzg5NDJSeVY1aWhRZ1VYRFdh?=
 =?utf-8?B?eWZiMkFiTDJiOUc5TmtlSXRNalV3cXo1S2RTRkE0SmdybzN2enBNdkhZck11?=
 =?utf-8?B?eUlKd09IWWN4Ulp2R2I0c0Q0Rm5aTlFpT3V4ZStsREUzTDJMUzhkbUlGTTE2?=
 =?utf-8?B?Q2Njbm42bUNYK3BKSGNHeUs5dmFsemZpU0J3UUQ2QzRncnR5cVN6N3owcUFl?=
 =?utf-8?B?RVlhY2NQZzVpaGNRQlU3R05SUm1McUdTTTRxT3ZwOHNyOTlTbkRHdEkraG9D?=
 =?utf-8?B?c05GTWMwVitFY0RlNVh3ZFZJTjE5YXVrcUtRUGErUVdCL2xaTGM3OG1CTGhi?=
 =?utf-8?B?RnovbG5qMmRycHEwWHZqN0Z1MGwwNW5EdHZmRDBhWWQydng0eEUwWUY0VWFN?=
 =?utf-8?B?RkpFWkdxL3VBTUNsZXBrbjdsbCtxZXNGajRGMmFKQnZJakU2KzRCY21DbCtE?=
 =?utf-8?B?cmpvZVJRMEduZWwvZ3NRUFVxalJoSjZ0U241MSt0Tm9SZ29KRk5pa1RVTkhv?=
 =?utf-8?B?ZTZSS21vQWtYRGs4L25mbHp1c1pxYWI4RTM2SXBneDBXZ0hxMnEwLzRSaVdG?=
 =?utf-8?B?cmNCY0JlV1hRVW1nVXZzS2xwaDJFcVB0YmZ6YXM4Z2o4TFFsakFxUnBFTHBC?=
 =?utf-8?B?VXZhT1IwMUsycThHWTB4YmxtOFVqWmZPSUxCRHRKSFZtNE8rQTFhVFVQcWU4?=
 =?utf-8?B?UDhEODFjdkNwUFhCOG5FUkxYSzh6eDRVdnlzMlBYV05ONVB5d2RwT094UENE?=
 =?utf-8?B?N1dFaC82SmZ1VHBHRVVuNFRyTkc2UGFxdW14MDdqKzdpVCs1K3YzcXU1cnZD?=
 =?utf-8?B?OEprdUxBWllNek0weVJibmpic3Jzc1ZqSnQ3U1FKUXl3SUx6M2RxUzQyYkI3?=
 =?utf-8?Q?4Z3V3CbxrPnEe2/ur35lKgk+9?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: c7f57223-58ea-4d50-7c18-08dc47ad1b2c
X-MS-Exchange-CrossTenant-AuthSource: BL1PR12MB5874.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Mar 2024 00:39:59.1562
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: U1IVlIQl3tCktj5Lkuoq3KY7XcnjS4LKnmwHTvcwSRZaZia23+MgZ9pc7p0JFxAs
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS7PR12MB6189


On 3/17/24 1:50 PM, Krzysztof Kozlowski wrote:
> On 15/03/2024 22:15, Tanmay Shah wrote:
>> AMD-Xilinx Versal platform is successor of ZynqMP platform. Real-time
>> Processor Unit R5 cluster IP on Versal is same as of ZynqMP Platform.
>> Only difference is power-domains ID needed by power management firmware.
>> Hence, keeping the compatible property same as of zynqmp node.
>> 
>> Signed-off-by: Tanmay Shah <tanmay.shah@amd.com>
> 
> There is no binding change, so NAK. Platform is not being added to
> examples. You changed nothing in  Versal support...

Thanks for reviews.

Okay got it. That means I don't really need to add anything related to Versal.
I will get rid of patch that says "Versal support". Looks like it's not needed
at all.

Thanks.

> 
> Best regards,
> Krzysztof
>