mbox series

[V2,0/8] remoteproc: imx_rproc: support i.MX7ULP/8MN/8MP

Message ID 1617095574-6764-1-git-send-email-peng.fan@oss.nxp.com (mailing list archive)
Headers show
Series remoteproc: imx_rproc: support i.MX7ULP/8MN/8MP | expand

Message

Peng Fan (OSS) March 30, 2021, 9:12 a.m. UTC
From: Peng Fan <peng.fan@nxp.com>

V2:
 Patch 1/8, use fsl as vendor, typo fix
 Because patchset [1] has v2 version, patch 5,6,7,8 are adapted that change.

This patchset is to support i.MX7ULP/8MN/8MP, also includes a patch to parse
imx,auto-boot
This patchset depends on [1]

[1] https://patchwork.kernel.org/project/linux-remoteproc/cover/1617082235-15923-1-git-send-email-peng.fan@oss.nxp.com/

Peng Fan (8):
  dt-bindings: remoteproc: imx_rproc: add fsl,auto-boot property
  dt-bindings: remoteproc: imx_rproc: add i.MX7ULP support
  dt-bindings: remoteproc: imx_rproc: support i.MX8MN/P
  remoteproc: imx_rproc: make clk optional
  remoteproc: imx_rproc: parse fsl,auto-boot
  remoteproc: imx_rproc: initial support for mutilple start/stop method
  remoteproc: imx_rproc: support i.MX7ULP
  remoteproc: imx_rproc: support i.MX8MN/P

 .../bindings/remoteproc/fsl,imx-rproc.yaml    |  11 +-
 drivers/remoteproc/imx_rproc.c                | 170 +++++++++++++++---
 2 files changed, 159 insertions(+), 22 deletions(-)

Comments

Mathieu Poirier April 7, 2021, 4:18 p.m. UTC | #1
On Tue, Mar 30, 2021 at 05:12:46PM +0800, peng.fan@oss.nxp.com wrote:
> From: Peng Fan <peng.fan@nxp.com>
> 
> V2:
>  Patch 1/8, use fsl as vendor, typo fix
>  Because patchset [1] has v2 version, patch 5,6,7,8 are adapted that change.
> 
> This patchset is to support i.MX7ULP/8MN/8MP, also includes a patch to parse
> imx,auto-boot
> This patchset depends on [1]
> 
> [1] https://patchwork.kernel.org/project/linux-remoteproc/cover/1617082235-15923-1-git-send-email-peng.fan@oss.nxp.com/
> 
> Peng Fan (8):
>   dt-bindings: remoteproc: imx_rproc: add fsl,auto-boot property
>   dt-bindings: remoteproc: imx_rproc: add i.MX7ULP support
>   dt-bindings: remoteproc: imx_rproc: support i.MX8MN/P
>   remoteproc: imx_rproc: make clk optional
>   remoteproc: imx_rproc: parse fsl,auto-boot
>   remoteproc: imx_rproc: initial support for mutilple start/stop method
>   remoteproc: imx_rproc: support i.MX7ULP
>   remoteproc: imx_rproc: support i.MX8MN/P
>

There are glaring checkpatch warning on this set - the "DT compatible string"
ones are fine but the others are not negotiable.  As such I am dropping this set
entirely.
 
>  .../bindings/remoteproc/fsl,imx-rproc.yaml    |  11 +-
>  drivers/remoteproc/imx_rproc.c                | 170 +++++++++++++++---
>  2 files changed, 159 insertions(+), 22 deletions(-)
> 
> -- 
> 2.30.0
>
Peng Fan (OSS) April 8, 2021, 2:18 a.m. UTC | #2
On 2021/4/8 0:18, Mathieu Poirier wrote:
> On Tue, Mar 30, 2021 at 05:12:46PM +0800, peng.fan@oss.nxp.com wrote:
>> From: Peng Fan <peng.fan@nxp.com>
>>
>> V2:
>>   Patch 1/8, use fsl as vendor, typo fix
>>   Because patchset [1] has v2 version, patch 5,6,7,8 are adapted that change.
>>
>> This patchset is to support i.MX7ULP/8MN/8MP, also includes a patch to parse
>> imx,auto-boot
>> This patchset depends on [1]
>>
>> [1] https://patchwork.kernel.org/project/linux-remoteproc/cover/1617082235-15923-1-git-send-email-peng.fan@oss.nxp.com/
>>
>> Peng Fan (8):
>>    dt-bindings: remoteproc: imx_rproc: add fsl,auto-boot property
>>    dt-bindings: remoteproc: imx_rproc: add i.MX7ULP support
>>    dt-bindings: remoteproc: imx_rproc: support i.MX8MN/P
>>    remoteproc: imx_rproc: make clk optional
>>    remoteproc: imx_rproc: parse fsl,auto-boot
>>    remoteproc: imx_rproc: initial support for mutilple start/stop method
>>    remoteproc: imx_rproc: support i.MX7ULP
>>    remoteproc: imx_rproc: support i.MX8MN/P
>>
> 
> There are glaring checkpatch warning on this set - the "DT compatible string"
> ones are fine but the others are not negotiable.  As such I am dropping this set
> entirely.

Ah, sorry for this. I just sent out V3 to address all the checkpatch 
warnings.

Thanks,
Peng.

>   
>>   .../bindings/remoteproc/fsl,imx-rproc.yaml    |  11 +-
>>   drivers/remoteproc/imx_rproc.c                | 170 +++++++++++++++---
>>   2 files changed, 159 insertions(+), 22 deletions(-)
>>
>> -- 
>> 2.30.0
>>