lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <2a1d1e8f3c580b6b12eb5e5e568320a9.squirrel@www.codeaurora.org>
Date:	Mon, 23 Feb 2015 09:16:06 -0000
From:	"Dov Levenglick" <dovl@...eaurora.org>
To:	"Gilad Broner" <gbroner@...eaurora.org>
Cc:	james.bottomley@...senpartnership.com,
	linux-kernel@...r.kernel.org, linux-scsi@...r.kernel.org,
	linux-arm-msm@...r.kernel.org, santoshsy@...il.com,
	linux-scsi-owner@...r.kernel.org, subhashj@...eaurora.org,
	ygardi@...eaurora.org, draviv@...eaurora.org,
	"Noa Rubens" <noag@...eaurora.org>,
	"Raviv Shvili" <rshvili@...eaurora.org>,
	"Vinayak Holikatti" <vinholikatti@...il.com>,
	"James E.J. Bottomley" <jbottomley@...allels.com>,
	"open list:ABI/API" <linux-api@...r.kernel.org>
Subject: Re: [PATCH v3 1/4] scsi: ufs: add ioctl interface for query request

> From: Dolev Raviv <draviv@...eaurora.org>
>
> This patch exposes the ioctl interface for UFS driver via SCSI device
> ioctl interface. As of now UFS driver would provide the ioctl for query
> interface to connected UFS device.
>
> Signed-off-by: Dolev Raviv <draviv@...eaurora.org>
> Signed-off-by: Noa Rubens <noag@...eaurora.org>
> Signed-off-by: Raviv Shvili <rshvili@...eaurora.org>
> Signed-off-by: Yaniv Gardi <ygardi@...eaurora.org>
> ---
>  drivers/scsi/ufs/ufs.h        |  53 +++-------
>  drivers/scsi/ufs/ufshcd.c     | 225
> +++++++++++++++++++++++++++++++++++++++++-
>  include/uapi/scsi/Kbuild      |   1 +
>  include/uapi/scsi/ufs/Kbuild  |   3 +
>  include/uapi/scsi/ufs/ioctl.h |  57 +++++++++++
>  include/uapi/scsi/ufs/ufs.h   |  66 +++++++++++++
>  6 files changed, 361 insertions(+), 44 deletions(-)
>  create mode 100644 include/uapi/scsi/ufs/Kbuild
>  create mode 100644 include/uapi/scsi/ufs/ioctl.h
>  create mode 100644 include/uapi/scsi/ufs/ufs.h
>
> diff --git a/drivers/scsi/ufs/ufs.h b/drivers/scsi/ufs/ufs.h
> index 42c459a..1f023c4 100644
> --- a/drivers/scsi/ufs/ufs.h
> +++ b/drivers/scsi/ufs/ufs.h
> @@ -38,6 +38,7 @@
>
>  #include <linux/mutex.h>
>  #include <linux/types.h>
> +#include <scsi/ufs/ufs.h>
>
>  #define MAX_CDB_SIZE	16
>  #define GENERAL_UPIU_REQUEST_SIZE 32
> @@ -71,6 +72,16 @@ enum {
>  	UFS_UPIU_RPMB_WLUN		= 0xC4,
>  };
>
> +/**
> + * ufs_is_valid_unit_desc_lun - checks if the given LUN has a unit
> descriptor
> + * @lun: LU number to check
> + * @return: true if the lun has a matching unit descriptor, false
> otherwise
> + */
> +static inline bool ufs_is_valid_unit_desc_lun(u8 lun)
> +{
> +	return (lun == UFS_UPIU_RPMB_WLUN || (lun <
> UFS_UPIU_MAX_GENERAL_LUN));
> +}
> +
>  /*
>   * UFS Protocol Information Unit related definitions
>   */
> @@ -126,35 +137,6 @@ enum {
>  	UPIU_QUERY_FUNC_STANDARD_WRITE_REQUEST          = 0x81,
>  };
>
> -/* Flag idn for Query Requests*/
> -enum flag_idn {
> -	QUERY_FLAG_IDN_FDEVICEINIT      = 0x01,
> -	QUERY_FLAG_IDN_PWR_ON_WPE	= 0x03,
> -	QUERY_FLAG_IDN_BKOPS_EN         = 0x04,
> -};
> -
> -/* Attribute idn for Query requests */
> -enum attr_idn {
> -	QUERY_ATTR_IDN_ACTIVE_ICC_LVL	= 0x03,
> -	QUERY_ATTR_IDN_BKOPS_STATUS	= 0x05,
> -	QUERY_ATTR_IDN_EE_CONTROL	= 0x0D,
> -	QUERY_ATTR_IDN_EE_STATUS	= 0x0E,
> -};
> -
> -/* Descriptor idn for Query requests */
> -enum desc_idn {
> -	QUERY_DESC_IDN_DEVICE		= 0x0,
> -	QUERY_DESC_IDN_CONFIGURAION	= 0x1,
> -	QUERY_DESC_IDN_UNIT		= 0x2,
> -	QUERY_DESC_IDN_RFU_0		= 0x3,
> -	QUERY_DESC_IDN_INTERCONNECT	= 0x4,
> -	QUERY_DESC_IDN_STRING		= 0x5,
> -	QUERY_DESC_IDN_RFU_1		= 0x6,
> -	QUERY_DESC_IDN_GEOMETRY		= 0x7,
> -	QUERY_DESC_IDN_POWER		= 0x8,
> -	QUERY_DESC_IDN_MAX,
> -};
> -
>  enum desc_header_offset {
>  	QUERY_DESC_LENGTH_OFFSET	= 0x00,
>  	QUERY_DESC_DESC_TYPE_OFFSET	= 0x01,
> @@ -247,19 +229,6 @@ enum bkops_status {
>  	BKOPS_STATUS_MAX		 = BKOPS_STATUS_CRITICAL,
>  };
>
> -/* UTP QUERY Transaction Specific Fields OpCode */
> -enum query_opcode {
> -	UPIU_QUERY_OPCODE_NOP		= 0x0,
> -	UPIU_QUERY_OPCODE_READ_DESC	= 0x1,
> -	UPIU_QUERY_OPCODE_WRITE_DESC	= 0x2,
> -	UPIU_QUERY_OPCODE_READ_ATTR	= 0x3,
> -	UPIU_QUERY_OPCODE_WRITE_ATTR	= 0x4,
> -	UPIU_QUERY_OPCODE_READ_FLAG	= 0x5,
> -	UPIU_QUERY_OPCODE_SET_FLAG	= 0x6,
> -	UPIU_QUERY_OPCODE_CLEAR_FLAG	= 0x7,
> -	UPIU_QUERY_OPCODE_TOGGLE_FLAG	= 0x8,
> -};
> -
>  /* Query response result code */
>  enum {
>  	QUERY_RESULT_SUCCESS                    = 0x00,
> diff --git a/drivers/scsi/ufs/ufshcd.c b/drivers/scsi/ufs/ufshcd.c
> index 5d60a86..cb357f8 100644
> --- a/drivers/scsi/ufs/ufshcd.c
> +++ b/drivers/scsi/ufs/ufshcd.c
> @@ -3,7 +3,7 @@
>   *
>   * This code is based on drivers/scsi/ufs/ufshcd.c
>   * Copyright (C) 2011-2013 Samsung India Software Operations
> - * Copyright (c) 2013-2014, The Linux Foundation. All rights reserved.
> + * Copyright (c) 2013-2015, The Linux Foundation. All rights reserved.
>   *
>   * Authors:
>   *	Santosh Yaraganavi <santosh.sy@...sung.com>
> @@ -39,6 +39,7 @@
>
>  #include <linux/async.h>
>  #include <linux/devfreq.h>
> +#include <scsi/ufs/ioctl.h>
>
>  #include "ufshcd.h"
>  #include "unipro.h"
> @@ -74,6 +75,9 @@
>  /* Interrupt aggregation default timeout, unit: 40us */
>  #define INT_AGGR_DEF_TO	0x02
>
> +/* IOCTL opcode for command - ufs set device read only */
> +#define UFS_IOCTL_BLKROSET      BLKROSET
> +
>  #define ufshcd_toggle_vreg(_dev, _vreg, _on)				\
>  	({                                                              \
>  		int _ret;                                               \
> @@ -1882,7 +1886,7 @@ static inline int ufshcd_read_unit_desc_param(struct
> ufs_hba *hba,
>  	 * Unit descriptors are only available for general purpose LUs
> (LUN id
>  	 * from 0 to 7) and RPMB Well known LU.
>  	 */
> -	if (lun != UFS_UPIU_RPMB_WLUN && (lun >=
> UFS_UPIU_MAX_GENERAL_LUN))
> +	if (!ufs_is_valid_unit_desc_lun(lun))
>  		return -EOPNOTSUPP;
>
>  	return ufshcd_read_desc_param(hba, QUERY_DESC_IDN_UNIT, lun,
> @@ -4201,6 +4205,222 @@ static void ufshcd_async_scan(void *data,
> async_cookie_t cookie)
>  	ufshcd_probe_hba(hba);
>  }
>
> +/**
> + * ufshcd_query_ioctl - perform user read queries
> + * @hba: per-adapter instance
> + * @lun: used for lun specific queries
> + * @buffer: user space buffer for reading and submitting query data and
> params
> + * @return: 0 for success negative error code otherwise
> + *
> + * Expected/Submitted buffer structure is struct ufs_ioctl_query_data.
> + * It will read the opcode, idn and buf_length parameters, and, put the
> + * response in the buffer field while updating the used size in
> buf_length.
> + */
> +static int ufshcd_query_ioctl(struct ufs_hba *hba, u8 lun, void __user
> *buffer)
> +{
> +	struct ufs_ioctl_query_data *ioctl_data;
> +	int err = 0;
> +	int length = 0;
> +	void *data_ptr;
> +	bool flag;
> +	u32 att;
> +	u8 index;
> +	u8 *desc = NULL;
> +
> +	ioctl_data = kzalloc(sizeof(struct ufs_ioctl_query_data),
> GFP_KERNEL);
> +	if (!ioctl_data) {
> +		err = -ENOMEM;
> +		goto out;
> +	}
> +
> +	/* extract params from user buffer */
> +	err = copy_from_user(ioctl_data, buffer,
> +			sizeof(struct ufs_ioctl_query_data));
> +	if (err) {
> +		dev_err(hba->dev,
> +			"%s: Failed copying buffer from user, err %d\n",
> +			__func__, err);
> +		goto out_release_mem;
> +	}
> +
> +	/* verify legal parameters & send query */
> +	switch (ioctl_data->opcode) {
> +	case UPIU_QUERY_OPCODE_READ_DESC:
> +		switch (ioctl_data->idn) {
> +		case QUERY_DESC_IDN_DEVICE:
> +		case QUERY_DESC_IDN_CONFIGURAION:
> +		case QUERY_DESC_IDN_INTERCONNECT:
> +		case QUERY_DESC_IDN_GEOMETRY:
> +		case QUERY_DESC_IDN_POWER:
> +			index = 0;
> +			break;
> +		case QUERY_DESC_IDN_UNIT:
> +			if (!ufs_is_valid_unit_desc_lun(lun)) {
> +				dev_err(hba->dev,
> +					"%s: No unit descriptor for lun
> 0x%x\n",
> +					__func__, lun);
> +				err = -EINVAL;
> +				goto out_release_mem;
> +			}
> +			index = lun;
> +			break;
> +		default:
> +			goto out_einval;
> +		}
> +		length = min_t(int, QUERY_DESC_MAX_SIZE,
> +				ioctl_data->buf_size);
> +		desc = kzalloc(length, GFP_KERNEL);
> +		if (!desc) {
> +			dev_err(hba->dev, "%s: Failed allocating %d
> bytes\n",
> +					__func__, length);
> +			err = -ENOMEM;
> +			goto out_release_mem;
> +		}
> +		err = ufshcd_query_descriptor(hba, ioctl_data->opcode,
> +				ioctl_data->idn, index, 0, desc, &length);
> +		break;
> +	case UPIU_QUERY_OPCODE_READ_ATTR:
> +		switch (ioctl_data->idn) {
> +		case QUERY_ATTR_IDN_BOOT_LU_EN:
> +		case QUERY_ATTR_IDN_POWER_MODE:
> +		case QUERY_ATTR_IDN_ACTIVE_ICC_LVL:
> +		case QUERY_ATTR_IDN_OOO_DATA_EN:
> +		case QUERY_ATTR_IDN_BKOPS_STATUS:
> +		case QUERY_ATTR_IDN_PURGE_STATUS:
> +		case QUERY_ATTR_IDN_MAX_DATA_IN:
> +		case QUERY_ATTR_IDN_MAX_DATA_OUT:
> +		case QUERY_ATTR_IDN_REF_CLK_FREQ:
> +		case QUERY_ATTR_IDN_CONF_DESC_LOCK:
> +		case QUERY_ATTR_IDN_MAX_NUM_OF_RTT:
> +		case QUERY_ATTR_IDN_EE_CONTROL:
> +		case QUERY_ATTR_IDN_EE_STATUS:
> +		case QUERY_ATTR_IDN_SECONDS_PASSED:
> +			index = 0;
> +			break;
> +		case QUERY_ATTR_IDN_DYN_CAP_NEEDED:
> +		case QUERY_ATTR_IDN_CORR_PRG_BLK_NUM:
> +			index = lun;
> +			break;
> +		default:
> +			goto out_einval;
> +		}
> +		err = ufshcd_query_attr(hba, ioctl_data->opcode,
> +					ioctl_data->idn, index, 0, &att);
> +		break;
> +	case UPIU_QUERY_OPCODE_READ_FLAG:
> +		switch (ioctl_data->idn) {
> +		case QUERY_FLAG_IDN_FDEVICEINIT:
> +		case QUERY_FLAG_IDN_PERMANENT_WPE:
> +		case QUERY_FLAG_IDN_PWR_ON_WPE:
> +		case QUERY_FLAG_IDN_BKOPS_EN:
> +		case QUERY_FLAG_IDN_PURGE_ENABLE:
> +		case QUERY_FLAG_IDN_FPHYRESOURCEREMOVAL:
> +		case QUERY_FLAG_IDN_BUSY_RTC:
> +			break;
> +		default:
> +			goto out_einval;
> +		}
> +		err = ufshcd_query_flag(hba, ioctl_data->opcode,
> +					ioctl_data->idn, &flag);
> +		break;
> +	default:
> +		goto out_einval;
> +	}
> +
> +	if (err) {
> +		dev_err(hba->dev, "%s: Query for idn %d failed\n",
> __func__,
> +					ioctl_data->idn);
> +		goto out_release_mem;
> +	}
> +
> +	/*
> +	 * copy response data
> +	 * As we might end up reading less data then what is specified in
> +	 * "ioct_data->buf_size". So we are updating "ioct_data->
> +	 * buf_size" to what exactly we have read.
> +	 */
> +	switch (ioctl_data->opcode) {
> +	case UPIU_QUERY_OPCODE_READ_DESC:
> +		ioctl_data->buf_size = min_t(int, ioctl_data->buf_size,
> length);
> +		data_ptr = desc;
> +		break;
> +	case UPIU_QUERY_OPCODE_READ_ATTR:
> +		ioctl_data->buf_size = sizeof(u32);
> +		data_ptr = &att;
> +		break;
> +	case UPIU_QUERY_OPCODE_READ_FLAG:
> +		ioctl_data->buf_size = 1;
> +		data_ptr = &flag;
> +		break;
> +	default:
> +		BUG_ON(true);
> +	}
> +
> +	/* copy to user */
> +	err = copy_to_user(buffer, ioctl_data,
> +			sizeof(struct ufs_ioctl_query_data));
> +	if (err)
> +		dev_err(hba->dev, "%s: Failed copying back to user.\n",
> +			__func__);
> +	err = copy_to_user(buffer + sizeof(struct ufs_ioctl_query_data),
> +			data_ptr, ioctl_data->buf_size);
> +	if (err)
> +		dev_err(hba->dev, "%s: err %d copying back to user.\n",
> +				__func__, err);
> +	goto out_release_mem;
> +
> +out_einval:
> +	dev_err(hba->dev,
> +		"%s: illegal ufs query ioctl data, opcode 0x%x, idn
> 0x%x\n",
> +		__func__, ioctl_data->opcode, (unsigned
> int)ioctl_data->idn);
> +	err = -EINVAL;
> +out_release_mem:
> +	kfree(ioctl_data);
> +	kfree(desc);
> +out:
> +	return err;
> +}
> +
> +/**
> + * ufshcd_ioctl - ufs ioctl callback registered in scsi_host
> + * @dev: scsi device required for per LUN queries
> + * @cmd: command opcode
> + * @buffer: user space buffer for transferring data
> + *
> + * Supported commands:
> + * UFS_IOCTL_QUERY
> + */
> +static int ufshcd_ioctl(struct scsi_device *dev, int cmd, void __user
> *buffer)
> +{
> +	struct ufs_hba *hba = shost_priv(dev->host);
> +	int err = 0;
> +
> +	BUG_ON(!hba);
> +	if (!buffer) {
> +		dev_err(hba->dev, "%s: User buffer is NULL!\n", __func__);
> +		return -EINVAL;
> +	}
> +
> +	switch (cmd) {
> +	case UFS_IOCTL_QUERY:
> +		pm_runtime_get_sync(hba->dev);
> +		err = ufshcd_query_ioctl(hba,
> ufshcd_scsi_to_upiu_lun(dev->lun),
> +				buffer);
> +		pm_runtime_put_sync(hba->dev);
> +		break;
> +	case UFS_IOCTL_BLKROSET:
> +		err = -ENOIOCTLCMD;
> +		break;
> +	default:
> +		err = -EINVAL;
> +		dev_err(hba->dev, "%s: Illegal ufs-IOCTL cmd %d\n",
> __func__,
> +				cmd);
> +		break;
> +	}
> +
> +	return err;
> +}
> +
>  static struct scsi_host_template ufshcd_driver_template = {
>  	.module			= THIS_MODULE,
>  	.name			= UFSHCD,
> @@ -4213,6 +4433,7 @@ static struct scsi_host_template
> ufshcd_driver_template = {
>  	.eh_abort_handler	= ufshcd_abort,
>  	.eh_device_reset_handler = ufshcd_eh_device_reset_handler,
>  	.eh_host_reset_handler   = ufshcd_eh_host_reset_handler,
> +	.ioctl			= ufshcd_ioctl,
>  	.this_id		= -1,
>  	.sg_tablesize		= SG_ALL,
>  	.cmd_per_lun		= UFSHCD_CMD_PER_LUN,
> diff --git a/include/uapi/scsi/Kbuild b/include/uapi/scsi/Kbuild
> index 75746d5..d404525 100644
> --- a/include/uapi/scsi/Kbuild
> +++ b/include/uapi/scsi/Kbuild
> @@ -1,5 +1,6 @@
>  # UAPI Header export list
>  header-y += fc/
> +header-y += ufs/
>  header-y += scsi_bsg_fc.h
>  header-y += scsi_netlink.h
>  header-y += scsi_netlink_fc.h
> diff --git a/include/uapi/scsi/ufs/Kbuild b/include/uapi/scsi/ufs/Kbuild
> new file mode 100644
> index 0000000..cc3ef20
> --- /dev/null
> +++ b/include/uapi/scsi/ufs/Kbuild
> @@ -0,0 +1,3 @@
> +# UAPI Header export list
> +header-y += ioctl.h
> +header-y += ufs.h
> diff --git a/include/uapi/scsi/ufs/ioctl.h b/include/uapi/scsi/ufs/ioctl.h
> new file mode 100644
> index 0000000..bc4eed7
> --- /dev/null
> +++ b/include/uapi/scsi/ufs/ioctl.h
> @@ -0,0 +1,57 @@
> +#ifndef UAPI_UFS_IOCTL_H_
> +#define UAPI_UFS_IOCTL_H_
> +
> +#include <linux/types.h>
> +
> +/*
> + *  IOCTL opcode for ufs queries has the following opcode after
> + *  SCSI_IOCTL_GET_PCI
> + */
> +#define UFS_IOCTL_QUERY			0x5388
> +
> +/**
> + * struct ufs_ioctl_query_data - used to transfer data to and from user
> via ioctl
> + * @opcode: type of data to query (descriptor/attribute/flag)
> + * @idn: id of the data structure
> + * @buf_size: number of allocated bytes/data size on return
> + * @buffer: data location
> + *
> + * Received: buffer and buf_size (available space for transferred data)
> + * Submitted: opcode, idn, length, buf_size
> + */
> +struct ufs_ioctl_query_data {
> +	/*
> +	 * User should select one of the opcode defined in "enum
> query_opcode".
> +	 * Please check include/uapi/scsi/ufs/ufs.h for the definition of
> it.
> +	 * Note that only UPIU_QUERY_OPCODE_READ_DESC,
> +	 * UPIU_QUERY_OPCODE_READ_ATTR & UPIU_QUERY_OPCODE_READ_FLAG are
> +	 * supported as of now. All other query_opcode would be considered
> +	 * invalid.
> +	 * As of now only read query operations are supported.
> +	 */
> +	__u32 opcode;
> +	/*
> +	 * User should select one of the idn from "enum flag_idn" or "enum
> +	 * attr_idn" or "enum desc_idn" based on whether opcode above is
> +	 * attribute, flag or descriptor.
> +	 * Please check include/uapi/scsi/ufs/ufs.h for the definition of
> it.
> +	 */
> +	__u8 idn;
> +	/*
> +	 * User should specify the size of the buffer (buffer[0] below)
> where
> +	 * it wants to read the query data (attribute/flag/descriptor).
> +	 * As we might end up reading less data then what is specified in
> +	 * buf_size. So we are updating buf_size to what exactly we have
> read.
> +	 */
> +	__u16 buf_size;
> +	/*
> +	 * placeholder for the start of the data buffer where kernel will
> copy
> +	 * the query data (attribute/flag/descriptor) read from the UFS
> device
> +	 * Note:
> +	 * For Read Attribute you will have to allocate 4 bytes
> +	 * For Read Flag you will have to allocate 1 byte
> +	 */
> +	__u8 buffer[0];
> +};
> +
> +#endif /* UAPI_UFS_IOCTL_H_ */
> diff --git a/include/uapi/scsi/ufs/ufs.h b/include/uapi/scsi/ufs/ufs.h
> new file mode 100644
> index 0000000..894ea45
> --- /dev/null
> +++ b/include/uapi/scsi/ufs/ufs.h
> @@ -0,0 +1,66 @@
> +#ifndef UAPI_UFS_H_
> +#define UAPI_UFS_H_
> +
> +/* Flag idn for Query Requests*/
> +enum flag_idn {
> +	QUERY_FLAG_IDN_FDEVICEINIT		= 0x01,
> +	QUERY_FLAG_IDN_PERMANENT_WPE		= 0x02,
> +	QUERY_FLAG_IDN_PWR_ON_WPE		= 0x03,
> +	QUERY_FLAG_IDN_BKOPS_EN			= 0x04,
> +	QUERY_FLAG_IDN_RESERVED1		= 0x05,
> +	QUERY_FLAG_IDN_PURGE_ENABLE		= 0x06,
> +	QUERY_FLAG_IDN_RESERVED2		= 0x07,
> +	QUERY_FLAG_IDN_FPHYRESOURCEREMOVAL      = 0x08,
> +	QUERY_FLAG_IDN_BUSY_RTC			= 0x09,
> +};
> +
> +/* Attribute idn for Query requests */
> +enum attr_idn {
> +	QUERY_ATTR_IDN_BOOT_LU_EN		= 0x00,
> +	QUERY_ATTR_IDN_RESERVED			= 0x01,
> +	QUERY_ATTR_IDN_POWER_MODE		= 0x02,
> +	QUERY_ATTR_IDN_ACTIVE_ICC_LVL		= 0x03,
> +	QUERY_ATTR_IDN_OOO_DATA_EN		= 0x04,
> +	QUERY_ATTR_IDN_BKOPS_STATUS		= 0x05,
> +	QUERY_ATTR_IDN_PURGE_STATUS		= 0x06,
> +	QUERY_ATTR_IDN_MAX_DATA_IN		= 0x07,
> +	QUERY_ATTR_IDN_MAX_DATA_OUT		= 0x08,
> +	QUERY_ATTR_IDN_DYN_CAP_NEEDED		= 0x09,
> +	QUERY_ATTR_IDN_REF_CLK_FREQ		= 0x0A,
> +	QUERY_ATTR_IDN_CONF_DESC_LOCK		= 0x0B,
> +	QUERY_ATTR_IDN_MAX_NUM_OF_RTT		= 0x0C,
> +	QUERY_ATTR_IDN_EE_CONTROL		= 0x0D,
> +	QUERY_ATTR_IDN_EE_STATUS		= 0x0E,
> +	QUERY_ATTR_IDN_SECONDS_PASSED		= 0x0F,
> +	QUERY_ATTR_IDN_CNTX_CONF		= 0x10,
> +	QUERY_ATTR_IDN_CORR_PRG_BLK_NUM		= 0x11,
> +};
> +
> +/* Descriptor idn for Query requests */
> +enum desc_idn {
> +	QUERY_DESC_IDN_DEVICE		= 0x0,
> +	QUERY_DESC_IDN_CONFIGURAION	= 0x1,
> +	QUERY_DESC_IDN_UNIT		= 0x2,
> +	QUERY_DESC_IDN_RFU_0		= 0x3,
> +	QUERY_DESC_IDN_INTERCONNECT	= 0x4,
> +	QUERY_DESC_IDN_STRING		= 0x5,
> +	QUERY_DESC_IDN_RFU_1		= 0x6,
> +	QUERY_DESC_IDN_GEOMETRY		= 0x7,
> +	QUERY_DESC_IDN_POWER		= 0x8,
> +	QUERY_DESC_IDN_RFU_2		= 0x9,
> +	QUERY_DESC_IDN_MAX,
> +};
> +
> +/* UTP QUERY Transaction Specific Fields OpCode */
> +enum query_opcode {
> +	UPIU_QUERY_OPCODE_NOP		= 0x0,
> +	UPIU_QUERY_OPCODE_READ_DESC	= 0x1,
> +	UPIU_QUERY_OPCODE_WRITE_DESC	= 0x2,
> +	UPIU_QUERY_OPCODE_READ_ATTR	= 0x3,
> +	UPIU_QUERY_OPCODE_WRITE_ATTR	= 0x4,
> +	UPIU_QUERY_OPCODE_READ_FLAG	= 0x5,
> +	UPIU_QUERY_OPCODE_SET_FLAG	= 0x6,
> +	UPIU_QUERY_OPCODE_CLEAR_FLAG	= 0x7,
> +	UPIU_QUERY_OPCODE_TOGGLE_FLAG	= 0x8,
> +};
> +#endif /* UAPI_UFS_H_ */
> --
> Qualcomm Israel, on behalf of Qualcomm Innovation Center, Inc.
> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
> a Linux Foundation Collaborative Project
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>

Reviewed-by: Dov Levenglick <dovl@...eaurora.org>

QUALCOMM ISRAEL, on behalf of Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ