[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <7c1d80b6-5db3-9955-0a67-908455bd77fa@linux.intel.com>
Date: Sun, 29 Jan 2023 14:54:43 +0800
From: Zhou Furong <furong.zhou@...ux.intel.com>
To: Greg KH <gregkh@...uxfoundation.org>,
Wesley Cheng <quic_wcheng@...cinc.com>
Cc: srinivas.kandagatla@...aro.org, mathias.nyman@...el.com,
perex@...ex.cz, lgirdwood@...il.com, andersson@...nel.org,
krzysztof.kozlowski+dt@...aro.org, Thinh.Nguyen@...opsys.com,
broonie@...nel.org, bgoswami@...cinc.com, tiwai@...e.com,
robh+dt@...nel.org, agross@...nel.org,
linux-kernel@...r.kernel.org, linux-arm-msm@...r.kernel.org,
alsa-devel@...a-project.org, devicetree@...r.kernel.org,
linux-usb@...r.kernel.org, quic_jackp@...cinc.com,
quic_plai@...cinc.com
Subject: Re: [RFC PATCH v2 07/22] ASoC: Add SOC USB APIs for adding an USB
backend
>> +void *snd_soc_usb_get_priv_data(struct device *usbdev)
>> +{
>> + struct snd_soc_usb *ctx;
>> +
>> + if (!usbdev)
>> + return NULL;
>
> How could usbdev ever be NULL?
The method is exported to public, valid check should be reasonable
as someone may call it by mistake
>
>> +
>> + ctx = snd_soc_find_usb_ctx(usbdev);
>> +
>> + return ctx ? ctx->priv_data : NULL;
>> +}
>> +EXPORT_SYMBOL_GPL(snd_soc_usb_get_priv_data);
>> +
Powered by blists - more mailing lists