lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4B0FD0B2.9070303@chelcom.ru>
Date:	Fri, 27 Nov 2009 18:14:26 +0500
From:	"A. Porodko" <panda@...lcom.ru>
To:	Mark Brown <broonie@...nsource.wolfsonmicro.com>
CC:	Chaithrika U S <chaithrika@...com>,
	'Liam Girdwood' <lrg@...mlogic.co.uk>,
	'Jaroslav Kysela' <perex@...ex.cz>,
	'Takashi Iwai' <tiwai@...e.de>,
	'Troy Kisky' <troy.kisky@...ndarydevices.com>,
	'David Brownell' <dbrownell@...rs.sourceforge.net>,
	'Kevin Hilman' <khilman@...prootsystems.com>,
	alsa-devel@...a-project.org, linux-kernel@...r.kernel.org
Subject: Re: [alsa-devel] Sound support for Neuros OSD2 Davinci based board.

Mark Brown wrote:
> On Thu, Nov 26, 2009 at 06:57:19PM +0530, Chaithrika U S wrote:
>
>   
>> @@ -246,6 +249,9 @@ static int __init evm_init(void)
>>  	if (machine_is_davinci_evm()) {
>>  		evm_snd_dev_data = &evm_snd_devdata;
>>  		index = 0;
>> +	} else if (machine_is_neuros_osd2()) {
>> +		evm_snd_dev_data = &evm_snd_devdata;
>> +		index = 0;
>>     
>
>   
>> The above can be combined with the check for DaVinci EVM
>> 	if (machine_is_davinci_evm() ||
>> 				(machine_is_neuros_osd2()) {
>>  		evm_snd_dev_data = &evm_snd_devdata;
>>  		index = 0;
>>     
>
> I raised the same query when Andrey sent an off-list copy to me by
> mistake - he said that in future he intends to extend the driver to 
> have additional features not present on the EVM (though for me that
> could always change the || later).
>
> Andrey, with Linux kernel patches reply to all is generally the way
> forward.
>   
Mark,

Understood. Shall I change if if to if || then?

-- 
Best regards
Andrey A. Porodko
 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ