All of lore.kernel.org
 help / color / mirror / Atom feed
From: Matthias Brugger <matthias.bgg@gmail.com>
To: Fabien Parent <fparent@baylibre.com>,
	Enric Balletbo i Serra <enric.balletbo@collabora.com>
Cc: linux-kernel <linux-kernel@vger.kernel.org>,
	drinkcat@chromium.org, hsinyi@chromium.org,
	Collabora Kernel ML <kernel@collabora.com>,
	weiyi.lu@mediatek.com, Matthias Brugger <mbrugger@suse.com>,
	Joerg Roedel <jroedel@suse.de>,
	Miles Chen <miles.chen@mediatek.com>,
	Linux ARM <linux-arm-kernel@lists.infradead.org>,
	"moderated list:ARM/Mediatek SoC support" 
	<linux-mediatek@lists.infradead.org>
Subject: Re: [PATCH v3 05/16] soc: mediatek: pm_domains: Make bus protection generic
Date: Thu, 29 Oct 2020 15:49:04 +0100	[thread overview]
Message-ID: <57c0aaf5-9650-a0ad-1cbe-8e89c2d9a669@gmail.com> (raw)
In-Reply-To: <CAOwMV_xJez8WB=XjBSWfRmQ7x6265+2iJ+nLN4BR5YjoQzingQ@mail.gmail.com>



On 27/10/2020 13:57, Fabien Parent wrote:
>> -               ret = mtk_infracfg_set_bus_protection(pd->infracfg,
>> -                                                     bpd[i].bus_prot_mask,
>> -                                                     bpd[i].bus_prot_reg_update);
> 
> [snip]
> 
>> -               ret = mtk_infracfg_clear_bus_protection(pd->infracfg,
>> -                                                       bpd[i].bus_prot_mask,
>> -                                                       bpd[i].bus_prot_reg_update);
> 
> Since you got rid of all the dependencies to mtk-infracfg.c, maybe you
> can also remove the "depends on MTK_INFRACFG" in the Kconfig.
> 

We still need that file for the old driver.

Regards,
Matthias

WARNING: multiple messages have this Message-ID (diff)
From: Matthias Brugger <matthias.bgg@gmail.com>
To: Fabien Parent <fparent@baylibre.com>,
	Enric Balletbo i Serra <enric.balletbo@collabora.com>
Cc: Matthias Brugger <mbrugger@suse.com>,
	weiyi.lu@mediatek.com,
	linux-kernel <linux-kernel@vger.kernel.org>,
	Miles Chen <miles.chen@mediatek.com>,
	drinkcat@chromium.org,
	"moderated list:ARM/Mediatek SoC support"
	<linux-mediatek@lists.infradead.org>,
	Joerg Roedel <jroedel@suse.de>,
	hsinyi@chromium.org, Collabora Kernel ML <kernel@collabora.com>,
	Linux ARM <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH v3 05/16] soc: mediatek: pm_domains: Make bus protection generic
Date: Thu, 29 Oct 2020 15:49:04 +0100	[thread overview]
Message-ID: <57c0aaf5-9650-a0ad-1cbe-8e89c2d9a669@gmail.com> (raw)
In-Reply-To: <CAOwMV_xJez8WB=XjBSWfRmQ7x6265+2iJ+nLN4BR5YjoQzingQ@mail.gmail.com>



On 27/10/2020 13:57, Fabien Parent wrote:
>> -               ret = mtk_infracfg_set_bus_protection(pd->infracfg,
>> -                                                     bpd[i].bus_prot_mask,
>> -                                                     bpd[i].bus_prot_reg_update);
> 
> [snip]
> 
>> -               ret = mtk_infracfg_clear_bus_protection(pd->infracfg,
>> -                                                       bpd[i].bus_prot_mask,
>> -                                                       bpd[i].bus_prot_reg_update);
> 
> Since you got rid of all the dependencies to mtk-infracfg.c, maybe you
> can also remove the "depends on MTK_INFRACFG" in the Kconfig.
> 

We still need that file for the old driver.

Regards,
Matthias

_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek

WARNING: multiple messages have this Message-ID (diff)
From: Matthias Brugger <matthias.bgg@gmail.com>
To: Fabien Parent <fparent@baylibre.com>,
	Enric Balletbo i Serra <enric.balletbo@collabora.com>
Cc: Matthias Brugger <mbrugger@suse.com>,
	weiyi.lu@mediatek.com,
	linux-kernel <linux-kernel@vger.kernel.org>,
	Miles Chen <miles.chen@mediatek.com>,
	drinkcat@chromium.org,
	"moderated list:ARM/Mediatek SoC support"
	<linux-mediatek@lists.infradead.org>,
	Joerg Roedel <jroedel@suse.de>,
	hsinyi@chromium.org, Collabora Kernel ML <kernel@collabora.com>,
	Linux ARM <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH v3 05/16] soc: mediatek: pm_domains: Make bus protection generic
Date: Thu, 29 Oct 2020 15:49:04 +0100	[thread overview]
Message-ID: <57c0aaf5-9650-a0ad-1cbe-8e89c2d9a669@gmail.com> (raw)
In-Reply-To: <CAOwMV_xJez8WB=XjBSWfRmQ7x6265+2iJ+nLN4BR5YjoQzingQ@mail.gmail.com>



On 27/10/2020 13:57, Fabien Parent wrote:
>> -               ret = mtk_infracfg_set_bus_protection(pd->infracfg,
>> -                                                     bpd[i].bus_prot_mask,
>> -                                                     bpd[i].bus_prot_reg_update);
> 
> [snip]
> 
>> -               ret = mtk_infracfg_clear_bus_protection(pd->infracfg,
>> -                                                       bpd[i].bus_prot_mask,
>> -                                                       bpd[i].bus_prot_reg_update);
> 
> Since you got rid of all the dependencies to mtk-infracfg.c, maybe you
> can also remove the "depends on MTK_INFRACFG" in the Kconfig.
> 

We still need that file for the old driver.

Regards,
Matthias

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2020-10-29 14:49 UTC|newest]

Thread overview: 101+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-10-26 17:55 [PATCH v3 00/16] soc: mediatek: pm-domains: Add new driver for SCPSYS power domains controller Enric Balletbo i Serra
2020-10-26 17:55 ` Enric Balletbo i Serra
2020-10-26 17:55 ` Enric Balletbo i Serra
2020-10-26 17:55 ` [PATCH v3 01/16] dt-bindings: power: Add bindings for the Mediatek " Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-28 13:55   ` Rob Herring
2020-10-28 13:55     ` Rob Herring
2020-10-28 13:55     ` Rob Herring
2020-10-28 13:57   ` Rob Herring
2020-10-28 13:57     ` Rob Herring
2020-10-28 13:57     ` Rob Herring
2020-10-26 17:55 ` [PATCH v3 02/16] soc: mediatek: Add MediaTek SCPSYS power domains Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-27  0:55   ` Nicolas Boichat
2020-10-27  0:55     ` Nicolas Boichat
2020-10-27  0:55     ` Nicolas Boichat
2020-10-30 10:54     ` Enric Balletbo i Serra
2020-10-30 10:54       ` Enric Balletbo i Serra
2020-10-30 10:54       ` Enric Balletbo i Serra
2020-10-26 17:55 ` [PATCH v3 03/16] arm64: dts: mediatek: Add mt8173 power domain controller Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-26 17:55 ` [PATCH v3 04/16] soc: mediatek: pm-domains: Add bus protection protocol Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-27  0:58   ` Nicolas Boichat
2020-10-27  0:58     ` Nicolas Boichat
2020-10-27  0:58     ` Nicolas Boichat
2020-10-27 11:07   ` Weiyi Lu
2020-10-27 11:07     ` Weiyi Lu
2020-10-27 11:07     ` Weiyi Lu
2020-10-30 10:55     ` Enric Balletbo i Serra
2020-10-30 10:55       ` Enric Balletbo i Serra
2020-10-30 10:55       ` Enric Balletbo i Serra
2020-10-26 17:55 ` [PATCH v3 05/16] soc: mediatek: pm_domains: Make bus protection generic Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-27  2:41   ` Nicolas Boichat
2020-10-27  2:41     ` Nicolas Boichat
2020-10-27  2:41     ` Nicolas Boichat
2020-10-27 12:57   ` Fabien Parent
2020-10-27 12:57     ` Fabien Parent
2020-10-27 12:57     ` Fabien Parent
2020-10-29 14:49     ` Matthias Brugger [this message]
2020-10-29 14:49       ` Matthias Brugger
2020-10-29 14:49       ` Matthias Brugger
2020-10-26 17:55 ` [PATCH v3 06/16] soc: mediatek: pm-domains: Add SMI block as bus protection block Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-27  2:44   ` Nicolas Boichat
2020-10-27  2:44     ` Nicolas Boichat
2020-10-27  2:44     ` Nicolas Boichat
2020-10-30 10:56     ` Enric Balletbo i Serra
2020-10-30 10:56       ` Enric Balletbo i Serra
2020-10-30 10:56       ` Enric Balletbo i Serra
2020-10-26 17:55 ` [PATCH v3 07/16] soc: mediatek: pm-domains: Add extra sram control Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-27  2:47   ` Nicolas Boichat
2020-10-27  2:47     ` Nicolas Boichat
2020-10-27  2:47     ` Nicolas Boichat
2020-10-26 17:55 ` [PATCH v3 08/16] soc: mediatek: pm-domains: Add subsystem clocks Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-26 17:55 ` [PATCH v3 09/16] soc: mediatek: pm-domains: Allow bus protection to ignore clear ack Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-26 17:55 ` [PATCH v3 10/16] dt-bindings: power: Add MT8183 power domains Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-26 17:55 ` [PATCH v3 11/16] soc: mediatek: pm-domains: Add support for mt8183 Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-26 17:55 ` [PATCH v3 12/16] arm64: dts: mediatek: Add smi_common node for MT8183 Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-26 17:55 ` [PATCH v3 13/16] arm64: dts: mediatek: Add mt8183 power domains controller Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-26 17:55 ` [PATCH v3 14/16] dt-bindings: power: Add MT8192 power domains Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-26 17:55 ` [PATCH v3 15/16] soc: mediatek: pm-domains: Add default power off flag Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-27 10:53   ` Matthias Brugger
2020-10-27 10:53     ` Matthias Brugger
2020-10-27 10:53     ` Matthias Brugger
2020-10-27 11:18     ` Weiyi Lu
2020-10-27 11:18       ` Weiyi Lu
2020-10-29 14:51       ` Matthias Brugger
2020-10-29 14:51         ` Matthias Brugger
2020-10-29 14:51         ` Matthias Brugger
2020-10-30 11:17         ` Enric Balletbo i Serra
2020-10-30 11:17           ` Enric Balletbo i Serra
2020-10-30 11:17           ` Enric Balletbo i Serra
2020-10-26 17:55 ` [PATCH v3 16/16] soc: mediatek: pm-domains: Add support for mt8192 Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra
2020-10-26 17:55   ` Enric Balletbo i Serra

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=57c0aaf5-9650-a0ad-1cbe-8e89c2d9a669@gmail.com \
    --to=matthias.bgg@gmail.com \
    --cc=drinkcat@chromium.org \
    --cc=enric.balletbo@collabora.com \
    --cc=fparent@baylibre.com \
    --cc=hsinyi@chromium.org \
    --cc=jroedel@suse.de \
    --cc=kernel@collabora.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=mbrugger@suse.com \
    --cc=miles.chen@mediatek.com \
    --cc=weiyi.lu@mediatek.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.