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 X-Spam-Level: X-Spam-Status: No, score=-3.9 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 99E15C2D0EB for ; Sun, 29 Mar 2020 20:32:45 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 6F25420781 for ; Sun, 29 Mar 2020 20:32:45 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=canb.auug.org.au header.i=@canb.auug.org.au header.b="AbaUTddb" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728501AbgC2Ucm (ORCPT ); Sun, 29 Mar 2020 16:32:42 -0400 Received: from ozlabs.org ([203.11.71.1]:57915 "EHLO ozlabs.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727370AbgC2Ucl (ORCPT ); Sun, 29 Mar 2020 16:32:41 -0400 Received: from authenticated.ozlabs.org (localhost [127.0.0.1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-256) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by mail.ozlabs.org (Postfix) with ESMTPSA id 48r6j235X1z9sRN; Mon, 30 Mar 2020 07:32:38 +1100 (AEDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=canb.auug.org.au; s=201702; t=1585513959; bh=wtdE/xlqjJigILuC52/ujbxaiSyFTL1AhRTSlpQdnnM=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=AbaUTddb47juNzth6lZa+NH56Eg7RuaCRJtp2nagH/0oG1OIHbFPC4URcbaBYxGnQ WBxXKWEc3igs/Zx8Rk574Q3yGaDrWAddEJuALT/uW5glh1XggiPpVl7kHp3WY8d/Jm pjpZuRqL2na7CDrcAPgp+xWbpmAGyn6F0OkSSESorOPSQ6Jyl+OXXxO9xwAVl3b837 8UVue+8QXFmOBoORBX8Jknc3Yrw8jsbBa8xaiBHWCX9U7p2/lym6BwptzCc3ZYmgOH daFHFPtdWMD0GM1qfz7Uy8Ep791bEjrboagTy2VEFDKAhX0Z41DzE1Q3vSU02v3+KC 2+VLjPx/vremQ== Date: Mon, 30 Mar 2020 07:32:31 +1100 From: Stephen Rothwell To: Nathan Chancellor Cc: Daniel Lezcano , Ohad Ben-Cohen , Bjorn Andersson , linux-remoteproc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-next@vger.kernel.org Subject: Re: Build failure in omap_remoteproc due to 02e6d546e3bd Message-ID: <20200330073231.6e04035e@canb.auug.org.au> In-Reply-To: <20200327185055.GA22438@ubuntu-m2-xlarge-x86> References: <20200327185055.GA22438@ubuntu-m2-xlarge-x86> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="Sig_/n6aWScVd=OTDo0fxVhMctuB"; protocol="application/pgp-signature"; micalg=pgp-sha256 Sender: linux-next-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-next@vger.kernel.org --Sig_/n6aWScVd=OTDo0fxVhMctuB Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable Hi Nathan, On Fri, 27 Mar 2020 11:50:55 -0700 Nathan Chancellor wrote: > > Apologies if this has already been reported, I did not see anything in > the archives. On next-20200327, I see the following build error: >=20 > drivers/remoteproc/omap_remoteproc.c:174:44: error: too many arguments > to function call, expected 2, have 3 > timer->timer_ops->set_load(timer->odt, 0, 0); > ~~~~~~~~~~~~~~~~~~~~~~~~~~ ^ > 1 error generated. >=20 > This is due to commit 02e6d546e3bd ("clocksource/drivers/timer-ti-dm: > Enable autoreload in set_pwm") in the clockevents tree interacting with > commit e28edc571925 ("remoteproc/omap: Request a timer(s) for remoteproc > usage") from the rpmsg tree. >=20 > Something like the diff below should resolve it but it cannot be applied > to the rpmsg tree because it does not have the clockevents change. It > will have to be applied by Linus as a merge fixup. >=20 > Stephen, would you mind applying this to whichever merge happens second? >=20 > Cheers, > Nathan >=20 > diff --git a/drivers/remoteproc/omap_remoteproc.c b/drivers/remoteproc/om= ap_remoteproc.c > index cdb546f7232e..6955fab0a78b 100644 > --- a/drivers/remoteproc/omap_remoteproc.c > +++ b/drivers/remoteproc/omap_remoteproc.c > @@ -171,7 +171,7 @@ static int omap_rproc_request_timer(struct device *de= v, struct device_node *np, > } > =20 > /* clean counter, remoteproc code will set the value */ > - timer->timer_ops->set_load(timer->odt, 0, 0); > + timer->timer_ops->set_load(timer->odt, 0); > =20 > return 0; > } Thanks, I will use that as a merge fix up from today. --=20 Cheers, Stephen Rothwell --Sig_/n6aWScVd=OTDo0fxVhMctuB Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- iQEzBAEBCAAdFiEENIC96giZ81tWdLgKAVBC80lX0GwFAl6BBeAACgkQAVBC80lX 0Gz/tQf+Jak6uV9MnoCsyPrLx1fCq0oYPt8FVj/npVTYLTuEFiwKn3i7vXHVdFw4 vEzpW0ZtPrccyWk8gEwksLrhbwYZNt40Pzy9pEjlPjRIj3fWhRJI6bJmaZXyhM31 +u785y0yaX8nxjZIihxu7ghuWbLx11iCgTDunvBvhv/xVrrCPmbtuY4xMzDKt1va OiHnGDfuOiENQckikS4gR6s1C1P97QHKSYSLyGhiB4DUuykzVQGqEPbfo2/WkfsP cVnA+Wgf0+iHEc74LAb5JGqDsfkGZohoqiVKswBdPe1NKwmJQM2OMZksUMKpaUzN 1o7g+18wyHN9AdTmP9vMjZusY4q1Hw== =eV0w -----END PGP SIGNATURE----- --Sig_/n6aWScVd=OTDo0fxVhMctuB--