Re: [PATCH] ppc: vio of_node_put cleanup

The comment used to be inside the "if" block, is this
change correct?

You'd prefer an empty line in there?

Obviously, you should change the comment to include the
conditional, if that is what is needed.

[And, do we want all these changes anyway?  I don't care
either way, both sides have their pros and their cons --
just asking :-) ]

You know my opinion already :-)

Heh.  Ok, I'll rephrase: is there _consensus_ that this is a
good thing :-)  [But never mind, I looked it up, and it is
*documented* as being supported, so fine with me].


Segher

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• References:
  • [PATCH] ppc: vio of_node_put cleanup
    • From: Mariusz Kozlowski <[email protected]>
  • Re: [PATCH] ppc: vio of_node_put cleanup
    • From: Segher Boessenkool <[email protected]>
  • Re: [PATCH] ppc: vio of_node_put cleanup
    • From: Mariusz Kozlowski <[email protected]>

• Prev by Date: Re: fuse, get_user_pages, flush_anon_page, aliasing caches and all that again
• Next by Date: Re: [parisc-linux] [RFC][PATCH] use cycle_t instead of u64 in struct
• Previous by thread: Re: [PATCH] ppc: vio of_node_put cleanup
• Next by thread: [PATCH] video: pvrusb2-hdw kfree cleanup
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]