Re: [PATCH v1 3/6] staging: unisys: Switch to use new generic UUID API

From: Andy Shevchenko
Date: Wed Jul 26 2017 - 06:01:56 EST

Next message: Andy Shevchenko: "Re: [PATCH v1 6/6] device property: Switch to use new generic UUID API"
Previous message: Marc Zyngier: "Re: [PATCH v2] irqchip/gic-v3-its: Allow GIC ITS number more than MAX_NUMNODES"
Next in thread: Greg Kroah-Hartman: "Re: [PATCH v1 3/6] staging: unisys: Switch to use new generic UUID API"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

On Wed, 2017-07-19 at 21:28 +0300, Andy Shevchenko wrote:
> 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.
>
> While here, re-indent couple of lines to increase readability.

This looks like no user space UUID API is involved, can be routed via
either tree (uuid or staging).

Anyone to comment?

>
> Cc: David Kershner <david.kershner@xxxxxxxxxx>
> Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
> Cc: sparmaintainer@xxxxxxxxxx
> Cc: devel@xxxxxxxxxxxxxxxxxxxx
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> ---
> Âdrivers/staging/unisys/Documentation/overview.txtÂÂ| 14 +++----
> Âdrivers/staging/unisys/include/channel.hÂÂÂÂÂÂÂÂÂÂÂ| 45 ++++++++++---
> -------
> Âdrivers/staging/unisys/include/iochannel.hÂÂÂÂÂÂÂÂÂ| 14 +------
> Âdrivers/staging/unisys/include/visorbus.hÂÂÂÂÂÂÂÂÂÂ| 12 +++---
> Âdrivers/staging/unisys/visorbus/controlvmchannel.h | 24 ++++-------
> Âdrivers/staging/unisys/visorbus/vbuschannel.hÂÂÂÂÂÂ|ÂÂ7 ++--
> Âdrivers/staging/unisys/visorbus/visorbus_main.cÂÂÂÂ| 28 ++++++-------
> Âdrivers/staging/unisys/visorbus/visorbus_private.h |ÂÂ6 +--
> Âdrivers/staging/unisys/visorbus/visorchannel.cÂÂÂÂÂ| 48 +++++++++++
> -----------
> Âdrivers/staging/unisys/visorbus/visorchipset.cÂÂÂÂÂ| 39 ++++++++++---
> -----
> Âdrivers/staging/unisys/visorhba/visorhba_main.cÂÂÂÂ|ÂÂ6 +--
> Âdrivers/staging/unisys/visorinput/visorinput.cÂÂÂÂÂ| 38 ++++++++-----
> ----
> Âdrivers/staging/unisys/visornic/visornic_main.cÂÂÂÂ|ÂÂ6 +--
> Â13 files changed, 135 insertions(+), 152 deletions(-)
>
> diff --git a/drivers/staging/unisys/Documentation/overview.txt
> b/drivers/staging/unisys/Documentation/overview.txt
> index e0466bfada2f..9ab30af265a5 100644
> --- a/drivers/staging/unisys/Documentation/overview.txt
> +++ b/drivers/staging/unisys/Documentation/overview.txt
> @@ -221,7 +221,7 @@ The following files exist under
> /sys/devices/visorbus<x>/vbus<x>:dev<y>:
> Â
> ÂThe visorhba driver registers with visorbus as the function driver to
> Âhandle virtual scsi disk devices, specified using the
> -VISOR_VHBA_CHANNEL_UUID type in the visorbus_register_visor_driver()
> +VISOR_VHBA_CHANNEL_GUID type in the visorbus_register_visor_driver()
> Âcall. visorhba uses scsi_add_host() to expose a Linux block device
> Â(e.g., /sys/block/) in the guest environment for each s-Par virtual
> device.
> Â
> @@ -240,7 +240,7 @@ When compiled as a module, visorhba can be
> autoloaded by visorbus in
> Âstandard udev/systemd environments, as it includes the modules.alias
> Âdefinition:
> Â
> -ÂÂÂÂ"visorbus:"+VISOR_VHBA_CHANNEL_UUID_STR
> +ÂÂÂÂ"visorbus:"+VISOR_VHBA_CHANNEL_GUID_STR
> Â
> Âi.e.:
> Â
> @@ -252,7 +252,7 @@ i.e.:
> Â
> ÂThe visornic driver registers with visorbus as the function driver to
> Âhandle virtual network devices, specified using the
> -VISOR_VNIC_CHANNEL_UUID type in the visorbus_register_visor_driver()
> +VISOR_VNIC_CHANNEL_GUID type in the visorbus_register_visor_driver()
> Âcall. visornic uses register_netdev() to expose a Linux device of
> class net
> Â(e.g., /sys/class/net/) in the guest environment for each s-Par
> virtual
> Âdevice.
> @@ -270,7 +270,7 @@ When compiled as a module, visornic can be
> autoloaded by visorbus in
> Âstandard udev/systemd environments, as it includes the modules.alias
> Âdefinition:
> Â
> -ÂÂÂÂ"visorbus:"+VISOR_VNIC_CHANNEL_UUID_STR
> +ÂÂÂÂ"visorbus:"+VISOR_VNIC_CHANNEL_GUID_STR
> Â
> Âi.e.:
> Â
> @@ -282,7 +282,7 @@ i.e.:
> Â
> ÂThe visorinput driver registers with visorbus as the function driver
> to
> Âhandle human input devices, specified using the
> -VISOR_KEYBOARD_CHANNEL_UUID and VISOR_MOUSE_CHANNEL_UUID
> +VISOR_KEYBOARD_CHANNEL_GUID and VISOR_MOUSE_CHANNEL_GUID
> Âtypes in the visorbus_register_visor_driver() call. visorinput uses
> Âinput_register_device() to expose devices of class input
> Â(e.g., /sys/class/input/) for virtual keyboard and virtual mouse
> devices.
> @@ -307,8 +307,8 @@ When compiled as a module, visorinput can be
> autoloaded by visorbus in
> Âstandard udev/systemd environments, as it includes the modules.alias
> Âdefinition:
> Â
> -ÂÂÂÂ"visorbus:"+VISOR_MOUSE_CHANNEL_UUID_STR
> -ÂÂÂÂ"visorbus:"+VISOR_KEYBOARD_CHANNEL_UUID_STR
> +ÂÂÂÂ"visorbus:"+VISOR_MOUSE_CHANNEL_GUID_STR
> +ÂÂÂÂ"visorbus:"+VISOR_KEYBOARD_CHANNEL_GUID_STR
> Â
> Âi.e.:
> Â
> diff --git a/drivers/staging/unisys/include/channel.h
> b/drivers/staging/unisys/include/channel.h
> index 692efcb38245..b09b1f55a6f1 100644
> --- a/drivers/staging/unisys/include/channel.h
> +++ b/drivers/staging/unisys/include/channel.h
> @@ -116,13 +116,13 @@ struct channel_header {
> Â u32 header_size; /* sizeof(struct channel_header) */
> Â u64 size; /* Total size of this channel in
> bytes */
> Â u64 features; /* Flags to modify behavior */
> - uuid_le chtype; /* Channel type: data, bus,
> control, etc. */
> + guid_t chtype; /* Channel type: data, bus,
> control, etc. */
> Â u64 partition_handle; /* ID of guest partition */
> Â u64 handle; /* Device number of this channel
> in client */
> Â u64 ch_space_offset; /* Offset in bytes to channel
> specific area */
> Â u32 version_id; /* struct channel_header
> Version ID */
> Â u32 partition_index; /* Index of guest partition */
> - uuid_le zone_uuid; /* Guid of Channel's zone */
> + guid_t zone_guid; /* Guid of Channel's zone */
> Â u32 cli_str_offset; /* offset from channel header to
> Â Â* nul-terminated ClientString (0 if
> Â Â* ClientString not present)
> @@ -205,25 +205,25 @@ struct signal_queue_header {
> Â */
> Âstatic inline int
> Âvisor_check_channel(struct channel_header *ch,
> - ÂÂÂÂuuid_le expected_uuid,
> + ÂÂÂÂconst guid_t *expected_guid,
> Â ÂÂÂÂchar *chname,
> Â ÂÂÂÂu64 expected_min_bytes,
> Â ÂÂÂÂu32 expected_version,
> Â ÂÂÂÂu64 expected_signature)
> Â{
> - if (uuid_le_cmp(expected_uuid, NULL_UUID_LE) != 0) {
> + if (!guid_is_null(expected_guid)) {
> Â /* caller wants us to verify type GUID */
> - if (uuid_le_cmp(ch->chtype, expected_uuid) != 0) {
> + if (!guid_equal(&ch->chtype, expected_guid)) {
> Â pr_err("Channel mismatch on channel=%s(%pUL)
> field=type expected=%pUL actual=%pUL\n",
> - ÂÂÂÂÂÂÂchname, &expected_uuid,
> - ÂÂÂÂÂÂÂ&expected_uuid, &ch->chtype);
> + ÂÂÂÂÂÂÂchname, expected_guid,
> + ÂÂÂÂÂÂÂexpected_guid, &ch->chtype);
> Â return 0;
> Â }
> Â }
> Â if (expected_min_bytes > 0) { /* verify channel size
> */
> Â if (ch->size < expected_min_bytes) {
> Â pr_err("Channel mismatch on channel=%s(%pUL)
> field=size expected=0x%-8.8Lx actual=0x%-8.8Lx\n",
> - ÂÂÂÂÂÂÂchname, &expected_uuid,
> + ÂÂÂÂÂÂÂchname, expected_guid,
> Â ÂÂÂÂÂÂÂ(unsigned long
> long)expected_min_bytes,
> Â ÂÂÂÂÂÂÂch->size);
> Â return 0;
> @@ -232,7 +232,7 @@ visor_check_channel(struct channel_header *ch,
> Â if (expected_version > 0) { /* verify channel version
> */
> Â if (ch->version_id != expected_version) {
> Â pr_err("Channel mismatch on channel=%s(%pUL)
> field=version expected=0x%-8.8lx actual=0x%-8.8x\n",
> - ÂÂÂÂÂÂÂchname, &expected_uuid,
> + ÂÂÂÂÂÂÂchname, expected_guid,
> Â ÂÂÂÂÂÂÂ(unsigned long)expected_version,
> Â ÂÂÂÂÂÂÂch->version_id);
> Â return 0;
> @@ -241,7 +241,7 @@ visor_check_channel(struct channel_header *ch,
> Â if (expected_signature > 0) { /* verify channel
> signature */
> Â if (ch->signature != expected_signature) {
> Â pr_err("Channel mismatch on channel=%s(%pUL)
> field=signature expected=0x%-8.8Lx actual=0x%-8.8Lx\n",
> - ÂÂÂÂÂÂÂchname, &expected_uuid,
> + ÂÂÂÂÂÂÂchname, expected_guid,
> Â ÂÂÂÂÂÂÂexpected_signature, ch->signature);
> Â return 0;
> Â }
> @@ -254,25 +254,22 @@ visor_check_channel(struct channel_header *ch,
> Â */
> Â
> Â/* {414815ed-c58c-11da-95a9-00e08161165f} */
> -#define VISOR_VHBA_CHANNEL_UUID \
> - UUID_LE(0x414815ed, 0xc58c, 0x11da, \
> - 0x95, 0xa9, 0x0, 0xe0, 0x81, 0x61, 0x16, 0x5f)
> -static const uuid_le visor_vhba_channel_uuid =
> VISOR_VHBA_CHANNEL_UUID;
> -#define VISOR_VHBA_CHANNEL_UUID_STR \
> +#define VISOR_VHBA_CHANNEL_GUID \
> + GUID_INIT(0x414815ed, 0xc58c, 0x11da, \
> + ÂÂ0x95, 0xa9, 0x0, 0xe0, 0x81, 0x61, 0x16, 0x5f)
> +#define VISOR_VHBA_CHANNEL_GUID_STR \
> Â "414815ed-c58c-11da-95a9-00e08161165f"
> Â
> Â/* {8cd5994d-c58e-11da-95a9-00e08161165f} */
> -#define VISOR_VNIC_CHANNEL_UUID \
> - UUID_LE(0x8cd5994d, 0xc58e, 0x11da, \
> - 0x95, 0xa9, 0x0, 0xe0, 0x81, 0x61, 0x16, 0x5f)
> -static const uuid_le visor_vnic_channel_uuid =
> VISOR_VNIC_CHANNEL_UUID;
> -#define VISOR_VNIC_CHANNEL_UUID_STR \
> +#define VISOR_VNIC_CHANNEL_GUID \
> + GUID_INIT(0x8cd5994d, 0xc58e, 0x11da, \
> + ÂÂ0x95, 0xa9, 0x0, 0xe0, 0x81, 0x61, 0x16, 0x5f)
> +#define VISOR_VNIC_CHANNEL_GUID_STR \
> Â "8cd5994d-c58e-11da-95a9-00e08161165f"
> Â
> Â/* {72120008-4AAB-11DC-8530-444553544200} */
> -#define VISOR_SIOVM_UUID \
> - UUID_LE(0x72120008, 0x4AAB, 0x11DC, \
> - 0x85, 0x30, 0x44, 0x45, 0x53, 0x54, 0x42, 0x00)
> -static const uuid_le visor_siovm_uuid = VISOR_SIOVM_UUID;
> +#define VISOR_SIOVM_GUID \
> + GUID_INIT(0x72120008, 0x4AAB, 0x11DC, \
> + ÂÂ0x85, 0x30, 0x44, 0x45, 0x53, 0x54, 0x42, 0x00)
> Â
> Â#endif
> diff --git a/drivers/staging/unisys/include/iochannel.h
> b/drivers/staging/unisys/include/iochannel.h
> index c7cb3fbde7b2..8f05bdd5197c 100644
> --- a/drivers/staging/unisys/include/iochannel.h
> +++ b/drivers/staging/unisys/include/iochannel.h
> @@ -49,18 +49,6 @@
> Â#define VISOR_VNIC_CHANNEL_VERSIONID 2
> Â#define VISOR_VSWITCH_CHANNEL_VERSIONID 1
> Â
> -#define VISOR_VHBA_CHANNEL_OK_CLIENT(ch) \
> - (visor_check_channel(ch, visor_vhba_channel_uuid, \
> - ÂÂÂÂÂ"vhba", MIN_IO_CHANNEL_SIZE, \
> - ÂÂÂÂÂVISOR_VHBA_CHANNEL_VERSIONID, \
> - ÂÂÂÂÂVISOR_VHBA_CHANNEL_SIGNATURE))
> -
> -#define VISOR_VNIC_CHANNEL_OK_CLIENT(ch) \
> - (visor_check_channel(ch, visor_vnic_channel_uuid, \
> - ÂÂÂÂÂ"vnic", MIN_IO_CHANNEL_SIZE, \
> - ÂÂÂÂÂVISOR_VNIC_CHANNEL_VERSIONID, \
> - ÂÂÂÂÂVISOR_VNIC_CHANNEL_SIGNATURE))
> -
> Â/*
> Â * Everything necessary to handle SCSI & NIC traffic between Guest
> Partition and
> Â * IO Partition is defined below.
> @@ -522,7 +510,7 @@ struct iochannel_vnic {
> Â u8 macaddr[6]; /* 6 bytes */
> Â u32 num_rcv_bufs; /* 4 bytes */
> Â u32 mtu; /* 4 bytes */
> - uuid_le zone_uuid; /* 16 bytes */
> + guid_t zone_guid; /* 16 bytes */
> Â} __packed;
> Â/*
> Â * This is just the header of the IO channel. It is assumed that
> directly after
> diff --git a/drivers/staging/unisys/include/visorbus.h
> b/drivers/staging/unisys/include/visorbus.h
> index de0635542fbd..38ce0878c55e 100644
> --- a/drivers/staging/unisys/include/visorbus.h
> +++ b/drivers/staging/unisys/include/visorbus.h
> @@ -58,7 +58,7 @@ struct visorchipset_state {
> Â *ÂÂGUID, name, and sizes.
> Â */
> Âstruct visor_channeltype_descriptor {
> - const uuid_le guid;
> + const guid_t guid;
> Â const char *name;
> Â};
> Â
> @@ -141,14 +141,14 @@ struct visor_driver {
> Â * hypervisor requests.
> Â * @vbus_hdr_info: A pointer to header info. Private
> use by bus
> Â * driver.
> - * @partition_uuid: Indicates client partion id. This
> should be the
> + * @partition_guid: Indicates client partion id. This
> should be the
> Â * same across all visor_devices in
> the current
> Â * guest. Private use by bus driver
> only.
> Â */
> Â
> Âstruct visor_device {
> Â struct visorchannel *visorchannel;
> - uuid_le channel_type_guid;
> + guid_t channel_type_guid;
> Â /* These fields are for private use by the bus driver only.
> */
> Â struct device device;
> Â struct list_head list_all;
> @@ -161,11 +161,11 @@ struct visor_device {
> Â u32 chipset_bus_no;
> Â u32 chipset_dev_no;
> Â struct visorchipset_state state;
> - uuid_le inst;
> + guid_t inst;
> Â u8 *name;
> Â struct controlvm_message_header *pending_msg_hdr;
> Â void *vbus_hdr_info;
> - uuid_le partition_uuid;
> + guid_t partition_guid;
> Â struct dentry *debugfs_dir;
> Â struct dentry *debugfs_client_bus_info;
> Â};
> @@ -207,7 +207,7 @@ int visorchannel_signalremove(struct visorchannel
> *channel, u32 queue,
> Âint visorchannel_signalinsert(struct visorchannel *channel, u32
> queue,
> Â ÂÂÂÂÂÂvoid *msg);
> Âbool visorchannel_signalempty(struct visorchannel *channel, u32
> queue);
> -uuid_le visorchannel_get_uuid(struct visorchannel *channel);
> +const guid_t *visorchannel_get_guid(struct visorchannel *channel);
> Â
> Â#define BUS_ROOT_DEVICE UINT_MAX
> Âstruct visor_device *visorbus_get_device_by_id(u32 bus_no, u32
> dev_no,
> diff --git a/drivers/staging/unisys/visorbus/controlvmchannel.h
> b/drivers/staging/unisys/visorbus/controlvmchannel.h
> index ed045eff0e33..d9d7628aff50 100644
> --- a/drivers/staging/unisys/visorbus/controlvmchannel.h
> +++ b/drivers/staging/unisys/visorbus/controlvmchannel.h
> @@ -19,9 +19,9 @@
> Â#include "channel.h"
> Â
> Â/* {2B3C2D10-7EF5-4ad8-B966-3448B7386B3D} */
> -#define VISOR_CONTROLVM_CHANNEL_UUID \
> - UUID_LE(0x2b3c2d10, 0x7ef5, 0x4ad8, \
> - 0xb9, 0x66, 0x34, 0x48, 0xb7, 0x38, 0x6b, 0x3d)
> +#define VISOR_CONTROLVM_CHANNEL_GUID \
> + GUID_INIT(0x2b3c2d10, 0x7ef5, 0x4ad8, \
> + ÂÂ0xb9, 0x66, 0x34, 0x48, 0xb7, 0x38, 0x6b, 0x3d)
> Â
> Â#define VISOR_CONTROLVM_CHANNEL_SIGNATURE VISOR_CHANNEL_SIGNATURE
> Â#define CONTROLVM_MESSAGE_MAX 64
> @@ -34,14 +34,6 @@
> Â */
> Â#define VISOR_CONTROLVM_CHANNEL_VERSIONID 1
> Â
> -#define VISOR_CONTROLVM_CHANNEL_OK_CLIENT(ch) \
> - (visor_check_channel(ch, \
> - ÂÂÂÂÂVISOR_CONTROLVM_CHANNEL_UUID, \
> - ÂÂÂÂÂ"controlvm", \
> - ÂÂÂÂÂsizeof(struct visor_controlvm_channel),
> \
> - ÂÂÂÂÂVISOR_CONTROLVM_CHANNEL_VERSIONID, \
> - ÂÂÂÂÂVISOR_CONTROLVM_CHANNEL_SIGNATURE))
> -
> Â/* Defines for various channel queues */
> Â#define CONTROLVM_QUEUE_REQUEST Â0
> Â#define CONTROLVM_QUEUE_RESPONSE 1
> @@ -222,8 +214,8 @@ struct controlvm_packet_device_createÂÂ{
> Â Â*/
> Â u64 channel_addr;
> Â u64 channel_bytes; /* specifies size of the channel in
> bytes */
> - uuid_le data_type_uuid; /* specifies format of data in
> channel */
> - uuid_le dev_inst_uuid; /* instance guid for the device
> */
> + guid_t data_type_guid; /* specifies format of data in
> channel */
> + guid_t dev_inst_guid; /* instance guid for the device
> */
> Â struct irq_info intr; /* specifies interrupt
> information */
> Â} __packed; /* for CONTROLVM_DEVICE_CREATE */
> Â
> @@ -258,8 +250,8 @@ struct controlvm_message_packetÂÂ{
> Â u64 channel_addr;
> Â u64 channel_bytes; /* size of the
> channel */
> Â /* indicates format of data in bus channel*/
> - uuid_le bus_data_type_uuid;
> - uuid_le bus_inst_uuid; /* instance
> uuid for the bus */
> + guid_t bus_data_type_guid;
> + guid_t bus_inst_guid; /* instance GUID
> for the bus */
> Â } __packed create_bus; /* for
> CONTROLVM_BUS_CREATE */
> Â structÂÂ{
> Â /* bus # (0..n-1) from the msg receiver's perspective */
> @@ -455,7 +447,7 @@ struct visor_controlvm_parameters_header {
> Â u32 client_length;
> Â u32 name_offset;
> Â u32 name_length;
> - uuid_le id;
> + guid_t id;
> Â u32 revision;
> Â u32 reserved; /* Natural alignment */
> Â} __packed;
> diff --git a/drivers/staging/unisys/visorbus/vbuschannel.h
> b/drivers/staging/unisys/visorbus/vbuschannel.h
> index 01d7d517dba7..8f256dbf7ad2 100644
> --- a/drivers/staging/unisys/visorbus/vbuschannel.h
> +++ b/drivers/staging/unisys/visorbus/vbuschannel.h
> @@ -27,10 +27,9 @@
> Â#include "channel.h"
> Â
> Â/* {193b331b-c58f-11da-95a9-00e08161165f} */
> -#define VISOR_VBUS_CHANNEL_UUID \
> - UUID_LE(0x193b331b, 0xc58f, 0x11da, \
> - 0x95, 0xa9, 0x0, 0xe0, 0x81, 0x61, 0x16, 0x5f)
> -static const uuid_le visor_vbus_channel_uuid =
> VISOR_VBUS_CHANNEL_UUID;
> +#define VISOR_VBUS_CHANNEL_GUID \
> + GUID_INIT(0x193b331b, 0xc58f, 0x11da, \
> + ÂÂ0x95, 0xa9, 0x0, 0xe0, 0x81, 0x61, 0x16, 0x5f)
> Â
> Â#define VISOR_VBUS_CHANNEL_SIGNATURE VISOR_CHANNEL_SIGNATURE
> Â
> diff --git a/drivers/staging/unisys/visorbus/visorbus_main.c
> b/drivers/staging/unisys/visorbus/visorbus_main.c
> index 1c785dd19ddd..dd932e39607b 100644
> --- a/drivers/staging/unisys/visorbus/visorbus_main.c
> +++ b/drivers/staging/unisys/visorbus/visorbus_main.c
> @@ -20,6 +20,8 @@
> Â#include "visorbus.h"
> Â#include "visorbus_private.h"
> Â
> +static const guid_t visor_vbus_channel_guid =
> VISOR_VBUS_CHANNEL_GUID;
> +
> Â#define MYDRVNAME "visorbus"
> Â
> Â/* Display string that is guaranteed to be no longer the 99
> characters*/
> @@ -40,11 +42,11 @@ static ssize_t modalias_show(struct device *dev,
> struct device_attribute *attr,
> Â ÂÂÂÂÂchar *buf)
> Â{
> Â struct visor_device *vdev;
> - uuid_le guid;
> + const guid_t *guid;
> Â
> Â vdev = to_visor_device(dev);
> - guid = visorchannel_get_uuid(vdev->visorchannel);
> - return sprintf(buf, "visorbus:%pUl\n", &guid);
> + guid = visorchannel_get_guid(vdev->visorchannel);
> + return sprintf(buf, "visorbus:%pUl\n", guid);
> Â}
> Âstatic DEVICE_ATTR_RO(modalias);
> Â
> @@ -77,12 +79,12 @@ static int
> Âvisorbus_uevent(struct device *xdev, struct kobj_uevent_env *env)
> Â{
> Â struct visor_device *dev;
> - uuid_le guid;
> + const guid_t *guid;
> Â
> Â dev = to_visor_device(xdev);
> - guid = visorchannel_get_uuid(dev->visorchannel);
> + guid = visorchannel_get_guid(dev->visorchannel);
> Â
> - return add_uevent_var(env, "MODALIAS=visorbus:%pUl", &guid);
> + return add_uevent_var(env, "MODALIAS=visorbus:%pUl", guid);
> Â}
> Â
> Â/*
> @@ -97,24 +99,22 @@ visorbus_uevent(struct device *xdev, struct
> kobj_uevent_env *env)
> Âstatic int
> Âvisorbus_match(struct device *xdev, struct device_driver *xdrv)
> Â{
> - uuid_le channel_type;
> + const guid_t *channel_type;
> Â int i;
> Â struct visor_device *dev;
> Â struct visor_driver *drv;
> Â
> Â dev = to_visor_device(xdev);
> Â drv = to_visor_driver(xdrv);
> - channel_type = visorchannel_get_uuid(dev->visorchannel);
> + channel_type = visorchannel_get_guid(dev->visorchannel);
> Â
> Â if (!drv->channel_types)
> Â return 0;
> Â
> Â for (i = 0;
> - ÂÂÂÂÂ(uuid_le_cmp(drv->channel_types[i].guid, NULL_UUID_LE)
> != 0) ||
> - ÂÂÂÂÂ(drv->channel_types[i].name);
> + ÂÂÂÂÂ!guid_is_null(&drv->channel_types[i].guid) || drv-
> >channel_types[i].name;
> Â ÂÂÂÂÂi++)
> - if (uuid_le_cmp(drv->channel_types[i].guid,
> - channel_type) == 0)
> + if (guid_equal(&drv->channel_types[i].guid,
> channel_type))
> Â return i + 1;
> Â
> Â return 0;
> @@ -283,7 +283,7 @@ static ssize_t partition_guid_show(struct device
> *dev,
> Â ÂÂÂchar *buf) {
> Â struct visor_device *vdev = to_visor_device(dev);
> Â
> - return sprintf(buf, "{%pUb}\n", &vdev->partition_uuid);
> + return sprintf(buf, "{%pUb}\n", &vdev->partition_guid);
> Â}
> Âstatic DEVICE_ATTR_RO(partition_guid);
> Â
> @@ -687,7 +687,7 @@ get_vbus_header_info(struct visorchannel *chan,
> Â int err;
> Â
> Â if (!visor_check_channel(visorchannel_get_header(chan),
> - Âvisor_vbus_channel_uuid,
> + Â&visor_vbus_channel_guid,
> Â Â"vbus",
> Â Âsizeof(struct visor_vbus_channel),
> Â ÂVISOR_VBUS_CHANNEL_VERSIONID,
> diff --git a/drivers/staging/unisys/visorbus/visorbus_private.h
> b/drivers/staging/unisys/visorbus/visorbus_private.h
> index 98a5af19189d..0856179338df 100644
> --- a/drivers/staging/unisys/visorbus/visorbus_private.h
> +++ b/drivers/staging/unisys/visorbus/visorbus_private.h
> @@ -48,10 +48,10 @@ void visorbus_exit(void);
> Â
> Âstruct visorchannel *visorchannel_create(u64 physaddr,
> Â Âunsigned long channel_bytes,
> - Âgfp_t gfp, uuid_le guid);
> + Âgfp_t gfp, const guid_t
> *guid);
> Âstruct visorchannel *visorchannel_create_with_lock(u64 physaddr,
> Â ÂÂÂunsigned long
> channel_bytes,
> - ÂÂÂgfp_t gfp, uuid_le
> guid);
> + ÂÂÂgfp_t gfp, const
> guid_t *guid);
> Âvoid visorchannel_destroy(struct visorchannel *channel);
> Âint visorchannel_read(struct visorchannel *channel, ulong offset,
> Â ÂÂÂÂÂÂvoid *dest, ulong nbytes);
> @@ -64,6 +64,6 @@ char *visorchannel_zoneid(struct visorchannel
> *channel, char *s);
> Âu64 visorchannel_get_clientpartition(struct visorchannel *channel);
> Âint visorchannel_set_clientpartition(struct visorchannel *channel,
> Â ÂÂÂÂÂu64 partition_handle);
> -char *visorchannel_uuid_id(uuid_le *guid, char *s);
> +char *visorchannel_guid_id(const guid_t *guid, char *s);
> Âvoid *visorchannel_get_header(struct visorchannel *channel);
> Â#endif
> diff --git a/drivers/staging/unisys/visorbus/visorchannel.c
> b/drivers/staging/unisys/visorbus/visorchannel.c
> index 6885c2cb7135..fd64be952628 100644
> --- a/drivers/staging/unisys/visorbus/visorchannel.c
> +++ b/drivers/staging/unisys/visorbus/visorchannel.c
> @@ -29,10 +29,10 @@
> Â#define MYDRVNAME "visorchannel"
> Â
> Â#define VISOR_CONSOLEVIDEO_CHANNEL_GUID \
> - UUID_LE(0x3cd6e705, 0xd6a2, 0x4aa5, \
> - 0xad, 0x5c, 0x7b, 0x8, 0x88, 0x9d, 0xff, 0xe2)
> + GUID_INIT(0x3cd6e705, 0xd6a2, 0x4aa5, \
> + ÂÂ0xad, 0x5c, 0x7b, 0x8, 0x88, 0x9d, 0xff, 0xe2)
> Â
> -static const uuid_le visor_video_guid =
> VISOR_CONSOLEVIDEO_CHANNEL_GUID;
> +static const guid_t visor_video_guid =
> VISOR_CONSOLEVIDEO_CHANNEL_GUID;
> Â
> Âstruct visorchannel {
> Â u64 physaddr;
> @@ -40,14 +40,14 @@ struct visorchannel {
> Â void *mapped;
> Â bool requested;
> Â struct channel_header chan_hdr;
> - uuid_le guid;
> + guid_t guid;
> Â bool needs_lock; /* channel creator knows if more than
> one */
> Â /* thread will be inserting or
> removing */
> Â spinlock_t insert_lock; /* protect head writes in chan_hdr */
> Â spinlock_t remove_lock; /* protect tail writes in
> chan_hdr */
> Â
> - uuid_le type;
> - uuid_le inst;
> + guid_t type;
> + guid_t inst;
> Â};
> Â
> Âvoid
> @@ -76,7 +76,7 @@ visorchannel_get_nbytes(struct visorchannel
> *channel)
> Â}
> Â
> Âchar *
> -visorchannel_uuid_id(uuid_le *guid, char *s)
> +visorchannel_guid_id(const guid_t *guid, char *s)
> Â{
> Â sprintf(s, "%pUL", guid);
> Â return s;
> @@ -85,13 +85,13 @@ visorchannel_uuid_id(uuid_le *guid, char *s)
> Âchar *
> Âvisorchannel_id(struct visorchannel *channel, char *s)
> Â{
> - return visorchannel_uuid_id(&channel->guid, s);
> + return visorchannel_guid_id(&channel->guid, s);
> Â}
> Â
> Âchar *
> Âvisorchannel_zoneid(struct visorchannel *channel, char *s)
> Â{
> - return visorchannel_uuid_id(&channel->chan_hdr.zone_uuid, s);
> + return visorchannel_guid_id(&channel->chan_hdr.zone_guid, s);
> Â}
> Â
> Âu64
> @@ -109,17 +109,17 @@ visorchannel_set_clientpartition(struct
> visorchannel *channel,
> Â}
> Â
> Â/**
> - * visorchannel_get_uuid() - queries the UUID of the designated
> channel
> + * visorchannel_get_guid() - queries the GUID of the designated
> channel
> Â * @channel: the channel to query
> Â *
> - * Return: the UUID of the provided channel
> + * Return: the GUID of the provided channel
> Â */
> -uuid_le
> -visorchannel_get_uuid(struct visorchannel *channel)
> +const guid_t *
> +visorchannel_get_guid(struct visorchannel *channel)
> Â{
> - return channel->guid;
> + return &channel->guid;
> Â}
> -EXPORT_SYMBOL_GPL(visorchannel_get_uuid);
> +EXPORT_SYMBOL_GPL(visorchannel_get_guid);
> Â
> Âint
> Âvisorchannel_read(struct visorchannel *channel, ulong offset,
> @@ -376,7 +376,7 @@ signalinsert_inner(struct visorchannel *channel,
> u32 queue, void *msg)
> Â *ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂback-end), in which case the actual channel size
> will be
> Â *ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂread from the channel header in memory
> Â * @gfp:ÂÂÂÂÂÂÂÂÂÂÂgfp_t to use when allocating memory for the data
> struct
> - * @guid:ÂÂÂÂÂÂÂÂÂÂuuid that identifies channel type; this may 0 if
> the channel
> + * @guid:ÂÂÂÂÂÂÂÂÂÂGUID that identifies channel type; this may 0 if
> the channel
> Â *ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂhas already been initialized in memory (which is
> true for all
> Â *ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂchannels provided to guest environments by the s-
> Par
> Â *ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂback-end), in which case the actual channel guid
> will be
> @@ -390,7 +390,7 @@ signalinsert_inner(struct visorchannel *channel,
> u32 queue, void *msg)
> Â */
> Âstatic struct visorchannel *
> Âvisorchannel_create_guts(u64 physaddr, unsigned long channel_bytes,
> - Âgfp_t gfp, uuid_le guid, bool needs_lock)
> + Âgfp_t gfp, const guid_t *guid, bool
> needs_lock)
> Â{
> Â struct visorchannel *channel;
> Â int err;
> @@ -415,7 +415,7 @@ visorchannel_create_guts(u64 physaddr, unsigned
> long channel_bytes,
> Â Â* release later on.
> Â Â*/
> Â channel->requested = request_mem_region(physaddr, size,
> MYDRVNAME);
> - if (!channel->requested && uuid_le_cmp(guid,
> visor_video_guid))
> + if (!channel->requested && !guid_equal(guid,
> &visor_video_guid))
> Â /* we only care about errors if this is not the video
> channel */
> Â goto err_destroy_channel;
> Â
> @@ -436,8 +436,8 @@ visorchannel_create_guts(u64 physaddr, unsigned
> long channel_bytes,
> Â /* we had better be a CLIENT of this channel */
> Â if (channel_bytes == 0)
> Â channel_bytes = (ulong)channel->chan_hdr.size;
> - if (uuid_le_cmp(guid, NULL_UUID_LE) == 0)
> - guid = channel->chan_hdr.chtype;
> + if (guid_is_null(guid))
> + guid = &channel->chan_hdr.chtype;
> Â
> Â memunmap(channel->mapped);
> Â if (channel->requested)
> @@ -445,7 +445,7 @@ visorchannel_create_guts(u64 physaddr, unsigned
> long channel_bytes,
> Â channel->mapped = NULL;
> Â channel->requested = request_mem_region(channel->physaddr,
> Â channel_bytes,
> MYDRVNAME);
> - if (!channel->requested && uuid_le_cmp(guid,
> visor_video_guid))
> + if (!channel->requested && !guid_equal(guid,
> &visor_video_guid))
> Â /* we only care about errors if this is not the video
> channel */
> Â goto err_destroy_channel;
> Â
> @@ -457,7 +457,7 @@ visorchannel_create_guts(u64 physaddr, unsigned
> long channel_bytes,
> Â }
> Â
> Â channel->nbytes = channel_bytes;
> - channel->guid = guid;
> + guid_copy(&channel->guid, guid);
> Â return channel;
> Â
> Âerr_destroy_channel:
> @@ -467,7 +467,7 @@ visorchannel_create_guts(u64 physaddr, unsigned
> long channel_bytes,
> Â
> Âstruct visorchannel *
> Âvisorchannel_create(u64 physaddr, unsigned long channel_bytes,
> - ÂÂÂÂgfp_t gfp, uuid_le guid)
> + ÂÂÂÂgfp_t gfp, const guid_t *guid)
> Â{
> Â return visorchannel_create_guts(physaddr, channel_bytes, gfp,
> guid,
> Â false);
> @@ -475,7 +475,7 @@ visorchannel_create(u64 physaddr, unsigned long
> channel_bytes,
> Â
> Âstruct visorchannel *
> Âvisorchannel_create_with_lock(u64 physaddr, unsigned long
> channel_bytes,
> - ÂÂÂÂÂÂgfp_t gfp, uuid_le guid)
> + ÂÂÂÂÂÂgfp_t gfp, const guid_t *guid)
> Â{
> Â return visorchannel_create_guts(physaddr, channel_bytes, gfp,
> guid,
> Â true);
> diff --git a/drivers/staging/unisys/visorbus/visorchipset.c
> b/drivers/staging/unisys/visorbus/visorchipset.c
> index 22150564b4fb..5e2bb9b8fa1b 100644
> --- a/drivers/staging/unisys/visorbus/visorchipset.c
> +++ b/drivers/staging/unisys/visorbus/visorchipset.c
> @@ -27,6 +27,10 @@
> Â#include "visorbus_private.h"
> Â#include "vmcallinterface.h"
> Â
> +static const guid_t visor_vhba_channel_guid =
> VISOR_VHBA_CHANNEL_GUID;
> +static const guid_t visor_siovm_guid = VISOR_SIOVM_GUID;
> +static const guid_t visor_controlvm_channel_guid =
> VISOR_CONTROLVM_CHANNEL_GUID;
> +
> Â#define CURRENT_FILE_PC VISOR_BUS_PC_visorchipset_c
> Â
> Â#define POLLJIFFIES_CONTROLVMCHANNEL_FAST 1
> @@ -277,13 +281,12 @@ static ssize_t remaining_steps_store(struct
> device *dev,
> Â}
> Âstatic DEVICE_ATTR_RW(remaining_steps);
> Â
> -static uuid_le
> -parser_id_get(struct parser_context *ctx)
> +static const guid_t *parser_id_get(struct parser_context *ctx)
> Â{
> Â struct visor_controlvm_parameters_header *phdr = NULL;
> Â
> Â phdr = (struct visor_controlvm_parameters_header *)(ctx-
> >data);
> - return phdr->id;
> + return &phdr->id;
> Â}
> Â
> Âstatic void parser_done(struct parser_context *ctx)
> @@ -595,7 +598,7 @@ visorbus_create(struct controlvm_message *inmsg)
> Â bus_info->chipset_bus_no = bus_no;
> Â bus_info->chipset_dev_no = BUS_ROOT_DEVICE;
> Â
> - if (uuid_le_cmp(cmd->create_bus.bus_inst_uuid,
> visor_siovm_uuid) == 0) {
> + if (guid_equal(&cmd->create_bus.bus_inst_guid,
> &visor_siovm_guid)) {
> Â err = save_crash_message(inmsg, CRASH_BUS);
> Â if (err)
> Â goto err_free_bus_info;
> @@ -617,7 +620,7 @@ visorbus_create(struct controlvm_message *inmsg)
> Â visorchannel = visorchannel_create(cmd-
> >create_bus.channel_addr,
> Â ÂÂÂcmd-
> >create_bus.channel_bytes,
> Â ÂÂÂGFP_KERNEL,
> - ÂÂÂcmd-
> >create_bus.bus_data_type_uuid);
> + ÂÂÂ&cmd-
> >create_bus.bus_data_type_guid);
> Â
> Â if (!visorchannel) {
> Â err = -ENOMEM;
> @@ -722,7 +725,9 @@ visorbus_configure(struct controlvm_message
> *inmsg,
> Â goto err_respond;
> Â
> Â if (parser_ctx) {
> - bus_info->partition_uuid = parser_id_get(parser_ctx);
> + const guid_t *partition_guid =
> parser_id_get(parser_ctx);
> +
> + guid_copy(&bus_info->partition_guid, partition_guid);
> Â bus_info->name = parser_name_get(parser_ctx);
> Â }
> Â
> @@ -781,7 +786,7 @@ visorbus_device_create(struct controlvm_message
> *inmsg)
> Â
> Â dev_info->chipset_bus_no = bus_no;
> Â dev_info->chipset_dev_no = dev_no;
> - dev_info->inst = cmd->create_device.dev_inst_uuid;
> + guid_copy(&dev_info->inst, &cmd-
> >create_device.dev_inst_guid);
> Â
> Â /* not sure where the best place to set the 'parent' */
> Â dev_info->device.parent = &bus_info->device;
> @@ -790,7 +795,7 @@ visorbus_device_create(struct controlvm_message
> *inmsg)
> Â ÂÂÂÂÂÂÂvisorchannel_create_with_lock(cmd-
> >create_device.channel_addr,
> Â ÂÂÂÂÂcmd-
> >create_device.channel_bytes,
> Â ÂÂÂÂÂGFP_KERNEL,
> - ÂÂÂÂÂcmd-
> >create_device.data_type_uuid);
> + ÂÂÂÂÂ&cmd-
> >create_device.data_type_guid);
> Â
> Â if (!visorchannel) {
> Â dev_err(&chipset_dev->acpi_device->dev,
> @@ -800,9 +805,8 @@ visorbus_device_create(struct controlvm_message
> *inmsg)
> Â goto err_free_dev_info;
> Â }
> Â dev_info->visorchannel = visorchannel;
> - dev_info->channel_type_guid = cmd-
> >create_device.data_type_uuid;
> - if (uuid_le_cmp(cmd->create_device.data_type_uuid,
> - visor_vhba_channel_uuid) == 0) {
> + guid_copy(&dev_info->channel_type_guid, &cmd-
> >create_device.data_type_guid);
> + if (guid_equal(&cmd->create_device.data_type_guid,
> &visor_vhba_channel_guid)) {
> Â err = save_crash_message(inmsg, CRASH_DEV);
> Â if (err)
> Â goto err_destroy_visorchannel;
> @@ -1817,7 +1821,6 @@ visorchipset_init(struct acpi_device
> *acpi_device)
> Â{
> Â int err = -ENODEV;
> Â u64 addr;
> - uuid_le uuid = VISOR_CONTROLVM_CHANNEL_UUID;
> Â struct visorchannel *controlvm_channel;
> Â
> Â chipset_dev = kzalloc(sizeof(*chipset_dev), GFP_KERNEL);
> @@ -1832,9 +1835,9 @@ visorchipset_init(struct acpi_device
> *acpi_device)
> Â
> Â chipset_dev->acpi_device = acpi_device;
> Â chipset_dev->poll_jiffies =
> POLLJIFFIES_CONTROLVMCHANNEL_FAST;
> - controlvm_channel = visorchannel_create_with_lock(addr,
> - ÂÂ0,
> GFP_KERNEL, uuid);
> Â
> + controlvm_channel = visorchannel_create_with_lock(addr, 0,
> GFP_KERNEL,
> + &visor_controlvm_chan
> nel_guid);
> Â if (!controlvm_channel)
> Â goto error_free_chipset_dev;
> Â
> @@ -1845,8 +1848,12 @@ visorchipset_init(struct acpi_device
> *acpi_device)
> Â if (err < 0)
> Â goto error_destroy_channel;
> Â
> - if (!VISOR_CONTROLVM_CHANNEL_OK_CLIENT(
> - visorchannel_get_header(controlvm_cha
> nnel)))
> + if
> (!visor_check_channel(visorchannel_get_header(controlvm_channel),
> + Â&visor_controlvm_channel_guid,
> + Â"controlvm",
> + Âsizeof(struct
> visor_controlvm_channel),
> + ÂVISOR_CONTROLVM_CHANNEL_VERSIONID,
> + ÂVISOR_CONTROLVM_CHANNEL_SIGNATURE))
> Â goto error_delete_groups;
> Â
> Â /* if booting in a crash kernel */
> diff --git a/drivers/staging/unisys/visorhba/visorhba_main.c
> b/drivers/staging/unisys/visorhba/visorhba_main.c
> index a6e7a6bbc428..d0e84ca08a5a 100644
> --- a/drivers/staging/unisys/visorhba/visorhba_main.c
> +++ b/drivers/staging/unisys/visorhba/visorhba_main.c
> @@ -39,12 +39,12 @@ static struct visor_channeltype_descriptor
> visorhba_channel_types[] = {
> Â /* Note that the only channel type we expect to be reported
> by the
> Â Â* bus driver is the VISOR_VHBA channel.
> Â Â*/
> - { VISOR_VHBA_CHANNEL_UUID, "sparvhba" },
> - { NULL_UUID_LE, NULL }
> + { VISOR_VHBA_CHANNEL_GUID, "sparvhba" },
> + {}
> Â};
> Â
> ÂMODULE_DEVICE_TABLE(visorbus, visorhba_channel_types);
> -MODULE_ALIAS("visorbus:" VISOR_VHBA_CHANNEL_UUID_STR);
> +MODULE_ALIAS("visorbus:" VISOR_VHBA_CHANNEL_GUID_STR);
> Â
> Âstruct visordisk_info {
> Â u32 valid;
> diff --git a/drivers/staging/unisys/visorinput/visorinput.c
> b/drivers/staging/unisys/visorinput/visorinput.c
> index 45bc340d4e9d..fbf4957ffa15 100644
> --- a/drivers/staging/unisys/visorinput/visorinput.c
> +++ b/drivers/staging/unisys/visorinput/visorinput.c
> @@ -33,16 +33,16 @@
> Â#include "ultrainputreport.h"
> Â
> Â/* Keyboard channel {c73416d0-b0b8-44af-b304-9d2ae99f1b3d} */
> -#define VISOR_KEYBOARD_CHANNEL_UUID \
> - UUID_LE(0xc73416d0, 0xb0b8, 0x44af, \
> - 0xb3, 0x4, 0x9d, 0x2a, 0xe9, 0x9f, 0x1b, 0x3d)
> -#define VISOR_KEYBOARD_CHANNEL_UUID_STR "c73416d0-b0b8-44af-b304-
> 9d2ae99f1b3d"
> +#define VISOR_KEYBOARD_CHANNEL_GUID \
> + GUID_INIT(0xc73416d0, 0xb0b8, 0x44af, \
> + ÂÂ0xb3, 0x4, 0x9d, 0x2a, 0xe9, 0x9f, 0x1b, 0x3d)
> +#define VISOR_KEYBOARD_CHANNEL_GUID_STR "c73416d0-b0b8-44af-b304-
> 9d2ae99f1b3d"
> Â
> Â/* Mouse channel {addf07d4-94a9-46e2-81c3-61abcdbdbd87} */
> -#define VISOR_MOUSE_CHANNEL_UUID \
> - UUID_LE(0xaddf07d4, 0x94a9, 0x46e2, \
> - 0x81, 0xc3, 0x61, 0xab, 0xcd, 0xbd, 0xbd, 0x87)
> -#define VISOR_MOUSE_CHANNEL_UUID_STR "addf07d4-94a9-46e2-81c3-
> 61abcdbdbd87"
> +#define VISOR_MOUSE_CHANNEL_GUID \
> + GUID_INIT(0xaddf07d4, 0x94a9, 0x46e2, \
> + ÂÂ0x81, 0xc3, 0x61, 0xab, 0xcd, 0xbd, 0xbd, 0x87)
> +#define VISOR_MOUSE_CHANNEL_GUID_STR "addf07d4-94a9-46e2-81c3-
> 61abcdbdbd87"
> Â
> Â#define PIXELS_ACROSS_DEFAULT 800
> Â#define PIXELS_DOWN_DEFAULTÂÂÂ600
> @@ -69,8 +69,8 @@ struct visorinput_devdata {
> Â unsigned char keycode_table[0];
> Â};
> Â
> -static const uuid_le visor_keyboard_channel_uuid =
> VISOR_KEYBOARD_CHANNEL_UUID;
> -static const uuid_le visor_mouse_channel_uuid =
> VISOR_MOUSE_CHANNEL_UUID;
> +static const guid_t visor_keyboard_channel_guid =
> VISOR_KEYBOARD_CHANNEL_GUID;
> +static const guid_t visor_mouse_channel_guid =
> VISOR_MOUSE_CHANNEL_GUID;
> Â
> Â/*
> Â * Borrowed from drivers/input/keyboard/atakbd.c
> @@ -449,13 +449,13 @@ devdata_create(struct visor_device *dev, enum
> visorinput_device_type devtype)
> Âstatic int
> Âvisorinput_probe(struct visor_device *dev)
> Â{
> - uuid_le guid;
> + const guid_t *guid;
> Â enum visorinput_device_type devtype;
> Â
> - guid = visorchannel_get_uuid(dev->visorchannel);
> - if (uuid_le_cmp(guid, visor_mouse_channel_uuid) == 0)
> + guid = visorchannel_get_guid(dev->visorchannel);
> + if (guid_equal(guid, &visor_mouse_channel_guid))
> Â devtype = visorinput_mouse;
> - else if (uuid_le_cmp(guid, visor_keyboard_channel_uuid) == 0)
> + else if (guid_equal(guid, &visor_keyboard_channel_guid))
> Â devtype = visorinput_keyboard;
> Â else
> Â return -ENODEV;
> @@ -727,9 +727,9 @@ visorinput_resume(struct visor_device *dev,
> Â
> Â/* GUIDS for all channel types supported by this driver. */
> Âstatic struct visor_channeltype_descriptor visorinput_channel_types[]
> = {
> - { VISOR_KEYBOARD_CHANNEL_UUID, "keyboard"},
> - { VISOR_MOUSE_CHANNEL_UUID, "mouse"},
> - { NULL_UUID_LE, NULL }
> + { VISOR_KEYBOARD_CHANNEL_GUID, "keyboard"},
> + { VISOR_MOUSE_CHANNEL_GUID, "mouse"},
> + {}
> Â};
> Â
> Âstatic struct visor_driver visorinput_driver = {
> @@ -764,5 +764,5 @@ MODULE_AUTHOR("Unisys");
> ÂMODULE_LICENSE("GPL");
> ÂMODULE_DESCRIPTION("s-Par human input driver for virtual
> keyboard/mouse");
> Â
> -MODULE_ALIAS("visorbus:" VISOR_MOUSE_CHANNEL_UUID_STR);
> -MODULE_ALIAS("visorbus:" VISOR_KEYBOARD_CHANNEL_UUID_STR);
> +MODULE_ALIAS("visorbus:" VISOR_MOUSE_CHANNEL_GUID_STR);
> +MODULE_ALIAS("visorbus:" VISOR_KEYBOARD_CHANNEL_GUID_STR);
> diff --git a/drivers/staging/unisys/visornic/visornic_main.c
> b/drivers/staging/unisys/visornic/visornic_main.c
> index 2891622eef18..10c1d036f996 100644
> --- a/drivers/staging/unisys/visornic/visornic_main.c
> +++ b/drivers/staging/unisys/visornic/visornic_main.c
> @@ -41,8 +41,8 @@ static struct visor_channeltype_descriptor
> visornic_channel_types[] = {
> Â /* Note that the only channel type we expect to be reported
> by the
> Â Â* bus driver is the VISOR_VNIC channel.
> Â Â*/
> - { VISOR_VNIC_CHANNEL_UUID, "ultravnic" },
> - { NULL_UUID_LE, NULL }
> + { VISOR_VNIC_CHANNEL_GUID, "ultravnic" },
> + {}
> Â};
> ÂMODULE_DEVICE_TABLE(visorbus, visornic_channel_types);
> Â/*
> @@ -52,7 +52,7 @@ MODULE_DEVICE_TABLE(visorbus,
> visornic_channel_types);
> Â * must be added to scripts/mode/file2alias.c, etc., to get this
> working
> Â * properly.
> Â */
> -MODULE_ALIAS("visorbus:" VISOR_VNIC_CHANNEL_UUID_STR);
> +MODULE_ALIAS("visorbus:" VISOR_VNIC_CHANNEL_GUID_STR);
> Â
> Âstruct chanstat {
> Â unsigned long got_rcv;

--
Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
Intel Finland Oy

Next message: Andy Shevchenko: "Re: [PATCH v1 6/6] device property: Switch to use new generic UUID API"
Previous message: Marc Zyngier: "Re: [PATCH v2] irqchip/gic-v3-its: Allow GIC ITS number more than MAX_NUMNODES"
Next in thread: Greg Kroah-Hartman: "Re: [PATCH v1 3/6] staging: unisys: Switch to use new generic UUID API"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]