Merge tag 'staging-5.18-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/gregkh...
[platform/kernel/linux-starfive.git] / drivers / staging / wfx / bus_spi.c
index fa0ff66..7fb1afb 100644 (file)
 #define SET_WRITE 0x7FFF        /* usage: and operation */
 #define SET_READ 0x8000         /* usage: or operation */
 
-#define WFX_RESET_INVERTED 1
+static const struct wfx_platform_data pdata_wf200 = {
+       .file_fw = "wfx/wfm_wf200",
+       .file_pds = "wfx/wf200.pds",
+       .use_rising_clk = true,
+};
+
+static const struct wfx_platform_data pdata_brd4001a = {
+       .file_fw = "wfx/wfm_wf200",
+       .file_pds = "wfx/brd4001a.pds",
+       .use_rising_clk = true,
+};
 
-static const struct wfx_platform_data wfx_spi_pdata = {
-       .file_fw = "wfm_wf200",
-       .file_pds = "wf200.pds",
+static const struct wfx_platform_data pdata_brd8022a = {
+       .file_fw = "wfx/wfm_wf200",
+       .file_pds = "wfx/brd8022a.pds",
+       .use_rising_clk = true,
+};
+
+static const struct wfx_platform_data pdata_brd8023a = {
+       .file_fw = "wfx/wfm_wf200",
+       .file_pds = "wfx/brd8023a.pds",
        .use_rising_clk = true,
 };
 
@@ -38,27 +54,25 @@ struct wfx_spi_priv {
        bool need_swab;
 };
 
-/* The chip reads 16bits of data at time and place them directly into (little
- * endian) CPU register. So, the chip expects bytes order to be "B1 B0 B3 B2"
- * (while LE is "B0 B1 B2 B3" and BE is "B3 B2 B1 B0")
+/* The chip reads 16bits of data at time and place them directly into (little endian) CPU register.
+ * So, the chip expects bytes order to be "B1 B0 B3 B2" (while LE is "B0 B1 B2 B3" and BE is
+ * "B3 B2 B1 B0")
  *
- * A little endian host with bits_per_word == 16 should do the right job
- * natively. The code below to support big endian host and commonly used SPI
- * 8bits.
+ * A little endian host with bits_per_word == 16 should do the right job natively. The code below to
+ * support big endian host and commonly used SPI 8bits.
  */
-static int wfx_spi_copy_from_io(void *priv, unsigned int addr,
-                               void *dst, size_t count)
+static int wfx_spi_copy_from_io(void *priv, unsigned int addr, void *dst, size_t count)
 {
        struct wfx_spi_priv *bus = priv;
        u16 regaddr = (addr << 12) | (count / 2) | SET_READ;
-       struct spi_message      m;
-       struct spi_transfer     t_addr = {
-               .tx_buf         = &regaddr,
-               .len            = sizeof(regaddr),
+       struct spi_message m;
+       struct spi_transfer t_addr = {
+               .tx_buf = &regaddr,
+               .len = sizeof(regaddr),
        };
-       struct spi_transfer     t_msg = {
-               .rx_buf         = dst,
-               .len            = count,
+       struct spi_transfer t_msg = {
+               .rx_buf = dst,
+               .len = count,
        };
        u16 *dst16 = dst;
        int ret, i;
@@ -80,22 +94,21 @@ static int wfx_spi_copy_from_io(void *priv, unsigned int addr,
        return ret;
 }
 
-static int wfx_spi_copy_to_io(void *priv, unsigned int addr,
-                             const void *src, size_t count)
+static int wfx_spi_copy_to_io(void *priv, unsigned int addr, const void *src, size_t count)
 {
        struct wfx_spi_priv *bus = priv;
        u16 regaddr = (addr << 12) | (count / 2);
        /* FIXME: use a bounce buffer */
        u16 *src16 = (void *)src;
        int ret, i;
-       struct spi_message      m;
-       struct spi_transfer     t_addr = {
-               .tx_buf         = &regaddr,
-               .len            = sizeof(regaddr),
+       struct spi_message m;
+       struct spi_transfer t_addr = {
+               .tx_buf = &regaddr,
+               .len = sizeof(regaddr),
        };
-       struct spi_transfer     t_msg = {
-               .tx_buf         = src,
-               .len            = count,
+       struct spi_transfer t_msg = {
+               .tx_buf = src,
+               .len = count,
        };
 
        WARN(count % 2, "buffer size must be a multiple of 2");
@@ -149,8 +162,7 @@ static int wfx_spi_irq_subscribe(void *priv)
                flags = IRQF_TRIGGER_HIGH;
        flags |= IRQF_ONESHOT;
        return devm_request_threaded_irq(&bus->func->dev, bus->func->irq, NULL,
-                                        wfx_spi_irq_handler, IRQF_ONESHOT,
-                                        "wfx", bus);
+                                        wfx_spi_irq_handler, flags, "wfx", bus);
 }
 
 static int wfx_spi_irq_unsubscribe(void *priv)
@@ -163,23 +175,23 @@ static int wfx_spi_irq_unsubscribe(void *priv)
 
 static size_t wfx_spi_align_size(void *priv, size_t size)
 {
-       /* Most of SPI controllers avoid DMA if buffer size is not 32bit aligned
-        */
+       /* Most of SPI controllers avoid DMA if buffer size is not 32bit aligned */
        return ALIGN(size, 4);
 }
 
-static const struct hwbus_ops wfx_spi_hwbus_ops = {
-       .copy_from_io = wfx_spi_copy_from_io,
-       .copy_to_io = wfx_spi_copy_to_io,
-       .irq_subscribe = wfx_spi_irq_subscribe,
+static const struct wfx_hwbus_ops wfx_spi_hwbus_ops = {
+       .copy_from_io    = wfx_spi_copy_from_io,
+       .copy_to_io      = wfx_spi_copy_to_io,
+       .irq_subscribe   = wfx_spi_irq_subscribe,
        .irq_unsubscribe = wfx_spi_irq_unsubscribe,
-       .lock                   = wfx_spi_lock,
-       .unlock                 = wfx_spi_unlock,
-       .align_size             = wfx_spi_align_size,
+       .lock            = wfx_spi_lock,
+       .unlock          = wfx_spi_unlock,
+       .align_size      = wfx_spi_align_size,
 };
 
 static int wfx_spi_probe(struct spi_device *func)
 {
+       struct wfx_platform_data *pdata;
        struct wfx_spi_priv *bus;
        int ret;
 
@@ -188,16 +200,19 @@ static int wfx_spi_probe(struct spi_device *func)
        ret = spi_setup(func);
        if (ret)
                return ret;
+       pdata = (struct wfx_platform_data *)spi_get_device_id(func)->driver_data;
+       if (!pdata) {
+               dev_err(&func->dev, "unable to retrieve driver data (please report)\n");
+               return -ENODEV;
+       }
+
        /* Trace below is also displayed by spi_setup() if compiled with DEBUG */
        dev_dbg(&func->dev, "SPI params: CS=%d, mode=%d bits/word=%d speed=%d\n",
-               func->chip_select, func->mode, func->bits_per_word,
-               func->max_speed_hz);
+               func->chip_select, func->mode, func->bits_per_word, func->max_speed_hz);
        if (func->bits_per_word != 16 && func->bits_per_word != 8)
-               dev_warn(&func->dev, "unusual bits/word value: %d\n",
-                        func->bits_per_word);
+               dev_warn(&func->dev, "unusual bits/word value: %d\n", func->bits_per_word);
        if (func->max_speed_hz > 50000000)
-               dev_warn(&func->dev, "%dHz is a very high speed\n",
-                        func->max_speed_hz);
+               dev_warn(&func->dev, "%dHz is a very high speed\n", func->max_speed_hz);
 
        bus = devm_kzalloc(&func->dev, sizeof(*bus), GFP_KERNEL);
        if (!bus)
@@ -207,25 +222,20 @@ static int wfx_spi_probe(struct spi_device *func)
                bus->need_swab = true;
        spi_set_drvdata(func, bus);
 
-       bus->gpio_reset = devm_gpiod_get_optional(&func->dev, "reset",
-                                                 GPIOD_OUT_LOW);
+       bus->gpio_reset = devm_gpiod_get_optional(&func->dev, "reset", GPIOD_OUT_LOW);
        if (IS_ERR(bus->gpio_reset))
                return PTR_ERR(bus->gpio_reset);
        if (!bus->gpio_reset) {
-               dev_warn(&func->dev,
-                        "gpio reset is not defined, trying to load firmware anyway\n");
+               dev_warn(&func->dev, "gpio reset is not defined, trying to load firmware anyway\n");
        } else {
                gpiod_set_consumer_name(bus->gpio_reset, "wfx reset");
-               if (spi_get_device_id(func)->driver_data & WFX_RESET_INVERTED)
-                       gpiod_toggle_active_low(bus->gpio_reset);
                gpiod_set_value_cansleep(bus->gpio_reset, 1);
                usleep_range(100, 150);
                gpiod_set_value_cansleep(bus->gpio_reset, 0);
                usleep_range(2000, 2500);
        }
 
-       bus->core = wfx_init_common(&func->dev, &wfx_spi_pdata,
-                                   &wfx_spi_hwbus_ops, bus);
+       bus->core = wfx_init_common(&func->dev, pdata, &wfx_spi_hwbus_ops, bus);
        if (!bus->core)
                return -EIO;
 
@@ -244,16 +254,20 @@ static void wfx_spi_remove(struct spi_device *func)
  * stripped.
  */
 static const struct spi_device_id wfx_spi_id[] = {
-       { "wfx-spi", WFX_RESET_INVERTED },
-       { "wf200", 0 },
+       { "wf200",    (kernel_ulong_t)&pdata_wf200 },
+       { "brd4001a", (kernel_ulong_t)&pdata_brd4001a },
+       { "brd8022a", (kernel_ulong_t)&pdata_brd8022a },
+       { "brd8023a", (kernel_ulong_t)&pdata_brd8023a },
        { },
 };
 MODULE_DEVICE_TABLE(spi, wfx_spi_id);
 
 #ifdef CONFIG_OF
 static const struct of_device_id wfx_spi_of_match[] = {
-       { .compatible = "silabs,wfx-spi", .data = (void *)WFX_RESET_INVERTED },
        { .compatible = "silabs,wf200" },
+       { .compatible = "silabs,brd4001a" },
+       { .compatible = "silabs,brd8022a" },
+       { .compatible = "silabs,brd8023a" },
        { },
 };
 MODULE_DEVICE_TABLE(of, wfx_spi_of_match);