leandrof/u-boot
1
0
Fork 1
Code Issues Pull Requests Packages Projects Releases Wiki Activity

net: ks8851: Replace malloc()+memset() with calloc()

Browse Source 
Replace combination of malloc()+memset() with calloc() as the behavior
is exactly the same and the amount of code is reduced. Moreover, remove
printf() in the fail path, as it is useless, and return proper -ENOMEM
return code.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Eugen Hristev <eugen.hristev@microchip.com>
Cc: Joe Hershberger <joe.hershberger@ni.com>
This commit is contained in:
Marek Vasut
2020-03-25 16:52:38 +01:00
parent 2fa581ba91
commit e3b54cda68
1 changed files with 3 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
drivers/net/ks8851_mll.c
View File

@@ -604,12 +604,9 @@ int ks8851_mll_initialize(u8 dev_num, int base_addr)
{ {
struct eth_device *dev; struct eth_device *dev;
dev = malloc(sizeof(*dev)); dev = calloc(1, sizeof(*dev));
if (!dev) { if (!dev)
printf("Error: Failed to allocate memory\n"); return -ENOMEM;
return -1;
}
memset(dev, 0, sizeof(*dev));
dev->iobase = base_addr; dev->iobase = base_addr;