From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752449AbbCHTui (ORCPT ); Sun, 8 Mar 2015 15:50:38 -0400 Received: from bh-25.webhostbox.net ([208.91.199.152]:46148 "EHLO bh-25.webhostbox.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751343AbbCHTuf (ORCPT ); Sun, 8 Mar 2015 15:50:35 -0400 Message-ID: <54FCA807.7080505@roeck-us.net> Date: Sun, 08 Mar 2015 12:50:31 -0700 From: Guenter Roeck User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0 MIME-Version: 1.0 To: Andy Lutomirski CC: Jean Delvare , Boaz Harrosh , One Thousand Gnomes , Rui Wang , Alun Evans , Robert Elliott , "linux-i2c@vger.kernel.org" , Wolfram Sang , Mauro Carvalho Chehab , Tony Luck , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH 1/2] i2c_imc: New driver for Intel's iMC, found on LGA2011 chips References: <13443f0542fb447a4c0e558a5f6077c6a76a6e95.1425695891.git.luto@amacapital.net> <54FB0D7D.8060402@roeck-us.net> <54FC6D2B.3060307@roeck-us.net> In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit X-Authenticated_sender: linux@roeck-us.net X-OutGoing-Spam-Status: No, score=-1.0 X-CTCH-PVer: 0000001 X-CTCH-Spam: Unknown X-CTCH-VOD: Unknown X-CTCH-Flags: 0 X-CTCH-RefID: str=0001.0A020206.54FCA80A.00B0,ss=1,re=0.001,recu=0.000,reip=0.000,cl=1,cld=1,fgs=0 X-CTCH-Score: 0.001 X-CTCH-ScoreCust: 0.000 X-CTCH-Rules: C_4847, X-CTCH-SenderID: linux@roeck-us.net X-CTCH-SenderID-Flags: 0 X-CTCH-SenderID-TotalMessages: 1 X-CTCH-SenderID-TotalSpam: 0 X-CTCH-SenderID-TotalSuspected: 0 X-CTCH-SenderID-TotalConfirmed: 0 X-CTCH-SenderID-TotalBulk: 0 X-CTCH-SenderID-TotalVirus: 0 X-CTCH-SenderID-TotalRecipients: 0 X-AntiAbuse: This header was added to track abuse, please include it with any abuse report X-AntiAbuse: Primary Hostname - bh-25.webhostbox.net X-AntiAbuse: Original Domain - vger.kernel.org X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12] X-AntiAbuse: Sender Address Domain - roeck-us.net X-Get-Message-Sender-Via: bh-25.webhostbox.net: mailgid no entry from get_relayhosts_entry X-Source: X-Source-Args: X-Source-Dir: Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 03/08/2015 12:30 PM, Andy Lutomirski wrote: [ ... ] >> >>> One other question: from my reading of the spec, it should be possible to >>> augment this driver to expose a temporate sensor subdevice that shows >>> recent cached temperatures from HW DIMM measurements. They would be >>> redundant with the jc42 outputs, but it would be safe to use them even on >>> systems without safe SMBUS arbitration. Should I do that as a followup >>> later on? >>> >> >> Without thinking too much about it, this should be a separate driver, >> and I think it might actually be more valuable (since less risky) >> than this entire patch set. > > The main problem there is that they'll fight over the PCI ids. > That is why we have mfd drivers. While that most likely won't be a solution here (and I don't suggest it), you can use the same idea. If can not be instantiated as pci driver, the alternative would be to instantiate it as platform driver and let it get its pci information from the parent device. Guenter