On Fri, Mar 24, 2023 at 11:15 AM Linus Walleij linus.walleij@linaro.org wrote:

Like the other calls in this function virt_to_page() expects a pointer, not an integer.

However since many architectures implement virt_to_pfn() as a macro, this function becomes polymorphic and accepts both a (unsigned long) and a (void *).

Fix this up with an explicit cast.

Signed-off-by: Linus Walleij linus.walleij@linaro.org

I'm picking up this.

Thanks, Jens

---

drivers/tee/tee_shm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/tee/tee_shm.c b/drivers/tee/tee_shm.c index b1c6231defad..673cf0359494 100644 --- a/drivers/tee/tee_shm.c +++ b/drivers/tee/tee_shm.c @@ -32,7 +32,7 @@ static int shm_get_kernel_pages(unsigned long start, size_t page_count, is_kmap_addr((void *)start))) return -EINVAL;

•   page = virt_to_page(start);
  

•   page = virt_to_page((void *)start);
    for (n = 0; n < page_count; n++) {
            pages[n] = page + n;
            get_page(pages[n]);
  

-- 2.34.1