Re: [patch 2.6.29-rc] regulator: add get_status()

From: Mark Brown
Date: Thu Jan 15 2009 - 11:05:40 EST


On Thu, Jan 15, 2009 at 07:35:29AM -0800, David Brownell wrote:
> On Thursday 15 January 2009, Liam Girdwood wrote:

> > Please send a separate patch for the kerneldoc as I've already applied.

> So that will be the first member of "struct regulator_ops" to
> grow kerneldoc ... out of a total of fifteen (now) members.
> Hmm...

> Maybe I'm not understanding what's meant by "kerneldoc adding".
> Perhaps it's nothing more than a sentence resembling "returns
> REGULATOR_STATUS_* code or negative errno", and not real
> kerneldoc?

No, real kerneldoc - this was added recently, check current mainline
(eg, commit c8e7e4640facbe99d10a6e262523b25be129b9b9). I'm not saying
it's great but it's there.

--
"You grabbed my hand and we fell into it, like a daydream - or a fever."
--
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/