* [PATCH] sound: lx_core: Few misc cleanups
@ 2014-04-08 14:15 Maxime Ripard
2014-04-09 8:01 ` [alsa-devel] " Takashi Iwai
0 siblings, 1 reply; 3+ messages in thread
From: Maxime Ripard @ 2014-04-08 14:15 UTC (permalink / raw)
To: perex, iwai; +Cc: alsa-devel, linux-kernel, Maxime Ripard
This file had a few comments in french that are now in english, and took the
opportunity to cleanup a bunch of #if 0 .. #endif and commented out code
Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
---
sound/pci/lx6464es/lx_core.c | 70 +++++++++++++++++---------------------------
1 file changed, 27 insertions(+), 43 deletions(-)
diff --git a/sound/pci/lx6464es/lx_core.c b/sound/pci/lx6464es/lx_core.c
index 626ecad4dae7..002049770ff7 100644
--- a/sound/pci/lx6464es/lx_core.c
+++ b/sound/pci/lx6464es/lx_core.c
@@ -22,8 +22,7 @@
*
*/
-/* #define RMH_DEBUG 1 */
-
+#include <linux/bitops.h>
#include <linux/module.h>
#include <linux/pci.h>
#include <linux/delay.h>
@@ -496,15 +495,17 @@ int lx_dsp_es_check_pipeline(struct lx6464es *chip)
int i;
for (i = 0; i != CSES_TIMEOUT; ++i) {
- /*
- * le bit CSES_UPDATE_LDSV est à 1 dés que le macprog
- * est pret. il re-passe à 0 lorsque le premier read a
- * été fait. pour l'instant on retire le test car ce bit
- * passe a 1 environ 200 à 400 ms aprés que le registre
- * confES à été écrit (kick du xilinx ES).
+ /*
+ * The bit CSES_UPDATE_LDSV is set to 1 as soon as the
+ * macprog is ready. He goes back to 0 whenever the
+ * first read has been issued.
+ *
+ * For now, we don't bother testing this bit because
+ * it's set to 1 between 200 to 400ms after the confES
+ * register has been written to (xilinx ES kick).
*
- * On ne teste que le bit CE.
- * */
+ * We only test the CE bit.
+ */
u32 cses = lx_dsp_reg_read(chip, eReg_CSES);
@@ -602,8 +603,7 @@ int lx_buffer_ask(struct lx6464es *chip, u32 pipe, int is_capture,
/* free */
*r_needed += 1;
}
-
-#if 0
+#if DEBUG
snd_printdd(LXP "CMD_08_ASK_BUFFERS: needed %d, freed %d\n",
*r_needed, *r_freed);
for (i = 0; i < MAX_STREAM_BUFFER; ++i) {
@@ -1049,9 +1049,9 @@ int lx_level_peaks(struct lx6464es *chip, int is_capture, int channels,
/* interrupt handling */
#define PCX_IRQ_NONE 0
-#define IRQCS_ACTIVE_PCIDB 0x00002000L /* Bit nø 13 */
-#define IRQCS_ENABLE_PCIIRQ 0x00000100L /* Bit nø 08 */
-#define IRQCS_ENABLE_PCIDB 0x00000200L /* Bit nø 09 */
+#define IRQCS_ACTIVE_PCIDB BIT(13)
+#define IRQCS_ENABLE_PCIIRQ BIT(8)
+#define IRQCS_ENABLE_PCIDB BIT(9)
static u32 lx_interrupt_test_ack(struct lx6464es *chip)
{
@@ -1108,25 +1108,21 @@ static int lx_interrupt_handle_async_events(struct lx6464es *chip, u32 irqsrc,
int err;
u32 stat[9]; /* answer from CMD_04_GET_EVENT */
- /* On peut optimiser pour ne pas lire les evenements vides
- * les mots de réponse sont dans l'ordre suivant :
- * Stat[0] mot de status général
- * Stat[1] fin de buffer OUT pF
- * Stat[2] fin de buffer OUT pf
- * Stat[3] fin de buffer IN pF
- * Stat[4] fin de buffer IN pf
- * Stat[5] underrun poid fort
- * Stat[6] underrun poid faible
- * Stat[7] overrun poid fort
- * Stat[8] overrun poid faible
+ /* We can optimize this to not read dumb events.
+ * Answer words are in the following order:
+ * Stat[0] general status
+ * Stat[1] end of buffer OUT pF
+ * Stat[2] end of buffer OUT pf
+ * Stat[3] end of buffer IN pF
+ * Stat[4] end of buffer IN pf
+ * Stat[5] MSB underrun
+ * Stat[6] LSB underrun
+ * Stat[7] MSB overrun
+ * Stat[8] LSB overrun
* */
u64 orun_mask;
u64 urun_mask;
-#if 0
- int has_underrun = (irqsrc & MASK_SYS_STATUS_URUN) ? 1 : 0;
- int has_overrun = (irqsrc & MASK_SYS_STATUS_ORUN) ? 1 : 0;
-#endif
int eb_pending_out = (irqsrc & MASK_SYS_STATUS_EOBO) ? 1 : 0;
int eb_pending_in = (irqsrc & MASK_SYS_STATUS_EOBI) ? 1 : 0;
@@ -1272,7 +1268,7 @@ irqreturn_t lx_interrupt(int irq, void *dev_id)
if (irqsrc & MASK_SYS_STATUS_CMD_DONE)
goto exit;
-#if 0
+#if DEBUG
if (irqsrc & MASK_SYS_STATUS_EOBI)
snd_printdd(LXP "interrupt: EOBI\n");
@@ -1310,18 +1306,6 @@ irqreturn_t lx_interrupt(int irq, void *dev_id)
"error during audio transfer\n");
}
- if (async_escmd) {
-#if 0
- /* backdoor for ethersound commands
- *
- * for now, we do not need this
- *
- * */
-
- snd_printdd("lx6464es: interrupt requests escmd handling\n");
-#endif
- }
-
exit:
spin_unlock(&chip->lock);
return IRQ_HANDLED; /* this device caused the interrupt */
--
1.9.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [alsa-devel] [PATCH] sound: lx_core: Few misc cleanups
2014-04-08 14:15 [PATCH] sound: lx_core: Few misc cleanups Maxime Ripard
@ 2014-04-09 8:01 ` Takashi Iwai
2014-04-17 9:52 ` Maxime Ripard
0 siblings, 1 reply; 3+ messages in thread
From: Takashi Iwai @ 2014-04-09 8:01 UTC (permalink / raw)
To: Maxime Ripard; +Cc: perex, alsa-devel, linux-kernel
At Tue, 8 Apr 2014 16:15:33 +0200,
Maxime Ripard wrote:
>
> This file had a few comments in french that are now in english, and took the
> opportunity to cleanup a bunch of #if 0 .. #endif and commented out code
>
> Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
The patch isn't applicable any longer. Could you rebase to either
Linus git tree or sound git tree for-linus branch, and resubmit?
thanks,
Takashi
> ---
> sound/pci/lx6464es/lx_core.c | 70 +++++++++++++++++---------------------------
> 1 file changed, 27 insertions(+), 43 deletions(-)
>
> diff --git a/sound/pci/lx6464es/lx_core.c b/sound/pci/lx6464es/lx_core.c
> index 626ecad4dae7..002049770ff7 100644
> --- a/sound/pci/lx6464es/lx_core.c
> +++ b/sound/pci/lx6464es/lx_core.c
> @@ -22,8 +22,7 @@
> *
> */
>
> -/* #define RMH_DEBUG 1 */
> -
> +#include <linux/bitops.h>
> #include <linux/module.h>
> #include <linux/pci.h>
> #include <linux/delay.h>
> @@ -496,15 +495,17 @@ int lx_dsp_es_check_pipeline(struct lx6464es *chip)
> int i;
>
> for (i = 0; i != CSES_TIMEOUT; ++i) {
> - /*
> - * le bit CSES_UPDATE_LDSV est à 1 dés que le macprog
> - * est pret. il re-passe à 0 lorsque le premier read a
> - * été fait. pour l'instant on retire le test car ce bit
> - * passe a 1 environ 200 à 400 ms aprés que le registre
> - * confES à été écrit (kick du xilinx ES).
> + /*
> + * The bit CSES_UPDATE_LDSV is set to 1 as soon as the
> + * macprog is ready. He goes back to 0 whenever the
> + * first read has been issued.
> + *
> + * For now, we don't bother testing this bit because
> + * it's set to 1 between 200 to 400ms after the confES
> + * register has been written to (xilinx ES kick).
> *
> - * On ne teste que le bit CE.
> - * */
> + * We only test the CE bit.
> + */
>
> u32 cses = lx_dsp_reg_read(chip, eReg_CSES);
>
> @@ -602,8 +603,7 @@ int lx_buffer_ask(struct lx6464es *chip, u32 pipe, int is_capture,
> /* free */
> *r_needed += 1;
> }
> -
> -#if 0
> +#if DEBUG
> snd_printdd(LXP "CMD_08_ASK_BUFFERS: needed %d, freed %d\n",
> *r_needed, *r_freed);
> for (i = 0; i < MAX_STREAM_BUFFER; ++i) {
> @@ -1049,9 +1049,9 @@ int lx_level_peaks(struct lx6464es *chip, int is_capture, int channels,
>
> /* interrupt handling */
> #define PCX_IRQ_NONE 0
> -#define IRQCS_ACTIVE_PCIDB 0x00002000L /* Bit nø 13 */
> -#define IRQCS_ENABLE_PCIIRQ 0x00000100L /* Bit nø 08 */
> -#define IRQCS_ENABLE_PCIDB 0x00000200L /* Bit nø 09 */
> +#define IRQCS_ACTIVE_PCIDB BIT(13)
> +#define IRQCS_ENABLE_PCIIRQ BIT(8)
> +#define IRQCS_ENABLE_PCIDB BIT(9)
>
> static u32 lx_interrupt_test_ack(struct lx6464es *chip)
> {
> @@ -1108,25 +1108,21 @@ static int lx_interrupt_handle_async_events(struct lx6464es *chip, u32 irqsrc,
> int err;
> u32 stat[9]; /* answer from CMD_04_GET_EVENT */
>
> - /* On peut optimiser pour ne pas lire les evenements vides
> - * les mots de réponse sont dans l'ordre suivant :
> - * Stat[0] mot de status général
> - * Stat[1] fin de buffer OUT pF
> - * Stat[2] fin de buffer OUT pf
> - * Stat[3] fin de buffer IN pF
> - * Stat[4] fin de buffer IN pf
> - * Stat[5] underrun poid fort
> - * Stat[6] underrun poid faible
> - * Stat[7] overrun poid fort
> - * Stat[8] overrun poid faible
> + /* We can optimize this to not read dumb events.
> + * Answer words are in the following order:
> + * Stat[0] general status
> + * Stat[1] end of buffer OUT pF
> + * Stat[2] end of buffer OUT pf
> + * Stat[3] end of buffer IN pF
> + * Stat[4] end of buffer IN pf
> + * Stat[5] MSB underrun
> + * Stat[6] LSB underrun
> + * Stat[7] MSB overrun
> + * Stat[8] LSB overrun
> * */
>
> u64 orun_mask;
> u64 urun_mask;
> -#if 0
> - int has_underrun = (irqsrc & MASK_SYS_STATUS_URUN) ? 1 : 0;
> - int has_overrun = (irqsrc & MASK_SYS_STATUS_ORUN) ? 1 : 0;
> -#endif
> int eb_pending_out = (irqsrc & MASK_SYS_STATUS_EOBO) ? 1 : 0;
> int eb_pending_in = (irqsrc & MASK_SYS_STATUS_EOBI) ? 1 : 0;
>
> @@ -1272,7 +1268,7 @@ irqreturn_t lx_interrupt(int irq, void *dev_id)
> if (irqsrc & MASK_SYS_STATUS_CMD_DONE)
> goto exit;
>
> -#if 0
> +#if DEBUG
> if (irqsrc & MASK_SYS_STATUS_EOBI)
> snd_printdd(LXP "interrupt: EOBI\n");
>
> @@ -1310,18 +1306,6 @@ irqreturn_t lx_interrupt(int irq, void *dev_id)
> "error during audio transfer\n");
> }
>
> - if (async_escmd) {
> -#if 0
> - /* backdoor for ethersound commands
> - *
> - * for now, we do not need this
> - *
> - * */
> -
> - snd_printdd("lx6464es: interrupt requests escmd handling\n");
> -#endif
> - }
> -
> exit:
> spin_unlock(&chip->lock);
> return IRQ_HANDLED; /* this device caused the interrupt */
> --
> 1.9.1
>
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [alsa-devel] [PATCH] sound: lx_core: Few misc cleanups
2014-04-09 8:01 ` [alsa-devel] " Takashi Iwai
@ 2014-04-17 9:52 ` Maxime Ripard
0 siblings, 0 replies; 3+ messages in thread
From: Maxime Ripard @ 2014-04-17 9:52 UTC (permalink / raw)
To: Takashi Iwai; +Cc: perex, alsa-devel, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 717 bytes --]
Hi,
On Wed, Apr 09, 2014 at 10:01:06AM +0200, Takashi Iwai wrote:
> At Tue, 8 Apr 2014 16:15:33 +0200,
> Maxime Ripard wrote:
> >
> > This file had a few comments in french that are now in english, and took the
> > opportunity to cleanup a bunch of #if 0 .. #endif and commented out code
> >
> > Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
>
> The patch isn't applicable any longer. Could you rebase to either
> Linus git tree or sound git tree for-linus branch, and resubmit?
I just did, and fixed a few stuff that have been introduced along the way.
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-04-17 10:06 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-04-08 14:15 [PATCH] sound: lx_core: Few misc cleanups Maxime Ripard
2014-04-09 8:01 ` [alsa-devel] " Takashi Iwai
2014-04-17 9:52 ` Maxime Ripard
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox