From mboxrd@z Thu Jan  1 00:00:00 1970
From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Date: Fri, 3 Apr 2020 13:56:55 +0300
Subject: [PATCH] serial: ns16550: Fix ordering of getting base address
In-Reply-To: <CAEUhbmXaaUrMn=cDQDcP+DnOr-49h1B-m34u1VU4RkZgDKonuA@mail.gmail.com>
References: <1585907888-12554-1-git-send-email-bmeng.cn@gmail.com>
 <20200403100556.GM1922688@smile.fi.intel.com>
 <CAEUhbmXaaUrMn=cDQDcP+DnOr-49h1B-m34u1VU4RkZgDKonuA@mail.gmail.com>
Message-ID: <20200403105655.GN1922688@smile.fi.intel.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Fri, Apr 03, 2020 at 06:16:51PM +0800, Bin Meng wrote:
> On Fri, Apr 3, 2020 at 6:05 PM Andy Shevchenko
> <andriy.shevchenko@linux.intel.com> wrote:
> > On Fri, Apr 03, 2020 at 02:58:08AM -0700, Bin Meng wrote:

...

> > > Fixes: 720f9e1fdb0c9 ("serial: ns16550: Move PCI access from
> > > ofdata_to_platdata() to probe()")
> >
> > I believe this should be one line.
> 
> I thought putting it in one line violates the 80 characters per column
> rule. And checkpatch.pl does not complain about this, so I think it's
> fine :)

Breaking tags to two or more lines actually breaks the scripts around this stuff.

JFYI, in Linux kernel it was now in documentation (not to break Fixes, etc to
two or more lines).

-- 
With Best Regards,
Andy Shevchenko