lkml.org 
[lkml]   [2018]   [Jun]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [RFC PATCH 4/5] fsi/scom: Add register definitions
From
Date


On 06/12/2018 12:19 AM, Benjamin Herrenschmidt wrote:
> Add a few more register and bit definitions, also define and use
> SCOM_READ_CMD (which is 0 but it makes the code clearer)
>
> Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>

Reviewed-by: Eddie James <eajames@linux.vnet.ibm.com>

> ---
> drivers/fsi/fsi-scom.c | 19 ++++++++++++++++++-
> 1 file changed, 18 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/fsi/fsi-scom.c b/drivers/fsi/fsi-scom.c
> index 6ddfb6021420..e98573ecdae1 100644
> --- a/drivers/fsi/fsi-scom.c
> +++ b/drivers/fsi/fsi-scom.c
> @@ -30,8 +30,25 @@
> #define SCOM_DATA0_REG 0x00
> #define SCOM_DATA1_REG 0x04
> #define SCOM_CMD_REG 0x08
> +#define SCOM_FSI2PIB_RESET_REG 0x18
> +#define SCOM_STATUS_REG 0x1C /* Read */
> +#define SCOM_PIB_RESET_REG 0x1C /* Write */
>
> +/* Command register */
> #define SCOM_WRITE_CMD 0x80000000
> +#define SCOM_READ_CMD 0x00000000
> +
> +/* Status register bits */
> +#define SCOM_STATUS_ERR_SUMMARY 0x80000000
> +#define SCOM_STATUS_PROTECTION 0x01000000
> +#define SCOM_STATUS_PIB_ABORT 0x00100000
> +#define SCOM_STATUS_PIB_RESP_MASK 0x00007000
> +#define SCOM_STATUS_PIB_RESP_SHIFT 12
> +
> +#define SCOM_STATUS_ANY_ERR (SCOM_STATUS_ERR_SUMMARY | \
> + SCOM_STATUS_PROTECTION | \
> + SCOM_STATUS_PIB_ABORT | \
> + SCOM_STATUS_PIB_RESP_MASK)
>
> struct scom_device {
> struct list_head link;
> @@ -85,7 +102,7 @@ static int get_scom(struct scom_device *scom_dev, uint64_t *value,
>
> mutex_lock(&scom_dev->lock);
> *value = 0ULL;
> - data = cpu_to_be32(addr);
> + data = cpu_to_be32(SCOM_READ_CMD | addr);
> rc = fsi_device_write(scom_dev->fsi_dev, SCOM_CMD_REG, &data,
> sizeof(uint32_t));
> if (rc)

\
 
 \ /
  Last update: 2018-06-13 16:58    [W:0.091 / U:0.528 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site