: Re: [PATCH v3] ASoC: rt286: fix headphone click/crack noise on Dell
> XPS 9343 I2S mode
>
> > If HPO is already muted as what we expected, it means "HPO L" and "HPO R"
> > work properly. And there is no reason we create an event to do the same
> > thing
[snip]
> Let me explain it in more detail. rt5670 need to set a serious of
> registers to prevent the pop noise of powering up/down muting/
> unmuting headphone. That's what rt5670_hp_event() does. But,
> what rt286_hp_power_event do is only mute/unmute headphone
> which is done by "HPO L" and "HP
: Re: [PATCH v3] ASoC: rt286: fix headphone click/crack noise on Dell
> XPS 9343 I2S mode
>
> >> What I really want to do is something rt5670's rt5670_hp_event(),
> >> maybe autodisable is not enough sometimes?
> >
> > It is different. rt5670_hp_event() is doin
a-project.org;
>> linux-kernel@vger.kernel.org
>> Subject: Re: [PATCH v3] ASoC: rt286: fix headphone click/crack noise on Dell
>> XPS 9343 I2S mode
>> >
>> >> I had the same thought originally, but printk under each case suggests
>> >> otherwise
; alsa-de...@alsa-project.org; linux-kernel@vger.kernel.org
>> Subject: Re: [PATCH v3] ASoC: rt286: fix headphone click/crack noise on Dell
>> XPS 9343 I2S mode
>> >>
>> >> + switch (event) {
>> >> + case SND_SOC_DAPM_PRE
v3] ASoC: rt286: fix headphone click/crack noise on Dell
> XPS 9343 I2S mode
> >
> >> I had the same thought originally, but printk under each case suggests
> >> otherwise - _POST_PMU is triggered not right after _PRE_PMU but right
> >> before _PRE_PMD.
> >
: Re: [PATCH v3] ASoC: rt286: fix headphone click/crack noise on Dell
> XPS 9343 I2S mode
> >>
> >> + switch (event) {
> >> + case SND_SOC_DAPM_PRE_PMD:
> >> + case SND_SOC_DAPM_POST_PMD:
> >> + case SND_SOC_DAPM_POST_PMU:
> >>
On Tue, Mar 21, 2017 at 11:07 AM, Bard Liao wrote:
>> -Original Message-
>> From: Kai-Heng Feng [mailto:kai.heng.f...@canonical.com]
>> Sent: Monday, March 20, 2017 11:59 AM
>> To: broo...@kernel.org
>> Cc: lgirdw...@gmail.com; Bard Liao; Oder Chiou;
>> alsa-de...@alsa-project.org; linux-k
On Tue, Mar 21, 2017 at 1:26 AM, Mark Brown wrote:
> On Tue, Mar 21, 2017 at 12:23:53AM +0800, Kai-Heng Feng wrote:
>> On Tue, Mar 21, 2017 at 12:06 AM, Mark Brown wrote:
>
>> > As it says there "...and inserted automatically following the three dash
>> > line".
>
>> I saw iteration changelog in
> -Original Message-
> From: Kai-Heng Feng [mailto:kai.heng.f...@canonical.com]
> Sent: Monday, March 20, 2017 11:59 AM
> To: broo...@kernel.org
> Cc: lgirdw...@gmail.com; Bard Liao; Oder Chiou;
> alsa-de...@alsa-project.org; linux-kernel@vger.kernel.org; Kai-Heng Feng
> Subject: [PATCH v3]
On Tue, Mar 21, 2017 at 12:23:53AM +0800, Kai-Heng Feng wrote:
> On Tue, Mar 21, 2017 at 12:06 AM, Mark Brown wrote:
> > As it says there "...and inserted automatically following the three dash
> > line".
> I saw iteration changelog in git log all over the place, maybe add a
> rule section for e
On Tue, Mar 21, 2017 at 12:06 AM, Mark Brown wrote:
> On Mon, Mar 20, 2017 at 03:46:13PM +, Kai-Heng Feng wrote:
>> On Mon, Mar 20, 2017 at 11:08 PM Mark Brown wrote:
>
>> > As covered in SubmittingPatches this should come after the ---, it
>> > doesn't need to end up in the changelogs.
>
>>
On Mon, Mar 20, 2017 at 03:46:13PM +, Kai-Heng Feng wrote:
> On Mon, Mar 20, 2017 at 11:08 PM Mark Brown wrote:
> > As covered in SubmittingPatches this should come after the ---, it
> > doesn't need to end up in the changelogs.
> Do you mean
> https://github.com/git/git/blob/master/Document
On Mon, Mar 20, 2017 at 11:58:31AM +0800, Kai-Heng Feng wrote:
> v3:
> Implicit conversion instead of tenary operator.
>
> v2:
> Use 'HP Power' instead of individual power events.
As covered in SubmittingPatches this should come after the ---, it
doesn't need to end up in the changelogs.
> +
14 matches
Mail list logo