All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 00/10] Generic USB Display driver
@ 2020-05-09 14:16 ` Noralf Trønnes
  0 siblings, 0 replies; 30+ messages in thread
From: Noralf Trønnes @ 2020-05-09 14:16 UTC (permalink / raw)
  To: dri-devel, linux-usb, lee.jones
  Cc: sam, daniel.thompson, jingoohan1, Noralf Trønnes

Hi,

A while back I had the idea to turn a Raspberry Pi Zero into a $5
USB to HDMI/SDTV/DSI/DPI display adapter.

This series adds a USB host driver and a device/gadget driver to achieve
that.

The reason for calling it 'Generic' is so anyone can make a USB
display/adapter against this driver, all that's needed is to add a USB
vid:pid. I was hoping to have someone working on a microcontroller based
USB display by now, but unfortunately that has been delayed. It would
have been nice to have a microcontroller implementation to ensure that I
haven't made things unnecessary difficult to implement.

Performance
The one thing that decides how useful this all is, is how smooth an
experience it gives. My hope was that it should not be noticeably laggy
with ordinary office use on 1920x1080@RG16. I'm pleased to see that it's
also possible to watch youtube movies, although not in fullscreen.

Some of the main factors that affects performance:
- Display resolution
- Userspace providing damage reports (FB_DAMAGE_CLIPS or
  DRM_IOCTL_MODE_DIRTYFB)
- Color depth (DRM_CAP_DUMB_PREFERRED_DEPTH = 16 if RGB565)
- How well the frames compress (lz4)
- USB2 vs. USB3
- Gadget device memory bandwidth, CPU power for decompression
- (Big endian hosts will have to do byte swapping on the frames)

I've tested these:
- xorg-server on Pi4. This was nice and smooth since it uses
  DRM_IOCTL_MODE_DIRTYFB and honours DRM_CAP_DUMB_PREFERRED_DEPTH.
- Ubuntu 20.04 GNOME on x86. This was useable, but not so good for
  movies. GNOME doesn't look at DRM_CAP_DUMB_PREFERRED_DEPTH and doesn't
  set FB_DAMAGE_CLIPS on the pageflips.

I've made a short video to show what it looks like[1].

Tearing

Host side
Compression and transfer happens in an async worker.
There are 2 tearing cases here:
1. The framebuffer fits in the kmalloc'ed transfer buffer, max 4MB by
   default (KMALLOC_MAX_SIZE). This can give occasional tearing when
   userspace gets ahead of the worker.
2. The framebuffer doesn't fit in the transfer buffer, so the host
   splits the transfer. When showing a movie this will almost always give
   tearing between each part of the frame.
   Fix: The device can increase KMALLOC_MAX_SIZE by setting
   CONFIG_FORCE_MAX_ZONEORDER since it probably has a custom kernel
   anyways. The host distro can't/won't do that, so I have look for
   solution here.
The host side tearing is worse than the device side, because it lasts
longer.

Device side
Received updates are decompressed/copied into the framebuffer that's
being scanned out, so tearing is possible. Double buffering/page
flipping could be used for full frames, but well behaving userspace
doesn't send many of those, so very little impact. Double buffering on
all updates would require memcpy between buffers locally, hampering
performance. Maybe a Pi4 could get away with it, but a Pi Zero would
certainly not (memory bandwidth).

I've tested this series with usbip[2] by connecting 2 Pi4's over cabled
gigabit network. It worked fine.

One use case for these drivers is reusing old tablets and cell phones as
USB displays.

The Pi4 has two hdmi ports and I was asked if the driver supports that,
and I've concluded that it would be too much work to implement at this
point. It is possible to extend the protocol and implement it later.

I have used a Pi4 as the gadget device during development since it has
much better memory bandwith (4000 vs 200 MBps)[3] and CPU than the Pi
Zero. They both have the same gadget controller (dwc2).

Changes in this version:
- Drop drm_backlight_helper
- Use devm_drm_dev_alloc()
- Addressed Sam's comments


Noralf.

[1] https://youtu.be/AhGZWwUm8JU
[2] tools/usb/usbip/README
[3] https://magpi.raspberrypi.org/articles/raspberry-pi-specs-benchmarks


Noralf Trønnes (10):
  backlight: Add backlight_device_get_by_name()
  drm/client: Add drm_client_init_from_id()
  drm/client: Add drm_client_framebuffer_flush()
  drm/client: Add drm_client_modeset_check()
  drm/client: Add drm_client_modeset_disable()
  drm/client: Add a way to set modeset, properties and rotation
  drm/format-helper: Add drm_fb_swab()
  drm: Add Generic USB Display driver
  drm/gud: Add functionality for the USB gadget side
  usb: gadget: function: Add Generic USB Display support

 .../ABI/testing/configfs-usb-gadget-gud_drm   |   10 +
 MAINTAINERS                                   |   10 +
 drivers/gpu/drm/Kconfig                       |    2 +
 drivers/gpu/drm/Makefile                      |    1 +
 drivers/gpu/drm/drm_client.c                  |   81 +-
 drivers/gpu/drm/drm_client_modeset.c          |  192 ++-
 drivers/gpu/drm/drm_format_helper.c           |   61 +-
 drivers/gpu/drm/drm_mipi_dbi.c                |    2 +-
 drivers/gpu/drm/gud/Kconfig                   |   20 +
 drivers/gpu/drm/gud/Makefile                  |    5 +
 drivers/gpu/drm/gud/gud_drm_connector.c       |  724 ++++++++++
 drivers/gpu/drm/gud/gud_drm_drv.c             |  641 +++++++++
 drivers/gpu/drm/gud/gud_drm_gadget.c          | 1165 +++++++++++++++++
 drivers/gpu/drm/gud/gud_drm_internal.h        |   66 +
 drivers/gpu/drm/gud/gud_drm_pipe.c            |  423 ++++++
 drivers/usb/gadget/Kconfig                    |   12 +
 drivers/usb/gadget/function/Makefile          |    2 +
 drivers/usb/gadget/function/f_gud_drm.c       |  678 ++++++++++
 drivers/video/backlight/backlight.c           |   21 +
 include/drm/drm_client.h                      |   45 +-
 include/drm/drm_format_helper.h               |    4 +-
 include/drm/gud_drm.h                         |  377 ++++++
 include/linux/backlight.h                     |    1 +
 23 files changed, 4513 insertions(+), 30 deletions(-)
 create mode 100644 Documentation/ABI/testing/configfs-usb-gadget-gud_drm
 create mode 100644 drivers/gpu/drm/gud/Kconfig
 create mode 100644 drivers/gpu/drm/gud/Makefile
 create mode 100644 drivers/gpu/drm/gud/gud_drm_connector.c
 create mode 100644 drivers/gpu/drm/gud/gud_drm_drv.c
 create mode 100644 drivers/gpu/drm/gud/gud_drm_gadget.c
 create mode 100644 drivers/gpu/drm/gud/gud_drm_internal.h
 create mode 100644 drivers/gpu/drm/gud/gud_drm_pipe.c
 create mode 100644 drivers/usb/gadget/function/f_gud_drm.c
 create mode 100644 include/drm/gud_drm.h

-- 
2.23.0


^ permalink raw reply	[flat|nested] 30+ messages in thread

end of thread, other threads:[~2020-06-03 12:19 UTC | newest]

Thread overview: 30+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-05-09 14:16 [PATCH v2 00/10] Generic USB Display driver Noralf Trønnes
2020-05-09 14:16 ` Noralf Trønnes
2020-05-09 14:16 ` [PATCH v2 01/10] backlight: Add backlight_device_get_by_name() Noralf Trønnes
2020-05-09 14:16   ` Noralf Trønnes
2020-05-10 21:44   ` Jingoo Han
2020-05-10 21:44     ` Jingoo Han
2020-05-11  6:40   ` Lee Jones
2020-05-11  6:40     ` Lee Jones
2020-05-09 14:16 ` [PATCH v2 02/10] drm/client: Add drm_client_init_from_id() Noralf Trønnes
2020-05-09 14:16   ` Noralf Trønnes
2020-05-09 14:16 ` [PATCH v2 03/10] drm/client: Add drm_client_framebuffer_flush() Noralf Trønnes
2020-05-09 14:16   ` Noralf Trønnes
2020-05-28 13:48   ` Emil Velikov
2020-05-28 13:48     ` Emil Velikov
2020-06-03 12:19     ` Noralf Trønnes
2020-06-03 12:19       ` Noralf Trønnes
2020-05-09 14:16 ` [PATCH v2 04/10] drm/client: Add drm_client_modeset_check() Noralf Trønnes
2020-05-09 14:16   ` Noralf Trønnes
2020-05-09 14:16 ` [PATCH v2 05/10] drm/client: Add drm_client_modeset_disable() Noralf Trønnes
2020-05-09 14:16   ` Noralf Trønnes
2020-05-09 14:16 ` [PATCH v2 06/10] drm/client: Add a way to set modeset, properties and rotation Noralf Trønnes
2020-05-09 14:16   ` Noralf Trønnes
2020-05-09 14:16 ` [PATCH v2 07/10] drm/format-helper: Add drm_fb_swab() Noralf Trønnes
2020-05-09 14:16   ` Noralf Trønnes
2020-05-09 14:16 ` [PATCH v2 08/10] drm: Add Generic USB Display driver Noralf Trønnes
2020-05-09 14:16   ` Noralf Trønnes
2020-05-09 14:16 ` [PATCH v2 09/10] drm/gud: Add functionality for the USB gadget side Noralf Trønnes
2020-05-09 14:16   ` Noralf Trønnes
2020-05-09 14:16 ` [PATCH v2 10/10] usb: gadget: function: Add Generic USB Display support Noralf Trønnes
2020-05-09 14:16   ` Noralf Trønnes

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.