Received: by 2002:ac0:a581:0:0:0:0:0 with SMTP id m1-v6csp571487imm; Wed, 4 Jul 2018 02:07:02 -0700 (PDT) X-Google-Smtp-Source: AAOMgpcvwkpvmElyWAAilCGEiZ/Ti6/HQG1zio9cmX0wM18151V7V6oS5ZfSlCGa51zuz5AAHFhi X-Received: by 2002:a63:9702:: with SMTP id n2-v6mr1139472pge.88.1530695222779; Wed, 04 Jul 2018 02:07:02 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1530695222; cv=none; d=google.com; s=arc-20160816; b=v9X1+k4Mw6Ov+uAqkPy1pANwDr8AXCuBELqEyq5nkSGo3krG25ZmTLMjj7c/CTC2Ju pVVPfqTZ+AMZ5FJtulKWwxTb0jeh2JT6+suSZbjwRvmV2KYa1i+qaamgoRUBK7D9wBWG QltrWwNFwhN0mkESDtF0EHyGoRClBwvbCEDTli3qQnfJtSZ2fLHZ5aRj/ep5WvcflPja 2VVvQtyJtwl0y6BcKkYzKXo2giv2fmLUag9hQ1QJZLltOBh8DS2rc9oxUMOAR6HPbagO Z7CzDkO9lLnPSJfu8m+k8guh5hsvDNPdDByPKOYNNCzw4NAmb3ZzrSQqwtIJEfoxWFP9 2ojw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject:dkim-signature :arc-authentication-results; bh=r2OE8sJUi9rHlS+9rxJszYaAItoKn2wRLCXTgktCMPA=; b=mG05Ynwp/zlEmN5LWZwYhwrf+RfY9w96V1cFT92himCu8fY8BBTqRvfI/EQQMwyTWT k2dRomYDrCt3zBtyo25riF4S2gi1IOvIvj4dVzY93ixyIfxTeXK0QqHr5BvIkO7QtFQS qQHq8g5iztdvxZo4DNdpVu4blKrXWyYkOYjpYyZp88YPEXnfhZQGqkJ/sAK7fcVWEooC Q0ShwgKhWSEGfSQvT1j/E/Um2h/Jzx36PoaqX/0W5xmiR+fyZ1vTHOXTV9DSZEpuP75A kB1TyY9o0EwL3VpTwKhyKuLiMncACHgUSpIqHhvxX9PQxWOChUzt8/d9Rbw62b06k9pP nDBg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@ti.com header.s=ti-com-17Q1 header.b=RdOaiR6W; 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=pass (p=QUARANTINE sp=NONE dis=NONE) header.from=ti.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id v127-v6si2859368pgv.212.2018.07.04.02.06.47; Wed, 04 Jul 2018 02:07:02 -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; dkim=pass header.i=@ti.com header.s=ti-com-17Q1 header.b=RdOaiR6W; 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=pass (p=QUARANTINE sp=NONE dis=NONE) header.from=ti.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934429AbeGDJF5 (ORCPT + 99 others); Wed, 4 Jul 2018 05:05:57 -0400 Received: from fllv0016.ext.ti.com ([198.47.19.142]:53658 "EHLO fllv0016.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932479AbeGDJFx (ORCPT ); Wed, 4 Jul 2018 05:05:53 -0400 Received: from dflxv15.itg.ti.com ([128.247.5.124]) by fllv0016.ext.ti.com (8.15.2/8.15.2) with ESMTP id w6494LUD065656; Wed, 4 Jul 2018 04:04:21 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1530695061; bh=r2OE8sJUi9rHlS+9rxJszYaAItoKn2wRLCXTgktCMPA=; h=Subject:To:CC:References:From:Date:In-Reply-To; b=RdOaiR6WThMrCmFfExPMLmmXr72OVguXF5i87g2QllKiihqMuizZcHaroGtFuvMZu qE8lJhXG7YZ3WgEkvmlblyP7bQhgu64cLxoGq0roJ73uN+biXDFakWh7uMq6Cnry9I HLKeik7EZSt14eEzlE9XgWRbTm8xniNZoT6rOO4E= Received: from DFLE114.ent.ti.com (dfle114.ent.ti.com [10.64.6.35]) by dflxv15.itg.ti.com (8.14.3/8.13.8) with ESMTP id w6494LhQ004281; Wed, 4 Jul 2018 04:04:21 -0500 Received: from DFLE100.ent.ti.com (10.64.6.21) by DFLE114.ent.ti.com (10.64.6.35) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1466.3; Wed, 4 Jul 2018 04:04:20 -0500 Received: from dflp32.itg.ti.com (10.64.6.15) by DFLE100.ent.ti.com (10.64.6.21) with Microsoft SMTP Server (version=TLS1_0, cipher=TLS_RSA_WITH_AES_256_CBC_SHA) id 15.1.1466.3 via Frontend Transport; Wed, 4 Jul 2018 04:04:20 -0500 Received: from [172.24.190.172] (ileax41-snat.itg.ti.com [10.172.224.153]) by dflp32.itg.ti.com (8.14.3/8.13.8) with ESMTP id w6494CDd007627; Wed, 4 Jul 2018 04:04:13 -0500 Subject: Re: [PATCH v4 08/18] net: davinci_emac: potentially get the MAC address from MTD To: Bartosz Golaszewski , Ladislav Michl CC: Florian Fainelli , Kevin Hilman , Russell King , Grygorii Strashko , "David S . Miller" , Srinivas Kandagatla , Lukas Wunner , Rob Herring , Dan Carpenter , Ivan Khoronzhuk , David Lechner , Greg Kroah-Hartman , Andrew Lunn , Jonathan Corbet , Linux ARM , Linux Kernel Mailing List , , , Bartosz Golaszewski References: <20180629094039.7543-1-brgl@bgdev.pl> <20180629094039.7543-9-brgl@bgdev.pl> <03b77e24-9ab9-fa01-2387-9de0408a9942@gmail.com> <20180704070919.GA14051@lenoch> From: Sekhar Nori Message-ID: Date: Wed, 4 Jul 2018 14:34:12 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.8.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 7bit X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wednesday 04 July 2018 01:59 PM, Bartosz Golaszewski wrote: > 2018-07-04 9:09 GMT+02:00 Ladislav Michl : >> On Tue, Jul 03, 2018 at 09:39:51AM -0700, Florian Fainelli wrote: >>> >>> >>> On 06/29/2018 02:40 AM, Bartosz Golaszewski wrote: >>>> From: Bartosz Golaszewski >>>> >>>> On da850-evm board we can read the MAC address from MTD. It's currently >>>> done in the relevant board file, but we want to get rid of all the MAC >>>> reading callbacks from the board file (SPI and NAND). Move the reading >>>> of the MAC address from SPI to the emac driver's probe function. >>> >>> This should be made something generic to all drivers, not just something >>> the davinci_emac driver does, something like this actually: >>> >>> https://lkml.org/lkml/2018/3/24/312 >> >> ...and that's would also make it work when MAC address is stored >> in 24c08 EEPROM, which is quite common. >> > > This is what the second patch for davinci_emac in this series does. I > agree that this should become more generic at some point - we should > probably have a routine somewhere in net that would try to get the MAC > address from all possible sources (nvmem, of etc.). This is somewhat > related to the work I want to do on nvmem to make the at24 setup() > callback more generic. > > Unfortunately we don't have it yet and I will not have time to work on > it before v4.20 so if there are no serious objections, I'd like to get > this series merged for v4.19 and then we can refactor the MAC reading > later. > > How does it sound? I don't think the series introduces any regressions. We need to have MTD and SPI flash built into the kernel even today to get mac address on DA850 EVM. So from that perspective, I don't have objections (I need to actually test still). OTOH, it will be nice to do the conversion once and not piecemeal. That way there is less churn and scope for regressions. So from a mach-davinci perspective, I don't have a very strong position either way. Thanks, Sekhar