public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
From: Sudeep Holla <sudeep.holla@arm.com>
To: Jens Wiklander <jens.wiklander@linaro.org>
Cc: linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	op-tee@lists.trustedfirmware.org,
	Sumit Garg <sumit.garg@linaro.org>,
	Marc Bonnici <marc.bonnici@arm.com>,
	Jerome Forissier <jerome@forissier.org>,
	sughosh.ganu@linaro.org
Subject: Re: [PATCH v6 5/5] optee: add FF-A support
Date: Fri, 8 Oct 2021 14:27:32 +0100	[thread overview]
Message-ID: <20211008132732.hbmkd3hftdydtrsc@bogus> (raw)
In-Reply-To: <20211006070902.2531311-6-jens.wiklander@linaro.org>

On Wed, Oct 06, 2021 at 09:09:02AM +0200, Jens Wiklander wrote:
> Adds support for using FF-A [1] as transport to the OP-TEE driver.
>
> Introduces struct optee_msg_param_fmem which carries all information
> needed when OP-TEE is calling FFA_MEM_RETRIEVE_REQ to get the shared
> memory reference mapped by the hypervisor in S-EL2. Register usage is
> also updated to include the information needed.
>
> The FF-A part of this driver is enabled if CONFIG_ARM_FFA_TRANSPORT is
> enabled.
>

I am not sure if I missed this with earlier version but I see the below
warning the second time I insert the optee module. I am sure I tested it
with previous version when I was fixing issues with FF-A as a module.

Not sure if I am missing something in my steps.

Regards,
Sudeep

-->8

 sysfs: cannot create duplicate filename '/devices/optee-ta-d96a5b40-c3e5-21e3-8794-1002a5d5c61b'
 CPU: 7 PID: 181 Comm: modprobe Not tainted 5.15.0-rc4+ #6
 Hardware name: FVP Base RevC (DT)
 Call trace:
  dump_backtrace+0x0/0x1d4
  show_stack+0x18/0x64
  dump_stack_lvl+0x64/0x7c
  dump_stack+0x18/0x38
  sysfs_create_dir_ns+0xe4/0x140
  kobject_add_internal+0x170/0x354
  kobject_add+0x94/0x100
  device_add+0x178/0x5f0
  device_register+0x20/0x30
  optee_enumerate_devices+0x204/0x2fc [optee]
  optee_ffa_probe+0x370/0x3bc [optee]
  ffa_device_probe+0x1c/0x28 [ffa_module]
  really_probe+0xc4/0x2f4
  __driver_probe_device+0xb4/0xe0
  driver_probe_device+0x40/0x134
  __driver_attach+0xe0/0x180 
  bus_for_each_dev+0x80/0xcc
  driver_attach+0x24/0x30
  bus_add_driver+0xfc/0x1dc
  driver_register+0x78/0x110
  ffa_driver_register+0x2c/0x40 [ffa_module]
  optee_ffa_abi_register+0x28/0x34 [optee]
  init_module+0x38/0x60 [optee]
  do_one_initcall+0xbc/0x230
  do_init_module+0x58/0x304
  load_module+0x1cb8/0x1f78

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2021-10-08 13:30 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-06  7:08 [PATCH v6 0/5] Add FF-A support in OP-TEE driver Jens Wiklander
2021-10-06  7:08 ` [PATCH v6 1/5] tee: add sec_world_id to struct tee_shm Jens Wiklander
2021-10-06  8:36   ` Jerome Forissier
2021-10-06 10:58     ` Jens Wiklander
2021-10-06  7:08 ` [PATCH v6 2/5] optee: simplify optee_release() Jens Wiklander
2021-10-06  7:09 ` [PATCH v6 3/5] optee: refactor driver with internal callbacks Jens Wiklander
2021-10-06  7:09 ` [PATCH v6 4/5] optee: isolate smc abi Jens Wiklander
2021-10-18  6:38   ` Jens Wiklander
2021-10-06  7:09 ` [PATCH v6 5/5] optee: add FF-A support Jens Wiklander
2021-10-08 13:27   ` Sudeep Holla [this message]
2021-10-08 16:32     ` Sudeep Holla
2021-10-08 19:21       ` Jerome Forissier
2021-10-11  9:29         ` Jens Wiklander
2021-10-12  7:35           ` Sumit Garg

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20211008132732.hbmkd3hftdydtrsc@bogus \
    --to=sudeep.holla@arm.com \
    --cc=jens.wiklander@linaro.org \
    --cc=jerome@forissier.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=marc.bonnici@arm.com \
    --cc=op-tee@lists.trustedfirmware.org \
    --cc=sughosh.ganu@linaro.org \
    --cc=sumit.garg@linaro.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox