From owner-freebsd-arm@freebsd.org Tue Sep 1 00:26:45 2020 Return-Path: Delivered-To: freebsd-arm@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id 5B7F43D24DD for ; Tue, 1 Sep 2020 00:26:45 +0000 (UTC) (envelope-from jmg@gold.funkthat.com) Received: from gold.funkthat.com (gate2.funkthat.com [208.87.223.18]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "gate2.funkthat.com", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4BgSYc3tDTz4KrF; Tue, 1 Sep 2020 00:26:44 +0000 (UTC) (envelope-from jmg@gold.funkthat.com) Received: from gold.funkthat.com (localhost [127.0.0.1]) by gold.funkthat.com (8.15.2/8.15.2) with ESMTPS id 0810QgLi043076 (version=TLSv1.2 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO); Mon, 31 Aug 2020 17:26:42 -0700 (PDT) (envelope-from jmg@gold.funkthat.com) Received: (from jmg@localhost) by gold.funkthat.com (8.15.2/8.15.2/Submit) id 0810QgXa043075; Mon, 31 Aug 2020 17:26:42 -0700 (PDT) (envelope-from jmg) Date: Mon, 31 Aug 2020 17:26:42 -0700 From: John-Mark Gurney To: Andriy Gapon Cc: freebsd-arm@freebsd.org Subject: Re: Kmod driver at iicbus. attach() and config_intrhook(9) Message-ID: <20200901002642.GV4213@funkthat.com> Mail-Followup-To: Andriy Gapon , freebsd-arm@freebsd.org References: <20200820223918.GC4213@funkthat.com> <496ed13afb2327d6416664b4dacfc346@mh.net.ru> <20200824204708.GH4213@funkthat.com> <6be8abd0-185c-dbbb-5964-1e2ad92ad213@FreeBSD.org> <20200825203313.GJ4213@funkthat.com> <41220fa2-0af3-22c2-0057-c7578f14d52b@FreeBSD.org> <20200826213343.GR4213@funkthat.com> <2cd30277-0628-9a4c-56c0-4ab7d4debaf8@FreeBSD.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <2cd30277-0628-9a4c-56c0-4ab7d4debaf8@FreeBSD.org> X-Operating-System: FreeBSD 11.3-STABLE amd64 X-PGP-Fingerprint: D87A 235F FB71 1F3F 55B7 ED9B D5FF 5A51 C0AC 3D65 X-Files: The truth is out there X-URL: https://www.funkthat.com/ X-Resume: https://www.funkthat.com/~jmg/resume.html X-TipJar: bitcoin:13Qmb6AeTgQecazTWph4XasEsP7nGRbAPE X-to-the-FBI-CIA-and-NSA: HI! HOW YA DOIN? can i haz chizburger? User-Agent: Mutt/1.6.1 (2016-04-27) X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.4.3 (gold.funkthat.com [127.0.0.1]); Mon, 31 Aug 2020 17:26:43 -0700 (PDT) X-Rspamd-Queue-Id: 4BgSYc3tDTz4KrF X-Spamd-Bar: + Authentication-Results: mx1.freebsd.org; dkim=none; dmarc=none; spf=none (mx1.freebsd.org: domain of jmg@gold.funkthat.com has no SPF policy when checking 208.87.223.18) smtp.mailfrom=jmg@gold.funkthat.com X-Spamd-Result: default: False [1.09 / 15.00]; RCVD_TLS_ALL(0.00)[]; ARC_NA(0.00)[]; FREEFALL_USER(0.00)[jmg]; FROM_HAS_DN(0.00)[]; TO_DN_SOME(0.00)[]; TO_MATCH_ENVRCPT_ALL(0.00)[]; MID_RHS_MATCH_FROM(0.00)[]; MIME_GOOD(-0.10)[text/plain]; DMARC_NA(0.00)[funkthat.com]; AUTH_NA(1.00)[]; NEURAL_SPAM_SHORT(0.03)[0.027]; NEURAL_HAM_LONG(-0.05)[-0.051]; NEURAL_HAM_MEDIUM(-0.09)[-0.087]; RCPT_COUNT_TWO(0.00)[2]; R_SPF_NA(0.00)[no SPF record]; FORGED_SENDER(0.30)[jmg@funkthat.com,jmg@gold.funkthat.com]; R_DKIM_NA(0.00)[]; MIME_TRACE(0.00)[0:+]; ASN(0.00)[asn:32354, ipnet:208.87.216.0/21, country:US]; FROM_NEQ_ENVFROM(0.00)[jmg@funkthat.com,jmg@gold.funkthat.com]; MAILMAN_DEST(0.00)[freebsd-arm]; RCVD_COUNT_TWO(0.00)[2] X-BeenThere: freebsd-arm@freebsd.org X-Mailman-Version: 2.1.33 Precedence: list List-Id: "Porting FreeBSD to ARM processors." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 01 Sep 2020 00:26:45 -0000 Andriy Gapon wrote this message on Thu, Aug 27, 2020 at 08:35 +0300: > On 27/08/2020 00:33, John-Mark Gurney wrote: > > Andriy Gapon wrote this message on Wed, Aug 26, 2020 at 23:55 +0300: > >> On 25/08/2020 23:33, John-Mark Gurney wrote: > >>> Andriy Gapon wrote this message on Tue, Aug 25, 2020 at 10:21 +0300: > >>>> On 24/08/2020 23:47, John-Mark Gurney wrote: > >>>>> What driver is used for i2c on the Orange? I've tried to firgure it > >>>>> out from dts + source, but it isn't clear. > >>>> > >>>> a10_twsi > >>> > >>> Thanks. And this explains why I couldn't find it in arm/allwinner.. :) > >>> > >>> Yeah, this is using bus_generic_attach... > >>> > >>> But it looks like it supports interruptless operation. Is there a flag > >>> for when interrupts are enabled? > >>> > >>> Because the check in sys/dev/iicbus/twsi/twsi.c:twsi_transfer could be > >>> changed from: > >>> if (sc->have_intr == false) > >>> > >>> to: > >>> if (sc->have_intr == false || interrupts_not_enabled) > >>> > >>> And that could fix things. > >>> > >> > >> sc->have_intr is already enabled from the interrupt enable hook. > > > > But that only detects if the interrupt are used, NOT that they are > > currently active and working which the "|| cold" would provide... > > The interrupt hook is called after interrupts are enabled and usable. > Specifically, I mean config_intrhook_oneshot(). Ok, then the OP shouldn't be having a problem then, maybe you can help then figure it out then? -- John-Mark Gurney Voice: +1 415 225 5579 "All that I will do, has been done, All that I have, has not."