From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752177AbdLFU72 (ORCPT ); Wed, 6 Dec 2017 15:59:28 -0500 Received: from lelnx194.ext.ti.com ([198.47.27.80]:30211 "EHLO lelnx194.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751544AbdLFU7Z (ORCPT ); Wed, 6 Dec 2017 15:59:25 -0500 Subject: Re: [RFC PATCH 1/3] ASoC: Add platforms directory To: Mark Brown CC: Liam Girdwood , , References: <20171205181448.18513-1-afd@ti.com> <20171205181448.18513-2-afd@ti.com> <20171206123908.GB1827@finisterre> <561a6ebd-c978-c52a-a13a-3300be9a58bb@ti.com> <20171206172924.mrd233uipzorrymc@sirena.org.uk> <20171206184219.fhhohxdihsm33kf6@sirena.org.uk> <20171206192707.vurfm6whlzku3ifk@sirena.org.uk> From: "Andrew F. Davis" Message-ID: <5aaa3cdc-613d-25fe-0149-db91f0b9d8c6@ti.com> Date: Wed, 6 Dec 2017 14:59:14 -0600 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 In-Reply-To: <20171206192707.vurfm6whlzku3ifk@sirena.org.uk> Content-Type: text/plain; charset="windows-1252" Content-Language: en-US Content-Transfer-Encoding: 7bit X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 12/06/2017 01:27 PM, Mark Brown wrote: > On Wed, Dec 06, 2017 at 12:49:39PM -0600, Andrew F. Davis wrote: >> On 12/06/2017 12:42 PM, Mark Brown wrote: > >>> We need to preserve old bindings to ensure DT compatiblity, the easiest >>> way to do that is to keep old machine drivers around. There are plenty >>> of older drivers that wouldn't be accepted now but would at least need >>> replacing with a compatibility layer that adapts the bindings onto one >>> of the generic drivers. That adaption layer would definitely be useful >>> (basically a big table of platform data) but it'd take time to implement >>> it. > >> We then should at least start depreciating them now so that someday we >> can drop that stuff. Isolating them would be the first step. > > Moving the drivers around is not going to help with that. For users the > drivers are not deprecated until someone actually steps up and makes > something that allows the generic drivers to handle the old bindings and > moves them over, at which point we'd just remove things that have been > converted. We can't just tell them not to use something without > providing an alternative. For developers they're just going to end up > with the simpler machine drivers sitting next to a bunch of machine > drivers that reasonably exist which I'm not sure clarifies anything. > It's an orthogonal problem. > It helps in that people would see an organized set of drivers to target, "this folder here (sound/soc/machines/) should be reduced as much as possible". Otherwise it is just, "go dig around for the machine drivers burred together with platform drivers, which file is which type? Who knows!?" I will help too, but there needs to be just a bit of organization first. >>> Wouldn't a few regexps in the MAINTAINERS file cover it? We've already >>> got a bunch of vendors doing this. > >> pcm* >> tas* >> tlv* >> twl* > >> It's messy how many prefixes we have :/ > > Eh, not that bad. And easy enough to do anyway. > And it is easy enough to group them into a directory by something like vendor, same as everyone else, including your very own sound/soc/. Yet codecs/ is a giant 400+ file blob, by C file count it is the largest directory in the whole kernel... Top 10: C files count: folder 130: drivers/staging/comedi/drivers 145: drivers/gpio 159: drivers/watchdog 164: drivers/rtc 165: drivers/hwmon 172: net/netfilter 175: drivers/acpi/acpica 194: lib 201: drivers/mfd 239: sound/soc/codecs Maybe a bit of grouping wouldn't be so bad