ALT Linux kernel packages development
 help / color / mirror / Atom feed
From: Anton Midyukov <antohami@altlinux.org>
To: devel-kernel@lists.altlinux.org
Subject: Re: [d-kernel] [PATCH 08/12] soc: bcm: bcm2835-power: Add support for BCM2711's RPiVid ASB
Date: Mon, 20 Apr 2026 09:01:17 +0300
Message-ID: <c7b976c5-9614-4be3-a08c-6616c5625286@altlinux.org> (raw)
In-Reply-To: <b3f4034b-2a1d-4ddc-9c93-b7a2e34eeb04@altlinux.org>

19.04.2026 08:08, Anton Midyukov пишет:
> 18.04.2026 21:57, Vitaly Chikunov пишет:
>>
>> Anton,
>>
>> Этот патч в std-def/p10 вызывает merge conflict с v5.10.253.
>>
> 
> Вроде бы конфликт разрешили. Проверю на работоспособность.
> 

Протестировал 3D на Raspberry Pi 4B с этим ядром на свежесобранном стартерките p10.
v3d работает, проблем не обнаружил.

>> On Tue, Mar 22, 2022 at 10:36:11PM +0700, antohami@basealt.ru wrote:
>>> From: Nicolas Saenz Julienne <nsaenzjulienne@suse.de>
>>>
>>> In BCM2711 the new RPiVid ASB took over V3D. The old ASB is still present
>>> with the ISP and H264 bits, and V3D is in the same place in the new ASB
>>> as the old one.
>>>
>>> Use the fact that 'pm->rpivid_asb' is populated as a hint that we're on
>>> BCM2711. On top of that introduce the macro ASB_BASE() which will select
>>> the correct ASB register base, based on whether we're trying to access
>>> V3D and which platform we're on.
>>>
>>> Signed-off-by: Nicolas Saenz Julienne <nsaenzjulienne@suse.de>
>>> ---
>>>  drivers/soc/bcm/bcm2835-power.c | 66 ++++++++++++++++++++-------------
>>>  1 file changed, 41 insertions(+), 25 deletions(-)
>>>
>>> diff --git a/drivers/soc/bcm/bcm2835-power.c b/drivers/soc/bcm/bcm2835-power.c
>>> index 1e0041ec8132..eea31f75dc64 100644
>>> --- a/drivers/soc/bcm/bcm2835-power.c
>>> +++ b/drivers/soc/bcm/bcm2835-power.c
>>> @@ -126,8 +126,9 @@
>>>  
>>>  #define ASB_AXI_BRDG_ID			0x20
>>>  
>>> -#define ASB_READ(reg) readl(power->asb + (reg))
>>> -#define ASB_WRITE(reg, val) writel(PM_PASSWORD | (val), power->asb + (reg))
>>> +#define ASB_BASE(is_v3d) (is_v3d && power->rpivid_asb ? power->rpivid_asb : power->asb)
>>> +#define ASB_READ(reg, is_v3d) readl(ASB_BASE(is_v3d) + (reg))
>>> +#define ASB_WRITE(reg, val, is_v3d) writel(PM_PASSWORD | (val), ASB_BASE(is_v3d) + (reg))
>>>  
>>>  struct bcm2835_power_domain {
>>>  	struct generic_pm_domain base;
>>> @@ -142,13 +143,16 @@ struct bcm2835_power {
>>>  	void __iomem		*base;
>>>  	/* AXI Async bridge registers. */
>>>  	void __iomem		*asb;
>>> +	/* RPiVid bridge registers. */
>>> +	void __iomem		*rpivid_asb;
>>>  
>>>  	struct genpd_onecell_data pd_xlate;
>>>  	struct bcm2835_power_domain domains[BCM2835_POWER_DOMAIN_COUNT];
>>>  	struct reset_controller_dev reset;
>>>  };
>>>  
>>> -static int bcm2835_asb_enable(struct bcm2835_power *power, u32 reg)
>>> +static int bcm2835_asb_enable(struct bcm2835_power *power, u32 reg,
>>> +			      bool is_v3d)
>>>  {
>>>  	u64 start;
>>>  
>>> @@ -158,8 +162,8 @@ static int bcm2835_asb_enable(struct bcm2835_power *power, u32 reg)
>>>  	start = ktime_get_ns();
>>>  
>>>  	/* Enable the module's async AXI bridges. */
>>> -	ASB_WRITE(reg, ASB_READ(reg) & ~ASB_REQ_STOP);
>>> -	while (ASB_READ(reg) & ASB_ACK) {
>>> +	ASB_WRITE(reg, ASB_READ(reg, is_v3d) & ~ASB_REQ_STOP, is_v3d);
>>> +	while (ASB_READ(reg, is_v3d) & ASB_ACK) {
>>>  		cpu_relax();
>>>  		if (ktime_get_ns() - start >= 1000)
>>>  			return -ETIMEDOUT;
>>> @@ -168,7 +172,8 @@ static int bcm2835_asb_enable(struct bcm2835_power *power, u32 reg)
>>>  	return 0;
>>>  }
>>>  
>>> -static int bcm2835_asb_disable(struct bcm2835_power *power, u32 reg)
>>> +static int bcm2835_asb_disable(struct bcm2835_power *power, u32 reg,
>>> +			       bool is_v3d)
>>>  {
>>>  	u64 start;
>>>  
>>> @@ -178,8 +183,8 @@ static int bcm2835_asb_disable(struct bcm2835_power *power, u32 reg)
>>>  	start = ktime_get_ns();
>>>  
>>>  	/* Enable the module's async AXI bridges. */
>>> -	ASB_WRITE(reg, ASB_READ(reg) | ASB_REQ_STOP);
>>> -	while (!(ASB_READ(reg) & ASB_ACK)) {
>>> +	ASB_WRITE(reg, ASB_READ(reg, is_v3d) | ASB_REQ_STOP, is_v3d);
>>> +	while (!(ASB_READ(reg, is_v3d) & ASB_ACK)) {
>>>  		cpu_relax();
>>>  		if (ktime_get_ns() - start >= 1000)
>>>  			return -ETIMEDOUT;
>>> @@ -274,7 +279,8 @@ static int bcm2835_asb_power_on(struct bcm2835_power_domain *pd,
>>>  				u32 pm_reg,
>>>  				u32 asb_m_reg,
>>>  				u32 asb_s_reg,
>>> -				u32 reset_flags)
>>> +				u32 reset_flags,
>>> +				bool is_v3d)
>>>  {
>>>  	struct bcm2835_power *power = pd->power;
>>>  	int ret;
>>> @@ -301,13 +307,13 @@ static int bcm2835_asb_power_on(struct bcm2835_power_domain *pd,
>>>  		goto err_enable_resets;
>>>  	}
>>>  
>>> -	ret = bcm2835_asb_enable(power, asb_m_reg);
>>> +	ret = bcm2835_asb_enable(power, asb_m_reg, is_v3d);
>>>  	if (ret) {
>>>  		dev_err(power->dev, "Failed to enable ASB master for %s\n",
>>>  			pd->base.name);
>>>  		goto err_disable_clk;
>>>  	}
>>> -	ret = bcm2835_asb_enable(power, asb_s_reg);
>>> +	ret = bcm2835_asb_enable(power, asb_s_reg, is_v3d);
>>>  	if (ret) {
>>>  		dev_err(power->dev, "Failed to enable ASB slave for %s\n",
>>>  			pd->base.name);
>>> @@ -317,7 +323,7 @@ static int bcm2835_asb_power_on(struct bcm2835_power_domain *pd,
>>>  	return 0;
>>>  
>>>  err_disable_asb_master:
>>> -	bcm2835_asb_disable(power, asb_m_reg);
>>> +	bcm2835_asb_disable(power, asb_m_reg, is_v3d);
>>>  err_disable_clk:
>>>  	clk_disable_unprepare(pd->clk);
>>>  err_enable_resets:
>>> @@ -329,22 +335,23 @@ static int bcm2835_asb_power_off(struct bcm2835_power_domain *pd,
>>>  				 u32 pm_reg,
>>>  				 u32 asb_m_reg,
>>>  				 u32 asb_s_reg,
>>> -				 u32 reset_flags)
>>> +				 u32 reset_flags,
>>> +				 bool is_v3d)
>>>  {
>>>  	struct bcm2835_power *power = pd->power;
>>>  	int ret;
>>>  
>>> -	ret = bcm2835_asb_disable(power, asb_s_reg);
>>> +	ret = bcm2835_asb_disable(power, asb_s_reg, is_v3d);
>>>  	if (ret) {
>>>  		dev_warn(power->dev, "Failed to disable ASB slave for %s\n",
>>>  			 pd->base.name);
>>>  		return ret;
>>>  	}
>>> -	ret = bcm2835_asb_disable(power, asb_m_reg);
>>> +	ret = bcm2835_asb_disable(power, asb_m_reg, is_v3d);
>>>  	if (ret) {
>>>  		dev_warn(power->dev, "Failed to disable ASB master for %s\n",
>>>  			 pd->base.name);
>>> -		bcm2835_asb_enable(power, asb_s_reg);
>>> +		bcm2835_asb_enable(power, asb_s_reg, is_v3d);
>>>  		return ret;
>>>  	}
>>>  
>>> @@ -369,7 +376,7 @@ static int bcm2835_power_pd_power_on(struct generic_pm_domain *domain)
>>>  	case BCM2835_POWER_DOMAIN_GRAFX_V3D:
>>>  		return bcm2835_asb_power_on(pd, PM_GRAFX,
>>>  					    ASB_V3D_M_CTRL, ASB_V3D_S_CTRL,
>>> -					    PM_V3DRSTN);
>>> +					    PM_V3DRSTN, true);
>>>  
>>>  	case BCM2835_POWER_DOMAIN_IMAGE:
>>>  		return bcm2835_power_power_on(pd, PM_IMAGE);
>>> @@ -377,17 +384,17 @@ static int bcm2835_power_pd_power_on(struct generic_pm_domain *domain)
>>>  	case BCM2835_POWER_DOMAIN_IMAGE_PERI:
>>>  		return bcm2835_asb_power_on(pd, PM_IMAGE,
>>>  					    0, 0,
>>> -					    PM_PERIRSTN);
>>> +					    PM_PERIRSTN, false);
>>>  
>>>  	case BCM2835_POWER_DOMAIN_IMAGE_ISP:
>>>  		return bcm2835_asb_power_on(pd, PM_IMAGE,
>>>  					    ASB_ISP_M_CTRL, ASB_ISP_S_CTRL,
>>> -					    PM_ISPRSTN);
>>> +					    PM_ISPRSTN, false);
>>>  
>>>  	case BCM2835_POWER_DOMAIN_IMAGE_H264:
>>>  		return bcm2835_asb_power_on(pd, PM_IMAGE,
>>>  					    ASB_H264_M_CTRL, ASB_H264_S_CTRL,
>>> -					    PM_H264RSTN);
>>> +					    PM_H264RSTN, false);
>>>  
>>>  	case BCM2835_POWER_DOMAIN_USB:
>>>  		PM_WRITE(PM_USB, PM_USB_CTRLEN);
>>> @@ -435,7 +442,7 @@ static int bcm2835_power_pd_power_off(struct generic_pm_domain *domain)
>>>  	case BCM2835_POWER_DOMAIN_GRAFX_V3D:
>>>  		return bcm2835_asb_power_off(pd, PM_GRAFX,
>>>  					     ASB_V3D_M_CTRL, ASB_V3D_S_CTRL,
>>> -					     PM_V3DRSTN);
>>> +					     PM_V3DRSTN, true);
>>>  
>>>  	case BCM2835_POWER_DOMAIN_IMAGE:
>>>  		return bcm2835_power_power_off(pd, PM_IMAGE);
>>> @@ -443,17 +450,17 @@ static int bcm2835_power_pd_power_off(struct generic_pm_domain *domain)
>>>  	case BCM2835_POWER_DOMAIN_IMAGE_PERI:
>>>  		return bcm2835_asb_power_off(pd, PM_IMAGE,
>>>  					     0, 0,
>>> -					     PM_PERIRSTN);
>>> +					     PM_PERIRSTN, false);
>>>  
>>>  	case BCM2835_POWER_DOMAIN_IMAGE_ISP:
>>>  		return bcm2835_asb_power_off(pd, PM_IMAGE,
>>>  					     ASB_ISP_M_CTRL, ASB_ISP_S_CTRL,
>>> -					     PM_ISPRSTN);
>>> +					     PM_ISPRSTN, false);
>>>  
>>>  	case BCM2835_POWER_DOMAIN_IMAGE_H264:
>>>  		return bcm2835_asb_power_off(pd, PM_IMAGE,
>>>  					     ASB_H264_M_CTRL, ASB_H264_S_CTRL,
>>> -					     PM_H264RSTN);
>>> +					     PM_H264RSTN, false);
>>>  
>>>  	case BCM2835_POWER_DOMAIN_USB:
>>>  		PM_WRITE(PM_USB, 0);
>>> @@ -626,13 +633,22 @@ static int bcm2835_power_probe(struct platform_device *pdev)
>>>  	power->dev = dev;
>>>  	power->base = pm->base;
>>>  	power->asb = pm->asb;
>>> +	power->rpivid_asb = pm->rpivid_asb;
>>>  
>>> -	id = ASB_READ(ASB_AXI_BRDG_ID);
>>> +	id = ASB_READ(ASB_AXI_BRDG_ID, false);
>>>  	if (id != 0x62726467 /* "BRDG" */) {
>>>  		dev_err(dev, "ASB register ID returned 0x%08x\n", id);
>>>  		return -ENODEV;
>>>  	}
>>>  
>>> +	if (pm->rpivid_asb) {
>>> +		id = ASB_READ(ASB_AXI_BRDG_ID, true);
>>> +		if (id != 0x62726467 /* "BRDG" */) {
>>> +			dev_err(dev, "RPiVid ASB register ID returned 0x%08x\n", id);
>>> +			return -ENODEV;
>>> +		}
>>> +	}
>>> +
>>>  	power->pd_xlate.domains = devm_kcalloc(dev,
>>>  					       ARRAY_SIZE(power_domain_names),
>>>  					       sizeof(*power->pd_xlate.domains),
>>> -- 
>>> 2.33.0
>>>
>>> _______________________________________________
>>> devel-kernel mailing list
>>> devel-kernel@lists.altlinux.org
>>> https://lists.altlinux.org/mailman/listinfo/devel-kernel
> 
> 

-- 
best regards, Anton Midyukov <antohami@altlinux.org>



  reply	other threads:[~2026-04-20  6:01 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-03-22 15:36 [d-kernel] [PATCH 00/12] *** Enable v3d on Raspberry Pi 4 for kernel 5.10 std-def *** antohami
2022-03-22 15:36 ` [d-kernel] [PATCH 01/12] dt-bindings: soc: bcm: bcm2835-pm: Convert bindings to DT schema antohami
2022-03-22 15:36 ` [d-kernel] [PATCH 02/12] dt-bindings: soc: bcm: bcm2835-pm: Introduce reg-names antohami
2022-03-22 15:36 ` [d-kernel] [PATCH 03/12] dt-bindings: soc: bcm: bcm2835-pm: Add support for bcm2711 antohami
2022-03-22 15:36 ` [d-kernel] [PATCH 04/12] ARM: dts: bcm2835/bcm2711: Introduce reg-names in watchdog node antohami
2022-03-22 15:36 ` [d-kernel] [PATCH 05/12] ARM: dts: bcm2711: Use proper compatible in PM/Watchdog node antohami
2022-03-22 15:36 ` [d-kernel] [PATCH 06/12] mfd: bcm2835-pm: Use 'reg-names' to get resources antohami
2022-03-22 15:36 ` [d-kernel] [PATCH 07/12] mfd: bcm2835-pm: Add support for BCM2711 antohami
2022-03-22 15:36 ` [d-kernel] [PATCH 08/12] soc: bcm: bcm2835-power: Add support for BCM2711's RPiVid ASB antohami
2026-04-18 18:57   ` Vitaly Chikunov
2026-04-19  5:08     ` Anton Midyukov
2026-04-20  6:01       ` Anton Midyukov [this message]
2026-04-20  6:14         ` Vitaly Chikunov
2022-03-22 15:36 ` [d-kernel] [PATCH 09/12] soc: bcm: bcm2835-power: Bypass power_on/off() calls antohami
2022-03-22 15:36 ` [d-kernel] [PATCH 10/12] drm/v3d: Get rid of pm code antohami
2022-03-22 15:36 ` [d-kernel] [PATCH 11/12] drm/v3d: Add support for bcm2711 antohami
2022-03-22 15:36 ` [d-kernel] [PATCH 12/12] ARM: dts: bcm2711: Enable V3D antohami
2022-03-22 16:24 ` [d-kernel] [PATCH 00/12] *** Enable v3d on Raspberry Pi 4 for kernel 5.10 std-def *** Vitaly Chikunov
2022-03-22 16:39   ` Антон Мидюков
2022-03-22 17:14     ` Vitaly Chikunov
2022-03-22 17:17       ` Антон Мидюков
2022-03-22 17:25         ` Vitaly Chikunov

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=c7b976c5-9614-4be3-a08c-6616c5625286@altlinux.org \
    --to=antohami@altlinux.org \
    --cc=devel-kernel@lists.altlinux.org \
    /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

ALT Linux kernel packages development

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel-kernel/0 devel-kernel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 devel-kernel devel-kernel/ http://lore.altlinux.org/devel-kernel \
		devel-kernel@altlinux.org devel-kernel@altlinux.ru devel-kernel@altlinux.com
	public-inbox-index devel-kernel

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.devel-kernel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git