From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: hp_pin was NULL value Date: Wed, 09 Jan 2019 10:42:42 +0100 Message-ID: References: <6FAB7C47BCF00940BB0999A99BE3547A18420E8B@RTITMBSV02.realtek.com.tw> Mime-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mx1.suse.de (mx2.suse.de [195.135.220.15]) by alsa0.perex.cz (Postfix) with ESMTP id 9730B2665E5 for ; Wed, 9 Jan 2019 10:42:43 +0100 (CET) In-Reply-To: <6FAB7C47BCF00940BB0999A99BE3547A18420E8B@RTITMBSV02.realtek.com.tw> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: Kailang Cc: " (alsa-devel@alsa-project.org)" List-Id: alsa-devel@alsa-project.org On Wed, 09 Jan 2019 10:31:33 +0100, Kailang wrote: > > Hi Takashi, > > Could I move the alc294_hp_init(codec) to below line. > Because hp_pin = spec->gen.autocfg.hp_pins[0] was null value when alc294_hp_init(codec) at original line. > Or move alc269_parse_auto_config() upward. It looks OK to me. But this made me wonder whether we don't need to call this function at resume as well? Currently it's called only at probing. thanks, Takashi