OK6410矩阵键盘驱动问题（linux-2.6.36.2）

飞凌键盘驱动怎么老是出问题阿。。。

涉及的文件：(内核版本linux-2.6.36.2)

file1: driver/input/keyboard/samsung-keypad.c

file2: arch/arm/plat-samsung/include/plat/keypad.h

file3: arch/arm/mach-s3c64xx/mach-s3c64xx.c

file4: arch/arm/plat-samsung/Dev-keypad.c

其中键盘的驱动代码主要为file1: samsung-keypad.c

在内核根目录，make menuconfig查看了内核的配置，默认使用了gpio-keys驱动，也就是开发板上的6个一排的小红色按键的驱动。而在此我们使用的矩阵键盘驱动应该使用samsung-keypad。内核默认没有编译，因此我们可以单独编译该模块加载运行。

照理说将file1拿出来单独编译成模块samsung-keypad.ko然后加载，就可以使用键盘了，但是世事难料，编译成功通过，但是加载后没有丝毫反应，于是在file1中的几个主要函数中添加了打印调试信息。再次加载发现，加载过程只进行到samsung_keypad_init（）。

也就是只是调用了  platform_driver_register(&samsung_keypad_driver);

而probe函数没用运行，由此发出疑问：是不是矩阵键盘对应的设备数据没有在Platformbus 总线上安装？

使用source insight 查看源代码 ，找到file3中定义的6410的设备列表：

staticstruct platform_device *smdk6410_devices[] __initdata = {

.。。。

&s3c_device_fb,

&s3c_device_ohci,

&s3c_device_usb_hsotg,

#if 0

&s3c64xx_device_iisv4,

&samsung_device_keypad,

#endif

 

&s3c64xx_device_ac97,

&s3c_device_adc,

。。。

};

发现其中果然将&samsung_device_keypad,给屏蔽掉了。

找到症结所在，去掉屏蔽，然后重新编译内核，编译驱动模块，加载运行，此时进入了probe函数，但是又出现了新的问题，内存错误，程序直接死掉。

通过加入调试信息，将问题锁定在probe函数中调用的matrix_keypad_build_keymap（）函数中，即：

matrix_keypad_build_keymap(keymap_data,row_shift, input_dev->keycode, input_dev->keybit);

 

使用source insight 查看，该函数如下：

staticinline void

matrix_keypad_build_keymap(conststruct matrix_keymap_data *keymap_data,unsigned int row_shift,unsigned short *keymap, unsigned long *keybit)

{

inti;

for(i = 0; i < keymap_data->keymap_size; i++) {

unsignedint key = keymap_data->keymap[i];

unsignedint row = KEY_ROW(key);

unsignedint col = KEY_COL(key);

unsignedshort code = KEY_VAL(key);

 

keymap[MATRIX_SCAN_CODE(row,col, row_shift)] = code;

__set_bit(code,keybit);

}

__clear_bit(KEY_RESERVED,keybit);

}

在调试时，我试图打印keymap_data->keymap_size发现结果很小的负数，打印keymap_data->keymap[i]的时候就会报内存出错，由此可见，keymap_data初始化出了问题。

 

继续查看源代码，在probe函数中，keymap_data= pdata->keymap_data; 而pdata= pdev->dev.platform_data;

即：keymap_data=pdev->dev.platform_data->keymap_data

其中，pdev也就是probe函数的参数：platform_device。.在这个模块中，应该就是file4:Dev-keypad.c文件中定义的：samsung_device_keypad，也即上面提到的6410设备列表中的samsung_device_keypad

structplatform_device samsung_device_keypad = {

.name ="samsung-keypad",

.id =-1,

.num_resources =ARRAY_SIZE(samsung_keypad_resources),

.resource =samsung_keypad_resources,

};

在系统开机时调用smdk6410_machine_init函数，在此函数中调用了

samsung_keypad_set_platdata(&smdk6410_keypad_data);

函数定义如下：

void__init samsung_keypad_set_platdata(struct samsung_keypad_platdata*pd)

{

structsamsung_keypad_platdata *npd;

npd= s3c_set_platdata(pd, sizeof(struct samsung_keypad_platdata),

&samsung_device_keypad);

 

if(!npd->cfg_gpio)

npd->cfg_gpio= samsung_keypad_cfg_gpio;

}

其中调用：

void__init *s3c_set_platdata(void *pd, size_t pdsize,

struct platform_device *pdev)

{

。。。

pdev->dev.platform_data= npd;

returnnpd;

}

所以pdev->dev.platform_data= &smdk6410_keypad_data

smdk6410_keypad_data及相关的数据结构定义如下：（在file3中）

 

static uint32_t smdk6410_keymap[] __initdata = {

/*KEY(row, col, keycode) */

KEY(0,3, KEY_1), KEY(0, 4, KEY_2), KEY(0, 5, KEY_3),

KEY(0,6, KEY_4), KEY(0, 7, KEY_5),

KEY(1,3, KEY_A), KEY(1, 4, KEY_B), KEY(1, 5, KEY_C),

KEY(1,6, KEY_D), KEY(1, 7, KEY_E)

};

static struct matrix_keymap_data smdk6410_keymap_data __initdata = { 
.keymap =smdk6410_keymap,

.keymap_size =ARRAY_SIZE(smdk6410_keymap),

};

static struct samsung_keypad_platdata smdk6410_keypad_data __initdata = { 
.keymap_data =&smdk6410_keymap_data,

.rows =2,

.cols =8,

};

到这里，我们可以知道，上面说的初始化有问题的keymap_data应该就是这里的smdk6410_keymap_data 了，如果说初始化有问题，那就是smdk6410_keypad_data有问题了，也即probe函数中的pdata有问题。

但是奇怪的是，我在probe函数中打印了pdata->rows,和pdata->cols确实是打印出2 & 8,那说明smdk6410_keymap_data没有问题阿。为何单独smdk6410_keymap_data的keymap_data字段出了问题呢，百思不得其解。有心去调试，可是发现该代码在内核代码中，不是模块可加载的代码，每次都需要编译内核、烧写内核，然后才能测试，比较麻烦。于是想在probe函数中动动手脚，绕过这块代码。

上面的这三个数据结构，主要就是提供了两方面的信息，一个是行和列的个数，第二个则是矩阵键盘的映射表。因此在probe函数中我们可以提供自己定义的行列值以及键盘映射表。

再看probe函数，该函数主要任务就是申请分配一些资源，以及对structsamsung_keypad *keypad; 和structinput_dev *input_dev; 两个数据结构进行初始化。

这里我定义了两个参数rows和cols代替原来的pdata->rows,pdata->cols.并全部初始化为8,假设64个键盘都用到了，当然以后可以按照需要随时改动。

之前调试遇到的内存错误问题出现在matrix_keypad_build_keymap(keymap_data,row_shift,input_dev->keycode,input_dev->keybit);函数中，

查看源代码发现该函数的主要功能就是对input_dev->keycode 和input_dev->keybit进行赋值初始化，因此我使用如下的代码代替该函数：

for(key= 0; key < keycodes_size; key++){

code= keypad->keycodes[key] = keypad_keycode[key];

if(code<=0)

continue;

__set_bit(code,input_dev->keybit);

}

__clear_bit(0,input_dev->keybit);

 

其中keycodes_size = rows ×cols = 8×8 =64

keypad->keycodes =input_dev->keycode

keypad_keycode·[]则是自己定义的键盘映射表，表内容如下：

unsignedshort keypad_keycode[] = {

KEY_A,KEY_B, KEY_C, KEY_D, KEY_E, KEY_F, KEY_G, KEY_H,

KEY_I,KEY_J, KEY_K, KEY_L, KEY_M, KEY_N, KEY_O, KEY_P,

KEY_Q,KEY_R, KEY_S, KEY_T, KEY_U, KEY_V, KEY_W, KEY_X,

KEY_Y,KEY_Z, KEY_1, KEY_2, KEY_3, KEY_4, KEY_5, KEY_6,

KEY_7,KEY_8, KEY_9, KEY_0, KEY_ENTER, KEY_ENTER, KEY_ENTER, KEY_ENTER,

KEY_F3,KEY_ENTER,KEY_6,KEY_Y, KEY_H, KEY_SPACE, 47, 48,

KEY_BACKSPACE,KEY_DOWN,KEY_7,KEY_U, KEY_J, KEY_N, 55, KEY_ENTER,

KEY_LEFT,KEY_RIGHT,KEY_9,KEY_8, KEY_I, KEY_K, KEY_B, 63, KEY_COMMA

};

为了方便管理，该表定义在file2: arch/arm/plat-samsung/include/plat/keypad.h中,表的内容以及排序可以根据自己需要随意改动。

至此，probe函数就可以检测成功了。。。

此外在samsung_keypad_scan还会遇到内存错误，该函数如下：

staticvoid samsung_keypad_scan(struct samsung_keypad *keypad,

unsignedint *row_state)

{

//structdevice *dev;

unsignedint col;

unsignedint val;

xyl_dbg("keypad_scanning...\n");

//dev= keypad->input_dev->dev.parent;

for(col = 0; col < keypad->cols; col++) {

/*if(samsung_keypad_is_s5pv210(dev))

{

xyl_dbg("is_s5pv210\n");

val= S5PV210_KEYIFCOLEN_MASK;

val&= ~(1 << col) << 8;

}

else

{*/

//xyl_dbg("is_not_s5pv210\n");

val= SAMSUNG_KEYIFCOL_MASK;

val&= ~(1 << col);

//}

writel(val,keypad->base + SAMSUNG_KEYIFCOL);

mdelay(1);

val= readl(keypad->base + SAMSUNG_KEYIFROW);

row_state[col]= ~val & ((1 << keypad->rows) - 1);

}

/*KEYIFCOL reg clear */

writel(0,keypad->base + SAMSUNG_KEYIFCOL);

}

出现内存错误的是dev= keypad->input_dev->dev.parent;语句，简单的处理办法为：注释调该语句，以及下面和s5pv210g相关的所有语句即可，如上面红色部分。

注意：2.6.28版本中键盘驱动存在的问题，该版本中依然存在，即：

voidsamsung_keypad_cfg_gpio(unsigned int rows, unsigned int cols)

{

unsignedint gpio;

unsignedint end;

 

/*Set all the necessary GPK pins to special-function 3: KP_ROW[x] */

end= S3C64XX_GPK(8 + rows);

for(gpio = S3C64XX_GPK(8); gpio < end; gpio++) {

s3c_gpio_cfgpin(gpio,S3C_GPIO_SFN(3));

s3c_gpio_setpull(gpio,S3C_GPIO_PULL_NONE);

}

 

/*Set all the necessary GPL pins to special-function 3: KP_COL[x] */

end= S3C64XX_GPL(0 + cols);

for(gpio = S3C64XX_GPL(0); gpio < end; gpio++) {

s3c_gpio_cfgpin(gpio,S3C_GPIO_SFN(3));

s3c_gpio_setpull(gpio,S3C_GPIO_PULL_NONE);

}

}

其中S3C_GPIO_PULL_NONE需要改成S3C_GPIO_PULL_UP

驱动终于可以用了。