[EXT] Re: [PATCH v5 01/11] dt-bindings: arm: fsl: add imx-se-fw binding doc
Varun Sethi
V.Sethi at nxp.com
Mon Aug 28 02:14:16 PDT 2023
Hi Krzysztof,
> -----Original Message-----
> From: Krzysztof Kozlowski <krzysztof.kozlowski at linaro.org>
> Sent: Monday, August 28, 2023 12:25 PM
> To: Varun Sethi <V.Sethi at nxp.com>; Greg Kroah-Hartman
> <gregkh at linuxfoundation.org>; Krzysztof Kozlowski <krzk at kernel.org>
> Cc: Rob Herring <robh at kernel.org>; Pankaj Gupta
> <pankaj.gupta at nxp.com>; shawnguo at kernel.org; s.hauer at pengutronix.de;
> kernel at pengutronix.de; clin at suse.com; conor+dt at kernel.org;
> pierre.gondois at arm.com; Jacky Bai <ping.bai at nxp.com>; Clark Wang
> <xiaoning.wang at nxp.com>; Wei Fang <wei.fang at nxp.com>; Peng Fan
> <peng.fan at nxp.com>; Bough Chen <haibo.chen at nxp.com>;
> festevam at gmail.com; dl-linux-imx <linux-imx at nxp.com>;
> davem at davemloft.net; krzysztof.kozlowski+dt at linaro.org; linux-arm-
> kernel at lists.infradead.org; devicetree at vger.kernel.org; linux-
> kernel at vger.kernel.org; Gaurav Jain <gaurav.jain at nxp.com>;
> alexander.stein at ew.tq-group.com; Sahil Malhotra
> <sahil.malhotra at nxp.com>; Aisheng Dong <aisheng.dong at nxp.com>
> Subject: Re: [EXT] Re: [PATCH v5 01/11] dt-bindings: arm: fsl: add
imx-se-fw
> binding doc
>
> Caution: This is an external email. Please take care when clicking links
or
> opening attachments. When in doubt, report the message using the 'Report
> this email' button
>
>
> On 28/08/2023 08:00, Varun Sethi wrote:
> > Hi Greg,
> >
> >> -----Original Message-----
> >> From: Greg Kroah-Hartman <gregkh at linuxfoundation.org>
> >> Sent: Friday, August 25, 2023 12:54 AM
> >> To: Krzysztof Kozlowski <krzk at kernel.org>
> >> Cc: Rob Herring <robh at kernel.org>; Pankaj Gupta
> >> <pankaj.gupta at nxp.com>; shawnguo at kernel.org;
> s.hauer at pengutronix.de;
> >> kernel at pengutronix.de; clin at suse.com; conor+dt at kernel.org;
> >> pierre.gondois at arm.com; Jacky Bai <ping.bai at nxp.com>; Clark Wang
> >> <xiaoning.wang at nxp.com>; Wei Fang <wei.fang at nxp.com>; Peng Fan
> >> <peng.fan at nxp.com>; Bough Chen <haibo.chen at nxp.com>;
> >> festevam at gmail.com; dl-linux-imx <linux-imx at nxp.com>;
> >> davem at davemloft.net; krzysztof.kozlowski+dt at linaro.org; linux-arm-
> >> kernel at lists.infradead.org; devicetree at vger.kernel.org; linux-
> >> kernel at vger.kernel.org; Gaurav Jain <gaurav.jain at nxp.com>;
> >> alexander.stein at ew.tq-group.com; Sahil Malhotra
> >> <sahil.malhotra at nxp.com>; Aisheng Dong <aisheng.dong at nxp.com>;
> Varun
> >> Sethi <V.Sethi at nxp.com>
> >> Subject: [EXT] Re: [PATCH v5 01/11] dt-bindings: arm: fsl: add
> >> imx-se-fw binding doc
> >>
> >> Caution: This is an external email. Please take care when clicking
> >> links
> > or
> >> opening attachments. When in doubt, report the message using the
> >> 'Report this email' button
> >>
> >>
> >> On Thu, Aug 24, 2023 at 08:45:41PM +0200, Krzysztof Kozlowski wrote:
> >>> On 23/08/2023 14:43, Rob Herring wrote:
> >>>>> + |
> >>>>> + +------------------------------------------------------ |
> >>>>> + | | | |
> >>>>> + userspace /dev/ele_muXch0 | | |
> >>>>> + /dev/ele_muXch1 | |
> >>>>> + /dev/ele_muXchY |
> >>>>> + |
> >>>>> +
> >>>>> + When a user sends a command to the firmware, it registers its
> >>>>> + device_ctx as waiter of a response from firmware.
> >>>>> +
> >>>>> + A user can be registered as receiver of command from the ELE.
> >>>>> + Create char devices in /dev as channels of the form
> >>>>> + /dev/ele_muXchY with X the id of the driver and Y for each
> >>>>> + users. It allows to send and receive messages to the NXP
> >>>>> + EdgeLock Enclave IP firmware on NXP SoC, where current possible
> >> value, i.e., supported SoC(s) are imx8ulp, imx93.
> >>>>
> >>>> Looks like a bunch of Linux details which don't belong in the
binding.
> >>>>
> >>>> Why do you need your own custom interface to userspace? No one
> else
> >>>> has a similar feature in their platforms? Something like virtio or
> >>>> rpmsg doesn't work?
> >>>
> >>> +Cc Greg,
> >>>
> >>> I doubt they care. This is some stub-driver to pass messages from
> >>> user-space to the firmware. The interface is undocumented, without
> >>> examples and no user-space user.
> >>
> >> Great, no user? Let's delete the code then :)
> >>
> > [Varun] We do have middleware applications that utilize the driver.
> > Following
> > are the links:
> > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgith
> > ub.com%2Fnxp-imx%2Fimx-secure-
> enclave&data=05%7C01%7CV.Sethi%40nxp.com
> > %7Ceda172f10a684cc926a908dba793b776%7C686ea1d3bc2b4c6fa92cd99c5c
> 301635
> > %7C0%7C0%7C638288025110398317%7CUnknown%7CTWFpbGZsb3d8eyJW
> IjoiMC4wLjAw
> >
> MDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%
> 7C&sda
> >
> ta=jJ%2FZl1myldPaciswIps2dDuuzp%2BqXGZDnvh07yD%2BEFU%3D&reserve
> d=0
> > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgith
> > ub.com%2Fnxp-imx%2Fimx-
> smw&data=05%7C01%7CV.Sethi%40nxp.com%7Ceda172f1
> >
> 0a684cc926a908dba793b776%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7
> C0%7C
> >
> 638288025110398317%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwM
> DAiLCJQIjo
> >
> iV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=U
> VjBwmzZ
> > myHu%2Fmy6WZiQcz2qksA67W1BtwFIJ8%2Ft7nQ%3D&reserved=0
> >
>
> Why this is not explained in the cover letter and in the patch adding the
> interfaces? You still need to document and explain the interface.
>
[Varun] Following information would be added in the next version of the
patchset:
1. Details of the users for this driver.
2. The ABI document would be enhanced to provide information about the
interfaces leveraged
by the user space middleware applications.
Please let me know if this would be sufficient.
Regards
Varun Sethi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 9707 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20230828/3330f6e8/attachment-0001.p7s>
More information about the linux-arm-kernel
mailing list