Received-SPF: pass (google.com: domain of linux-kernel+bounces-154571-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
Date: Tue, 23 Apr 2024 08:48:52 +0200
From: Robert Richter <rrichter@amd.com>
To: Dan Williams <dan.j.williams@intel.com>
Cc: "Rafael J. Wysocki" <rafael@kernel.org>,
	Dave Hansen <dave.hansen@linux.intel.com>,
	Alison Schofield <alison.schofield@intel.com>,
	linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-cxl@vger.kernel.org, Len Brown <lenb@kernel.org>
Subject: Re: [PATCH v3 3/5] ACPI/NUMA: Remove architecture dependent
 remainings
Message-ID: <ZidZ1PUhQTKFlU4q@rric.localdomain>
References: <20240419140203.1996635-1-rrichter@amd.com>
 <20240419140203.1996635-4-rrichter@amd.com>
 <66271ccd13e5c_691029471@dwillia2-xfh.jf.intel.com.notmuch>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <66271ccd13e5c_691029471@dwillia2-xfh.jf.intel.com.notmuch>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?LlrivH/bmzuFW/SkWfPAhBX/VpxpxvgaVAtzK/4lXbsxkET1Zm3oyjvgXxEl?=
 =?us-ascii?Q?sVaoaBN7d7QlR2sa6kxxa69VNdZ9nrewFvFFf26H5/jCFqMSuUMDegRXpIqy?=
 =?us-ascii?Q?vcz1/W4tEuM/QyjrNFmXCbNMGO9lLyj0aRl9cOR5cBvj/hl2vFqNPzY49u4G?=
 =?us-ascii?Q?bwmfKTGHTgqZ2rzZGNi6lkN3DZCcMD5gAZP3eCD5k9uKu4FMfnPZnOlmiBNx?=
 =?us-ascii?Q?7ySOT4Ye22AJPh9p6RXO/Xa3ecxhDgmniRvuDzaLX7MVH7TWwukqev8TBH0F?=
 =?us-ascii?Q?QV59AfCDDMjI1XofqtXAI5p3MpbRznbjw6mvcjb3l3CXO9UwdcVeg7idRKDv?=
 =?us-ascii?Q?PEZ234s4fJiDtx2iBy6C1iFPIycZIY46WVYLnnAKfhVf0muiM9ivDTfxp5Jr?=
 =?us-ascii?Q?50GEz8MD/RnmpP4b0quZKnENALM/riuQP8QRdoAOb57vhXXEA4GjqP6BN4kF?=
 =?us-ascii?Q?tkbOzwf0QjJyTRaXWMelmmm9CnYtwQ7qHZGKjZzrMrhfyiaENSnP4B5q0msA?=
 =?us-ascii?Q?XMkVQLUXbOvCmizm04u+iu/WgPOCD8taqObQbeWKllcq6ZCmdVYvlrKgIWjd?=
 =?us-ascii?Q?O0YL80oTbYNoGcGodS4yaazwN+UJjRK4hv3gRTnFy6mzS7qWmfPtamwIAxu/?=
 =?us-ascii?Q?COlhDWtgrBAq3wyOVm2daqMhAwvZoWGLD2Iojx6PN+zdzCsX+mCXToTV1wle?=
 =?us-ascii?Q?IJ/fNzvHvriaja4zP51NKWsa1tHlQexJaQHrXoSjDJbmGxZKTJjj2/dy27Kl?=
 =?us-ascii?Q?2vnCibq5ECxZmgxGc4IZqMeH68bbSHh3RJ/fGPVOhzr1cncvogEJf0avhhhn?=
 =?us-ascii?Q?yDY1SYwXkl4666Dvgd6Cs1g9nCYqfuQcYYK91R9rTHPs3B/gIoUhGkomRAnw?=
 =?us-ascii?Q?a0bDMmmdtDO3O6S5K4+PLxF7bz8D3Iup3gXDShKxRUQH/3vQn5jLqn52Og3s?=
 =?us-ascii?Q?RoSwYiQMyqPWY49Fxu773nsZM5zfLurlhwXnnvLk/ca6DEduQwpOrfD5ZvZf?=
 =?us-ascii?Q?uBGAGuBdo0rPzY+KbtmTaqYm2qhP1R9MVfAQ5P3zYPYHZSlHe+rfKftsLtol?=
 =?us-ascii?Q?ZucSrUkhgbs3Z+ccePB5XIa+WGpxTpLXhd0StE86fPkg1Hd2LfUDRQwtw0dw?=
 =?us-ascii?Q?3BsbJRRMyYPcPclvrZ+5di7/E8WfpkTn7UaW/Cw/0PWz8F4JMJip4QA9PfLq?=
 =?us-ascii?Q?aEnn15nEnPer4yKdRsdbqSXWWgcsRo0BrHJWDmMq2WuyOZopNugbf5TJZ1yA?=
 =?us-ascii?Q?8L2IoWaow7uhOK3H/vdeevwld4irpNIYBP1RcLKQsj7U8mwvoXtsqWFTyjGU?=
 =?us-ascii?Q?v7U2FCV1vLUpqmpXlSJ97XrFLcEOGu6vgOEPoOx8zL1thigg7kruaqxykbIU?=
 =?us-ascii?Q?SyxXCZB9tBTh05V1oChZ64U2OV4QlPtUcDHFaWmxeTdnJQXmq9A2edisypNZ?=
 =?us-ascii?Q?q8RZ4U8eSUlLMEDhRfyHrhOkEp4B2/v6ngZ84AhIURegcbhsUnOlDTPMTwKY?=
 =?us-ascii?Q?+nl8ICXcI6JWBXfBQcQNDrtESPX1KNgbeWHXfVBGLXJFAThAZ15pJNA3rPg3?=
 =?us-ascii?Q?M9bPKz18K6geDtBcRxWR41SSzjg07RBsvfDLBclP?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 9ed8a4e1-cf63-47f7-b5e0-08dc636173eb
X-MS-Exchange-CrossTenant-AuthSource: CYYPR12MB8750.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Apr 2024 06:48:58.8697
 (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: rKmUqsXxAqMt+fkLVagAIh4dFuxI5t/t3PxyPP16Gg8ntNtv4hS4TjXE8MhI7cYRy9vd5CnTUwmLa0H9CzDrKg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW4PR12MB6708

On 22.04.24 19:28:29, Dan Williams wrote:
> Robert Richter wrote:
> > With the removal of the Itanium architecture [1] the last architecture
> > dependent functions:
> > 
> >  acpi_numa_slit_init(), acpi_numa_memory_affinity_init()
> > 
> > were removed. Remove its remainings in the header files too an make
> > them static.
> > 
> > [1] commit cf8e8658100d ("arch: Remove Itanium (IA-64) architecture")
> > 
> > Signed-off-by: Robert Richter <rrichter@amd.com>
> > ---
> >  drivers/acpi/numa/srat.c | 17 ++---------------
> >  include/linux/acpi.h     |  5 -----
> >  2 files changed, 2 insertions(+), 20 deletions(-)
> 
> Looks good,
> 
> Reviewed-by: Dan Williams <dan.j.williams@intel.com>

I assume this is for 4/5?

Thanks for review.

-Robert