| Message ID | 20190725105132.22545-1-noralf@tronnes.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | drm/tinydrm: Rename to drm/tiny | expand |
On Thu, Jul 25, 2019 at 12:51:29PM +0200, Noralf Trønnes wrote: > This is the final polish on tinydrm turning it into _the_ place for tiny > DRM drivers. > > Noralf. > > Noralf Trønnes (3): > drm/tinydrm/Kconfig: Remove menuconfig DRM_TINYDRM > drm/tinydrm: Rename folder to tiny > drm/gm12u320: Move driver to drm/tiny I wonder whether we could place a documentation hint somewhere to put tiny drm drivers (i.e. one file only, and that one pretty small) into drm/tiny. But I couldn't come up with a good place. Maybe re-add tiny.rst with a short overview paragraph that points at the most useful helpers for tiny drivers, plus how to integrate them? Kinda as a FAQ to the "I want to port an fbdev driver, halp pls?" question. Anyway that's just an idea aside, on the series: Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch> > > MAINTAINERS | 14 +++---- > drivers/gpu/drm/Kconfig | 4 +- > drivers/gpu/drm/Makefile | 3 +- > drivers/gpu/drm/gm12u320/Kconfig | 9 ---- > drivers/gpu/drm/gm12u320/Makefile | 2 - > drivers/gpu/drm/{tinydrm => tiny}/Kconfig | 41 +++++++++++++------ > drivers/gpu/drm/{tinydrm => tiny}/Makefile | 1 + > drivers/gpu/drm/{gm12u320 => tiny}/gm12u320.c | 0 > drivers/gpu/drm/{tinydrm => tiny}/hx8357d.c | 0 > drivers/gpu/drm/{tinydrm => tiny}/ili9225.c | 0 > drivers/gpu/drm/{tinydrm => tiny}/ili9341.c | 0 > drivers/gpu/drm/{tinydrm => tiny}/mi0283qt.c | 0 > drivers/gpu/drm/{tinydrm => tiny}/repaper.c | 0 > drivers/gpu/drm/{tinydrm => tiny}/st7586.c | 0 > drivers/gpu/drm/{tinydrm => tiny}/st7735r.c | 0 > 15 files changed, 38 insertions(+), 36 deletions(-) > delete mode 100644 drivers/gpu/drm/gm12u320/Kconfig > delete mode 100644 drivers/gpu/drm/gm12u320/Makefile > rename drivers/gpu/drm/{tinydrm => tiny}/Kconfig (73%) > rename drivers/gpu/drm/{tinydrm => tiny}/Makefile (89%) > rename drivers/gpu/drm/{gm12u320 => tiny}/gm12u320.c (100%) > rename drivers/gpu/drm/{tinydrm => tiny}/hx8357d.c (100%) > rename drivers/gpu/drm/{tinydrm => tiny}/ili9225.c (100%) > rename drivers/gpu/drm/{tinydrm => tiny}/ili9341.c (100%) > rename drivers/gpu/drm/{tinydrm => tiny}/mi0283qt.c (100%) > rename drivers/gpu/drm/{tinydrm => tiny}/repaper.c (100%) > rename drivers/gpu/drm/{tinydrm => tiny}/st7586.c (100%) > rename drivers/gpu/drm/{tinydrm => tiny}/st7735r.c (100%) > > -- > 2.20.1 >
Den 25.07.2019 14.06, skrev Daniel Vetter: > On Thu, Jul 25, 2019 at 12:51:29PM +0200, Noralf Trønnes wrote: >> This is the final polish on tinydrm turning it into _the_ place for tiny >> DRM drivers. >> >> Noralf. >> >> Noralf Trønnes (3): >> drm/tinydrm/Kconfig: Remove menuconfig DRM_TINYDRM >> drm/tinydrm: Rename folder to tiny >> drm/gm12u320: Move driver to drm/tiny > > I wonder whether we could place a documentation hint somewhere to put > tiny drm drivers (i.e. one file only, and that one pretty small) into > drm/tiny. But I couldn't come up with a good place. Maybe re-add tiny.rst > with a short overview paragraph that points at the most useful helpers for > tiny drivers, plus how to integrate them? Kinda as a FAQ to the "I want to > port an fbdev driver, halp pls?" question. > Sam, is this something you could do, you giving a talk about this and everything :-) A link to the driver example in drivers/gpu/drm/drm_drv.c: "DOC: driver instance overview" would make sense in such a doc. Noralf. > Anyway that's just an idea aside, on the series: > > Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch> > >> >> MAINTAINERS | 14 +++---- >> drivers/gpu/drm/Kconfig | 4 +- >> drivers/gpu/drm/Makefile | 3 +- >> drivers/gpu/drm/gm12u320/Kconfig | 9 ---- >> drivers/gpu/drm/gm12u320/Makefile | 2 - >> drivers/gpu/drm/{tinydrm => tiny}/Kconfig | 41 +++++++++++++------ >> drivers/gpu/drm/{tinydrm => tiny}/Makefile | 1 + >> drivers/gpu/drm/{gm12u320 => tiny}/gm12u320.c | 0 >> drivers/gpu/drm/{tinydrm => tiny}/hx8357d.c | 0 >> drivers/gpu/drm/{tinydrm => tiny}/ili9225.c | 0 >> drivers/gpu/drm/{tinydrm => tiny}/ili9341.c | 0 >> drivers/gpu/drm/{tinydrm => tiny}/mi0283qt.c | 0 >> drivers/gpu/drm/{tinydrm => tiny}/repaper.c | 0 >> drivers/gpu/drm/{tinydrm => tiny}/st7586.c | 0 >> drivers/gpu/drm/{tinydrm => tiny}/st7735r.c | 0 >> 15 files changed, 38 insertions(+), 36 deletions(-) >> delete mode 100644 drivers/gpu/drm/gm12u320/Kconfig >> delete mode 100644 drivers/gpu/drm/gm12u320/Makefile >> rename drivers/gpu/drm/{tinydrm => tiny}/Kconfig (73%) >> rename drivers/gpu/drm/{tinydrm => tiny}/Makefile (89%) >> rename drivers/gpu/drm/{gm12u320 => tiny}/gm12u320.c (100%) >> rename drivers/gpu/drm/{tinydrm => tiny}/hx8357d.c (100%) >> rename drivers/gpu/drm/{tinydrm => tiny}/ili9225.c (100%) >> rename drivers/gpu/drm/{tinydrm => tiny}/ili9341.c (100%) >> rename drivers/gpu/drm/{tinydrm => tiny}/mi0283qt.c (100%) >> rename drivers/gpu/drm/{tinydrm => tiny}/repaper.c (100%) >> rename drivers/gpu/drm/{tinydrm => tiny}/st7586.c (100%) >> rename drivers/gpu/drm/{tinydrm => tiny}/st7735r.c (100%) >> >> -- >> 2.20.1 >> >
Hi Noralf. On Thu, Jul 25, 2019 at 02:46:08PM +0200, Noralf Trønnes wrote: > > > Den 25.07.2019 14.06, skrev Daniel Vetter: > > On Thu, Jul 25, 2019 at 12:51:29PM +0200, Noralf Trønnes wrote: > >> This is the final polish on tinydrm turning it into _the_ place for tiny > >> DRM drivers. > >> > >> Noralf. > >> > >> Noralf Trønnes (3): > >> drm/tinydrm/Kconfig: Remove menuconfig DRM_TINYDRM > >> drm/tinydrm: Rename folder to tiny > >> drm/gm12u320: Move driver to drm/tiny I have thrown the patch-set after my usual build testing. Looks good - so you can consider the full series: Acked-by: Sam Ravnborg <sam@ravnborg.org> > > > > I wonder whether we could place a documentation hint somewhere to put > > tiny drm drivers (i.e. one file only, and that one pretty small) into > > drm/tiny. But I couldn't come up with a good place. Maybe re-add tiny.rst > > with a short overview paragraph that points at the most useful helpers for > > tiny drivers, plus how to integrate them? Kinda as a FAQ to the "I want to > > port an fbdev driver, halp pls?" question. > > > > Sam, is this something you could do, you giving a talk about this and > everything :-) Talk proposal is accepted, but I need to get a few details in order here. (work, family stuff). And then I also hope to actaully create a first tiny driver - awaiting display to arrive by mail. But if that all succeeds then I will be happy to try to come up with something that would have helped me. Sam
Den 25.07.2019 12.51, skrev Noralf Trønnes: > This is the final polish on tinydrm turning it into _the_ place for tiny > DRM drivers. > > Noralf. > > Noralf Trønnes (3): > drm/tinydrm/Kconfig: Remove menuconfig DRM_TINYDRM > drm/tinydrm: Rename folder to tiny > drm/gm12u320: Move driver to drm/tiny > Series applied to drm-misc-next, thanks for reviewing. Noralf.
This is the final polish on tinydrm turning it into _the_ place for tiny DRM drivers. Noralf. Noralf Trønnes (3): drm/tinydrm/Kconfig: Remove menuconfig DRM_TINYDRM drm/tinydrm: Rename folder to tiny drm/gm12u320: Move driver to drm/tiny MAINTAINERS | 14 +++---- drivers/gpu/drm/Kconfig | 4 +- drivers/gpu/drm/Makefile | 3 +- drivers/gpu/drm/gm12u320/Kconfig | 9 ---- drivers/gpu/drm/gm12u320/Makefile | 2 - drivers/gpu/drm/{tinydrm => tiny}/Kconfig | 41 +++++++++++++------ drivers/gpu/drm/{tinydrm => tiny}/Makefile | 1 + drivers/gpu/drm/{gm12u320 => tiny}/gm12u320.c | 0 drivers/gpu/drm/{tinydrm => tiny}/hx8357d.c | 0 drivers/gpu/drm/{tinydrm => tiny}/ili9225.c | 0 drivers/gpu/drm/{tinydrm => tiny}/ili9341.c | 0 drivers/gpu/drm/{tinydrm => tiny}/mi0283qt.c | 0 drivers/gpu/drm/{tinydrm => tiny}/repaper.c | 0 drivers/gpu/drm/{tinydrm => tiny}/st7586.c | 0 drivers/gpu/drm/{tinydrm => tiny}/st7735r.c | 0 15 files changed, 38 insertions(+), 36 deletions(-) delete mode 100644 drivers/gpu/drm/gm12u320/Kconfig delete mode 100644 drivers/gpu/drm/gm12u320/Makefile rename drivers/gpu/drm/{tinydrm => tiny}/Kconfig (73%) rename drivers/gpu/drm/{tinydrm => tiny}/Makefile (89%) rename drivers/gpu/drm/{gm12u320 => tiny}/gm12u320.c (100%) rename drivers/gpu/drm/{tinydrm => tiny}/hx8357d.c (100%) rename drivers/gpu/drm/{tinydrm => tiny}/ili9225.c (100%) rename drivers/gpu/drm/{tinydrm => tiny}/ili9341.c (100%) rename drivers/gpu/drm/{tinydrm => tiny}/mi0283qt.c (100%) rename drivers/gpu/drm/{tinydrm => tiny}/repaper.c (100%) rename drivers/gpu/drm/{tinydrm => tiny}/st7586.c (100%) rename drivers/gpu/drm/{tinydrm => tiny}/st7735r.c (100%)