From: Bjorn Helgaas <helgaas@kernel.org>
To: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
Cc: "Lorenzo Pieralisi" <lorenzo.pieralisi@arm.com>,
linuxarm@huawei.com, mauro.chehab@huawei.com,
"Songxiaowei (Kirin_DRV)" <songxiaowei@hisilicon.com>,
"Krzysztof Wilczyński" <kw@linux.com>,
"Alex Dewar" <alex.dewar90@gmail.com>,
"Andy Shevchenko" <andriy.shevchenko@linux.intel.com>,
"Binghui Wang" <wangbinghui@hisilicon.com>,
"Bjorn Helgaas" <bhelgaas@google.com>,
"Jaehoon Chung" <jh80.chung@samsung.com>,
"Jingoo Han" <jingoohan1@gmail.com>,
"Luca Ceresoli" <luca@lucaceresoli.net>,
"Manivannan Sadhasivam" <mani@kernel.org>,
"Nobuhiro Iwamatsu" <nobuhiro1.iwamatsu@toshiba.co.jp>,
"Paul Walmsley" <paul.walmsley@sifive.com>,
"Rob Herring" <robh@kernel.org>, "Simon Xue" <xxm@rock-chips.com>,
"Srikanth Thokala" <srikanth.thokala@intel.com>,
"Wesley Sheng" <wesley.sheng@amd.com>,
linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org
Subject: Re: [PATCH v14 00/11] Add support for Hikey 970 PCIe
Date: Wed, 20 Oct 2021 14:02:14 -0500 [thread overview]
Message-ID: <20211020190214.GA2630871@bhelgaas> (raw)
In-Reply-To: <20211020064142.0807ae70@sal.lan>
On Wed, Oct 20, 2021 at 06:41:42AM +0100, Mauro Carvalho Chehab wrote:
> Em Tue, 19 Oct 2021 14:27:58 -0500
> Bjorn Helgaas <helgaas@kernel.org> escreveu:
>
> > On Tue, Oct 19, 2021 at 07:06:37AM +0100, Mauro Carvalho Chehab wrote:
> >
> > > Mauro Carvalho Chehab (11):
> > > PCI: kirin: Reorganize the PHY logic inside the driver
> > > PCI: kirin: Add support for a PHY layer
> > > PCI: kirin: Use regmap for APB registers
> > > PCI: kirin: Add support for bridge slot DT schema
> > > PCI: kirin: give more time for PERST# reset to finish
> > > PCI: kirin: Add Kirin 970 compatible
> > > PCI: kirin: Add MODULE_* macros
> > > PCI: kirin: Allow building it as a module
> > > PCI: kirin: Add power_off support for Kirin 960 PHY
> > > PCI: kirin: fix poweroff sequence
> > > PCI: kirin: Allow removing the driver
> >
> > Don't repost for this, but if you have occasion to repost for other
> > reasons, two of these are not capitalized like the others:
> >
> > > PCI: kirin: give more time for PERST# reset to finish
> > > PCI: kirin: fix poweroff sequence
> >
> > These are write-once for you, but I'll be reading them many times in
> > the future and they're minor distractions.
>
> Ok, changed on my working repository. On media and on other subsystems
> I contribute regularly, the practice is just the opposite: to use
> lowercase after colons.
Thanks. I wouldn't mind a kernel-wide convention, but there isn't
one. I just try to pay attention to whatever convention there is for
a given file or subsystem.
Bjorn
prev parent reply other threads:[~2021-10-20 19:02 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-10-19 6:06 [PATCH v14 00/11] Add support for Hikey 970 PCIe Mauro Carvalho Chehab
2021-10-19 6:06 ` [PATCH v14 01/11] PCI: kirin: Reorganize the PHY logic inside the driver Mauro Carvalho Chehab
2021-10-19 6:06 ` [PATCH v14 02/11] PCI: kirin: Add support for a PHY layer Mauro Carvalho Chehab
2021-10-19 6:06 ` [PATCH v14 03/11] PCI: kirin: Use regmap for APB registers Mauro Carvalho Chehab
2021-10-19 6:06 ` [PATCH v14 04/11] PCI: kirin: Add support for bridge slot DT schema Mauro Carvalho Chehab
2022-05-24 17:19 ` Bjorn Helgaas
2022-05-24 18:59 ` Bjorn Helgaas
2022-05-24 19:55 ` Mauro Carvalho Chehab
2022-05-24 21:29 ` Bjorn Helgaas
2021-10-19 6:06 ` [PATCH v14 05/11] PCI: kirin: give more time for PERST# reset to finish Mauro Carvalho Chehab
2021-10-21 12:27 ` Lorenzo Pieralisi
2021-10-21 12:40 ` Mauro Carvalho Chehab
2021-10-22 15:16 ` Pali Rohár
2021-10-23 9:30 ` Mauro Carvalho Chehab
2021-10-23 10:40 ` Pali Rohár
2021-10-23 13:45 ` Mauro Carvalho Chehab
2021-10-23 14:55 ` Pali Rohár
2021-10-25 10:25 ` Lorenzo Pieralisi
2021-10-25 10:40 ` Mauro Carvalho Chehab
2021-10-26 17:06 ` Lorenzo Pieralisi
2021-10-19 6:06 ` [PATCH v14 06/11] PCI: kirin: Add Kirin 970 compatible Mauro Carvalho Chehab
2021-10-19 6:06 ` [PATCH v14 07/11] PCI: kirin: Add MODULE_* macros Mauro Carvalho Chehab
2021-10-19 6:06 ` [PATCH v14 08/11] PCI: kirin: Allow building it as a module Mauro Carvalho Chehab
2021-10-19 6:06 ` [PATCH v14 09/11] PCI: kirin: Add power_off support for Kirin 960 PHY Mauro Carvalho Chehab
2021-10-19 6:06 ` [PATCH v14 10/11] PCI: kirin: fix poweroff sequence Mauro Carvalho Chehab
2021-10-19 6:06 ` [PATCH v14 11/11] PCI: kirin: Allow removing the driver Mauro Carvalho Chehab
2021-10-19 19:27 ` [PATCH v14 00/11] Add support for Hikey 970 PCIe Bjorn Helgaas
2021-10-20 5:41 ` Mauro Carvalho Chehab
2021-10-20 19:02 ` Bjorn Helgaas [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20211020190214.GA2630871@bhelgaas \
--to=helgaas@kernel.org \
--cc=alex.dewar90@gmail.com \
--cc=andriy.shevchenko@linux.intel.com \
--cc=bhelgaas@google.com \
--cc=jh80.chung@samsung.com \
--cc=jingoohan1@gmail.com \
--cc=kw@linux.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=linuxarm@huawei.com \
--cc=lorenzo.pieralisi@arm.com \
--cc=luca@lucaceresoli.net \
--cc=mani@kernel.org \
--cc=mauro.chehab@huawei.com \
--cc=mchehab+huawei@kernel.org \
--cc=nobuhiro1.iwamatsu@toshiba.co.jp \
--cc=paul.walmsley@sifive.com \
--cc=robh@kernel.org \
--cc=songxiaowei@hisilicon.com \
--cc=srikanth.thokala@intel.com \
--cc=wangbinghui@hisilicon.com \
--cc=wesley.sheng@amd.com \
--cc=xxm@rock-chips.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).