Re: [alsa-devel] [Device-drivers-devel] [PATCH] Add driver for AnalogDevices ADAU1701 SigmaDSP

[Mike Frysinger]

On Wed, Mar 9, 2011 at 05:00, Mark Brown wrote:
> On Wed, Mar 09, 2011 at 03:25:05PM +0800, Cliff Cai wrote:
>> On Mon, Mar 7, 2011 at 8:29 PM, Mike Frysinger wrote:
>> > i dont see the issue here. Âthe firmware is *optional* and does not
>> > impair basic audio output. Âfurther, the firmware is fully
>> > written/compiled/maintained by the end customer, just like the
>> > application. Âwhich means there is no "magic" here -- the end customer
>> > is the wizard.
>>
>> it's a DSP,so firmware is not optional,actually there is default
>> internal program can be used
>> if no external firmware is downloaded,of cause, the internal program
>> is only used to test analog audio pass-through.
>
> If there is default firmware then even if it's not particularly useful
> the kernel should probably not *require* that additional firmware is
> provided during driver startup - the system may wish to wait until later
> in the boot to provide it, for example because it needs to mount media
> to get to the firmware or because it needs to do additional work to
> decide what firmware is required for the current system setup.

sorry, i was thinking this part was like the other ADI ones where the
DSP is there as an enhancement.  so many of my comments were tailored
towards that.

there shouldnt be any problem with us providing a stub firmware with
the kernel (in firmware/) that simply does a "pass through" or
whatever.
-mike
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/