Re: [PATCH] gpio: mxs: implement get_direction callback

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




W dniu 2014-11-17 o 10:26, Richard Genoud pisze:
2014-11-17 0:59 GMT+01:00 Janusz Użycki <j.uzycki@xxxxxxxxxxxxxx>:
W dniu 2014-11-16 o 22:42, Uwe Kleine-König pisze:

Thanks Uwe. I fully agree with you.
a) was just a starter to your suggestion. My options were too conservative -
I just
wanted to avoid tests on hardware I don't have.
I don't understand why gpiod_get_direction() always requires the callback
and b) would be broken (I'm not so familiar with gpiolib) but I don't need
it now.

So, it looks we can drop the gpio-mxs patch, yes?
And, I or Richard should submit a patch for
mctrl_gpio/atmel_serial/mxs-auart
to introduce the irq helper, yes?
You're welcome to do it !
At the time the mctrl_helpers were introduced, there was only one user
(atmel_serial), so the line between specific code and factorizable
code was not so clear.
But clearly, the more we factorize, the better !

You wrote passing uart_port is enough. Argument "name" for request_irq() can
be
recovered from dev_name(dev) or dev_driver_string(dev)  where dev =
port_uart->dev.
And, honestly, I'm not sure dev_name(dev) is a good name.
Having something like dev_name(dev)_port_id_CTS may be better.


For names other than device's or driver's name I would need to allocate
string. Is it so important? You can simple check the interrupt and corresponding port:
cat /proc/interrupts
 50:          0  gpio-mxs  21  80072000.serial
 80:          0  gpio-mxs  18  8006c000.serial
 81:          0  gpio-mxs  19  8006c000.serial
 83:          0  gpio-mxs  21  8006c000.serial
 84:          0  gpio-mxs  22  8006c000.serial

best regards
Janusz

Seems ok !

Richard.

--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux PPP]     [Linux FS]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Linmodem]     [Device Mapper]     [Linux Kernel for ARM]

  Powered by Linux