On Tue, 6 Feb 2024 15:28:30 -0700 Dave Jiang <dave.jiang@xxxxxxxxx> wrote: > Both generic node and HMAT handling code have been using magic numbers to > indicate access classes for 'struct access_coordinate'. Introduce enums to > enumerate the access0 and access1 classes shared by the two subsystems. > Update the function parameters and callers as appropriate to utilize the > new enum. > > Access0 is named to ACCESS_COORDINATE_LOCAL in order to indicate that the > access class is for 'struct access_coordinate' between a target node and > the nearest initiator node. > > Access1 is named to ACCESS_COORDINATE_CPU in order to indicate that the > access class is for 'struct access_coordinate' between a target node and > the nearest CPU node. > > Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > Cc: Rafael J. Wysocki <rafael@xxxxxxxxxx> > Signed-off-by: Dave Jiang <dave.jiang@xxxxxxxxx> Reviewed-by: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx> I resisted bikeshedding on names. These are clear enough for me.