Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755246Ab3HPSxC (ORCPT ); Fri, 16 Aug 2013 14:53:02 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:54388 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754934Ab3HPSw5 (ORCPT ); Fri, 16 Aug 2013 14:52:57 -0400 Date: Fri, 16 Aug 2013 11:52:56 -0700 From: Greg Kroah-Hartman To: Josh Cartwright Cc: Grant Likely , Rob Herring , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-arm-msm@vger.kernel.org, Sagar Dharia , Gilad Avidov , Michael Bohan Subject: Re: [PATCH RFC 2/3] spmi: Add MSM PMIC Arbiter SPMI controller Message-ID: <20130816185256.GC31510@kroah.com> References: <2b97e8deb412ae0087bffe227690a26ea9dd9bba.1376596224.git.joshc@codeaurora.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <2b97e8deb412ae0087bffe227690a26ea9dd9bba.1376596224.git.joshc@codeaurora.org> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 757 Lines: 23 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 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/