Re: [PATCH] regulator: hi655x: Add DT bindings so module autoloads

[Mark Brown]

On Tue, Feb 28, 2017 at 07:07:06PM -0600, Jeremy Linton wrote:
> The hi655x driver is required for mmc/sd functionality on the
> 96boards hikey, and likely other platforms. When built as
> a standalone module it doesn't get automatically loaded because
> it is missing the module probe hooks.

> +static const struct of_device_id hi655x_dt_ids[] = {
> +	{ .compatible = "hisilicon,hi655x-pmic",  },
> +	{},
> +};
> +MODULE_DEVICE_TABLE(of, hi655x_dt_ids);

This is an MFD subdevice, I'd expect it to instantiate as a platform
device from the MFD - there shouldn't be a compatible string in the DT
and indeed one is not documented (I see something slipped into the
example but it's not in the actual binding).  It's not like this device
can exist independently of the parent device and that compatible string
is not specific enough anyway.

Attachment: signature.asc
Description: PGP signature