Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1470094

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

Show all headers | View raw

> -----Original Message-----
> From: David Miller [mailto:davem@davemloft.net]
> Sent: Thursday, August 25, 2016 5:54 AM
> To: Salil Mehta
> Cc: dledford@redhat.com; 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
> 
> From: Salil Mehta <salil.mehta@huawei.com>
> Date: Wed, 24 Aug 2016 04:44:48 +0800
> 
> > This patch is meant to add support of ACPI to the Hisilicon RoCE
> driver.
> > Following changes have been made in the driver(s):
> >
> > Patch 1/2: HNS Ethernet Driver: changes to support ACPI have been
> done in
> >    the RoCE reset function part of the HNS ethernet driver. Earlier
> it only
> >    supported DT/syscon.
> >
> > Patch 2/2. HNS RoCE driver: changes done in RoCE driver are meant to
> detect
> >    the type and then either use DT specific or ACPI spcific
> functions. Where
> >    ever possible, this patch tries to make use of "Unified Device
> Property
> >    Interface" APIs to support both DT and ACPI through single
> interface.
> >
> > NOTE 1: ACPI changes done in both of the drivers depend upon the ACPI
> Table
> >      (DSDT and IORT tables) changes part of UEFI/BIOS. These changes
> are NOT
> >      part of this patch-set.
> > NOTE 2: Reset function in Patch 1/2 depends upon the reset function
> added in
> >      ACPI tables(basically DSDT table) part of the UEFI/BIOS. Again,
> this
> >      change is NOT reflected in this patch-set.
> 
> I can't apply this series to my tree because the hns infiniband driver
> doesn't exist in it.
Hi David,
I understand your point. This patch-set was primarily sent for Doug Ledford
and is based on his internal repository (which has been rebased on the
net-next). 

Though we were hoping, if by any chance, we can expedite the acceptance of the
below patch part of patch-set in the net-next. This might help Doug Ledford as
well later on when he pushes the already accepted RoCE driver and ACPI patches
to linux-next,

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

Below HNS RoCE reset function patch has already been accepted and is part of your
net-next,
https://patchwork.kernel.org/patch/9287497/

Above ACPI support of RoCE Reset patch cleanly applies over the already accepted
patch in the link. It is not dependent on other accompanying RoCE driver ACPI
changes related patch or even the presence of the Infiniband/RoCE Driver in the
net-next repository.

Could you please suggest anything here?  

Thanks 
Salil

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH for-next 0/2] {IB,net}/hns: Add support of ACPI to the Hisilicon RoCE Driver Salil Mehta <salil.mehta@huawei.com> - 2016-08-23 22:30 +0200
  [PATCH for-next 2/2] IB/hns: Add support of ACPI to the Hisilicon RoCE driver Salil Mehta <salil.mehta@huawei.com> - 2016-08-23 22:30 +0200
    Re: [PATCH for-next 2/2] IB/hns: Add support of ACPI to the  Hisilicon RoCE driver Leon Romanovsky <leon@kernel.org> - 2016-08-24 16:10 +0200
      RE: [PATCH for-next 2/2] IB/hns: Add support of ACPI to the  Hisilicon RoCE driver Salil Mehta <salil.mehta@huawei.com> - 2016-08-24 16:40 +0200
        Re: [PATCH for-next 2/2] IB/hns: Add support of ACPI to the  Hisilicon RoCE driver Leon Romanovsky <leon@kernel.org> - 2016-08-24 17:00 +0200
  [PATCH for-next 1/2] net: hns: Add support of ACPI to HNS driver RoCE Reset function Salil Mehta <salil.mehta@huawei.com> - 2016-08-23 22:30 +0200
  Re: [PATCH for-next 0/2] {IB,net}/hns: Add support of ACPI to the  Hisilicon RoCE Driver David Miller <davem@davemloft.net> - 2016-08-25 07:00 +0200
    RE: [PATCH for-next 0/2] {IB,net}/hns: Add support of ACPI to the  Hisilicon RoCE Driver Salil Mehta <salil.mehta@huawei.com> - 2016-08-25 14:10 +0200
      Re: [PATCH for-next 0/2] {IB,net}/hns: Add support of ACPI to the  Hisilicon RoCE Driver Doug Ledford <dledford@redhat.com> - 2016-08-25 16:00 +0200
        RE: [PATCH for-next 0/2] {IB,net}/hns: Add support of ACPI to the  Hisilicon RoCE Driver Salil Mehta <salil.mehta@huawei.com> - 2016-08-25 17:00 +0200
          Re: [PATCH for-next 0/2] {IB,net}/hns: Add support of ACPI to the  Hisilicon RoCE Driver Doug Ledford <dledford@redhat.com> - 2016-08-25 17:10 +0200
            RE: [PATCH for-next 0/2] {IB,net}/hns: Add support of ACPI to the  Hisilicon RoCE Driver Salil Mehta <salil.mehta@huawei.com> - 2016-08-25 17:40 +0200
            RE: [PATCH for-next 0/2] {IB,net}/hns: Add support of ACPI to the  Hisilicon RoCE Driver Salil Mehta <salil.mehta@huawei.com> - 2016-09-05 12:20 +0200
    RE: [PATCH for-next 0/2] {IB,net}/hns: Add support of ACPI to the  Hisilicon RoCE Driver Salil Mehta <salil.mehta@huawei.com> - 2016-08-25 14:10 +0200
      Re: [PATCH for-next 0/2] {IB,net}/hns: Add support of ACPI to the  Hisilicon RoCE Driver Doug Ledford <dledford@redhat.com> - 2016-08-25 16:10 +0200
        RE: [PATCH for-next 0/2] {IB,net}/hns: Add support of ACPI to the  Hisilicon RoCE Driver Salil Mehta <salil.mehta@huawei.com> - 2016-08-25 17:00 +0200
    Re: [PATCH for-next 0/2] {IB,net}/hns: Add support of ACPI to the  Hisilicon RoCE Driver Doug Ledford <dledford@redhat.com> - 2016-08-25 14:10 +0200
      RE: [PATCH for-next 0/2] {IB,net}/hns: Add support of ACPI to the  Hisilicon RoCE Driver Salil Mehta <salil.mehta@huawei.com> - 2016-09-05 15:00 +0200
        Re: [PATCH for-next 0/2] {IB,net}/hns: Add support of ACPI to the  Hisilicon RoCE Driver David Miller <davem@davemloft.net> - 2016-09-05 21:50 +0200

csiph-web