From owner-freebsd-current@freebsd.org Thu Oct 6 05:39:05 2016 Return-Path: Delivered-To: freebsd-current@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id ACC8BBD3007 for ; Thu, 6 Oct 2016 05:39:05 +0000 (UTC) (envelope-from avg@FreeBSD.org) Received: from citapm.icyb.net.ua (citapm.icyb.net.ua [212.40.38.140]) by mx1.freebsd.org (Postfix) with ESMTP id A4B5AB94; Thu, 6 Oct 2016 05:39:04 +0000 (UTC) (envelope-from avg@FreeBSD.org) Received: from porto.starpoint.kiev.ua (porto-e.starpoint.kiev.ua [212.40.38.100]) by citapm.icyb.net.ua (8.8.8p3/ICyb-2.3exp) with ESMTP id IAA06467; Thu, 06 Oct 2016 08:39:02 +0300 (EEST) (envelope-from avg@FreeBSD.org) Received: from localhost ([127.0.0.1]) by porto.starpoint.kiev.ua with esmtp (Exim 4.34 (FreeBSD)) id 1bs1OY-00099v-Gz; Thu, 06 Oct 2016 08:39:02 +0300 Subject: Re: [request for testing] isl, cyapa on chromebooks To: Matthias Apitz , Michael Gmelin , FreeBSD Current References: <20161003180750.744dae0a@bsd64.grem.de> <20161003222528.56d46787@bsd64.grem.de> <20161004114635.64857a29@bsd64.grem.de> <20161005004831.2a2fdc4b@bsd64.grem.de> <1bcfd282-63fd-f8ee-4dad-393b51f14bcd@FreeBSD.org> <302FDA6E-DEC7-49F0-8F2C-8C26C8A884AF@freebsd.org> <53bca4d2-975f-f4a0-d12a-5d73fba01a0c@FreeBSD.org> <20161006044732.GA2393@c720-r292778-amd64.oa.oclc.org> From: Andriy Gapon Message-ID: <734c7ac0-9018-051e-1df4-a3b719057e19@FreeBSD.org> Date: Thu, 6 Oct 2016 08:37:40 +0300 User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 MIME-Version: 1.0 In-Reply-To: <20161006044732.GA2393@c720-r292778-amd64.oa.oclc.org> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 06 Oct 2016 05:39:05 -0000 On 06/10/2016 07:47, Matthias Apitz wrote: > El día Wednesday, October 05, 2016 a las 04:01:25PM +0300, Andriy Gapon escribió: > >> On 05/10/2016 14:19, Michael Gmelin wrote: >>> >>> ig4iic_start is called, but iicbus_hinted_child, isl_probe, iicbus_probe and >>> iicbus_attach are not. >> >> Thank you! >> Now I think I see where I made a silly mistake. >> Please try an updated version of the patch from here >> https://people.freebsd.org/~avg/ig4-i2c.v3.diff >> It contains a fix and some cosmetic changes on top of the previous patch. > > Hi Andriy, > > I have an Acer C720 too, since around two years and it works fine, > thanks to Michael, with the cyapa chip. I have my C720 in daily heavy > usage, actually with r292778. Should I apply the above patch or do I > need to update before to a more recent CURRENT? I think that the patch should apply. But if it doesn't... > I have a second C720 > where I could do such test/update more easy, but this is current not in > my hands and has, after a repair at Acer, a Elan touch pad. > But I could prepare an USB key with 12-CURRENT, just for test. The more testing the better! > Btw: What is the reason for this change of ig4/i2c/cyapa driver? The reason is that the hardware that ig4 handles is really an I2C controller. Moreover, isl and cyapa need to issue I2C commands that can not be mapped to SMBus commands to talk to their hardware. I am not sure why Matt Dillon (who I believe is the original author of the code) chose to use smbus(4) instead of iicbus(4). And to make that work he had to "extend" smbus(4) with smbus_trans() method which does not really map to anything defined by the SMBus specification and which can not be implemented on any of real pure SMBus controllers (like intpm or ichsmb). The closest command that SMBus supports is 'Block write - block read process call', but it's not quite the same. > Last thing: I propose to remove freebsd-mobile@ from the thread. Okay. I just was not sure where I can find FreeBSD Chromebook owners. -- Andriy Gapon