From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id AD558C4167B for ; Wed, 6 Dec 2023 22:37:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=ggjlWjtq2OeqOXsemfBbrQ5zgkoT7QBZlz9MOFccWPY=; b=FiOHebht9f9Fvj xgXvLYndClOkwxh3QIQomgfRDobJVAncJjX/ptEjE+RTTaW9HY+Vp79FNkcdHfOD7a8OczUQVb6+0 7IBLB3i9PIZ1vNEBe4EUPy/frn9fAdLAUOdKeFBUMBZ6yMNftAY0E4+LSLkew7cOja/jfui9QGOIA d2lonBSGGl+nOliAat0f+p+wuj9yViBUpRC+R+WqufwOFX7e4r6RorZJbCmQArhqGDhP+uhmTPJPA XHT0UYnOhBCs+RpT5RZd0a4bbbQQT8ESLpT3G59yF7+4OKH+uKRYJ2r/bjIFsz3U9jNUyyv3OZVPy XqrK3HtJ31bvb6IPrfPQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1rB0VN-00BPv7-2B; Wed, 06 Dec 2023 22:36:33 +0000 Received: from perceval.ideasonboard.com ([213.167.242.64]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1rB0VL-00BPuh-0m for linux-arm-kernel@lists.infradead.org; Wed, 06 Dec 2023 22:36:32 +0000 Received: from pendragon.ideasonboard.com (213-243-189-158.bb.dnainternet.fi [213.243.189.158]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 871E9556; Wed, 6 Dec 2023 23:35:48 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1701902148; bh=gYjzyVxIoQATrShVZeAAv2CPas5m9EdbtIMdRBgw1cI=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=H0sw/cWMfIcg38tVBmswoaVCrsilBadsETT6THqaV7+bkJde98YANDhLt1Emxnlg6 IG713lsGlCC7UFwboS9enM3LD04YcaLTXaACOnzLIWtLS7pZ2vAoxM9P1AvbgmN8Eo a9ZM9Y5ygBeRG9CxSb+zB0XOtm9g1SvxIVtvqNAQ= Date: Thu, 7 Dec 2023 00:36:36 +0200 From: Laurent Pinchart To: Alexander Stein Cc: Mauro Carvalho Chehab , Shawn Guo , Sascha Hauer , Fabio Estevam , Pengutronix Kernel Team , NXP Linux Team , linux-media@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH 1/1] media: nxp: imx8-isi-debug: Add missing registers to debugfs output Message-ID: <20231206223636.GE29417@pendragon.ideasonboard.com> References: <20231206094401.491100-1-alexander.stein@ew.tq-group.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20231206094401.491100-1-alexander.stein@ew.tq-group.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231206_143631_424302_229E0DB9 X-CRM114-Status: GOOD ( 15.47 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Alexander, On Wed, Dec 06, 2023 at 10:44:01AM +0100, Alexander Stein wrote: > The extended address registers are missing in the debug output register > list. Add them. > > Signed-off-by: Alexander Stein > --- > drivers/media/platform/nxp/imx8-isi/imx8-isi-debug.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/drivers/media/platform/nxp/imx8-isi/imx8-isi-debug.c b/drivers/media/platform/nxp/imx8-isi/imx8-isi-debug.c > index 6709ab7ea1f3..3ac5685d6cc1 100644 > --- a/drivers/media/platform/nxp/imx8-isi/imx8-isi-debug.c > +++ b/drivers/media/platform/nxp/imx8-isi/imx8-isi-debug.c > @@ -66,6 +66,13 @@ static int mxc_isi_debug_dump_regs_show(struct seq_file *m, void *p) > MXC_ISI_DEBUG_REG(CHNL_OUT_BUF2_ADDR_V), > MXC_ISI_DEBUG_REG(CHNL_SCL_IMG_CFG), > MXC_ISI_DEBUG_REG(CHNL_FLOW_CTRL), > + MXC_ISI_DEBUG_REG(CHNL_Y_BUF1_XTND_ADDR), > + MXC_ISI_DEBUG_REG(CHNL_U_BUF1_XTND_ADDR), > + MXC_ISI_DEBUG_REG(CHNL_V_BUF1_XTND_ADDR), > + MXC_ISI_DEBUG_REG(CHNL_Y_BUF2_XTND_ADDR), > + MXC_ISI_DEBUG_REG(CHNL_U_BUF2_XTND_ADDR), > + MXC_ISI_DEBUG_REG(CHNL_V_BUF2_XTND_ADDR), > + MXC_ISI_DEBUG_REG(CHNL_IN_BUF_XTND_ADDR), Those registers only exist in the i.MX8MP. The i.MX8MN referenece manual lists them as reserved and indicates they read as 0's, so it should be safe. The i.MX93 reference manual, however, doesn't list those registers at all, so accessing them may lead to issues. How what platform(s) have you tested this patch ? > }; > > struct mxc_isi_pipe *pipe = m->private; -- Regards, Laurent Pinchart _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel