[LinuxBIOS] #20: Cleanup of all CHIP_NAME() entries

LinuxBIOS svn at openbios.org
Sun Nov 5 20:56:44 CET 2006


#20: Cleanup of all CHIP_NAME() entries
----------------------------+-----------------------------------------------
     Reporter:  uwe         |          Owner:  uwe           
         Type:  defect      |         Status:  closed        
     Priority:  minor       |      Milestone:  Cosmetic fixes
    Component:  code        |        Version:  v2            
   Resolution:  fixed       |       Keywords:                
Include_gantt:  0           |   Dependencies:                
   Due_assign:  DD/MM/YYYY  |      Due_close:  DD/MM/YYYY    
----------------------------+-----------------------------------------------
Comment (by anonymous):

 > In my opinion we should go further and drop the CHIP_NAME macro
 completely.
 > So instead of CHIP_NAME("Via VT8263") it should just read "Via VT8263"

 No, it would read   .name = "Via VT8263"  .  _Always_ use designated
 initialisers, that's what they're there for (well there are exceptions of
 course, but those are just that: exceptions).

 > The CHIP_NAME macro was invented at a time when we were really tight in
 rom space, but it is obviously the wrong approach.

 I don't see how a macro would help any in this regard?


 Segher

-- 
Ticket URL: <http://tracker.linuxbios.org/trac/LinuxBIOS/ticket/20#comment:3>
LinuxBIOS <http://www.linuxbios.org/>


More information about the coreboot mailing list