[alsa-devel] [PATCH v1] ALSA: usb-audio: sound: usb: usb true/false for bool return type
Use true/false for bool type return in uac_clock_source_is_valid().
Signed-off-by: Saurav Girepunje saurav.girepunje@gmail.com ---
Changes in v1:
- Update the patch subject suggested by Takashi Iwai tiwai@suse.de
- Drop "!!" but keep the ternary operator in last return statement so that function will return always true/false.
sound/usb/clock.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/sound/usb/clock.c b/sound/usb/clock.c index 72e9bdf76115..8d5538063598 100644 --- a/sound/usb/clock.c +++ b/sound/usb/clock.c @@ -167,21 +167,21 @@ static bool uac_clock_source_is_valid(struct snd_usb_audio *chip, snd_usb_find_clock_source_v3(chip->ctrl_intf, source_id);
if (!cs_desc) - return 0; + return false; bmControls = le32_to_cpu(cs_desc->bmControls); } else { /* UAC_VERSION_1/2 */ struct uac_clock_source_descriptor *cs_desc = snd_usb_find_clock_source(chip->ctrl_intf, source_id);
if (!cs_desc) - return 0; + return false; bmControls = cs_desc->bmControls; }
/* If a clock source can't tell us whether it's valid, we assume it is */ if (!uac_v2v3_control_is_readable(bmControls, UAC2_CS_CONTROL_CLOCK_VALID)) - return 1; + return true;
err = snd_usb_ctl_msg(dev, usb_rcvctrlpipe(dev, 0), UAC2_CS_CUR, USB_TYPE_CLASS | USB_RECIP_INTERFACE | USB_DIR_IN, @@ -193,10 +193,10 @@ static bool uac_clock_source_is_valid(struct snd_usb_audio *chip, dev_warn(&dev->dev, "%s(): cannot get clock validity for id %d\n", __func__, source_id); - return 0; + return false; }
- return !!data; + return data ? true : false; }
static int __uac_clock_find_source(struct snd_usb_audio *chip, int entity_id,
On Tue, 29 Oct 2019 18:52:00 +0100, Saurav Girepunje wrote:
Use true/false for bool type return in uac_clock_source_is_valid().
Signed-off-by: Saurav Girepunje saurav.girepunje@gmail.com
Changes in v1:
Update the patch subject suggested by Takashi Iwai tiwai@suse.de
- Drop "!!" but keep the ternary operator in last return statement so that function will return always true/false.
Applied, thanks.
Takashi
participants (2)
-
Saurav Girepunje
-
Takashi Iwai