RE: [PATCH for-next 0/2] {IB,net}/hns: Add support of ACPI to the Hisilicon RoCE Driver

-----Original Message-----
From: linux-rdma-owner@vger.kernel.org [mailto:linux-rdma-
owner@vger.kernel.org] On Behalf Of Doug Ledford
Sent: Thursday, August 25, 2016 3:09 PM
To: Salil Mehta; David Miller
Cc: Huwei (Xavier); oulijun; Zhuangyuzeng (Yisen);
mehta.salil.lnk@gmail.com; linux-rdma@vger.kernel.org;
netdev@vger.kernel.org; linux-kernel@vger.kernel.org; Linuxarm
Subject: Re: [PATCH for-next 0/2] {IB,net}/hns: Add support of ACPI to
the Hisilicon RoCE Driver

On 8/25/2016 8:08 AM, Salil Mehta wrote:

quoted

quoted

-----Original Message-----
From: Doug Ledford [mailto:dledford@redhat.com]
Sent: Thursday, August 25, 2016 12:57 PM
To: David Miller; Salil Mehta
Cc: Huwei (Xavier); oulijun; Zhuangyuzeng (Yisen);
mehta.salil.lnk@gmail.com; linux-rdma@vger.kernel.org;
netdev@vger.kernel.org; linux-kernel@vger.kernel.org; Linuxarm
Subject: Re: [PATCH for-next 0/2] {IB,net}/hns: Add support of ACPI

to

quoted

quoted

the Hisilicon RoCE Driver

On 8/25/2016 12:53 AM, David Miller wrote:

quoted

From: Salil Mehta <redacted>
Date: Wed, 24 Aug 2016 04:44:48 +0800

quoted

This patch is meant to add support of ACPI to the Hisilicon RoCE

driver.

quoted

quoted

Following changes have been made in the driver(s):

Patch 1/2: HNS Ethernet Driver: changes to support ACPI have been

done in

quoted

quoted

   the RoCE reset function part of the HNS ethernet driver.

Earlier

quoted

quoted

it only

quoted

quoted

   supported DT/syscon.

Patch 2/2. HNS RoCE driver: changes done in RoCE driver are meant

to

quoted

quoted

detect

quoted

quoted

   the type and then either use DT specific or ACPI spcific

functions. Where

quoted

quoted

   ever possible, this patch tries to make use of "Unified Device

Property

quoted

quoted

   Interface" APIs to support both DT and ACPI through single

interface.

quoted

quoted

NOTE 1: ACPI changes done in both of the drivers depend upon the

ACPI Table

quoted

quoted

     (DSDT and IORT tables) changes part of UEFI/BIOS. These

changes

quoted

quoted

are NOT

quoted

quoted

     part of this patch-set.
NOTE 2: Reset function in Patch 1/2 depends upon the reset

function

quoted

quoted

added in

quoted

quoted

     ACPI tables(basically DSDT table) part of the UEFI/BIOS.

Again,

quoted

quoted

this

quoted

quoted

     change is NOT reflected in this patch-set.

I can't apply this series to my tree because the hns infiniband

driver

quoted

doesn't exist in it.

No.  This probably needs to go through my tree.  Although with all

of

quoted

quoted

the requirements, I'm a bit concerned about those being present
elsewhere.

--
Doug Ledford [off-list ref]
    GPG Key ID: 0E572FDD

Hello Doug,
Thanks for your reply. I have just replied to David email as well and

did

quoted

not realize your response was already on the way. Sorry for this!

I would just like to request, if by any chance, we can expedite the

acceptance

quoted

of the below patch (part of patch-set) in the net-next. This might

help you as

quoted

well in future when you will actually push the RoCE driver to the

linux-next.

quoted

"[PATCH for-next 1/2] net: hns: Add support of ACPI to HNS driver

RoCE Reset

quoted

 function"

Below HNS RoCE reset function patch has already been accepted by Dave

Miller and

quoted

is part of net-next,
https://patchwork.kernel.org/patch/9287497/

Also, above ACPI support of RoCE Reset patch cleanly applies over the

already

quoted

accepted patch in the link and is not dependent on other accompanying

RoCE

quoted

driver ACPI changes or even the presence of the Infiniband/RoCE

Driver in the

quoted

net-next repository.

Could you please suggest if this the something which can be

considered?

I've pulled both of these patches in.  I usually merge late in the
merge
window, so it won't be any stretch to wait until the ACPI tree has been
merged before I send Linus my pull request.

Thanks David! Hope we can take care of the delta which might get created
because of unrelated (not related to RoCE driver from other people) HNS
Ethernet changes?

The pace & magnitude at which HNS development is going on and at which RoCE
Development is going on is different.

Best regards
Salil

--
Doug Ledford [off-list ref]
    GPG Key ID: 0E572FDD