From mboxrd@z Thu Jan 1 00:00:00 1970 From: Brian Norris Subject: Re: [PATCH for-4.15] ASoC: rt5514: don't assume rt5514 component was "attached" Date: Mon, 18 Dec 2017 09:42:36 -0800 Message-ID: <20171218174235.GA113663@google.com> References: <20171216011230.107527-1-briannorris@chromium.org> <20171216031738.GA220575@google.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: 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: Cheng-yi Chiang Cc: Oder Chiou , alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org, Jeffy Chen , Takashi Iwai , Liam Girdwood , linux-rockchip@lists.infradead.org, Mark Brown , Bard Liao , Matthias Kaehlcke List-Id: linux-rockchip.vger.kernel.org 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