Message-ID: <CO1PR11MB5026F694E4A231BAFA3F5184DAA29@CO1PR11MB5026.namprd11.prod.outlook.com>
Date:   Wed, 20 Jan 2021 13:05:34 +0000
From:   "Sia, Jee Heng" <jee.heng.sia@...el.com>
To:     Eugeniy Paltsev <Eugeniy.Paltsev@...opsys.com>,
        "vkoul@...nel.org" <vkoul@...nel.org>
CC:     "andriy.shevchenko@...ux.intel.com" 
        <andriy.shevchenko@...ux.intel.com>,
        "dmaengine@...r.kernel.org" <dmaengine@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "robh+dt@...nel.org" <robh+dt@...nel.org>
Subject: Re: [PATCH v9 00/16] dmaengine: dw-axi-dmac: support Intel KeemBay
 AxiDMA

Hi Eugeniy Paltsev,

Could you please resend your below Reviewed-by and tested-by tag with plain text format?
I am not seeing your below acknowledgement reflected to the mailing list.

In fact, when I replied on top of your mail, I am receiving below errors:

The following message to <linux-kernel@...r.kernel.org> was undeliverable.
The reason for the problem:
5.3.0 - Other mail system problem 550-'5.7.1 Content-Policy reject msg: The message contains HTML subpart, therefore we consider it SPAM or Outlook Virus.  TEXT/PLAIN is accepted.! BF:<H 0.000529886>; S1731521AbhATMql'

The following message to <devicetree@...r.kernel.org> was undeliverable.
The reason for the problem:
5.3.0 - Other mail system problem 550-'5.7.1 Content-Policy reject msg: The message contains HTML subpart, therefore we consider it SPAM or Outlook Virus.  TEXT/PLAIN is accepted.! BF:<H 0.000529886>; S1731521AbhATMql'

The following message to <dmaengine@...r.kernel.org> was undeliverable.
The reason for the problem:
5.3.0 - Other mail system problem 550-'5.7.1 Content-Policy reject msg: The message contains HTML subpart, therefore we consider it SPAM or Outlook Virus.  TEXT/PLAIN is accepted.! BF:<H 0.000529886>; S1731521AbhATMql'

Thanks
Regards
Jee Heng

________________________________________
From: Eugeniy Paltsev <Eugeniy.Paltsev@...opsys.com> 
Sent: 18 January 2021 8:54 PM
To: Sia, Jee Heng <jee.heng.sia@...el.com>; vkoul@...nel.org
Cc: andriy.shevchenko@...ux.intel.com; dmaengine@...r.kernel.org; linux-kernel@...r.kernel.org; devicetree@...r.kernel.org; robh+dt@...nel.org
Subject: Re: [PATCH v9 00/16] dmaengine: dw-axi-dmac: support Intel KeemBay AxiDMA

Hi Sia Jee Heng,

Reviewed-by: Eugeniy Paltsev <Eugeniy.Paltsev@...opsys.com>

I've runtime tested this series on HSDK SoC/board, so
Tested-by: Eugeniy Paltsev <Eugeniy.Paltsev@...opsys.com>

---
 Eugeniy Paltsev

________________________________________
From: Sia Jee Heng <jee.heng.sia@...el.com>
Sent: Tuesday, January 5, 2021 03:42
To: vkoul@...nel.org <vkoul@...nel.org>; Eugeniy Paltsev <paltsev@...opsys.com>; robh+dt@...nel.org <robh+dt@...nel.org>
Cc: andriy.shevchenko@...ux.intel.com <andriy.shevchenko@...ux.intel.com>; dmaengine@...r.kernel.org <dmaengine@...r.kernel.org>; linux-kernel@...r.kernel.org <linux-kernel@...r.kernel.org>; devicetree@...r.kernel.org <devicetree@...r.kernel.org>
Subject: [PATCH v9 00/16] dmaengine: dw-axi-dmac: support Intel KeemBay AxiDMA 
 
The below patch series are to support AxiDMA running on Intel KeemBay SoC.
The base driver is dw-axi-dmac. This driver only support DMA memory copy
transfers. Code refactoring is needed so that additional features can be
supported.
The features added in this patch series are:
- Replacing Linked List with virtual descriptor management.
- Remove unrelated hw desc stuff from dma memory pool.
- Manage dma memory pool alloc/destroy based on channel activity.
- Support dmaengine device_sync() callback.
- Support dmaengine device_config().
- Support dmaengine device_prep_slave_sg().
- Support dmaengine device_prep_dma_cyclic().
- Support of_dma_controller_register().
- Support burst residue granularity.
- Support Intel KeemBay AxiDMA registers.
- Support Intel KeemBay AxiDMA device handshake.
- Support Intel KeemBay AxiDMA BYTE and HALFWORD device operation.
- Add constraint to Max segment size.
- Virtually split the linked-list.

This patch series are tested on Intel KeemBay platform.

v9:
- Logic checked on apb_regs inside the function.
- Improved code scalability so that missing of apb_regs wouldn't failed
  the common callback functions.

v8:
- Rebased to kernel v5.11-rc1.
- Added reviewed-by tag from Rob.

v7:
- Added 'allOf' and '$ref:dma-controller.yaml#' in DT binding.
- Removed the dma-channels common description in DT binding.
- Removed the default fields in DT binding.

v6:
- Removed 'allOf' cases in DT binding.
- Added '>' at the end of the email address.
- Removed additional '|' at the start of description.
- Fixed space indent.
- Added proper constraint in DT binding.
- Removed second example in DT binding.

v5:
- Added comment to the Apb registers used by Intel KeemBay Soc.
- Renamed "hs_num" to "handshake_num".
- Conditional check for the compatible property and return error
  instead of printing warning.
- Added patch 16th to virtually split the linked-list as per
  request from ALSA team.

v4:
- Fixed bot found errors running make_dt_binding_check.
- Added minItems: 1 to the YAML schemas DT binding.
- Updated "reg" field to the YAML schemas DT binding.

v3:
- Added additionalProperties: false to the YAML schemas DT binding.
- Reordered patch sequence for patch 10th, 11th and 12th so that
  DT binding come first, follow by adding Intel KeemBay SoC registers
  and update .compatible field.
- Checked txstate NULL condition.
- Created helper function dw_axi_dma_set_hw_desc() to handle common code.

v2:
- Rebased to v5.10-rc1 kernel.
- Added support for dmaengine device_config().
- Added support for dmaengine device_prep_slave_sg().
- Added support for dmaengine device_prep_dma_cyclic().
- Added support for of_dma_controller_register().
- Added support for burst residue granularity.
- Added support for Intel KeemBay AxiDMA registers.
- Added support for Intel KeemBay AxiDMA device handshake.
- Added support for Intel KeemBay AxiDMA BYTE and HALFWORD device operation.
- Added constraint to Max segment size.

v1:
- Initial version. Patch on top of dw-axi-dma driver. This version improve
  the descriptor management by replacing Linked List Item (LLI) with
  virtual descriptor management, only allocate hardware LLI memories from
  DMA memory pool, manage DMA memory pool alloc/destroy based on channel
  activity and to support device_sync callback.

Sia Jee Heng (16):
  dt-bindings: dma: Add YAML schemas for dw-axi-dmac
  dmaengine: dw-axi-dmac: simplify descriptor management
  dmaengine: dw-axi-dmac: move dma_pool_create() to
    alloc_chan_resources()
  dmaengine: dw-axi-dmac: Add device_synchronize() callback
  dmaengine: dw-axi-dmac: Add device_config operation
  dmaengine: dw-axi-dmac: Support device_prep_slave_sg
  dmaegine: dw-axi-dmac: Support device_prep_dma_cyclic()
  dmaengine: dw-axi-dmac: Support of_dma_controller_register()
  dmaengine: dw-axi-dmac: Support burst residue granularity
  dt-binding: dma: dw-axi-dmac: Add support for Intel KeemBay AxiDMA
  dmaengine: dw-axi-dmac: Add Intel KeemBay DMA register fields
  dmaengine: dw-axi-dmac: Add Intel KeemBay AxiDMA support
  dmaengine: dw-axi-dmac: Add Intel KeemBay AxiDMA handshake
  dmaengine: dw-axi-dmac: Add Intel KeemBay AxiDMA BYTE and HALFWORD
    registers
  dmaengine: dw-axi-dmac: Set constraint to the Max segment size
  dmaengine: dw-axi-dmac: Virtually split the linked-list

 .../bindings/dma/snps,dw-axi-dmac.txt         |  39 -
 .../bindings/dma/snps,dw-axi-dmac.yaml        | 126 ++++
 .../dma/dw-axi-dmac/dw-axi-dmac-platform.c    | 697 +++++++++++++++---
 drivers/dma/dw-axi-dmac/dw-axi-dmac.h         |  34 +-
 4 files changed, 764 insertions(+), 132 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.txt
 create mode 100644 Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml


base-commit: e71ba9452f0b5b2e8dc8aa5445198cd9214a6a62
-- 
2.18.0

Powered by blists - more mailing lists