linux-kernel - Re: [PATCH] Staging: fsl-mc/bus: mc-bus.c: Fixed coding style issues

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Fri, 04 Sep 2015 11:33:36 -0700
From:	Joe Perches <joe@...ches.com>
To:	"gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>
Cc:	Jose Rivera <German.Rivera@...escale.com>,
	Nayeemahmed Badebade <itachi.opsrc@...il.com>,
	Stuart Yoder <stuart.yoder@...escale.com>,
	"agraf@...e.de" <agraf@...e.de>,
	"hgujulan@...teon.com" <hgujulan@...teon.com>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] Staging: fsl-mc/bus: mc-bus.c: Fixed coding style issues

On Fri, 2015-09-04 at 11:00 -0700, gregkh@...uxfoundation.org wrote:
> On Fri, Sep 04, 2015 at 05:48:55PM +0000, Jose Rivera wrote:
> > > -----Original Message-----
> > > From: Nayeemahmed Badebade [mailto:itachi.opsrc@...il.com]
[]
> > > Fixed coding style issues where kernel types u16,u64,u32 should be
> > > preferred over uint16_t,uint64_t,uint32_t
[]
> > Thanks for your interests in helping us clean up this coding style issue.
> > However, doing this for all the files related to the MC bus driver (not just
> > for one) is something that I intend to do in a future patch as
> > part of the "Cleanup" item of our TODO list.
> > Doing this clean up across the board for all files is preferable than 
> > doing it just for this file in this patch.
> 
> Sorry, but we never postpone a change submitted by someone just because
> someone says "sometime in the future we will be making much the same
> change".  That's a sure way to kill productivity and stop developers
> from helping you out.
> 
> If you really wanted to make this type of change, you would have done so
> already, so obviously it's not a high priority.  I'll take Nayeemahmed's
> patch.

There's a problem with doing this on a file-by-file
basis for a particular subsystem/driver.

Doing so can break compilation because of differences
between header files and compilation units.

It's likely much better to do something like:

$ git ls-files -- drivers/staging/fsl-mc/*.[ch] | \
  xargs sed -r -i \
	-e 's/\buint(8|16|32|64)_t\b/u\1/g' \
	-e 's/\bint(8|16|32|64)_t\b/s\1/g'

then verify, compile, test, and submit the diff.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/