linux-kernel - Re: [PATCH] progear_bl: use platform_device_register

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <200811281017.17809.marcin@haerwu.biz>
Date:	Fri, 28 Nov 2008 10:17:16 +0100
From:	Marcin Juszkiewicz <marcin@...rwu.biz>
To:	Akinobu Mita <akinobu.mita@...il.com>,
	Richard Purdie <rpurdie@...ys.net>
Cc:	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] progear_bl: use platform_device_register_simple()

Sunday 21 of September 2008 16:38:29 Akinobu Mita napisał(a):
> This change also fixes error handling when platform_device_alloc()
> fails. (When platform_device_alloc() failed, it returns error without
> unregistering progearbl_driver)
>
> Signed-off-by: Akinobu Mita <akinobu.mita@...il.com>
> Cc: Marcin Juszkiewicz <linux@....one.pl>
> Cc: Richard Purdie <rpurdie@...ys.net>

Acked-by: Marcin Juszkiewicz <marcin@...rwu.biz>

> ---
>  drivers/video/backlight/progear_bl.c |   20 ++++++++------------
>  1 file changed, 8 insertions(+), 12 deletions(-)
>
> Index: 2.6-git/drivers/video/backlight/progear_bl.c
> ===================================================================
> --- 2.6-git.orig/drivers/video/backlight/progear_bl.c
> +++ 2.6-git/drivers/video/backlight/progear_bl.c
> @@ -119,20 +119,16 @@ static int __init progearbl_init(void)
>  {
>  	int ret = platform_driver_register(&progearbl_driver);
>
> -	if (!ret) {
> -		progearbl_device = platform_device_alloc("progear-bl", -1);
> -		if (!progearbl_device)
> -			return -ENOMEM;
> -
> -		ret = platform_device_add(progearbl_device);
> -
> -		if (ret) {
> -			platform_device_put(progearbl_device);
> -			platform_driver_unregister(&progearbl_driver);
> -		}
> +	if (ret)
> +		return ret;
> +	progearbl_device = platform_device_register_simple("progear-bl",
> -1, +								NULL, 0);
> +	if (IS_ERR(progearbl_device)) {
> +		platform_driver_unregister(&progearbl_driver);
> +		return PTR_ERR(progearbl_device);
>  	}
>
> -	return ret;
> +	return 0;
>  }
>
>  static void __exit progearbl_exit(void)


Regards, 
-- 
JID:      hrw@...ber.org
Website:  http://blog.haerwu.biz/
LinkedIn: http://www.linkedin.com/in/marcinjuszkiewicz


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/