linux-kernel - Re: [PATCH] regulator: vexpress: rename vexpress regulator implementation

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: <56C21981.6090604@arm.com>
Date:	Mon, 15 Feb 2016 18:31:29 +0000
From:	Sudeep Holla <sudeep.holla@....com>
To:	Mark Brown <broonie@...nel.org>
Cc:	Sudeep Holla <sudeep.holla@....com>, linux-kernel@...r.kernel.org,
	Rusty Russell <rusty@...tcorp.com.au>,
	Liviu Dudau <liviu.dudau@....com>,
	Lorenzo Pieralisi <lorenzo.pieralisi@....com>,
	Liam Girdwood <lgirdwood@...il.com>
Subject: Re: [PATCH] regulator: vexpress: rename vexpress regulator
 implementation



On 15/02/16 18:10, Mark Brown wrote:
> On Wed, Feb 10, 2016 at 06:49:54PM +0000, Sudeep Holla wrote:
>> On 29/01/16 17:37, Sudeep Holla wrote:
>
>>> The vexpress regulator implementation is currently just called vexpress.
>>> This is a problem because it clashes with another module with the same
>>> name in hardware monitors.
>
>>> This patch renames the vexpress regulator implementation to
>>> vexpress-regulator so that there will be no clash in the module namespace.
>
>> Can you take this patch for v4.6 ? It was reported @[1]
>
> Please allow a reasonable time for review.  People get busy, go on
> holiday, attend conferences and so on so unless there is some reason for
> urgency (like critical bug fixes) please allow at least a couple of
> weeks for review.  Sending content free pings just adds to the mail
> volume (if they are seen at all) and if something has gone wrong you'll
> have to resend the patches anyway.
>

Sorry for that. It was not a ping rather I just wanted to provide the
reference for the report since I realized I missed that initially.
I just asked the question along with that and didn't mean to show any
urgency. I do understand this patch is clearly not at all an urgent/
critical fix.

>> [1] https://lkml.org/lkml/2016/1/29/7
>
> Please include human readable descriptions of things like commits and
> issues being discussed in e-mail in your mails, this makes them much
> easier for humans to read especially when they have no internet access.
> I do frequently catch up on my mail on flights or while otherwise
> travelling so this is even more pressing for me than just being about
> making things a bit easier to read.
>

Understood. I knew lkml.org is broken if we tried very old links but
expected to work at-least for couple of month old patches. Anyways
hopefully [1] still works.

-- 
Regards,
Sudeep

[1] http://www.spinics.net/lists/linux-crypto/msg18316.html