From mboxrd@z Thu Jan 1 00:00:00 1970 Received: with ECARTIS (v1.0.0; list linux-mips); Thu, 11 Feb 2016 09:15:50 +0100 (CET) Received: from mailapp01.imgtec.com ([195.59.15.196]:40141 "EHLO mailapp01.imgtec.com" rhost-flags-OK-OK-OK-OK) by eddie.linux-mips.org with ESMTP id S27011013AbcBKIPszwTmy (ORCPT ); Thu, 11 Feb 2016 09:15:48 +0100 Received: from hhmail02.hh.imgtec.org (unknown [10.100.10.20]) by Websense Email Security Gateway with ESMTPS id B66ABE7F071FE; Thu, 11 Feb 2016 08:15:41 +0000 (GMT) Received: from LEMAIL01.le.imgtec.org (192.168.152.62) by hhmail02.hh.imgtec.org (10.100.10.20) with Microsoft SMTP Server (TLS) id 14.3.266.1; Thu, 11 Feb 2016 08:15:42 +0000 Received: from [192.168.154.116] (192.168.154.116) by LEMAIL01.le.imgtec.org (192.168.152.62) with Microsoft SMTP Server (TLS) id 14.3.210.2; Thu, 11 Feb 2016 08:15:42 +0000 Subject: Re: [PATCH v5] mmc: OCTEON: Add host driver for OCTEON MMC controller To: Florian Fainelli , David Daney , Aaro Koskinen References: <1455125775-7245-1-git-send-email-matt.redfearn@imgtec.com> <56BB7B2F.60307@caviumnetworks.com> <20160210234907.GC1640@darkstar.musicnaut.iki.fi> <56BBD6AD.2090902@caviumnetworks.com> <56BBF83B.8020908@gmail.com> CC: , , , , , , , Aleksey Makarov , Chandrakala Chavva , Aleksey Makarov , Leonid Rosenboim , Peter Swain , Aaron Williams From: Matt Redfearn Message-ID: <56BC432E.8000600@imgtec.com> Date: Thu, 11 Feb 2016 08:15:42 +0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 MIME-Version: 1.0 In-Reply-To: <56BBF83B.8020908@gmail.com> Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 8bit X-Originating-IP: [192.168.154.116] Return-Path: X-Envelope-To: <"|/home/ecartis/ecartis -s linux-mips"> (uid 0) X-Orcpt: rfc822;linux-mips@linux-mips.org Original-Recipient: rfc822;linux-mips@linux-mips.org X-archive-position: 51992 X-ecartis-version: Ecartis v1.0.0 Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org X-original-sender: matt.redfearn@imgtec.com Precedence: bulk List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-Id: linux-mips X-List-ID: linux-mips List-subscribe: List-owner: List-post: List-archive: X-list: linux-mips Hi Florian. On 11/02/16 02:55, Florian Fainelli wrote: > Le 10/02/2016 16:32, David Daney a écrit : >> On 02/10/2016 03:49 PM, Aaro Koskinen wrote: >>> Hi, >>> >>> On Wed, Feb 10, 2016 at 10:02:23AM -0800, David Daney wrote: >>>> On 02/10/2016 09:36 AM, Matt Redfearn wrote: >>>>> + pr_warn(FW_WARN "%s: Legacy property '%s'. Please remove\n", >>>>> + node->full_name, legacy_name); >>>> I don't like this warning message. >>>> >>>> The vast majority of people that see it will not be able to change their >>>> firmware. So it will be forever cluttering up their boot logs. >>> Until they switch to use APPENDED_DTB. :-) >>> >> I am philosophically opposed to making the DTB an internal kernel >> implementation detail. >> >> For OCTEON boards, it is an ABI between the boot firmware and the >> kernel, and is impractical to change. >> >> One could argue that many years ago, when the decision was made (by me), >> that we should have opted to carry in the kernel source code tree the >> DTS files for all OCTEON boards ever made, but we did not do that. Due >> to the non-reversibility of time, the decision is hard to reverse. >> >> In the case of this MMC driver, the only real difference is that two >> properties have legacy names that later had differing "official" names. >> The overhead of carrying the legacy bindings is very low. > Since there is an existing FDT patching infrastructure in > arch/mips/cavium-octeon/ would not that be a place where you could put > an adaptation layer between your legacy firmware properties and the > upstream binding? Thanks for your constructive advice. That does, indeed, look like a better place to put this. Thanks, Matt From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailapp01.imgtec.com ([195.59.15.196]:40141 "EHLO mailapp01.imgtec.com" rhost-flags-OK-OK-OK-OK) by eddie.linux-mips.org with ESMTP id S27011013AbcBKIPszwTmy (ORCPT ); Thu, 11 Feb 2016 09:15:48 +0100 Subject: Re: [PATCH v5] mmc: OCTEON: Add host driver for OCTEON MMC controller References: <1455125775-7245-1-git-send-email-matt.redfearn@imgtec.com> <56BB7B2F.60307@caviumnetworks.com> <20160210234907.GC1640@darkstar.musicnaut.iki.fi> <56BBD6AD.2090902@caviumnetworks.com> <56BBF83B.8020908@gmail.com> From: Matt Redfearn Message-ID: <56BC432E.8000600@imgtec.com> Date: Thu, 11 Feb 2016 08:15:42 +0000 MIME-Version: 1.0 In-Reply-To: <56BBF83B.8020908@gmail.com> Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 8bit Return-Path: Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-subscribe: List-owner: List-post: List-archive: To: Florian Fainelli , David Daney , Aaro Koskinen Cc: linux-mmc@vger.kernel.org, david.daney@cavium.com, ulf.hansson@linaro.org, linux-mips@linux-mips.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, Zubair.Kakakhel@imgtec.com, Aleksey Makarov , Chandrakala Chavva , Aleksey Makarov , Leonid Rosenboim , Peter Swain , Aaron Williams Message-ID: <20160211081542.v4pBi0yPPzy5g8NUqaPHBclwxaruLrSD9fRoKRSJufw@z> Hi Florian. On 11/02/16 02:55, Florian Fainelli wrote: > Le 10/02/2016 16:32, David Daney a écrit : >> On 02/10/2016 03:49 PM, Aaro Koskinen wrote: >>> Hi, >>> >>> On Wed, Feb 10, 2016 at 10:02:23AM -0800, David Daney wrote: >>>> On 02/10/2016 09:36 AM, Matt Redfearn wrote: >>>>> + pr_warn(FW_WARN "%s: Legacy property '%s'. Please remove\n", >>>>> + node->full_name, legacy_name); >>>> I don't like this warning message. >>>> >>>> The vast majority of people that see it will not be able to change their >>>> firmware. So it will be forever cluttering up their boot logs. >>> Until they switch to use APPENDED_DTB. :-) >>> >> I am philosophically opposed to making the DTB an internal kernel >> implementation detail. >> >> For OCTEON boards, it is an ABI between the boot firmware and the >> kernel, and is impractical to change. >> >> One could argue that many years ago, when the decision was made (by me), >> that we should have opted to carry in the kernel source code tree the >> DTS files for all OCTEON boards ever made, but we did not do that. Due >> to the non-reversibility of time, the decision is hard to reverse. >> >> In the case of this MMC driver, the only real difference is that two >> properties have legacy names that later had differing "official" names. >> The overhead of carrying the legacy bindings is very low. > Since there is an existing FDT patching infrastructure in > arch/mips/cavium-octeon/ would not that be a place where you could put > an adaptation layer between your legacy firmware properties and the > upstream binding? Thanks for your constructive advice. That does, indeed, look like a better place to put this. Thanks, Matt From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matt Redfearn Subject: Re: [PATCH v5] mmc: OCTEON: Add host driver for OCTEON MMC controller Date: Thu, 11 Feb 2016 08:15:42 +0000 Message-ID: <56BC432E.8000600@imgtec.com> References: <1455125775-7245-1-git-send-email-matt.redfearn@imgtec.com> <56BB7B2F.60307@caviumnetworks.com> <20160210234907.GC1640@darkstar.musicnaut.iki.fi> <56BBD6AD.2090902@caviumnetworks.com> <56BBF83B.8020908@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: <56BBF83B.8020908-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Florian Fainelli , David Daney , Aaro Koskinen Cc: linux-mmc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, david.daney-YGCgFSpz5w/QT0dZR+AlfA@public.gmane.org, ulf.hansson-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org, linux-mips-6z/3iImG2C8G8FEW9MqTrA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Zubair.Kakakhel-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org, Aleksey Makarov , Chandrakala Chavva , Aleksey Makarov , Leonid Rosenboim , Peter Swain , Aaron Williams List-Id: linux-mmc@vger.kernel.org Hi Florian. On 11/02/16 02:55, Florian Fainelli wrote: > Le 10/02/2016 16:32, David Daney a =C3=A9crit : >> On 02/10/2016 03:49 PM, Aaro Koskinen wrote: >>> Hi, >>> >>> On Wed, Feb 10, 2016 at 10:02:23AM -0800, David Daney wrote: >>>> On 02/10/2016 09:36 AM, Matt Redfearn wrote: >>>>> + pr_warn(FW_WARN "%s: Legacy property '%s'. Please remove= \n", >>>>> + node->full_name, legacy_name); >>>> I don't like this warning message. >>>> >>>> The vast majority of people that see it will not be able to change= their >>>> firmware. So it will be forever cluttering up their boot logs. >>> Until they switch to use APPENDED_DTB. :-) >>> >> I am philosophically opposed to making the DTB an internal kernel >> implementation detail. >> >> For OCTEON boards, it is an ABI between the boot firmware and the >> kernel, and is impractical to change. >> >> One could argue that many years ago, when the decision was made (by = me), >> that we should have opted to carry in the kernel source code tree th= e >> DTS files for all OCTEON boards ever made, but we did not do that. = Due >> to the non-reversibility of time, the decision is hard to reverse. >> >> In the case of this MMC driver, the only real difference is that two >> properties have legacy names that later had differing "official" nam= es. >> The overhead of carrying the legacy bindings is very low. > Since there is an existing FDT patching infrastructure in > arch/mips/cavium-octeon/ would not that be a place where you could pu= t > an adaptation layer between your legacy firmware properties and the > upstream binding? Thanks for your constructive advice. That does, indeed, look like a=20 better place to put this. Thanks, Matt -- To unsubscribe from this list: send the line "unsubscribe devicetree" i= n the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html