From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?Q?Andreas_F=c3=a4rber?= Subject: Re: Sense of soc bus? (was: [PATCH] base: soc: Export soc_device_to_device() helper) Date: Fri, 15 Nov 2019 13:00:59 +0100 Message-ID: References: <20191103013645.9856-3-afaerber@suse.de> <20191111045609.7026-1-afaerber@suse.de> <20191111052741.GB3176397@kroah.com> <586fa37c-6292-aca4-fa7c-73064858afaf@suse.de> <20191111064040.GA3502217@kroah.com> <20191112052347.GA1197504@kroah.com> <20191112072926.isjxfa4ci6akhx56@pengutronix.de> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: In-Reply-To: Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org To: Geert Uytterhoeven Cc: Neil Armstrong , Greg Kroah-Hartman , Geert Uytterhoeven , linux-realtek-soc@lists.infradead.org, Tony Lindgren , Linus Walleij , Bjorn Andersson , Thierry Reding , Lee Jones , Rob Herring , Kevin Hilman , "Rafael J. Wysocki" , Michal Simek , Jonathan Hunter , NXP Linux Team , =?UTF-8?Q?Uwe_Kleine-K=c3=b6nig?= , boot-architecture@lists.linaro.org, Sascha Hauer , Fabio List-Id: linux-tegra@vger.kernel.org Hi Geert, Am 15.11.19 um 09:58 schrieb Geert Uytterhoeven: > On Fri, Nov 15, 2019 at 9:52 AM Neil Armstrong wrote: >> On 12/11/2019 11:47, Andreas Färber wrote: >>> For example, RTD1295 will support LSADC only from revision B00 >>> on (and it's not the first time I'm seeing such things in the industry). >>> So if a user complains, it will be helpful to see that information. >>> >>> Referencing your Amlogic review, with all due respect for its authors, >>> the common framework here just lets that information evaporate into the >>> deeps of sysfs. >> >> Hopefully we never had the case where needed to use the soc info in drivers, >> but now we have one and having such infrastructure already in-place will help. >> >> Renesas platforms makes a extensive usage of the soc info infrastructure to >> figure out plenty of HW parameters at runtime and lower their DT changes. > > We do our best to use it solely for detecting quirks in early SoC revisions. Got a pointer? I fail to immediately understand how sysfs would help drivers (as opposed to userspace) detect quirks: Parsing strings back doesn't sound efficient, and I don't see you exporting any custom APIs in drivers/soc/renesas/renesas-soc.c? Regards, Andreas -- SUSE Software Solutions Germany GmbH Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer HRB 36809 (AG Nürnberg)