From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Date: Tue, 29 Oct 2013 23:55:06 +0000 Subject: Re: [PATCH v7 1/5] omap3isp: Modify clocks registration to avoid circular references Message-Id: <15201376.PVPE5NrGdO@avalon> MIME-Version: 1 Content-Type: multipart/mixed; boundary="nextPart24267327.NWieDcb5ZA" List-Id: References: <1383076268-8984-1-git-send-email-s.nawrocki@samsung.com> <16467881.81yEf9zq68@avalon> <20131029232837.GB2266@earth.universe> In-Reply-To: <20131029232837.GB2266@earth.universe> To: linux-arm-kernel@lists.infradead.org --nextPart24267327.NWieDcb5ZA Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Hi Sebastian, On Wednesday 30 October 2013 00:28:39 Sebastian Reichel wrote: > On Tue, Oct 29, 2013 at 11:28:37PM +0100, Laurent Pinchart wrote: > > On Tuesday 29 October 2013 20:51:04 Sylwester Nawrocki wrote: > > > The clock core code is going to be modified so clk_get() takes > > > reference on the clock provider module. Until the potential circular > > > reference issue is properly addressed, we pass NULL as as the first > > > argument to clk_register(), in order to disallow sub-devices taking > > > a reference on the ISP module back trough clk_get(). This should > > > prevent locking the modules in memory. > > > > > > Cc: Laurent Pinchart > > > Signed-off-by: Sylwester Nawrocki > > > Signed-off-by: Kyungmin Park > > > > Acked-by: Laurent Pinchart > > > > Do you plan to push this to mainline as part of this patch series ? I > > don't have pending patches for the omap3isp that would conflict with this > > patch, so that would be fine with me. > > I plan to add support for DT to omap3isp + ADP1653 shortly. I have > not yet started to work on this, but expect to send some first RFC > patches in November. That's very nice to hear ! And thanks for the heads up, I would have started working on it shortly. -- Regards, Laurent Pinchart --nextPart24267327.NWieDcb5ZA Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part. Content-Transfer-Encoding: 7Bit -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.22 (GNU/Linux) iQEcBAABAgAGBQJScErfAAoJEIkPb2GL7hl1IekH/0CY1YWUNcUAQpUr0aeJoBOl 1zcvSIjk2nF0jgf+ayjndSr16epbblODJh5aFcYz+VCweP/aA+xArg5p6tmDO4f9 Ijk76QN6GjdsiEMvRYnARqysDJEFPvepurpQDrN3YNFmH56tsFytCzBGSzA4qIQy 4+ouLUFz6q8lOkSZ39F51NYOOw8XyX66y2XfV99psUEnUlaOyizyxocDmmRxUg2s 9gq6I7TFNTgipo9zbGKxCCwGbxacUPnc2cEg7SuGQuHitwkPSL9yTbNoScKDwoeb tTUtPKN9JZzeGmbtvHPPt8GILQsejskvW3Iaj5lp8Tvr8hoOqd8AaesTfha6P6U= =dhkA -----END PGP SIGNATURE----- --nextPart24267327.NWieDcb5ZA--