linux-kernel - Re: [PATCH v13 2/3] power-domain: rockchip: add power domain driver

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: <55351064.7090409@rock-chips.com>
Date:	Mon, 20 Apr 2015 22:42:44 +0800
From:	Caesar Wang <wxt@...k-chips.com>
To:	Paul Bolle <pebolle@...cali.nl>
CC:	heiko@...ech.de, khilman@...aro.org,
	linux-arm-kernel@...ts.infradead.org, tomasz.figa@...il.com,
	mark.rutland@....com, linux-kernel@...r.kernel.org,
	devicetree@...r.kernel.org, linux-doc@...r.kernel.org,
	linux-rockchip@...ts.infradead.org, dianders@...omium.org,
	ulf.hansson@...aro.org, dmitry.torokhov@...il.com,
	broonie@...nel.org, grant.likely@...aro.org,
	"jinkun.hong" <jinkun.hong@...k-chips.com>
Subject: Re: [PATCH v13 2/3] power-domain: rockchip: add power domain driver

Paul,

Thanks your commnets!:-)

在 2015年04月20日 17:26, Paul Bolle 写道:
> On Sun, 2015-04-19 at 18:44 +0800, Caesar Wang wrote:
>> --- a/arch/arm/mach-rockchip/Makefile
>> +++ b/arch/arm/mach-rockchip/Makefile
> +obj-$(CONFIG_PM_GENERIC_DOMAINS) += pm_domains.o
>
> PM_GENERIC_DOMAINS is a bool symbol, so pm_domains.o can never be part
> of a module. Is that correct?

Yeah, So this platform_driver does not need to set an owner, it will be 
populated by the
driver core.
>
>> --- /dev/null
>> +++ b/arch/arm/mach-rockchip/pm_domains.c
>> +#include <linux/module.h>
> This include could probably be dropped.

Yeah. As the about said.
>> +static const struct of_device_id rockchip_pm_domain_dt_match[] = {
>> +	{
>> +		.compatible = "rockchip,rk3288-power-controller",
>> +		.data = (void *)&rk3288_pmu,
>> +	},
>> +	{ /* sentinel */ },
>> +};
>> +MODULE_DEVICE_TABLE(of, rockchip_pm_domain_dt_match);
> The MODULE_DEVICE_TABLE() macro will be preprocessed away for built-in
> code, according to include/linux/module.h.

So,do you think we should remove it?
wow.....,looks we should remove it.
>> +static struct platform_driver rockchip_pm_domain_driver = {
>> +	.probe = rockchip_pm_domain_probe,
>> +	.driver = {
>> +		.name   = "rockchip-pm-domain",
>> +		.owner  = THIS_MODULE,
> According to include/linux/export.h THIS_MODULE is equivalent to NULL
> for built-in code. So I think this line is not needed.
>

Yeah,you are right.
>> +		.of_match_table = rockchip_pm_domain_dt_match,
>> +		/*
>> +		 * We can't forcibly eject devices form power domain,
>> +		 * so we can't really remove power domains once they
>> +		 * were added.
>> +		 */
>> +		.suppress_bind_attrs = true,
>> +	},
>> +};
> Thanks,
>
>
> Paul Bolle
>
>
>
>
Thanks,

Caesar

-- 
**************************************************************************************
王晓腾    Caesar Wang
Product R&D Dept.III
Fuzhou Rockchip Electronics Co.Ltd
Addr：  NO.18 Building, A District, Fuzhou Software Park,Gulou District,Fuzhou, Fujian,China(Fuzhou Headquarters)
               21F,Malata Building,Kejizhongyi Avenue,Nanshan District,Shenzhen  (Shenzhen Office)
Tel：+86-591-83991906/07 - 8221
Mobile:+86 15059456742
E-mail : wxt@...k-chips.com
***************************************************************************************
***************************************************************************************
IMPORTANT NOTICE: This email is from Fuzhou Rockchip Electronics Co., Ltd .The contents of this email and any attachments may
contain information that is privileged, confidential and/or exempt from disclosure under applicable law and relevant NDA.
If you are not the intended recipient, you are hereby notified that any disclosure, copying, distribution, or use of the
information is STRICTLY PROHIBITED. Please immediately contact the sender as soon as possible and destroy the material
in its entirety in any format. Thank you.
***************************************************************************************


--
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/