[PATCH 2/4] arm: Move asm/utils.h to log2.h

Simon Glass sjg at chromium.org
Mon Feb 3 16:31:02 CET 2020

Hi Sean,

On Mon, 3 Feb 2020 at 08:27, Sean Anderson <seanga2 at gmail.com> wrote:
> On 2/3/20 10:11 AM, Simon Glass wrote:
> > Hi Marek,
> >
> > On Mon, 3 Feb 2020 at 01:28, Marek Vasut <marex at denx.de> wrote:
> >> How could this be reviewed if it's missing basic things, like a SoB line ?
> I added that in a reply to the original patch; again my apologies for
> missing it the first time around.
> >
> > I saw you requested that on another patch. Apart from that it looks OK to me.
> >
> > People should use patman :-)
> I usually add my signed-off-by lines with git rebase --sign, but I
> forgot to do that this time. Though reasonably the thing which should
> have reminded me was running checkpatch (which I also forgot to run).

That's why patman exists, to automate adding sign-off, checkpath, get
maintainers, dropping unwanted tags, cover letter, etc.

It's so simple to set up. It really doesn't make much sense to send
patches manually, IMO.


More information about the U-Boot mailing list