All of lore.kernel.org
 help / color / mirror / Atom feed
* Re: [PATCH] ALSA: oxygen - Fix snd_oxygen module not loading for some (new?) Xonar DG SI cards.
  2017-03-29 21:13 [PATCH] ALSA: oxygen - Fix snd_oxygen module not loading for some (new?) Xonar DG SI cards Eugene Ganeev
@ 2017-03-29 18:56   ` Clemens Ladisch
  0 siblings, 0 replies; 5+ messages in thread
From: Clemens Ladisch @ 2017-03-29 18:56 UTC (permalink / raw)
  To: Eugene Ganeev; +Cc: Jaroslav Kysela, Takashi Iwai, alsa-devel, linux-kernel

Eugene Ganeev wrote:
> My Xonar DG SI card is showing up in lspci but no module is loaded for
> it.
>
> The patch just adds a new value with card's PCI ID to oxygen_ids array.

Is the hardware identical?  Do all the inputs and outputs work?

> Signed-off-by: Eugene Ganeev <easymodofrf@gmail.com>
> ---
>  sound/pci/oxygen/oxygen.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/sound/pci/oxygen/oxygen.c b/sound/pci/oxygen/oxygen.c
> index ada6c256378e..99ba0354d4cc 100644
> --- a/sound/pci/oxygen/oxygen.c
> +++ b/sound/pci/oxygen/oxygen.c
> @@ -110,6 +110,7 @@ static DEFINE_PCI_DEVICE_TABLE(oxygen_ids) = {
>  	{ OXYGEN_PCI_SUBID(0x1a58, 0x0910), .driver_data = MODEL_CMEDIA_REF },
>  	/* Asus Xonar DG */
>  	{ OXYGEN_PCI_SUBID(0x1043, 0x8467), .driver_data = MODEL_XONAR_DG },
> +	{ OXYGEN_PCI_SUBID(0x1043, 0x855e), .driver_data = MODEL_XONAR_DG },

Please add the correct name to the names[] array.


Regards.
Clemens

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] ALSA: oxygen - Fix snd_oxygen module not loading for some (new?) Xonar DG SI cards.
@ 2017-03-29 18:56   ` Clemens Ladisch
  0 siblings, 0 replies; 5+ messages in thread
From: Clemens Ladisch @ 2017-03-29 18:56 UTC (permalink / raw)
  To: Eugene Ganeev; +Cc: linux-kernel, alsa-devel, Takashi Iwai

Eugene Ganeev wrote:
> My Xonar DG SI card is showing up in lspci but no module is loaded for
> it.
>
> The patch just adds a new value with card's PCI ID to oxygen_ids array.

Is the hardware identical?  Do all the inputs and outputs work?

> Signed-off-by: Eugene Ganeev <easymodofrf@gmail.com>
> ---
>  sound/pci/oxygen/oxygen.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/sound/pci/oxygen/oxygen.c b/sound/pci/oxygen/oxygen.c
> index ada6c256378e..99ba0354d4cc 100644
> --- a/sound/pci/oxygen/oxygen.c
> +++ b/sound/pci/oxygen/oxygen.c
> @@ -110,6 +110,7 @@ static DEFINE_PCI_DEVICE_TABLE(oxygen_ids) = {
>  	{ OXYGEN_PCI_SUBID(0x1a58, 0x0910), .driver_data = MODEL_CMEDIA_REF },
>  	/* Asus Xonar DG */
>  	{ OXYGEN_PCI_SUBID(0x1043, 0x8467), .driver_data = MODEL_XONAR_DG },
> +	{ OXYGEN_PCI_SUBID(0x1043, 0x855e), .driver_data = MODEL_XONAR_DG },

Please add the correct name to the names[] array.


Regards.
Clemens

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH] ALSA: oxygen - Fix snd_oxygen module not loading for some (new?) Xonar DG SI cards.
@ 2017-03-29 21:13 Eugene Ganeev
  2017-03-29 18:56   ` Clemens Ladisch
  0 siblings, 1 reply; 5+ messages in thread
From: Eugene Ganeev @ 2017-03-29 21:13 UTC (permalink / raw)
  To: Clemens Ladisch; +Cc: Jaroslav Kysela, Takashi Iwai, alsa-devel, linux-kernel

My Xonar DG SI card is showing up in lspci but no module is loaded for
it.

The patch just adds a new value with card's PCI ID to oxygen_ids array.

Signed-off-by: Eugene Ganeev <easymodofrf@gmail.com>
---
 sound/pci/oxygen/oxygen.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/sound/pci/oxygen/oxygen.c b/sound/pci/oxygen/oxygen.c
index ada6c256378e..99ba0354d4cc 100644
--- a/sound/pci/oxygen/oxygen.c
+++ b/sound/pci/oxygen/oxygen.c
@@ -110,6 +110,7 @@ static DEFINE_PCI_DEVICE_TABLE(oxygen_ids) = {
 	{ OXYGEN_PCI_SUBID(0x1a58, 0x0910), .driver_data = MODEL_CMEDIA_REF },
 	/* Asus Xonar DG */
 	{ OXYGEN_PCI_SUBID(0x1043, 0x8467), .driver_data = MODEL_XONAR_DG },
+	{ OXYGEN_PCI_SUBID(0x1043, 0x855e), .driver_data = MODEL_XONAR_DG },
 	/* Asus Xonar DGX */
 	{ OXYGEN_PCI_SUBID(0x1043, 0x8521), .driver_data = MODEL_XONAR_DGX },
 	/* PCI 2.0 HD Audio */
-- 
2.11.1

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] ALSA: oxygen - Fix snd_oxygen module not loading for some (new?) Xonar DG SI cards.
  2017-03-29 18:56   ` Clemens Ladisch
  (?)
@ 2017-03-30  2:55   ` Eugene Ganeev
  2017-03-30  7:20     ` Clemens Ladisch
  -1 siblings, 1 reply; 5+ messages in thread
From: Eugene Ganeev @ 2017-03-30  2:55 UTC (permalink / raw)
  To: Clemens Ladisch; +Cc: Jaroslav Kysela, Takashi Iwai, alsa-devel, linux-kernel

On Wed, Mar 29, 2017 at 08:56:16PM +0200, Clemens Ladisch wrote:
> Eugene Ganeev wrote:
> > My Xonar DG SI card is showing up in lspci but no module is loaded for
> > it.
> >
> > The patch just adds a new value with card's PCI ID to oxygen_ids array.
> 
> Is the hardware identical?  Do all the inputs and outputs work?
 
According to asus website they both use C-MEDIA CMI8786. The difference is that DG SI is 7.1 surround compared to 5.1 in DG.
I've tested the patch, all ports seem to work except rear out. So basically the card is usable unless you use 7.1 setup. 

> Please add the correct name to the names[] array.
> 
> 
> Regards.
> Clemens.

[PATCH v2] sound: oxygen: add Xonar DG SI support

Add PCI ID for the Asus Xonar DG SI card.

Signed-off-by: Eugene Ganeev <easymodofrf@gmail.com>

---

v2:
	- Added model name to names[] array.

 sound/pci/oxygen/oxygen.c | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/sound/pci/oxygen/oxygen.c b/sound/pci/oxygen/oxygen.c
index 74afb6b75976..7ba1099fefb1 100644
--- a/sound/pci/oxygen/oxygen.c
+++ b/sound/pci/oxygen/oxygen.c
@@ -94,6 +94,7 @@ enum {
 	MODEL_2CH_OUTPUT,
 	MODEL_HG2PCI,
 	MODEL_XONAR_DG,
+	MODEL_XONAR_DG_SI,
 	MODEL_XONAR_DGX,
 };
 
@@ -110,6 +111,8 @@ static const struct pci_device_id oxygen_ids[] = {
 	{ OXYGEN_PCI_SUBID(0x1a58, 0x0910), .driver_data = MODEL_CMEDIA_REF },
 	/* Asus Xonar DG */
 	{ OXYGEN_PCI_SUBID(0x1043, 0x8467), .driver_data = MODEL_XONAR_DG },
+	/* Asus Xonar DG SI */
+	{ OXYGEN_PCI_SUBID(0x1043, 0x855e), .driver_data = MODEL_XONAR_DG_SI },
 	/* Asus Xonar DGX */
 	{ OXYGEN_PCI_SUBID(0x1043, 0x8521), .driver_data = MODEL_XONAR_DGX },
 	/* PCI 2.0 HD Audio */
@@ -767,6 +770,7 @@ static int get_oxygen_model(struct oxygen *chip,
 		[MODEL_FANTASIA]	= "TempoTec HiFier Fantasia",
 		[MODEL_SERENADE]	= "TempoTec HiFier Serenade",
 		[MODEL_HG2PCI]		= "CMI8787-HG2PCI",
+		[MODEL_XONAR_DG_SI]	= "Asus Xonar DG SI",
 	};
 
 	chip->model = model_generic;
@@ -832,6 +836,9 @@ static int get_oxygen_model(struct oxygen *chip,
 		chip->model = model_xonar_dg;
 		chip->model.shortname = "Xonar DG";
 		break;
+	case MODEL_XONAR_DG_SI:
+		chip->model = model_xonar_dg;
+		break;
 	case MODEL_XONAR_DGX:
 		chip->model = model_xonar_dg;
 		chip->model.shortname = "Xonar DGX";

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] ALSA: oxygen - Fix snd_oxygen module not loading for some (new?) Xonar DG SI cards.
  2017-03-30  2:55   ` Eugene Ganeev
@ 2017-03-30  7:20     ` Clemens Ladisch
  0 siblings, 0 replies; 5+ messages in thread
From: Clemens Ladisch @ 2017-03-30  7:20 UTC (permalink / raw)
  To: Eugene Ganeev; +Cc: alsa-devel, Takashi Iwai

Eugene Ganeev wrote:
> On Wed, Mar 29, 2017 at 08:56:16PM +0200, Clemens Ladisch wrote:
>> Eugene Ganeev wrote:
>>> My Xonar DG SI card is showing up in lspci but no module is loaded for
>>> it.
>>>
>>> The patch just adds a new value with card's PCI ID to oxygen_ids array.
>>
>> Is the hardware identical?  Do all the inputs and outputs work?
>
> According to asus website they both use C-MEDIA CMI8786. The difference is that DG SI is 7.1 surround compared to 5.1 in DG.
> I've tested the patch, all ports seem to work except rear out. So basically the card is usable unless you use 7.1 setup.

Strange.  This would imply that the CS4245 is used for both headphones
output and back surround output.

And of course this should be supported.  Are you willing to experiment?
(In theory, it should suffice to adjust model.dac_channels_pcm and
model.adjust_dac_routing.)

>> Please add the correct name to the names[] array.
>
> [PATCH v2] sound: oxygen: add Xonar DG SI support
>
> Add PCI ID for the Asus Xonar DG SI card.
>
> Signed-off-by: Eugene Ganeev <easymodofrf@gmail.com>
>
> ---
>
> v2:
> 	- Added model name to names[] array.
>
>  sound/pci/oxygen/oxygen.c | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/sound/pci/oxygen/oxygen.c b/sound/pci/oxygen/oxygen.c
> index 74afb6b75976..7ba1099fefb1 100644
> --- a/sound/pci/oxygen/oxygen.c
> +++ b/sound/pci/oxygen/oxygen.c
> @@ -94,6 +94,7 @@ enum {
>  	MODEL_2CH_OUTPUT,
>  	MODEL_HG2PCI,
>  	MODEL_XONAR_DG,
> +	MODEL_XONAR_DG_SI,
>  	MODEL_XONAR_DGX,
>  };
>
> @@ -110,6 +111,8 @@ static const struct pci_device_id oxygen_ids[] = {
>  	{ OXYGEN_PCI_SUBID(0x1a58, 0x0910), .driver_data = MODEL_CMEDIA_REF },
>  	/* Asus Xonar DG */
>  	{ OXYGEN_PCI_SUBID(0x1043, 0x8467), .driver_data = MODEL_XONAR_DG },
> +	/* Asus Xonar DG SI */
> +	{ OXYGEN_PCI_SUBID(0x1043, 0x855e), .driver_data = MODEL_XONAR_DG_SI },
>  	/* Asus Xonar DGX */
>  	{ OXYGEN_PCI_SUBID(0x1043, 0x8521), .driver_data = MODEL_XONAR_DGX },
>  	/* PCI 2.0 HD Audio */
> @@ -767,6 +770,7 @@ static int get_oxygen_model(struct oxygen *chip,
>  		[MODEL_FANTASIA]	= "TempoTec HiFier Fantasia",
>  		[MODEL_SERENADE]	= "TempoTec HiFier Serenade",
>  		[MODEL_HG2PCI]		= "CMI8787-HG2PCI",
> +		[MODEL_XONAR_DG_SI]	= "Asus Xonar DG SI",
>  	};
>
>  	chip->model = model_generic;
> @@ -832,6 +836,9 @@ static int get_oxygen_model(struct oxygen *chip,
>  		chip->model = model_xonar_dg;
>  		chip->model.shortname = "Xonar DG";
>  		break;
> +	case MODEL_XONAR_DG_SI:
> +		chip->model = model_xonar_dg;
> +		break;
>  	case MODEL_XONAR_DGX:
>  		chip->model = model_xonar_dg;
>  		chip->model.shortname = "Xonar DGX";

This does no longer apply because of the other names[] patch.

And I forgot to mention: please also update sound/pci/Kconfig and
Documentation/sound/alsa-configuration.rst.


Regards,
Clemens

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2017-03-30  7:20 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-03-29 21:13 [PATCH] ALSA: oxygen - Fix snd_oxygen module not loading for some (new?) Xonar DG SI cards Eugene Ganeev
2017-03-29 18:56 ` Clemens Ladisch
2017-03-29 18:56   ` Clemens Ladisch
2017-03-30  2:55   ` Eugene Ganeev
2017-03-30  7:20     ` Clemens Ladisch

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.