Received-SPF: pass (google.com: domain of linux-kernel+bounces-112389-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
From: Frank Li <Frank.Li@nxp.com>
Subject: [PATCH v3 0/5] dmaengine: fsl-edma: add 8ulp support
Date: Sat, 23 Mar 2024 11:34:49 -0400
Message-Id: <20240323-8ulp_edma-v3-0-c0e981027c05@nxp.com>
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
To: Vinod Koul <vkoul@kernel.org>, Rob Herring <robh@kernel.org>, 
 Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, 
 Conor Dooley <conor+dt@kernel.org>, Peng Fan <peng.fan@nxp.com>
Cc: imx@lists.linux.dev, dmaengine@vger.kernel.org, 
 linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, 
 Frank Li <Frank.Li@nxp.com>, Joy Zou <joy.zou@nxp.com>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?TmtKWCtsYTFqc0h2ZzNhTGc3bHZROUM5MjUybTJzRFdNWDJ0ZG90VjdSRCs3?=
 =?utf-8?B?WGs5WEN2Z1p6NHV1bTR3VTh1S0NvSWZocGQwVmt4Q3BwbUYrWkNuczhVTkNX?=
 =?utf-8?B?L3g0U1R1OXAwdG45QXU1ODVTaVpzK1kyaTZRNGpNbWJ2akJzcnJHZ3dPTWx3?=
 =?utf-8?B?RkN1ZndKcWEvQ0Zjb1VlV1AwYjZFZXRxZlFoUHZabXdkMTBKUDRsbFVPV3Zz?=
 =?utf-8?B?ZTQ2NjVlUVdmRVhRVkM2djJiZ1lRb3hsVHJkQyt0dkkzVjFxL2o1RllLamQ2?=
 =?utf-8?B?R2VGdTg0NCtNZmQzU2x0WEE5RUNLWFJmVmJuc0FqYjNSL2E1Y3l5NkNIOTQx?=
 =?utf-8?B?cVluVlRvSUZuVEQ3OU52REhQOXcxbWZlUTJuUmlmTlNPU25QUU9qZmJlVWlV?=
 =?utf-8?B?b3E2ODVsVnBObjV6TVk5ZCtBYm9wK092Q1U3QlpObi9wemsxK0lSTjdQMG5W?=
 =?utf-8?B?OUxWazBGbGJvcFpVWHpWMkJNUlJMU01HdHZPckE0b0ZHNDRtbHRJVFlPOWdD?=
 =?utf-8?B?Yi96Z1VsRGpWazl5b3Q2VjFVWW5kRUxYVVZ3dVJtNTNvRFZjS0RrbGxhV3RB?=
 =?utf-8?B?Si9kZENnRE8reG1XcW9IaFhzOFRjb0N3TU9MOG5LcnRtRzd3NmlVdUdUQWht?=
 =?utf-8?B?NUJFVG9uSnh1VHIxWSt5bmdOa1pMS1YzeUVIWC9scWozUFBGOHBZY0ZyYXdz?=
 =?utf-8?B?R2ljWjF6VGZjRWdJNEQ5eGhkOU0vS2FYY2hhRzRuZWo1NGJRemQ5UlJUYllW?=
 =?utf-8?B?b09NaVgzcTlEZFpLcWN6RVNiODFzK3BGUjdPR3N1Y1RiWExEczd4NU92TWVZ?=
 =?utf-8?B?TjNldmdXeUhWWFo3OHl0RGRITFRWOE15OG1GLzgva3pBV3JMdzg3KzFaSTd2?=
 =?utf-8?B?NVhEUVVEQWNnSE14akdMbzMzTUtBYVU5N2oyUWVXYWlaVVVsTWloVG1QWHRo?=
 =?utf-8?B?MjNibzkrOXhQRWlGamg0OUV1TTVTUkoyS1VrcDZYS3ZCb0hnSTVMMWpWeG5r?=
 =?utf-8?B?MHEram10UE1teS9hSjVlQTc5Nk0yMTVBNUdUWWR1WnI3NGdkVnRFYnQyMWZ0?=
 =?utf-8?B?OFhERTE2V1JBbk9xTDBJL0JpbUZWVTRwYVNFcXBzbXhCbUxZUU9FTkdEbHhD?=
 =?utf-8?B?RDNOQ3NQRzVFcVFIcFhqV2hWRXRYZWdUcHg3THYzL0tIUC9hcjRVVSszd1k2?=
 =?utf-8?B?Q3FHM0xwbEt1MzJSVDlDNzYreFRieDd2dFY0SVBNVlZmRFY1cEVZd3ZwSHRM?=
 =?utf-8?B?YjZKMkJKcWdjNkhWeUhSeHpkMUx1OStBa1ZZQTRjMmE3dFNQREFWdXZGREhn?=
 =?utf-8?B?c1N2NXJ0QTU0VERPOU12UUZlVmQ2T3Zxb2lsejZtMVdzTVh6bm9hKzBCZ0ZD?=
 =?utf-8?B?bmdZblJpOHJhMUdTZVdIOGxZc2dCOHorVDcvZVZkRyt4ajdMSHJqdTU5NXlZ?=
 =?utf-8?B?UWRwM1N3YUdOVVYvRlUrd0hGc1B3UUdVc2c1NzhPUEtmOElzdXI4WmJ0bGpV?=
 =?utf-8?B?b3lJR2VXb1QrSXZuNXRJd2RsamtGRTRBam9vbUJBdCtJU1lmbGg1aTJtRlJW?=
 =?utf-8?B?T284RWFRblR6dUR2NXJwVk9YemtVdjdtKzkzL2lRZFdRNlFlYmduTW1GMkYr?=
 =?utf-8?B?Mjc3ZjdFbHhxdkY5ZThvZzF3ejczTWUxSklNekJvZVlaS2FacUx4MTZ4SHBu?=
 =?utf-8?B?dzNDdU9DMUs5Wm1yTWJqWW1ERFdxQ1lQU1NhUlE1WUdNUWJnZko4VkxoaWhZ?=
 =?utf-8?B?L0NpUmV6aHF5Q053elNiR0x5ckxsa2pXZVdJTFlYU1FaeXh0enZadCtseUs0?=
 =?utf-8?B?TXEveFdwcmxxL1JEVzlzZ3JzbWJVYk92WlZnNy80SVk5S2NyWlhxWTFnekxP?=
 =?utf-8?B?QVdJLzFUc2Z5a1NraTc1VUQxV2dUZDVOaGRDQzNPWm5udVNaVkdoWXRSTnRp?=
 =?utf-8?B?MUYvb24walNRS3N4dVI3VjMxUUtLTUErZ09pYU1EaGFZTFJmSDFINGJRM1hG?=
 =?utf-8?B?Q3pmRVhxWDRMb1Z0Ukh1Vk9jcG5EeTl3YlRmOW1HcnJyM1NVSnJPY0V1WkNN?=
 =?utf-8?B?cWdDY0RvUEpvMEFkOXp4c2RHU3lqbzFvbi9xNkJuTXF6dWVwOFdudzNQajUr?=
 =?utf-8?Q?Gtd5kLGK+fK5wMNWTdxyf9Agv?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 3ead93ba-e8b7-4644-4f31-08dc4b4ed62d
X-MS-Exchange-CrossTenant-AuthSource: PAXPR04MB9642.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Mar 2024 15:35:15.3792
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: sHgcl8ZTBMGlT1izlRhRADGODh1Gd8lP7TSnfOLhXJQdDs5kNyj+O3uEKaaY78vuiH7TLGGwzdj2wO1GKNoYWA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PA4PR04MB7981

Do some small clean up.                                                    

0c562876972ee dmaengine: fsl-edma: remove 'slave_id' from fsl_edma_chan    
d9b66cb5fdf62 dmaengine: fsl-edma: add safety check for 'srcid'            
aae21b7528311 dmaengine: fsl-edma: clean up chclk and FSL_EDMA_DRV_HAS_CHCLK

Update binding doc.                                                        
23a1d1a6609fa dt-bindings: fsl-dma: fsl-edma: add fsl,imx8ulp-edma compatible string

Add 8ulp support.                                                          
dmaengine: fsl-edma: add i.MX8ULP edma support

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
Changes in v3:
- Change clock name form CHXX-CLK to chxx
- Fix typeo 'clock'
- Add dma-cell description
- About clock-names:
  items:
    oneOf:
      - const: dma
      - pattern: ...

Which already detect naming wrong, for example:

clock-names = "dma", "ch00", "ch01", "ch02", "ch03",                       
              ....                                                         
              "ch28", "ch29", "ch30", "abcc";                              
                                                                           
                                                                           
arch/arm64/boot/dts/freescale/imx8ulp-evk.dtb: dma-controller@29010000: clock-names:32: 'oneOf' conditional failed, one must be fixed:
        'dma' was expected                                                 
        'abcc' does not match '^ch(0[0-9]|[1-2][0-9]|3[01])$'  

Only lose order check, such as ch00, dma, ch03, ch02, can pass check.
I think it is good enough. 

I tried rob's suggestion, but met some technology issue. Detail see

https://lore.kernel.org/imx/20240229-8ulp_edma-v2-0-9d12f883c8f7@nxp.com/T/#mc5767dd505d4b7cfc66586a0631684a57e735476

- Link to v2: https://lore.kernel.org/r/20240229-8ulp_edma-v2-0-9d12f883c8f7@nxp.com

Changes in v2:
  Fixed dt-bind about clocks number restriction for vf610. Keep the same
restriction for other compatible string.

  Send out v2 to avoid consiste test rebot report build error.
  Fixed build error

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202402292240.149bq00a-lkp@intel.com/

  Rework dt-binding commit message. Add reason why change clk number to 33.

- Link to v1: https://lore.kernel.org/r/20240227-8ulp_edma-v1-0-7fcfe1e265c2@nxp.com

---
Frank Li (3):
      dmaengine: fsl-edma: remove 'slave_id' from fsl_edma_chan
      dmaengine: fsl-edma: add safety check for 'srcid'
      dmaengine: fsl-edma: clean up chclk and FSL_EDMA_DRV_HAS_CHCLK

Joy Zou (2):
      dt-bindings: dma: fsl-edma: add fsl,imx8ulp-edma compatible string
      dmaengine: fsl-edma: add i.MX8ULP edma support

 .../devicetree/bindings/dma/fsl,edma.yaml          | 40 +++++++++++++++++-
 drivers/dma/fsl-edma-common.c                      |  6 +++
 drivers/dma/fsl-edma-common.h                      |  2 -
 drivers/dma/fsl-edma-main.c                        | 47 ++++++++++++++++------
 drivers/dma/mcf-edma-main.c                        |  4 +-
 5 files changed, 80 insertions(+), 19 deletions(-)
---
base-commit: 37cbf12c590dbe3e66c6d489aaf9b35f7b6a0670
change-id: 20240227-8ulp_edma-12ee2648d74b

Best regards,
---
Frank Li <Frank.Li@nxp.com>