Re: [PATCH 1/3] PM / OPP: rename functions to dev_pm_opp*

From: Viresh Kumar
Date: Fri Sep 20 2013 - 12:51:16 EST


On 20 September 2013 18:08, Nishanth Menon <nm@xxxxxx> wrote:

> I am open to suggestions if any one feels we can improve this better.

I didn't really had one.. I thought of pm_opp** instead of dev_pm_opp**
though..

> I believe that change was from Patch #2[1]

Yeah.. I just replied on a single patch :)

> yes, you are right, I had squashed this patch in to squelch checkpatch
> warnings:

I see..

> I had added a comment:
> " Minor checkpatch warning fixes as a result of this change was fixed as
> well."

I really missed that..

> Would you suggest I split the change off to a separate patch or improve
> the comment a little more?

don't really know, maybe leave those as is and let checkpatch warn you..
--
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/