From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933472AbcBYR7u (ORCPT ); Thu, 25 Feb 2016 12:59:50 -0500 Received: from down.free-electrons.com ([37.187.137.238]:52141 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S932381AbcBYR7s (ORCPT ); Thu, 25 Feb 2016 12:59:48 -0500 Date: Thu, 25 Feb 2016 09:59:43 -0800 From: Maxime Ripard To: Henry Paulissen Cc: linux-sunxi , draakje197@gmail.com, linus.walleij@linaro.org, wens@csie.org, patrice.chotard@st.com, plagnioj@jcrosoft.com, maxime.coquelin@st.com, fabf@skynet.be, linux-gpio@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [linux-sunxi] Re: [PATCH v3] Fix sun7i pin assignment for IRQ's Message-ID: <20160225175943.GD4736@lukather> References: <1456060845-20692-1-git-send-email-henry@nitronetworks.nl> <20160221171830.GD19169@lukather> <4cf9dbe3-faa6-4729-87fd-0e7d9ce463b0@googlegroups.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="cHMo6Wbp1wrKhbfi" Content-Disposition: inline In-Reply-To: <4cf9dbe3-faa6-4729-87fd-0e7d9ce463b0@googlegroups.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --cHMo6Wbp1wrKhbfi Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi, On Sun, Feb 21, 2016 at 11:27:46AM -0800, Henry Paulissen wrote: >=20 > Op zondag 21 februari 2016 18:18:37 UTC+1 schreef Maxime Ripard: >=20 > > > > Your commit log is going to need some work. Which bugs? What tests did= =20 > > you make? Why are you making these changes while the datasheet says=20 > > otherwise?=20 > > >=20 > Its a fix for a not yet existing bug. I was fiddling around with IRQ's an= d=20 > couldn't get them to work. > I took a dumpster dive into it and found a shitload of contradicting=20 > manuals and datasheets. >=20 >=20 > Take for example the A20 user manual: > http://dl.linux-sunxi.org/A20/A20%20user%20manual%20v1.3%2020141010.pdf >=20 > (pin PI14) > Page 237: EINT26 is on mux *5* in the pin overview. > Page 288: EINT26 is on mux *6* in the registers. > =20 > Page 233: EINT12 is on pin PC19 mux6 in the pin overview. > Page 236: EINT12 is on pin PH12 mux6 in the pin overview. > Page 253: EINT12 is *not* on pin PC19 on the registers. > Page 281: EINT12 is on pin PH12 mux6 in the registers. Ok, so i guess you're actually fixing two different things: the first one is that some interrupts are using the wrong function, while some others are just not there at all. It would be great if you could make two different patches for these. I guess we could also change SUNXI_FUNCTION_IRQ to enforce the mux value 6, since you're removing the last users of a different value, but that can be done as a followup > So manual may say otherwise, but I hope I have proven that the manual isn= 't=20 > to be trusted. >=20 > My patch is based onto testing from both me and Andre (apritzel). > He with a Banana PI M1 and me with a Cubietruck (both A20 soc). >=20 > We did a basic test by connecting a pulsing signal to a port and configur= e=20 > kernel to use irq. >=20 > e.g. > echo pin# > /sys/class/gpio/export > echo in > /sys/class/gpio/gpio#/direction > echo rising > /sys/class/gpio/gpio#/edge >=20 > and check on /proc/interrupts to see if a irq was attached and if it was= =20 > receiving. >=20 > Im not sure what andre his pulse source was, but mine was a 1pps coming= =20 > from a gps. That's a great explanation overall, it should just be in the commit log itself. A git commit is easy to find, the discussion that was triggered by it not so much. Thanks! Maxime --=20 Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com --cHMo6Wbp1wrKhbfi Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJWz0EPAAoJEBx+YmzsjxAgyTEP/iNYVSNIQBE3gv7A7kq8xfNl qvuL/K3BcI7/niP8mjiZ2VuCXmjf8tsH4SJ2zlhpczt9m7NBGsaptg83zldS64vc X9UsClUUDbx8C4wMH1SieaQt9wvpShjUNYax2m4e0VFy8GK/pwDrMuzj+U2k5jFu 7k4xnI2UYomJkFEDWb+5KyqH4PMbflRB37GTM7zkj1fYqns+VdoXLoyKEb+Z8rYi LfGox3lBv/leGMKqsEcIKY9zuJToLyhydXptsW04vyB6gctnlbZS/+S56C1EdrJA h03ZDeUh765ljxT5dr3nDixvqmTjau8Sd6s/BIBhFnaZtqSioOAUhEhm8C3xEB99 1udzJfnmanJnZTwOyM/1+Z5EDv/Dftqr2CysF7xS0Lhw7dWdtkRvykDFgAmUHxo5 u0be4keP4gwyQsY8lCFTgQ3FJNoJHzObQ3J2iTOsx/qNUe1lr0mVFrwOVoQEXVd/ Zzc+/+Ax1XRWfS3R0zmulvtNSqpjS9AdRuEJ5Gxdk1MIXmPWemsWNXpRSUVD/I1g Z7PqkCgZNVWHEb1rI+MiwLLuCvX66YHvfJsPBnyF8KP9qjgwAlhkZVh1BEkffogM 48WpkVEnNJ5dHQPjT2bv/VjV5OHBHbtT0cc5K59XuZ95ULaQTZUILkcYD0B/9b2C WcRs27zC5PieTxd51PDE =G1Vj -----END PGP SIGNATURE----- --cHMo6Wbp1wrKhbfi--