[PATCH 1/7] ARM: add GPMI support for imx23/imx28
Lothar Waßmann
LW at KARO-electronics.de
Thu Mar 17 06:36:22 EDT 2011
Hi,
Huang Shijie writes:
> Hi:
>
> > Huang Shijie writes:
> >> Hi Lothar:
> >>> Hi,
> >>>
> >>> All in all I do not like this driver in any way. I already vomitted
> >>> over it, when I saw it in the Freescale BSP.
> >> :) I have already removed a lot of code.
> >>
> >>> There is no need for the multiple layers of abstraction that are
> >>> papered over the MTD layer.
> >> I think this is good. Make the code clear.
> >>
> > I'll comment when the remaining patch has got through.
> >
> thanks a lot .
>
>
> >>> There is no need for spilling the driver over a dozen files.
> >> I can merge some files into one. thanks.
> >>
> > IMO all the source code could be kept in one file (except for the
> > include files of course) and the gpmi-nfc subdirectory removed.
> I really can merge some files, but IMHO, it's not a good solution to
> keep all in one file.
>
> The reasons are :
> [1] The GPMI support the NAND boot mode,but the imx23 and imx28 is a
> little different in
> the NAND boot procedure due the different firmwares in the chips. If I
> merge the rom-imx23.c
> rom-imx28.c to one file, the code will very mess. ugly enough!
>
You can use platform_ids to differentiate between the versions and
have separate functions for imx23 and imx28 where appropriate and
select between those depending on the platform_id.
Lothar Waßmann
--
___________________________________________________________
Ka-Ro electronics GmbH | Pascalstraße 22 | D - 52076 Aachen
Phone: +49 2408 1402-0 | Fax: +49 2408 1402-10
Geschäftsführer: Matthias Kaussen
Handelsregistereintrag: Amtsgericht Aachen, HRB 4996
www.karo-electronics.de | info at karo-electronics.de
___________________________________________________________
More information about the linux-arm-kernel
mailing list