Recently introduced helpers take pointers to uuid_{be|le} instead of reference.
Using them makes code less ugly.
Cc: Tomas Winkler tomas.winkler@intel.com Signed-off-by: Andy Shevchenko andriy.shevchenko@linux.intel.com --- drivers/misc/mei/bus-fixup.c | 2 +- drivers/misc/mei/bus.c | 2 +- drivers/misc/mei/client.c | 4 ++-- 3 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/misc/mei/bus-fixup.c b/drivers/misc/mei/bus-fixup.c index 0208c4b027c5..de0fcd98d162 100644 --- a/drivers/misc/mei/bus-fixup.c +++ b/drivers/misc/mei/bus-fixup.c @@ -416,7 +416,7 @@ void mei_cl_bus_dev_fixup(struct mei_cl_device *cldev)
f = &mei_fixups[i]; if (uuid_le_cmp(f->uuid, MEI_UUID_ANY) == 0 || - uuid_le_cmp(f->uuid, *uuid) == 0) + uuid_le_cmp_p(uuid, f->uuid) == 0) f->hook(cldev); } } diff --git a/drivers/misc/mei/bus.c b/drivers/misc/mei/bus.c index d1928fdd0f43..e15549f71891 100644 --- a/drivers/misc/mei/bus.c +++ b/drivers/misc/mei/bus.c @@ -615,7 +615,7 @@ struct mei_cl_device_id *mei_cl_device_find(struct mei_cl_device *cldev,
id = cldrv->id_table; while (uuid_le_cmp(NULL_UUID_LE, id->uuid)) { - if (!uuid_le_cmp(*uuid, id->uuid)) { + if (!uuid_le_cmp_p(uuid, id->uuid)) { match = true;
if (cldev->name[0]) diff --git a/drivers/misc/mei/client.c b/drivers/misc/mei/client.c index be64969d986a..590accb369bd 100644 --- a/drivers/misc/mei/client.c +++ b/drivers/misc/mei/client.c @@ -148,7 +148,7 @@ static struct mei_me_client *__mei_me_cl_by_uuid(struct mei_device *dev,
list_for_each_entry(me_cl, &dev->me_clients, list) { pn = &me_cl->props.protocol_name; - if (uuid_le_cmp(*uuid, *pn) == 0) + if (uuid_le_cmp_pp(uuid, pn) == 0) return mei_me_cl_get(me_cl); }
@@ -228,7 +228,7 @@ static struct mei_me_client *__mei_me_cl_by_uuid_id(struct mei_device *dev,
list_for_each_entry(me_cl, &dev->me_clients, list) { pn = &me_cl->props.protocol_name; - if (uuid_le_cmp(*uuid, *pn) == 0 && + if (uuid_le_cmp_pp(uuid, pn) == 0 && me_cl->client_id == client_id) return mei_me_cl_get(me_cl); }