Messages in this thread | | | Date | Tue, 14 Jul 2015 19:12:50 -0700 | From | Greg KH <> | Subject | Re: [PATCH 03/10] Staging: fbtft: Use a struct to describe each LCD controller |
| |
On Tue, Jun 30, 2015 at 08:43:10AM +0200, Fabio Falzoi wrote: > Use a struct flexfb_lcd_controller to holds chip properties, instead of > relying on a long 'if - else if' chain. > This allows to: > - use a simple linear search to verify if a certain LCD controller > model is supported or not. > - add support for a new LCD chip controller simply defining a new > flexfb_lcd_controller struct. > > Signed-off-by: Fabio Falzoi <fabio.falzoi84@gmail.com> > --- > drivers/staging/fbtft/fbtft.h | 20 ++++ > drivers/staging/fbtft/flexfb.c | 212 ++++++++++++++++++++++------------------- > 2 files changed, 136 insertions(+), 96 deletions(-)
I need the maintainers to sign off on this before I can take it, as it's not just a "simple" coding style fix.
thanks,
greg k-h
| |