From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from bali.collaboradmins.com (bali.collaboradmins.com [148.251.105.195]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id BE7FC286439 for ; Mon, 30 Jun 2025 15:50:19 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=148.251.105.195 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751298622; cv=none; b=iAeyJcoxTK4zw7mNUf212wJz1MBtcX9cEvthtgjxAl9l6sgF1Pwvwl2mVryDXZRZHarxc3YKhanXYh1CFYfpfgikqGUvpRZlBg670B/WuKrBDQpxT/jePHCE6PGaBhVGhVMt1rDWgEEM/nwKH3+jrHfjYLH1TQcMCdDh4pjH15E= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751298622; c=relaxed/simple; bh=OFNpuSwskHZ5cDNYtd7s1ZDhkEVY62Cgbtcf8oU9wd4=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=ho/8p9ZSPEr0vJpuwm8swAYcuxE+PV4fDEKiPFDaiDEZTZPAWLQqw0teget3TzlQ44tFvCbX+q9TNaPWIu9ysqIoi94NtliIjwbiEVcXXotKix5bNkpqumwob2Tky9AHOUoGkliu7OE7nj2aAzNqYzq3En9M8GWoDgnZApP72L4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com; spf=pass smtp.mailfrom=collabora.com; dkim=pass (2048-bit key) header.d=collabora.com header.i=@collabora.com header.b=Jum/g7KB; arc=none smtp.client-ip=148.251.105.195 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=collabora.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=collabora.com header.i=@collabora.com header.b="Jum/g7KB" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1751298615; bh=OFNpuSwskHZ5cDNYtd7s1ZDhkEVY62Cgbtcf8oU9wd4=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=Jum/g7KBXEDCMUrkBm/2+AX51VoD6/ln8Z1HUMzW1l/hkzHaZapoMESM2O9Nl8tLB u/p33kBj+OlYZzd8jCjZqim83y0XMvV05OYvqu2WyHI/N57hwyN5ZNVn6CQnZk8az5 bUTeIfGU3Hwn/dhD05aSK+z3iRzE2dD7v8suodOpcCiJIQY9XBwku7iNWpBJMBmyIi QpDIxrpSNL9NNQ3P8h941zKNheiYu7xHcmmi+N8xkH4qhjBiTju+ZLhBXJW5WL+ja7 xdBxszdlcvjRzZqj0rMtRYIwOXRc82IesxwoXyr+w6sJM0+5KmiO5HrLXCaP7FZoZ5 KIluQjx8xTsvA== Received: from fedora (unknown [IPv6:2a01:e0a:2c:6930:16f1:973:959b:9b0c]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: bbrezillon) by bali.collaboradmins.com (Postfix) with ESMTPSA id 512C317E0391; Mon, 30 Jun 2025 17:50:15 +0200 (CEST) Date: Mon, 30 Jun 2025 17:50:11 +0200 From: Boris Brezillon To: Steven Price Cc: Liviu Dudau , Karunika Choo , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] drm/panthor: Wait for _READY register when powering on Message-ID: <20250630175011.7bd395d9@fedora> In-Reply-To: <20250630140704.432409-1-steven.price@arm.com> References: <20250630140704.432409-1-steven.price@arm.com> Organization: Collabora X-Mailer: Claws Mail 4.3.1 (GTK 3.24.49; x86_64-redhat-linux-gnu) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Mon, 30 Jun 2025 15:07:02 +0100 Steven Price wrote: > panthor_gpu_block_power_on() takes a register offset (rdy_reg) for the > purpose of waiting for the power transition to complete. However, a > copy/paste error converting to use the new 64 register functions > switched it to using the pwrtrans_reg register instead. Fix the function > to use the correct register. > > Fixes: 4d230aa209ed ("drm/panthor: Add 64-bit and poll register accessors") > Signed-off-by: Steven Price Reviewed-by: Boris Brezillon > --- > drivers/gpu/drm/panthor/panthor_gpu.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/panthor/panthor_gpu.c b/drivers/gpu/drm/panthor/panthor_gpu.c > index 534735518824..cb7a335e07d7 100644 > --- a/drivers/gpu/drm/panthor/panthor_gpu.c > +++ b/drivers/gpu/drm/panthor/panthor_gpu.c > @@ -297,8 +297,9 @@ int panthor_gpu_block_power_on(struct panthor_device *ptdev, > > gpu_write64(ptdev, pwron_reg, mask); > > - ret = gpu_read64_relaxed_poll_timeout(ptdev, pwrtrans_reg, val, > - !(mask & val), 100, timeout_us); > + ret = gpu_read64_relaxed_poll_timeout(ptdev, rdy_reg, val, > + (mask & val) == val, > + 100, timeout_us); > if (ret) { > drm_err(&ptdev->base, "timeout waiting on %s:%llx readiness", > blk_name, mask);