[PATCH 0/30] Remove unneeded casts of [kv][mzc]alloc() return values

[PATCH 0/30] Remove unneeded casts of [kv][mzc]alloc() return values

[Jesper Juhl]

Hi,

The [kv][mzc]alloc() functions return a void pointer, so when 
assigning to a pointer type variable there is no need to cast 
the return value.

This is something that is already commonly being cleaned up when 
people change nearby code and it's also something people are 
usually asked not to do when submitting new code. 

I took a look at how many such casts are currently left and didn't 
find that many, so I cleaned them up.
I've probably missed some, but this patch series should clean up 
the majority of the remaining ones at least.

The patches in this series are

 [PATCH 1/30] ia64: Remove unnecessary cast of allocation return value in sn_hwperf_enum_objects()
 [PATCH 1/30] cris: Remove unnecessary cast of allocation return value in intmem.c
 [PATCH 1/30] um: Don't unnecessarily cast allocation return value in ubd_kern.c
 [PATCH 1/30] powerpc: Don't cast kmalloc return value in ibmebus.c
 [PATCH 1/30] atm: No need to cast vmalloc() return value
 [PATCH 1/30] i2o: No need to cast kmalloc() return value in cfg_open()
 [PATCH 1/30] mtd: Get rid of pointless cast of kzalloc() return value in AT26xxx driver
 [PATCH 1/30] mtd: Avoid a pointless kmalloc() return value cast in TQM8xxL mapping handling code
 [PATCH 1/30] mtd: Don't cast kmalloc() return value in drivers/mtd/maps/pmcmsp-flash.c
 [PATCH 1/30] irda: Do not do pointless kmalloc return value cast in KingSun driver
 [PATCH 1/30] net: No need to cast vmalloc() return values in NetXen NIC driver
 [PATCH 1/30] net: No point in casting kmalloc return values in Gianfar Ethernet Driver
 [PATCH 1/30] net: Don't do pointless kmalloc return value casts in zd1211 driver
 [PATCH 1/30] net: Kill some unneeded allocation return value casts in libertas
 [PATCH 1/30] net: No need to cast kmalloc() return value in ipw2100 driver
 [PATCH 1/30] net: Avoid pointless allocation casts in BSD compression module
 [PATCH 1/30] isdn: Get rid of some pointless allocation casts in common and bsd comp.
 [PATCH 1/30] isdn: eicon - get rid of a pointless vmalloc() return value cast
 [PATCH 1/30] scsi: Remove explicit casts of [kv]alloc return values in osst driver
 [PATCH 1/30] scsi: In the Advansys driver, do not cast allocation function return values
 [PATCH 1/30] oss: Remove unneeded vmalloc() return value casts in OSS
 [PATCH 1/30] ivtv: kzalloc() returns void pointer, no need to cast
 [PATCH 1/30] video: Remove pointless kmalloc() return value cast in Zoran PCI controller driver
 [PATCH 1/30] dvb: remove some unneeded vmalloc() return value casts from av7110
 [PATCH 1/30] tty: dont needlessly cast kmalloc() return value
 [PATCH 1/30] md: vmalloc() returns void pointer so we don't need to cast it in dm-ioctl
 [PATCH 1/30] usb: avoid redundant cast of kmalloc() return value in OTi-6858 driver
 [PATCH 1/30] jfs: avoid pointless casts of kmalloc() return val
 [PATCH 1/30] mm: No need to cast vmalloc() return value in zone_wait_table_init()
 [PATCH 1/30] emu10k1: There's no need to cast vmalloc() return value in snd_emu10k1_create()

I'll send them all as replies to this email, to LKML and maintainers.


Kind regards,
  Jesper

Re: [PATCH 0/30] Remove unneeded casts of [kv][mzc]alloc() return values

[Jesper Juhl]

On 24/08/07, Jesper Juhl <jesper.juhl@gmail.com> wrote:
[snip]
> The patches in this series are
> 
Damn, I managed to mess up that list. Here's how it should have looked : 

 [PATCH 01/30] ia64: Remove unnecessary cast of allocation return value in sn_hwperf_enum_objects()
 [PATCH 02/30] cris: Remove unnecessary cast of allocation return value in intmem.c
 [PATCH 03/30] um: Don't unnecessarily cast allocation return value in ubd_kern.c
 [PATCH 04/30] powerpc: Don't cast kmalloc return value in ibmebus.c
 [PATCH 05/30] atm: No need to cast vmalloc() return value
 [PATCH 06/30] i2o: No need to cast kmalloc() return value in cfg_open()
 [PATCH 07/30] mtd: Get rid of pointless cast of kzalloc() return value in AT26xxx driver
 [PATCH 08/30] mtd: Avoid a pointless kmalloc() return value cast in TQM8xxL mapping handling code
 [PATCH 09/30] mtd: Don't cast kmalloc() return value in drivers/mtd/maps/pmcmsp-flash.c
 [PATCH 10/30] irda: Do not do pointless kmalloc return value cast in KingSun driver
 [PATCH 11/30] net: No need to cast vmalloc() return values in NetXen NIC driver
 [PATCH 12/30] net: No point in casting kmalloc return values in Gianfar Ethernet Driver
 [PATCH 13/30] net: Don't do pointless kmalloc return value casts in zd1211 driver
 [PATCH 14/30] net: Kill some unneeded allocation return value casts in libertas
 [PATCH 15/30] net: No need to cast kmalloc() return value in ipw2100 driver
 [PATCH 16/30] net: Avoid pointless allocation casts in BSD compression module
 [PATCH 17/30] isdn: Get rid of some pointless allocation casts in common and bsd comp.
 [PATCH 18/30] isdn: eicon - get rid of a pointless vmalloc() return value cast
 [PATCH 19/30] scsi: Remove explicit casts of [kv]alloc return values in osst driver
 [PATCH 20/30] scsi: In the Advansys driver, do not cast allocation function return values
 [PATCH 21/30] oss: Remove unneeded vmalloc() return value casts in OSS
 [PATCH 22/30] ivtv: kzalloc() returns void pointer, no need to cast
 [PATCH 23/30] video: Remove pointless kmalloc() return value cast in Zoran PCI controller driver
 [PATCH 24/30] dvb: remove some unneeded vmalloc() return value casts from av7110
 [PATCH 25/30] tty: dont needlessly cast kmalloc() return value
 [PATCH 26/30] md: vmalloc() returns void pointer so we don't need to cast it in dm-ioctl
 [PATCH 27/30] usb: avoid redundant cast of kmalloc() return value in OTi-6858 driver
 [PATCH 28/30] jfs: avoid pointless casts of kmalloc() return val
 [PATCH 29/30] mm: No need to cast vmalloc() return value in zone_wait_table_init()
 [PATCH 30/30] emu10k1: There's no need to cast vmalloc() return value in snd_emu10k1_create()

Kind regards,
  Jesper