Various small consistency fixes (trivial):
authorUwe Hermann <uwe@hermann-uwe.de>
Fri, 11 Apr 2008 18:01:50 +0000 (18:01 +0000)
committerUwe Hermann <uwe@hermann-uwe.de>
Fri, 11 Apr 2008 18:01:50 +0000 (18:01 +0000)
 - Use _FOO_H include guard format everywhere.

 - Add missing speaker.c prototypes to libpayload.h.

 - Consistently use short form u8/u16/u32 instead of uint8_t et. al.

 - kcofig: Use 'depends on' instead of 'depends', which seems deprecated.

Signed-off-by: Uwe Hermann <uwe@hermann-uwe.de>
Acked-by: Uwe Hermann <uwe@hermann-uwe.de>
git-svn-id: svn://svn.coreboot.org/coreboot/trunk@3234 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1

15 files changed:
payloads/libpayload/Config.in
payloads/libpayload/curses/local.h
payloads/libpayload/drivers/video/font8x16.h
payloads/libpayload/i386/coreboot.c
payloads/libpayload/include/arch/io.h
payloads/libpayload/include/arch/msr.h
payloads/libpayload/include/arch/rdtsc.h
payloads/libpayload/include/arch/types.h
payloads/libpayload/include/coreboot_tables.h
payloads/libpayload/include/curses.h
payloads/libpayload/include/curses.priv.h
payloads/libpayload/include/libpayload.h
payloads/libpayload/include/pci.h
payloads/libpayload/include/sysinfo.h
payloads/libpayload/include/video_console.h

index 43f7b5e34fb193e350cc8115495a96e497cd36f5..ea869a5bdc09855dc81ab5af7b6f67425e745ece 100644 (file)
@@ -46,17 +46,17 @@ config SERIAL_CONSOLE
 
 config SERIAL_IOBASE
        hex "I/O base for the serial port (default 0x3f8)"
-       depends SERIAL_CONSOLE
+       depends on SERIAL_CONSOLE
        default 0x3f8
 
 config SERIAL_SET_SPEED
        bool "Override the serial console baud rate"
        default n
-       depends SERIAL_CONSOLE
+       depends on SERIAL_CONSOLE
 
 config SERIAL_BAUD_RATE
        int "Serial console baud rate (default 115200)"
-       depends SERIAL_SET_SPEED
+       depends on SERIAL_SET_SPEED
        default 115200
 
 config VIDEO_CONSOLE
index 7e1fb64b930b10d4197ad8a51d4793e58459cb22..0a861ceed86085b4d2de60168e9d5035339c9cdf 100644 (file)
@@ -27,8 +27,8 @@
  * SUCH DAMAGE.
  */
 
-#ifndef TINYCURSES_H
-#define TINYCURSES_H
+#ifndef _CURSES_LOCAL_H
+#define _CURSES_LOCAL_H
 
 /* For curses.priv.h: */
 #define USE_RCS_IDS 0
@@ -91,4 +91,4 @@ void speaker_enable(u16 freq);
 void speaker_disable(void);
 void speaker_tone(u16 freq, unsigned int duration);
 
-#endif                         /* TINYCURSES_H */
+#endif
index 02f40de5378ad4c4e6f2fa6389fdd3d2f9a00c9b..33b0319642deca35ada1612a28a51b8a802db557 100644 (file)
@@ -27,8 +27,8 @@
  * SUCH DAMAGE.
  */
 
-#ifndef FONT_8X16_H_
-#define FONT_8X16_H_
+#ifndef _DRIVERS_VIDEO_FONT8X16_H
+#define _DRIVERS_VIDEO_FONT8X16_H
 
 #ifdef HAVE_CONSOLE_FONT
 #error "You have already defined a console font!"
index 8e4a19a5eedeeccaed84e71dd456f9053398a339..e9bc40fc414e03779f01a32db198972b977f640b 100644 (file)
@@ -95,10 +95,10 @@ static int cb_parse_header(void *addr, int len, struct sysinfo_t *info)
                return 0;
 
        /* Make sure the checksums match. */
-       if (ipchksum((uint16_t *) header, sizeof(*header)) != 0)
+       if (ipchksum((u16 *) header, sizeof(*header)) != 0)
                return -1;
 
-       if (ipchksum((uint16_t *) (ptr + sizeof(*header)),
+       if (ipchksum((u16 *) (ptr + sizeof(*header)),
                     header->table_bytes) != header->table_checksum)
                return -1;
 
index dcf5bd97eaf799c40e53517d68ad6175ec2407fe..538b296a117683f6c7d9f69b460f6eae6a3be021 100644 (file)
@@ -27,8 +27,8 @@
  * SUCH DAMAGE.
  */
 
-#ifndef _X86_ARCH_IO_H_
-#define _X86_ARCH_IO_H_
+#ifndef _ARCH_IO_H
+#define _ARCH_IO_H
 
 #define readb(_a) (*(volatile unsigned char *) (_a))
 #define readw(_a) (*(volatile unsigned short *) (_a))
index 5985ab81d8dd5a425896a5d1e5b2105590d71b88..c28b56cde106d5e6c951aa418882a58b00c52f18 100644 (file)
@@ -27,8 +27,8 @@
  * SUCH DAMAGE.
  */
 
-#ifndef _ARCH_MSR_H_
-#define _ARCH_MSR_H_
+#ifndef _ARCH_MSR_H
+#define _ARCH_MSR_H
 
 static inline unsigned long long _rdmsr(unsigned int msr)
 {
index 1addedafc808429c62467ca9426cad442e63be8e..da7949d808db8e5e554801d5d4596a7e35b51017 100644 (file)
@@ -27,8 +27,8 @@
  * SUCH DAMAGE.
  */
 
-#ifndef _RDTSC_H_
-#define _RDTSC_H_
+#ifndef _ARCH_RDTSC_H
+#define _ARCH_RDTSC_H
 
 static inline unsigned long long rdtsc(void)
 {
index d1f3acda5bebabb2a1fd7a0d4696102738c521fd..641fb0aef83bb424de57cc2e479c1b4d8e890de6 100644 (file)
@@ -27,8 +27,8 @@
  * SUCH DAMAGE.
  */
 
-#ifndef _ARCH_X86_TYPES_H
-#define _ARCH_X86_TYPES_H
+#ifndef _ARCH_TYPES_H
+#define _ARCH_TYPES_H
 
 typedef unsigned char uint8_t;
 typedef unsigned char u8;
index f1563838620cf59c34d506aa228284387e10f463..88dc251cd881a1b5a950b40c1ada078a66a3529a 100644 (file)
  * SUCH DAMAGE.
  */
 
-#ifndef _COREBOOT_TABLES_H_
-#define _COREBOOT_TABLES_H_
+#ifndef _COREBOOT_TABLES_H
+#define _COREBOOT_TABLES_H
 
 #include <arch/types.h>
 
 struct cbuint64 {
-       uint32_t lo;
-       uint32_t hi;
+       u32 lo;
+       u32 hi;
 };
 
 struct cb_header {
-       uint8_t signature[4];
-       uint32_t header_bytes;
-       uint32_t header_checksum;
-       uint32_t table_bytes;
-       uint32_t table_checksum;
-       uint32_t table_entries;
+       u8 signature[4];
+       u32 header_bytes;
+       u32 header_checksum;
+       u32 table_bytes;
+       u32 table_checksum;
+       u32 table_entries;
 };
 
 struct cb_record {
-       uint32_t tag;
-       uint32_t size;
+       u32 tag;
+       u32 size;
 };
 
 #define CB_TAG_UNUSED     0x0000
@@ -57,7 +57,7 @@ struct cb_record {
 struct cb_memory_range {
        struct cbuint64 start;
        struct cbuint64 size;
-       uint32_t type;
+       u32 type;
 };
 
 #define CB_MEM_RAM      1
@@ -65,27 +65,27 @@ struct cb_memory_range {
 #define CB_MEM_TABLE    16
 
 struct cb_memory {
-       uint32_t tag;
-       uint32_t size;
+       u32 tag;
+       u32 size;
        struct cb_memory_range map[0];
 };
 
 #define CB_TAG_HWRPB      0x0002
 
 struct cb_hwrpb {
-       uint32_t tag;
-       uint32_t size;
-       uint64_t hwrpb;
+       u32 tag;
+       u32 size;
+       u64 hwrpb;
 };
 
 #define CB_TAG_MAINBOARD  0x0003
 
 struct cb_mainboard {
-       uint32_t tag;
-       uint32_t size;
-       uint8_t vendor_idx;
-       uint8_t part_number_idx;
-       uint8_t strings[0];
+       u32 tag;
+       u32 size;
+       u8 vendor_idx;
+       u8 part_number_idx;
+       u8 strings[0];
 };
 
 #define CB_TAG_VERSION        0x0004
@@ -100,25 +100,25 @@ struct cb_mainboard {
 #define CB_TAG_ASSEMBLER      0x000d
 
 struct cb_string {
-       uint32_t tag;
-       uint32_t size;
-       uint8_t string[0];
+       u32 tag;
+       u32 size;
+       u8 string[0];
 };
 
 #define CB_TAG_SERIAL         0x000f
 
 struct cb_serial {
-       uint32_t tag;
-       uint32_t size;
-       uint16_t ioport;
+       u32 tag;
+       u32 size;
+       u16 ioport;
 };
 
 #define CB_TAG_CONSOLE       0x00010
 
 struct cb_console {
-       uint32_t tag;
-       uint32_t size;
-       uint16_t type;
+       u32 tag;
+       u32 size;
+       u16 type;
 };
 
 #define CB_TAG_CONSOLE_SERIAL8250 0
@@ -136,7 +136,7 @@ struct cb_console {
        (((_rec)->size - sizeof(*(_rec))) / sizeof((_rec)->map[0]))
 
 #define MEM_RANGE_PTR(_rec, _idx) \
-       (((uint8_t *) (_rec)) + sizeof(*(_rec)) \
+       (((u8 *) (_rec)) + sizeof(*(_rec)) \
        + (sizeof((_rec)->map[0]) * (_idx)))
 
 #define MB_VENDOR_STRING(_mb) \
@@ -146,6 +146,6 @@ struct cb_console {
        (((unsigned char *) ((_mb)->strings)) + (_mb)->part_number_idx)
 
 #define UNPACK_CB64(_in) \
-       ( (((uint64_t) _in.hi) << 32) | _in.lo )
+       ( (((u64) _in.hi) << 32) | _in.lo )
 
 #endif
index dc64ad36af47779f50ee7a29e61ec341f6d72066..993bb3c962da9b245053ae58e01ae5824e95e150 100644 (file)
@@ -34,8 +34,8 @@
 
 /* $Id: curses.h.in,v 1.167 2006/11/26 01:14:54 tom Exp $ */
 
-#ifndef __NCURSES_H
-#define __NCURSES_H
+#ifndef _CURSES_H
+#define _CURSES_H
 
 #define CURSES 1
 #define CURSES_H 1
@@ -1673,4 +1673,4 @@ extern NCURSES_EXPORT(const char *) _nc_visbuf (const char *);
 void curses_enable_vga(int);
 void curses_enable_serial(int);
 
-#endif /* __NCURSES_H */
+#endif /* _CURSES_H */
index 72d9e82db10986304b97b056f3a0f9bb9ff20f1e..720e6789221d3852d32f8c43694f1100f3729289 100644 (file)
@@ -43,8 +43,8 @@
  *
  */
 
-#ifndef CURSES_PRIV_H
-#define CURSES_PRIV_H 1
+#ifndef _CURSES_PRIV_H
+#define _CURSES_PRIV_H 1
 
 //// XXX
 extern void *memset(void *s, int c, size_t len);
index 32deabf2b3b387916320a166fda0e56600d2c15b..e197e430fde893e304e004d961ef1d925b434942 100644 (file)
@@ -27,8 +27,8 @@
  * SUCH DAMAGE.
  */
 
-#ifndef LIBPAYLOAD_H
-#define LIBPAYLOAD_H
+#ifndef _LIBPAYLOAD_H
+#define _LIBPAYLOAD_H
 
 #include <autoconf.h>
 #include <stddef.h>
@@ -78,6 +78,11 @@ void serial_putchar(unsigned char c);
 int serial_havechar(void);
 int serial_getchar(void);
 
+/* drivers/speaker.c */
+void speaker_enable(u16 freq);
+void speaker_disable(void);
+void speaker_tone(u16 freq, unsigned int duration);
+
 /* video/video.c */
 int video_console_init(void);
 void video_console_putchar(unsigned int ch);
index 32ad3fb96d09565d5b8448e2c1f1ec602173a42f..df88e098ab5272be56707a71bd9dd4521d33c218 100644 (file)
@@ -27,8 +27,8 @@
  * SUCH DAMAGE.
  */
 
-#ifndef _PCI_H_
-#define _PCI_H_
+#ifndef _PCI_H
+#define _PCI_H
 
 typedef unsigned short pcidev_t;
 
index 50d331a043e6aabe8e8cbdec36dc0f4d6f007594..aebbff6481399a729a3d1cda60cf1c30919a3bb7 100644 (file)
@@ -27,8 +27,8 @@
  * SUCH DAMAGE.
  */
 
-#ifndef SYSINFO_H
-#define SYSINFO_H
+#ifndef _SYSINFO_H
+#define _SYSINFO_H
 
 /* Allow a maximum of 16 memory range definitions. */
 #define SYSINFO_MAX_MEM_RANGES 16
index 55deedf65ff5daa2c443ded14ead9c95302e2d0c..1646ec60567af8922b80e6f6fdd5e370dc582006 100644 (file)
@@ -27,8 +27,8 @@
  * SUCH DAMAGE.
  */
 
-#ifndef VIDEO_CONSOLE_H_
-#define VIDEO_CONSOLE_H_
+#ifndef _VIDEO_CONSOLE_H
+#define _VIDEO_CONSOLE_H
 
 #define VIDEO_ROWS 25
 #define VIDEO_COLS 80