[U-Boot-Users] [PATCH] Refreshed 85xx and 83xx patches

Wolfgang,
Here is a URL referencing a refresh of all of the outstanding 85xx and 83xx U-Boot patches.
http://www.jdl.com/pub/software/u-boot-jdl.patch
These patches are summarized like this:
- All outstanding 85{40,60}ADS patches - All outstanging 85{55,41}CDS patches - New 8548CDS board support - TSEC driver moved from cpu/85xx to drivers/ - New 8349ADS board support - New 8540EVAL board support
It does not contain the I2C patch Adam sent for 8548. It does not contain the Murray Jensen CPM2 I/O pin functions patch.
These patches are 17K lines at 500K. If people would also like them posted here, let me know.
Thanks, jdl

In message 1122660203.7750.24.camel@cashmere.sps.mot.com you wrote:
Here is a URL referencing a refresh of all of the outstanding 85xx and 83xx U-Boot patches.
Sorry, but I have to reject this monster patch. Please stick to the coding style. A quick scan showed the following problems:
* C++ comments in
board/mpc8349ads/mpc8349ads.c cpu/mpc83xx/pci.c doc/README.PIP405 drivers/sk98lin/skgeinit.c drivers/sk98lin/skxmac2.c include/configs/MPC8349ADS.h
* Trailing white space in
common/cmd_flash.c common/cmd_usb.c common/usb.c common/usb_storage.c cpu/arm920t/s3c24x0/usb_ohci.c cpu/mpc5xxx/usb_ohci.c cpu/mpc83xx/pci.c cpu/mpc83xx/resetvec.S cpu/mpc83xx/spd_sdram.c cpu/mpc83xx/speed.c cpu/mpc83xx/start.S cpu/mpc85xx/spd_sdram.c drivers/tsec.c include/asm-ppc/e300.h include/asm-ppc/i2c.h include/asm-ppc/immap_83xx.h include/mpc83xx.h
* Indentation not by TAB in
doc/README.mpc83xxads include/asm-ppc/immap_83xx.h include/configs/MPC8349ADS.h include/configs/MPC8540EVAL.h
* Excessive amount of empty lines in
board/LEOX/elpt860/README.LEOX board/mpc8540eval/mpc8540eval.c cpu/arm920t/s3c24x0/usb_ohci.c cpu/mpc83xx/spd_sdram.c include/asm-ppc/immap_85xx.h include/configs/MPC8349ADS.h
* Trailing empty lines in
board/mpc8349ads/mpc8349ads.c cpu/mpc83xx/cpu_init.c cpu/mpc83xx/speed.c include/spd.h
Also, the patch contains stuff that does not belong into it, like this chunk:
Merge with rsync://git-user@source.denx.net/git/u-boot.git
Please cleanup and resubmit.
Best regards,
Wolfgang Denk

On Fri, 2005-07-29 at 15:19, Wolfgang Denk wrote:
Sorry, but I have to reject this monster patch. Please stick to the coding style. A quick scan showed the following problems:
- C++ comments in
- Trailing white space in
- Indentation not by TAB in
- Excessive amount of empty lines in
- Trailing empty lines in
OK, I'll fix those.
Also, the patch contains stuff that does not belong into it, like this chunk:
But GIT doesn't care! It knows!
Merge with rsync://git-user@source.denx.net/git/u-boot.git
When I posted it, it was merged, I believe. Naturally, I'll update it.
Please cleanup and resubmit.
Will do.
jdl

In message 1122669513.7750.39.camel@cashmere.sps.mot.com you wrote:
OK, I'll fix those.
Thanks.
Also, the patch contains stuff that does not belong into it, like this chunk:
But GIT doesn't care! It knows!
This was a patch sent to the public mailing list, and here we don't know much about git yet. We use CVS, and your patch does not apply cleanly to this tree.
[Sorry for being so anal retentive here, but remember that there are others on this list who might be interested in using your patch.]
Best regards,
Wolfgang Denk

On Fri, 2005-07-29 at 15:38, Jon Loeliger wrote:
On Fri, 2005-07-29 at 15:19, Wolfgang Denk wrote:
Sorry, but I have to reject this monster patch. Please stick to the coding style. A quick scan showed the following problems:
- C++ comments in
- Trailing white space in
- Indentation not by TAB in
- Excessive amount of empty lines in
- Trailing empty lines in
OK, I'll fix those.
I have fixed up the request style issues now.
Naturally, I'll update it.
Please cleanup and resubmit.
Will do.
I have also remerged with the current head as of a couple hours ago. Also, I've brought up an rsync server, so my git trees at http://www.jdl.com/git_repos/ will feed you http or rsync as needed.
jdl

In message 1123024299.2614.37.camel@cashmere.sps.mot.com you wrote:
I have fixed up the request style issues now.
Thanks.
I have also remerged with the current head as of a couple hours ago. Also, I've brought up an rsync server, so my git trees at http://www.jdl.com/git_repos/ will feed you http or rsync as needed.
Thanks, worked fine for me.
I understand that it would cause an unacceptable anmount of work for you to re-unwind all these modifications, but please note that this is a one time exception I will make. From now on please use separate branches for boards and processors as discussed.
I see the following problems with your new code (I guess you missed to run the MAKEALL script ???)
MPC8349ADS:
cpu.c: In function `do_reset': cpu.c:77: warning: unused variable `addr' pci.c: In function `pci_mpc83xx_init': pci.c:119: warning: implicit declaration of function `i2c_init' pci.c:122: warning: implicit declaration of function `i2c_write' pci.c:172: warning: passing arg 1 of `pci_set_region' discards qualifiers from pointer target type pci.c:178: warning: passing arg 1 of `pci_set_region' discards qualifiers from pointer target type pci.c:184: warning: passing arg 1 of `pci_setup_indirect' discards qualifiers from pointer target type pci.c:188: warning: passing arg 1 of `pci_hose_write_config_byte' discards qualifiers from pointer target type pci.c:190: warning: passing arg 1 of `pci_hose_read_config_word' discards qualifiers from pointer target type pci.c:192: warning: passing arg 1 of `pci_hose_write_config_word' discards qualifiers from pointer target type pci.c:197: warning: passing arg 1 of `pci_hose_write_config_word' discards qualifiers from pointer target type pci.c:198: warning: passing arg 1 of `pci_hose_write_config_byte' discards qualifiers from pointer target type pci.c:207: warning: passing arg 1 of `pci_set_region' discards qualifiers from pointer target type pci.c:213: warning: passing arg 1 of `pci_set_region' discards qualifiers from pointer target type pci.c:219: warning: passing arg 1 of `pci_setup_indirect' discards qualifiers from pointer target type pci.c:221: warning: passing arg 1 of `pci_hose_write_config_byte' discards qualifiers from pointer target type pci.c:222: warning: passing arg 1 of `pci_hose_read_config_word' discards qualifiers from pointer target type pci.c:224: warning: passing arg 1 of `pci_hose_write_config_word' discards qualifiers from pointer target type pci.c:229: warning: passing arg 1 of `pci_hose_write_config_word' discards qualifiers from pointer target type pci.c:230: warning: passing arg 1 of `pci_hose_write_config_byte' discards qualifiers from pointer target type pci.c:247: warning: passing arg 1 of `pci_register_hose' discards qualifiers from pointer target type pci.c:248: warning: passing arg 1 of `pci_hose_scan' discards qualifiers from pointer target type pci.c:54: warning: unused variable `tmp8' pci.c:55: warning: unused variable `tmp16' spd_sdram.c:84:2: warning: #warning Current spd_sdram does not fit its usage... adjust implementation or API...
Please cleanup.
Best regards,
Wolfgang Denk

On Wed, 2005-08-03 at 04:31, Wolfgang Denk wrote:
In message 1123024299.2614.37.camel@cashmere.sps.mot.com you wrote:
I have fixed up the request style issues now.
Thanks.
I have also remerged with the current head as of a couple hours ago. Also, I've brought up an rsync server, so my git trees at http://www.jdl.com/git_repos/ will feed you http or rsync as needed.
Thanks, worked fine for me.
Good to hear!
I understand that it would cause an unacceptable anmount of work for you to re-unwind all these modifications, but please note that this is a one time exception I will make. From now on please use separate branches for boards and processors as discussed.
Thanks. Will do.
I see the following problems with your new code (I guess you missed to run the MAKEALL script ???)
I ran MAKEALL on 85xx and 83xx boards.
MPC8349ADS:
cpu.c: In function `do_reset': cpu.c:77: warning: unused variable `addr' pci.c: In function `pci_mpc83xx_init': pci.c:119: warning: implicit declaration of function `i2c_init' pci.c:122: warning: implicit declaration of function `i2c_write' ... pci.c:247: warning: passing arg 1 of `pci_register_hose' discards qualifiers from pointer target type pci.c:248: warning: passing arg 1 of `pci_hose_scan' discards qualifiers from pointer target type pci.c:54: warning: unused variable `tmp8' pci.c:55: warning: unused variable `tmp16' spd_sdram.c:84:2: warning: #warning Current spd_sdram does not fit its usage... adjust implementation or API...
Grumble. They didn't hit the LOG/ file. I'll try to update an 8349 (branch) and clean some of this up!
Thanks, jdl
participants (2)
-
Jon Loeliger
-
Wolfgang Denk