linux-kernel - Re: [PATCH 13/13] clocksource/drivers/dw

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: <a70866b1-cf62-ea03-e389-e08906e8580a@linaro.org>
Date:   Sun, 7 Oct 2018 14:16:17 +0200
From:   Daniel Lezcano <daniel.lezcano@...aro.org>
To:     Marek Vasut <marex@...x.de>, Thomas Gleixner <tglx@...utronix.de>
Cc:     Dinh Nguyen <dinguyen@...nel.org>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 13/13] clocksource/drivers/dw_apb: Add reset control

On 07/10/2018 14:10, Marek Vasut wrote:
> On 10/06/2018 07:07 PM, Thomas Gleixner wrote:
>> On Sat, 6 Oct 2018, Daniel Lezcano wrote:
>>
>>> On 06/10/2018 14:34, Thomas Gleixner wrote:
>>>> On Fri, 5 Oct 2018, Daniel Lezcano wrote:
>>>>
>>>>> From: Dinh Nguyen <dinguyen@...nel.org>
>>>>>
>>>>> Add code to retrieve the reset property from the dw-apb timers and if
>>>>> the property is available, the safe operation is to assert the timer
>>>>> into reset, and followed by a deassert of the timer reset (brings the
>>>>> timer out of reset).
>>>>>
>>>>> This patch is needed for systems where the bootloader has left the timer
>>>>> not used in reset.
>>>>>
>>>>>  - Trivial conflict with commit a74bd1ad7a:
>>>>>     "Convert to using %pOFn instead of device_node.name"
>>>>>
>>>>> Signed-off-by: Marek Vasut <marex@...x.de>
>>>>> Signed-off-by: Dinh Nguyen <dinguyen@...nel.org>
>>>>
>>>> Ingo just noticed, that this SOB chain is wrong. How is Marek involved
>>>> here? Can you please fix that up and send me another pull request?
>>>
>>> Thomas,
>>>
>>> I will fix it as soon as possible but I would like to give the
>>> opportunity to Dinh to explain why he sent the patch with Marek's SOB.
>>>
>>> Dinh ?
>>
>> No rush.
> 
> I reported this to Dinh and gave him some prototype patches to start
> with, since I didn't have the time to write a patch. You can change it
> to Reported-by: if you want.

Ok, thanks


-- 
 <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog