Bug#612376: flash-kernel: please include efikamx support
On Mon, Apr 04, 2011, Hector Oron wrote:
> > Finally, is "mx51" going to be the name of the kernel flavour? Can
> > you have mx5 and mx6 in one kernel? What about different mx5x
> > platforms in one kernel?
> I am not sure which kernel flavour we should use, hence I dropped an
> email to firstname.lastname@example.org, there is contributed code, probably by
> ubuntu, which adds "imx51", "mx51", etc... I think that needs to be
> clarified, so I am working on it.
It would be nice if you could file a bug against linux to get the new
flavor; I personally would recommend -mx5. Currently, the upstream
kernel doesn't support building both i.MX51 and i.MX53 in the same
kernel but the Linux Linaro branch and the Freescale BSP kernel do;
also, I think the patches by Eric Miao allowing this (runtime phys
offset patches -- i.MX51 has phys RAM at 0x90000000 while i.MX53 has it
at 0x70000000, see arch/arm/mach-mx5/Makefile.boot) are pending in
Sacha Hauer's tree, but I'm not entirely sure.
Technically, mx5 would only support mx51 right now, but would soon
allow supporting mx53 as well, notably for mx53loco / quickstart boards
which I bet a bunch of Debian ARM users will get.
I reviewed your commits, some notes:
- fa8fdbc9607ebd892eba1e1c3b4ad12f710c901f Add support for Efika devices:
please sort machine names alphabetically in the case statement
- 7e6bd315830ad89c11aadebbe45f1636dd686055 README: Add Efika support:
kernel uses "nettop" in cpuinfo, but vendors uses "Smarttop"; also,
vendor uses "Efika MX" not just "Efika". I would prefer just the
vendor name in the README but you could have both with "Efika MX
Smarttop (nettop)"; the kernel name is "set in stone" now, but it's
not too nice
flash-kernel-installer.isinstallable: Add mx51 as subarchitecture:
I realize this comes from other parts of d-i, but would it make sense
to use arm*/mx5? My preference would be to avoid subarches as long
as possible but use upstream names when required; arch/arm/mach-mx5
is the current name, so that's what I'd use.
- 5deea668506d7a26a8b79fea218101c7900b9752 changelog: add Efika support
please amend the currently UNRELEASED changelog entry
Othewise looked good; thanks!
I'm also in favor of a boot.scr.
Outside of Martin's remarks, we need to agree on whether this is meant
for installation of Debian to PATA, to SD card, or both and whether
this is meant to work with vendor's u-boot, upstream u-boot, or both.