Re: [PATCH] ASoc: hdmi-codec: remove HDMI device unregister

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Vincent

> >> While unregistering the hdmi-codec, the hdmi device list must be
> >> cleaned up. It avoids kernel page fault when registering again the
> >> hdmi-codec.
> >>
> >> Cc: Liam Girdwood <lgirdwood@xxxxxxxxx>
> >> Cc: Mark Brown <broonie@xxxxxxxxxx>
> >> Cc: Jaroslav Kysela <perex@xxxxxxxx>
> >> Cc: Takashi Iwai <tiwai@xxxxxxxx>
> >> Cc: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx>
> >> Cc: Jyri Sarha <jsarha@xxxxxx>
> >> Cc: Arnaud Pouliquen <arnaud.pouliquen@xxxxxx>
> >> Cc: Philipp Zabel <p.zabel@xxxxxxxxxxxxxx>
> >>
> >> Signed-off-by: Vincent Abriou <vincent.abriou@xxxxxx>
> >> ---
> >
> > Thank you about this patch.
> > I added original "hdmi_device_list" to this driver for DW-HDMI device purpose.
> > But, actually, 1) DW-HDMI binding method was exchanged, 2) I assumed it
> > will not be exchanged, and 3) this patch was accepted under such
> > (wrong) assumption.
> >
> > Thus, this "hdmi_device_list" is no longer needed.
> > So, I'm planing to remove 9731f82d60166a19af6914f998092bbd1560f783
> > ("ASoC: hdmi-codec: enable multi probe for same device")
> > and its related patch soon.
> >
> > Can you agree about it ?
> >
> 
> I am fine with you proposal.
> Can you keep Arnaud Pouliquen and myself in the loop of your new patche 
> series so that we are able to test it on our side.

Thanks.
Above remove patch is one of my next patch-set which is
based on my current OF-graph sound card patch-set.
Thus, I'm waiting its result.

But above removing patch itself is just single patch.
So, I will post it if OF-graph patch-set progress was too late.

Best regards
---
Kuninori Morimoto
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux