Commit 14106501 authored by Andy Shevchenko's avatar Andy Shevchenko Committed by Jiri Kosina

HID: intel-ish-hid: Switch to use new generic UUID API

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.
Signed-off-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: default avatarChristoph Hellwig <hch@lst.de>
Acked-by: default avatarSrinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Signed-off-by: default avatarJiri Kosina <jkosina@suse.cz>
parent 2edefc05
...@@ -788,8 +788,8 @@ static int hid_ishtp_cl_probe(struct ishtp_cl_device *cl_device) ...@@ -788,8 +788,8 @@ static int hid_ishtp_cl_probe(struct ishtp_cl_device *cl_device)
if (!cl_device) if (!cl_device)
return -ENODEV; return -ENODEV;
if (uuid_le_cmp(hid_ishtp_guid, if (!guid_equal(&hid_ishtp_guid,
cl_device->fw_client->props.protocol_name) != 0) &cl_device->fw_client->props.protocol_name))
return -ENODEV; return -ENODEV;
client_data = devm_kzalloc(&cl_device->dev, sizeof(*client_data), client_data = devm_kzalloc(&cl_device->dev, sizeof(*client_data),
......
...@@ -29,9 +29,9 @@ ...@@ -29,9 +29,9 @@
client->cl_device->ishtp_dev, __VA_ARGS__) client->cl_device->ishtp_dev, __VA_ARGS__)
/* ISH Transport protocol (ISHTP in short) GUID */ /* ISH Transport protocol (ISHTP in short) GUID */
static const uuid_le hid_ishtp_guid = UUID_LE(0x33AECD58, 0xB679, 0x4E54, static const guid_t hid_ishtp_guid =
0x9B, 0xD9, 0xA0, 0x4D, 0x34, GUID_INIT(0x33AECD58, 0xB679, 0x4E54,
0xF0, 0xC2, 0x26); 0x9B, 0xD9, 0xA0, 0x4D, 0x34, 0xF0, 0xC2, 0x26);
/* ISH HID message structure */ /* ISH HID message structure */
struct hostif_msg_hdr { struct hostif_msg_hdr {
......
...@@ -133,18 +133,15 @@ int ishtp_write_message(struct ishtp_device *dev, struct ishtp_msg_hdr *hdr, ...@@ -133,18 +133,15 @@ int ishtp_write_message(struct ishtp_device *dev, struct ishtp_msg_hdr *hdr,
* *
* Return: fw client index or -ENOENT if not found * Return: fw client index or -ENOENT if not found
*/ */
int ishtp_fw_cl_by_uuid(struct ishtp_device *dev, const uuid_le *uuid) int ishtp_fw_cl_by_uuid(struct ishtp_device *dev, const guid_t *uuid)
{ {
int i, res = -ENOENT; unsigned int i;
for (i = 0; i < dev->fw_clients_num; ++i) { for (i = 0; i < dev->fw_clients_num; ++i) {
if (uuid_le_cmp(*uuid, dev->fw_clients[i].props.protocol_name) if (guid_equal(uuid, &dev->fw_clients[i].props.protocol_name))
== 0) { return i;
res = i;
break;
}
} }
return res; return -ENOENT;
} }
EXPORT_SYMBOL(ishtp_fw_cl_by_uuid); EXPORT_SYMBOL(ishtp_fw_cl_by_uuid);
...@@ -158,7 +155,7 @@ EXPORT_SYMBOL(ishtp_fw_cl_by_uuid); ...@@ -158,7 +155,7 @@ EXPORT_SYMBOL(ishtp_fw_cl_by_uuid);
* Return: pointer of client information on success, NULL on failure. * Return: pointer of client information on success, NULL on failure.
*/ */
struct ishtp_fw_client *ishtp_fw_cl_get_client(struct ishtp_device *dev, struct ishtp_fw_client *ishtp_fw_cl_get_client(struct ishtp_device *dev,
const uuid_le *uuid) const guid_t *uuid)
{ {
int i; int i;
unsigned long flags; unsigned long flags;
...@@ -401,7 +398,7 @@ static const struct device_type ishtp_cl_device_type = { ...@@ -401,7 +398,7 @@ static const struct device_type ishtp_cl_device_type = {
* Return: ishtp_cl_device pointer or NULL on failure * Return: ishtp_cl_device pointer or NULL on failure
*/ */
static struct ishtp_cl_device *ishtp_bus_add_device(struct ishtp_device *dev, static struct ishtp_cl_device *ishtp_bus_add_device(struct ishtp_device *dev,
uuid_le uuid, char *name) guid_t uuid, char *name)
{ {
struct ishtp_cl_device *device; struct ishtp_cl_device *device;
int status; int status;
...@@ -629,7 +626,7 @@ int ishtp_bus_new_client(struct ishtp_device *dev) ...@@ -629,7 +626,7 @@ int ishtp_bus_new_client(struct ishtp_device *dev)
int i; int i;
char *dev_name; char *dev_name;
struct ishtp_cl_device *cl_device; struct ishtp_cl_device *cl_device;
uuid_le device_uuid; guid_t device_uuid;
/* /*
* For all reported clients, create an unconnected client and add its * For all reported clients, create an unconnected client and add its
...@@ -639,7 +636,7 @@ int ishtp_bus_new_client(struct ishtp_device *dev) ...@@ -639,7 +636,7 @@ int ishtp_bus_new_client(struct ishtp_device *dev)
*/ */
i = dev->fw_client_presentation_num - 1; i = dev->fw_client_presentation_num - 1;
device_uuid = dev->fw_clients[i].props.protocol_name; device_uuid = dev->fw_clients[i].props.protocol_name;
dev_name = kasprintf(GFP_KERNEL, "{%pUL}", device_uuid.b); dev_name = kasprintf(GFP_KERNEL, "{%pUL}", &device_uuid);
if (!dev_name) if (!dev_name)
return -ENOMEM; return -ENOMEM;
......
...@@ -112,8 +112,8 @@ void ishtp_cl_driver_unregister(struct ishtp_cl_driver *driver); ...@@ -112,8 +112,8 @@ void ishtp_cl_driver_unregister(struct ishtp_cl_driver *driver);
int ishtp_register_event_cb(struct ishtp_cl_device *device, int ishtp_register_event_cb(struct ishtp_cl_device *device,
void (*read_cb)(struct ishtp_cl_device *)); void (*read_cb)(struct ishtp_cl_device *));
int ishtp_fw_cl_by_uuid(struct ishtp_device *dev, const uuid_le *cuuid); int ishtp_fw_cl_by_uuid(struct ishtp_device *dev, const guid_t *cuuid);
struct ishtp_fw_client *ishtp_fw_cl_get_client(struct ishtp_device *dev, struct ishtp_fw_client *ishtp_fw_cl_get_client(struct ishtp_device *dev,
const uuid_le *uuid); const guid_t *uuid);
#endif /* _LINUX_ISHTP_CL_BUS_H */ #endif /* _LINUX_ISHTP_CL_BUS_H */
...@@ -126,7 +126,7 @@ struct ishtp_cl { ...@@ -126,7 +126,7 @@ struct ishtp_cl {
}; };
/* Client connection managenment internal functions */ /* Client connection managenment internal functions */
int ishtp_can_client_connect(struct ishtp_device *ishtp_dev, uuid_le *uuid); int ishtp_can_client_connect(struct ishtp_device *ishtp_dev, guid_t *uuid);
int ishtp_fw_cl_by_id(struct ishtp_device *dev, uint8_t client_id); int ishtp_fw_cl_by_id(struct ishtp_device *dev, uint8_t client_id);
void ishtp_cl_send_msg(struct ishtp_device *dev, struct ishtp_cl *cl); void ishtp_cl_send_msg(struct ishtp_device *dev, struct ishtp_cl *cl);
void recv_ishtp_cl_msg(struct ishtp_device *dev, void recv_ishtp_cl_msg(struct ishtp_device *dev,
......
...@@ -149,7 +149,7 @@ struct hbm_host_enum_response { ...@@ -149,7 +149,7 @@ struct hbm_host_enum_response {
} __packed; } __packed;
struct ishtp_client_properties { struct ishtp_client_properties {
uuid_le protocol_name; guid_t protocol_name;
uint8_t protocol_version; uint8_t protocol_version;
uint8_t max_number_of_connections; uint8_t max_number_of_connections;
uint8_t fixed_address; uint8_t fixed_address;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment