lkml.org 
[lkml]   [2019]   [Nov]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [alsa-devel] [PATCH 07/14] soundwire: add initial definitions for sdw_master_device
On 04-11-19, 08:42, Pierre-Louis Bossart wrote:
>
>
> On 11/3/19 1:30 AM, Vinod Koul wrote:
> > On 23-10-19, 16:28, Pierre-Louis Bossart wrote:
> > > Since we want an explicit support for the SoundWire Master device, add
> > > the definitions, following the Grey Bus example.
> > >
> > > Open: do we need to set a variable when dealing with the master uevent?
> >
> > I dont think we want that or we need that!
>
> In GreyBus there are events and variables set, not sure what they were used
> for. The code works without setting an event, but we'd need to make a
> conscious design decision, and I am not too sure what usersace would use the
> informatio for.
>
> >
> > And to prevent that rather than adding a variable, can you please
> > modify the device_type and use separate ones for master_device and
> > slave_device
>
> sorry, I don't get the comment. There is only already a different device
> type
>
>
> struct bus_type sdw_bus_type = {
> .name = "soundwire",
> .match = sdw_bus_match,
> .uevent = sdw_uevent,

We can remove this

> };
>
> struct device_type sdw_slave_type = {
> .name = "sdw_slave",
> .release = sdw_slave_release,

Add here:

uevent = sdw_uevent,

> };
>
> struct device_type sdw_md_type = {
> .name = "soundwire_master",
> .release = sdw_md_release,
> };

And not have here!

Problem solved!

--
~Vinod

\
 
 \ /
  Last update: 2019-11-08 05:05    [W:0.865 / U:0.040 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site