Messages in this thread Patch in this message | | | Subject | Re: [PATCH v2 2/6] mei: Switch to use new generic UUID API | From | Andy Shevchenko <> | Date | Tue, 25 Jul 2017 13:37:56 +0300 |
| |
On Tue, 2017-07-25 at 10:25 +0000, Winkler, Tomas wrote: > > > > On Tue, 2017-07-25 at 00:12 +0300, Tomas Winkler wrote: > > > From: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > > > > > > There are new types and helpers that are supposed to be used in > > > new > > > code. > > > > > > As a preparation to get rid of legacy types and API functions do > > > the > > > conversion here. > > > > > > Cc: Tomas Winkler <tomas.winkler@intel.com> > > > Cc: Guenter Roeck <linux@roeck-us.net> > > > Cc: Samuel Ortiz <sameo@linux.intel.com> > > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > > > Signed-off-by: Tomas Winkler <tomas.winkler@intel.com> > > > --- > >
+Cc: Christoph.
(How come you missed him in Cc list?)
> > > > > V2: > > > 1. Fixed file2alias compilation breakage > > > 2. Dropped amt version sample from the patch > > > a. We need to see yet how to not break all our user space. > > > c. It had some checkpatch issues. > > > > Thanks, though I didn't get a context. > > Sorry, not sure what you are referring...
I mean if it's supposed to be your patch series or what? Below you clarified already. Usually we have sent the comments on top of the patch.
> Once the kernel hasn't compiled,
Yes, kbuild bot reported that.
> second I'm not very happy about this guid renaming.
There are two choices at least: 1) leave a mess in kernel 2) move to better naming scheme with a pain regarding to this action
> I understand the motivations but we have user space API named uuid > out there. Now we have guid/uuid mix mess. I don't see it as an > improvement.
Currently yes we have, this series is targeting to resolve uuid vs guid issues.
> Last looks like amt version wasn't dropped it was left out of the > index, late hour commit :(
Why it should?
> > Is it supposed to be injected in my patch series instead of what I > > have as > > patch 2? > > Hmm, yes, though you didn't send me the rest of the series so I hope > it's okay.
Yes, you were Cc'ed to cover letter and one patch I considered independent (looks like we need to figure out interaction with user space first).
> > > > > > > > > Still this is naming mess will require addition re-spin > > > > > > drivers/misc/mei/bus-fixup.c | 35 +++++++++++++++++++--------- > > > ---- > > > --- > > > drivers/misc/mei/bus.c | 14 +++++++------- > > > drivers/misc/mei/client.c | 21 ++++++++++----------- > > > drivers/misc/mei/client.h | 12 ++++++------ > > > drivers/misc/mei/hbm.c | 2 +- > > > drivers/misc/mei/hw.h | 2 +- > > > drivers/misc/mei/main.c | 2 +- > > > drivers/nfc/mei_phy.h | 5 +++-- > > > drivers/watchdog/mei_wdt.c | 5 +++-- > > > include/linux/mei_cl_bus.h | 2 +- > > > include/linux/mod_devicetable.h | 8 ++++---- > > > include/uapi/linux/mei.h | 4 +--- > > > samples/mei/mei-amt-version.c | 23 ++++++++++++++++++----- > > > scripts/mod/file2alias.c | 6 +++--- > > > 14 files changed, 78 insertions(+), 63 deletions(-) > > > > > > diff --git a/drivers/misc/mei/bus-fixup.c b/drivers/misc/mei/bus- > > > fixup.c index 0208c4b027c5..9eaac4f2d427 100644 > > > --- a/drivers/misc/mei/bus-fixup.c > > > +++ b/drivers/misc/mei/bus-fixup.c > > > @@ -27,21 +27,27 @@ > > > #include "mei_dev.h" > > > #include "client.h" > > > > > > -#define MEI_UUID_NFC_INFO UUID_LE(0xd2de1625, 0x382d, 0x417d, \ > > > - 0x48, 0xa4, 0xef, 0xab, 0xba, 0x8a, 0x12, > > > 0x06) > > > +#define MEI_UUID_NFC_INFO > > > \ > > > + GUID_INIT(0xd2de1625, 0x382d, 0x417d, > > > \ > > > + 0x48, 0xa4, 0xef, 0xab, 0xba, 0x8a, 0x12, 0x06) > > > > > > -static const uuid_le mei_nfc_info_guid = MEI_UUID_NFC_INFO; > > > +static const guid_t mei_nfc_info_guid = MEI_UUID_NFC_INFO; > > > > > > -#define MEI_UUID_NFC_HCI UUID_LE(0x0bb17a78, 0x2a8e, 0x4c50, \ > > > - 0x94, 0xd4, 0x50, 0x26, 0x67, 0x23, 0x77, > > > 0x5c) > > > +#define MEI_UUID_NFC_HCI > > > \ > > > + GUID_INIT(0x0bb17a78, 0x2a8e, 0x4c50, > > > \ > > > + 0x94, 0xd4, 0x50, 0x26, 0x67, 0x23, 0x77, 0x5c) > > > > > > -#define MEI_UUID_WD UUID_LE(0x05B79A6F, 0x4628, 0x4D7F, \ > > > - 0x89, 0x9D, 0xA9, 0x15, 0x14, 0xCB, > > > 0x32, > > > 0xAB) > > > +#define MEI_UUID_WD > > > \ > > > + GUID_INIT(0x05B79A6F, 0x4628, 0x4D7F, > > > \ > > > + 0x89, 0x9D, 0xA9, 0x15, 0x14, 0xCB, 0x32, 0xAB) > > > > > > -#define MEI_UUID_MKHIF_FIX UUID_LE(0x55213584, 0x9a29, 0x4916, \ > > > - 0xba, 0xdf, 0xf, 0xb7, 0xed, 0x68, 0x2a, > > > 0xeb) > > > +#define MEI_UUID_MKHIF_FIX > > > \ > > > + GUID_INIT(0x55213584, 0x9a29, 0x4916, > > > \ > > > + 0xba, 0xdf, 0x0f, 0xb7, 0xed, 0x68, 0x2a, 0xeb) > > > > > > -#define MEI_UUID_ANY NULL_UUID_LE > > > +#define MEI_UUID_ANY > > > \ > > > + GUID_INIT(0x00000000, 0x0000, 0x0000, > > > \ > > > + 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00) > > > > > > /** > > > * number_of_connections - determine whether an client be on the > > > bus > > > @@ -391,7 +397,7 @@ static void mei_nfc(struct mei_cl_device > > > *cldev) > > > > > > static struct mei_fixup { > > > > > > - const uuid_le uuid; > > > + const guid_t uuid; > > > void (*hook)(struct mei_cl_device *cldev); > > > } mei_fixups[] = { > > > MEI_FIXUP(MEI_UUID_ANY, number_of_connections), @@ > > > -409,15 > > > > +415,12 > > > @@ static struct mei_fixup { > > > void mei_cl_bus_dev_fixup(struct mei_cl_device *cldev) > > > { > > > struct mei_fixup *f; > > > - const uuid_le *uuid = mei_me_cl_uuid(cldev->me_cl); > > > + const guid_t *uuid = mei_me_cl_uuid(cldev->me_cl); > > > int i; > > > > > > for (i = 0; i < ARRAY_SIZE(mei_fixups); i++) { > > > - > > > f = &mei_fixups[i]; > > > - if (uuid_le_cmp(f->uuid, MEI_UUID_ANY) == 0 || > > > - uuid_le_cmp(f->uuid, *uuid) == 0) > > > + if (guid_is_null(&f->uuid) || guid_equal(&f- > > > >uuid, > > > uuid)) > > > f->hook(cldev); > > > } > > > } > > > - > > > diff --git a/drivers/misc/mei/bus.c b/drivers/misc/mei/bus.c index > > > 40c79089e548..c5165f2ec842 100644 > > > --- a/drivers/misc/mei/bus.c > > > +++ b/drivers/misc/mei/bus.c > > > @@ -417,7 +417,7 @@ EXPORT_SYMBOL_GPL(mei_cldev_set_drvdata); > > > * > > > * Return: me client uuid > > > */ > > > -const uuid_le *mei_cldev_uuid(const struct mei_cl_device *cldev) > > > +const guid_t *mei_cldev_uuid(const struct mei_cl_device *cldev) > > > { > > > return mei_me_cl_uuid(cldev->me_cl); > > > } > > > @@ -606,7 +606,7 @@ struct mei_cl_device_id > > > *mei_cl_device_find(struct > > > mei_cl_device *cldev, > > > struct mei_cl_driver > > > *cldrv) > > > { > > > const struct mei_cl_device_id *id; > > > - const uuid_le *uuid; > > > + const guid_t *uuid; > > > u8 version; > > > bool match; > > > > > > @@ -614,8 +614,8 @@ struct mei_cl_device_id > > > *mei_cl_device_find(struct > > > mei_cl_device *cldev, > > > version = mei_me_cl_ver(cldev->me_cl); > > > > > > id = cldrv->id_table; > > > - while (uuid_le_cmp(NULL_UUID_LE, id->uuid)) { > > > - if (!uuid_le_cmp(*uuid, id->uuid)) { > > > + while (!guid_is_null(&id->uuid)) { > > > + if (guid_equal(uuid, &id->uuid)) { > > > match = true; > > > > > > if (cldev->name[0]) > > > @@ -742,7 +742,7 @@ static ssize_t uuid_show(struct device *dev, > > > struct device_attribute *a, > > > char *buf) > > > { > > > struct mei_cl_device *cldev = to_mei_cl_device(dev); > > > - const uuid_le *uuid = mei_me_cl_uuid(cldev->me_cl); > > > + const guid_t *uuid = mei_me_cl_uuid(cldev->me_cl); > > > > > > return scnprintf(buf, PAGE_SIZE, "%pUl", uuid); > > > } > > > @@ -762,7 +762,7 @@ static ssize_t modalias_show(struct device > > > *dev, > > > struct device_attribute *a, > > > char *buf) > > > { > > > struct mei_cl_device *cldev = to_mei_cl_device(dev); > > > - const uuid_le *uuid = mei_me_cl_uuid(cldev->me_cl); > > > + const guid_t *uuid = mei_me_cl_uuid(cldev->me_cl); > > > u8 version = mei_me_cl_ver(cldev->me_cl); > > > > > > return scnprintf(buf, PAGE_SIZE, "mei:%s:%pUl:%02X:", @@ > > > -790,7 > > > +790,7 @@ ATTRIBUTE_GROUPS(mei_cldev); > > > static int mei_cl_device_uevent(struct device *dev, struct > > > kobj_uevent_env *env) > > > { > > > struct mei_cl_device *cldev = to_mei_cl_device(dev); > > > - const uuid_le *uuid = mei_me_cl_uuid(cldev->me_cl); > > > + const guid_t *uuid = mei_me_cl_uuid(cldev->me_cl); > > > u8 version = mei_me_cl_ver(cldev->me_cl); > > > > > > if (add_uevent_var(env, "MEI_CL_VERSION=%d", version)) > > > diff --git > > > a/drivers/misc/mei/client.c b/drivers/misc/mei/client.c index > > > be64969d986a..a44fb96452a9 100644 > > > --- a/drivers/misc/mei/client.c > > > +++ b/drivers/misc/mei/client.c > > > @@ -139,16 +139,16 @@ void mei_me_cl_add(struct mei_device *dev, > > > struct mei_me_client *me_cl) > > > * Locking: dev->me_clients_rwsem > > > */ > > > static struct mei_me_client *__mei_me_cl_by_uuid(struct > > > mei_device > > > *dev, > > > - const uuid_le *uuid) > > > + const guid_t *uuid) > > > { > > > struct mei_me_client *me_cl; > > > - const uuid_le *pn; > > > + const guid_t *pn; > > > > > > WARN_ON(!rwsem_is_locked(&dev->me_clients_rwsem)); > > > > > > list_for_each_entry(me_cl, &dev->me_clients, list) { > > > pn = &me_cl->props.protocol_name; > > > - if (uuid_le_cmp(*uuid, *pn) == 0) > > > + if (guid_equal(uuid, pn)) > > > return mei_me_cl_get(me_cl); > > > } > > > > > > @@ -167,7 +167,7 @@ static struct mei_me_client > > > *__mei_me_cl_by_uuid(struct mei_device *dev, > > > * Locking: dev->me_clients_rwsem > > > */ > > > struct mei_me_client *mei_me_cl_by_uuid(struct mei_device *dev, > > > - const uuid_le *uuid) > > > + const guid_t *uuid) > > > { > > > struct mei_me_client *me_cl; > > > > > > @@ -219,17 +219,16 @@ struct mei_me_client *mei_me_cl_by_id(struct > > > mei_device *dev, u8 client_id) > > > * Locking: dev->me_clients_rwsem > > > */ > > > static struct mei_me_client *__mei_me_cl_by_uuid_id(struct > > > mei_device > > > *dev, > > > - const uuid_le *uuid, > > > u8 > > > client_id) > > > + const guid_t *uuid, u8 > > > client_id) > > > { > > > struct mei_me_client *me_cl; > > > - const uuid_le *pn; > > > + const guid_t *pn; > > > > > > WARN_ON(!rwsem_is_locked(&dev->me_clients_rwsem)); > > > > > > list_for_each_entry(me_cl, &dev->me_clients, list) { > > > pn = &me_cl->props.protocol_name; > > > - if (uuid_le_cmp(*uuid, *pn) == 0 && > > > - me_cl->client_id == client_id) > > > + if (guid_equal(uuid, pn) && me_cl->client_id == > > > client_id) > > > return mei_me_cl_get(me_cl); > > > } > > > > > > @@ -248,7 +247,7 @@ static struct mei_me_client > > > *__mei_me_cl_by_uuid_id(struct mei_device *dev, > > > * Return: me client or null if not found > > > */ > > > struct mei_me_client *mei_me_cl_by_uuid_id(struct mei_device > > > *dev, > > > - const uuid_le *uuid, > > > u8 > > > client_id) > > > + const guid_t *uuid, u8 > > > client_id) > > > { > > > struct mei_me_client *me_cl; > > > > > > @@ -267,7 +266,7 @@ struct mei_me_client > > > > *mei_me_cl_by_uuid_id(struct > > > mei_device *dev, > > > * > > > * Locking: called under "dev->device_lock" lock > > > */ > > > -void mei_me_cl_rm_by_uuid(struct mei_device *dev, const uuid_le > > > *uuid) > > > +void mei_me_cl_rm_by_uuid(struct mei_device *dev, const guid_t > > > *uuid) > > > { > > > struct mei_me_client *me_cl; > > > > > > @@ -288,7 +287,7 @@ void mei_me_cl_rm_by_uuid(struct mei_device > > > > *dev, > > > const uuid_le *uuid) > > > * > > > * Locking: called under "dev->device_lock" lock > > > */ > > > -void mei_me_cl_rm_by_uuid_id(struct mei_device *dev, const > > > uuid_le > > > *uuid, u8 id) > > > +void mei_me_cl_rm_by_uuid_id(struct mei_device *dev, const guid_t > > > *uuid, u8 id) > > > { > > > struct mei_me_client *me_cl; > > > > > > diff --git a/drivers/misc/mei/client.h b/drivers/misc/mei/client.h > > > index 5371df4d8af3..5d996e1e39fa 100644 > > > --- a/drivers/misc/mei/client.h > > > +++ b/drivers/misc/mei/client.h > > > @@ -34,13 +34,13 @@ void mei_me_cl_add(struct mei_device *dev, > > > struct > > > mei_me_client *me_cl); > > > void mei_me_cl_del(struct mei_device *dev, struct mei_me_client > > > *me_cl); > > > > > > struct mei_me_client *mei_me_cl_by_uuid(struct mei_device *dev, > > > - const uuid_le *uuid); > > > + const guid_t *uuid); > > > struct mei_me_client *mei_me_cl_by_id(struct mei_device *dev, u8 > > > client_id); > > > struct mei_me_client *mei_me_cl_by_uuid_id(struct mei_device > > > *dev, > > > - const uuid_le *uuid, > > > u8 > > > client_id); > > > -void mei_me_cl_rm_by_uuid(struct mei_device *dev, const uuid_le > > > *uuid); > > > + const guid_t *uuid, u8 > > > client_id); > > > +void mei_me_cl_rm_by_uuid(struct mei_device *dev, const guid_t > > > *uuid); > > > void mei_me_cl_rm_by_uuid_id(struct mei_device *dev, > > > - const uuid_le *uuid, u8 id); > > > + const guid_t *uuid, u8 id); > > > void mei_me_cl_rm_all(struct mei_device *dev); > > > > > > /** > > > @@ -62,7 +62,7 @@ static inline bool mei_me_cl_is_active(const > > > struct > > > mei_me_client *me_cl) > > > * > > > * Return: me client protocol name > > > */ > > > -static inline const uuid_le *mei_me_cl_uuid(const struct > > > mei_me_client *me_cl) > > > +static inline const guid_t *mei_me_cl_uuid(const struct > > > mei_me_client > > > *me_cl) > > > { > > > return &me_cl->props.protocol_name; > > > } > > > @@ -177,7 +177,7 @@ static inline bool > > > mei_cl_is_single_recv_buf(const > > > struct mei_cl *cl) > > > * > > > * Return: return uuid of connected me client > > > */ > > > -static inline const uuid_le *mei_cl_uuid(const struct mei_cl *cl) > > > +static inline const guid_t *mei_cl_uuid(const struct mei_cl *cl) > > > { > > > return mei_me_cl_uuid(cl->me_cl); > > > } > > > diff --git a/drivers/misc/mei/hbm.c b/drivers/misc/mei/hbm.c index > > > fe6595fe94f1..059b8dfef67b 100644 > > > --- a/drivers/misc/mei/hbm.c > > > +++ b/drivers/misc/mei/hbm.c > > > @@ -329,7 +329,7 @@ static int mei_hbm_me_cl_add(struct mei_device > > > *dev, > > > struct hbm_props_response *res) > > > { > > > struct mei_me_client *me_cl; > > > - const uuid_le *uuid = &res- > > > >client_properties.protocol_name; > > > + const guid_t *uuid = &res- > > > >client_properties.protocol_name; > > > > > > mei_me_cl_rm_by_uuid(dev, uuid); > > > > > > diff --git a/drivers/misc/mei/hw.h b/drivers/misc/mei/hw.h index > > > 5c8286b40b62..e061f449ea75 100644 > > > --- a/drivers/misc/mei/hw.h > > > +++ b/drivers/misc/mei/hw.h > > > @@ -288,7 +288,7 @@ struct hbm_host_enum_response { > > > } __packed; > > > > > > struct mei_client_properties { > > > - uuid_le protocol_name; > > > + guid_t protocol_name; > > > u8 protocol_version; > > > u8 max_number_of_connections; > > > u8 fixed_address; > > > diff --git a/drivers/misc/mei/main.c b/drivers/misc/mei/main.c > > > index > > > e825f013e54e..be576c625f9f 100644 > > > --- a/drivers/misc/mei/main.c > > > +++ b/drivers/misc/mei/main.c > > > @@ -342,7 +342,7 @@ static int mei_ioctl_connect_client(struct > > > file > > > *file, > > > return -EBUSY; > > > > > > /* find ME client we're trying to connect to */ > > > - me_cl = mei_me_cl_by_uuid(dev, &data->in_client_uuid); > > > + me_cl = mei_me_cl_by_uuid(dev, (guid_t *)&data- > > > > in_client_uuid); > > > > > > if (!me_cl) { > > > dev_dbg(dev->dev, "Cannot connect to FW Client > > > UUID = > > > > %pUl\n", > > > &data->in_client_uuid); > > > diff --git a/drivers/nfc/mei_phy.h b/drivers/nfc/mei_phy.h index > > > acd3a1fc69e6..bf4775a111e8 100644 > > > --- a/drivers/nfc/mei_phy.h > > > +++ b/drivers/nfc/mei_phy.h > > > @@ -5,8 +5,9 @@ > > > #include <net/nfc/hci.h> > > > #include <linux/uuid.h> > > > > > > -#define MEI_NFC_UUID UUID_LE(0x0bb17a78, 0x2a8e, 0x4c50, \ > > > - 0x94, 0xd4, 0x50, 0x26, 0x67, 0x23, 0x77, 0x5c) > > > +#define MEI_NFC_UUID > > > \ > > > + GUID_INIT(0x0bb17a78, 0x2a8e, 0x4c50, > > > \ > > > + 0x94, 0xd4, 0x50, 0x26, 0x67, 0x23, 0x77, 0x5c) > > > #define MEI_NFC_HEADER_SIZE 10 > > > #define MEI_NFC_MAX_HCI_PAYLOAD 300 > > > > > > diff --git a/drivers/watchdog/mei_wdt.c > > > b/drivers/watchdog/mei_wdt.c > > > index b29c6fde7473..a0f052a96913 100644 > > > --- a/drivers/watchdog/mei_wdt.c > > > +++ b/drivers/watchdog/mei_wdt.c > > > @@ -667,8 +667,9 @@ static int mei_wdt_remove(struct mei_cl_device > > > *cldev) > > > return 0; > > > } > > > > > > -#define MEI_UUID_WD UUID_LE(0x05B79A6F, 0x4628, 0x4D7F, \ > > > - 0x89, 0x9D, 0xA9, 0x15, 0x14, 0xCB, > > > 0x32, > > > 0xAB) > > > +#define MEI_UUID_WD > > > \ > > > + GUID_INIT(0x05B79A6F, 0x4628, 0x4D7F, > > > \ > > > + 0x89, 0x9D, 0xA9, 0x15, 0x14, 0xCB, 0x32, 0xAB) > > > > > > static struct mei_cl_device_id mei_wdt_tbl[] = { > > > { .uuid = MEI_UUID_WD, .version = MEI_CL_VERSION_ANY }, > > > diff --git > > > a/include/linux/mei_cl_bus.h b/include/linux/mei_cl_bus.h index > > > a0d274fe08f1..81a860bfd076 100644 > > > --- a/include/linux/mei_cl_bus.h > > > +++ b/include/linux/mei_cl_bus.h > > > @@ -94,7 +94,7 @@ int mei_cldev_register_rx_cb(struct > > > mei_cl_device > > > *cldev, mei_cldev_cb_t rx_cb); > > > int mei_cldev_register_notif_cb(struct mei_cl_device *cldev, > > > mei_cldev_cb_t notif_cb); > > > > > > -const uuid_le *mei_cldev_uuid(const struct mei_cl_device *cldev); > > > +const guid_t *mei_cldev_uuid(const struct mei_cl_device *cldev); > > > u8 mei_cldev_ver(const struct mei_cl_device *cldev); > > > > > > void *mei_cldev_get_drvdata(const struct mei_cl_device *cldev); > > > diff > > > --git a/include/linux/mod_devicetable.h > > > b/include/linux/mod_devicetable.h index 3f74ef2281e8..c36d274b7c91 > > > 100644 > > > --- a/include/linux/mod_devicetable.h > > > +++ b/include/linux/mod_devicetable.h > > > @@ -405,7 +405,7 @@ struct virtio_device_id { > > > * For Hyper-V devices we use the device guid as the id. > > > */ > > > struct hv_vmbus_device_id { > > > - uuid_le guid; > > > + guid_t guid; > > > kernel_ulong_t driver_data; /* Data private to the > > > driver */ > > > }; > > > > > > @@ -627,15 +627,15 @@ struct ipack_device_id { > > > /** > > > * struct mei_cl_device_id - MEI client device identifier > > > * @name: helper name > > > - * @uuid: client uuid > > > + * @uuid: client GUID > > > * @version: client protocol version > > > * @driver_info: information used by the driver. > > > * > > > - * identifies mei client device by uuid and name > > > + * identifies mei client device by GUID and name > > > */ > > > struct mei_cl_device_id { > > > char name[MEI_CL_NAME_SIZE]; > > > - uuid_le uuid; > > > + guid_t uuid; > > > __u8 version; > > > kernel_ulong_t driver_info; > > > }; > > > diff --git a/include/uapi/linux/mei.h b/include/uapi/linux/mei.h > > > index > > > 7c3b64f6a215..cf02d56a13e6 100644 > > > --- a/include/uapi/linux/mei.h > > > +++ b/include/uapi/linux/mei.h > > > @@ -67,8 +67,6 @@ > > > #ifndef _LINUX_MEI_H > > > #define _LINUX_MEI_H > > > > > > -#include <linux/uuid.h> > > > - > > > /* > > > * This IOCTL is used to associate the current file descriptor > > > with a > > > * FW Client (given by UUID). This opens a communication channel > > > @@ > > > -102,7 +100,7 @@ struct mei_client { > > > */ > > > struct mei_connect_client_data { > > > union { > > > - uuid_le in_client_uuid; > > > + __u8 in_client_uuid[16]; > > > struct mei_client out_client_properties; > > > }; > > > }; > > > diff --git a/samples/mei/mei-amt-version.c b/samples/mei/mei-amt- > > > version.c index 57d0d871dcf7..396915e37d3b 100644 > > > --- a/samples/mei/mei-amt-version.c > > > +++ b/samples/mei/mei-amt-version.c > > > @@ -89,8 +89,20 @@ > > > fprintf(stderr, "Error: " fmt, ##ARGS); \ > > > } while (0) > > > > > > +/* FIXME: Use libuuid instead */ > > > +typedef struct { > > > + __u8 b[16]; > > > +} guid_t; > > > + > > > +#define GUID_INIT(a, b, c, d0, d1, d2, d3, d4, d5, d6, d7) > > > \ > > > +((guid_t) > > > \ > > > +{{ (a) & 0xff, ((a) >> 8) & 0xff, ((a) >> 16) & 0xff, ((a) >> 24) > > > & > > > 0xff, \ > > > + (b) & 0xff, ((b) >> 8) & 0xff, > > > \ > > > + (c) & 0xff, ((c) >> 8) & 0xff, > > > \ > > > + (d0), (d1), (d2), (d3), (d4), (d5), (d6), (d7) }}) > > > + > > > struct mei { > > > - uuid_le guid; > > > + guid_t guid; > > > bool initialized; > > > bool verbose; > > > unsigned int buf_size; > > > @@ -108,8 +120,8 @@ static void mei_deinit(struct mei *cl) > > > cl->initialized = false; > > > } > > > > > > -static bool mei_init(struct mei *me, const uuid_le *guid, > > > - unsigned char req_protocol_version, bool verbose) > > > +static bool mei_init(struct mei *me, const guid_t *guid, > > > + unsigned char req_protocol_version, bool > > > verbose) > > > { > > > int result; > > > struct mei_client *cl; > > > @@ -270,8 +282,9 @@ struct amt_host_if_resp_header { > > > unsigned char data[0]; > > > } __attribute__((packed)); > > > > > > -const uuid_le MEI_IAMTHIF = UUID_LE(0x12f80028, 0xb4b7, > > > 0x4b2d, \ > > > - 0xac, 0xa8, 0x46, 0xe0, 0xff, > > > 0x65, > > > 0x81, 0x4c); > > > +const guid_t MEI_IAMTHIF = > > > + GUID_INIT(0x12f80028, 0xb4b7, 0x4b2d, > > > + 0xac, 0xa8, 0x46, 0xe0, 0xff, 0x65, 0x81, > > > 0x4c); > > > > > > #define AMT_HOST_IF_CODE_VERSIONS_REQUEST 0x0400001A > > > #define AMT_HOST_IF_CODE_VERSIONS_RESPONSE 0x0480001A diff --git > > > a/scripts/mod/file2alias.c b/scripts/mod/file2alias.c index > > > 29d6699d5a06..045ec1a10e25 100644 > > > --- a/scripts/mod/file2alias.c > > > +++ b/scripts/mod/file2alias.c > > > @@ -36,7 +36,7 @@ typedef uint16_t __u16; > > > typedef unsigned char __u8; > > > typedef struct { > > > __u8 b[16]; > > > -} uuid_le; > > > +} guid_t; > > > > > > /* Big exception to the "don't include kernel headers into > > > userspace, > > > which > > > * even potentially has different endianness and word sizes, > > > since @@ > > > -134,7 +134,7 @@ static inline void add_wildcard(char *str) > > > strcat(str + len, "*"); > > > } > > > > > > -static inline void add_uuid(char *str, uuid_le uuid) > > > +static inline void add_guid(char *str, guid_t uuid) > > > { > > > int len = strlen(str); > > > > > > @@ -1230,7 +1230,7 @@ static int do_mei_entry(const char > > > *filename, > > > void *symval, > > > > > > sprintf(alias, MEI_CL_MODULE_PREFIX); > > > sprintf(alias + strlen(alias), "%s:", (*name)[0] ? > > > *name : > > > "*"); > > > - add_uuid(alias, *uuid); > > > + add_guid(alias, *uuid); > > > ADD(alias, ":", version != MEI_CL_VERSION_ANY, version); > > > > > > strcat(alias, ":*"); > > > > -- > > Andy Shevchenko <andriy.shevchenko@linux.intel.com> > > Intel Finland Oy
-- Andy Shevchenko <andriy.shevchenko@linux.intel.com> Intel Finland Oy
| |