Re: [PATCH] regulator: pfuze100: Add terminate entry for [i2c|of]_device_id tables
On Tue, Mar 04, 2014 at 06:20:14PM +0800, Axel Lin wrote: > Also remove PFUZE_NUM to avoid below build warnings: Applied, thanks. signature.asc Description: Digital signature
RE: [PATCH] regulator: pfuze100: Add terminate entry for [i2c|of]_device_id tables
Acked-by: Robin Gong -Original Message- From: Axel Lin [mailto:axel@ingics.com] Sent: Tuesday, March 04, 2014 6:20 PM To: Mark Brown Cc: Gong Yibin-B38343; Liam Girdwood; linux-kernel@vger.kernel.org Subject: [PATCH] regulator: pfuze100: Add terminate entry for [i2c|of]_device_id tables Also remove PFUZE_NUM to avoid below build warnings: CC [M] drivers/regulator/pfuze100-regulator.o drivers/regulator/pfuze100-regulator.c:86:2: warning: excess elements in array initializer [enabled by default] drivers/regulator/pfuze100-regulator.c:86:2: warning: (near initialization for 'pfuze_device_id') [enabled by default] drivers/regulator/pfuze100-regulator.c:93:2: warning: excess elements in array initializer [enabled by default] drivers/regulator/pfuze100-regulator.c:93:2: warning: (near initialization for 'pfuze_dt_ids') [enabled by default] Signed-off-by: Axel Lin --- drivers/regulator/pfuze100-regulator.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/drivers/regulator/pfuze100-regulator.c b/drivers/regulator/pfuze100-regulator.c index 44075f8..f87ec70 100644 --- a/drivers/regulator/pfuze100-regulator.c +++ b/drivers/regulator/pfuze100-regulator.c @@ -56,7 +56,7 @@ #define PFUZE100_VGEN5VOL 0x70 #define PFUZE100_VGEN6VOL 0x71 -enum chips {PFUZE100, PFUZE200, PFUZE_NUM}; +enum chips { PFUZE100, PFUZE200 }; struct pfuze_regulator { struct regulator_desc desc; @@ -80,15 +80,17 @@ static const int pfuze100_vsnvs[] = { 100, 110, 120, 130, 150, 180, 300, }; -static const struct i2c_device_id pfuze_device_id[PFUZE_NUM] = { +static const struct i2c_device_id pfuze_device_id[] = { {.name = "pfuze100", .driver_data = PFUZE100}, {.name = "pfuze200", .driver_data = PFUZE200}, + { } }; MODULE_DEVICE_TABLE(i2c, pfuze_device_id); -static const struct of_device_id pfuze_dt_ids[PFUZE_NUM] = { +static const struct of_device_id pfuze_dt_ids[] = { { .compatible = "fsl,pfuze100", .data = (void *)PFUZE100}, { .compatible = "fsl,pfuze200", .data = (void *)PFUZE200}, + { } }; MODULE_DEVICE_TABLE(of, pfuze_dt_ids); -- 1.8.1.2 N�r��yb�X��ǧv�^�){.n�+{zX����ܨ}���Ơz�&j:+v���zZ+��+zf���h���~i���z��w���?�&�)ߢf��^jǫy�m��@A�a��� 0��h���i
[PATCH] regulator: pfuze100: Add terminate entry for [i2c|of]_device_id tables
Also remove PFUZE_NUM to avoid below build warnings: CC [M] drivers/regulator/pfuze100-regulator.o drivers/regulator/pfuze100-regulator.c:86:2: warning: excess elements in array initializer [enabled by default] drivers/regulator/pfuze100-regulator.c:86:2: warning: (near initialization for 'pfuze_device_id') [enabled by default] drivers/regulator/pfuze100-regulator.c:93:2: warning: excess elements in array initializer [enabled by default] drivers/regulator/pfuze100-regulator.c:93:2: warning: (near initialization for 'pfuze_dt_ids') [enabled by default] Signed-off-by: Axel Lin --- drivers/regulator/pfuze100-regulator.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/drivers/regulator/pfuze100-regulator.c b/drivers/regulator/pfuze100-regulator.c index 44075f8..f87ec70 100644 --- a/drivers/regulator/pfuze100-regulator.c +++ b/drivers/regulator/pfuze100-regulator.c @@ -56,7 +56,7 @@ #define PFUZE100_VGEN5VOL 0x70 #define PFUZE100_VGEN6VOL 0x71 -enum chips {PFUZE100, PFUZE200, PFUZE_NUM}; +enum chips { PFUZE100, PFUZE200 }; struct pfuze_regulator { struct regulator_desc desc; @@ -80,15 +80,17 @@ static const int pfuze100_vsnvs[] = { 100, 110, 120, 130, 150, 180, 300, }; -static const struct i2c_device_id pfuze_device_id[PFUZE_NUM] = { +static const struct i2c_device_id pfuze_device_id[] = { {.name = "pfuze100", .driver_data = PFUZE100}, {.name = "pfuze200", .driver_data = PFUZE200}, + { } }; MODULE_DEVICE_TABLE(i2c, pfuze_device_id); -static const struct of_device_id pfuze_dt_ids[PFUZE_NUM] = { +static const struct of_device_id pfuze_dt_ids[] = { { .compatible = "fsl,pfuze100", .data = (void *)PFUZE100}, { .compatible = "fsl,pfuze200", .data = (void *)PFUZE200}, + { } }; MODULE_DEVICE_TABLE(of, pfuze_dt_ids); -- 1.8.1.2 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/