diff mbox series

[v2,RESEND,2/2] iio: accel: mma8452: Sort match tables

Message ID 20230818184033.335502-3-biju.das.jz@bp.renesas.com (mailing list archive)
State Accepted
Delegated to: Geert Uytterhoeven
Headers show
Series Match data improvements for mma8452 driver | expand

Commit Message

Biju Das Aug. 18, 2023, 6:40 p.m. UTC 
Sort ID table alphabetically by name and OF table by compatible.

Suggested-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
---
v2:
 * New patch
---
 drivers/iio/accel/mma8452.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Andy Shevchenko Aug. 21, 2023, 1:10 p.m. UTC | #1 
On Fri, Aug 18, 2023 at 07:40:33PM +0100, Biju Das wrote:
> Sort ID table alphabetically by name and OF table by compatible.

Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Jonathan Cameron Aug. 28, 2023, 11:21 a.m. UTC | #2 
On Mon, 21 Aug 2023 16:10:50 +0300
Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote:

> On Fri, Aug 18, 2023 at 07:40:33PM +0100, Biju Das wrote:
> > Sort ID table alphabetically by name and OF table by compatible.  
> 
> Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

Series applied to the togreg branch of iio.git. I'll push that out
as testing until I can rebase on rc1.

Thanks

Jonathan

>
diff mbox series

Patch

diff --git a/drivers/iio/accel/mma8452.c b/drivers/iio/accel/mma8452.c
index 5864ad726e97..d3fd0318e47b 100644
--- a/drivers/iio/accel/mma8452.c
+++ b/drivers/iio/accel/mma8452.c
@@ -1535,12 +1535,12 @@  static int mma8452_reset(struct i2c_client *client)
 }
 
 static const struct of_device_id mma8452_dt_ids[] = {
+	{ .compatible = "fsl,fxls8471", .data = &mma_chip_info_table[fxls8471] },
 	{ .compatible = "fsl,mma8451", .data = &mma_chip_info_table[mma8451] },
 	{ .compatible = "fsl,mma8452", .data = &mma_chip_info_table[mma8452] },
 	{ .compatible = "fsl,mma8453", .data = &mma_chip_info_table[mma8453] },
 	{ .compatible = "fsl,mma8652", .data = &mma_chip_info_table[mma8652] },
 	{ .compatible = "fsl,mma8653", .data = &mma_chip_info_table[mma8653] },
-	{ .compatible = "fsl,fxls8471", .data = &mma_chip_info_table[fxls8471] },
 	{ }
 };
 MODULE_DEVICE_TABLE(of, mma8452_dt_ids);
@@ -1824,12 +1824,12 @@  static const struct dev_pm_ops mma8452_pm_ops = {
 };
 
 static const struct i2c_device_id mma8452_id[] = {
+	{ "fxls8471", (kernel_ulong_t)&mma_chip_info_table[fxls8471] },
 	{ "mma8451", (kernel_ulong_t)&mma_chip_info_table[mma8451] },
 	{ "mma8452", (kernel_ulong_t)&mma_chip_info_table[mma8452] },
 	{ "mma8453", (kernel_ulong_t)&mma_chip_info_table[mma8453] },
 	{ "mma8652", (kernel_ulong_t)&mma_chip_info_table[mma8652] },
 	{ "mma8653", (kernel_ulong_t)&mma_chip_info_table[mma8653] },
-	{ "fxls8471", (kernel_ulong_t)&mma_chip_info_table[fxls8471] },
 	{ }
 };
 MODULE_DEVICE_TABLE(i2c, mma8452_id);