On Thu, Nov 6, 2008 at 7:06 PM, Stefan Reinauer <span dir="ltr"><<a href="mailto:stepan@coresystems.de">stepan@coresystems.de</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
ron minnich schrieb:<div><div></div><div class="Wj3C7c"><br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
On Thu, Nov 6, 2008 at 2:58 PM, Carl-Daniel Hailfinger<br>
<<a href="mailto:c-d.hailfinger.devel.2006@gmx.net" target="_blank">c-d.hailfinger.devel.2006@gmx.net</a>> wrote:<br>
  <br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
On 25.10.2008 13:05, Uwe Hermann wrote:<br>
    <br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
On Fri, Oct 24, 2008 at 03:31:20AM +0200, Peter Stuge wrote:<br>
<br>
      <br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Uwe Hermann wrote:<br>
<br>
        <br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Drop the 'BIOS_' prefix from all printk() log-levels.<br>
<br>
          <br>
</blockquote>
It would be nice to have another prefix though. Maybe LOG_ ? Without<br>
a prefix some of the names look a bit too generic.<br>
<br>
        <br>
</blockquote>
I've though about that, but I don't see it as a big problem. Only<br>
"DEBUG" tends to be misused in v2, but we can easily avoid that by<br>
using DEBUG_SMBUS, DEBUG_FOOBAR #defines in v3 (if at all; all of<br>
those custom DEBUG #defines should be printk()-loglevels anyway;<br>
I think the reason for DEBUG in v2 is usually romcc/size relåted(?))<br>
<br>
If we all desperately want a prefix, how about something really short,<br>
e.g. LDEBUG, LWARN, LINFO, ... ? I'd really like to keep the printk()<br>
lines as short as possible for readability reasons.<br>
<br>
      <br>
</blockquote>
Can't we just keep the names as they are?<br>
<br>
    <br>
</blockquote>
<br>
<br>
I think the majority prefer that they not change ...<br>
  <br>
</blockquote></div></div>
Add me to that majority. I think those names are good as they are.<br>
<br>
Please don't change such stuff just for the sake of it.<br>
<br>
a) it really does not matter<br>
b) don't make it longer than it is<br>
c) keep it comprehensible.</blockquote><div><br>I'm agreeing here too. changing it breaks most outstanding patches, screws up everyone that's used to BIOS_X, and I can't really see any benefit to a different name, aside from less typing, but that's what copy and paste are for ;)<br>
<br>-Corey <br></div></div><br>