Re: [PATCH 1/3 v6] cpufreq: Add debugfs directory for cpufreq

From: Viresh Kumar
Date: Wed Jul 24 2013 - 02:16:13 EST


On Wed, Jul 24, 2013 at 11:44 AM, Chanwoo Choi <cw00.choi@xxxxxxxxxxx> wrote:
> I considered the parameter type of cpufreq_create_debugfs_symlink() and then
> I need following function declaration because this function didn't always need to
> create symbolic link to policy->cpu. This function declaration is capable of creating
> symbolic link as 'dest_cpu -> src_cpu'
>
> +static int cpufreq_create_debugfs_symlink(unsigned int src_cpu,
> + unsigned int dest_cpu)

You need policy structure as well, right? And src_cpu is always going to
be policy->cpu. So you can have policy and cpu as two parameters.. cpu
is dest cpu btw.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/