Received-SPF: pass (google.com: domain of linux-kernel+bounces-205570-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Message-ID: <02dba55e-39a3-4853-ac52-834cb1c0fc7f@marvell.com>
Date: Fri, 7 Jun 2024 13:23:48 +0530
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v6 1/1] irqchip/gic-v3: Enable non-coherent
 redistributors/ITSes ACPI probing
To: Lorenzo Pieralisi <lpieralisi@kernel.org>
Cc: linux-kernel@vger.kernel.org, Robin Murphy <robin.murphy@arm.com>,
        Marc Zyngier <maz@kernel.org>, Thomas Gleixner <tglx@linutronix.de>,
        Mark Rutland <mark.rutland@arm.com>,
        "Rafael J. Wysocki"
 <rafael@kernel.org>,
        linux-arm-kernel@lists.infradead.org, linux-acpi@vger.kernel.org,
        acpica-devel@lists.linux.dev
References: <20240123110332.112797-1-lpieralisi@kernel.org>
 <20240606094238.757649-1-lpieralisi@kernel.org>
 <20240606094238.757649-2-lpieralisi@kernel.org>
 <f901fd06-3b6b-4444-a3da-2b75c59059d3@marvell.com>
 <ZmK5Ilk37GyJ4QEa@lpieralisi>
Content-Language: en-US
From: Amit Singh Tomar <amitsinght@marvell.com>
In-Reply-To: <ZmK5Ilk37GyJ4QEa@lpieralisi>
Content-Type: text/plain; charset=UTF-8; format=flowed
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?SVREbXpleFFtUytldmlhRUpKYytJRVdKYkVtWlk2VG91T3JxL3AzdWdWd3l6?=
 =?utf-8?B?RTFSbmdDOFRPdHYvMGpDMXVMREFwOXExVzFKYVJOWWxVS2Y5cld2MVo1bkFB?=
 =?utf-8?B?R29wTWNmYkJXUWp4cVBNdjVNVHozS1B1cllUckNvTmZPbXdndUZxeXVJVDhp?=
 =?utf-8?B?b0ZLQ3MwUHVwUGN0MXQ2VlJTTGdRSVpCRnhHR0U3dDN6dkJlU0hPTDhBNGxU?=
 =?utf-8?B?OWZvMVlkZGhsYStKUUpYNnpuM2RSaG9KVlp4ekk2RFA1SldEZE1WOGJJOW5l?=
 =?utf-8?B?SmtkMHl6OTlXanVINkp1cHZiWVViaUQyT2JhcGhyVGU2TjgzSUt4anAxanlh?=
 =?utf-8?B?ZDRtMTlxSFRhd2VNdXJBajVtT3hRSlF3RjJ5S1ZYbnNiK1gyak11ZnFyMWN3?=
 =?utf-8?B?ajlFU2w1WlNDODkySXlSNnFHZG80WkhIWjFvcFZ3dEJTcHBLdjB0Rm9EUFg4?=
 =?utf-8?B?dXBVejJXd1RQbUFIZ1ppTDVFOGdEV3ZhUE53WVJzbDJ4Y3lQc1hJTW0rUDRv?=
 =?utf-8?B?dTBjNmdtT2J0VGNYNUxqZlYrdzgva0hTTE5pS3gwZzBadWhVRUVKQW9zaXh2?=
 =?utf-8?B?Vks3MlFZZ0NPeGNJRUFFTW1vcGxzRVJkOHZURTJJR2xFeExLRmV0emNwVUZC?=
 =?utf-8?B?OE4wbXcvenZCa3drWFVsR3FheFY0T2FpbGtqd0JDdHhLbHI1NCtaQzR6bk9s?=
 =?utf-8?B?VWd5b1dKMEVkRjViY2NTaUFabXpHbzlUZVIrQndrbEtnR3AzelRNYlNkdWNH?=
 =?utf-8?B?RHBscjRPUHE0Y1RjUXpYOXhpamdZTWZUc0J0V0NTT0F3UjQ4R2Q3czlpYzM3?=
 =?utf-8?B?V0FHNmxLNnVhTUNFZFpjOFNRMWNEQm8zeFgxQW5ra0Y1ZWxWV3JOVElhaStP?=
 =?utf-8?B?dXRVdnFWa0ttOWwvbXpKMUQrNEJzQ1hpWjVWemY3NEVubFJ4K1l4Y0ZkUSth?=
 =?utf-8?B?Zk9iVVNSeGlxK2xFUnpqaDhJMS91RGFNZERLUHpkVk5MUzNRS0xPQzRVYS9P?=
 =?utf-8?B?cVkxQTZjL014c1dOQ3RsYk85Tytzd0hsR0V2UjgxTkdvOUZ5b3V0OEZlQ3N3?=
 =?utf-8?B?OGJnY3VkTEJGcWpVbmE3N0JXYkQ5akpBZW4yMXdvUXY5RDlncWIxUnFuME52?=
 =?utf-8?B?WXYwRHV6M0JqeHpZVjNWTGp1NlpJOFpFQ3BUQk5TZE1qdzVQMGhJQkJwQkh2?=
 =?utf-8?B?Y0dUemYrM3RoclhYYjBSN2VaZlJMUmhzNHNTSklJdjk3U0JJZ1BlSFlUY3Mw?=
 =?utf-8?B?S3N1Vy8vQWp4dDUrV0N2a1lCbHI4blh1MndLbk4xQzRPbHlucThxczlmNVlD?=
 =?utf-8?B?UzNSbDVHRXp4SHZmTGRTSHRaeElGaGNQOFM4d1JoLys3OGV1M2xUcVp2bWlW?=
 =?utf-8?B?dzhMMzlzdXB0eXZPSWIzcUU0VU1GLzRlR01sVm4rSGt3ZWZ2SEprMXByeURr?=
 =?utf-8?B?cHQySlVHNnZZWXFpSVpmdUNRRnNKWExsUjZYSnlGSDV2WWYwdFRVTDBiRWlK?=
 =?utf-8?B?QjhZeDdSSmwyYkRHYjEyRkFmY3dpT0dTWG0vSm11L1RSL0dIc002Q2lRRC9u?=
 =?utf-8?B?NkZNall1blNoV2dTTHNORzM5aEJZMHlnZ256MkhzS25qK1VUR1RjZ1dTZ3A3?=
 =?utf-8?B?WnNNdGl5UW1XTjNQSkk5bS9nSHJrK2lGN1I4Zmc4NEtSOXZjWE91aFNBd2Er?=
 =?utf-8?B?RThZakhYVS9YWVJZN0RpNEczSFFQTCtrUGtsaWdaaWUvekg5NVlLb2R3c09w?=
 =?utf-8?B?bFBqM1NiRFV3YkZ6bkdhQ050VlJvUFpVdFl3MWlwakhIT2t3TjBCL244YXNa?=
 =?utf-8?B?QkhEMXQvc3dzR0ZGV1A5bWptYjg2MG1ESWIrVWlGenJ3VmRHUkhsSjFZSUM3?=
 =?utf-8?B?eUNuMStWc2tUQ1NEZHhoVWUySldQOERsWDNlWmhSakxHQzZ4V2dCak9UaHVM?=
 =?utf-8?B?eDhRWjBUcFpqNDJYOGNoVkVkUUQ5bWgzZzBLcTZJMzJ2UG5vZVpSSjBnbzc5?=
 =?utf-8?B?R2dOR2tGZnRZT1dnNXVSV2ZWaTNJb2tVU2tvNXYrMEl4N2tJTjcyWmtGellz?=
 =?utf-8?B?SHVPeHp3Zks4dkVJRWFJcjlDeDQvMkhsWXgyaXF0N2lYNWpMNy9vdDJGcnpK?=
 =?utf-8?Q?Mkh+1QyffFa/8xoUGO5daT8Rz?=
X-OriginatorOrg: marvell.com
X-MS-Exchange-CrossTenant-Network-Message-Id: d4e71c7c-0f89-4d90-e8ff-08dc86c6fbd0
X-MS-Exchange-CrossTenant-AuthSource: MW4PR18MB5084.namprd18.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Jun 2024 07:53:56.9241
 (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: rYJ0twQc47aN4jbrBAO64YkE9i4AoAmUbI8I5cXSk6MiUQD5xeOUze3afHXfISRGlzyIHmb/VFhwaDrurlNrBg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR18MB4430
X-Proofpoint-ORIG-GUID: _0XePVrKMm0xtspbDNhDwZAE2k0YX9Hb
X-Proofpoint-GUID: _0XePVrKMm0xtspbDNhDwZAE2k0YX9Hb
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.293,Aquarius:18.0.1039,Hydra:6.0.680,FMLib:17.12.28.16
 definitions=2024-06-07_02,2024-06-06_02,2024-05-17_01

> On Fri, Jun 07, 2024 at 12:21:54AM +0530, Amit Singh Tomar wrote:
> 
> [...]
> 
>>> diff --git a/drivers/acpi/processor_core.c b/drivers/acpi/processor_core.c
>>> index b203cfe28550..915713c0e9b7 100644
>>> --- a/drivers/acpi/processor_core.c
>>> +++ b/drivers/acpi/processor_core.c
>>> @@ -215,6 +215,21 @@ phys_cpuid_t __init acpi_map_madt_entry(u32 acpi_id)
>>>    	return rv;
>>>    }
>>> +int __init acpi_get_madt_revision(void)
>>
>> Wondering, if we can have a generic function (acpi_get_tbl_revision) to
>> obtain the revision number for any ACPI table, not just specific to MADT?
> 
> We could - I don't think there would be users other than code in this
> patch though so I thought it would not be necessary.
> 

Right, it might not be essential now but I see that MPAM will be another 
user of it once the MPAM patches are out.

https://git.kernel.org/pub/scm/linux/kernel/git/morse/linux.git/tree/drivers/acpi/arm64/mpam.c?h=mpam/snapshot/v6.7-rc2#n299

Thanks
-Amit