From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter Ujfalusi Subject: Re: Use of of_parse_phandle()/of_node_put() Date: Thu, 14 Feb 2013 10:26:41 +0100 Message-ID: <511CADD1.9060104@ti.com> References: <511BB6C2.3060105@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: Sender: linux-omap-owner@vger.kernel.org To: Anil Kumar Cc: "Kumar, Anil" , Mark Brown , Liam Girdwood , "jarkko.nikula@bitmer.com" , "devicetree-discuss@lists.ozlabs.org" , "linux-omap@vger.kernel.org" , Alsa Devel List List-Id: devicetree@vger.kernel.org On 02/13/2013 06:02 PM, Anil Kumar wrote: > ------------8--------------- > diff --git a/sound/soc/soc-core.c b/sound/soc/soc-core.c > index b7e84a7..9000f4a 100644 > --- a/sound/soc/soc-core.c > +++ b/sound/soc/soc-core.c > @@ -1044,6 +1044,13 @@ static void soc_remove_dai_links(struct > snd_soc_card *card) > soc_remove_link_dais(card, dai, order); > } >=20 > + /* release cpu_of_node */ > + if(card->dai_link) { > + int i; > + for(i =3D 0; i < card->num_links; i++) > + of_node_put(card->dai_link[i].cpu_of_node); and the same for codec_of_node and platform_of_node for that matter. Mark: what do you think? Does it make sense to do this in the core or s= hould we let the machine drivers to take care of this? > + } > + >=20 --=20 P=E9ter -- To unsubscribe from this list: send the line "unsubscribe linux-omap" i= n the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html