Re: [PATCH v4 1/2] ucm: Load device-specific configuration file based on the card long name

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> -----Original Message-----
> From: Takashi Iwai [mailto:tiwai@xxxxxxx]
> Sent: Monday, January 16, 2017 6:41 PM
> To: Lin, Mengdong <mengdong.lin@xxxxxxxxx>
> Cc: mengdong.lin@xxxxxxxxxxxxxxx; alsa-devel@xxxxxxxxxxxxxxxx;
> liam.r.girdwood@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH v4 1/2] ucm: Load device-specific configuration file based
> on the card long name
> 
> On Sun, 15 Jan 2017 11:01:15 +0100,
> Lin, Mengdong wrote:
> >
> > > -----Original Message-----
> > > From: Takashi Iwai [mailto:tiwai@xxxxxxx]
> > > Sent: Sunday, January 15, 2017 4:25 PM
> > >
> > > On Sat, 14 Jan 2017 09:23:55 +0100,
> > > mengdong.lin@xxxxxxxxxxxxxxx wrote:
> > > >
> > > > From: Mengdong Lin <mengdong.lin@xxxxxxxxxxxxxxx>
> > > >
> > > > Intel DSP platform drivers are used by many different devices. For
> > > > user space to differentiate them, ASoC machine drivers may use the
> > > > DMI info
> > > > (vendor-product-version-board) as card long name. Possible card
> > > > long names
> > > > are:
> > > > DellInc.-XPS139343-01-0310JH
> > > > ASUSTeKCOMPUTERINC.-T100TA-1.0-T100TA
> > > > Circuitco-MinnowboardMaxD0PLATFORM-D0-MinnowBoardMAX
> > > > ...
> > > >
> > > > And user space can define configuration files like
> > > > longname\longname.conf for a specific device.
> > >
> > > Do you mean a file name containing a backslash?
> > > I didn't find the relevant code in your patch...
> >
> > No, the file name doesn't contain a backslash. It means we're using the
> card long name as both the directory name that contains the UCM config file
> and the name of the config file itself.
> > E.g. for the laptop Dell XPS 13, UCM will try to find a directory "DellInc.-
> XPS139343-01-0310JH", and then find the file " DellInc.-XPS139343-01-
> 0310JH.conf" under the directory. If it cannot find the this file, it will fall back
> to open file "broadwell-rt286.conf" under directory "broadwell-rt286".
> 
> Ah, a backslash as on DOS.  Use a slash for Linux instead :)
> 
> 
> Takashi

Sorry, I fixed this in v6. Now the kernel side patches have been applied to ASoC so there is no dependency now.

Thanks
Mengdong
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux