]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
iio: accel: bma220: move bma220_power function
authorPetre Rodan <petre.rodan@subdimension.ro>
Sun, 5 Oct 2025 13:12:14 +0000 (16:12 +0300)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Sun, 19 Oct 2025 10:59:19 +0000 (11:59 +0100)
Move bma220_power() before bma220_init() as a precursor to a
patch that removes code duplication.

Signed-off-by: Petre Rodan <petre.rodan@subdimension.ro>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/accel/bma220_spi.c

index a5d2906321ae5fc67c0e1dc44651833c5e7fcb39..45ac0d7ee27de65b204bd2766f26024e4ed57f4c 100644 (file)
@@ -199,6 +199,31 @@ static const struct iio_info bma220_info = {
        .read_avail             = bma220_read_avail,
 };
 
+static int bma220_power(struct spi_device *spi, bool up)
+{
+       int ret;
+       unsigned int i;
+
+       /*
+        * The chip can be suspended/woken up by a simple register read.
+        * So, we need up to 2 register reads of the suspend register
+        * to make sure that the device is in the desired state.
+        */
+       for (i = 0; i < 2; i++) {
+               ret = bma220_read_reg(spi, BMA220_REG_SUSPEND);
+               if (ret < 0)
+                       return ret;
+
+               if (up && ret == BMA220_SUSPEND_SLEEP)
+                       return 0;
+
+               if (!up && ret == BMA220_SUSPEND_WAKE)
+                       return 0;
+       }
+
+       return -EBUSY;
+}
+
 static int bma220_init(struct spi_device *spi)
 {
        int ret;
@@ -224,30 +249,6 @@ static int bma220_init(struct spi_device *spi)
        return 0;
 }
 
-static int bma220_power(struct spi_device *spi, bool up)
-{
-       int i, ret;
-
-       /*
-        * The chip can be suspended/woken up by a simple register read.
-        * So, we need up to 2 register reads of the suspend register
-        * to make sure that the device is in the desired state.
-        */
-       for (i = 0; i < 2; i++) {
-               ret = bma220_read_reg(spi, BMA220_REG_SUSPEND);
-               if (ret < 0)
-                       return ret;
-
-               if (up && ret == BMA220_SUSPEND_SLEEP)
-                       return 0;
-
-               if (!up && ret == BMA220_SUSPEND_WAKE)
-                       return 0;
-       }
-
-       return -EBUSY;
-}
-
 static void bma220_deinit(void *spi)
 {
        bma220_power(spi, false);