From: Pavel Hofman <pavel.hofman@ivitera.com>
To: Takashi Iwai <tiwai@suse.de>
Cc: alsa-devel@alsa-project.org
Subject: Re: [PATCH 1/7] ak4114 - fix errors in output selector bits
Date: Wed, 16 Sep 2009 09:00:59 +0200 [thread overview]
Message-ID: <4AB08D2B.5070604@ivitera.com> (raw)
In-Reply-To: <s5hbplbzarg.wl%tiwai@suse.de>
Takashi Iwai wrote:
> At Wed, 16 Sep 2009 00:39:50 +0200,
> pavel.hofman@ivitera.com wrote:
>> From: Pavel Hofman <pavel.hofman@ivitera.com>
>>
>>
>> Signed-off-by: Pavel Hofman <pavel.hofman@ivitera.com>
>
> Could you describe *what* are exactly fixed?
> Do these change influence on any driver behavior?
It is just a fix - the original version had a typo - see the equal
values for
AK4114_OPS12 = AK4114_OPS02
AK4114_OPS11 = AK4114_OPS01
AK4114_OPS10 = AK4114_OPS00
Since no cards actually use this feature, the bug was not uncovered earlier.
Thanks,
Pavel.
>
>
> thanks,
>
> Takashi
>
>> diff --git a/include/ak4114.h b/include/ak4114.h
>> index d293d36..3ce69fd 100644
>> --- a/include/ak4114.h
>> +++ b/include/ak4114.h
>> @@ -95,13 +95,13 @@
>>
>> /* AK4114_REG_IO0 */
>> #define AK4114_TX1E (1<<7) /* TX1 Output Enable (1 = enable) */
>> -#define AK4114_OPS12 (1<<2) /* Output Though Data Selector for TX1 pin */
>> -#define AK4114_OPS11 (1<<1) /* Output Though Data Selector for TX1 pin */
>> -#define AK4114_OPS10 (1<<0) /* Output Though Data Selector for TX1 pin */
>> +#define AK4114_OPS12 (1<<6) /* Output Data Selector for TX1 pin */
>> +#define AK4114_OPS11 (1<<5) /* Output Data Selector for TX1 pin */
>> +#define AK4114_OPS10 (1<<4) /* Output Data Selector for TX1 pin */
>> #define AK4114_TX0E (1<<3) /* TX0 Output Enable (1 = enable) */
>> -#define AK4114_OPS02 (1<<2) /* Output Though Data Selector for TX0 pin */
>> -#define AK4114_OPS01 (1<<1) /* Output Though Data Selector for TX0 pin */
>> -#define AK4114_OPS00 (1<<0) /* Output Though Data Selector for TX0 pin */
>> +#define AK4114_OPS02 (1<<2) /* Output Data Selector for TX0 pin */
>> +#define AK4114_OPS01 (1<<1) /* Output Data Selector for TX0 pin */
>> +#define AK4114_OPS00 (1<<0) /* Output Data Selector for TX0 pin */
>>
>> /* AK4114_REG_IO1 */
>> #define AK4114_EFH1 (1<<7) /* Interrupt 0 pin Hold */
>> --
>> 1.5.6.3
>>
--
IVITERA a.s., www.ivitera.com
Kafkova 16, 160 00 Praha 6
Rubesova 29, 326 00 Plzen
Czech Republic
Mobil +420 603 163 973
Tel. +420 377 449 358
Fax +420 222 745 553
pavel.hofman@ivitera.com
www.educity.cz www.jobcity.cz
www.meetings.cz www.hrzive.cz
www.zamestnaneckevyhody.cz
www.comben.cz
--------------------------------
Navstivte nejvetsi nabidku kurzu a skoleni
na ceskem internetu www.educity.cz,
nejvetsi vyber prostor pro firemni akce www.meetings.cz
a www.jobcity.cz s nabidkami lepsi prace.
next prev parent reply other threads:[~2009-09-16 7:00 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-09-15 22:39 [PATCH 1/7] ak4114 - fix errors in output selector bits pavel.hofman
2009-09-15 22:39 ` [PATCH 2/7] ak4620 support, codec regs listed in proc pavel.hofman
2009-09-15 22:39 ` [PATCH 3/7] ak4113 support pavel.hofman
2009-09-15 22:39 ` [PATCH 4/7] ice1724 - adding GPIO routines for mask and direction pavel.hofman
2009-09-15 22:39 ` [PATCH 5/7] ice1724 - pro-rate-locking makes sense only for internal clock mode pavel.hofman
2009-09-15 22:39 ` [PATCH 6/7] Support for multiple external clock types pavel.hofman
2009-09-15 22:39 ` [PATCH 7/7] Infrasonic Quartet support pavel.hofman
2009-09-16 6:07 ` [PATCH 4/7] ice1724 - adding GPIO routines for mask and direction Takashi Iwai
2009-09-16 7:15 ` Pavel Hofman
2009-09-16 7:48 ` Takashi Iwai
2009-09-16 8:10 ` Pavel Hofman
2009-09-16 8:27 ` Takashi Iwai
2009-09-16 20:15 ` Pavel Hofman
2009-09-17 16:30 ` Takashi Iwai
2009-09-16 6:03 ` [PATCH 1/7] ak4114 - fix errors in output selector bits Takashi Iwai
2009-09-16 7:00 ` Pavel Hofman [this message]
2009-09-16 7:45 ` Takashi Iwai
2009-09-16 7:48 ` Pavel Hofman
-- strict thread matches above, loose matches on Subject: below --
2009-09-16 20:25 pavel.hofman
2009-09-21 14:10 ` Takashi Iwai
2009-09-21 18:38 ` Pavel Hofman
2009-09-21 19:26 ` Takashi Iwai
2009-09-23 9:16 ` Pavel Hofman
2009-09-23 9:21 ` Takashi Iwai
2009-09-23 9:32 ` Pavel Hofman
2009-09-23 9:56 ` Takashi Iwai
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=4AB08D2B.5070604@ivitera.com \
--to=pavel.hofman@ivitera.com \
--cc=alsa-devel@alsa-project.org \
--cc=tiwai@suse.de \
/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
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.