| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <20210308160815.GB3977653@xps15>
Date: Mon, 8 Mar 2021 09:08:15 -0700
From: Mathieu Poirier <mathieu.poirier@...aro.org>
To: peng.fan@....nxp.com
Cc: ohad@...ery.com, bjorn.andersson@...aro.org,
o.rempel@...gutronix.de, robh+dt@...nel.org,
devicetree@...r.kernel.org, shawnguo@...nel.org,
s.hauer@...gutronix.de, kernel@...gutronix.de, festevam@...il.com,
linux-remoteproc@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
paul@...pouillou.net, matthias.bgg@...il.com, agross@...nel.org,
patrice.chotard@...com, Peng Fan <peng.fan@....com>
Subject: Re: [PATCH V13 00/10] remoteproc: imx_rproc: support iMX8MQ/M
On Sat, Mar 06, 2021 at 07:24:15PM +0800, peng.fan@....nxp.com wrote:
> From: Peng Fan <peng.fan@....com>
>
> V13:
> Add R-b tag from Rob for patch 1.
> Drop the reserved memory node from patch 2 per Rob's comment.
> Mathieu, Bjorn
> Only patch 2 not have R-b/A-b tag, but since Rob's only has a minor comment, and
> addressed in this version, is it ok for you take into remoteproc next branch?
> Thanks.
As much as I want to, there is no way to move forward without an acknowledgement
from Rob.
>
> V12:
> Add maxItems to avoid dt_bindings_check fail
> Rebased on top of linux-next
>
> V11:
> Per Rob's comments, fix memory-region in patch 1/10
> Rebased on top of Linux-next
>
> V10:
> Per Rob's comments, fix patch 1/10
>
> V9:
> Per Mathieu's comments,
> update the tile of yaml in patch 2/10
> update the Kconfig and MODULE_DESCRIPTION, I merge this change in patch 8/10,
> since this is a minor change, I still keep Mathieu's R-b tag. If any objection, I could remove.
> Add R-b tag in Patch 10/10
>
> Rob, please help review patch 1/10 and 2/10
>
> V8:
> Address sparse warning in patch 4/10 reported by kernel test robot
>
> V7:
> Add R-b tag from Mathieu
> vdevbuffer->vdev0buffer in patch 1/10, 7/10
> correct err msg and shutdown seq per Mathieu's comments in patch 10/10
> Hope this version is ok to be merged.
>
> V6:
> Add R-b tag from Mathieu
> Convert imx-rproc.txt to yaml and add dt-bindings support for i.MX8MQ/M, patch 1/10 2/10
> No other changes.
>
> V5:
> Apply on Linux next
> Add V5 subject prefix
> Add R-b tag from Bjorn for 1/8, 2/8, 3/8
> https://patchwork.kernel.org/project/linux-remoteproc/cover/20201229033019.25899-1-peng.fan@nxp.com/
>
> V4:
> According to Bjorn's comments, add is_iomem for da to va usage
> 1/8, 2/8 is new patch
> 3/8, follow Bjorn's comments to correct/update the err msg.
> 6/8, new patch
> 8/8, use dev_err_probe to simplify code, use queue_work instead schedule_delayed_work
>
> V3:
> Since I was quite busy in the past days, V3 is late
> Rebased on Linux-next
> Add R-b tags
> 1/7: Add R-b tag of Mathieu, add comments
> 4/7: Typo fix
> 5/7: Add R-b tag of Mathieu, drop index Per Mathieu's comments
> 6/7: Add R-b tag of Mathieu
> 7/7: Add comment for vqid << 16, drop unneeded timeout settings of mailbox
> Use queue_work instead of schedule_delayed_work
> free mbox channels when remove
> https://lkml.org/lkml/2020/12/4/82
>
> V2:
> Rebased on linux-next
> Dropped early boot feature to make patchset simple.
> Drop rsc-da
> https://patchwork.kernel.org/project/linux-remoteproc/cover/20200927064131.24101-1-peng.fan@nxp.com/
>
> V1:
> https://patchwork.kernel.org/cover/11682461/
>
> This patchset is to support i.MX8MQ/M coproc.
> The early boot feature was dropped to make the patchset small in V2.
>
> Since i.MX specific TCM memory requirement, add elf platform hook.
> Several patches have got reviewed by Oleksij and Mathieu in v1.
>
>
> Peng Fan (10):
> dt-bindings: remoteproc: convert imx rproc bindings to json-schema
> dt-bindings: remoteproc: imx_rproc: add i.MX8MQ/M support
> remoteproc: introduce is_iomem to rproc_mem_entry
> remoteproc: add is_iomem to da_to_va
> remoteproc: imx_rproc: correct err message
> remoteproc: imx_rproc: use devm_ioremap
> remoteproc: imx_rproc: add i.MX specific parse fw hook
> remoteproc: imx_rproc: support i.MX8MQ/M
> remoteproc: imx_rproc: ignore mapping vdev regions
> remoteproc: imx_proc: enable virtio/mailbox
>
> .../bindings/remoteproc/fsl,imx-rproc.yaml | 90 ++++++
> .../bindings/remoteproc/imx-rproc.txt | 33 ---
> drivers/remoteproc/Kconfig | 6 +-
> drivers/remoteproc/imx_rproc.c | 262 +++++++++++++++++-
> drivers/remoteproc/ingenic_rproc.c | 2 +-
> drivers/remoteproc/keystone_remoteproc.c | 2 +-
> drivers/remoteproc/mtk_scp.c | 6 +-
> drivers/remoteproc/omap_remoteproc.c | 2 +-
> drivers/remoteproc/pru_rproc.c | 2 +-
> drivers/remoteproc/qcom_q6v5_adsp.c | 2 +-
> drivers/remoteproc/qcom_q6v5_pas.c | 2 +-
> drivers/remoteproc/qcom_q6v5_wcss.c | 2 +-
> drivers/remoteproc/qcom_wcnss.c | 2 +-
> drivers/remoteproc/remoteproc_core.c | 7 +-
> drivers/remoteproc/remoteproc_coredump.c | 8 +-
> drivers/remoteproc/remoteproc_debugfs.c | 2 +-
> drivers/remoteproc/remoteproc_elf_loader.c | 21 +-
> drivers/remoteproc/remoteproc_internal.h | 2 +-
> drivers/remoteproc/st_slim_rproc.c | 2 +-
> drivers/remoteproc/ti_k3_dsp_remoteproc.c | 2 +-
> drivers/remoteproc/ti_k3_r5_remoteproc.c | 2 +-
> drivers/remoteproc/wkup_m3_rproc.c | 2 +-
> include/linux/remoteproc.h | 4 +-
> 23 files changed, 393 insertions(+), 72 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/remoteproc/fsl,imx-rproc.yaml
> delete mode 100644 Documentation/devicetree/bindings/remoteproc/imx-rproc.txt
>
> --
> 2.30.0
>
Powered by blists - more mailing lists