From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1751851AbcGORFe (ORCPT <rfc822;w@1wt.eu>);
	Fri, 15 Jul 2016 13:05:34 -0400
Received: from bhuna.collabora.co.uk ([46.235.227.227]:45234 "EHLO
	bhuna.collabora.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1751241AbcGORFb (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Fri, 15 Jul 2016 13:05:31 -0400
Message-ID: <1468602325.3819.6.camel@ndufresne.ca>
Subject: Re: [RFT PATCH v2] [media] exynos4-is: Fix
 fimc_is_parse_sensor_config() nodes handling
From: Nicolas Dufresne <nicolas@ndufresne.ca>
Reply-To: nicolas@ndufresne.ca
To: Javier Martinez Canillas <javier@osg.samsung.com>,
        Sylwester Nawrocki <s.nawrocki@samsung.com>
Cc: linux-kernel@vger.kernel.org, linux-samsung-soc@vger.kernel.org,
        Krzysztof Kozlowski <k.kozlowski@samsung.com>,
        Mauro Carvalho Chehab <mchehab@osg.samsung.com>,
        Kyungmin Park <kyungmin.park@samsung.com>,
        Kukjin Kim <kgene@kernel.org>,
        Andreas =?ISO-8859-1?Q?F=E4rber?= <afaerber@suse.de>,
        linux-media@vger.kernel.org
Date: Fri, 15 Jul 2016 13:05:25 -0400
In-Reply-To: <1468601608.3819.4.camel@ndufresne.ca>
References: <1458780100-8865-1-git-send-email-javier@osg.samsung.com>
	 <5718D3DC.20004@osg.samsung.com> <1468601608.3819.4.camel@ndufresne.ca>
Content-Type: multipart/signed; micalg="pgp-sha1"; protocol="application/pgp-signature";
	boundary="=-ykrAIx091FSnLflAVduC"
X-Mailer: Evolution 3.20.3 (3.20.3-1.fc24) 
Mime-Version: 1.0
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


--=-ykrAIx091FSnLflAVduC
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Le vendredi 15 juillet 2016 =C3=A0 12:53 -0400, Nicolas Dufresne a =C3=A9cr=
it=C2=A0:
> Le jeudi 21 avril 2016 =C3=A0 09:21 -0400, Javier Martinez Canillas a
> =C3=A9crit=C2=A0:
> > Hello Sylwester,
> >=20
> > On 03/23/2016 08:41 PM, Javier Martinez Canillas wrote:
> > > The same struct device_node * is used for looking up the I2C
> > > sensor, OF
> > > graph endpoint and port. So the reference count is incremented
> > > but
> > > not
> > > decremented for the endpoint and port nodes.
> > >=20
> > > Fix this by having separate pointers for each node looked up.
> > >=20
> > > Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
> > >=20
> >=20
> > Any comments about this patch?
>=20
> Tested-by: Nicolas Dufresne <nicoas.dufresne@collabora.com>

There a typo here.

Tested-by: Nicolas Dufresne <nicolas.dufresne@collabora.com>

>=20
> Note: I could not verify that leak is gone, but I could verify that
> this driver is still working properly after the change.
>=20
> >=20
> > Best regards,
--=-ykrAIx091FSnLflAVduC
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

iEYEABECAAYFAleJF9UACgkQcVMCLawGqBy+MgCg22w0W7Fa7MNuPkTaTmEU65r9
sXEAoMorG212uIxsUwagbNdzQWPB7Bwo
=DvkU
-----END PGP SIGNATURE-----

--=-ykrAIx091FSnLflAVduC--