Hi Naresh,
On Fri, Feb 18, 2022 at 3:36 PM Naresh Kamboju
<naresh.kamboju@xxxxxxxxxx> wrote:
>
> While building stable rc queues for arch arm64 on queue/5.15 and
> queue/5.16 the following build errors / warnings were noticed.
>
> ## Fails
> * arm64, build
> - gcc-11-defconfig-5e73d44a
>
> Committing details,
> optee: use driver internal tee_context for some rpc
> commit aceeafefff736057e8f93f19bbfbef26abd94604 upstream.
>
>
> build error / warning.
> drivers/tee/optee/core.c: In function 'optee_remove':
> drivers/tee/optee/core.c:591:9: error: implicit declaration of
> function 'teedev_close_context'; did you mean
> 'tee_client_close_context'? [-Werror=implicit-function-declaration]
> 591 | teedev_close_context(optee->ctx);
> | ^~~~~~~~~~~~~~~~~~~~
> | tee_client_close_context
> drivers/tee/optee/core.c: In function 'optee_probe':
> drivers/tee/optee/core.c:724:15: error: implicit declaration of
> function 'teedev_open' [-Werror=implicit-function-declaration]
> 724 | ctx = teedev_open(optee->teedev);
> | ^~~~~~~~~~~
> drivers/tee/optee/core.c:724:13: warning: assignment to 'struct
> tee_context *' from 'int' makes pointer from integer without a cast
> [-Wint-conversion]
> 724 | ctx = teedev_open(optee->teedev);
> | ^
> drivers/tee/optee/core.c:726:20: warning: operation on 'rc' may be
> undefined [-Wsequence-point]
> 726 | rc = rc = PTR_ERR(ctx);
> | ~~~^~~~~~~~~~~~~~~~~~~
> cc1: some warnings being treated as errors
>
>
>
It looks like 1e2c3ef0496e ("tee: export teedev_open() and
teedev_close_context()") is missing. I noted the dependency as:
Cc: stable@xxxxxxxxxxxxxxx # 1e2c3ef0496e tee: export
teedev_open() and teedev_close_context()
in the commit. Perhaps I've misunderstood how this is supposed to be done.
Thanks,
Jens
> --
> Linaro LKFT
> https://lkft.linaro.org
>
> [1] https://qa-reports.linaro.org/lkft/linux-stable-rc-queues-queue_5.16/build/v5.16.10-87-gb5b4ed62d295
> [2] https://qa-reports.linaro.org/lkft/linux-stable-rc-queues-queue_5.16/build/v5.16.10-87-gb5b4ed62d295
