On Wed, Jun 07, 2023 at 09:28:25PM +0100, Conor Dooley wrote: > From: Conor Dooley <conor.dooley@xxxxxxxxxxxxx> > > Saving off the `isa` pointer to a temp variable, followed by checking if > it has been incremented is a bit of an odd pattern. Perhaps it was done > to avoid a funky looking if statement mixed with the ifdeffery. > > Now that we use IS_ENABLED() here just return from the parser as soon as > we detect a mismatch between the string and the currently running > kernel. > > Reviewed-by: Andrew Jones <ajones@xxxxxxxxxxxxxxxx> > Signed-off-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx> > --- Reviewed-by: Sunil V L <sunilvl@xxxxxxxxxxxxxxxx>