lkml.org 
[lkml]   [2019]   [Dec]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [alsa-devel] [PATCH v5 06/17] soundwire: add support for sdw_slave_type
On 27-12-19, 17:26, Pierre-Louis Bossart wrote:
>
> > > static int sdw_uevent(struct device *dev, struct kobj_uevent_env *env)
> > > {
> > > - struct sdw_slave *slave = to_sdw_slave_device(dev);
> > > + struct sdw_slave *slave;
> > > char modalias[32];
> > > - sdw_slave_modalias(slave, modalias, sizeof(modalias));
> > > -
> > > - if (add_uevent_var(env, "MODALIAS=%s", modalias))
> > > - return -ENOMEM;
> > > + if (is_sdw_slave(dev)) {
> > > + slave = to_sdw_slave_device(dev);
> > > +
> > > + sdw_slave_modalias(slave, modalias, sizeof(modalias));
> > > +
> > > + if (add_uevent_var(env, "MODALIAS=%s", modalias))
> > > + return -ENOMEM;
> > > + } else {
> > > + /*
> > > + * We only need to handle uevents for the Slave device
> > > + * type. This error cannot happen unless the .uevent
> > > + * callback is set to use this function for a
> > > + * different device type (e.g. Master or Monitor)
> > > + */
> > > + dev_err(dev, "uevent for unknown Soundwire type\n");
> > > + return -EINVAL;
> >
> > At this point and after next patch, the above code would be a no-op, do
> > we want this here, if so why?
>
> to be future proof if someone wants to add support for a monitor, as
> explained above.
> I can remove this if you don't want it.

It can be added with monitor support whenever that comes. We dont like
dead code in kernel, the piece can come when future arrives :)

--
~Vinod

\
 
 \ /
  Last update: 2019-12-28 13:07    [W:0.072 / U:0.452 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site