[Etherboot-developers] Fwd: etherboot sis900 source code - problem

[Marty C. <md...@et...>]

Alright, here's a strange problem. It could indicate anything from a  
compiler bug to an alignment bug. I am forwarding it in case it makes  
sense to anyone else.
I've seen this sort of thing, but it is usually an initialization  
problem.
Does anyone have any ideas?  Clearly the "patch" would be, um,  
problematic ;)

Marty

Begin forwarded message:

> From: "Rajesh S. Ghanekar" <raj...@ne...>
> Date: Sat Nov 9, 2002  2:47:26  AM US/Eastern
> To: md...@th...
> Cc: raj...@ne...
> Subject: etherboot sis900 source code - problem
>
> Hi Marty,
>     First of all sorry that i am sending a personal mail. I have posted
> this in etherboot mailing list, but no luck with answer.
>     I was facing a problem with sis900 etherboot image file
> (sis900.lzdsk). I have on-board sis900 ethernet controller. So i
> downloaded the source code (5.0.7) and recent patches. After  
> recompiling
> still the problem persisted. So i edited sis900.c file and added a
> simple printf for testing. But that simple printf solved my problem and
> i got my machine booting on LAN. I was getting following error  
> messeges:
>
> sis900_probe: Error MAC address not found
> No adapter found
> <sleep>
> <abort>
>
>
> My edited sis900.c file is as follows :
>
> --------------------------- sis900.c ----------------------------------
>
> /* save for use later in sis900_reset() */
>     pci_revision = revision;
>     printf ("Netcore Solutions\n");
>
>     if (revision == SIS630E_900_REV)
>         ret = sis630e_get_mac_addr(pci, nic);
>     else if ((revision > 0x81) && (revision <= 0x90))
>         ret = sis635_get_mac_addr(pci, nic);
>     else
>         ret = sis900_get_mac_addr(pci, nic);
>
>
>     if (ret == 0)
>     {
>         printf ("sis900_probe: Error MAC address not found\n");
>         return NULL;
>     }
>
>     /* 630ET : set the mii access mode as software-mode */
>
> ----------------------------------------------------------------------- 
> --
>
> This is the change that i have added just for testing:
>
> printf ("Netcore Solutions\n");
>
>
> But it worked for my booting. I don't know how it worked because i am
> not a programmer. In the above sample code, the first if statement gets
> executed, i.e.
>      if (revision == SIS630E_900_REV)
>
> So i guess my chip revision is that. I thought this might be helpful to
> you, so i mailed you.
>
>
> Regards,
> Rajesh
>
>

-- 
     Try: http://rom-o-matic.net/ to make Etherboot images instantly.

    Name: Marty Connor
US Mail: Entity Cyber, Inc.; P.O. Box 391827;
          Cambridge, MA 02139; USA
   Voice: (617) 491-6935; Fax: (617) 491-7046
   Email: md...@et...
     Web: http://www.etherboot.org/