[PATCH] rtlwifi: Change long delays to sleeps

ByeoungWook Kim quddnr145 at gmail.com
Sat Feb 20 14:29:09 UTC 2016


2016-02-20 23:06 GMT+09:00 Larry Finger <Larry.Finger at lwfinger.net>:
> On 02/19/2016 11:48 PM, ByeoungWook Kim wrote:
>>
>> 2016-02-16 7:12 GMT+09:00 Larry Finger <Larry.Finger at lwfinger.net>:
>>>
>>> Routine rtl_addr_delay() uses delay statements in code that can
>>> sleep. To improve system responsiveness, the various delay statements
>>> are changed.
>>>
>>> In addition, routines rtl_rfreg_delay() and rtl_bb_delay() are
>>> rewritten to use the code in rtl_addr_delay() for most of their
>>> input values.
>>>
>>> Suggested-by: Byeoungwook Kim <quddnr145 at gmail.com>
>>> Signed-off-by: Larry Finger <Larry.Finger at lwfinger.net>
>>> ---
>>
>>
>> Why you are using 'Suggested-by:' of me?
>> I think to that your commit was included a part of my
>> commit(https://patchwork.kernel.org/patch/8197071/).
>
>
> Your commit was placed in the deferred list of patches. It did lead me to
> look at that section and realize that using delays there was a bug, and that
> it should be fixed as quickly as possible. That argues against waiting for
> your patches to be implemented. I did use some of your changes. How do you
> think I should have referenced your material?
>
> Larry
>
>

I'm understood your mail.
Only, I wondered why 'Suggested-by' was used.

Thank for your answer.

Regards,
Byeoungwook


More information about the devel mailing list