Re: [PATCH v4 4/5] regulator: tps65912: Add regulator driver for the TPS65912 PMIC

On Thu, Nov 05, 2015 at 12:04:00PM -0600, Andrew F. Davis wrote:

On 11/05/2015 04:14 AM, Mark Brown wrote:

quoted

That sounds like a bug to me, it'll have broken a bunch of existing
devices.

Most OF drivers have the OF MODALIAS.

That's nice but not relevant to non-OF devices.

'platform_uevent' can only emit one MODALIAS string per device (only
the last emitted one seems to count), so for any device with
'dev->of_node' set it will be the OF MODALIAS string. So I need
that table (to generate the OF MODALIAS) or this sub-device module
will not be loaded.

No, you need to fix the bug that is causing dev->of_node to be populated
for the MFD function device.  Probably the issue is that you have put
this pointless compatible string in your DT.

Please stop this.  I don't understand why you are pushing so hard to put
the Linux device model representation of the device into DT but it's
getting very repetitive.