Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Sun, 21 Aug 2022 10:35:23 +0200
From:   Greg KH <gregkh@linuxfoundation.org>
To:     Paul Lemmermann <thepaulodoom@thepaulodoom.com>
Cc:     arnd@arndb.de, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] drivers: char: mem: changed 'unsigned' to 'unsigned int'
Message-ID: <YwHuSyur7AQ7PUsC@kroah.com>
References: <YvxF0yn07ztg9r4A@gus-fring.localdomain>
 <Yv5SG8XMR2mZGAqG@kroah.com>
 <Yv7s5dedhRaDlsAC@gus-fring.localdomain>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <Yv7s5dedhRaDlsAC@gus-fring.localdomain>
Precedence: bulk

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