[PATCH V4 0/4] MPC85xx support

Sascha Hauer s.hauer at pengutronix.de
Thu May 3 03:12:33 EDT 2012

Hi Renaud,

On Tue, May 01, 2012 at 10:25:59AM +0100, Renaud Barbier wrote:
> The patchset V4 introduces the MPC85xx architecture to Barebox and
> addresses the comments of the version 3. The existing PPC code is
> modified to enable multiple PowerPC architecture support.
> All existing architecture specific MPC5xxx code which prevents
> adding additional architectures is moved into the architecture specific
> subdirectories and the PCM030 configuration fixed to use this new
> configuration.
> Base MPC85xx infrastructure is added to enable the required
> functionality for a minimal boot target in 32 bit addressing mode
> using an e500v2 core. To address V3 comments global variables that
> helped handling TLBs and LAWs are replaced by search functions.
> Finally support is added for a minimal Freescale P2020RDB platform with
> driver support currently limited to NOR and serial port only.
> Renaud Barbier (4):
>   Preparation stage to support multiple PPC architectures
>   Minimal support of the MPC85xx architecture
>   Header files update to support the mpc85xx.
>   Minimal P2020RDB platform support and configuration file

You might have noticed I applied parts of your patches, namely 1/4 and
the bulk of 3/4. A git rebase should make your patches a bit smaller and
should make you less dependent on future ppc changes.

Generally smaller patches are easier to review and will make merging
your series faster, so it would be nice if you could split this up a bit
more. This is especially important for the parts of the patches that
affect other users like mpc5200 aswell.

Sorry for not replying with a more detailed review, but your patch is
quite big and my spare time is rather small at the moment.


Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

More information about the barebox mailing list