ucm: Don't log errors during normal operation
authorTanu Kaskinen <tanuk@iki.fi>
Tue, 17 Mar 2020 07:26:57 +0000 (09:26 +0200)
committerTanu Kaskinen <tanuk@iki.fi>
Tue, 17 Mar 2020 07:31:38 +0000 (09:31 +0200)
It's completely normal to not have explicit channel configuration for
stereo devices. In fact, the ALSA developers actively avoid configuring
the channels for stereo devices.

I also dropped the word "duplex" from the messages, because "stereo
duplex" implies bidirectionality, but most devices use one direction
only.

src/modules/alsa/alsa-ucm.c

index 8df24e7..37cbafa 100644 (file)
@@ -379,8 +379,8 @@ static int ucm_get_device_property(
 
     if (pa_proplist_gets(device->proplist, PA_ALSA_PROP_UCM_SINK) &&
         device->playback_channels == 0) {
-        pa_log("UCM file does not specify 'PlaybackChannels' "
-                    "for device %s, assuming stereo duplex.", device_name);
+        pa_log_info("UCM file does not specify 'PlaybackChannels' "
+                    "for device %s, assuming stereo.", device_name);
         device->playback_channels = 2;
     }
 
@@ -400,8 +400,8 @@ static int ucm_get_device_property(
 
     if (pa_proplist_gets(device->proplist, PA_ALSA_PROP_UCM_SOURCE) &&
         device->capture_channels == 0) {
-        pa_log("UCM file does not specify 'CaptureChannels' "
-                    "for device %s, assuming stereo duplex.", device_name);
+        pa_log_info("UCM file does not specify 'CaptureChannels' "
+                    "for device %s, assuming stereo.", device_name);
         device->capture_channels = 2;
     }