[PATCH v3] Documentation: checkpatch: Add SYMBOLIC_PERMS message
dwaipayanray1 at gmail.com
Fri Aug 6 19:56:36 UTC 2021
On Fri, Aug 6, 2021 at 7:16 PM Utkarsh Verma <utkarshverma294 at gmail.com> wrote:
> On Thu, Aug 05, 2021 at 07:21:50PM +0530, Dwaipayan Ray wrote:
> > On Wed, Aug 4, 2021 at 9:02 PM Utkarsh Verma <utkarshverma294 at gmail.com> wrote:
> > >
> > > Add a new message type SYMBOLIC_PERMS under the 'Permissions'
> > > subsection. Checkpatch documentation that recommends the user to use
> > Did you mean: Checkpatch recommends the user to use octal
> > permission bits...?
> This patch adds documentation that explains to the user why octal
> permission should be used. So in the commit description, which explains
> my patch I wrote, "Checkpatch documentation that recommends the user ...."
> If it's sounding vague, then should I change the commit description to:
> "Add a new message type SYMBOLIC_PERMS under the 'Permissions'
> subsection. Users should use octal permission bits instead of their
> symbolic macro names."
I think it reads better that way. Please make the change.
More information about the Linux-kernel-mentees