[PATCH 1/1] tee: optee: do not check memref size on return from Secure World

2021-03-22 Thread Jerome Forissier
without dynamic shared memory support (CFG_CORE_DYN_SHM=n). Suggested-by: Jens Wiklander Signed-off-by: Jerome Forissier --- drivers/tee/optee/core.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/drivers/tee/optee/core.c b/drivers/tee/optee/core.c index 319a1e701163..ddb8f9ecf307

Re: [PATCH v8 2/4] KEYS: trusted: Introduce TEE based Trusted Keys

2021-01-21 Thread Jerome Forissier
On 1/21/21 4:24 PM, Jarkko Sakkinen wrote: > On Thu, Jan 21, 2021 at 05:07:42PM +0200, Jarkko Sakkinen wrote: >> On Thu, Jan 21, 2021 at 09:44:07AM +0100, Jerome Forissier wrote: >>> >>> >>> On 1/21/21 1:02 AM, Jarkko Sakkinen via OP-TEE wrote: >>&

Re: [PATCH v8 2/4] KEYS: trusted: Introduce TEE based Trusted Keys

2021-01-21 Thread Jerome Forissier
On 1/21/21 1:02 AM, Jarkko Sakkinen via OP-TEE wrote: > On Wed, Jan 20, 2021 at 12:53:28PM +0530, Sumit Garg wrote: >> On Wed, 20 Jan 2021 at 07:01, Jarkko Sakkinen wrote: >>> >>> On Tue, Jan 19, 2021 at 12:30:42PM +0200, Jarkko Sakkinen wrote: On Fri, Jan 15, 2021 at 11:32:31AM +0530,

[PATCH v4] checkpatch: add --kconfig-prefix

2020-08-18 Thread Jerome Forissier
the --kconfig-prefix command line option. [1] https://github.com/OP-TEE/optee_os Signed-off-by: Jerome Forissier Acked-by: Joe Perches --- scripts/checkpatch.pl | 12 1 file changed, 8 insertions(+), 4 deletions(-) v4: - Add missing link - Apply Joe's Acked-by: tag v3: - Use ${CONFIG_

Re: [PATCH v3] checkpatch: add --kconfig-prefix

2020-08-18 Thread Jerome Forissier
On 8/18/20 9:43 AM, Jerome Forissier wrote: > Kconfig allows to customize the CONFIG_ prefix via the $CONFIG_ > environment variable. Out-of-tree projects may therefore use Kconfig > with a different prefix, or they may use a custom configuration tool > which does not use the CO

[PATCH v3] checkpatch: add --kconfig-prefix

2020-08-18 Thread Jerome Forissier
the --kconfig-prefix command line option. Signed-off-by: Jerome Forissier --- scripts/checkpatch.pl | 12 1 file changed, 8 insertions(+), 4 deletions(-) v3: - Use ${CONFIG_} instead of $CONFIG_. - Expand the commit message to mention OP-TEE. v2: - Use a command-line/.checkpatch.conf

[PATCH v2] checkpatch: add --kconfig-prefix

2020-08-17 Thread Jerome Forissier
to the Linux kernel coding style and run checkpatch.pl. To make this possible, add the --kconfig-prefix command line option. Signed-off-by: Jerome Forissier --- scripts/checkpatch.pl | 12 1 file changed, 8 insertions(+), 4 deletions(-) v2: - Use a command-line/.checkpatch.conf option instead

Re: [PATCH] checkpatch: get CONFIG_ prefix from the environment

2020-08-17 Thread Jerome Forissier
On 8/17/20 4:09 PM, Joe Perches wrote: > On Mon, 2020-08-17 at 11:50 +0200, Jerome Forissier wrote: >> Kconfig allows to customize the CONFIG_ prefix via the $CONFIG_ >> environment variable. Out-of-tree projects may therefore use Kconfig >> with a different prefix, or

[PATCH] checkpatch: get CONFIG_ prefix from the environment

2020-08-17 Thread Jerome Forissier
to the Linux kernel coding style and run checkpatch.pl. To make this possible, update checkpatch to use the value of $CONFIG_ if defined or "CONFIG_" otherwise. Signed-off-by: Jerome Forissier --- scripts/checkpatch.pl | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git

Re: [Tee-dev] [PATCHv8 1/3] optee: use uuid for sysfs driver entry

2020-06-24 Thread Jerome Forissier
On 6/24/20 5:21 PM, James Bottomley wrote: > On Wed, 2020-06-24 at 16:17 +0530, Sumit Garg wrote: >> Apologies for delay in my reply as I was busy with some other stuff. >> >> On Fri, 19 Jun 2020 at 20:30, James Bottomley >> wrote: > [...] >>> it's about consistency with what the kernel types

Re: [Tee-dev] [PATCHv8 1/3] optee: use uuid for sysfs driver entry

2020-06-18 Thread Jerome Forissier
On 6/18/20 6:59 AM, Sumit Garg wrote: > Hi Jerome, > > On Wed, 17 Jun 2020 at 20:46, Jerome Forissier wrote: >> >> >> >> On 6/17/20 3:58 PM, Sumit Garg wrote: >>> Hi Maxim, >>> >>> On Thu, 4 Jun 2020 at 23:28, Maxim Uvarov wrote:

Re: [Tee-dev] [PATCHv8 1/3] optee: use uuid for sysfs driver entry

2020-06-17 Thread Jerome Forissier
On 6/17/20 9:52 PM, Maxim Uvarov wrote: > On Wed, 17 Jun 2020 at 18:16, Jerome Forissier wrote: >> >> On 6/17/20 3:58 PM, Sumit Garg wrote: >>> Hi Maxim, >>> >>> On Thu, 4 Jun 2020 at 23:28, Maxim Uvarov wrote: >>>> >>>> Wit

Re: [Tee-dev] [PATCHv8 1/3] optee: use uuid for sysfs driver entry

2020-06-17 Thread Jerome Forissier
On 6/17/20 3:58 PM, Sumit Garg wrote: > Hi Maxim, > > On Thu, 4 Jun 2020 at 23:28, Maxim Uvarov wrote: >> >> With the evolving use-cases for TEE bus, now it's required to support >> multi-stage enumeration process. But using a simple index doesn't >> suffice this requirement and instead leads

Re: [PATCH v5 2/4] riscv: Introduce CONFIG_RELOCATABLE

2020-06-10 Thread Jerome Forissier
On 6/7/20 9:59 AM, Alexandre Ghiti wrote: [...] > +config RELOCATABLE > + bool > + depends on MMU > + help > + This builds a kernel as a Position Independent Executable (PIE), > + which retains all relocation metadata required to relocate the > + kernel

Re: [Tee-dev] [PATCHv3 2/3] optee: use uuid for sysfs driver entry

2020-05-25 Thread Jerome Forissier
On 5/25/20 3:36 PM, Maxim Uvarov wrote: > On Mon, 25 May 2020 at 15:10, Jerome Forissier wrote: >> >> >> >> On 5/25/20 1:52 PM, Maxim Uvarov wrote: >>> Optee device names for sysfs needed to be unique >> >> s/Optee/OP-TEE/ >> s/needed/ne

Re: [Tee-dev] [PATCHv3 2/3] optee: use uuid for sysfs driver entry

2020-05-25 Thread Jerome Forissier
On 5/25/20 1:52 PM, Maxim Uvarov wrote: > Optee device names for sysfs needed to be unique s/Optee/OP-TEE/ s/needed/need/ > and it's better if they will mean something. UUID for name > looks like good solution: > /sys/bus/tee/devices/optee-clnt- How about mentioning it is the UUID of the

[PATCH] tee: optee: update optee_msg.h and optee_smc.h to dual license

2019-02-08 Thread Jerome Forissier
/optee_msg.h [2] https://github.com/OP-TEE/optee_os/blob/master/core/arch/arm/include/sm/optee_smc.h Signed-off-by: Jerome Forissier --- drivers/tee/optee/optee_msg.h | 26 ++ drivers/tee/optee/optee_smc.h | 26 ++ 2 files changed, 4 insertions(+), 48

[PATCH v2] Documentation: dt: Add binding for /secure-chosen/stdout-path

2018-10-08 Thread Jerome Forissier
e is meant to be used by the secure firmware to pass data to the secure OS. Only the stdout-path property is supported for now. Signed-off-by: Jerome Forissier --- Notes: Sending this again, slightly modified. Previous submission was in March 2017 [1]. Since then, OP-TEE has implemented th

[PATCH v2] Documentation: dt: Add binding for /secure-chosen/stdout-path

2018-10-08 Thread Jerome Forissier
e is meant to be used by the secure firmware to pass data to the secure OS. Only the stdout-path property is supported for now. Signed-off-by: Jerome Forissier --- Notes: Sending this again, slightly modified. Previous submission was in March 2017 [1]. Since then, OP-TEE has implemented th

Re: [PATCH 2/2] clk: imx: imx7d: remove clks_init_on array

2018-08-31 Thread Jerome Forissier
On 08/31/2018 03:29 AM, Stephen Boyd wrote: > Quoting Peng Fan (2018-08-12 18:15:41) >> Hi Anson, >> > -Original Message- > From: Anson Huang > Sent: 2018年8月8日 12:39 > To: shawn...@kernel.org; s.ha...@pengutronix.de; > ker...@pengutronix.de; Fabio Estevam ; >

Re: [PATCH 2/2] clk: imx: imx7d: remove clks_init_on array

2018-08-31 Thread Jerome Forissier
On 08/31/2018 03:29 AM, Stephen Boyd wrote: > Quoting Peng Fan (2018-08-12 18:15:41) >> Hi Anson, >> > -Original Message- > From: Anson Huang > Sent: 2018年8月8日 12:39 > To: shawn...@kernel.org; s.ha...@pengutronix.de; > ker...@pengutronix.de; Fabio Estevam ; >

Re: [PATCH v2] firmware: qcom: scm: Fix incorrect of_node_put call in scm_init

2017-12-07 Thread Jerome Forissier
On 12/06/2017 09:06 PM, Stephen Boyd wrote: > On 12/06, Loys Ollivier wrote: >> When using other platform architectures, in the init of the qcom_scm >> driver, of_node_put is called on /firmware if no qcom dt is found. >> This results in a kernel error: Bad of_node_put() on /firmware. >> >> The

Re: [PATCH v2] firmware: qcom: scm: Fix incorrect of_node_put call in scm_init

2017-12-07 Thread Jerome Forissier
On 12/06/2017 09:06 PM, Stephen Boyd wrote: > On 12/06, Loys Ollivier wrote: >> When using other platform architectures, in the init of the qcom_scm >> driver, of_node_put is called on /firmware if no qcom dt is found. >> This results in a kernel error: Bad of_node_put() on /firmware. >> >> The

Re: [PATCH] tee: add forward declaration for struct device

2017-06-14 Thread Jerome Forissier
On 06/14/2017 01:01 PM, Arnd Bergmann wrote: > On Wed, Jun 14, 2017 at 11:46 AM, Jens Wiklander > <jens.wiklan...@linaro.org> wrote: >> On Wed, Jun 14, 2017 at 11:39:50AM +0200, Jerome Forissier wrote: >>> [+Arnd] >>> >>> Ping? >>> >&g

Re: [PATCH] tee: add forward declaration for struct device

2017-06-14 Thread Jerome Forissier
On 06/14/2017 01:01 PM, Arnd Bergmann wrote: > On Wed, Jun 14, 2017 at 11:46 AM, Jens Wiklander > wrote: >> On Wed, Jun 14, 2017 at 11:39:50AM +0200, Jerome Forissier wrote: >>> [+Arnd] >>> >>> Ping? >>> >>> Thanks, >>> --

Re: [PATCH] tee: add forward declaration for struct device

2017-06-14 Thread Jerome Forissier
[+Arnd] Ping? Thanks, -- Jerome On 05/31/2017 01:21 PM, Jerome Forissier wrote: > tee_drv.h references struct device, but does not include device.h nor > platform_device.h. Therefore, if tee_drv.h is included by some file > that does not pull device.h nor platform_device.h beforehand

Re: [PATCH] tee: add forward declaration for struct device

2017-06-14 Thread Jerome Forissier
[+Arnd] Ping? Thanks, -- Jerome On 05/31/2017 01:21 PM, Jerome Forissier wrote: > tee_drv.h references struct device, but does not include device.h nor > platform_device.h. Therefore, if tee_drv.h is included by some file > that does not pull device.h nor platform_device.h beforehand

[PATCH] tee: add forward declaration for struct device

2017-05-31 Thread Jerome Forissier
tee_drv.h references struct device, but does not include device.h nor platform_device.h. Therefore, if tee_drv.h is included by some file that does not pull device.h nor platform_device.h beforehand, we have a compile warning. Fix this by adding a forward declaration. Signed-off-by: Jerome

[PATCH] tee: add forward declaration for struct device

2017-05-31 Thread Jerome Forissier
tee_drv.h references struct device, but does not include device.h nor platform_device.h. Therefore, if tee_drv.h is included by some file that does not pull device.h nor platform_device.h beforehand, we have a compile warning. Fix this by adding a forward declaration. Signed-off-by: Jerome

Re: [PATCH] checkpatch: add --typedefsfile

2017-04-27 Thread Jerome Forissier
On 04/21/2017 08:31 AM, Jerome Forissier wrote: > On 04/20/2017 06:49 PM, Joe Perches wrote: >> On Thu, 2017-04-20 at 17:39 +0200, Jerome Forissier wrote: >>> When using checkpatch on out-of-tree code, it may occur that some >>> project-specific types are used, which w

Re: [PATCH] checkpatch: add --typedefsfile

2017-04-27 Thread Jerome Forissier
On 04/21/2017 08:31 AM, Jerome Forissier wrote: > On 04/20/2017 06:49 PM, Joe Perches wrote: >> On Thu, 2017-04-20 at 17:39 +0200, Jerome Forissier wrote: >>> When using checkpatch on out-of-tree code, it may occur that some >>> project-specific types are used, which w

Re: [PATCH] checkpatch: add --typedefsfile

2017-04-21 Thread Jerome Forissier
On 04/20/2017 06:49 PM, Joe Perches wrote: > On Thu, 2017-04-20 at 17:39 +0200, Jerome Forissier wrote: >> When using checkpatch on out-of-tree code, it may occur that some >> project-specific types are used, which will cause spurious warnings. >> Add the --typedefsfile opti

Re: [PATCH] checkpatch: add --typedefsfile

2017-04-21 Thread Jerome Forissier
On 04/20/2017 06:49 PM, Joe Perches wrote: > On Thu, 2017-04-20 at 17:39 +0200, Jerome Forissier wrote: >> When using checkpatch on out-of-tree code, it may occur that some >> project-specific types are used, which will cause spurious warnings. >> Add the --typedefsfile opti

[PATCH] checkpatch: add --typedefsfile

2017-04-20 Thread Jerome Forissier
When using checkpatch on out-of-tree code, it may occur that some project-specific types are used, which will cause spurious warnings. Add the --typedefsfile option as a way to extend the known types and deal with this issue. Signed-off-by: Jerome Forissier <jerome.foriss...@linaro.

[PATCH] checkpatch: add --typedefsfile

2017-04-20 Thread Jerome Forissier
When using checkpatch on out-of-tree code, it may occur that some project-specific types are used, which will cause spurious warnings. Add the --typedefsfile option as a way to extend the known types and deal with this issue. Signed-off-by: Jerome Forissier --- scripts/checkpatch.pl | 56

Re: [PATCH] checkpatch: don't match against empty $const_structs

2016-10-20 Thread Jerome Forissier
On 10/20/2016 10:16 AM, Joe Perches wrote: > On Thu, 2016-10-20 at 10:12 +0200, Jerome Forissier wrote: >> When $conststructsfile does not exist or is empty, we may get false >> warnings such as: >> WARNING: struct should normally be const >> >> Fix tha

Re: [PATCH] checkpatch: don't match against empty $const_structs

2016-10-20 Thread Jerome Forissier
On 10/20/2016 10:16 AM, Joe Perches wrote: > On Thu, 2016-10-20 at 10:12 +0200, Jerome Forissier wrote: >> When $conststructsfile does not exist or is empty, we may get false >> warnings such as: >> WARNING: struct should normally be const >> >> Fix tha

[PATCH] checkpatch: don't match against empty $const_structs

2016-10-20 Thread Jerome Forissier
When $conststructsfile does not exist or is empty, we may get false warnings such as: WARNING: struct should normally be const Fix that by not running the string match if $const_structs is empty. Signed-off-by: Jerome Forissier <jerome.foriss...@linaro.org> --- scripts/checkpatch.pl | 3

[PATCH] checkpatch: don't match against empty $const_structs

2016-10-20 Thread Jerome Forissier
When $conststructsfile does not exist or is empty, we may get false warnings such as: WARNING: struct should normally be const Fix that by not running the string match if $const_structs is empty. Signed-off-by: Jerome Forissier --- scripts/checkpatch.pl | 3 ++- 1 file changed, 2 insertions

[PATCH] checkpatch: don't try to get maintained status when --no-tree is given

2016-10-17 Thread Jerome Forissier
Fixes the following warning: Use of uninitialized value $root in concatenation (.) or string at /path/to/checkpatch.pl line 764. Signed-off-by: Jerome Forissier <jerome.foriss...@linaro.org> --- scripts/checkpatch.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/s

[PATCH] checkpatch: don't try to get maintained status when --no-tree is given

2016-10-17 Thread Jerome Forissier
Fixes the following warning: Use of uninitialized value $root in concatenation (.) or string at /path/to/checkpatch.pl line 764. Signed-off-by: Jerome Forissier --- scripts/checkpatch.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/checkpatch.pl b/scripts

[PATCH] mmc: dw_mmc: k3: add MMC_CAP_CMD23

2016-06-01 Thread Jerome Forissier
Enables RPMB support for the on-board eMMC of the HiKey board as well as for eMMC modules connected to the microSD slot. Signed-off-by: Jerome Forissier <jerome.foriss...@linaro.org> --- drivers/mmc/host/dw_mmc-k3.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/drivers/mm

[PATCH] mmc: dw_mmc: k3: add MMC_CAP_CMD23

2016-06-01 Thread Jerome Forissier
Enables RPMB support for the on-board eMMC of the HiKey board as well as for eMMC modules connected to the microSD slot. Signed-off-by: Jerome Forissier --- drivers/mmc/host/dw_mmc-k3.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/drivers/mmc/host/dw_mmc-k3.c b/drivers/mmc/host

Re: [PATCH 1/3] mtd: hisilicon: add device tree node for nand controller

2014-07-09 Thread Jerome FORISSIER
On 30-Jun-14 10:03, Zhou Wang wrote: > Signed-off-by: Zhou Wang > --- > arch/arm/boot/dts/hip04.dtsi | 31 +++ > 1 file changed, 31 insertions(+) > > diff --git a/arch/arm/boot/dts/hip04.dtsi b/arch/arm/boot/dts/hip04.dtsi > index abb42ca..e63fc61 100644 > ---

Re: [PATCH 1/3] mtd: hisilicon: add device tree node for nand controller

2014-07-09 Thread Jerome FORISSIER
On 30-Jun-14 10:03, Zhou Wang wrote: Signed-off-by: Zhou Wang wangzhou@gmail.com --- arch/arm/boot/dts/hip04.dtsi | 31 +++ 1 file changed, 31 insertions(+) diff --git a/arch/arm/boot/dts/hip04.dtsi b/arch/arm/boot/dts/hip04.dtsi index abb42ca..e63fc61