On Wed, 21 Oct 2015, Mark Brown wrote: > On Wed, Oct 21, 2015 at 09:46:33AM +0100, Lee Jones wrote: > > On Tue, 20 Oct 2015, Andrew F. Davis wrote: > > > > I could change the core to be "tps65086-pmic", then call the regulator > > > driver "tps65086-regulator" if this works for you, this seems to be the > > > way new drivers name the regulator driver (max77843.c). > > > Yes, this is what I was thinking of. > > > ti,tps65086 is uninformative/undescriptive and tells me nothing. > > It is however the normal way we write compatible strings - the class > information would normaly go in the node name (ie, i2c@7000c000 or > whatever). I didn't say it hasn't been done before, just that I didn't like it for the aforementioned reasons. I can also find 1000's of compatible strings which do append "-<device_type>", so it's not exactly an unheard of practice. -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
