mbox series

[v3,0/2] BugLink: https://bugs.launchpad.net/bugs/1727235

Message ID 20180906081920.32298-1-acelan.kao@canonical.com
Headers show
Series BugLink: https://bugs.launchpad.net/bugs/1727235 | expand

Message

AceLan Kao Sept. 6, 2018, 8:19 a.m. UTC 
[Impact]
New Dell AIO platforms requires a driver that can communicate with the
scalar IC FW through UART interface to get and set the brightness level.

[Test]
Verified on some new Dell AIO machines and confirmed it works.

[Fix]
Write a new driver to handle this task.

[Regression Potential]
Low, it's a new driver for new interface.

[Misc]
Upstream maintainer ask use to re-implement this driver by the new
interface, serdev, but serdev seems doesn't work with ACPI, so submit
this driver as Ubuntu sauce patch for now. I'll try re-write it when I'm
available.

AceLan Kao (2):
  UBUNTU: SAUCE: platform/x86: dell-uart-backlight: new backlight driver
    for DELL AIO
  UBUNTU: updateconfigs for Dell UART backlight driver

 debian.master/config/config.common.ubuntu  |   1 +
 drivers/platform/x86/Kconfig               |  14 +
 drivers/platform/x86/Makefile              |   1 +
 drivers/platform/x86/dell-uart-backlight.c | 463 +++++++++++++++++++++
 drivers/platform/x86/dell-uart-backlight.h |  36 ++
 5 files changed, 515 insertions(+)
 create mode 100644 drivers/platform/x86/dell-uart-backlight.c
 create mode 100644 drivers/platform/x86/dell-uart-backlight.h

Comments

Kleber Sacilotto de Souza Sept. 27, 2018, 3:55 p.m. UTC | #1 
On 09/06/18 10:19, AceLan Kao wrote:
> [Impact]
> New Dell AIO platforms requires a driver that can communicate with the
> scalar IC FW through UART interface to get and set the brightness level.
> 
> [Test]
> Verified on some new Dell AIO machines and confirmed it works.
> 
> [Fix]
> Write a new driver to handle this task.
> 
> [Regression Potential]
> Low, it's a new driver for new interface.
> 
> [Misc]
> Upstream maintainer ask use to re-implement this driver by the new
> interface, serdev, but serdev seems doesn't work with ACPI, so submit
> this driver as Ubuntu sauce patch for now. I'll try re-write it when I'm
> available.
> 
> AceLan Kao (2):
>   UBUNTU: SAUCE: platform/x86: dell-uart-backlight: new backlight driver
>     for DELL AIO
>   UBUNTU: updateconfigs for Dell UART backlight driver
> 
>  debian.master/config/config.common.ubuntu  |   1 +
>  drivers/platform/x86/Kconfig               |  14 +
>  drivers/platform/x86/Makefile              |   1 +
>  drivers/platform/x86/dell-uart-backlight.c | 463 +++++++++++++++++++++
>  drivers/platform/x86/dell-uart-backlight.h |  36 ++
>  5 files changed, 515 insertions(+)
>  create mode 100644 drivers/platform/x86/dell-uart-backlight.c
>  create mode 100644 drivers/platform/x86/dell-uart-backlight.h
> 

A v4 of this patchset has been sent.

Thanks,
Kleber