Re: [PATCH v5 2/3] iommu/mediatek: Introduce new flag TF_PORT_TO_ADDR_MT8173

[AngeloGioacchino Del Regno]

Il 13/09/22 16:00, Krzysztof Kozlowski ha scritto:
> On 13/09/2022 14:24, AngeloGioacchino Del Regno wrote:
> > In preparation for adding support for MT6795, add a new flag named
> > TF_PORT_TO_ADDR_MT8173 and use that instead of checking for m4u_plat
> > type in mtk_iommu_hw_init() to avoid seeing a long list of m4u_plat
> > checks there in the future.
> >
> > Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@xxxxxxxxxxxxx>
> > Reviewed-by: Yong Wu <yong.wu@xxxxxxxxxxxx>
> > ---
> >   drivers/iommu/mtk_iommu.c | 6 ++++--
> >   drivers/memory/mtk-smi.c  | 1 +
> >   2 files changed, 5 insertions(+), 2 deletions(-)

..snip..

> > diff --git a/drivers/memory/mtk-smi.c b/drivers/memory/mtk-smi.c
> > index 5a9754442bc7..cd415ed1f4ca 100644
> > --- a/drivers/memory/mtk-smi.c
> > +++ b/drivers/memory/mtk-smi.c
> > @@ -462,6 +462,7 @@ static int mtk_smi_larb_sleep_ctrl_enable(struct mtk_smi_larb *larb)
> >   	if (ret) {
> >   		/* TODO: Reset this larb if it fails here. */
> >   		dev_err(larb->smi.dev, "sleep ctrl is not ready(0x%x).\n", tmp);
> > +		ret = -EAGAIN;
>
> Doesn't look related nor explained in commit msg.

This is because it's not related.. not explained... and embarassing, as something
went wrong during the rebase.

Many thanks for pinging me about that, I'm immediately sending a new version.

Regards,
Angelo