By convention MODULE_DEVICE_TABLE() immediately follows the ID table it
exports, because this is easier to read and verify. It also makes more
sense since #ifdef for ACPI or OF could hide both of them.
Most of the drivers already have this correctly placed, so adjust
the missing ones. No functional impact.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
{ .compatible = "sensirion,sgpc3", .data = &sgp_devices[SGPC3] },
{ }
};
+MODULE_DEVICE_TABLE(of, sgp_dt_ids);
static int sgp_probe(struct i2c_client *client)
{
{ "sgpc3", (kernel_ulong_t)&sgp_devices[SGPC3] },
{ }
};
-
MODULE_DEVICE_TABLE(i2c, sgp_id);
-MODULE_DEVICE_TABLE(of, sgp_dt_ids);
static struct i2c_driver sgp_driver = {
.driver = {
{ "cm3232" },
{ }
};
+MODULE_DEVICE_TABLE(i2c, cm3232_id);
static int cm3232_suspend(struct device *dev)
{
static DEFINE_SIMPLE_DEV_PM_OPS(cm3232_pm_ops, cm3232_suspend, cm3232_resume);
-MODULE_DEVICE_TABLE(i2c, cm3232_id);
-
static const struct of_device_id cm3232_of_match[] = {
{.compatible = "capella,cm3232"},
{ }