The patch titled
sparse-pointer-use-of-zero-as-null-checkpatch-fixes
has been removed from the -mm tree. Its filename was
sparse-pointer-use-of-zero-as-null-checkpatch-fixes.patch
This patch was dropped because it was folded into
sparse-pointer-use-of-zero-as-null.patch
------------------------------------------------------
Subject: sparse-pointer-use-of-zero-as-null-checkpatch-fixes
From: Andrew Morton <[EMAIL PROTECTED]>
ERROR: do not use assignment in if condition
#50: FILE: drivers/base/dmapool.c:369:
+ if ((page = pool_find_page (pool, dma)) == NULL) {
ERROR: no space after that open parenthesis '('
#74: FILE: fs/autofs/waitq.c:185:
+ for ( wql = &sbi->queues; (wq = *wql) != NULL; wql = &wq->next ) {
ERROR: no space before that close parenthesis ')'
#74: FILE: fs/autofs/waitq.c:185:
+ for ( wql = &sbi->queues; (wq = *wql) != NULL; wql = &wq->next ) {
WARNING: line over 80 characters
#155: FILE: fs/jbd/journal.c:232:
+ wait_event(journal->j_wait_done_commit, journal->j_task ==
NULL);
Your patch has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
Cc: Andi Kleen <[EMAIL PROTECTED]>
Cc: Arnd Bergmann <[EMAIL PROTECTED]>
Cc: Davide Libenzi <[EMAIL PROTECTED]>
Cc: Ian Kent <[EMAIL PROTECTED]>
Cc: Jeff Garzik <[EMAIL PROTECTED]>
Cc: Matt Mackall <[EMAIL PROTECTED]>
Cc: Stephen Hemminger <[EMAIL PROTECTED]>
Cc: Stephen Smalley <[EMAIL PROTECTED]>
Signed-off-by: Andrew Morton <[EMAIL PROTECTED]>
---
drivers/base/dmapool.c | 2 +-
fs/autofs/waitq.c | 2 +-
fs/jbd/journal.c | 3 ++-
3 files changed, 4 insertions(+), 3 deletions(-)
diff -puN
drivers/base/dmapool.c~sparse-pointer-use-of-zero-as-null-checkpatch-fixes
drivers/base/dmapool.c
--- a/drivers/base/dmapool.c~sparse-pointer-use-of-zero-as-null-checkpatch-fixes
+++ a/drivers/base/dmapool.c
@@ -366,7 +366,7 @@ dma_pool_free (struct dma_pool *pool, vo
unsigned long flags;
int map, block;
- if ((page = pool_find_page (pool, dma)) == NULL) {
+ if ((page = pool_find_page(pool, dma)) == NULL) {
if (pool->dev)
dev_err(pool->dev, "dma_pool_free %s, %p/%lx (bad
dma)\n",
pool->name, vaddr, (unsigned long) dma);
diff -puN fs/autofs/waitq.c~sparse-pointer-use-of-zero-as-null-checkpatch-fixes
fs/autofs/waitq.c
--- a/fs/autofs/waitq.c~sparse-pointer-use-of-zero-as-null-checkpatch-fixes
+++ a/fs/autofs/waitq.c
@@ -182,7 +182,7 @@ int autofs_wait_release(struct autofs_sb
{
struct autofs_wait_queue *wq, **wql;
- for ( wql = &sbi->queues; (wq = *wql) != NULL; wql = &wq->next ) {
+ for (wql = &sbi->queues; (wq = *wql) != NULL; wql = &wq->next) {
if ( wq->wait_queue_token == wait_queue_token )
break;
}
diff -puN fs/jbd/journal.c~sparse-pointer-use-of-zero-as-null-checkpatch-fixes
fs/jbd/journal.c
--- a/fs/jbd/journal.c~sparse-pointer-use-of-zero-as-null-checkpatch-fixes
+++ a/fs/jbd/journal.c
@@ -230,7 +230,8 @@ static void journal_kill_thread(journal_
while (journal->j_task) {
wake_up(&journal->j_wait_commit);
spin_unlock(&journal->j_state_lock);
- wait_event(journal->j_wait_done_commit, journal->j_task ==
NULL);
+ wait_event(journal->j_wait_done_commit,
+ journal->j_task == NULL);
spin_lock(&journal->j_state_lock);
}
spin_unlock(&journal->j_state_lock);
_
Patches currently in -mm which might be from [EMAIL PROTECTED] are
origin.patch
stop-using-dma_xxbit_mask.patch
fuse-fix-race-between-getattr-and-write.patch
fuse-add-file-handle-to-getattr-operation.patch
sparse-pointer-use-of-zero-as-null.patch
sparse-pointer-use-of-zero-as-null-checkpatch-fixes.patch
i-oat-tighten-descriptor-setup-performance-fix.patch
i-oat-add-completion-callback-for-async_tx-interface-use-fix.patch
i-oat-add-completion-callback-for-async_tx-interface-use-fix-fix.patch
-
To unsubscribe from this list: send the line "unsubscribe mm-commits" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
