From: Randy Dunlap <randy.dunlap@oracle•com>
To: Stephen Rothwell <sfr@canb•auug.org.au>,
gregkh@suse•de, Al Cho <acho@novell•com>
Cc: linux-next@vger•kernel.org, LKML <linux-kernel@vger•kernel.org>,
driverdevel <devel@driverdev•osuosl.org>
Subject: [PATCH -next] staging/keucr: update TODO list and ask question about it
Date: Tue, 14 Sep 2010 15:41:48 -0700 [thread overview]
Message-ID: <20100914154148.826a0334.randy.dunlap@oracle.com> (raw)
In-Reply-To: <20100913134302.a77e9a3e.sfr@canb.auug.org.au>
From: Randy Dunlap <randy.dunlap@oracle•com>
Update keucr TODO with additional work items.
Signed-off-by: Randy Dunlap <randy.dunlap@oracle•com>
Cc: Al Cho <acho@novell•com>
---
drivers/staging/keucr/TODO | 3 +++
1 file changed, 3 insertions(+)
and I don't understand this TODO comment:
" - determine if the driver should not be using a duplicate
version of the usb-storage scsi interface code, but should
be merged into the drivers/usb/storage/ directory and
infrastructure instead."
Of course the driver should not use a duplicate of the usb-storage
SCSI interface code. Did someone seriously think that it should?
--- linux-next-20100913.orig/drivers/staging/keucr/TODO
+++ linux-next-20100913/drivers/staging/keucr/TODO
@@ -6,6 +6,9 @@ TODO:
be merged into the drivers/usb/storage/ directory and
infrastructure instead.
- review by the USB developer community
+ - common.h: use kernel swap, le, & be functions
+ - smcommon.h & smilsub.c: use kernel hweight8(), hweight16(),
+ strcmp(), & strcpy()
Please send any patches for this driver to Al Cho <acho@novell•com> and
Greg Kroah-Hartman <gregkh@suse•de>.
next prev parent reply other threads:[~2010-09-14 22:41 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-09-13 3:43 linux-next: Tree for September 13 Stephen Rothwell
2010-09-14 22:39 ` [PATCH -next] staging/bcm: fix build for CONFIG_PM not enabled Randy Dunlap
2010-09-14 22:39 ` [PATCH -next] staging/bcm: fix printk format warnings Randy Dunlap
2010-09-14 22:41 ` [PATCH -next] staging/keucr: fix build when CONFIG_PM is not enabled Randy Dunlap
2010-09-14 22:41 ` Randy Dunlap [this message]
2010-09-16 19:28 ` [PATCH -next] staging/keucr: update TODO list and ask question about it Greg KH
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20100914154148.826a0334.randy.dunlap@oracle.com \
--to=randy.dunlap@oracle$(echo .)com \
--cc=acho@novell$(echo .)com \
--cc=devel@driverdev$(echo .)osuosl.org \
--cc=gregkh@suse$(echo .)de \
--cc=linux-kernel@vger$(echo .)kernel.org \
--cc=linux-next@vger$(echo .)kernel.org \
--cc=sfr@canb$(echo .)auug.org.au \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox