Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751223AbaKZJNy (ORCPT ); Wed, 26 Nov 2014 04:13:54 -0500 Received: from mail-ie0-f174.google.com ([209.85.223.174]:36257 "EHLO mail-ie0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750811AbaKZJNv (ORCPT ); Wed, 26 Nov 2014 04:13:51 -0500 MIME-Version: 1.0 In-Reply-To: <54754364.80705@roeck-us.net> References: <1416930423-12179-1-git-send-email-bgolaszewski@baylibre.com> <1416930423-12179-2-git-send-email-bgolaszewski@baylibre.com> <5474A72D.2000309@roeck-us.net> <5474B564.9080902@roeck-us.net> <5474C36C.1020004@roeck-us.net> <5474CAC6.8030608@roeck-us.net> <54754364.80705@roeck-us.net> Date: Wed, 26 Nov 2014 10:13:50 +0100 Message-ID: Subject: Re: [PATCH 1/5] hwmon: ina2xx: bail-out from ina2xx_probe() in case of configuration errors From: Bartosz Golaszewski To: Guenter Roeck Cc: LKML , Benoit Cousson , Patrick Titiano Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 2014-11-26 4:05 GMT+01:00 Guenter Roeck : > On 11/25/2014 10:30 AM, Guenter Roeck wrote: >> Repeating my earlier question: Is the resistor value changed by software >> or by changing the probe ? >> > > Looking into the available documents, I am quite sure that the resistor > is changed by replacing the probe, in other words by pulling the board > with the ina226 and replacing it with another one. Given that, configuring > the shunt resistor value with a sysfs attribute is really the wrong way > to do it; you should use probe specific devicetree overlays instead. Yes, it's changed by replacing the probes. As for the averaging rate: it's a programmable feature of the chip and it's useful for our user interface for noise reduction (or the opposite - to be able to see the actual distortion), can this be accepted after applying the fixes according to your review? Bart -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/