[PATCH RFC 2/3] spmi: Add MSM PMIC Arbiter SPMI controller

Greg Kroah-Hartman gregkh at linuxfoundation.org
Fri Aug 16 14:52:56 EDT 2013


On Fri, Aug 09, 2013 at 01:37:07PM -0700, Josh Cartwright wrote:
> +struct spmi_pmic_arb_dev {
> +	struct spmi_controller	controller;
> +	struct device		*dev;
> +	struct device		*slave;

This is problematic.

Why do you have the driver "own" the controller?  What is dev for,
there's already a struct device within the controller.  Same for slave,
what is it?

You have 3 struct devices here, which one controls the lifecycle of the
object (hint, I know the answer, but I think it's wrong...)

thanks,

greg k-h



More information about the linux-arm-kernel mailing list