Re: [PATCH] atmel ac97c: using software reset instead hardware reset if not available

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



On 12/16/2011 02:52 AM, Shen, Voice :
> Hi Nicolas,
>   No one response for this.
>   Would you please merge it?

Unfortunately I cannot.

You will have to make sure that the *sound* maintainer is taking your
patch into account. It can take several iterations but you must be sure
that your work is included into maintainer's git tree.

Good luck... Bye,

> -----Original Message-----
> From: Nicolas Ferre [mailto:nicolas.ferre@xxxxxxxxx] 
> Sent: Thursday, December 15, 2011 21:04 voice
> To: Shen, Voice
> Cc: linux-kernel@xxxxxxxxxxxxxxx; linux-sound@xxxxxxxxxxxxxxx; Lin, JM;
> Xu, Hong
> Subject: Re: [PATCH] atmel ac97c: using software reset instead hardware
> reset if not available
> 
> On 11/23/2011 11:05 AM, voice :
>> Using software reset instead hardware reset if not available
> 
> Ping?
> Can you please tell me what is the status of this fix? Will it go
> upstream for 3.3?
> 
>> Signed-off-by: Bo Shen <voice.shen@xxxxxxxxx>
>> ---
>>  sound/atmel/ac97c.c |    4 ++++
>>  1 files changed, 4 insertions(+), 0 deletions(-)
>>
>> diff --git a/sound/atmel/ac97c.c b/sound/atmel/ac97c.c
>> index 6e5adde..73516f6 100644
>> --- a/sound/atmel/ac97c.c
>> +++ b/sound/atmel/ac97c.c
>> @@ -899,6 +899,10 @@ static void atmel_ac97c_reset(struct atmel_ac97c
> *chip)
>>  		/* AC97 v2.2 specifications says minimum 1 us. */
>>  		udelay(2);
>>  		gpio_set_value(chip->reset_pin, 1);
>> +	} else {
>> +		ac97c_writel(chip, MR, AC97C_MR_WRST | AC97C_MR_ENA);
>> +		udelay(2);
>> +		ac97c_writel(chip, MR, AC97C_MR_ENA);
>>  	}
>>  }
>>  
> 
> 


-- 
Nicolas Ferre
--
To unsubscribe from this list: send the line "unsubscribe linux-sound" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Pulseaudio]     [Linux Audio Users]     [ALSA Devel]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux