From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guenter Roeck Subject: Re: [PATCH 2/2] hwmon: (ntc_thermistor): add support for B57891S0103 from Epcos Date: Mon, 19 Nov 2018 13:28:06 -0800 Message-ID: <20181119212806.GC5795@roeck-us.net> References: <20181117121248.32143-1-peda@axentia.se> <20181117121248.32143-2-peda@axentia.se> <20181119170732.GA27435@roeck-us.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Peter Rosin Cc: "linux-kernel@vger.kernel.org" , Jean Delvare , Rob Herring , Mark Rutland , "linux-hwmon@vger.kernel.org" , "devicetree@vger.kernel.org" List-Id: devicetree@vger.kernel.org On Mon, Nov 19, 2018 at 09:16:12PM +0000, Peter Rosin wrote: > > > > Guess I deserve the non-alphabetic order as penalty for not enforcing it > > earlier. I'll accept the patch after DT approval and submit another one > > myself afterwards to restore alphabetic order. > > Right, I'm thinking another good change would be to introduce an enum > into the ntc_thermistor_id array, because the hard-coded numbering in Yes, sounds like a good idea. > the ntc_match variable is a bit fragile in my taste, and that list would > also benefit from being alphabetic. Currently there's simply no way to > add things in the middle without causing mayhem. I thought about doing > that, but didn't want to waste energy doing it up front without knowing > if it would be well received (the driver might have been superseded or It would. Guenter > something). I'm glad that you have volunteered to clean things up. > Ha! :-) > > Cheers, > Peter > > > Guenter > > > >> }; > >> > >> struct ntc_thermistor_platform_data { > >> -- > >> 2.11.0 > >> >