kernelnewbies.kernelnewbies.org archive mirror
 help / color / mirror / Atom feed
From: premdas.prajosh@gmail.com (Prajosh Premdas)
To: kernelnewbies@lists.kernelnewbies.org
Subject: Tool for automatic style of coding guidelines
Date: Mon, 30 Jan 2012 20:36:18 +0530	[thread overview]
Message-ID: <CAAFf3B9zGnCKr15pA_jOnPav1aKEd9GUDpA5jnmd5gGU_Vm-MQ@mail.gmail.com> (raw)
In-Reply-To: <CANBXnMmeaJhDPWA4jTBx0HpjAekBJBJN+Dr3eNj5qX4xSJH9cQ@mail.gmail.com>

Thanks for the suggestions and video links

Got the patches clean for review

On Sun, Jan 29, 2012 at 12:05 AM, Adil Mujeeb <mujeeb.adil@gmail.com> wrote:

> On Fri, Jan 27, 2012 at 11:45 PM, Greg KH <greg@kroah.com> wrote:
> > On Fri, Jan 27, 2012 at 05:50:30PM +0530, Prajosh Premdas wrote:
> >> Hi
> >>
> >> I have developed some kernel drivers and would like to send the patches
> for
> >> review. But i find that I have not followed the coding guideline for
> tabs and
> >> spaces properly, which can be easily corrected by using some tools like
> >> astyle.
> >>
> >> Can any body please help me with the proper commands string for the
> astyle
> >> tool? Or could any body please suggest me with any new tool like indent
> etc
> >> which you are using
> >
> > scripts/Lindent will give you a good start, but odds are you really want
> > to do this by hand, it will be easier in the end as indent can do some
> > strange things at times.
> >
> > good luck,
> >
> > greg k-h
>
> You can see this excellent video [1] where Greg has explained how to
> submit your first patch.
>
> [1] http://www.youtube.com/watch?v=LLBrBBImJt4
>
> Regards,
> Adil
>



-- 
Regards,

Prajosh Premdas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.kernelnewbies.org/pipermail/kernelnewbies/attachments/20120130/c2bde284/attachment.html 

      reply	other threads:[~2012-01-30 15:06 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-27 12:20 Tool for automatic style of coding guidelines Prajosh Premdas
2012-01-27 14:18 ` Philipp Ittershagen
2012-01-27 18:55   ` Greg Freemyer
2012-01-28 11:40     ` Bernd Petrovitsch
2012-01-27 18:15 ` Greg KH
2012-01-28 18:35   ` Adil Mujeeb
2012-01-30 15:06     ` Prajosh Premdas [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAAFf3B9zGnCKr15pA_jOnPav1aKEd9GUDpA5jnmd5gGU_Vm-MQ@mail.gmail.com \
    --to=premdas.prajosh@gmail.com \
    --cc=kernelnewbies@lists.kernelnewbies.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).