VirtualBox

Changeset 9329 in vbox for trunk/src/VBox/Devices/Audio


Ignore:
Timestamp:
Jun 2, 2008 9:44:21 PM (17 years ago)
Author:
vboxsync
Message:

Some cleanup; mainly adding some missing spaces and correcting the construction of an error string.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/VBox/Devices/Audio/DevIchAc97.cpp

    r8155 r9329  
     1/* $Id$ */
    12/** @file
    2  *
    3  * VBox ICH AC97 Audio Controller
     3 * VBox ICH AC97 Audio Controller.
    44 */
    55
    66/*
    7  * Copyright (C) 2006-2007 Sun Microsystems, Inc.
     7 * Copyright (C) 2006-2008 Sun Microsystems, Inc.
    88 *
    99 * This file is part of VirtualBox Open Source Edition (OSE), as
    … …  
    8080
    8181#ifndef VBOX
    82 #define SOFT_VOLUME
     82# define SOFT_VOLUME
    8383#else
    84 #undef  SOFT_VOLUME
     84# undef  SOFT_VOLUME
    8585#endif
    8686#define SR_FIFOE RT_BIT(4)          /* rwc, fifo error */
    … …  
    228228
    229229#define MKREGS(prefix, start)                   \
    230 enum {                                          \
    231     prefix ## _BDBAR = start,                   \
    232     prefix ## _CIV   = start + 4,               \
    233     prefix ## _LVI   = start + 5,               \
    234     prefix ## _SR    = start + 6,               \
    235     prefix ## _PICB  = start + 8,               \
    236     prefix ## _PIV   = start + 10,              \
    237     prefix ## _CR    = start + 11               \
    238 }
     230    enum {                                      \
     231        prefix ## _BDBAR = start,               \
     232        prefix ## _CIV   = start + 4,           \
     233        prefix ## _LVI   = start + 5,           \
     234        prefix ## _SR    = start + 6,           \
     235        prefix ## _PICB  = start + 8,           \
     236        prefix ## _PIV   = start + 10,          \
     237        prefix ## _CR    = start + 11           \
     238    }
    239239
    240240enum
    … …  
    282282    r->bd_valid   = 1;
    283283#if !defined(RT_ARCH_X86) && !defined(RT_ARCH_AMD64)
    284 #error Please adapt the code (audio buffers are little endian)!
     284# error Please adapt the code (audio buffers are little endian)!
    285285#else
    286286    r->bd.addr    = (*(uint32_t *) &b[0]) & ~3;
    … …  
    417417                                           s, pi_callback, &as);
    418418#ifdef LOG_VOICES
    419                 LogRel(("AC97: open PI freq=%d (%s)\n", freq, s->voice_pi ? "ok" : "FAIL"));
     419                LogRel (("AC97: open PI freq=%d (%s)\n", freq, s->voice_pi ? "ok" : "FAIL"));
    420420#endif
    421421                break;
    … …  
    425425                                            s, po_callback, &as);
    426426#ifdef LOG_VOICES
    427                 LogRel(("AC97: open PO freq=%d (%s)\n", freq, s->voice_po ? "ok" : "FAIL"));
     427                LogRel (("AC97: open PO freq=%d (%s)\n", freq, s->voice_po ? "ok" : "FAIL"));
    428428#endif
    429429                break;
    … …  
    433433                                           s, mc_callback, &as);
    434434#ifdef LOG_VOICES
    435                 LogRel(("AC97: open MC freq=%d (%s)\n", freq, s->voice_mc ? "ok" : "FAIL"));
     435                LogRel (("AC97: open MC freq=%d (%s)\n", freq, s->voice_mc ? "ok" : "FAIL"));
    436436#endif
    437437                break;
    … …  
    445445                AUD_close_in (&s->card, s->voice_pi);
    446446#ifdef LOG_VOICES
    447                 LogRel(("AC97: Closing PCM IN\n"));
     447                LogRel (("AC97: Closing PCM IN\n"));
    448448#endif
    449449                s->voice_pi = NULL;
    … …  
    453453                AUD_close_out (&s->card, s->voice_po);
    454454#ifdef LOG_VOICES
    455                 LogRel(("AC97: Closing PCM OUT\n"));
     455                LogRel (("AC97: Closing PCM OUT\n"));
    456456#endif
    457457                s->voice_po = NULL;
    … …  
    461461                AUD_close_in (&s->card, s->voice_mc);
    462462#ifdef LOG_VOICES
    463                 LogRel(("AC97: Closing MIC IN\n"));
     463                LogRel (("AC97: Closing MIC IN\n"));
    464464#endif
    465465                s->voice_mc = NULL;
    … …  
    487487
    488488#ifdef USE_MIXER
     489
    489490static void set_volume (AC97LinkState *s, int index,
    490491                        audmixerctl_t mt, uint32_t val)
    … …  
    496497    lvol = 255 * lvol / VOL_MASK;
    497498
    498 #ifdef SOFT_VOLUME
     499# ifdef SOFT_VOLUME
    499500    if (index == AC97_Master_Volume_Mute)
    500501        AUD_set_volume_out (s->voice_po, mute, lvol, rvol);
    501502    else
    502503        AUD_set_volume (mt, &mute, &lvol, &rvol);
    503 #else
     504# else
    504505    AUD_set_volume (mt, &mute, &lvol, &rvol);
    505 #endif
     506# endif
    506507
    507508    rvol = VOL_MASK - ((VOL_MASK * rvol) / 255);
    508509    lvol = VOL_MASK - ((VOL_MASK * lvol) / 255);
    509510
    510 #ifdef VBOX
    511511    /*
    512512     * From AC'97 SoundMax Codec AD1981A: "Because AC '97 defines 6-bit volume registers, to
    … …  
    521521    if (val & RT_BIT(13))
    522522        val |= RT_BIT(12) | RT_BIT(11) | RT_BIT(10) | RT_BIT(9) | RT_BIT(8);
    523 #endif
    524523
    525524    mixer_store (s, index, val);
    … …  
    567566    mixer_store (s, AC97_Record_Select, rs | (ls << 8));
    568567}
    569 #endif
     568
     569#endif /* USE_MIXER */
    570570
    571571static void mixer_reset (AC97LinkState *s)
    … …  
    804804        }
    805805
    806         Log(("r->picb = %d\n", r->picb));
     806        Log (("r->picb = %d\n", r->picb));
    807807
    808808        if (!r->picb)
    … …  
    12751275                    record_select (s, u32);
    12761276                    break;
    1277 #else
     1277#else  /* !USE_MIXER */
    12781278                case AC97_Master_Volume_Mute:
    12791279                case AC97_PCM_Out_Volume_Mute:
    … …  
    12821282                    mixer_store (s, index, u32);
    12831283                    break;
    1284 #endif
     1284#endif /* !USE_MIXER */
    12851285                case AC97_Vendor_ID1:
    12861286                case AC97_Vendor_ID2:
    … …  
    14971497#ifdef USE_MIXER
    14981498    record_select (s, mixer_load (s, AC97_Record_Select));
    1499 #define V_(a, b) set_volume (s, a, b, mixer_load (s, a))
     1499# define V_(a, b) set_volume (s, a, b, mixer_load (s, a))
    15001500    V_ (AC97_Master_Volume_Mute,  AUD_MIXER_VOLUME);
    15011501    V_ (AC97_PCM_Out_Volume_Mute, AUD_MIXER_PCM);
    15021502    V_ (AC97_Line_In_Volume_Mute, AUD_MIXER_LINE_IN);
    1503 #undef V_
    1504 #endif
     1503# undef V_
     1504#endif /* USE_MIXER */
    15051505    reset_voices (s, active);
    15061506
    … …  
    15201520 *          make sense to me so we'll do it.
    15211521 */
    1522 static DECLCALLBACK(void)  ac97Reset(PPDMDEVINS pDevIns)
     1522static DECLCALLBACK(void)  ac97Reset (PPDMDEVINS pDevIns)
    15231523{
    15241524    PCIAC97LinkState *pData = PDMINS2DATA(pDevIns, PCIAC97LinkState *);
    … …  
    15361536     * the codec manually.
    15371537     */
    1538     mixer_reset(&pData->ac97);
     1538    mixer_reset (&pData->ac97);
    15391539}
    15401540
    … …  
    15991599
    16001600    /* PCI Device */
     1601/** @todo convert to PCI*(). */
    16011602    pData->dev.config[0x00] = 0x86;      /* vid vendor id intel ro */
    16021603    pData->dev.config[0x01] = 0x80;      /* intel */
    … …  
    16431644     */
    16441645    rc = PDMDevHlpPCIRegister (pDevIns, &pData->dev);
    1645     if (VBOX_FAILURE(rc))
     1646    if (VBOX_FAILURE (rc))
    16461647        return rc;
    16471648
    16481649    rc = PDMDevHlpPCIIORegionRegister (pDevIns, 0, 256, PCI_ADDRESS_SPACE_IO,
    16491650                                       ichac97IOPortMap);
    1650     if (VBOX_FAILURE(rc))
     1651    if (VBOX_FAILURE (rc))
    16511652        return rc;
    16521653
    16531654    rc = PDMDevHlpPCIIORegionRegister (pDevIns, 1, 64, PCI_ADDRESS_SPACE_IO,
    16541655                                       ichac97IOPortMap);
    1655     if (VBOX_FAILURE(rc))
     1656    if (VBOX_FAILURE (rc))
    16561657        return rc;
    16571658
    … …  
    16601661                               NULL, ichac97SaveExec, NULL,
    16611662                               NULL, ichac97LoadExec, NULL);
    1662     if (VBOX_FAILURE(rc))
     1663    if (VBOX_FAILURE (rc))
    16631664        return rc;
    16641665
    … …  
    16701671    if (rc == VERR_PDM_NO_ATTACHED_DRIVER)
    16711672        Log (("ac97: No attached driver!\n"));
    1672     else if (VBOX_FAILURE(rc))
    1673     {
    1674         AssertMsgFailed(("Failed to attach AC97 LUN #0! rc=%Vrc\n", rc));
     1673    else if (VBOX_FAILURE (rc))
     1674    {
     1675        AssertMsgFailed (("Failed to attach AC97 LUN #0! rc=%Vrc\n", rc));
    16751676        return rc;
    16761677    }
    … …  
    16781679    AUD_register_card ("ICH0", &s->card);
    16791680
    1680     ac97Reset(pDevIns);
     1681    ac97Reset (pDevIns);
    16811682
    16821683#ifndef RT_OS_DARWIN /* coreaudio doesn't supply these. */
    16831684    if (!s->voice_pi)
    1684         LogRel(("AC97: WARNING: Unable to open PCM IN!\n"));
     1685        LogRel (("AC97: WARNING: Unable to open PCM IN!\n"));
    16851686    if (!s->voice_mc)
    1686         LogRel(("AC97: WARNING: Unable to open PCM MC!\n"));
     1687        LogRel (("AC97: WARNING: Unable to open PCM MC!\n"));
    16871688#endif
    16881689    if (!s->voice_po)
    1689         LogRel(("AC97: WARNING: Unable to open PCM OUT!\n"));
     1690        LogRel (("AC97: WARNING: Unable to open PCM OUT!\n"));
    16901691
    16911692    if (!s->voice_pi && !s->voice_po && !s->voice_mc)
    … …  
    16981699        s->voice_pi = NULL;
    16991700        s->voice_mc = NULL;
    1700         AUD_init_null();
    1701         ac97Reset(pDevIns);
    1702 
    1703         PDMDevHlpVMSetRuntimeError(pDevIns, false, "HostAudioNotResponding",
    1704              N_("No audio devices could be opened. Selecting the NULL audio backend "
     1701        AUD_init_null ();
     1702        ac97Reset (pDevIns);
     1703
     1704        PDMDevHlpVMSetRuntimeError (pDevIns, false, "HostAudioNotResponding",
     1705            N_ ("No audio devices could be opened. Selecting the NULL audio backend "
    17051706                "with the consequence that no sound is audible."));
    17061707    }
    … …  
    17101711        char   szMissingVoices[128];
    17111712        size_t len = 0;
    1712         bool   fComma = false;
    17131713        if (!s->voice_pi)
    1714         {
    1715             len = RTStrPrintf(szMissingVoices, sizeof(szMissingVoices), "PCM_in");
    1716             fComma = true;
    1717         }
     1714            len = RTStrPrintf (szMissingVoices, sizeof(szMissingVoices), "PCM_in");
    17181715        if (!s->voice_po)
    1719         {
    1720             len = RTStrPrintf(szMissingVoices + len, sizeof(szMissingVoices)-len, "%sPCM_out",
    1721                               fComma ? ", " : "");
    1722             fComma = true;
    1723         }
     1716            len += RTStrPrintf (szMissingVoices + len, sizeof(szMissingVoices) - len, len ? ", PCM_out" : "PCM_out");
    17241717        if (!s->voice_mc)
    1725         {
    1726             len = RTStrPrintf(szMissingVoices + len, sizeof(szMissingVoices)-len, "%sPCM_mic",
    1727                               fComma ? ", " : "");
    1728         }
    1729 
    1730         PDMDevHlpVMSetRuntimeError(pDevIns, false, "HostAudioNotResponding",
    1731              N_("Some audio devices (%s) could not be opened. Guest applications generating audio "
     1718            len += RTStrPrintf (szMissingVoices + len, sizeof(szMissingVoices) - len, len ? ", PCM_mic" : "PCM_mic");
     1719
     1720        PDMDevHlpVMSetRuntimeError (pDevIns, false, "HostAudioNotResponding",
     1721            N_ ("Some audio devices (%s) could not be opened. Guest applications generating audio "
    17321722                "output or depending on audio input may hang. Make sure your host audio device "
    17331723                "is working properly. Check the logfile for error messages of the audio "
Note: See TracChangeset for help on using the changeset viewer.

© 2024 Oracle Support Privacy / Do Not Sell My Info Terms of Use Trademark Policy Automated Access Etiquette