LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Connor McAdams <conmanx360@gmail.com>
To: Connor McAdams <conmanx360@gmail.com>
Cc: "Takashi Sakamoto" <o-takashi@sakamocchi.jp>,
	"Jaroslav Kysela" <perex@perex.cz>,
	"Takashi Iwai" <tiwai@suse.com>,
	"Jérémy Lefaure" <jeremy.lefaure@lse.epita.fr>,
	alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/9] R3Di and SBZ quirk entires + alt firmware loading
Date: Fri, 4 May 2018 00:30:14 -0400	[thread overview]
Message-ID: <CAM8Agx0s1Rg5bGCXP5gQftcfsqbQoCvZtNyg4fmUpNaJsix=-A@mail.gmail.com> (raw)
In-Reply-To: <1525407594-25644-1-git-send-email-conmanx360@gmail.com>

Sorry for sending these twice, I made a formatting mistake in the
first series, and they would not apply properly. Hopefully these do
not show up as spam because of this.

I went through and fixed them all individually and re-committed them,
but kept the same commit messages. I still have a lot to learn.

Thanks,
Connor

On Fri, May 4, 2018 at 12:19 AM, Connor McAdams <conmanx360@gmail.com> wrote:
> This patch adds PCI quirk ID's for the Sound Blaster Z and Recon3Di.
> Only the currently tested ID's have been added.
>
> This patch also adds the ability to load alternative firmwares for each
> card, the firmwares can be obtained from within the Windows driver.
> The Recon3Di uses "ctefx-r3di.bin" and the Sound Blaster Z uses
> "ctefx-sbz.bin". If the alternative firmware for the given quirk is not
> found, the original ctefx.bin will be used. This has been confirmed to
> work for both the R3Di and the SBZ.
>
> This patch also makes the character array *dirstr a const.
>
> Signed-off-by: Connor McAdams <conmanx360@gmail.com>
> ---
>  sound/pci/hda/patch_ca0132.c | 61 +++++++++++++++++++++++++++++++++++++++-----
>  1 file changed, 55 insertions(+), 6 deletions(-)
>
> diff --git a/sound/pci/hda/patch_ca0132.c b/sound/pci/hda/patch_ca0132.c
> index 768ea86..8346100 100644
> --- a/sound/pci/hda/patch_ca0132.c
> +++ b/sound/pci/hda/patch_ca0132.c
> @@ -72,12 +72,16 @@
>  #define SCP_GET    1
>
>  #define EFX_FILE   "ctefx.bin"
> +#define SBZ_EFX_FILE   "ctefx-sbz.bin"
> +#define R3DI_EFX_FILE  "ctefx-r3di.bin"
>
>  #ifdef CONFIG_SND_HDA_CODEC_CA0132_DSP
>  MODULE_FIRMWARE(EFX_FILE);
> +MODULE_FIRMWARE(SBZ_EFX_FILE);
> +MODULE_FIRMWARE(R3DI_EFX_FILE);
>  #endif
>
> -static char *dirstr[2] = { "Playback", "Capture" };
> +static const char *dirstr[2] = { "Playback", "Capture" };
>
>  enum {
>         SPEAKER_OUT,
> @@ -734,6 +738,7 @@ struct ca0132_spec {
>         unsigned int scp_resp_header;
>         unsigned int scp_resp_data[4];
>         unsigned int scp_resp_count;
> +       bool alt_firmware_present;
>
>         /* mixer and effects related */
>         unsigned char dmic_ctl;
> @@ -762,6 +767,8 @@ struct ca0132_spec {
>  enum {
>         QUIRK_NONE,
>         QUIRK_ALIENWARE,
> +       QUIRK_SBZ,
> +       QUIRK_R3DI,
>  };
>
>  static const struct hda_pintbl alienware_pincfgs[] = {
> @@ -782,6 +789,10 @@ static const struct snd_pci_quirk ca0132_quirks[] = {
>         SND_PCI_QUIRK(0x1028, 0x0685, "Alienware 15 2015", QUIRK_ALIENWARE),
>         SND_PCI_QUIRK(0x1028, 0x0688, "Alienware 17 2015", QUIRK_ALIENWARE),
>         SND_PCI_QUIRK(0x1028, 0x0708, "Alienware 15 R2 2016", QUIRK_ALIENWARE),
> +       SND_PCI_QUIRK(0x1102, 0x0010, "Sound Blaster Z", QUIRK_SBZ),
> +       SND_PCI_QUIRK(0x1102, 0x0023, "Sound Blaster Z", QUIRK_SBZ),
> +       SND_PCI_QUIRK(0x1458, 0xA016, "Recon3Di", QUIRK_R3DI),
> +       SND_PCI_QUIRK(0x1458, 0xA036, "Recon3Di", QUIRK_R3DI),
>         {}
>  };
>
> @@ -3207,7 +3218,7 @@ static int ca0132_select_out(struct hda_codec *codec)
>                                     pin_ctl & ~PIN_HP);
>                 /* enable speaker node */
>                 pin_ctl = snd_hda_codec_read(codec, spec->out_pins[0], 0,
> -                                            AC_VERB_GET_PIN_WIDGET_CONTROL, 0);
> +                               AC_VERB_GET_PIN_WIDGET_CONTROL, 0);
>                 snd_hda_set_pin_ctl(codec, spec->out_pins[0],
>                                     pin_ctl | PIN_OUT);
>         } else {
> @@ -4370,11 +4381,49 @@ static void ca0132_set_dsp_msr(struct hda_codec *codec, bool is96k)
>  static bool ca0132_download_dsp_images(struct hda_codec *codec)
>  {
>         bool dsp_loaded = false;
> +       struct ca0132_spec *spec = codec->spec;
>         const struct dsp_image_seg *dsp_os_image;
>         const struct firmware *fw_entry;
> -
> -       if (request_firmware(&fw_entry, EFX_FILE, codec->card->dev) != 0)
> -               return false;
> +       /*
> +        * Alternate firmwares for different variants. The Recon3Di apparently
> +        * can use the default firmware, but I'll leave the option in case
> +        * it needs it again.
> +        */
> +       switch (spec->quirk) {
> +       case QUIRK_SBZ:
> +               if (request_firmware(&fw_entry, SBZ_EFX_FILE,
> +                                       codec->card->dev) != 0) {
> +                       codec_dbg(codec, "SBZ alt firmware not detected. ");
> +                       spec->alt_firmware_present = false;
> +               } else {
> +                       codec_dbg(codec, "Sound Blaster Z firmware selected.");
> +                       spec->alt_firmware_present = true;
> +               }
> +               break;
> +       case QUIRK_R3DI:
> +               if (request_firmware(&fw_entry, R3DI_EFX_FILE,
> +                                       codec->card->dev) != 0) {
> +                       codec_dbg(codec, "Recon3Di alt firmware not detected.");
> +                       spec->alt_firmware_present = false;
> +               } else {
> +                       codec_dbg(codec, "Recon3Di firmware selected.");
> +                       spec->alt_firmware_present = true;
> +               }
> +               break;
> +       default:
> +               spec->alt_firmware_present = false;
> +               break;
> +       }
> +       /*
> +        * Use default ctefx.bin if no alt firmware is detected, or if none
> +        * exists for your particular codec.
> +        */
> +       if (!spec->alt_firmware_present) {
> +               codec_dbg(codec, "Default firmware selected.");
> +               if (request_firmware(&fw_entry, EFX_FILE,
> +                                       codec->card->dev) != 0)
> +                       return false;
> +       }
>
>         dsp_os_image = (struct dsp_image_seg *)(fw_entry->data);
>         if (dspload_image(codec, dsp_os_image, 0, 0, true, 0)) {
> @@ -4476,7 +4525,7 @@ static struct hda_verb ca0132_base_exit_verbs[] = {
>         {}
>  };
>
> -/* Other verbs tables.  Sends after DSP download. */
> +/* Other verbs tables. Sends after DSP download. */
>  static struct hda_verb ca0132_init_verbs0[] = {
>         /* chip init verbs */
>         {0x15, 0x70D, 0xF0},
> --
> 2.7.4
>

  parent reply	other threads:[~2018-05-04  4:30 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-05-04  4:19 Connor McAdams
2018-05-04  4:19 ` [PATCH 2/9] Add pincfg for SBZ + R3Di, add fp hp auto-detect Connor McAdams
2018-05-04  4:19 ` [PATCH 3/9] Add PCI region2 iomap for SBZ Connor McAdams
2018-05-04  4:19 ` [PATCH 4/9] Add extra exit functions for R3Di and SBZ Connor McAdams
2018-05-04  4:19 ` [PATCH 5/9] add/change helper " Connor McAdams
2018-05-04  4:19 ` [PATCH 6/9] add alt_select_in/out for R3Di + SBZ Connor McAdams
2018-05-04  4:19 ` [PATCH 7/9] Add DSP Volume set and New mixers for SBZ + R3Di Connor McAdams
2018-05-04  4:19 ` [PATCH 8/9] add ca0132_alt_set_vipsource Connor McAdams
2018-05-04  4:19 ` [PATCH 9/9] Add new control changes for SBZ + R3Di Connor McAdams
2018-05-04  4:30 ` Connor McAdams [this message]
2018-05-04  8:25   ` [PATCH 1/9] R3Di and SBZ quirk entires + alt firmware loading Takashi Sakamoto
2018-05-04  9:40 ` Takashi Iwai
  -- strict thread matches above, loose matches on Subject: below --
2018-05-03 21:42 Connor McAdams

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAM8Agx0s1Rg5bGCXP5gQftcfsqbQoCvZtNyg4fmUpNaJsix=-A@mail.gmail.com' \
    --to=conmanx360@gmail.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=jeremy.lefaure@lse.epita.fr \
    --cc=linux-kernel@vger.kernel.org \
    --cc=o-takashi@sakamocchi.jp \
    --cc=perex@perex.cz \
    --cc=tiwai@suse.com \
    --subject='Re: [PATCH 1/9] R3Di and SBZ quirk entires + alt firmware loading' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).