Thread (16 messages) 16 messages, 6 authors, 2016-01-29

Re: [PATCH 0/3] 8250: Split Fintek PCIE to UART to independent file

From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Date: 2016-01-19 13:21:18
Also in: lkml

On Tue, 2016-01-19 at 12:33 +0000, One Thousand Gnomes wrote:
quoted
The architecture of 8250_pci.c is good for PCIE device with 8250
compatible serial ports. We want to implement all functions of
F81504/508/512, but it'll make 8250_pci.c bloated and complex if we
implement GPIOLIB in 8250_pci.c
Your device is multi-function. Create an MFD driver for it. Make the
8250 driver bind to the MFD, and provide your own baud rate methods
within the standard 8250 layer
Ouch, somehow I missed this one!

Peter, Alan's suggestion is really worth to try.
Implement the GPIO lines in a GPIO driver that also binds to the MFD
and
lives in drivers/gpio

All the needed pieces already exist to implement it cleanly this way
without duplicating a ton of code.

Alan
-- 
Andy Shevchenko [off-list ref]
Intel Finland Oy
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help