From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753372AbdGSJJj (ORCPT ); Wed, 19 Jul 2017 05:09:39 -0400 Received: from esa5.microchip.iphmx.com ([216.71.150.166]:43319 "EHLO esa5.microchip.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752582AbdGSJJf (ORCPT ); Wed, 19 Jul 2017 05:09:35 -0400 X-IronPort-AV: E=Sophos;i="5.40,380,1496127600"; d="scan'208";a="2437383" Subject: Re: [PATCH 3/3] dt-bindings: i2c: eeprom: document "mac-offset" binding To: Rob Herring CC: , , , , , , , , References: <1499336217-16913-1-git-send-email-claudiu.beznea@microchip.com> <1499336217-16913-4-git-send-email-claudiu.beznea@microchip.com> <20170710034613.fc4vj6h22kagce24@rob-hp-laptop> From: m18063 Message-ID: <96d6eeed-9272-1460-8c5d-64f06918b09a@microchip.com> Date: Wed, 19 Jul 2017 12:08:39 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: <20170710034613.fc4vj6h22kagce24@rob-hp-laptop> Content-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 10.07.2017 06:46, Rob Herring wrote: > On Thu, Jul 06, 2017 at 01:16:57PM +0300, Claudiu Beznea wrote: >> Document "mac-offset" binding that will be used by at24 EEPROM driver. >> >> Signed-off-by: Claudiu Beznea >> --- >> Documentation/devicetree/bindings/eeprom/eeprom.txt | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/eeprom/eeprom.txt b/Documentation/devicetree/bindings/eeprom/eeprom.txt >> index a50dc01..3dd267c 100644 >> --- a/Documentation/devicetree/bindings/eeprom/eeprom.txt >> +++ b/Documentation/devicetree/bindings/eeprom/eeprom.txt >> @@ -35,10 +35,13 @@ Optional properties: >> >> - read-only: this parameterless property disables writes to the eeprom >> >> + - mac-offset: offset in EEPROM where MAC address starts >> + > > This doesn't scale if you have multiple things you need the offset to, > and we already have a binding for this. Use the nvmem binding. Are you talking about nvmem-cells, nvmem-cell-names bindings? Since at24 is i2c driver, I can only think at it as a nvmem provider. Using these bindings it will imply, as per my understanding about nvmem subsystem, that this should also become a nvmem consumer. It looks a little strange to me but I don't have deep knowledge about subsystem, correct me if I'm wrong. Please let me know if you are talking about using these bindings and reading them (just to get the offset passed in "reg" binding) by not passing the nvmem APIs defined in nvmem core. Thank you, Claudiu > > Rob >