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:	Thu, 13 Mar 2014 11:35:03 +0100
From:	Antoine Ténart 
	<antoine.tenart@...e-electrons.com>
To:	Sebastian Hesselbarth <sebastian.hesselbarth@...il.com>,
	Alexandre Belloni <alexandre.belloni@...e-electrons.com>
CC:	zmxu@...vell.com, jszhang@...vell.com,
	linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH 0/2] ARM: add initial support for the Marvell BG2-Q DMP

Hi Sebastian,

On 13/03/2014 10:51, Sebastian Hesselbarth wrote:
> On 03/12/2014 11:35 AM, Alexandre Belloni wrote:
>> On 12/03/2014 at 12:06:02 +0100, Antoine Ténart wrote :
>>> This serie adds the initial support for the Marvell BG2-Q DMP (part
>>> of the
>>> Berlin family). SoC has nodes for cpu, l2 cache controller, interrupt
>>> controllers, local timer, apb timers and uarts for now.
>>>
>>> Homepage: http://www.marvell.com/digital-entertainment/armada-1500-pro/
>>>
>>
>> I would move that description to the commit logs. Else one would have to
>> lookup the mailing list archive to read it.
>
> Antoine,
>
> thanks for posting this. I also prefer each patch to have a commit log
> no matter how obvious the contents may be. Please have a simple commit
> log for each of the single patches.
>
> You can leave out the link to marvell.com in the commit log as you
> update Marvell SoC documentation.

I'll cook up a v2 today. Thanks for the feedback !

-- 
Antoine Ténart, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ