2023-07-13 12:10:57

by Praveen Talari

[permalink] [raw]
Subject: [PATCH v6 1/2] soc: qcom: geni-se: Add SPI Device mode support for GENI based QuPv3

From: Praveen Talari <[email protected]>

Add device mode supported registers and masks.

Signed-off-by: Praveen Talari <[email protected]>
---
v2 -> v3:
- modified commit message to use device mode instead of slave mode

v1 -> v2:
- modified commit message
---
include/linux/soc/qcom/geni-se.h | 9 +++++++++
1 file changed, 9 insertions(+)

diff --git a/include/linux/soc/qcom/geni-se.h b/include/linux/soc/qcom/geni-se.h
index 821a19135bb6..29e06905bc1f 100644
--- a/include/linux/soc/qcom/geni-se.h
+++ b/include/linux/soc/qcom/geni-se.h
@@ -35,6 +35,7 @@ enum geni_se_protocol_type {
GENI_SE_UART,
GENI_SE_I2C,
GENI_SE_I3C,
+ GENI_SE_SPI_SLAVE,
};

struct geni_wrapper;
@@ -73,12 +74,14 @@ struct geni_se {

/* Common SE registers */
#define GENI_FORCE_DEFAULT_REG 0x20
+#define GENI_OUTPUT_CTRL 0x24
#define SE_GENI_STATUS 0x40
#define GENI_SER_M_CLK_CFG 0x48
#define GENI_SER_S_CLK_CFG 0x4c
#define GENI_IF_DISABLE_RO 0x64
#define GENI_FW_REVISION_RO 0x68
#define SE_GENI_CLK_SEL 0x7c
+#define SE_GENI_CFG_SEQ_START 0x84
#define SE_GENI_DMA_MODE_EN 0x258
#define SE_GENI_M_CMD0 0x600
#define SE_GENI_M_CMD_CTRL_REG 0x604
@@ -111,6 +114,9 @@ struct geni_se {
/* GENI_FORCE_DEFAULT_REG fields */
#define FORCE_DEFAULT BIT(0)

+/* GENI_OUTPUT_CTRL fields */
+#define GENI_IO_MUX_0_EN BIT(0)
+
/* GENI_STATUS fields */
#define M_GENI_CMD_ACTIVE BIT(0)
#define S_GENI_CMD_ACTIVE BIT(12)
@@ -130,6 +136,9 @@ struct geni_se {
/* GENI_CLK_SEL fields */
#define CLK_SEL_MSK GENMASK(2, 0)

+/* SE_GENI_CFG_SEQ_START fields */
+#define START_TRIGGER BIT(0)
+
/* SE_GENI_DMA_MODE_EN */
#define GENI_DMA_MODE_EN BIT(0)

--
2.17.1



2023-07-14 04:19:06

by Bjorn Andersson

[permalink] [raw]
Subject: Re: [PATCH v6 1/2] soc: qcom: geni-se: Add SPI Device mode support for GENI based QuPv3

On Thu, Jul 13, 2023 at 05:21:44PM +0530, Praveen Talari wrote:
> From: Praveen Talari <[email protected]>
>
> Add device mode supported registers and masks.
>
> Signed-off-by: Praveen Talari <[email protected]>

Shouldn't this be [email protected]

Regards,
Bjorn