Re: [PATCH 15/20] hda: cs35l41: Remove unnecessary log

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

To: Mark Brown <broonie@xxxxxxxxxx>
Subject: Re: [PATCH 15/20] hda: cs35l41: Remove unnecessary log
From: Lucas tanure <tanureal@xxxxxxxxxxxxxxxxxxxxx>
Date: Fri, 4 Mar 2022 15:05:54 +0000
Cc: devicetree@xxxxxxxxxxxxxxx, alsa-devel@xxxxxxxxxxxxxxxx, patches@xxxxxxxxxxxxxxxxxxxxx, Takashi Iwai <tiwai@xxxxxxxx>, Liam Girdwood <lgirdwood@xxxxxxxxx>, Rob Herring <robh+dt@xxxxxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx
In-reply-to: <YiD/0ZMcSGVRTZxx@sirena.org.uk>
References: <20220303173059.269657-1-tanureal@opensource.cirrus.com> <20220303173059.269657-16-tanureal@opensource.cirrus.com> <YiD/0ZMcSGVRTZxx@sirena.org.uk>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.6.1

On 3/3/22 17:50, Mark Brown wrote:

On Thu, Mar 03, 2022 at 05:30:54PM +0000, Lucas Tanure wrote:

If regmap fails would fail during probe of the device.


I'm pretty sure circumstances could arise which would break things at
runtime.

Yes, so instead of not logging that, we can log the last regmap accessper switch case, instead of all of them.

References:
- [PATCH 00/20] Support external boost at CS35l41 ASoC driver
  - From: Lucas Tanure
- [PATCH 15/20] hda: cs35l41: Remove unnecessary log
  - From: Lucas Tanure
- Re: [PATCH 15/20] hda: cs35l41: Remove unnecessary log
  - From: Mark Brown

Prev by Date: [PATCH v3 17/17] ASoC: Intel: avs: Code loading over HDA
Next by Date: [PATCH v2 00/20] Support external boost at CS35l41 ASoC driver
Previous by thread: Re: [PATCH 15/20] hda: cs35l41: Remove unnecessary log
Next by thread: [PATCH 14/20] hda: cs35l41: Move boost config to initialization code
Index(es):
- Date
- Thread

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

Powered by Linux