linux-kernel - Re: [PATCH 1/5] Staging: bcm: DDRInit: Fixed coding style issue, replaced spaces w/ tabs.(patch set)

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]

Message-ID: <20131205085736.GY28413@mwanda>
Date:	Thu, 5 Dec 2013 11:57:36 +0300
From:	Dan Carpenter <dan.carpenter@...cle.com>
To:	Gary Rookard <garyrookard@...il.com>
Cc:	gregkh@...uxfoundation.org, devel@...verdev.osuosl.org,
	lisa@...apiadmin.com, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/5] Staging: bcm: DDRInit: Fixed coding style issue,
 replaced spaces w/ tabs.(patch set)

These five patches all do the same thing.  It's ok to merge them into
one patch.

The subject lines are all the same and that's not ok.  The subject lines
are too long as well.

On Wed, Dec 04, 2013 at 07:26:19PM -0500, Gary Rookard wrote:
> This is the first patch of a series.

Don't put this in the description.

> Replaced spaces in margin w/ 1 tab for lines:
>  11-15, 17-23, 25-58, 60, 62, 64
> 

No signed off by line.

>  On branch staging-next

Put this between the --- and the diffstat.
> ---

<-- here.

>  drivers/staging/bcm/DDRInit.c | 98 +++++++++++++++++++++----------------------
>  1 file changed, 49 insertions(+), 49 deletions(-)

regards,
dan carpenter
--
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/