[pulseaudio-discuss] [PATCH] card: Log availability status when choosing initial profile

[João Paulo Rechi Vita]

On Wed, Jun 27, 2018 at 3:01 PM, João Paulo Rechi Vita
<jprvita at gmail.com> wrote:
> ---
>  src/pulsecore/card.c | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/src/pulsecore/card.c b/src/pulsecore/card.c
> index 0112cffb1..4ead3a606 100644
> --- a/src/pulsecore/card.c
> +++ b/src/pulsecore/card.c
> @@ -204,7 +204,10 @@ void pa_card_choose_initial_profile(pa_card *card) {
>       * or if all profiles are unavailable, pick the profile with the highest
>       * priority regardless of its availability. */
>
> +    pa_log_debug("Looking for an initial profile for card %s", card->name);
>      PA_HASHMAP_FOREACH(profile, card->profiles, state) {
> +        pa_log_debug("profile %s availability %s", profile->name, profile->available == PA_AVAILABLE_YES ? "yes" :
> +                                                                  profile->available == PA_AVAILABLE_NO ? "no" : "unknown");
>          if (profile->available == PA_AVAILABLE_NO)
>              continue;
>
> @@ -213,6 +216,7 @@ void pa_card_choose_initial_profile(pa_card *card) {
>      }
>
>      if (!best) {
> +        pa_log_info("No profile with availability status 'yes' or 'unknown' found for card %s", card->name);
>          PA_HASHMAP_FOREACH(profile, card->profiles, state) {
>              if (!best || profile->priority > best->priority)
>                  best = profile;
> @@ -220,6 +224,7 @@ void pa_card_choose_initial_profile(pa_card *card) {
>      }
>      pa_assert(best);
>
> +    pa_log_info("Initial profile %s chosen for card %s", best->name, card->name);
>      card->active_profile = best;
>      card->save_profile = false;
>


Just to give the reasoning behind this patch (which I should probably
have added to the commit message, sorry): this was very helpful when
debugging Bluetooth headset profiles failing to connect depending on
the order they were getting connected, so I figured it would be useful
to have upstream for future problems.

--
João Paulo Rechi Vita
http://about.me/jprvita