2008/07/10 4:28, Jean-Christophe PLAGNIOL-VILLARD wrote:
>> +include $(TOPDIR)/config.mk
>> +
>> +LIB = lib$(BOARD).a
>> +
>> +COBJS := r0p7785lc0011rl.o selfcheck.o rtl8169_mac.o
>> +SOBJS := lowlevel_init.o
>> +
>> +$(LIB): $(COBJS) $(SOBJS)
>> + $(AR) crv $@ $(COBJS) $(SOBJS)
> please use $(ARFLAGS) instead of 'crv'
I will fix it.
>> +
>> +clean:
>> + rm -f $(SOBJS) $(OBJS)
>> +
>> +distclean: clean
>> + rm -f $(LIB) core *.bak .depend
> pelase add the $(obj)
I will fix it, too.
>> +
>> +#########################################################################
>> +
>> +.depend: Makefile $(SOBJS:.o=.S) $(OBJS:.o=.c)
>> + $(CC) -M $(CPPFLAGS) $(SOBJS:.o=.S) $(OBJS:.o=.c) > $@
>> +
>> +-include .depend
>> +
>> + * MA 02111-1307 USA
>> + */
>> +
>> +#define PCIREG_8(_adr) (*(volatile unsigned char *)(_adr))
>> +#define PCIREG_32(_adr) (*(volatile unsigned long *)(_adr))
>> +#define PCI_PAR PCIREG_32(0xfe0401c0)
>> +#define PCI_PDR PCIREG_32(0xfe040220)
>> +#define PCI_CR PCIREG_32(0xfe040100)
>> +#define PCI_CONF1 PCIREG_32(0xfe040004)
>> +
>> +#define EEPROM_ADD 0x00000050
>> +#define HIGH 1
>> +#define LOW 0
>> +
>> +#define PCI_PROG 0x80
>> +
>> +#define PCI_EEP_ADDRESS (unsigned short)0x0007
>> +#define PCI_MAC_ADDRESS_SIZE 3
>> +
>> +#define TIME1 100
>> +#define TIME2 20000
>
>> +#define BIT_DUMMY 0
>> +#define MAC_EEP_READ 1
>> +#define MAC_EEP_WRITE 2
>> +#define MAC_EEP_ERACE 3
>> +#define MAC_EEP_EWEN 4
>> +#define MAC_EEP_EWDS 5
>> +
>> +#define DEBUG 0
> Please remove
I will remove DEBUG.
> Could you add few comment about the data?
I'm sorry. I cannot add comment, because I don't know
detail of the data.
>> +
>> +/* RTL8169 */
>> +const unsigned short EEPROM_W_Data_8169_A[] = {
>> + 0x8129, 0x10ec, 0x8169, 0x1154, 0x032b,
>> + 0x4020, 0xa101
>> +};
>> +const unsigned short EEPROM_W_Data_8169_B[] = {
>> + 0x4d15, 0xf7c2, 0x8000, 0x0000, 0x0000, 0x1300,
>> + 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000,
>> + 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x2000,
>> + 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000,
>> + 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000,
>> + 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000,
>> + 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000
>> +};
>> +
>
>> +
>> +int do_set_mac(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
>> +{
>> + int i;
>> + unsigned char mac[6];
>> + char *s, *e;
>> +
>> + if (argc != 2) {
>> + printf("Usage:\n%s\n", cmdtp->usage);
>> + return 1;
>> + }
>> +
>> + s = argv[1];
>> +
>> + for (i = 0; i < 6; i++) {
>> + mac[i] = s ? simple_strtoul(s, &e, 16) : 0;
>> + if (s)
>> + s = (*e) ? e + 1 : e;
>> + }
>> + mac_write(mac);
>> +
>> + return 0;
>> +}
>> +
>> +U_BOOT_CMD(
>> + setmac, 2, 1, do_set_mac,
>> + "setmac - write MAC address for RTL8110SCL\n",
>> + "\n"
>> + "setmac <mac address> - write MAC address for RTL8110SCL\n"
>> +);
>> +
>> +int do_print_mac(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
>> +{
>> + int i;
>> + uchar mac[6];
>> +
>> + if (argc != 1) {
>> + printf("Usage:\n%s\n", cmdtp->usage);
>> + return 1;
>> + }
>> +
>> + mac_read();
>> +
>> + return 0;
>> +}
>> +
>> +U_BOOT_CMD(
>> + printmac, 1, 1, do_print_mac,
>> + "printmac - print MAC address for RTL8110\n",
>> + "\n"
>> + " - print MAC address for RTL8110\n"
>> +);
> Could you add a README for this board and describe its specific command?
Yes, I will add a README.
>> +
>> +
>> +U_BOOT_CMD(
>> + hwtest, 2, 1, do_hw_test,
>> + "hwtest - hardware test for R0P7785LC0011RL board\n",
>> + "\n"
>> + "hwtest all - test all hardware\n"
>> + "hwtest pld - output pld version\n"
>> + "hwtest led - test LED\n"
>> + "hwtest dipsw - test DIPSW\n"
>> + "hwtest sm107 - output SM107 version\n"
>> + "hwtest usb - test R8A66597\n"
>> + "hwtest i2c - output PCA9564 version\n"
>> + "hwtest net - compare RTL8110 ID\n"
>> + "hwtest sata - compare SiI3512 ID\n"
>> + "hwtest pci - output PCI slot device ID\n"
>> +);
> idem
>> +
> +
>> +#define CONFIG_NETMASK 255.255.255.0
>> +#define CONFIG_IPADDR 192.168.10.100
>> +#define CONFIG_SERVERIP 192.168.10.77
>> +#define CONFIG_GATEWAYIP 192.168.10.77
> please remove this 4 defines
OK. I will remove it.
>> +
>> +#define __io
>> +#define __mem_pci
> is it really the good place to define it??
I will remove it, Because It is unnecessary.
>> +
>> +#define CONFIG_PCI_MEM_BUS 0xFD000000 /* Memory space base addr */
>> +#define CONFIG_PCI_MEM_PHYS CONFIG_PCI_MEM_BUS
>> +#define CONFIG_PCI_MEM_SIZE 0x01000000 /* Size of Memory window */
>> +
Best Regards,
Yoshihiro Shimoda
-------------------------------------------------------------------------
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
_______________________________________________
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users
