Re: [PATCH 1/2] ASoC: img-i2s-in: Add suspend / resume handling

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

 



On Thu, Oct 05, 2017 at 02:27:49PM +0100, Ed Blake wrote:
> On 04/10/17 11:37, Mark Brown wrote:

> > Why use the ASoC specific power management callbacks and not the system
> > ones?  It does also look a bit like there's a case for converting the
> > driver to regmap and using the regmap cache code.

> Looking at other drivers, most seemed to use the ASoC specific
> callbacks.  Are the system ones preferred?  If so I'll change it for v2.

Yes, look at newer drivers more here.  Unless you've a need to do some
ASoC specific stuff but that's not the case for this.

> > It does also look a bit like there's a case for converting the
> > driver to regmap and using the regmap cache code.

> Is it ok to do this in a future patch?

Well, the comment was there because you were open coding the regmap
cache sync code but it's not the end of the world to do it later.

Attachment: signature.asc
Description: PGP signature

_______________________________________________
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