Received: by 2002:ac0:a581:0:0:0:0:0 with SMTP id m1-v6csp6038105imm; Wed, 27 Jun 2018 00:48:17 -0700 (PDT) X-Google-Smtp-Source: AAOMgpe7lwo1WmNsyrd0hy8H1Z60QvJC6HVmJAyExVyYOahlE5L1gMlHg+COe3dnCpxTdIbFFGpG X-Received: by 2002:a63:b742:: with SMTP id w2-v6mr4131014pgt.200.1530085697572; Wed, 27 Jun 2018 00:48:17 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1530085697; cv=none; d=google.com; s=arc-20160816; b=ZK0kwZGHLmGCiLGMfyZZExX50JEBBUPMpw1EcZLY+Ibs8ZtBMwgUZQYJmdVDnrJuAh 31Fz1eGHA5Z8XHh2stlzjHZdoLumhOgBqU52ifw1VpBDXoVXxV6Lh/1KfocpGJEPikNR 8pDY6BwykkjV1Sku32e025W63ryODpocy1VBY6BRhvTAar/ZbwdWKb3YZfuRMSM4R6GU /Ts2J1aIJzOn8xgQcdmhGNURDNmKxYi0kHKUTe89JxDAMA4HTPM4qXdnYBUeqKGya03N mpKA6tu9vyrE2Da5JGfT/wnefkTBsgkgPvTKz57RpegxLBMPFWYeLHV9OrtqRlWvCblK Frgw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:arc-authentication-results; bh=Tq29/eQMORbLVuGqnVw7StAz9UhP5gWEdCY7W046VX4=; b=Yz5vFFBQ84K2X5nVBWz8XF4ZKCAszCDfT5wjrTVPouB/0TVDhxoYRJhcI/KAidGofR FYqB6wP6K2d7mEQnsBl/BeGRoaUUJl8r8qEeMb1QFSmeLxoEwhaF68wlkQ1TFgF83rVG zEsowUMP6jOr1YKFt8/986K2mvgx9Abx232foxr/Zl3vp+qKywdaGrg5v+zcV6EebnRD xtJYJdk801ZInLKAfX7ugvyBTJjNylBeAnaPz7h4fBDRnDXLp7N9obORBMImWn9r6up5 0L943eY3NqN0ntDpn4Ti3JsTS80yg43o4yJIFf6E5Ia/JydmiYGEc3l8mlbwkLIzSr/y inHQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id g10-v6si2926585pgp.479.2018.06.27.00.48.00; Wed, 27 Jun 2018 00:48:17 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932979AbeF0Hq0 (ORCPT + 99 others); Wed, 27 Jun 2018 03:46:26 -0400 Received: from mga03.intel.com ([134.134.136.65]:23337 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752979AbeF0HqY (ORCPT ); Wed, 27 Jun 2018 03:46:24 -0400 X-Amp-Result: UNSCANNABLE X-Amp-File-Uploaded: False Received: from orsmga003.jf.intel.com ([10.7.209.27]) by orsmga103.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 27 Jun 2018 00:46:24 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.51,278,1526367600"; d="scan'208";a="62544843" Received: from paasikivi.fi.intel.com ([10.237.72.42]) by orsmga003.jf.intel.com with ESMTP; 27 Jun 2018 00:46:20 -0700 Received: by paasikivi.fi.intel.com (Postfix, from userid 1000) id 2D5A2205FC; Wed, 27 Jun 2018 10:46:19 +0300 (EEST) Date: Wed, 27 Jun 2018 10:46:19 +0300 From: Sakari Ailus To: alanx.chiang@intel.com Cc: linux-i2c@vger.kernel.org, andy.yeh@intel.com, andriy.shevchenko@linux.intel.com, andriy.shevchenko@intel.com, rajmohan.mani@intel.com, andy.shevchenko@gmail.com, tfiga@chromium.org, jcliang@chromium.org, brgl@bgdev.pl, robh+dt@kernel.org, mark.rutland@arm.com, arnd@arndb.de, gregkh@linuxfoundation.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org Subject: Re: [PATCH v3 2/2] eeprom: at24: Add support for address-width property Message-ID: <20180627074618.65wcohw5fohbu7cv@paasikivi.fi.intel.com> References: <1530078385-1546-1-git-send-email-alanx.chiang@intel.com> <1530078385-1546-3-git-send-email-alanx.chiang@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1530078385-1546-3-git-send-email-alanx.chiang@intel.com> User-Agent: NeoMutt/20170113 (1.7.2) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jun 27, 2018 at 01:46:25PM +0800, alanx.chiang@intel.com wrote: > From: Alan Chiang > > Provide a flexible way to determine the addressing bits of eeprom. > Pass the addressing bits to driver through address-width property. > > Signed-off-by: Alan Chiang > Signed-off-by: Andy Yeh > > --- > since v1 > -- Add a warn message for 8-bit addressing. > since v2 > -- Modify the warning message for clear. > -- Move the clearing bit operation outside of a statement. > > --- > drivers/misc/eeprom/at24.c | 18 ++++++++++++++++++ > 1 file changed, 18 insertions(+) > > diff --git a/drivers/misc/eeprom/at24.c b/drivers/misc/eeprom/at24.c > index 0c125f2..d606f3b 100644 > --- a/drivers/misc/eeprom/at24.c > +++ b/drivers/misc/eeprom/at24.c > @@ -478,6 +478,24 @@ static void at24_properties_to_pdata(struct device *dev, > if (device_property_present(dev, "no-read-rollover")) > chip->flags |= AT24_FLAG_NO_RDROL; > > + err = device_property_read_u32(dev, "address-width", &val); > + if (!err) { > + switch (val) { > + case 8: > + if (chip->flags & AT24_FLAG_ADDR16) > + dev_warn(dev, "Override address width to be 8," > + "while default is 16\n"); Please don't wrap strings --- it breaks grep. Exceeding 80 is ok in this case. > + chip->flags &= ~AT24_FLAG_ADDR16; > + break; > + case 16: > + chip->flags |= AT24_FLAG_ADDR16; > + break; > + default: > + dev_warn(dev, "Bad \"address-width\" property: %u\n", > + val); > + } > + } > + > err = device_property_read_u32(dev, "size", &val); > if (!err) > chip->byte_len = val; > -- > 2.7.4 > -- Sakari Ailus sakari.ailus@linux.intel.com