Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Fri, 20 Nov 2020 16:46:23 -0800
From:   Andres Freund <andres@anarazel.de>
To:     Daniel Lezcano <daniel.lezcano@linaro.org>
Cc:     Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>,
        Tushar Dave <tushar.n.dave@intel.com>,
        Zhang Rui <rui.zhang@intel.com>, linux-pm@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH] thermal: intel_pch_thermal: Add PCI ids for Lewisburg
 PCH.
Message-ID: <20201121004623.mgbcn5ddo7hjdpfq@alap3.anarazel.de>
References: <2fd3733b-ed67-80e0-7b27-8e3c421eeb9c@linaro.org>
 <20201113204916.1144907-1-andres@anarazel.de>
 <116a81e6-7989-8077-1fc4-c9f047256e9a@linaro.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <116a81e6-7989-8077-1fc4-c9f047256e9a@linaro.org>
Precedence: bulk

Hi,

On 2020-11-16 10:28:04 +0100, Daniel Lezcano wrote:
> On 13/11/2020 21:49, Andres Freund wrote:
> > I noticed that I couldn't read the PCH temperature on my workstation
> > (C620 series chipset, w/ 2x Xeon Gold 5215 CPUs) directly, but had to go
> > through IPMI. Looking at the data sheet, it looks to me like the
> > existing intel PCH thermal driver should work without changes for
> > Lewisburg.
> > 
> > I suspect there's some other PCI IDs missing. But I hope somebody at
> > Intel would have an easier time figuring that out than I...
> > 
> > Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
> > Cc: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
> > Cc: Tushar Dave <tushar.n.dave@intel.com>
> > Cc: Zhang Rui <rui.zhang@intel.com>
> > Cc: linux-pm@vger.kernel.org
> > Cc: linux-kernel@vger.kernel.org
> > Link: https://lore.kernel.org/lkml/20200115184415.1726953-1-andres@anarazel.de/
> > Signed-off-by: Andres Freund <andres@anarazel.de>
> > ---
> 
> Applied, thanks

Thanks for review and applying.

Greetings,

Andres Freund