24 Apr, 2008
40 commits
-
Inner loop redeclares err with u32 rather than int, stupid fix here
is to change the inner err to err2.
sound/pci/pcxhr/pcxhr_core.c:1008:8: warning: symbol 'err' shadows an earlier one
sound/pci/pcxhr/pcxhr_core.c:983:6: originally declared hereSigned-off-by: Harvey Harrison
Signed-off-by: Takashi Iwai -
Use priv_idx as an identifier.
sound/pci/oxygen/virtuoso.c:277:15: warning: symbol 'index' shadows an earlier one
sound/pci/oxygen/virtuoso.c:56:12: originally declared hereSigned-off-by: Harvey Harrison
Signed-off-by: Takashi Iwai -
In all four case, adding a private value to the iooff index,
call it priv_idx.
sound/pci/ice1712/ice1712.c:1300:6: warning: symbol 'index' shadows an earlier one
sound/pci/ice1712/ice1712.c:85:12: originally declared here
sound/pci/ice1712/ice1712.c:1312:6: warning: symbol 'index' shadows an earlier one
sound/pci/ice1712/ice1712.c:85:12: originally declared here
sound/pci/ice1712/ice1712.c:1338:6: warning: symbol 'index' shadows an earlier one
sound/pci/ice1712/ice1712.c:85:12: originally declared here
sound/pci/ice1712/ice1712.c:1350:6: warning: symbol 'index' shadows an earlier one
sound/pci/ice1712/ice1712.c:85:12: originally declared here[tiwai - fixed coding issues as well]
Signed-off-by: Harvey Harrison
Signed-off-by: Takashi Iwai -
enable in these contexts refers specifically to intr enable, as
per the two functions it is found in. Use intr_enable instead.
sound/pci/emu10k1/emu10k1x.c:330:15: warning: symbol 'enable' shadows an earlier one
sound/pci/emu10k1/emu10k1x.c:53:12: originally declared here
sound/pci/emu10k1/emu10k1x.c:341:15: warning: symbol 'enable' shadows an earlier one
sound/pci/emu10k1/emu10k1x.c:53:12: originally declared hereinstead of shadowing, use cap_voice as we test for the capture
voice in this statement.
sound/pci/emu10k1/emu10k1x.c:798:25: warning: symbol 'pvoice' shadows an earlier one
sound/pci/emu10k1/emu10k1x.c:787:24: originally declared hereSigned-off-by: Harvey Harrison
Signed-off-by: Takashi Iwai -
Reading regs from the fpga into an int instead of a u32, trivial
fix.
sound/pci/emu10k1/emuproc.c:422:34: warning: incorrect type in argument 3 (different signedness)
sound/pci/emu10k1/emuproc.c:422:34: expected unsigned int [usertype] *value
sound/pci/emu10k1/emuproc.c:422:34: got int *Signed-off-by: Harvey Harrison
Signed-off-by: Takashi Iwai -
sound/pci/au88x0/au88x0_pcm.c:508:15: warning: Using plain integer as NULL pointer
Also some small codingstyle fixes.
Signed-off-by: Harvey Harrison
Signed-off-by: Takashi Iwai -
when the time interval for a period is smaller than kernel HZ, then
snd-aloop and snd-dummy cannot call snd_pcm_period_elapsed as fast enough
annymore. this happens for example with games. but the app still needs to
see, that the buffer actually did go further, which is provided by these
patches.Signed-off-by: Ahmet İnan mathematik.uni-freiburg.de>
Signed-off-by: Takashi Iwai -
Change the variable err to _err within the ADD_CTLS macro to avoid
shadowing the local variable.sound/pci/ca0106/ca0106_mixer.c:710:2: warning: symbol 'err' shadows an earlier one
sound/pci/ca0106/ca0106_mixer.c:663:6: originally declared here
sound/pci/ca0106/ca0106_mixer.c:712:3: warning: symbol 'err' shadows an earlier one
sound/pci/ca0106/ca0106_mixer.c:663:6: originally declared here
sound/pci/ca0106/ca0106_mixer.c:721:3: warning: symbol 'err' shadows an earlier one
sound/pci/ca0106/ca0106_mixer.c:663:6: originally declared hereSigned-off-by: Harvey Harrison
Signed-off-by: Takashi Iwai -
change to intr_enable as per the two functions it is defined in.
sound/pci/ca0106/ca0106_main.c:438:15: warning: symbol 'enable' shadows an earlier one
sound/pci/ca0106/ca0106_main.c:159:12: originally declared here
sound/pci/ca0106/ca0106_main.c:449:15: warning: symbol 'enable' shadows an earlier one
sound/pci/ca0106/ca0106_main.c:159:12: originally declared hereSigned-off-by: Harvey Harrison
Signed-off-by: Takashi Iwai -
enable is used to test for whether or not spdif should be enabled,
change to spdif_enable.sound/pci/ali5451/ali5451.c:1812:15: warning: symbol 'enable' shadows an earlier one
sound/pci/ali5451/ali5451.c:63:12: originally declared here
sound/pci/ali5451/ali5451.c:1840:27: warning: symbol 'enable' shadows an earlier one
sound/pci/ali5451/ali5451.c:63:12: originally declared hereSigned-off-by: Harvey Harrison
Signed-off-by: Takashi Iwai -
err is always assigned before it is used, no need to declare another
inside the if statement.
sound/pci/ac97/ac97_pcm.c:577:7: warning: symbol 'err' shadows an earlier one
sound/pci/ac97/ac97_pcm.c:572:6: originally declared hereSigned-off-by: Harvey Harrison
Signed-off-by: Takashi Iwai -
kernel style does assignment outside of if() block
sound/pci/rme96.c:1562:71: warning: Using plain integer as NULL pointerSigned-off-by: Harvey Harrison
Signed-off-by: Takashi Iwai -
kernel style does assignment outside of if() statements.
sound/pci/rme32.c:1353:71: warning: Using plain integer as NULL pointerSigned-off-by: Harvey Harrison
Signed-off-by: Takashi Iwai -
change id to elem_id as it is used to initialize each mixer element
sound/pci/maestro3.c:2071:25: warning: symbol 'id' shadows an earlier one
sound/pci/maestro3.c:67:13: originally declared hereindex is used in each of these places to count over the dsp's memory,
change to the name dsp_index
sound/pci/maestro3.c:2572:9: warning: symbol 'index' shadows an earlier one
sound/pci/maestro3.c:66:12: originally declared here
sound/pci/maestro3.c:2604:9: warning: symbol 'index' shadows an earlier one
sound/pci/maestro3.c:66:12: originally declared here[tiwai - fixed coding style issues as well]
Signed-off-by: Harvey Harrison
Signed-off-by: Takashi Iwai -
id was only used as a counter in a for loop, move the declaration
to where it is used and change it to i.
sound/pci/fm801.c:1288:6: warning: symbol 'id' shadows an earlier one
sound/pci/fm801.c:51:13: originally declared here[tiwai - fixed a coding style issue as well]
Signed-off-by: Harvey Harrison
Signed-off-by: Takashi Iwai -
id is used when initializing the mixer elements, use elem_id here
instead.
sound/pci/es1968.c:1963:25: warning: symbol 'id' shadows an earlier one
sound/pci/es1968.c:129:13: originally declared hereSigned-off-by: Harvey Harrison
Signed-off-by: Takashi Iwai -
index is incremented only when AC97_EI_SPDIF and then assigned to
the index field. Change the temporary name to is_spdif.sound/pci/ens1370.c:1638:10: warning: symbol 'index' shadows an earlier one
sound/pci/ens1370.c:84:12: originally declared hereSigned-off-by: Harvey Harrison
Signed-off-by: Takashi Iwai -
A temporary variable for each mixer element is used in an initialization
loop, use the name elem_id.sound/pci/cmipci.c:2747:26: warning: symbol 'id' shadows an earlier one
sound/pci/cmipci.c:56:13: originally declared here[tiwai - fixed a coding style issue as well]
Signed-off-by: Harvey Harrison
Signed-off-by: Takashi Iwai -
Signed-off-by: Mark Brown
Signed-off-by: Takashi Iwai -
Added the docking-stationc mic input to the capture source list
for Thinkpad X61.Signed-off-by: Takashi Iwai
-
Fix the initial num_dacs of 92HD71bxx codecs.
Signed-off-by: Takashi Iwai
-
Move some entries to their proper position so that the list is again
sorted by vendor/product ID.Signed-off-by: Clemens Ladisch
-
Reduce the number of times to check for a non-empty Tx FIFO from 100 to
2 because there is no MPU-401 implementation that needs more than one or
two reads to determine the actual FIFO status.Signed-off-by: Clemens Ladisch
-
The bits indicating SPDIF frequency in the status register are not the same for
the 9632 than for the other cards, because it also supports 192kHz. A specific
bitmask has thus been added (used in hdsp_spdif_sample_rate()).
The 9632 does not seem to report external sample rates greater than 96kHz. In
this case, the best seems to report spdif rate when autosync reference is
spdif. This also required to move function hdsp_spdif_sample_rate().Signed-off-by: Remy Bruno
Signed-off-by: Takashi Iwai -
Added logic to check if AUTO_PIN_FRONT_MIC is available for output
switch, if AUTO_PIN_MIC isn't.Signed-off-by: Matthew Ranostay
Signed-off-by: Takashi Iwai -
Added the missing descriptions for STAC codec models.
Signed-off-by: Takashi Iwai
-
Removed invalid BUG() - the driver should handle the error case properly
rather than issuing BUG().Signed-off-by: Takashi Iwai
-
Clean up the pci id table using PCI_DEVICE() macro.
Signed-off-by: Takashi Iwai
-
Signed-off-by: Ahmet İnan mathematik.uni-freiburg.de>
Signed-off-by: Takashi Iwai -
Some laptops have a internal analog microphone that is not setup by the BIOS.
Signed-off-by: Matthew Ranostay
Signed-off-by: Takashi Iwai -
Added PCI_QUIRKS for laptop that have the 92HDxxx family of codecs.
Signed-off-by: Matthew Ranostay
Signed-off-by: Takashi Iwai -
STAC_DELL_BIOS quirks were setting the association value wrong
for port 0x0f, which prevented it from being included in hp_outs[].Signed-off-by: Matthew Ranostay
Signed-off-by: Takashi Iwai -
Fix issue on STAC927x codecs that first DAC was getting powered down
even if was being used.Signed-off-by: Matthew Ranostay
Signed-off-by: Takashi Iwai -
Signed-off-by: Jarkko Nikula
Signed-off-by: Takashi Iwai -
Added a brief description of aw2 driver to ALSA-Configuration.txt.
Signed-off-by: Takashi Iwai
-
Added the missing declarations for module parameters.
Signed-off-by: Takashi Iwai
-
Signed-off-by: Cedric Bregardis
Signed-off-by: Jean-Christian Hassler
Signed-off-by: Takashi Iwai -
When the device has only one input source, it makes no sense to have
multiple capture streams.Signed-off-by: Takashi Iwai
-
Initialize the capture source properly for auto model.
It's especially important for cases that only mic is detected.Signed-off-by: Takashi Iwai
-
Added the beep volume control to ALC268 codec support code.
Since the codec doesn't return the correct AMP caps, we need to override
the value.Signed-off-by: Takashi Iwai