Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754943AbdCWO0F (ORCPT ); Thu, 23 Mar 2017 10:26:05 -0400 Received: from nbd.name ([46.4.11.11]:42266 "EHLO nbd.name" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751349AbdCWO0D (ORCPT ); Thu, 23 Mar 2017 10:26:03 -0400 Subject: Re: [PATCH net-next v2 5/5] net-next: dsa: add dsa support for Mediatek MT7530 switch To: Felix Fietkau , Sean Wang , Andrew Lunn References: <1490088910-19405-1-git-send-email-sean.wang@mediatek.com> <1490088910-19405-6-git-send-email-sean.wang@mediatek.com> <20170323072233.GA10076@lunn.ch> <1490256416.14184.2.camel@mtkswgap22> Cc: mark.rutland@arm.com, devicetree@vger.kernel.org, Landen.Chao@mediatek.com, f.fainelli@gmail.com, keyhaede@gmail.com, vivien.didelot@savoirfairelinux.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, robh+dt@kernel.org, linux-mediatek@lists.infradead.org, objelf@gmail.com, matthias.bgg@gmail.com, davem@davemloft.net From: John Crispin Message-ID: Date: Thu, 23 Mar 2017 15:25:39 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Icedove/45.6.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 962 Lines: 35 On 23/03/17 15:09, Felix Fietkau wrote: > On 2017-03-23 09:06, Sean Wang wrote: >> Hi Andrew, >> >> The purpose for the regmap table registered is to >> >> provide a way which helps us to look up a specific >> >> register on the switch through regmap-debugfs. >> >> >> And not all ranges of register is defined >> >> so I only include the meaningful ones in a sparse way >> >> for the table. > I think in that case it might be nice to make regmap support optional in > order to avoid pulling in bloat on platforms that don't need it. > > - Felix > The 2 relevant platforms are mips/ralink and arm/mediatek. both require regmap for the eth_sysctl syscon if they want to utilize the mtk_soc_eth driver which is a prereq for mt7530. so regmap cannot be optional here. John > > _______________________________________________ > Linux-mediatek mailing list > Linux-mediatek@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-mediatek