From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759801AbcAMOFo (ORCPT ); Wed, 13 Jan 2016 09:05:44 -0500 Received: from mx12-09.smtp.antispamcloud.com ([46.165.232.179]:58210 "EHLO mx12-09.smtp.antispamcloud.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754982AbcAMOFm convert rfc822-to-8bit (ORCPT ); Wed, 13 Jan 2016 09:05:42 -0500 Subject: Re: [PATCH v2] hwmon: Add LTC2990 sensor driver To: Guenter Roeck , References: <568FD131.4080308@roeck-us.net> <1452067627-24939-1-git-send-email-mike.looijmans@topic.nl> <1452683150-27747-1-git-send-email-mike.looijmans@topic.nl> <5696500E.4090905@roeck-us.net> <56965661.2010307@topic.nl> <569657C8.6070607@roeck-us.net> CC: , From: Mike Looijmans Organization: TOPIC Message-ID: <5696593D.8030908@topic.nl> Date: Wed, 13 Jan 2016 15:03:41 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.4.0 MIME-Version: 1.0 In-Reply-To: <569657C8.6070607@roeck-us.net> Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 8BIT X-Originating-IP: [192.168.80.121] X-EXCLAIMER-MD-CONFIG: 9833cda7-5b21-4d34-9a38-8d025ddc3664 X-EXCLAIMER-MD-BIFURCATION-INSTANCE: 0 X-Filter-ID: s0sct1PQhAABKnZB5plbIbbvfIHzQjPVmPLZeVYSu3xU9luQrU+8/8qthi+0Jd/W6KAUC/fjyuDn NXFr4uarwyNwZSpVdakq5X66Q8VYQRca9vh+1iGKuQPOclYnYikQ0/2nHbqxWziKbKPiSpdwVJeh 6oj6+vlVI7HORWkrem3u4ONX+wVRdCv4IOs1C4geKx6e82Qhrl5Lh95/VjZqVHB+NvhiSkt9FGXX bXB10m9VPlSGMfU9Lzo0uxWN7kMhLncc/qbbv57GdnyRbm8yEkZPqmZ/NKOCdpsk+9g1ybADCsJb 7z7kpgWXWHwL5rm4Ibtf63VNbf0lrvssY+k7ACgUUZ6G78uMfV2DLUDoJYSPCwPTzw22wYdX31FM YXOOFl5Bgdd0xwbfE0yjjQqqICERbInMiTBIUBbQ/Dy6Ip5J/ASe1B0hWvEIK5cPBacYKufgB7Ba elLkNTG3g1L+53OYYq89G8LTxEq5+I242X/3Y80OmAux3oN13+ztUzneK7xP+LD+FavI7Ol7e9pH /Pe4Uvw3tUqH8Ib6WPRGPWZ1++6w8PqYuLgGb7u7tZKksfrTSy315T+eQnDC8Q/96JFVgpT1b21u ZVckGp0ccOYWZk7PZs7jDt+VcZIxsBahwbRYKWMNtROYKfbuaAYREoGDA1m3o6oTfuAaw/GRbb+4 REj+VRyQT2g/K9wiHAQyiNldDtb4wgJqb3SEHNi1T4VUdYndRiyhyQb8o5SNcNQUrz12+SBJmQRv giHhvJsV4JtjPJ4R6ft//lif1yYpLLiLFZJhYijeIOrG82fwxtlvEjSGiKhzEHocL7YhFpj+ X-Report-Abuse-To: spam@mx99.antispamcloud.com X-Filter-Fingerprint: IFrWXGses7OKB5S5G8/dJUb3OPwsHaH0Fvg5oXltHd/JUWjZ8+qhjyB23tbDuyLOYL8Ff78gYsez 4Rl08xudmXi4esCQ0R1MchVjt7wblGlvhFgW0MjUMRkF5sMCDfftTXNFDzN17hnrWeZYOJvLq0Ic WjZ+XcEjj/7Pkld0zkmvziDInX9WdMov2kn2yXjdwv61T+KDYyYtREgszdyFwv8IxCB3p/oCKvxr eyISh3JGb7OS5oVgiO+kDxZrVPLz3MmEGC2PrUKqLq5WmHK+Nw== X-Originating-IP: 88.159.208.100 X-Spampanel-Domain: topic.nl X-Spampanel-Username: 88.159.208.100 Authentication-Results: antispamcloud.com; auth=pass smtp.auth=88.159.208.100@topic.nl X-Spampanel-Outgoing-Class: ham X-Spampanel-Outgoing-Evidence: Combined (0.00) X-Recommended-Action: accept Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 13-01-16 14:57, Guenter Roeck wrote: > On 01/13/2016 05:51 AM, Mike Looijmans wrote: >> On 13-01-16 14:24, Guenter Roeck wrote: >>> On 01/13/2016 03:05 AM, Mike Looijmans wrote: >>>> This adds support for the Linear Technology LTC2990 I2C System Monitor. >>>> The LTC2990 supports a combination of voltage, current and temperature >>>> monitoring. This driver currently only supports reading two currents >>>> by measuring two differential voltages across series resistors, in >>>> addition to the Vcc supply voltage and internal temperature. >>>> >>>> This is sufficient to support the Topic Miami SOM which uses this chip >>>> to monitor the currents flowing into the FPGA and the CPU parts. >>>> >>>> Signed-off-by: Mike Looijmans >>> >>> Mike, >>> >>> That looks much better. Can you send me the output of i2cdump for the chip ? >>> That would help me writing module test code for it. >> >> I'm kinda interested into how that would work. >> > https://github.com/groeck/module-tests > >> I'll have to remove the driver first to get i2cdump to work on the chip. I >> cannot force a device removal from user space while running, can I? >> And i suppose you want a dump of a chip in running status? (On boot, all are >> registers are simply set to zero) >> > > Just use i2cdump -f while the driver is active. Here's the dump (twice so you can see the A/D converter is actually running): root@topic-miami-florida-pci-xc7z015:~# i2cdump -y -f -r 0-0xf 1 0x4c b 0 1 2 3 4 5 6 7 8 9 a b c d e f 0123456789abcdef 00: 7f 1e 7f 7f 82 5f 80 44 80 44 80 3f 80 3f 89 b5 ?????_?D?D?????? root@topic-miami-florida-pci-xc7z015:~# i2cdump -y -f -r 0-0xf 1 0x4c b 0 1 2 3 4 5 6 7 8 9 a b c d e f 0123456789abcdef 00: 7f 1e 7f 7f 82 65 80 42 80 42 80 3d 80 3d 89 b4 ?????e?B?B?=?=?? Kind regards, Mike Looijmans System Expert TOPIC Embedded Products Eindhovenseweg 32-C, NL-5683 KH Best Postbus 440, NL-5680 AK Best Telefoon: +31 (0) 499 33 69 79 E-mail: mike.looijmans@topicproducts.com Website: www.topicproducts.com Please consider the environment before printing this e-mail