Код

struct pin_group lcd_pins = {
.pins = lcd_lms430_desc,
.nr_pins = ARRAY_SIZE(lcd_lms430_desc),
};

эти строки точно закомментированы, при этом на этапе компиляции ошибок не выдает... что делать? могу отправить свой конфиг (сюда не прикрепляется почему-то)

может вообще ничего не инициализировать, а только описать структуры пинов клавиатуры? драйвер судя по всему их сам инитит:

Код

/* initialized strobe lines as outputs, activated */
for (i = 0; i < pdata->num_col_gpios; i++) {
err = gpio_request(pdata->col_gpios[i], "matrix_kbd_col");
if (err) {
dev_err(&pdev->dev,
"failed to request GPIO%d for COL%d\n",
pdata->col_gpios[i], i);
goto err_free_cols;
}

gpio_direction_output(pdata->col_gpios[i], !pdata->active_low);

Убрал инициализацию всех ног клавы, оставил в ините борды только структуры и это:

Код

platform_device_register(&imx233_kbd_device);

теперь на gpio_request не ругается, но ругается на это:

Код

setting trigger mode 3 for irq 165 failed (stmp3xxx_set_irqtype+0x0/0xe0)
matrix-keypad matrix-keypad: Unable to acquire interrupt for GPIO line 37

не знаю, относится ли первая строчка к клаве? клава по прежнему не работает...

Похоже облом с matrix-keypad на imx233 - там требуется генерировать прерывания по обоим фронтам

Цитата

err = request_irq(gpio_to_irq(pdata->row_gpios[i]),
matrix_keypad_interrupt,
IRQF_DISABLED |
IRQF_TRIGGER_RISING | IRQF_TRIGGER_FALLING,
"matrix-keypad", keypad);

у imx233 контроллер PIO может только по одному фронту генерировать прерывание.

1) А на одном прерывании никак не получится?
2) Можно-ли ROWS настраивать не как GPIO (External interrupt например) ?
3) Есть ли еще способ организации пары десятков клавиш, поддерживаемый ядром линуха? может IO expander? (Это конечно крайний вариант)

1 см 2
2 не совсем понял вопрос, matrix-keypad работает с GPIO. Конечно можно изменить немного драйвер - достаточно одного прерывания, номер прерывания не используется - вся матрица сканируется после прерывания от любого row, но тогда схему матрицы нужно дополнять.
3 USB клавиатура :)

Можно изменить немного драйвер - зарегистрировать обработчики по одному фронту в зависимости от активного уровеня (уровень который соответствует нажатой клавише), он определен в бордфайле, у вас он
.active_low = 1,
Изначально предполагаем что все клавиши отпущены и мы ожидаем первого нажатия

Цитата

for (i = 0; i < pdata->num_row_gpios; i++) {
err = request_irq(gpio_to_irq(pdata->row_gpios[i]),
matrix_keypad_interrupt,
IRQF_DISABLED |
(pdata->active_low ? IRQF_TRIGGER_FALLING : IRQF_TRIGGER_RISING),
"matrix-keypad", keypad);

в самом обработчике irq никакой работы не делается - там только отключаются прерывания и ставится в очередь отложенный обработчик прерывания (matrix_keypad_scan), это нужно чтобы исключить влияние "дребезга" контактов. В отложенном обработчике сканируется вся матрица и определяются клавиши изменившие свое состояние - тут нам нужно переключать триггер для прерывания у row которые изменили состояние в зависимости от нового их состояния

Цитата

static void matrix_keypad_scan(struct work_struct *work)
....
for (row = 0; row < pdata->num_row_gpios; row++) {
if ((bits_changed & (1 << row)) == 0)
continue;

code = MATRIX_SCAN_CODE(row, col, keypad->row_shift);
input_event(input_dev, EV_MSC, MSC_SCAN, code);
input_report_key(input_dev,
keypad->keycodes[code],
new_state[col] & (1 << row));

if (!(new_state[col] & (1 << row))) // если клавиша отпущена
set_irq_type(gpio_to_irq(pdata->row_gpios[row]), pdata->active_low ? IRQ_TYPE_EDGE_FALLING : IRQ_TYPE_EDGE_RISING);
else
set_irq_type(gpio_to_irq(pdata->row_gpios[row]), pdata->active_low ? IRQ_TYPE_EDGE_RISING : IRQ_TYPE_EDGE_FALLING);
}

....
}

работать это должно но кривовато - например сразу непонятно - как поведет себя клава при одновременном нажатии клавиш с одинаковым row.

Пока отказался от матричной клавиатуры и завел кнопки просто на оставшиеся ноги МК. Как теперь организовать простую GPIO клавиатуру?

Цитата

Пока отказался от матричной клавиатуры и завел кнопки просто на оставшиеся ноги МК. Как теперь организовать простую GPIO клавиатуру?

http://www.starterkit.ru/html/index.php?name=forum&op=view&id=12359&num=2#12476