-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Daniel Gimpelevich wrote:
> Here is a patch against the current SVN that reflects what I built:
> 
> http://homepage.mac.com/danielg4/2066.patch
> 
> Can someone please take a look to see how obvious what I did wrong might
> be? Thanks.

Hi Daniel,
        I think that what you are doing wrong is using a780 as base for your
work, use a1200 instead.

        May i suggest a different way for adding a910 support? It will make
debugging much easier.

1. Start over, apply all patches up to ezx-core.patch.
2. quilt edit ezx-a910.c
3. copy ezx-a1200.c to ezx-a910.c
4. edit ezx-a910.c and s/a1200/a910/
5. add any gpio assignment you feel necessary to ezx.h, and the
Kconfig/Makefile entries for a910.
6. create a defconfig-a910 based on defconfig-1200.
7. quilt refresh.
8. compile and test, continue to 9 if you get a successful boot, fix it
(or report to ML) if it doesnt work.
9. copy all a1200-*.patch to a910-*.patch
10. edit all a910-*.patch, s/a1200/a910/
11. add all a910-*.patch to the series file, after the respective
a1200-*.patch reference.
12. quilt push until the next a910-*.patch
13. do any changes you feel necessary to properly support a910.
14. quilt refresh if you changed anything, compile and test.
15. goto 12.

        The most important thing after step 8 is PCAP, to test if it is working
enable the DEBUG macro on ezx-pcap.c, and assure that you are getting
valid register reads (no null and/or random values).

        Please, report your results.

        Thank you for your work, send me a diff to ezx-core.patch,
defconfig-a910, and the a910-*.patch files (in separate files please!)
and i will be happy to review/incorporate them.


- --
Daniel Ribeiro
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)

iD8DBQFHG/e+w3OYl0G0liQRAuU/AJ0VyJi5Jm6AW5Gnd0j7gKxyYplQ7gCfTEdK
P3xD+S5Pdr+/oJdK3gaz1HU=
=TrbM
-----END PGP SIGNATURE-----

Reply via email to