Changeset 2133

Show
Ignore:
Timestamp:
05/01/12 04:56:44 (11 years ago)
Author:
jwoithe
Message:

dice: for all currently known devices, 4x sample rates requires dual-wire mode. FFADO doesn't implement this yet, so don't advertise 4x rates as being available for these devices. See also tickets #343 and #353.

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • trunk/libffado/src/dice/dice_avdevice.cpp

    r2120 r2133  
    245245    DICE_CHECK_AND_ADD_SR(frequencies, 88200, DICE_CLOCKCAP_RATE_88K2); 
    246246    DICE_CHECK_AND_ADD_SR(frequencies, 96000, DICE_CLOCKCAP_RATE_96K); 
    247     DICE_CHECK_AND_ADD_SR(frequencies, 176400, DICE_CLOCKCAP_RATE_176K4); 
    248     DICE_CHECK_AND_ADD_SR(frequencies, 192000, DICE_CLOCKCAP_RATE_192K); 
     247 
     248    // In order to run at 4x rates on the DICE platform, one needs to 
     249    // implement support for "dual-wire" mode within the FFADO streaming 
     250    // engine.  Until this is done there's no point advertising 4x rate 
     251    // capabilities.  While theoretically devices based on newer DICE 
     252    // versions don't require dual-wire mode at these rates, as of 1 May 
     253    // 2012 no such devices are known.  See also ticket #343. 
     254    // 
     255    // DICE_CHECK_AND_ADD_SR(frequencies, 176400, DICE_CLOCKCAP_RATE_176K4); 
     256    // DICE_CHECK_AND_ADD_SR(frequencies, 192000, DICE_CLOCKCAP_RATE_192K); 
     257 
    249258    return frequencies; 
    250259} 
     
    314323            select=DICE_RATE_96K; 
    315324            break; 
     325 
     326// We currently can't support 4x rates on these devices.  See note 
     327// in getSupportedSamplingFrequencies(). 
     328#if 0 
    316329        case 176400: 
    317330            supported=maskedCheckNotZeroGlobalReg( 
     
    326339            select=DICE_RATE_192K; 
    327340            break; 
     341#endif 
    328342        } 
    329343