Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752064AbbHaF1f (ORCPT ); Mon, 31 Aug 2015 01:27:35 -0400 Received: from mail-ob0-f175.google.com ([209.85.214.175]:36503 "EHLO mail-ob0-f175.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750908AbbHaF1e (ORCPT ); Mon, 31 Aug 2015 01:27:34 -0400 MIME-Version: 1.0 In-Reply-To: <20150818115645.GA3345@localhost> References: <20150818115645.GA3345@localhost> From: Alexandre Courbot Date: Mon, 31 Aug 2015 14:27:14 +0900 Message-ID: Subject: Re: [PATCH] Revert part of "ASoC: tegra: Convert to managed resources" To: Vaishali Thakkar Cc: Mark Brown , Jaroslav Kysela , Takashi Iwai , Liam Girdwood , Stephen Warren , Wolfram Sang , "alsa-devel@alsa-project.org" , "linux-tegra@vger.kernel.org" , Linux Kernel Mailing List Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 838 Lines: 19 On Tue, Aug 18, 2015 at 8:56 PM, Vaishali Thakkar wrote: > Revert the problematic part of commit 470805eb9f31 ("ASoC: tegra: > Convert to managed resources"). Before this commit, PM cleanup was > performed after the component was unregistered. But returning > directly will skip PM cleanup. So, to be on safe side it is better > to use snd_soc_register_component instead of > devm_snd_soc_register_component. > > Signed-off-by: Vaishali Thakkar I'm probably late for the party, but FWIW: Reviewed-by: Alexandre Courbot -- 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/