linux-kernel - Re: [PATCH v4 0/6] FM V4L2 drivers for WL128x

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <AANLkTi=ZT0E=A9ZYAM86qu4P8eStkF2PLep6-DofCX-s@mail.gmail.com>
Date:	Wed, 17 Nov 2010 15:02:07 +0200
From:	Ohad Ben-Cohen <ohad@...ery.com>
To:	manjunatha_halli@...com
Cc:	mchehab@...radead.org, hverkuil@...all.nl,
	linux-kernel@...r.kernel.org, linux-media@...r.kernel.org
Subject: Re: [PATCH v4 0/6] FM V4L2 drivers for WL128x

Hi Manjunatha,

On Tue, Nov 16, 2010 at 3:18 PM,  <manjunatha_halli@...com> wrote:
>  drivers/staging/ti-st/Kconfig        |   10 +
>  drivers/staging/ti-st/Makefile       |    2 +
>  drivers/staging/ti-st/fmdrv.h        |  259 ++++
>  drivers/staging/ti-st/fmdrv_common.c | 2141 ++++++++++++++++++++++++++++++++++
>  drivers/staging/ti-st/fmdrv_common.h |  458 ++++++++
>  drivers/staging/ti-st/fmdrv_rx.c     |  979 ++++++++++++++++
>  drivers/staging/ti-st/fmdrv_rx.h     |   59 +
>  drivers/staging/ti-st/fmdrv_tx.c     |  461 ++++++++
>  drivers/staging/ti-st/fmdrv_tx.h     |   37 +
>  drivers/staging/ti-st/fmdrv_v4l2.c   |  757 ++++++++++++
>  drivers/staging/ti-st/fmdrv_v4l2.h   |   32 +
>  11 files changed, 5195 insertions(+), 0 deletions(-)

Usually when a driver is added to staging, it should also have a TODO
file specifying what needs to be done before the driver can be taken
out of staging (at least as far as the author knows of today).

It helps keeping track of the open issues in the driver, which is good
for everyone - the author, the random contributor/observer, and
probably even the staging maintainer.

Can you please add such a TODO file ?

Thanks,
Ohad.

>  create mode 100644 drivers/staging/ti-st/fmdrv.h
>  create mode 100644 drivers/staging/ti-st/fmdrv_common.c
>  create mode 100644 drivers/staging/ti-st/fmdrv_common.h
>  create mode 100644 drivers/staging/ti-st/fmdrv_rx.c
>  create mode 100644 drivers/staging/ti-st/fmdrv_rx.h
>  create mode 100644 drivers/staging/ti-st/fmdrv_tx.c
>  create mode 100644 drivers/staging/ti-st/fmdrv_tx.h
>  create mode 100644 drivers/staging/ti-st/fmdrv_v4l2.c
>  create mode 100644 drivers/staging/ti-st/fmdrv_v4l2.h
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/