Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
References: <20210202194752.247301-1-swboyd@chromium.org>
In-Reply-To: <20210202194752.247301-1-swboyd@chromium.org>
From:   Fritz Koenig <frkoenig@chromium.org>
Date:   Fri, 5 Feb 2021 15:20:53 -0800
Message-ID: <CAMfZQbyM-3mMchUgFZgiX48x7WG=BG=dZ6aBwAyu-K_y05=qkw@mail.gmail.com>
Subject: Re: [PATCH -next] media: venus: Include io.h for memremap()
To:     Stephen Boyd <swboyd@chromium.org>
Cc:     Stanimir Varbanov <stanimir.varbanov@linaro.org>,
        LKML <linux-kernel@vger.kernel.org>,
        Mauro Carvalho Chehab <mchehab@kernel.org>,
        linux-arm-msm@vger.kernel.org,
        Linux Media Mailing List <linux-media@vger.kernel.org>,
        Dikshita Agarwal <dikshita@codeaurora.org>
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

On Tue, Feb 2, 2021 at 11:51 AM Stephen Boyd <swboyd@chromium.org> wrote:
>
> This file uses memremap() now, so we should include io.h instead of
> relying on any sort of implicit include elsewhere.
>
> Cc: Dikshita Agarwal <dikshita@codeaurora.org>
> Fixes: 0ca0ca980505 ("media: venus: core: add support to dump FW region")
> Signed-off-by: Stephen Boyd <swboyd@chromium.org>
> ---
>  drivers/media/platform/qcom/venus/core.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/drivers/media/platform/qcom/venus/core.c b/drivers/media/platform/qcom/venus/core.c
> index 1471c7f9c89d..915b3ed8ed64 100644
> --- a/drivers/media/platform/qcom/venus/core.c
> +++ b/drivers/media/platform/qcom/venus/core.c
> @@ -5,6 +5,7 @@
>   */
>  #include <linux/init.h>
>  #include <linux/interconnect.h>
> +#include <linux/io.h>
>  #include <linux/ioctl.h>
>  #include <linux/delay.h>
>  #include <linux/devcoredump.h>
>
> base-commit: 0ca0ca9805055bb0efc16890f9d6433c65bd07cc
> --
> https://chromeos.dev

Reviewed-by: Fritz Koenig <frkoenig@chromium.org>