[PATCH 2/6] ASoC: SOF: Introduce descriptors for SOF client

Sridharan, Ranjani ranjani.sridharan at intel.com
Thu Oct 1 18:48:44 CEST 2020


On Thu, 2020-10-01 at 14:38 +0100, Mark Brown wrote:
> On Wed, Sep 30, 2020 at 03:50:47PM -0700, Dave Ertman wrote:
> 
> > +	ret = ancillary_device_initialize(ancildev);
> > +	if (ret < 0) {
> > +		dev_err(sdev->dev, "error: failed to initialize client
> > dev %s\n", name);
> > +		ida_simple_remove(client_ida, ancildev->id);
> > +		goto err_free;
> > +	}
> > +
> > +	return cdev;
> > +
> > +err_free:
> > +	kfree(cdev);
> 
> It would be nice to have the ida_simple_remove() done in this unwind
> section here too for consistency.
ack
> 
> > +int sof_client_dev_register(struct snd_sof_dev *sdev, const char
> > *name, struct ida *client_ida)
> > +{
> 
> I know the hard limit on line length got raised but can we wrap this
> please?
ack.

Thanks,Ranjani


More information about the Alsa-devel mailing list