[PATCH]: sound: ioremap/iounmap function balancing audit

2007-07-31 Thread Scott Thompson
s a memory leak). i810_audio.c had a 'fixme' issue regarding ioremap fail that was addressed as well. Signed-off-by: Scott Thompson hushmail.com> - --- diff --git a/sound/oss/i810_audio.c b/sound/oss/i810_audio.c index f5e31f1..407958c 100644 - --- a/sound/oss/i810_audio.c +++ b

Re: [PATCH]: sound: ioremap/iounmap function balancing audit

2007-07-31 Thread Scott Thompson
>please >split this patch into at least one for sound/oss and one for >sound/soc and >sound/pci, thanks. Will do, though wordwrapping issues will still be there. The part of the function I was patching in i810_probe is nested so deep birds have taken up residence. ----

[PATCH 2/3] sound/pci ioremap/iounmap balancing

2007-07-31 Thread Scott Thompson
ioremap / iounmap balancing in sound/pci tree Signed-off-by: Scott Thompson hushmail.com> --- diff --git a/sound/pci/mixart/mixart.c b/sound/pci/mixart/mixart.c index ac007ce..871b09f 100644 --- a/sound/pci/mixart/mixart.c +++ b/sound/pci/mixart/mixart.c @@ -1319,6 +1319,13 @@ static

[PATCH 3/3] sound/soc ioremap/iounmap balancing

2007-07-31 Thread Scott Thompson
ioremap / iounmap balancing in sound/soc tree Signed-off-by: Scott Thompson hushmail.com> --- diff --git a/sound/soc/s3c24xx/s3c24xx-i2s.c b/sound/soc/s3c24xx/s3c24xx-i2s.c index 39f0246..cd89c41 100644 --- a/sound/soc/s3c24xx/s3c24xx-i2s.c +++ b/sound/soc/s3c24xx/s3c24xx-i2s.c @@ -385,6 +38

Re: [PATCH 2/3] sound/pci ioremap/iounmap balancing

2007-08-01 Thread Scott Thompson
t. Thus, you don't need iounmap in each place. > > >thanks, > >Takashi Missed the callback on the audit, will repost this patch (sound/pci) with just the unchecked returns... --- Scott Thompson / [EMAIL PROTECTED] --

[PATCH 1/4] /drivers/net ioremap balancing/returncode check (es3210.c)

2007-08-10 Thread Scott Thompson
f for iounmap this audit was only for files directly in drivers/net directory, not the full tree. I split patches up by file changed as each file had different 'owner' to ease accept/reject/revision process. Signed-off-by: Scott Thompso

[PATCH 3/4] /drivers/net ioremap balancing/returncode check (myri_sbus.c)

2007-08-10 Thread Scott Thompson
f for iounmap this audit was only for files directly in drivers/net directory, not the full tree. I split patches up by file changed as each file had different 'owner' to ease accept/reject/revision process. Signed-off-by: Scott Thompso

[PATCH 2/4] /drivers/net ioremap balancing/returncode check (lne390.c)

2007-08-10 Thread Scott Thompson
f for iounmap this audit was only for files directly in drivers/net directory, not the full tree. I split patches up by file changed as each file had different 'owner' to ease accept/reject/revision process. Signed-off-by: Scott Thompso

[PATCH 4/4] /drivers/net ioremap balancing/returncode check (ucc_geth.c.c)

2007-08-10 Thread Scott Thompson
f for iounmap this audit was only for files directly in drivers/net directory, not the full tree. I split patches up by file changed as each file had different 'owner' to ease accept/reject/revision process. Signed-off-by: Scott Thompso

[PATCH ] /drivers/ata ioremap returncode check (pata_ixp4xx_cf.c)

2007-08-12 Thread Scott Thompson
patch against 2.6.23-rc2. this corrects two missing ioremap return checks. Signed-off-by: Scott Thompson hushmail.com> -- diff --git a/drivers/ata/pata_ixp4xx_cf.c b/drivers/ata/pata_ixp4xx_cf.c index 4ca7fd6..eaebc4d 100644 --

[PATCH ] /drivers/atm ioremap balancing/ returncode check

2007-08-12 Thread Scott Thompson
patchset against 2.6.23-rc2. corrects missing ioremap return checks and balancing on iounmap calls in /drivers/atm... Signed-off-by: Scott Thompson hushmail.com> -- diff --git a/drivers/atm/fore200e.c b/drivers/atm/fore200e.c in

[PATCH 1 of 5 ] /drivers/char ioremap balancing/ returncode check

2007-08-12 Thread Scott Thompson
patchset against 2.6.23-rc2 and this set is an audit of /drivers/char/a* through drivers/char . this corrects missing ioremap return checks and balancing on iounmap calls.. Signed-off-by: Scott Thompson hushmail.com> -- diff --gi

[PATCH 2 of 5 ] /drivers/char/agp ioremap balancing/ returncode check

2007-08-12 Thread Scott Thompson
patchset against 2.6.23-rc2 and this set is an audit of /drivers/char/a* through drivers/char . this corrects missing ioremap return checks and balancing on iounmap calls.. Signed-off-by: Scott Thompson hushmail.com> -- diff --gi

[PATCH 3 of 5 ] /drivers/char/drm ioremap balancing/ returncode check

2007-08-12 Thread Scott Thompson
patchset against 2.6.23-rc2 and this set is an audit of /drivers/char/a* through drivers/char . this corrects missing ioremap return checks and balancing on iounmap calls.. Signed-off-by: Scott Thompson hushmail.com> -- diff --gi

[PATCH 4 of 5 ] /drivers/char/rio ioremap balancing/ returncode check

2007-08-12 Thread Scott Thompson
patchset against 2.6.23-rc2 and this set is an audit of /drivers/char/a* through drivers/char . this corrects missing ioremap return checks and balancing on iounmap calls.. Signed-off-by: Scott Thompson hushmail.com> -- diff --gi

[PATCH 5 of 5 ] /drivers/char/watchdog ioremap balancing/ returncode check

2007-08-12 Thread Scott Thompson
patchset against 2.6.23-rc2 and this set is an audit of /drivers/char/a* through drivers/char . this corrects missing ioremap return checks and balancing on iounmap calls.. Signed-off-by: Scott Thompson hushmail.com> -- diff --gi

Re: [PATCH 4 of 5 ] /drivers/char/rio ioremap balancing/ returncode check

2007-08-13 Thread Scott Thompson
up ioremap usage. And, yes, I contributed *one more* wordwrap issue to the file, but when in Rome... --- Scott Thompson / [EMAIL PROTECTED] --- - To unsubscribe from this list: send the line "unsubscribe linux-kerne

Withdrawn for now: /drivers/char/* /drivers/net/* drivers/atm/* ioremap balancing/ returncode check

2007-08-13 Thread Scott Thompson
he server side of my mail client. Will resubmit after resolving issue w/ mail provider... ------- Scott Thompson / [EMAIL PROTECTED] --- - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the

[PATCH ] /drivers/ata ioremap returncode check

2007-08-13 Thread Scott Thompson
different mail client. Signed-off-by: Scott Thompson hushmail.com> -- diff --git a/drivers/ata/pata_ixp4xx_cf.c b/drivers/ata/pata_ixp4xx_cf.c index 4ca7fd6..eaebc4d 100644 --- a/drivers/ata/pata_ixp4xx_cf.c +++ b/drivers/ata/pata_ixp4xx_c

Re: Withdrawn for now: /drivers/char/* /drivers/net/* drivers/atm/* ioremap balancing/ returncode check

2007-08-13 Thread Scott Thompson
hushmail users can be warned that wordwrapping will be an issue on patch submission and another route will be required. ------- Scott Thompson / [EMAIL PROTECTED] --- - To unsubscribe from this list: send the line "unsubscribe lin

[PATCH ] /drivers/atm ioremap balancing/ returncode check (resend)

2007-08-13 Thread Scott Thompson
wordwrap issues, resending with different mail client. Signed-off-by: Scott Thompson hushmail.com> -- diff --git a/drivers/atm/fore200e.c b/drivers/atm/fore200e.c index 405ee5e..99e4ae4 100644 --- a/drivers/atm/fore200e.c +++ b/drivers/

Re: [PATCH 4 of 5 ] /drivers/char/rio ioremap balancing/ returncode check

2007-08-13 Thread Scott Thompson
ries). If there was a better list, or just to linux-kernel, please let me know as tracking down the 'right' list for these kernel-janitor kind of tasks can be tricky --- Scott Thompson / [EMAIL PROTECTED] ---

Re: [PATCH ] /drivers/atm ioremap balancing/ returncode check (re-resend)

2007-08-13 Thread Scott Thompson
AM"); Warning -- cleanup handler here may miss additional required cleanup as has occurred on other portions of ioremap audit. This patch had been submitted previously but hushmail client caused wordwrap issues, resending with different mail client.

[PATCH 1/3] /drivers/net ioremap balancing/returncode check (resend)

2007-08-13 Thread Scott Thompson
lieve had incorrect/not ideal variable check on if for iounmap this audit was only for files directly in drivers/net directory, not the full tree. Signed-off-by: Scott Thompson hushmail.com> -- diff --git a/drivers/net/es3210.c b/dri

[PATCH 2/3] /drivers/net myri_sbus.c ioremap balancing/returncode check (resend)

2007-08-13 Thread Scott Thompson
lieve had incorrect/not ideal variable check on if for iounmap this audit was only for files directly in drivers/net directory, not the full tree. Signed-off-by: Scott Thompson hushmail.com> -- diff --git a/drivers/net/myri_sbus.c b/dri

[PATCH 3/3] /drivers/net ucc_geth ioremap balancing/returncode check (resend)

2007-08-13 Thread Scott Thompson
lieve had incorrect/not ideal variable check on if for iounmap this audit was only for files directly in drivers/net directory, not the full tree. Signed-off-by: Scott Thompson hushmail.com> -- diff --git a/drivers/net/ucc_geth.c b/dri

Re: [PATCH 1 of 5 ] /drivers/char ioremap balancing/ returncode check

2007-08-14 Thread Scott Thompson
values should be changed. Let me know your preferences on these issues and I will include in the next pass. Thanks for your comments and review... --- Scott Thompson / [EMAIL PROTECTED] --- - To unsubscribe from this list: send the line "unsubsc

Re: [PATCH 4 of 5 ] /drivers/char/rio ioremap balancing/ returncode check

2007-08-15 Thread Scott Thompson
nitor faqs. Some groups seem more receptive to these kind of fixes than others, though they may just be more active maintainers. I don't know if anyone does or doesn't roll these up collectively, maybe the list can advise...

[PATCH resubmit] /drivers/ata ioremap returncode check

2007-08-22 Thread Scott Thompson
Patchset against 2.6.23-rc3. corrects missing ioremap return checks, resending after making changes suggested Signed-off-by: Scott Thompson hushmail.com> diff --git a/drivers/ata/pata_ixp4xx_cf.c b/drivers/ata/pata_ixp4xx_c

[PATCH 3 of 5 resend ] /drivers/char/drm ioremap balancing/ returncode check

2007-08-22 Thread Scott Thompson
audit. This patch had been submitted previously but hushmail client caused wordwrap issues, resending with different mail client. Signed-off-by: Scott Thompson hushmail.com> -- diff --git a/drivers/char/drm/drm_bufs.c b/drivers/char/

[PATCH 1 of 5 resend ] /drivers/char ioremap balancing/ returncode check

2007-08-22 Thread Scott Thompson
nup handler here may miss additional required cleanup as has occurred on other portions of ioremap audit. This patch had been submitted previously but hushmail client caused wordwrap issues, resending with different mail client. Signed-off-by: Scott Thompson hushmail.

[PATCH 2 of 5 resend ] /drivers/char/agp ioremap balancing/ returncode check

2007-08-22 Thread Scott Thompson
audit. This patch had been submitted previously but hushmail client caused wordwrap issues, resending with different mail client. Signed-off-by: Scott Thompson hushmail.com> -- diff --git a/drivers/char/agp/amd-k7-agp.c b/drivers/char/agp/

[PATCH 4 of 5 resend] /drivers/char/rio ioremap balancing/ returncode check

2007-08-22 Thread Scott Thompson
eas... Warning -- cleanup handler here may miss additional required cleanup as has occurred on other portions of ioremap audit. This patch had been submitted previously but hushmail client caused wordwrap issues, resending with different mail client. Signed-off-by: Scott Thompson hushmail.

[PATCH 5 of 5 resend ] /drivers/char/watchdog ioremap balancing/ returncode check

2007-08-22 Thread Scott Thompson
audit. This patch had been submitted previously but hushmail client caused wordwrap issues, resending with different mail client. Signed-off-by: Scott Thompson hushmail.com> -- diff --git a/drivers/char/watchdog/iTCO_wdt.c b/drivers/c

Ideas on column length in kernel "problem"?

2007-08-22 Thread Scott Thompson
ments for the current state of the linux tree please advise. --- Scott Thompson / [EMAIL PROTECTED] --- - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info

[PATCH resubmit] /drivers/ata ioremap returncode check

2007-08-23 Thread Scott Thompson
t contain these messages. When auditing (as I am for ioremap checking here) I am attempting to keep changes "in the style of their surroundings". Otherwise it's massive scope screep on these patches. Signed-off-by

Re: Ideas on column length in kernel "problem"?

2007-08-24 Thread Scott Thompson
ic __devinit int ixp4xx_pata_probe(struct platform_device *pdev) Common advice appears to be to get smtp working on gmail, and I thank the list for their suggestions. If I get one/two ways working that are compliant for patch submittal w/o wordwrap (et al) I'll start a 'howto' fr

[PATCH] /drivers/char sx.c ioremap -> pci_ioremap api

2007-08-24 Thread Scott Thompson
patchset against 2.6.23-rc3. moves the iomap/iounmap call to pci 'flavor'. this was a recommendation from a previously submitted patch that handles the unchecked iomap (or, now, pci_iomap) return code. Signed-off-by: Scott Thompson hu

Re: Ideas on column length in kernel "problem"?

2007-08-24 Thread Scott Thompson
email clients + settings and will post a link once a copy is up on the interweb. --- Scott Thompson / [EMAIL PROTECTED] --- - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body

Re: [PATCH] /drivers/char sx.c ioremap -> pci_ioremap api

2007-08-24 Thread Scott Thompson
r NAK is fine here and I won't resubmit w/ the hwbase var pulled out or the readl -> ioread32 switchover. --- Scott Thompson / [EMAIL PROTECTED] --- - To unsubscribe from this list: send the line "unsubscribe