On Wed, Aug 28, 2024 at 04:40:29AM +0800, Miao Wang wrote: > > > > 2024年8月8日 17:43,Hanjun Guo <guohanjun@xxxxxxxxxx> 写道: > > > > On 2024/8/8 17:00, Miao Wang via B4 Relay wrote: > >> From: Miao Wang <shankerwangmiao@xxxxxxxxx> > >> To avoid arch-specific code in general ACPI initialization flow, > >> we introduce a weak symbol acpi_arch_init. Currently, arm64 can > >> utillize this to insert its specific flow. In the future, > >> other architectures can also have chance to define their own > >> arch-specific acpi initialization process if necessary. > >> Signed-off-by: Miao Wang <shankerwangmiao@xxxxxxxxx> > >> Reviewed-by: Sunil V L <sunilvl@xxxxxxxxxxxxxxxx> > >> Reviewed-by: Sudeep Holla <sudeep.holla@xxxxxxx> > >> --- > >> Changes from v1 > >> - Change acpi_arch_init from a static inline stub to a weak function > >> according to Haijun Guo's advice > >> --- > >> Changes from v2: > >> - Add __init attribute to the weak acpi_arch_init stub > >> - Link to v2: https://lore.kernel.org/r/20240807-intro-acpi-arch-init-v2-1-9231e23a7721@xxxxxxxxx > > > > Thanks for the quick update, > > > > Acked-by: Hanjun Guo <guohanjun@xxxxxxxxxx> > > Hi, all. I wonder whether this patch is good to be applied or > any improvement is needed. > LGTM. Rafael, do you want to take this via your tree or arm64(need your ack) once you are happy with the change ? -- Regards, Sudeep