Re: [PATCH v38 1/7] Documentation/firmware: add imx/se to other_interfaces
| Newsgroups | dev.linux.lists.imx,org.kernel.vger.linux-devicetree |
|---|---|
| Message-ID | <[email protected]> |
Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider: - [Low] The patch includes a `kernel-doc` directive for a file (`drivers/firmware/imx/se_ctrl.c`) that does not exist in the source tree at this revision. -- --- Patch [1]: [PATCH v38 1/7] Documentation/firmware: add imx/se to other_interfaces --- commit 9d7548655ebb737efce2e2011a588d419c6d261b Author: Pankaj Gupta <[email protected]> Documentation/firmware: add imx/se to other_interfaces This commit documents the i.MX SoC's Service layer and C_DEV driver for selected SoCs containing NXP hardware IPs for Secure Enclaves. > diff --git a/Documentation/driver-api/firmware/other_interfaces.rst b/Documentation/driver-api/firmware/other_interfaces.rst > index 06ac89adaafba..6c6fa9a0ba1df 100644 > --- a/Documentation/driver-api/firmware/other_interfaces.rst > +++ b/Documentation/driver-api/firmware/other_interfaces.rst [ ... ] > +When a user sends a command to the firmware, it registers its device_ctx > +as waiter of a response from firmware. > + > +Enclave's Firmware owns the storage management over a Linux filesystem. > +For this c_dev provisions a dedicated slave device called "receiver". > + > +.. kernel-doc:: drivers/firmware/imx/se_ctrl.c > + :export: [Severity: Low] Does this file exist in the source tree at this revision? It appears that building the htmldocs will generate warnings because drivers/firmware/imx/se_ctrl.c is not present in the tree yet. -- Sashiko AI review · https://sashiko.dev/#/patchset/[email protected]?part=1