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]

Date:   Sun, 7 Oct 2018 14:10:38 +0200
From:   Marek Vasut <marex@...x.de>
To:     Thomas Gleixner <tglx@...utronix.de>,
        Daniel Lezcano <daniel.lezcano@...aro.org>
Cc:     Dinh Nguyen <dinguyen@...nel.org>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 13/13] clocksource/drivers/dw_apb: Add reset control

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.

-- 
Best regards,
Marek Vasut