Re: [PATCH v2 20/22] media: docs: move driver-specific info to driver-api

From: Hans Verkuil
Date: Thu Mar 12 2020 - 03:26:10 EST


On 3/10/20 2:43 PM, Mauro Carvalho Chehab wrote:
> Those documents don't really describe the driver API.
>
> Instead, they contain development-specific information.
>
> Yet, as the main index file describes the content of it as:
>
> "how specific kernel subsystems work
> from the point of view of a kernel developer"
>
> It seems to be the better fit.
>
> Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@xxxxxxxxxx>
> ---
> .../media/drivers}/bttv-devel.rst | 0
> .../media/drivers}/contributors.rst | 0
> .../media/drivers}/cpia2_devel.rst | 0
> .../media/drivers}/cx2341x-devel.rst | 0
> .../media/drivers}/cx88-devel.rst | 0
> .../media/drivers}/davinci-vpbe-devel.rst | 0
> .../media/drivers}/dvb-usb.rst | 0
> .../media/drivers}/fimc-devel.rst | 0
> .../media/drivers}/frontends.rst | 0
> .../media/drivers}/index.rst | 29 ++++++++++++++++++-
> .../media/drivers}/pvrusb2.rst | 0
> .../media/drivers}/pxa_camera.rst | 0
> .../media/drivers}/radiotrack.rst | 0
> .../media/drivers}/saa7134-devel.rst | 0
> .../media/drivers}/sh_mobile_ceu_camera.rst | 0
> .../media/drivers}/tuners.rst | 0
> .../media/drivers}/vimc-devel.rst | 0
> Documentation/driver-api/media/index.rst | 2 ++
> Documentation/media/index.rst | 1 -
> Documentation/media/v4l-drivers/index.rst | 14 ---------
> MAINTAINERS | 8 ++---
> drivers/media/dvb-frontends/dib3000.h | 2 +-
> drivers/media/dvb-frontends/dib3000mb.c | 2 +-
> drivers/media/dvb-frontends/eds1547.h | 2 +-
> drivers/media/dvb-frontends/z0194a.h | 2 +-
> drivers/media/pci/cx18/cx18-streams.c | 4 +--
> drivers/media/platform/pxa_camera.c | 4 +--
> drivers/media/radio/Kconfig | 2 +-
> drivers/media/usb/dvb-usb-v2/Kconfig | 2 +-
> drivers/media/usb/dvb-usb-v2/gl861.c | 2 +-
> drivers/media/usb/dvb-usb-v2/lmedm04.c | 2 +-
> drivers/media/usb/dvb-usb-v2/lmedm04.h | 2 +-
> drivers/media/usb/dvb-usb-v2/mxl111sf.c | 2 +-
> drivers/media/usb/dvb-usb-v2/mxl111sf.h | 2 +-
> drivers/media/usb/dvb-usb/Kconfig | 2 +-
> drivers/media/usb/dvb-usb/a800.c | 2 +-
> drivers/media/usb/dvb-usb/af9005-fe.c | 2 +-
> drivers/media/usb/dvb-usb/af9005-remote.c | 2 +-
> drivers/media/usb/dvb-usb/af9005.c | 2 +-
> drivers/media/usb/dvb-usb/af9005.h | 2 +-
> drivers/media/usb/dvb-usb/az6027.c | 2 +-
> drivers/media/usb/dvb-usb/cxusb.c | 2 +-
> drivers/media/usb/dvb-usb/dibusb-common.c | 2 +-
> drivers/media/usb/dvb-usb/dibusb-mb.c | 2 +-
> drivers/media/usb/dvb-usb/dibusb-mc-common.c | 2 +-
> drivers/media/usb/dvb-usb/dibusb-mc.c | 2 +-
> drivers/media/usb/dvb-usb/dibusb.h | 2 +-
> drivers/media/usb/dvb-usb/digitv.c | 2 +-
> drivers/media/usb/dvb-usb/dtt200u-fe.c | 2 +-
> drivers/media/usb/dvb-usb/dtt200u.c | 2 +-
> drivers/media/usb/dvb-usb/dtt200u.h | 2 +-
> drivers/media/usb/dvb-usb/dvb-usb-init.c | 2 +-
> drivers/media/usb/dvb-usb/dw2102.c | 2 +-
> drivers/media/usb/dvb-usb/gp8psk.c | 2 +-
> drivers/media/usb/dvb-usb/gp8psk.h | 2 +-
> drivers/media/usb/dvb-usb/m920x.c | 2 +-
> drivers/media/usb/dvb-usb/nova-t-usb2.c | 2 +-
> drivers/media/usb/dvb-usb/opera1.c | 2 +-
> drivers/media/usb/dvb-usb/ttusb2.c | 2 +-
> drivers/media/usb/dvb-usb/ttusb2.h | 2 +-
> drivers/media/usb/dvb-usb/umt-010.c | 2 +-
> drivers/media/usb/dvb-usb/vp702x-fe.c | 2 +-
> drivers/media/usb/dvb-usb/vp702x.c | 2 +-
> drivers/media/usb/dvb-usb/vp7045-fe.c | 2 +-
> drivers/media/usb/dvb-usb/vp7045.c | 2 +-
> drivers/media/usb/dvb-usb/vp7045.h | 2 +-
> 66 files changed, 81 insertions(+), 67 deletions(-)
> rename Documentation/{media/v4l-drivers => driver-api/media/drivers}/bttv-devel.rst (100%)
> rename Documentation/{media/dvb-drivers => driver-api/media/drivers}/contributors.rst (100%)
> rename Documentation/{media/v4l-drivers => driver-api/media/drivers}/cpia2_devel.rst (100%)
> rename Documentation/{media/v4l-drivers => driver-api/media/drivers}/cx2341x-devel.rst (100%)
> rename Documentation/{media/v4l-drivers => driver-api/media/drivers}/cx88-devel.rst (100%)
> rename Documentation/{media/v4l-drivers => driver-api/media/drivers}/davinci-vpbe-devel.rst (100%)
> rename Documentation/{media/dvb-drivers => driver-api/media/drivers}/dvb-usb.rst (100%)
> rename Documentation/{media/v4l-drivers => driver-api/media/drivers}/fimc-devel.rst (100%)
> rename Documentation/{media/dvb-drivers => driver-api/media/drivers}/frontends.rst (100%)
> rename Documentation/{media/dvb-drivers => driver-api/media/drivers}/index.rst (67%)
> rename Documentation/{media/v4l-drivers => driver-api/media/drivers}/pvrusb2.rst (100%)
> rename Documentation/{media/v4l-drivers => driver-api/media/drivers}/pxa_camera.rst (100%)
> rename Documentation/{media/v4l-drivers => driver-api/media/drivers}/radiotrack.rst (100%)
> rename Documentation/{media/v4l-drivers => driver-api/media/drivers}/saa7134-devel.rst (100%)
> rename Documentation/{media/v4l-drivers => driver-api/media/drivers}/sh_mobile_ceu_camera.rst (100%)
> rename Documentation/{media/v4l-drivers => driver-api/media/drivers}/tuners.rst (100%)
> rename Documentation/{media/v4l-drivers => driver-api/media/drivers}/vimc-devel.rst (100%)

Same question here: can we keep the v4l/dvb split?

Regards,

Hans