Re: [PATCH] [KERNEL-DOC] kill warnings when building mandocs

2007-04-17 Thread Borislav Petkov
On Sun, Apr 15, 2007 at 09:10:50AM +0200, Borislav Petkov wrote: On Sat, Apr 14, 2007 at 09:13:41AM -0700, Randy Dunlap wrote: On Fri, 13 Apr 2007 11:14:22 +0200 Borislav Petkov wrote: This patch shuts warnings of the sort: make -C /mnt/samsung_200/sam/kernel/trees/21-rc6/build

Re: [PATCH] [KERNEL-DOC] kill warnings when building mandocs

2007-04-19 Thread Borislav Petkov
snip I'm pretty sure the reason you cannot reproduce this warning is the line xsl:param name=refentry.version.suppress1/xsl:param which can be found in param.xsl, it being a part of the docbook-xsl distribution. The parameter's name is self-explanatory and a '1' suppresses the

Re: [PATCH] [KERNEL-DOC] kill warnings when building mandocs

2007-04-19 Thread Borislav Petkov
On Wed, Apr 18, 2007 at 10:16:54AM -0700, Randy Dunlap wrote: On Fri, 13 Apr 2007 11:29:43 +0200 Borislav Petkov wrote: Sorry for the improper whitespaces, here's a correct version. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] Index: 21-rc6/scripts/kernel-doc

[PATCHv2] [KERNEL-DOC] kill warnings when building mandocs

2007-04-19 Thread Borislav Petkov
A fixed version of the patch shutting up missing version warnings when building mandocs. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] Index: 21-rc7/scripts/kernel-doc === --- 21-rc7.orig/scripts/kernel-doc +++ 21-rc7/scripts

Re: [PATCHv2] [KERNEL-DOC] kill warnings when building mandocs

2007-04-20 Thread Borislav Petkov
On Thu, Apr 19, 2007 at 09:21:22AM -0700, Randy Dunlap wrote: On Thu, 19 Apr 2007 09:19:32 +0200 Borislav Petkov wrote: A fixed version of the patch shutting up missing version warnings when building mandocs. http://www.zip.com.au/~akpm/linux/patches/stuff/tpp.txt :: Please include

Re: [PATCH] [KERNEL-DOC] kill warnings when building mandocs

2007-04-20 Thread Borislav Petkov
On Thu, Apr 19, 2007 at 08:53:36AM -0700, Randy Dunlap wrote: On Thu, 19 Apr 2007 08:20:59 +0200 Borislav Petkov wrote: snip I'm pretty sure the reason you cannot reproduce this warning is the line xsl:param name=refentry.version.suppress1/xsl:param which can

[PATCHv3] [KERNEL-DOC] kill warnings when building mandocs

2007-04-22 Thread Borislav Petkov
On Sat, Apr 21, 2007 at 12:20:45AM -0700, Andrew Morton wrote: On Thu, 19 Apr 2007 09:19:32 +0200 Borislav Petkov [EMAIL PROTECTED] wrote: we already did this in the top-level Makefile. It's in $(KERNELVERSION) and is printed by `make kernelversion'. Cannot we use that info somehow? Yes

[PATCHv3] [KERNEL-DOC] kill warnings when building mandocs (fixed whitespaces)

2007-04-22 Thread Borislav Petkov
Sorry about the wrong whitespaces again :), here's a correct one - From: Borislav Petkov [EMAIL PROTECTED] This patch shuts warnings of the sort: make -C /mnt/samsung_200/sam/kernel/trees/21-rc6/build \ KBUILD_SRC=/mnt/samsung_200/sam/kernel/trees/21-rc6 \ KBUILD_EXTMOD= -f

[PATCHv4] [KERNEL-DOC] kill warnings when building mandocs (final)

2007-04-22 Thread Borislav Petkov
in Documentation/DocBook/Makefile): - From: Borislav Petkov [EMAIL PROTECTED] This patch shuts warnings of the sort: make -C /mnt/samsung_200/sam/kernel/trees/21-rc6/build \ KBUILD_SRC=/mnt/samsung_200/sam/kernel/trees/21-rc6 \ KBUILD_EXTMOD= -f /mnt/samsung_200/sam/kernel/trees/21-rc6

[PATCH] kernel-doc: handle arrays with arithmetic expressions as initializers was Re: [PATCH v2] kernel-doc: handle spaces in array size

2007-04-07 Thread Borislav Petkov
makes the array subscripts unreadable in the docs. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- 21-rc6/scripts/kernel-doc.orig 2007-04-07 16:48:51.0 +0200 +++ 21-rc6/scripts/kernel-doc 2007-04-07 16:51:17.0 +0200 @@ -1456,7 +1456,16 @@ sub create_parameterlist

Re: 2.6.21-rc6-mm1

2007-04-09 Thread Borislav Petkov
Andrew, you can drop +kernel-doc-handle-spaces-in-array-size.patch since +kernel-doc-handle-arrays-with-arithmetic-expressions-as.patch fixes the issue in a slightly better way. -- Regards/Gruß, Boris. - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of

[PATCH] [KERNEL-DOC] generate main index page when building 'htmldocs'

2007-04-10 Thread Borislav Petkov
Here's a patch for kernel-doc that enables the generation of a global, TOC-like index.html page after building 'htmldocs' Signed-off-by: Borislav Petkov [EMAIL PROTECTED] Index: 21-rc6/Documentation/DocBook/Makefile === --- 21

[PATCH] [KERNEL-DOC] alphabetically-sorted entries in index.html of 'htmldocs'

2007-04-11 Thread Borislav Petkov
This patch sorts the DOCBOOKS entries in Documentation/DocBook/Makefile alphabetically. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] Index: 21-rc6/Documentation/DocBook/Makefile === --- 21-rc6.orig/Documentation/DocBook

[PATCH] [KERNEL-DOC] fix tex error when building pdfdocs

2007-04-12 Thread Borislav Petkov
\fi * kernel-hacking.tex /log This is a rather arbitrary fix, so suggest away. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] Index: 21-rc6/Documentation/DocBook/kernel-hacking.tmpl === --- 21-rc6.orig/Documentation/DocBook

Re: [PATCH] [KERNEL-DOC] fix tex error when building pdfdocs

2007-04-13 Thread Borislav Petkov
On Thu, Apr 12, 2007 at 04:48:51PM -0700, Randy Dunlap wrote: On Thu, 12 Apr 2007 22:38:42 +0200 Borislav Petkov wrote: When building pdfdocs, the db2pdf converter bails out because of an latex-reserved token - '#' - in the intermediary .tex file which ends up in a conversion error

[PATCH] [KERNEL-DOC] kill warnings when building mandocs

2007-04-13 Thread Borislav Petkov
DTD for the man header, I believe, for the troff output is truncated too. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] Index: 21-rc6/scripts/kernel-doc === --- 21-rc6.orig/scripts/kernel-doc +++ 21-rc6/scripts/kernel-doc @@ -326,6

Re: [PATCH] [KERNEL-DOC] kill warnings when building mandocs

2007-04-13 Thread Borislav Petkov
Sorry for the improper whitespaces, here's a correct version. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] Index: 21-rc6/scripts/kernel-doc === --- 21-rc6.orig/scripts/kernel-doc +++ 21-rc6/scripts/kernel-doc @@ -326,6 +326,32

Re: [PATCH] [KERNEL-DOC] fix tex error when building pdfdocs

2007-04-14 Thread Borislav Petkov
On Fri, Apr 13, 2007 at 08:20:49AM -0700, Randy Dunlap wrote: On Fri, 13 Apr 2007 08:35:00 +0200 Borislav Petkov wrote: On Thu, Apr 12, 2007 at 04:48:51PM -0700, Randy Dunlap wrote: On Thu, 12 Apr 2007 22:38:42 +0200 Borislav Petkov wrote: When building pdfdocs, the db2pdf

Re: [PATCH] [KERNEL-DOC] kill warnings when building mandocs

2007-04-15 Thread Borislav Petkov
On Sat, Apr 14, 2007 at 09:13:41AM -0700, Randy Dunlap wrote: On Fri, 13 Apr 2007 11:14:22 +0200 Borislav Petkov wrote: This patch shuts warnings of the sort: make -C /mnt/samsung_200/sam/kernel/trees/21-rc6/build \ KBUILD_SRC=/mnt/samsung_200/sam/kernel/trees/21-rc6

[PATCH 20/21] ide-floppy: merge idefloppy_{input,output}_buffers

2008-01-11 Thread Borislav Petkov
variable instead. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c | 67 +++-- 1 files changed, 17 insertions(+), 50 deletions(-) diff --git a/drivers/ide/ide-floppy.c b/drivers/ide/ide-floppy.c index 3d9b1e5..4106eb4 100644

[PATCH 06/21] ide-floppy: remove struct idefloppy_flexible_disk_page

2008-01-11 Thread Borislav Petkov
medium type, this comparison is now done by simply checking whether the medium has changed. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c | 89 - 1 files changed, 32 insertions(+), 57 deletions(-) diff --git a/drivers/ide

[PATCH 07/21] ide-floppy: remove struct idefloppy_capacity_descriptor

2008-01-11 Thread Borislav Petkov
the current/maximum capacity descriptor. i.e the first one. - fix comments formatting Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c | 132 + 1 files changed, 50 insertions(+), 82 deletions(-) diff --git a/drivers/ide/ide

[PATCH 15/21] ide-floppy: disambiguate function names

2008-01-11 Thread Borislav Petkov
which releases that last reference. Rename ide_floppy_release() to idefloppy_cleanup_obj() in order to make its purpose more clear. There should be no functionality change resulting from this patch. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c |6 +++--- 1

[PATCH 14/21] ide-floppy: mv idefloppy_{should_,}report_error

2008-01-11 Thread Borislav Petkov
In addition to shortening the function name, move the printk-call into the function thereby saving some code lines. Also, make the function out_of_line since it is not on a performance critical path. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c | 37

[PATCH 01/21] ide-floppy: convert to generic packet commands

2008-01-11 Thread Borislav Petkov
Replace the ide-floppy packet commands opcode defines with the generic ones. Add a missing GPCMD_WRITE_12 (opcode 0xaa) to the generic ones in cdrom.h. The last one can be found in the current version of INF-8090, p.905. CC: Jens Axboe [EMAIL PROTECTED] Signed-off-by: Borislav Petkov [EMAIL

[PATCH 08/21] ide-floppy: remove struct idefloppy_inquiry_result

2008-01-11 Thread Borislav Petkov
Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c | 41 - 1 files changed, 0 insertions(+), 41 deletions(-) diff --git a/drivers/ide/ide-floppy.c b/drivers/ide/ide-floppy.c index 5c85833..d98264e 100644 --- a/drivers/ide/ide

[PATCH 11/21] ide-floppy: fix comments formatting

2008-01-11 Thread Borislav Petkov
That is, - remove unnecessary comments - shorten comments - shorten lines longer 80 columns - cleanup whitespace - add a missing loglevel KERN_ to a printk-call - fix misc checkpatch warnings Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c | 402

[PATCH 04/21] ide-floppy: cleanup and unify debugging macro calls

2008-01-11 Thread Borislav Petkov
* some debug_log() calls were not using ide-floppy: prefix * a few used printk levels different than KERN_INFO (KERN_NOTICE and KERN_ERR, which is the default one if no level is given) There should be no functional change resulting from this patch. Signed-off-by: Borislav Petkov [EMAIL

[PATCH 03/21] ide-floppy: remove unnecessary -handler != NULL check

2008-01-11 Thread Borislav Petkov
This BUG_ON is unneeded since the -handler != NULL check is performed in ide_set_handler(). Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c |7 +++ 1 files changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/ide/ide-floppy.c b/drivers/ide/ide

[PATCH 00/21] ide-floppy redux v2

2008-01-11 Thread Borislav Petkov
Hi Bart, here's the second version of the ide-floppy refactoring trail. All the patches are based on the version of your quilt tree from the 05.01. Also, you've already applied patch 5 in this series but i'm submitting it still for the sake of completeness. drivers/ide/ide-cd.c |2

[PATCH 09/21] ide-floppy: remove struct idefloppy_request_sense_result

2008-01-11 Thread Borislav Petkov
While at it, collapse idefloppy_analyze_error() into idefloppy_request_sense_callback() since the latter was its only user. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c | 82 + 1 files changed, 24 insertions(+), 58

[PATCH 17/21] ide-floppy: include the proper headers

2008-01-11 Thread Borislav Petkov
Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/ide/ide-floppy.c b/drivers/ide/ide-floppy.c index c889c16..89b26ea 100644 --- a/drivers/ide/ide-floppy.c +++ b/drivers/ide/ide

[PATCH 16/21] ide-floppy: use an xfer_func_t and io_buf_t typedefs in order to unify rw

2008-01-11 Thread Borislav Petkov
Also, move xfer_func_t typedef to the ide.h since it is used by two drivers now (more coming). Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-cd.c |2 -- drivers/ide/ide-floppy.c | 37 ++--- include/linux/ide.h |3 +++ 3

[PATCH 19/21] ide-floppy: fix most of the remaining checkpatch.pl issues

2008-01-11 Thread Borislav Petkov
-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c | 147 +++-- 1 files changed, 75 insertions(+), 72 deletions(-) diff --git a/drivers/ide/ide-floppy.c b/drivers/ide/ide-floppy.c index 0729df5..3d9b1e5 100644 --- a/drivers/ide/ide

[PATCH 21/21] ide-floppy: remove atomic test_*bit macros

2008-01-11 Thread Borislav Petkov
This change is temporary and after unification of the IDE subsystem proper bit setting and testing macros will be introduced. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c | 82 +- 1 files changed, 45 insertions(+), 37

[PATCH 10/21] ide-floppy: remove struct idefloppy_mode_parameter_header

2008-01-11 Thread Borislav Petkov
Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c | 25 - 1 files changed, 4 insertions(+), 21 deletions(-) diff --git a/drivers/ide/ide-floppy.c b/drivers/ide/ide-floppy.c index 7d4ac0b..11c2c9b 100644 --- a/drivers/ide/ide-floppy.c +++ b

[PATCH 18/21] ide-floppy: fix error handling in idefloppy_probe()

2008-01-11 Thread Borislav Petkov
Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/drivers/ide/ide-floppy.c b/drivers/ide/ide-floppy.c index 89b26ea..0729df5 100644 --- a/drivers/ide/ide-floppy.c +++ b/drivers/ide/ide-floppy.c

[PATCH 02/21] ide-floppy: replace ntoh{s,l} and hton{s,l} calls with the generic byteorder

2008-01-11 Thread Borislav Petkov
Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c | 28 ++-- 1 files changed, 14 insertions(+), 14 deletions(-) diff --git a/drivers/ide/ide-floppy.c b/drivers/ide/ide-floppy.c index e4ebb21..e63758a 100644 --- a/drivers/ide/ide-floppy.c

[PATCH 05/21] ide-floppy: remove struct idefloppy_capabilities_page

2008-01-11 Thread Borislav Petkov
typedefs and teach idefloppy_create_mode_sense_cmd() to work directly on u8 buffers. Also, since idefloppy_get_capability_page() was used to read only the sfrp bit, rename the latter so that the name reflects what it does. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c

[PATCH 12/21] ide-floppy: factor out ioctl handlers from idefloppy_ioctl()

2008-01-11 Thread Borislav Petkov
idefloppy_format_start to idefloppy_format_unit Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c | 169 + 1 files changed, 79 insertions(+), 90 deletions(-) diff --git a/drivers/ide/ide-floppy.c b/drivers/ide/ide-floppy.c

Re: [PATCH 06/21] ide-floppy: remove struct idefloppy_flexible_disk_page

2008-01-12 Thread Borislav Petkov
[...] This is not an equivalent transformation: header-wp is 0 or 1 pc.buffer[3] 0x80 is 0 or 0x80 It seems to work fine for -wp (because it is needlessly defined as 'int') but may seriously confuse set_disk_ro() and thus bdev_read_only() users. Should be fixed to '(pc.buffer[3]

Re: [PATCH 00/21] ide-floppy redux v2

2008-01-12 Thread Borislav Petkov
On Sat, Jan 12, 2008 at 09:14:39PM +0100, Bartlomiej Zolnierkiewicz wrote: [...] PS1 Please rebase the patches still needing polishing on top of updated IDE quilt tree, recast them and respin the patch series (no need to post already merged patches). sure, will do. PS2 what happend to fix

Re: [PATCH 18/21] ide-floppy: fix error handling in idefloppy_probe()

2008-01-12 Thread Borislav Petkov
On Sat, Jan 12, 2008 at 09:18:01PM +0100, Bartlomiej Zolnierkiewicz wrote: On Friday 11 January 2008, Borislav Petkov wrote: Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git

[PATCH 2/12] ide-floppy: remove struct idefloppy_flexible_disk_page

2008-01-13 Thread Borislav Petkov
Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c | 90 ++--- 1 files changed, 36 insertions(+), 54 deletions(-) diff --git a/drivers/ide/ide-floppy.c b/drivers/ide/ide-floppy.c index 7648684..06d83cb 100644 --- a/drivers/ide

[PATCH 1/12] ide-floppy: remove struct idefloppy_mode_parameter_header

2008-01-13 Thread Borislav Petkov
Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c | 26 +++--- 1 files changed, 3 insertions(+), 23 deletions(-) diff --git a/drivers/ide/ide-floppy.c b/drivers/ide/ide-floppy.c index 2b9885f..7648684 100644 --- a/drivers/ide/ide-floppy.c +++ b

[PATCH 3/12] ide-floppy: remove struct idefloppy_capacity_descriptor

2008-01-13 Thread Borislav Petkov
-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c | 176 -- 1 files changed, 76 insertions(+), 100 deletions(-) diff --git a/drivers/ide/ide-floppy.c b/drivers/ide/ide-floppy.c index 06d83cb..caeffb7 100644 --- a/drivers/ide/ide-floppy.c +++ b

[PATCH 4/12] ide-floppy: factor out ioctl handlers from idefloppy_ioctl()

2008-01-13 Thread Borislav Petkov
() to idefloppy_format_unit(). Finally, rename the reworked functions to ide_floppy..(). Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c | 153 +++-- 1 files changed, 78 insertions(+), 75 deletions(-) diff --git a/drivers/ide/ide-floppy.c b

[PATCH 5/12] ide-floppy: report DMA handling in idefloppy_pc_intr() properly

2008-01-13 Thread Borislav Petkov
Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c |8 +++- 1 files changed, 7 insertions(+), 1 deletions(-) diff --git a/drivers/ide/ide-floppy.c b/drivers/ide/ide-floppy.c index 00ea8d6..5004773 100644 --- a/drivers/ide/ide-floppy.c +++ b/drivers/ide/ide

[PATCH 7/12] ide-floppy: remove IDEFLOPPY_DEBUG_BUGS macro

2008-01-13 Thread Borislav Petkov
Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c |7 +-- 1 files changed, 1 insertions(+), 6 deletions(-) diff --git a/drivers/ide/ide-floppy.c b/drivers/ide/ide-floppy.c index b826ac5..e7cc9dd 100644 --- a/drivers/ide/ide-floppy.c +++ b/drivers/ide/ide

[PATCH 8/12] ide-floppy: merge idefloppy_{input,output}_buffers

2008-01-13 Thread Borislav Petkov
() to ide_floppy_io_buffers(). Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c | 67 + 1 files changed, 20 insertions(+), 47 deletions(-) diff --git a/drivers/ide/ide-floppy.c b/drivers/ide/ide-floppy.c index e7cc9dd..6c917fc 100644

[PATCH 11/12] ide-floppy: remove IDEFLOPPY_DEBUG_INFO

2008-01-13 Thread Borislav Petkov
This info is already available through ioctl() and /proc Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c | 63 + 1 files changed, 13 insertions(+), 50 deletions(-) diff --git a/drivers/ide/ide-floppy.c b/drivers/ide/ide

[PATCH 9/12] ide-floppy: remove unused IDEFLOPPY_USE_READ12

2008-01-13 Thread Borislav Petkov
This flag is not being set anywhere in the driver so it can go. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c | 17 ++--- 1 files changed, 6 insertions(+), 11 deletions(-) diff --git a/drivers/ide/ide-floppy.c b/drivers/ide/ide-floppy.c index

[PATCH 6/12] ide-floppy: mv idefloppy_{should_,}report_error

2008-01-13 Thread Borislav Petkov
In addition to shortening the function name, move the printk-call into the function thereby saving some code lines. Also, make the function out_of_line since it is not on a performance critical path. Finally, rename the reworked function to ide_floppy..(). Signed-off-by: Borislav Petkov [EMAIL

[PATCH 0/12] ide-floppy redux v2.5

2008-01-13 Thread Borislav Petkov
Hi Bart, here are the remaining patches which had issues to be worked out. I decided to keep the Flexible Disk Page and Capacity Descriptor in idefloppy_floppy_t for the sake of the two printk calls for which they are used. Otherwise, we'll be changing long-known driver behavior and this won't be

[PATCH 10/12] ide-floppy: remove atomic test_*bit macros

2008-01-13 Thread Borislav Petkov
..and replace them with flag enums. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c | 132 + 1 files changed, 73 insertions(+), 59 deletions(-) diff --git a/drivers/ide/ide-floppy.c b/drivers/ide/ide-floppy.c index

[PATCH 12/12] ide-floppy: fix most of the remaining checkpatch.pl issues

2008-01-13 Thread Borislav Petkov
on. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c | 357 +- 1 files changed, 161 insertions(+), 196 deletions(-) diff --git a/drivers/ide/ide-floppy.c b/drivers/ide/ide-floppy.c index f9faf26..9361dfd 100644

Re: [PATCH 10/12] ide-floppy: remove atomic test_*bit macros

2008-01-14 Thread Borislav Petkov
On Mon, Jan 14, 2008 at 10:50:58PM +0100, Bartlomiej Zolnierkiewicz wrote: On Sunday 13 January 2008, Borislav Petkov wrote: ..and replace them with flag enums. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c | 132

Re: [PATCH 8/12] ide-floppy: merge idefloppy_{input,output}_buffers

2008-01-14 Thread Borislav Petkov
On Mon, Jan 14, 2008 at 10:38:35PM +0100, Bartlomiej Zolnierkiewicz wrote: On Sunday 13 January 2008, Borislav Petkov wrote: We merge idefloppy_{input,output}_buffers() into idefloppy_io_buffers() by introducing a 4th arg. called direction. According to its value we atapi_input_bytes

[PATCH 0/2] unify DMA_..BIT_MASK definitions

2007-09-17 Thread Borislav Petkov
This patches remove redundant DMA_..BIT_MASK definitions across two drivers. First off, consolidate dma bitmask definitions in the proper header file... Signed-off-by: Borislav Petkov [EMAIL PROTECTED] -- Index: 23-rc6/include/linux/dma-mapping.h

[PATCH 2/2] unify DMA_..BIT_MASK definitions: cleanup drivers/scsi/gdth.c

2007-09-17 Thread Borislav Petkov
move local #defines to the dma-mappings header. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] -- Index: 23-rc6/drivers/scsi/gdth.c === --- 23-rc6/drivers/scsi/gdth.c.orig 2007-09-17 17:53:26.0 +0200 +++ 23-rc6/drivers

[PATCH 1/2] unify DMA_..BIT_MASK definitions: netxen local defs

2007-09-17 Thread Borislav Petkov
Move dma bitmask definitions into the dma-mappings header. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] -- Index: 23-rc6/drivers/net/netxen/netxen_nic_main.c === --- 23-rc6/drivers/net/netxen/netxen_nic_main.c.orig2007-09-17

Re: [PATCH 0/2] unify DMA_..BIT_MASK definitions

2007-09-17 Thread Borislav Petkov
On Mon, Sep 17, 2007 at 11:14:48AM -0700, Jeremy Fitzhardinge wrote: Borislav Petkov wrote: This patches remove redundant DMA_..BIT_MASK definitions across two drivers. First off, consolidate dma bitmask definitions in the proper header file... Signed-off-by: Borislav Petkov [EMAIL

[PATCH 2/2] unify DMA_..BIT_MASK definitions v1: cleanup drivers/scsi/gdth.c

2007-09-17 Thread Borislav Petkov
Move dma bitmask definitions into the dma-mappings header. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] Cc: Jeremy Fitzhardinge [EMAIL PROTECTED] -- Index: 23-rc6/drivers/scsi/gdth.c === --- 23-rc6/drivers/scsi/gdth.c.orig

[PATCH 0/2] unify DMA_..BIT_MASK definitions: v1

2007-09-17 Thread Borislav Petkov
These patches remove redundant DMA_..BIT_MASK definitions across two drivers. In this version of the patches, the computation of the bitmasks is done by the compiler. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] Cc: Jeremy Fitzhardinge [EMAIL PROTECTED] -- Index: 23-rc6/include/linux/dma

[PATCH 1/2] unify DMA_..BIT_MASK definitions v1: netxen local defs

2007-09-17 Thread Borislav Petkov
Move dma bitmask definitions into the dma-mappings header. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] Cc: Jeremy Fitzhardinge [EMAIL PROTECTED] -- Index: 23-rc6/drivers/net/netxen/netxen_nic_main.c === --- 23-rc6/drivers/net

Re: [patch 4/7] Immediate Values - i386 Optimization

2007-09-18 Thread Borislav Petkov
On Mon, Sep 17, 2007 at 02:42:28PM -0400, Mathieu Desnoyers wrote: i386 optimization of the immediate values which uses a movl with code patching to set/unset the value used to populate the register used as variable source. Changelog: - Use text_poke_early with cr0 WP save/restore to patch

Re: [PATCH 0/2] unify DMA_..BIT_MASK definitions: v1

2007-09-18 Thread Borislav Petkov
On Mon, Sep 17, 2007 at 11:01:21PM -0700, Jeremy Fitzhardinge wrote: Muli Ben-Yehuda wrote: +#define DMA_64BIT_MASKDMA_BIT_MASK(64) This one does not do what you mean. You need an explicit mask or a ~0ULL here. Yeah, I was just about to comment on it. Its possible the

[PATCH 0/2] unify DMA_..BIT_MASK definitions: v2

2007-09-18 Thread Borislav Petkov
These patches remove redundant DMA_..BIT_MASK definitions across two drivers. In this version of the patches, the computation of the majority of the bitmasks is done by the compiler. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] Cc: Jeremy Fitzhardinge [EMAIL PROTECTED] Cc: Muli Ben-Yehuda

[PATCH 1/2] unify DMA_..BIT_MASK definitions v2: netxen local defs

2007-09-18 Thread Borislav Petkov
Move dma bitmask definitions into the dma-mappings header. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] Cc: Jeremy Fitzhardinge [EMAIL PROTECTED] Cc: Muli Ben-Yehuda [EMAIL PROTECTED] --- 23-rc6/drivers/net/netxen/netxen_nic_main.c |3 --- 1 file changed, 3 deletions(-) Index: b/23-rc6

[PATCH 2/2] unify DMA_..BIT_MASK definitions v2: cleanup drivers/scsi/gdth.c

2007-09-18 Thread Borislav Petkov
Move dma bitmask definitions into the dma-mappings header. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] Cc: Jeremy Fitzhardinge [EMAIL PROTECTED] Cc: Muli Ben-Yehuda [EMAIL PROTECTED] --- 23-rc6/drivers/scsi/gdth.c |5 - 1 file changed, 5 deletions(-) Index: b/23-rc6/drivers/scsi

Re: [PATCH 0/2] unify DMA_..BIT_MASK definitions: v1

2007-09-18 Thread Borislav Petkov
On Tue, Sep 18, 2007 at 11:46:40AM +0530, Satyam Sharma wrote: Hi Borislav, On Tue, 18 Sep 2007, Borislav Petkov wrote: On Mon, Sep 17, 2007 at 11:01:21PM -0700, Jeremy Fitzhardinge wrote: Muli Ben-Yehuda wrote: +#define DMA_64BIT_MASKDMA_BIT_MASK(64

[PATCH 1/1] unify DMA_..BIT_MASK definitions: v3

2007-09-18 Thread Borislav Petkov
this patch for it touches drivers maintained by different people and there might be responsibility issues, imho. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] Cc: Jeremy Fitzhardinge [EMAIL PROTECTED] Cc: Muli Ben-Yehuda [EMAIL PROTECTED] drivers/net/netxen/netxen_nic_main.c |3 --- drivers/scsi

[PATCH 1/1] unify DMA_..BIT_MASK definitions: v3.1

2007-09-18 Thread Borislav Petkov
this patch for it touches drivers maintained by different people and i there might be responsibility issues, imho. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] Cc: Jeremy Fitzhardinge [EMAIL PROTECTED] Cc: Muli Ben-Yehuda [EMAIL PROTECTED] drivers/net/netxen/netxen_nic_main.c |3 --- drivers

[PATCH] fs/isofs/namei.c: Remove uninitialized local vars warning, was: Re: Arr! Linux 2.6.23-rc7

2007-09-19 Thread Borislav Petkov
Hi Andrew, since there's no iso9660 maintainer i send this one to you :). --- From: Borislav Petkov [EMAIL PROTECTED] shut up those: fs/isofs/namei.c: In function 'isofs_lookup': fs/isofs/namei.c:161: warning: 'offset' may be used uninitialized in this function fs/isofs/namei.c:161: warning

Re: [PATCH] keyspan: init termios properly

2007-11-15 Thread Borislav Petkov
On Thu, Nov 15, 2007 at 09:49:22AM -0800, Lucy McCoy wrote: Greg Boris, I've been out of town and just saw this. This patch will work with the 19HS but WILL BREAK all other Keyspan adapters. It will take me a few days to get to looking at a correct fix but that keyspan_send_setup(port, 1)

Re: [PATCH] keyspan: init termios properly

2007-11-15 Thread Borislav Petkov
On Thu, Nov 15, 2007 at 01:10:16PM -0800, Lucy McCoy wrote: Hi Lucy, I'm not familiar with the termios stuff on Linux so can you take a look at the following modified code to see if this solves your NULL ptr problem? will test the patch tomorrow and get back to you with results. Does cflag

Re: [rfc-patch 0/9] Immediate Values for 2.6.24-rc2-git5

2007-11-17 Thread Borislav Petkov
On Fri, Nov 16, 2007 at 03:02:38PM -0500, Mathieu Desnoyers wrote: Hi, just a conventions proposal: have you thought of shortening all those immediate_foo prefixes to 'imm_foo', for example? This'll make the code much more readable, i think. - Regards/Gruß, Boris. - To unsubscribe from

Re: [PATCH] keyspan: init termios properly

2007-11-18 Thread Borislav Petkov
On Thu, Nov 15, 2007 at 01:10:16PM -0800, Lucy McCoy wrote: static int keyspan_open (struct usb_serial_port *port, struct file *filp) { struct keyspan_port_private *p_priv; struct keyspan_serial_private *s_priv; struct usb_serial *serial = port-serial;

[PATCH] scripts/checkpatch.pl: add a check for the patch level (patch -pnum)

2007-12-16 Thread Borislav Petkov
Being bitten by this several times myself here's a quick hack for checking the patch level of the diffs in a patch file. It works only when checkpatch.pl is called from within the kernel tree. --- Signed-off-by: Borislav Petkov [EMAIL PROTECTED] -- scripts/checkpatch.pl | 18

[PATCH] drivers/cdrom/cdrom.c: simplify logic in cdrom_release()

2007-12-17 Thread Borislav Petkov
--- From: Borislav Petkov [EMAIL PROTECTED] Simplify logic in cdrom_release() without semantical change. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] -- drivers/cdrom/cdrom.c | 17 + 1 files changed, 9 insertions(+), 8 deletions(-) diff --git a/drivers/cdrom/cdrom.c b

Re: [PATCH] scripts/checkpatch.pl: add a check for the patch level (patch -pnum)

2007-12-17 Thread Borislav Petkov
On Mon, Dec 17, 2007 at 08:11:05AM +0100, Borislav Petkov wrote: A slightly microoptimized version 1.1: --- From: Borislav Petkov [EMAIL PROTECTED] Check the patch level of the single hunks in a patch file, however only when checkpatch.pl is called from within the kernel tree. Signed-off

Re: [PATCH 00/63] ide-cd: redux

2007-12-20 Thread Borislav Petkov
On Thu, Dec 20, 2007 at 01:48:49AM +0100, Bartlomiej Zolnierkiewicz wrote: Hi Bart, Hi all, PS ide-cd Maintainer position is still open... i'd like to take that position if nobody objects. I've done some work around the kernel here and there and i think its time for me to get serious. Thanks.

Re: [PATCH 00/63] ide-cd: redux

2007-12-22 Thread Borislav Petkov
On Fri, Dec 21, 2007 at 08:57:49PM +0100, Bartlomiej Zolnierkiewicz wrote: On Thursday 20 December 2007, Borislav Petkov wrote: On Thu, Dec 20, 2007 at 01:48:49AM +0100, Bartlomiej Zolnierkiewicz wrote: Hi Bart, Hi all, PS ide-cd Maintainer position is still open... i'd like to take

Re: [PATCH] scripts/checkpatch.pl: add a check for the patch level (patch -pnum)

2007-12-22 Thread Borislav Petkov
On Fri, Dec 21, 2007 at 06:12:02PM +, Andy Whitcroft wrote: On Tue, Dec 18, 2007 at 06:46:41AM +0100, Borislav Petkov wrote: On Mon, Dec 17, 2007 at 08:11:05AM +0100, Borislav Petkov wrote: A slightly microoptimized version 1.1: --- From: Borislav Petkov [EMAIL PROTECTED

Re: [PATCH] keyspan: init termios properly

2007-11-29 Thread Borislav Petkov
On Mon, Nov 26, 2007 at 02:18:52PM -0800, Andrew Morton wrote: On Sun, 18 Nov 2007 14:11:30 +0100 Borislav Petkov [EMAIL PROTECTED] wrote: On Thu, Nov 15, 2007 at 01:10:16PM -0800, Lucy McCoy wrote: ... yes, after testing this i can confirm that this one fixes the NULL ptr problem here

Re: [PATCH] USB: option: Bind to the correct interface of the Huawei E220

2007-12-01 Thread Borislav Petkov
On Fri, Nov 30, 2007 at 04:30:11PM +, Jaime Velasco Juan wrote: Signed-off-by: Oliver Neukum [EMAIL PROTECTED] Signed-off-by: Jaime Velasco Juan [EMAIL PROTECTED] --- diff --git a/drivers/usb/serial/option.c b/drivers/usb/serial/option.c index a18659e..352c94c 100644 ---

Re: [PATCH] keyspan: init termios properly

2007-12-02 Thread Borislav Petkov
On Fri, Nov 30, 2007 at 09:23:43AM -0800, Lucy McCoy wrote: Hi All, I've been too busy to get to this but i'd rather not use the code fragment i sent Boris to try. It would be better to go ahead with the tty setup if the pointer is not NULL, otherwise use the defaults and not reference the

Re: [PATCH] keyspan: init termios properly

2007-12-02 Thread Borislav Petkov
On Sun, Dec 02, 2007 at 01:57:34PM +, Alan Cox wrote: On Sun, 2 Dec 2007 09:03:40 +0100 Borislav Petkov [EMAIL PROTECTED] wrote: On Fri, Nov 30, 2007 at 09:23:43AM -0800, Lucy McCoy wrote: Hi All, I've been too busy to get to this but i'd rather not use the code fragment

[RFC] swap image signature check upon resume

2007-12-06 Thread Borislav Petkov
Hi Pavel, hi Rafael, after a quick search i couldn't find anything dealing with the topic in the subject line so here we go: One sometimes can mix up (and by one i mean me) the kernel images one boots after having suspended the machine previously. There can be at least two reasons for that: 1.

Re: [RFC] swap image signature check upon resume

2007-12-07 Thread Borislav Petkov
On Thu, Dec 06, 2007 at 10:46:48PM +0100, Rafael J. Wysocki wrote: On Thursday, 6 of December 2007, Borislav Petkov wrote: Hi Pavel, hi Rafael, after a quick search i couldn't find anything dealing with the topic in the subject line so here we go: One sometimes can mix up

Re: [RFC] swap image signature check upon resume

2007-12-08 Thread Borislav Petkov
need a tester when you have some prototypes ready. By the way, what do you do when the suspend image header mismatches and it is unsafe to continue booting? Also, there's a freakishly long comment in suspend_64.c, might wanna shorten it: Signed-off-by: Borislav Petkov [EMAIL PROTECTED] diff --git

Re: [rfc-patch 0/9] Immediate Values for 2.6.24-rc2-git5

2007-11-19 Thread Borislav Petkov
On Mon, Nov 19, 2007 at 10:31:39AM -0500, Mathieu Desnoyers wrote: * Borislav Petkov ([EMAIL PROTECTED]) wrote: On Fri, Nov 16, 2007 at 03:02:38PM -0500, Mathieu Desnoyers wrote: Hi, just a conventions proposal: have you thought of shortening all those immediate_foo prefixes to 'imm_foo

[PATCH] vmlinux_32.lds.S: remove repeated comment from the x86-32 linker script

2007-11-25 Thread Borislav Petkov
From: Borislav Petkov [EMAIL PROTECTED] Remove repeated comment from the linker script for the x86-32 target. Signed-off-by: Borislav Petkov [EMAIL PROTECTED] -- arch/x86/kernel/vmlinux_32.lds.S |6 -- 1 files changed, 0 insertions(+), 6 deletions(-) diff --git a/arch/x86/kernel

Re: [RFC] swap image signature check upon resume

2007-12-08 Thread Borislav Petkov
On Sat, Dec 08, 2007 at 11:50:33PM +0100, Rafael J. Wysocki wrote: On Saturday, 8 of December 2007, Borislav Petkov wrote: On Fri, Dec 07, 2007 at 09:19:09PM +0100, Rafael J. Wysocki wrote: ... Well, there's a patchset in the current mainline that allows you to use

2.6.24-rc4-mm1: acpi reboots machine

2007-12-08 Thread Borislav Petkov
Hi Andrew, Hi Len, after booting 2.6.24-rc4-mm1 (2.6.24-rc4-190-g94545ba, otoh, boots just fine) on my asus laptop, the machine reboots after claiming that Critical temperature reached (255 C). However, the degrees number is kinda hinting at 0xff all-ones field. Will try dump_stack in

Re: 2.6.24-rc4-mm1: acpi reboots machine

2007-12-09 Thread Borislav Petkov
On Sun, Dec 09, 2007 at 08:50:02AM +0100, Borislav Petkov wrote: Hi Andrew, Hi Len, after booting 2.6.24-rc4-mm1 (2.6.24-rc4-190-g94545ba, otoh, boots just fine) on my asus laptop, the machine reboots after claiming that Critical temperature reached (255 C). However, the degrees number

Re: [RFC] swap image signature check upon resume

2007-12-09 Thread Borislav Petkov
On Sun, Dec 09, 2007 at 03:27:57PM +0100, Rafael J. Wysocki wrote: ... Instead, I'd rather issue a warning that the swsusp header mismatches, say with which kernel the machine got suspended with and then start the countdown for reboot. What exactly would that change? You need to

Re: [RFC] swap image signature check upon resume

2007-12-09 Thread Borislav Petkov
On Sun, Dec 09, 2007 at 10:46:35PM +0100, Rafael J. Wysocki wrote: On Sunday, 9 of December 2007, Borislav Petkov wrote: On Sun, Dec 09, 2007 at 03:27:57PM +0100, Rafael J. Wysocki wrote: ... Instead, I'd rather issue a warning that the swsusp header mismatches, say with which

Re: 2.6.24-rc4-mm1: acpi reboots machine... solved

2007-12-11 Thread Borislav Petkov
On Sun, Dec 09, 2007 at 10:19:47AM +0100, Borislav Petkov wrote: On Sun, Dec 09, 2007 at 08:50:02AM +0100, Borislav Petkov wrote: Hi Andrew, Hi Len, after booting 2.6.24-rc4-mm1 (2.6.24-rc4-190-g94545ba, otoh, boots just fine) on my asus laptop, the machine reboots after claiming

Re: 2.6.24-rc4-mm1: acpi reboots machine... solved

2007-12-11 Thread Borislav Petkov
On Tue, Dec 11, 2007 at 01:00:24PM -0700, Bjorn Helgaas wrote: On Tuesday 11 December 2007 10:44:43 am Borislav Petkov wrote: On Sun, Dec 09, 2007 at 10:19:47AM +0100, Borislav Petkov wrote: On Sun, Dec 09, 2007 at 08:50:02AM +0100, Borislav Petkov wrote: Hi Andrew, Hi Len

  1   2   3   4   5   6   7   8   9   10   >