[PATCH v10 03/10] qcom: spm: Add Subsystem Power Manager driver

Stephen Boyd sboyd at codeaurora.org
Wed Nov 26 12:00:36 PST 2014


On 11/21/2014 10:03 AM, Lina Iyer wrote:
> +
> +static const struct platform_device_info  qcom_cpuidle_info = {

nitpick, why two spaces between info and qcom here?

> +	.name	= "qcom_cpuidle",
> +	.id	= -1,
> +	.data = &lpm_ops,
> +	.size_data = sizeof(lpm_ops),
> +};

This struct could go on the stack too so that it doesn't lay around forever.

>
> +
> +MODULE_LICENSE("GPL v2");
> +MODULE_DESCRIPTION("SAW power controller driver");
> +MODULE_ALIAS("saw");

"platform:saw" please. Namespacing aliases is good.

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project




More information about the linux-arm-kernel mailing list