| Message ID | 20220409091315.1663410-1-tanureal@opensource.cirrus.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | Support external boost at CS35l41 ASoC driver | expand |
On Sat, 09 Apr 2022 11:12:59 +0200, Lucas Tanure wrote: > > Move the support for CS35L41 external boost to its shared library > for ASoC use. > This move resulted in cs35l41_hda_reg_sequence being removed, > and its steps were broken down into regmap writes or functions > from the library. And hardware configuration struct was unified > for its use in the shared lib. > While at it, some minor bugs were found and fixed it. > > v6 changelog: > - Rebased on top of Linux Next with community patches for CS35L41 > - Document patch acked by Charles Keepax > > v5 changelog: > - Fixed wrong indentation at Documentation patch > - Use of consistent prefix > > v4 changelog: > - Separated GPIO 1 and 2 function enums > > v3 changelog: > - Remove patches already accepted > - Improved logic in documentation patch > - Documentation patch goes before its code > - Fixed missing Signed-off-by > - Fixed subject for HDA patches > > v2 changelog: > - Instead of removing the log, playback actions will log the last regmap access. > - Documentation patch with the correct subject line and fixed bug reported by Rob Herring on the > provided example. > > Previous versions: > v1: https://lkml.org/lkml/2022/3/3/759 > v2: https://lkml.org/lkml/2022/3/4/743 > v3: https://lkml.org/lkml/2022/3/8/975 > v4: https://lkml.org/lkml/2022/3/17/267 > v5: https://lkml.org/lkml/2022/3/22/696 > > David Rhodes (1): > ASoC: cs35l41: Document CS35l41 External Boost > > Lucas Tanure (15): > ALSA: cs35l41: Unify hardware configuration > ALSA: cs35l41: Check hw_config before using it > ALSA: cs35l41: Move cs35l41_gpio_config to shared lib > ALSA: hda: cs35l41: Fix I2S params comments > ALSA: hda: cs35l41: Always configure the DAI > ALSA: hda: cs35l41: Add Boost type flag > ALSA: hda: cs35l41: Put the device into safe mode for external boost > ALSA: hda: cs35l41: Mute the device before shutdown > ALSA: cs35l41: Enable Internal Boost in shared lib > ALSA: hda: cs35l41: Move boost config to initialization code > ALSA: hda: cs35l41: Remove cs35l41_hda_reg_sequence struct > ALSA: hda: cs35l41: Reorganize log for playback actions > ALSA: hda: cs35l41: Handle all external boost setups the same way > ALSA: hda: cs35l41: Move external boost handling to lib for ASoC use > ASoC: cs35l41: Support external boost If it's OK for Mark, I'm going to merge those to my tree for 5.18. thanks, Takashi
On Sat, 09 Apr 2022 11:12:59 +0200, Lucas Tanure wrote: > > Move the support for CS35L41 external boost to its shared library > for ASoC use. > This move resulted in cs35l41_hda_reg_sequence being removed, > and its steps were broken down into regmap writes or functions > from the library. And hardware configuration struct was unified > for its use in the shared lib. > While at it, some minor bugs were found and fixed it. The patch series seem inapplicable via git am. Could you check the setup of your mailer? thanks, Takashi
On 4/13/22 8:51 AM, Takashi Iwai <tiwai@suse.de> wrote: > On Sat, 09 Apr 2022 11:12:59 +0200, > Lucas Tanure wrote: > > > > Move the support for CS35L41 external boost to its shared library > > for ASoC use. > > This move resulted in cs35l41_hda_reg_sequence being removed, > > and its steps were broken down into regmap writes or functions > > from the library. And hardware configuration struct was unified > > for its use in the shared lib. > > While at it, some minor bugs were found and fixed it. > > The patch series seem inapplicable via git am. > Could you check the setup of your mailer? > > > thanks, > > Takashi > Hi, Could you explain how you apply this series? So I can reproduce it and see what's going on. To apply a series of patches I usually: 1 - Find the patch series on https://patchwork.kernel.org/ 2 - Click the series button on the right top corner, which will give me the single patch file with all series changes 3 - git am ~/Downloads/Support-external-boost-at-CS35l41-ASoC-driver.patch And that works fine for me here. Thanks Lucas
On Wed, 13 Apr 2022 10:10:16 +0200, <tanureal@opensource.cirrus.com> wrote: > > On 4/13/22 8:51 AM, Takashi Iwai <tiwai@suse.de> wrote: > > On Sat, 09 Apr 2022 11:12:59 +0200, > > Lucas Tanure wrote: > > > > > > Move the support for CS35L41 external boost to its shared library > > > for ASoC use. > > > This move resulted in cs35l41_hda_reg_sequence being removed, > > > and its steps were broken down into regmap writes or functions > > > from the library. And hardware configuration struct was unified > > > for its use in the shared lib. > > > While at it, some minor bugs were found and fixed it. > > > > The patch series seem inapplicable via git am. > > Could you check the setup of your mailer? > > > > > > thanks, > > > > Takashi > > > Hi, > Could you explain how you apply this series? So I can reproduce it and see what's going on. > To apply a series of patches I usually: > > 1 - Find the patch series on https://patchwork.kernel.org/ > 2 - Click the series button on the right top corner, which will give me the single patch file with all series changes > 3 - git am ~/Downloads/Support-external-boost-at-CS35l41-ASoC-driver.patch I tried b4 am and git am. Takashi
On Wed, 13 Apr 2022 10:12:25 +0200, Takashi Iwai wrote: > > On Wed, 13 Apr 2022 10:10:16 +0200, > <tanureal@opensource.cirrus.com> wrote: > > > > On 4/13/22 8:51 AM, Takashi Iwai <tiwai@suse.de> wrote: > > > On Sat, 09 Apr 2022 11:12:59 +0200, > > > Lucas Tanure wrote: > > > > > > > > Move the support for CS35L41 external boost to its shared library > > > > for ASoC use. > > > > This move resulted in cs35l41_hda_reg_sequence being removed, > > > > and its steps were broken down into regmap writes or functions > > > > from the library. And hardware configuration struct was unified > > > > for its use in the shared lib. > > > > While at it, some minor bugs were found and fixed it. > > > > > > The patch series seem inapplicable via git am. > > > Could you check the setup of your mailer? > > > > > > > > > thanks, > > > > > > Takashi > > > > > Hi, > > Could you explain how you apply this series? So I can reproduce it and see what's going on. > > To apply a series of patches I usually: > > > > 1 - Find the patch series on https://patchwork.kernel.org/ > > 2 - Click the series button on the right top corner, which will give me the single patch file with all series changes > > 3 - git am ~/Downloads/Support-external-boost-at-CS35l41-ASoC-driver.patch > > I tried b4 am and git am. Maybe your patch base isn't aligned with the current upstream. Could you rebase to 5.18-rc2? Takashi
On Wed, 13 Apr 2022 10:21:39 +0200, Takashi Iwai wrote: > > On Wed, 13 Apr 2022 10:12:25 +0200, > Takashi Iwai wrote: > > > > On Wed, 13 Apr 2022 10:10:16 +0200, > > <tanureal@opensource.cirrus.com> wrote: > > > > > > On 4/13/22 8:51 AM, Takashi Iwai <tiwai@suse.de> wrote: > > > > On Sat, 09 Apr 2022 11:12:59 +0200, > > > > Lucas Tanure wrote: > > > > > > > > > > Move the support for CS35L41 external boost to its shared library > > > > > for ASoC use. > > > > > This move resulted in cs35l41_hda_reg_sequence being removed, > > > > > and its steps were broken down into regmap writes or functions > > > > > from the library. And hardware configuration struct was unified > > > > > for its use in the shared lib. > > > > > While at it, some minor bugs were found and fixed it. > > > > > > > > The patch series seem inapplicable via git am. > > > > Could you check the setup of your mailer? > > > > > > > > > > > > thanks, > > > > > > > > Takashi > > > > > > > Hi, > > > Could you explain how you apply this series? So I can reproduce it and see what's going on. > > > To apply a series of patches I usually: > > > > > > 1 - Find the patch series on https://patchwork.kernel.org/ > > > 2 - Click the series button on the right top corner, which will give me the single patch file with all series changes > > > 3 - git am ~/Downloads/Support-external-boost-at-CS35l41-ASoC-driver.patch > > > > I tried b4 am and git am. > > Maybe your patch base isn't aligned with the current upstream. > Could you rebase to 5.18-rc2? ... and don't forget to pick up Mark's Acks. thanks, Takashi