Commit b2a4cc3a authored Oct 16, 2016 by Guenter Roeck

hwmon: (core) Explain why at least two attribute groups are allocated

A list of sysfs attribute groups is NULL-terminated, so we always need
to allocate data for at least two groups (the dynamically generated group
plus the NULL pointer). Add a comment to explain the situation.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

parent 239552f4

drivers/hwmon/hwmon.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -561,7 +561,7 @@ __hwmon_device_register(struct device dev, const char name, void *drvdata,

		if (chip) {
		struct attribute **attrs;
		int ngroups = 2;
		int ngroups = 2; /* terminating NULL plus &hwdev->groups */

		if (groups)
		for (i = 0; groups[i]; i++)