linux-kernel - Re: [PATCH 3/6] regulator: rk808: Make rk808 generic, rename rk808-regulator.c

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:	Fri, 15 Apr 2016 18:24:06 +0200
From:	Alexandre Belloni <alexandre.belloni@...e-electrons.com>
To:	Mark Brown <broonie@...nel.org>
Cc:	Wadim Egorov <w.egorov@...tec.de>, linux-kernel@...r.kernel.org,
	linux-arm-kernel@...ts.infradead.org, linux-clk@...r.kernel.org,
	rtc-linux@...glegroups.com, linux-rockchip@...ts.infradead.org,
	robh+dt@...nel.org, pawel.moll@....com, mark.rutland@....com,
	ijc+devicetree@...lion.org.uk, galak@...eaurora.org,
	linux@....linux.org.uk, mturquette@...libre.com,
	sboyd@...eaurora.org, lee.jones@...aro.org, lgirdwood@...il.com,
	a.zummo@...ertech.it, dianders@...omium.org, zyw@...k-chips.com,
	arnd@...db.de, k.kozlowski@...sung.com, javier@....samsung.com,
	kgene@...nel.org, olof@...om.net, geert+renesas@...der.be,
	sjoerd.simons@...labora.co.uk, treding@...dia.com,
	public_timo.s@...entcreek.de
Subject: Re: [PATCH 3/6] regulator: rk808: Make rk808 generic, rename
 rk808-regulator.c

On 15/04/2016 at 11:26:02 +0100, Mark Brown wrote :
> On Fri, Apr 15, 2016 at 11:16:27AM +0200, Wadim Egorov wrote:
> 
> > This patch just renames the rk808 driver so we can reuse this driver
> > to add more regulator devices from the RK8XX PMIC family later.
> 
> We normally manage to cope fine without renaming things - it's quite
> common to have drivers that were initially named after one part
> supporting multiple parts, avoiding the rename makes things like
> backports easier and avoids needing git log --follow and so on.

I agree with that and I will also add that probably at some point in
time a new part may appear with a name matching rk8xx but may not be
compatible with the rk8xx driver and that is when the real mess starts.

Finally, renaming the config options and the modules is not nice for the
user.

-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com