linux-kernel - Re: [PATCH 1/8] DMA: PL330: use prefix in reg names to build under x86

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 for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Fri, 23 Nov 2012 21:21:44 +0530
From:	viresh kumar <viresh.kumar@...aro.org>
To:	Davide Ciminaghi <ciminaghi@...dd.com>
Cc:	linux@....linux.org.uk, tglx@...utronix.de, mingo@...hat.com,
	hpa@...or.com, djbw@...com, vinod.koul@...el.com,
	grant.likely@...retlab.ca, linus.walleij@...aro.org,
	rubini@...dd.com, wim@...ana.be, cjb@...top.org,
	davidb@...eaurora.org, nico@...xnic.net, ben-linux@...ff.org,
	rajeev-dlh.kumar@...com, linux-kernel@...r.kernel.org,
	linux-arm-kernel@...ts.infradead.org, giancarlo.asnaghi@...com
Subject: Re: [PATCH 1/8] DMA: PL330: use prefix in reg names to build under x86

On Fri, Nov 23, 2012 at 8:38 PM, Davide Ciminaghi <ciminaghi@...dd.com> wrote:
> From: Alessandro Rubini <rubini@...dd.com>
>
> This driver would not compile if ARM_AMBA is selected under x86,
> because "CS" and "DS" are already defined there.  But AMBA
> is used in the x86 world by a PCI-to-AMBA bridge, to be submitted.
>
> The patch adds the "PL330_" prefix to all register and bit fields,
> so it can be built by randomconfig after ARM_AMBA appears within x86.
>
> Signed-off-by: Alessandro Rubini <rubini@...dd.com>
> Acked-by: Giancarlo Asnaghi <giancarlo.asnaghi@...com>

Hi Davide,

Its impossible to review due to obvious reasons :)

Though git provides an option to people to review such stuff easily.

git diff HEAD^.. --word-diff

and HEAD being the commit in discussion.

Problem, is people reviewing it need to apply it and not everybody
can do that. Don't know if adding this diff after the three --- line,
which form the non-commit log would be the right solution ?

--
viresh
--
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/