Messages in this thread | | | Date | Mon, 3 Nov 2014 12:29:48 +0000 | From | Mark Brown <> | Subject | Re: [PATCH 2/2] regulator: of: Add support for parsing microvolts for suspend state |
| |
On Sat, Nov 01, 2014 at 09:49:42AM +0100, Javier Martinez Canillas wrote: > On 11/01/2014 04:52 AM, Doug Anderson wrote:
> > + > > of_node_put(suspend_np); > > suspend_state = NULL; > > suspend_np = NULL;
> Looks good to me.
> Reviewed-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk>
If you're reviewing something please don't quote the entire patch, the same rule applies as always - just include relevant context - and especially don't include your one line of content at the very end of the patch. Otherwise people have to page through the entire thing to find what you've said (if they don't get fed up first). [unhandled content-type:application/pgp-signature] | |