[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAFv8Nw+bcVSugD5hEBj7nQSg3VGj2E+ROpzf+5LxexEmFoX=oA@mail.gmail.com>
Date: Tue, 19 Dec 2017 15:38:30 +0800
From: Cheng-yi Chiang <cychiang@...omium.org>
To: Brian Norris <briannorris@...omium.org>
Cc: Mark Brown <broonie@...nel.org>,
Oder Chiou <oder_chiou@...ltek.com>,
Bard Liao <bardliao@...ltek.com>,
Liam Girdwood <lgirdwood@...il.com>,
Jaroslav Kysela <perex@...ex.cz>,
Takashi Iwai <tiwai@...e.com>, alsa-devel@...a-project.org,
linux-kernel@...r.kernel.org, Matthias Kaehlcke <mka@...omium.org>,
Jeffy Chen <jeffy.chen@...k-chips.com>,
linux-rockchip@...ts.infradead.org
Subject: Re: [PATCH for-4.15] ASoC: rt5514: don't assume rt5514 component was "attached"
Hi Brian, I am sorry for not using the plain text mode in the previous mail.
I agree with you on other points.
On Tue, Dec 19, 2017 at 1:42 AM, Brian Norris <briannorris@...omium.org> wrote:
> Hi!
>
> (By the way, your mail is HTML and likely will get rejected by many
> mailing lists and/or people reading these mailing lists.)
>
> On Mon, Dec 18, 2017 at 12:23:18PM +0800, Cheng-yi Chiang wrote:
>> Hi Brian,
>> Oder has posted the same fix : [1]https://patchwork.kernel.org/
>> patch/10066257/ and it has been applied.
>
> OK cool. Obviously I'm biased, but I prefer mine, as it has less
> needless whitespace, and is appropriately documented ;) But it should be
> a fine replacement.
>
>> Perhaps you can cherry-pick it to v4.15 ?
>
> I have no say in that; that would be up to Mark, I think. It's most
> certainly a regression in 4.15-rc1, so IMO it should be given a proper
> 'Fixes: e9c50aa6bd39 ("ASoC: rt5514-spi: check irq status to schedule
> data copy in resume function")' tag and sent for 4.15, not 4.16.
>
> Brian
Powered by blists - more mailing lists