lkml.org 
[lkml]   [2017]   [Apr]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 2/2] ASoC: hisilicon: Minor fixups to hi6210 i2s audio driver
On Mon, Apr 10, 2017 at 12:35:12PM -0700, John Stultz wrote:

> This patch adds a few extra error returns in cases that
> shouldn't happen, some style nits adding breaks to final
> default cases in switch statements, and tweaks to use
> devm_ variant of snd_soc_register_component.

Please don't make multiple unrelated changes in a single patch, it's bad
practice which can slow down the bits of the patch which are OK and
makes it harder to review if the changes match up with the changelog.

> static int hi6210_i2s_remove(struct platform_device *pdev)
> {
> snd_soc_unregister_component(&pdev->dev);

Since you converted to devm_ this is now going to be a double free.
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2017-04-11 20:35    [W:0.924 / U:0.296 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site