linux-kernel - Re: [GIT PULL] ARM: amba: Enable module alias autogeneration for AMBA drivers

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:	Tue, 22 Nov 2011 12:07:56 +0000
From:	Dave Martin <dave.martin@...aro.org>
To:	Alessandro Rubini <rubini@...pv.it>
Cc:	linux@....linux.org.uk, patches@...aro.org,
	linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
	alan@...ux.intel.com, a.zummo@...ertech.it,
	alsa-devel@...a-project.org, cjb@...top.org,
	dan.j.williams@...el.com, dmitry.torokhov@...il.com,
	grant.likely@...retlab.ca, perex@...ex.cz,
	jassisinghbrar@...il.com, julia@...u.dk, linus.walleij@...aro.org,
	linux-fbdev@...r.kernel.org, linux-input@...r.kernel.org,
	linux-mmc@...r.kernel.org, linux-serial@...r.kernel.org,
	linux-watchdog@...r.kernel.org, lethal@...ux-sh.org,
	Pawel.Moll@....com, rtc-linux@...glegroups.com,
	spi-devel-general@...ts.sourceforge.net, tiwai@...e.de,
	vinod.koul@...el.com, wim@...ana.be
Subject: Re: [GIT PULL] ARM: amba: Enable module alias autogeneration for
 AMBA drivers

On Tue, Nov 22, 2011 at 12:56:32PM +0100, Alessandro Rubini wrote:
> > I have Acks on some of the driver patches and no comments on the
> > rest.  I've been circulating these for some time, so if you're
> > happy to pull those driver patches via your tree, please go ahead.
> 
> Sure I have no problem with the nomadik-rng.c, (so please have my
> Acked-by: -- sorry for not noting it earlier) but I think this

thanks

> file2alias thing needs a serious rework to split each bus in a
> different file, using ELF sections like we do elsewhere.
> 
> I posted a patch last week but I got no comment at all. Maybe this
> is the time to ask for comments? I can rebase and repost if needed.
> 
>   lkml.org/lkml/2011/11/4/125 (0/2: description of the work)
>   lkml.org/lkml/2011/11/4/126 (1/2: turn the if plethora to table lookup)
>   lkml.org/lkml/2011/11/4/127 (2/2: partial split to type-specific files).

Splitting this up doesn't look like a silly idea -- I thought it was
unfortunate to have to patch a core file just to add a bus type.

I guess the question is how many new buses we get over time.  If they
only get added gradually, people might not be too concerned.  Of course,
this is one of those maintenance issues which can only get worse as time
goes on.

Cheers
---Dave
--
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/