linux-kernel - Re: [PATCHv2 00/15] Consolidate Orion platform information setup.

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] [day] [month] [year] [list]

Message-id: <alpine.LFD.2.00.1105161608340.14430@xanadu.home>
Date:	Mon, 16 May 2011 16:31:49 -0400 (EDT)
From:	Nicolas Pitre <nico@...xnic.net>
To:	Andrew Lunn <andrew@...n.ch>
Cc:	kernel@...tstofly.org, linux-kernel@...r.kernel.org,
	linux-arm-kernel@...ts.infradead.org, saeed.bishara@...il.com
Subject: Re: [PATCHv2 00/15] Consolidate Orion platform information setup.

On Sun, 15 May 2011, Andrew Lunn wrote:

> This patchset consolidates some of the code used for creating
> platform data for various orion based SoC, namely Dove, Loki, Kirkwood,
> mv78xx0 and Orion5x. Where possible, this code has been put into the
> orion platform, with the mach-*/common.c calling the common platform
> code.
> 
> Additionally the setup of the multi purpose pins has been consolidated.
> Kirkwood and mv78xx0 had identical code, which has been generalized and
> put into the platform. Orion5x and Dove mpp code has been re-written to 
> use the consolidated method.
> 
> Tested on a Kirkwood device, otherwise compile tested.
> 
> Changes since last version:
> 
>  - Typos found by Saeed Bishara fixed
>  - All ethernets reserve the same size
>  - Add Kirkwood RTC interrupt as a seperate patch
>  - Rather than drop the dove mpp code, refactor to use the platform code.

This looks great. Thanks for this work. That will certainly make any 
transition to device tree much simpler.

Some further consolidation could be achieved by dynamically allocating 
the platform device structures as needed and having a single function 
for each class of devices.  But that can be done in a subsequent series.


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