On Wed, Jan 04, 2017 at 03:22:12PM +0300, Alexandrov Stansilav wrote: This looks okay, but title is same for both of the patches. It would be better to describe the change you are doing and not the outcome. so a better title could be: ASoC: rt5640: Add "10EC3276" ACPI ID and in log say: HP Pavilion x2 10-p000 comes with rt5640 with ACPI ID "10EC3276" so add that. > Signed-off-by: Alexandrov Stansilav <neko@xxxxxx> > --- > sound/soc/codecs/rt5640.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/sound/soc/codecs/rt5640.c b/sound/soc/codecs/rt5640.c > index e29a6de..b857a71 100644 > --- a/sound/soc/codecs/rt5640.c > +++ b/sound/soc/codecs/rt5640.c > @@ -2313,6 +2313,7 @@ MODULE_DEVICE_TABLE(of, rt5640_of_match); > #ifdef CONFIG_ACPI > static const struct acpi_device_id rt5640_acpi_match[] = { > { "INT33CA", 0 }, > + { "10EC3276", 0 }, > { "10EC5640", 0 }, > { "10EC5642", 0 }, > { "INTCCFFD", 0 }, > -- > 2.10.2 > -- ~Vinod _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel