OSDev.org

kb.o: In function `keyboard_handler':
kb.c:(.text+0xe): undefined reference to `inportb'

>>>> unsigned short keyboard_handler()
{
    unsigned char scancode;

    /* Read from the keyboard's data buffer */
>>>> scancode = inportb(0x60);
    /* If the top bit of the byte we read from the keyboard is
    *  set, that means that a key has just been released */
    if (scancode & 0x80)
    {
        /* You can use this one to see if the user released the
        *  shift, alt, or control keys... */
    }
    else
    {
        /* Here, a key was just pressed. Please note that if you
        *  hold a key down, you will get repeated key press
        *  interrupts. */

        /* Just to show you how this works, we simply translate
        *  the keyboard scancode into an ASCII value, and then
        *  display it to the screen. You can get creative and
        *  use some flags to see if a shift is pressed and use a
        *  different layout, or you can add another 128 entries
        *  to the above layout to correspond to 'shift' being
        *  held. If shift is held using the larger lookup table,
        *  you would add 128 to the scancode when you look for it */
        print_text(kbdus[scancode]);
    }
	return (scancode,kbdus[scancode]);
}

I am trying use the bran tutorial kb.c to intergrate with my operating system.

JohnnyTheDon wrote:Undefined reference means inportb isn't being linked into your binary. Make sure you are running ld (or whatever you linker is) with the object file that contains inportb.

01000101 wrote:have you written an "inportb(x)" function?

the undefined reference (as stated previously) is due to the function "inportb" being left out from the linking process, or just not existing in the first place.

unsigned Keyboard_Data_Control()
{
 inportb(0x60);
}

cjhawley001 wrote:You need to write the function, "Inportb" is not defined anywhere. Your os does not have this function in it. So write it.

PatrickV wrote:Thanks. I just don't know where the this function is to go. Will it go on my Multi-boot header file, kernel.c ,or inside the kb.c file it's self. I never wrote a function before. I will do some research on it.

neon wrote: Just create a source file (io.c, perhaps?) inside of your kernel and define inportb() and outportb(). Then in your kernels include directory, create "io.h" and declare the two functions extern. Now, just #include <io.h> to use inportb() and outportb(). This is the way I would do it, anyways...

unsigned char inportb (unsigned short _port)
{
    	unsigned char rv;
    	__asm__ __volatile__ ("inb %1, %0" : "=a" (rv) : "dN" (_port));
    	return rv;
}

AlexExtreme wrote:Ok, so you're saying never written a function before, and you're trying to write an OS? If that's the case, how much of your code is directly copied and pasted from a tutorial?

http://wiki.osdev.org/Getting_Started#R ... _Knowledge - Learn to program in C first, and come back and try again when you have.