Re: [PATCH] Changed DPRINTK to pr_debug in /drivers/macintosh/smu.c

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Dec 25, 2006 at 05:11:35PM +1000, Badai Aqrandista wrote:
> I chose this point in the kj todo list:
> 
> - pr_debug() from kernel.h could replace a lot of DPRINTK and similar
> macros.
> Attached is my first patch to implement that point in
> /drivers/macintosh/smu.c.

Patch looks good to me.

> I've few questions here:
> 
> Can I just send it to kernel-janitors@xxxxxxxxxxxxxx to be discussed there?

Yeah, but your mail needs some extra work:

- Do not EVER use HTML messages or your message *will* be ignored.
- Please inline patches to make it easy to comment on them. Only if
  your mailer mangles patches attach the patch (preferably as plain
  text, not base64 encoded).
- Add a Signed-off-by line so we can trace who submitted the patch.

See Documentation/SubmittingPatches for more information.

> Should I make a lot of changes and then post it in kernel janitor list as a
> patch?

One change per patch is the prefered way. One huge patch with lots of
changes is too much to review.

> Should I make my git available for kernel janitor maintainer to pull from?

You could do both but patches always need to go through the appropriate
mailinglist/maintainer by mail (unless your name is "Linus Torvalds").


Erik

-- 
They're all fools. Don't worry. Darwin may be slow, but he'll
eventually get them. -- Matthew Lammers in alt.sysadmin.recovery

--
Kernelnewbies: Help each other learn about the Linux kernel.
Archive:       http://mail.nl.linux.org/kernelnewbies/
FAQ:           http://kernelnewbies.org/faq/


[Index of Archives]     [Newbies FAQ]     [Linux Kernel Mentors]     [Linux Kernel Development]     [IETF Annouce]     [Git]     [Networking]     [Security]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux SCSI]     [Linux ACPI]
  Powered by Linux