lkml.org 
[lkml]   [2022]   [Aug]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] drivers: char: mem: changed 'unsigned' to 'unsigned int'
On Thu, Aug 18, 2022 at 08:52:37PM -0500, Paul Lemmermann wrote:
> On Thu, Aug 18, 2022 at 04:52:11PM +0200, Greg KH wrote:
> > On Tue, Aug 16, 2022 at 08:35:31PM -0500, Paul Lemmermann wrote:
> > > found with checkpatch.pl
> >
> > That is a very sparse changelog text.
> >
> > Why make this change? Please be descriptive.
> >
> I wanted to make this change to be more fitting with the kernel coding
> guidelines, and to reduce a WARNING made with checkpatch preferring the
> use of 'unsigned int' over 'unsigned'. With NOMMU_MAP_COPY defined in
> 'include/linux/fs.h' as an unsigned integer, I felt like the change was
> fitting. I was quickly reminded that the changelog was not properly done,
> and I now understand that, and will learn from my mistake in the future.
> As I am a new kernel contributor, and looking to hone my skills, what are
> my next steps?
>
> Should I:
>
> o Drop the patch and resubmit it,
> o Simply drop the patch,
> o Could I change it somehow?,
> o Or something else?

Try something else. Start out by working in drivers/staging/ on basic
coding style fixes and other things listed in the drivers/staging/*/TODO
files. That's the best place for new developers to get started and then
after you have some experience with how the development process works,
you can branch out into other subsystems doing real work.

hope this helps,

greg k-h

\
 
 \ /
  Last update: 2022-08-21 10:36    [W:0.043 / U:0.464 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site