From: John Garry <john.garry@huawei.com>
To: Bjorn Helgaas <helgaas@kernel.org>
Cc: "mika.westerberg@linux.intel.com"
<mika.westerberg@linux.intel.com>,
"rafael@kernel.org" <rafael@kernel.org>,
"lorenzo.pieralisi@arm.com" <lorenzo.pieralisi@arm.com>,
"rjw@rjwysocki.net" <rjw@rjwysocki.net>,
"hanjun.guo@linaro.org" <hanjun.guo@linaro.org>,
"robh+dt@kernel.org" <robh+dt@kernel.org>,
"bhelgaas@google.com" <bhelgaas@google.com>,
"arnd@arndb.de" <arnd@arndb.de>,
"mark.rutland@arm.com" <mark.rutland@arm.com>,
"olof@lixom.net" <olof@lixom.net>,
"dann.frazier@canonical.com" <dann.frazier@canonical.com>,
"andy.shevchenko@gmail.com" <andy.shevchenko@gmail.com>,
"robh@kernel.org" <robh@kernel.org>,
"andriy.shevchenko@linux.intel.com"
<andriy.shevchenko@linux.intel.com>,
"joe@perches.com" <joe@perches.com>,
"benh@kernel.crashing.org" <benh@kernel.crashing.org>,
"linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>,
Linuxarm <linuxarm@huawei.com>,
"minyard@acm.org" <minyard@acm.org>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"linux-arch@vger.kernel.org" <linux-arch@vger.kernel.org>,
"rdunlap@infradead.org" <rdunlap@infradead.org>,
"gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
"akpm@linux-foundation.org" <akpm@linux-foundation.org>,
"frowand.list@gmail.com" <frowand.list@gmail.com>,
"agraf@suse.de" <agraf@suse.de>
Subject: Re: [PATCH v17 00/10] LPC: legacy ISA I/O support
Date: Thu, 22 Mar 2018 10:38:37 +0000 [thread overview]
Message-ID: <334da3b2-5233-5ad3-728e-7ae95231bf01@huawei.com> (raw)
In-Reply-To: <20180321233940.GI38649@bhelgaas-glaptop.roam.corp.google.com>
On 21/03/2018 23:39, Bjorn Helgaas wrote:
> On Thu, Mar 15, 2018 at 02:15:49AM +0800, John Garry wrote:
>> This patchset supports the IPMI-bt device attached to the Low-Pin-Count
>> interface implemented on Hisilicon Hip06/Hip07 SoC.
>> -----------
>> | LPC host|
>> | |
>> -----------
>> |
>> _____________V_______________LPC
>> | |
>> V V
>> ------------
>> | BT(ipmi)|
>> ------------
>> ...
>
>> Gabriele Paoloni (2):
>> PCI: Remove unused __weak attribute in pci_register_io_range()
>> PCI: Add fwnode handler as input param of pci_register_io_range()
>>
>> John Garry (4):
>> ACPI / scan: rename acpi_is_serial_bus_slave() to widen use
>> ACPI / scan: do not enumerate Indirect IO host children
>> HISI LPC: Add ACPI support
>> MAINTAINERS: Add maintainer for HiSilicon LPC driver
>>
>> Zhichang Yuan (4):
>> LIB: Introduce a generic PIO mapping method
>> PCI: Apply the new generic I/O management on PCI IO hosts
>> OF: Add missing I/O range exception for indirect-IO devices
>> HISI LPC: Support the LPC host on Hip06/Hip07 with DT bindings
>>
>> .../arm/hisilicon/hisilicon-low-pin-count.txt | 33 ++
>> MAINTAINERS | 7 +
>> drivers/acpi/pci_root.c | 8 +-
>> drivers/acpi/scan.c | 33 +-
>> drivers/bus/Kconfig | 8 +
>> drivers/bus/Makefile | 2 +
>> drivers/bus/hisi_lpc.c | 623 +++++++++++++++++++++
>> drivers/of/address.c | 96 +++-
>> drivers/pci/pci.c | 95 +---
>> include/acpi/acpi_bus.h | 2 +-
>> include/asm-generic/io.h | 4 +-
>> include/linux/logic_pio.h | 124 ++++
>> include/linux/pci.h | 3 +-
>> lib/Kconfig | 15 +
>> lib/Makefile | 2 +
>> lib/logic_pio.c | 282 ++++++++++
>> 16 files changed, 1229 insertions(+), 108 deletions(-)
>> create mode 100644 Documentation/devicetree/bindings/arm/hisilicon/hisilicon-low-pin-count.txt
>> create mode 100644 drivers/bus/hisi_lpc.c
>> create mode 100644 include/linux/logic_pio.h
>> create mode 100644 lib/logic_pio.c
>
> I applied this whole series to pci/lpc for v4.17.
>
> I made the following whitespace and other trivial corrections.
> Hopefully I didn't break anything.
>
Hi Bjorn,
Super thanks for doing this. In general the changes look ok. However a
build issue has appeared, below.
I retested your pci/lpc branch (with the build fix), and it seems fine.
BTW, I am also testing with a "Serial controller: MosChip Semiconductor
Technology Ltd. PCIe 9912 Multi-I/O Controller" in loopback mode to
ensure PCI host IO space is not broken and this is ok.
Thanks again, and to everyone who has helped with this patchset!
John
>
> --- changelogs.orig 2018-03-21 18:37:47.209217927 -0500
> +++ changelogs 2018-03-21 18:37:35.993074570 -0500
> @@ -1,29 +1,31 @@
> -commit cc88cacce96a
> +commit eb3a2ff7e72e
> Author: John Garry <john.garry@huawei.com>
> Date: Thu Mar 15 02:15:59 2018 +0800
>
[ ... ]
>
> @@ -134,21 +132,20 @@ static struct logic_pio_hwaddr *find_io_range(unsigned long pio)
> * logic_pio_to_hwaddr - translate logical PIO to HW address
> * @pio: logical PIO value
> *
> - * Returns HW address if valid, ~0 otherwise
> + * Returns HW address if valid, ~0 otherwise.
> *
> - * Translate the input logical pio to the corresponding hardware address.
> - * The input pio should be unique in the whole logical PIO space.
> + * Translate the input logical PIO to the corresponding hardware address.
> + * The input PIO should be unique in the whole logical PIO space.
> */
> resource_size_t logic_pio_to_hwaddr(unsigned long pio)
> {
> struct logic_pio_hwaddr *range;
> - resource_size_t hwaddr = (resource_size_t)~0;
>
> range = find_io_range(pio);
> if (range)
> - hwaddr = range->hw_start + pio - range->io_start;
> + return = range->hw_start + pio - range->io_start;
Please remove '='
>
> - return hwaddr;
> + return (resource_size_t)~0;
> }
>
> /**
> @@ -159,15 +156,14 @@ resource_size_t logic_pio_to_hwaddr(unsigned long pio)
> *
> * Returns Logical PIO value if successful, ~0UL otherwise
> */
next prev parent reply other threads:[~2018-03-22 10:38 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-14 18:15 [PATCH v17 00/10] LPC: legacy ISA I/O support John Garry
2018-03-14 18:15 ` [PATCH v17 01/10] LIB: Introduce a generic PIO mapping method John Garry
2018-04-03 14:04 ` Thierry Reding
2018-04-03 14:39 ` Thierry Reding
2018-04-03 16:01 ` John Garry
2018-04-03 16:37 ` Thierry Reding
2018-04-03 17:02 ` John Garry
2018-04-03 17:53 ` Bjorn Helgaas
2018-04-03 18:24 ` John Garry
2018-03-14 18:15 ` [PATCH v17 02/10] PCI: Remove unused __weak attribute in pci_register_io_range() John Garry
2018-03-14 18:15 ` [PATCH v17 03/10] PCI: Add fwnode handler as input param of pci_register_io_range() John Garry
2018-03-14 18:15 ` [PATCH v17 04/10] PCI: Apply the new generic I/O management on PCI IO hosts John Garry
2018-04-03 13:45 ` Thierry Reding
2018-04-03 14:02 ` John Garry
2018-03-14 18:15 ` [PATCH v17 05/10] OF: Add missing I/O range exception for indirect-IO devices John Garry
2018-03-14 18:15 ` [PATCH v17 06/10] HISI LPC: Support the LPC host on Hip06/Hip07 with DT bindings John Garry
2018-03-14 18:15 ` [PATCH v17 07/10] ACPI / scan: rename acpi_is_serial_bus_slave() to widen use John Garry
2018-03-19 10:27 ` Rafael J. Wysocki
2018-03-14 18:15 ` [PATCH v17 08/10] ACPI / scan: do not enumerate Indirect IO host children John Garry
2018-03-19 10:30 ` Rafael J. Wysocki
2018-03-19 10:48 ` John Garry
2018-03-19 10:57 ` Rafael J. Wysocki
2018-03-19 11:13 ` John Garry
2018-03-14 18:15 ` [PATCH v17 09/10] HISI LPC: Add ACPI support John Garry
2018-03-14 18:15 ` [PATCH v17 10/10] MAINTAINERS: Add maintainer for HiSilicon LPC driver John Garry
2018-03-21 23:39 ` [PATCH v17 00/10] LPC: legacy ISA I/O support Bjorn Helgaas
2018-03-22 10:38 ` John Garry [this message]
2018-03-22 13:35 ` Bjorn Helgaas
2018-03-22 14:18 ` John Garry
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=334da3b2-5233-5ad3-728e-7ae95231bf01@huawei.com \
--to=john.garry@huawei.com \
--cc=agraf@suse.de \
--cc=akpm@linux-foundation.org \
--cc=andriy.shevchenko@linux.intel.com \
--cc=andy.shevchenko@gmail.com \
--cc=arnd@arndb.de \
--cc=benh@kernel.crashing.org \
--cc=bhelgaas@google.com \
--cc=dann.frazier@canonical.com \
--cc=devicetree@vger.kernel.org \
--cc=frowand.list@gmail.com \
--cc=gregkh@linuxfoundation.org \
--cc=hanjun.guo@linaro.org \
--cc=helgaas@kernel.org \
--cc=joe@perches.com \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-arch@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=linuxarm@huawei.com \
--cc=lorenzo.pieralisi@arm.com \
--cc=mark.rutland@arm.com \
--cc=mika.westerberg@linux.intel.com \
--cc=minyard@acm.org \
--cc=olof@lixom.net \
--cc=rafael@kernel.org \
--cc=rdunlap@infradead.org \
--cc=rjw@rjwysocki.net \
--cc=robh+dt@kernel.org \
--cc=robh@kernel.org \
/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).