video: add support for the AUO-K190X epd controllers

Started by Heiko Stübner on Mar 28, 2012 · 4 messages from 2 authors · Last reply Apr 26, 2012

STALE5314d SUPERSEDED

Status: Superseded by v2 (posted 2012-04-28). [go to v2]

Revision v1 (2 revisions archived).

Revisions (2)
  1. v1 current
  2. v2 [diff against previous]

open the first message on its own page

Message details and related content
This series adds basic support for the AUO-K190X family of epaper
controllers which consists at the moment of the K1900 and K1901.

These controllers are used mostly in the OEM line of Qisda ebook readers
sold under different brands worldwide. Among those are the Medion/Thalia
Oyo 1 + 2, the bq Avant and Avant 3, the Asus eeeReader DR-900, BenQ K60
and dozens more.

The _dpy_update, _dpy_update_pages and _need_refresh functions are
intentional duplicated. Currently they do the same, but as the
controllers emit a greatly different runtime behaviour there is a lot
of room for future controller-specific performance improvements.

The series also includes the patch which adds a first_io callback to
defio. This creates a nice way to parallelise the wakup when it is known
that the controller is needed at the end of the defio cycle.

Heiko Stuebner (4):
  fb_defio: add first_io callback
  video: auo_k190x: add code shared by controller drivers
  video: auo_k190x: add driver for AUO-K1900 variant
  video: auo_k190x: add driver for the AUO-K1901 variant

 drivers/video/Kconfig       |   28 ++
 drivers/video/Makefile      |    3 +
 drivers/video/auo_k1900fb.c |  198 ++++++++
 drivers/video/auo_k1901fb.c |  251 +++++++++++
 drivers/video/auo_k190x.c   | 1038 +++++++++++++++++++++++++++++++++++++++++++
 drivers/video/auo_k190x.h   |  129 ++++++
 drivers/video/fb_defio.c    |    4 +
 include/linux/fb.h          |    1 +
 include/video/auo_k190xfb.h |  106 +++++
 9 files changed, 1758 insertions(+), 0 deletions(-)
 create mode 100644 drivers/video/auo_k1900fb.c
 create mode 100644 drivers/video/auo_k1901fb.c
 create mode 100644 drivers/video/auo_k190x.c
 create mode 100644 drivers/video/auo_k190x.h
 create mode 100644 include/video/auo_k190xfb.h

-- 

2 more lines

Message details and related content
Hi Florian,

Am Mittwoch, 28. März 2012, 19:32:54 schrieb Heiko Stübner:
This series adds basic support for the AUO-K190X family of epaper
controllers which consists at the moment of the K1900 and K1901.

These controllers are used mostly in the OEM line of Qisda ebook readers
sold under different brands worldwide. Among those are the Medion/Thalia
Oyo 1 + 2, the bq Avant and Avant 3, the Asus eeeReader DR-900, BenQ K60
and dozens more.

The _dpy_update, _dpy_update_pages and _need_refresh functions are
intentional duplicated. Currently they do the same, but as the
controllers emit a greatly different runtime behaviour there is a lot
of room for future controller-specific performance improvements.

The series also includes the patch which adds a first_io callback to
defio. This creates a nice way to parallelise the wakup when it is known
that the controller is needed at the end of the defio cycle.
ping on this series?

Is anything more needed?


Thanks
Heiko
Heiko Stuebner (4):
  fb_defio: add first_io callback
  video: auo_k190x: add code shared by controller drivers
  video: auo_k190x: add driver for AUO-K1900 variant
  video: auo_k190x: add driver for the AUO-K1901 variant

 drivers/video/Kconfig       |   28 ++
 drivers/video/Makefile      |    3 +
 drivers/video/auo_k1900fb.c |  198 ++++++++
 drivers/video/auo_k1901fb.c |  251 +++++++++++
 drivers/video/auo_k190x.c   | 1038
+++++++++++++++++++++++++++++++++++++++++++ drivers/video/auo_k190x.h   | 
129 ++++++
 drivers/video/fb_defio.c    |    4 +
 include/linux/fb.h          |    1 +
 include/video/auo_k190xfb.h |  106 +++++
 9 files changed, 1758 insertions(+), 0 deletions(-)
 create mode 100644 drivers/video/auo_k1900fb.c
 create mode 100644 drivers/video/auo_k1901fb.c
 create mode 100644 drivers/video/auo_k190x.c
 create mode 100644 drivers/video/auo_k190x.h
 create mode 100644 include/video/auo_k190xfb.h
Message details and related content
Hi Heiko,

On 04/24/2012 01:44 PM, Heiko Stübner wrote:
Hi Florian,

Am Mittwoch, 28. März 2012, 19:32:54 schrieb Heiko Stübner:
quoted
This series adds basic support for the AUO-K190X family of epaper
controllers which consists at the moment of the K1900 and K1901.

These controllers are used mostly in the OEM line of Qisda ebook readers
sold under different brands worldwide. Among those are the Medion/Thalia
Oyo 1 + 2, the bq Avant and Avant 3, the Asus eeeReader DR-900, BenQ K60
and dozens more.

The _dpy_update, _dpy_update_pages and _need_refresh functions are
intentional duplicated. Currently they do the same, but as the
controllers emit a greatly different runtime behaviour there is a lot
of room for future controller-specific performance improvements.

The series also includes the patch which adds a first_io callback to
defio. This creates a nice way to parallelise the wakup when it is known
that the controller is needed at the end of the defio cycle.
ping on this series?

Is anything more needed?
I wanted to do do a patch to show you what I want but didn't find the
time to do so, so I'll try to explain it:

If I remember correctly the problem with the patches is the Kconfig menu
structure as "make menuconfig" shows it: Your driver ends up in the
graphics system and not in the framebuffer subsystem. The problem is the
invisible entry you add, it has no dependency on the FB subsystem and
therefore terminates the framebuffer menuconfig. The easy fix would be
to make it depend on fb but I'd consider it nicer to change the hidden
entry into a menunconfig one and let the individual drivers depend on it
rather than select it.


Best regards,

Florian Tobias Schandinat

Thanks
Heiko
quoted
Heiko Stuebner (4):
  fb_defio: add first_io callback
  video: auo_k190x: add code shared by controller drivers
  video: auo_k190x: add driver for AUO-K1900 variant
  video: auo_k190x: add driver for the AUO-K1901 variant

 drivers/video/Kconfig       |   28 ++
 drivers/video/Makefile      |    3 +
 drivers/video/auo_k1900fb.c |  198 ++++++++
 drivers/video/auo_k1901fb.c |  251 +++++++++++
 drivers/video/auo_k190x.c   | 1038
+++++++++++++++++++++++++++++++++++++++++++ drivers/video/auo_k190x.h   | 
129 ++++++
 drivers/video/fb_defio.c    |    4 +
 include/linux/fb.h          |    1 +
 include/video/auo_k190xfb.h |  106 +++++
 9 files changed, 1758 insertions(+), 0 deletions(-)
 create mode 100644 drivers/video/auo_k1900fb.c
 create mode 100644 drivers/video/auo_k1901fb.c
 create mode 100644 drivers/video/auo_k190x.c
 create mode 100644 drivers/video/auo_k190x.h
 create mode 100644 include/video/auo_k190xfb.h
Message details and related content
Hi Florian,

Am Donnerstag, 26. April 2012, 08:44:11 schrieb Florian Tobias Schandinat:
Hi Heiko,

On 04/24/2012 01:44 PM, Heiko Stübner wrote:
quoted
Hi Florian,

Am Mittwoch, 28. März 2012, 19:32:54 schrieb Heiko Stübner:
quoted
This series adds basic support for the AUO-K190X family of epaper
controllers which consists at the moment of the K1900 and K1901.

These controllers are used mostly in the OEM line of Qisda ebook readers
sold under different brands worldwide. Among those are the Medion/Thalia
Oyo 1 + 2, the bq Avant and Avant 3, the Asus eeeReader DR-900, BenQ K60
and dozens more.

The _dpy_update, _dpy_update_pages and _need_refresh functions are
intentional duplicated. Currently they do the same, but as the
controllers emit a greatly different runtime behaviour there is a lot
of room for future controller-specific performance improvements.

The series also includes the patch which adds a first_io callback to
defio. This creates a nice way to parallelise the wakup when it is known
that the controller is needed at the end of the defio cycle.
ping on this series?

Is anything more needed?
I wanted to do do a patch to show you what I want but didn't find the
time to do so, so I'll try to explain it:

If I remember correctly the problem with the patches is the Kconfig menu
structure as "make menuconfig" shows it: Your driver ends up in the
graphics system and not in the framebuffer subsystem. The problem is the
invisible entry you add, it has no dependency on the FB subsystem and
therefore terminates the framebuffer menuconfig. The easy fix would be
to make it depend on fb but I'd consider it nicer to change the hidden
entry into a menunconfig one and let the individual drivers depend on it
rather than select it.
ok, will change it to work like this.

I was unsure on what was the correct way to do it. At first I had it 
implemented like you described but as the base driver does not provide any 
usable functionality in itself changed it to the invisible config node.

Personally I have no hard preference, and will resubmit the series with the 
change.


Thanks
Heiko


quoted
quoted
Heiko Stuebner (4):
  fb_defio: add first_io callback
  video: auo_k190x: add code shared by controller drivers
  video: auo_k190x: add driver for AUO-K1900 variant
  video: auo_k190x: add driver for the AUO-K1901 variant
 
 drivers/video/Kconfig       |   28 ++
 drivers/video/Makefile      |    3 +
 drivers/video/auo_k1900fb.c |  198 ++++++++
 drivers/video/auo_k1901fb.c |  251 +++++++++++
 drivers/video/auo_k190x.c   | 1038

+++++++++++++++++++++++++++++++++++++++++++ drivers/video/auo_k190x.h  
| 129 ++++++

 drivers/video/fb_defio.c    |    4 +
 include/linux/fb.h          |    1 +
 include/video/auo_k190xfb.h |  106 +++++
 9 files changed, 1758 insertions(+), 0 deletions(-)
 create mode 100644 drivers/video/auo_k1900fb.c
 create mode 100644 drivers/video/auo_k1901fb.c
 create mode 100644 drivers/video/auo_k190x.c
 create mode 100644 drivers/video/auo_k190x.h
 create mode 100644 include/video/auo_k190xfb.h
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help