Re: [PATCH v2 01/13] conf/ucm: bytcr-rt5645: Use the generic bytcr/PlatformEnableSeq.conf

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

 



Dne 26.11.2018 v 15:18 Takashi Iwai napsal(a):
> On Mon, 26 Nov 2018 14:40:48 +0100,
> Hans de Goede wrote:
>>
>>> some of the recently added UCM profiles are the files to be included
>>> by others, and they are not card targets.  Unfortunately alsaucm
>>> doesn't know about it, and it aborts with error, e.g.
>>>
>>> ====
>>> % alsaucm
>>> ALSA lib utils.c:67:(uc_mgr_config_load) could not open configuration file /usr/share/alsa/ucm/bytcr/bytcr.conf
>>> ALSA lib parser.c:1427:(load_master_config) error: could not parse configuration for card bytcr
>>> alsaucm: unable to obtain card list: No such file or directory
>>> ====
>>>
>>> The similar error is found in nau8824, rt5640 and rt5651.  After
>>> putting a placeholder card config file (e.g. bytcr.conf), the error is
>>> gone.  But certainly we don't want to allow user to choose this as the
>>> proper card.
>>>
>>> So, I guess we need to put some flag to each such directory indicating
>>> that it's no card config.  For example, putting
>>> /usr/share/alsa/ucm/bytcr/.nocard file or such...
>>>
>>> Not sure whether it's wise to have a file starting with dot, though.
>>> Maybe clearer to be "nocard" or any other visible one?
>>
>> All the HiFi.conf files using the "include" snippets I added to avoid
>> copy and pasting a lot, already need to have this at the top for
>> this to work:
>>
>> <searchdir:ucm>
>>
>> So it might be best to move all the dirs holding include files rather
>> then proper card configs from /usr/share/alsa/ucm to
>> /usr/share/alsa/ucm-includes and then change the searchdir part of
>> the configs using these to:
>>
>> <searchdir:ucm-includes>
> 
> This sounds like a good solution, indeed.
> Care to submit a fix patch?  I'll apply it unless anyone objects.

Yep, it looks like a good idea.

				Jaroslav

-- 
Jaroslav Kysela <perex@xxxxxxxx>
Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel



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

  Powered by Linux