On Fri, 20 Sep 2024, WangYuli wrote: > There is a spelling mistake of 'ACCCESS_LOCAL' which should be > instead of 'ACCESS_LOCAL'. "instead of ACCESS_LOCAL" means opposite of what you intended. I'd just drop "instead of". > Cc: Thomas Zimmermann <tzimmermann@xxxxxxx> > Signed-off-by: WangYuli <wangyuli@xxxxxxxxxxxxx> > --- > drivers/platform/x86/intel/pmt/class.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/platform/x86/intel/pmt/class.c b/drivers/platform/x86/intel/pmt/class.c > index c04bb7f97a4d..7680474c4f96 100644 > --- a/drivers/platform/x86/intel/pmt/class.c > +++ b/drivers/platform/x86/intel/pmt/class.c > @@ -207,7 +207,7 @@ static int intel_pmt_populate_entry(struct intel_pmt_entry *entry, > /* > * Some hardware use a different calculation for the base address > * when access_type == ACCESS_LOCAL. On the these systems > - * ACCCESS_LOCAL refers to an address in the same BAR as the > + * ACCESS_LOCAL refers to an address in the same BAR as the The change is fine. -- i.