Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <5171511b-4021-589e-3606-10a690b48850@gmail.com>
Date:   Sun, 2 Apr 2023 18:52:21 +0200
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.6.1
Subject: Re: [PATCH v2 0/9] soc/mediatek: Cleanup mmsys, mutex, cmdq header
Content-Language: en-US
To:     AngeloGioacchino Del Regno 
        <angelogioacchino.delregno@collabora.com>
Cc:     jason-jh.lin@mediatek.com, chunkuang.hu@kernel.org,
        linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-mediatek@lists.infradead.org
References: <20230222094253.23678-1-angelogioacchino.delregno@collabora.com>
From:   Matthias Brugger <matthias.bgg@gmail.com>
In-Reply-To: <20230222094253.23678-1-angelogioacchino.delregno@collabora.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk


On 22/02/2023 10:42, AngeloGioacchino Del Regno wrote:
> Changes in v2:
>   - Rebased onto Matthias' v6.3-tmp/soc branch for VPP mutex conflict
>     on patch [5/9]
> 
> This series adds a proper remove callback for mtk-mmsys, adds light
> cleanups on mtk-mutex and, more importantly, adds a cleanup for the
> CMDQ helpers.
> 
> Special mention goes to the latter: this is just in preparation for
> a larger cleanup that I plan to do as soon as possible on other cmdq
> helpers users, being mediatek-drm and mtk-mdp3; that's not performed
> right now because, in the case of drm/mdp3, the cleanup is not entirely
> trivial.
> 
> AngeloGioacchino Del Regno (9):
>    soc: mediatek: mtk-mmsys: Add .remove() callback
>    soc: mediatek: mtk-mmsys: Use module_platform_driver() macro
>    soc: mediatek: mtk-mmsys: Compress of_device_id array entries
>    soc: mediatek: mtk-mmsys: Add MODULE_DEVICE_TABLE() to allow auto-load
>    soc: mediatek: mtk-mutex: Compress of_device_id array entries
>    soc: mediatek: mtk-mutex: Replace max handles number with definition
>    soc: mediatek: mtk-mutex: Use module_platform_driver() macro
>    soc: mediatek: cmdq: Add inline functions for !CONFIG_MTK_CMDQ
>    soc: mediatek: Cleanup ifdefs for IS_REACHABLE(CONFIG_MTK_CMDQ)
> 
>   drivers/soc/mediatek/mtk-mmsys.c      | 135 +++++++++-----------------
>   drivers/soc/mediatek/mtk-mutex.c      |  80 +++++----------
>   include/linux/soc/mediatek/mtk-cmdq.h | 114 ++++++++++++++++++++++
>   3 files changed, 182 insertions(+), 147 deletions(-)
> 

Applied, thanks!