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 BECD2C433EF for ; Mon, 31 Jan 2022 10:17:50 +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:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject: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=c0kjmsazKGym/QZLXFFlj2/8U3Zwcv+SZ20C8uqQQJo=; b=fkDL9botxoU2/+ socCTiJwsDYr9e++z3kNDpQwaGVLKDe+zw/iQVapSVyVGKTROqmv2WKrFasGJCp2lMJdSKC6eeH/g 5sqmfl6vW0dmYYr1JnyBo0sDa3E1BnbZUidnNrTSLTFfPEtUfHSRA/J4WvfzqDXKXO6IKMmolcJRk SxA6sfs1QCBrKK0L98lua4IZOnw55b8pbNqbwcUBBY7K9bZDkte5IQwwU03ixciC8qji2C2oU+gf3 koJLDvO4bXCJ9TEkz9dMhYotgP+JET67wiDHTSqMNCMpbi60/uTPENV7Qbxp5EA+sfmtFChd5hvmc iOhi3qJDw2QxLcFhzvPA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nETjO-008oIz-Cm; Mon, 31 Jan 2022 10:16:18 +0000 Received: from muru.com ([72.249.23.125]) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nETjK-008oH0-GT for linux-arm-kernel@lists.infradead.org; Mon, 31 Jan 2022 10:16:15 +0000 Received: from localhost (localhost [127.0.0.1]) by muru.com (Postfix) with ESMTPS id 5F72B8027; Mon, 31 Jan 2022 10:15:59 +0000 (UTC) Date: Mon, 31 Jan 2022 12:16:06 +0200 From: Tony Lindgren To: Drew Fustini Subject: Re: [PATCH 2/2] clocksource/drivers/timer-ti-dm: Handle dra7 timer wrap errata i940 Message-ID: References: <20210323074326.28302-1-tony@atomide.com> <20210323074326.28302-3-tony@atomide.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220131_021614_636499_B335B56B X-CRM114-Status: GOOD ( 18.68 ) 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: , Cc: Tero Kristo , Keerthy , Daniel Lezcano , linux-kernel@vger.kernel.org, Thomas Gleixner , linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org 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, * Drew Fustini [220129 21:05]: > On Tue, Mar 23, 2021 at 09:43:26AM +0200, Tony Lindgren wrote: > > There is a timer wrap issue on dra7 for the ARM architected timer. > > In a typical clock configuration the timer fails to wrap after 388 days. > > > > To work around the issue, we need to use timer-ti-dm percpu timers instead. > > > > Let's configure dmtimer3 and 4 as percpu timers by default, and warn about > > the issue if the dtb is not configured properly. > > Hi Tony, > > This causes a conflict for IPU2 which is using timer 3 and 4. > > From arch/arm/boot/dts/dra7-ipu-dsp-common.dtsi: > > &ipu2 { > mboxes = <&mailbox6 &mbox_ipu2_ipc3x>; > ti,timers = <&timer3>; > ti,watchdog-timers = <&timer4>, <&timer9>; > }; OK, sorry I missed that part. > I noticed an error ("could not get timer platform device") when booting > mainline on a BeagleBoard X15 (AM578): > > omap-rproc 55020000.ipu: assigned reserved memory node ipu2-memory@95800000 > remoteproc remoteproc1: 55020000.ipu is available > remoteproc remoteproc1: powering up 55020000.ipu > remoteproc remoteproc1: Booting fw image dra7-ipu2-fw.xem4, size 3747220 > omap-rproc 55020000.ipu: could not get timer platform device > omap-rproc 55020000.ipu: omap_rproc_enable_timers failed: -19 > remoteproc remoteproc1: can't start rproc 55020000.ipu: -19 > > I switched this errata fix to use timer 15 and 16 instead which resolves > the error. Do you think that is an acceptable solution? I think the only difference is that timers 15 and 16 are in l4_per3 instead of l4_per1. I doubt that matters as they are pretty much always clocked in this case. If you want to check you can run cyclictest :) > If so, I will post the patch to the list. OK thanks, Tony _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel