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:   Wed, 17 Jun 2020 23:07:35 +0800
From:   Jacky Hu <hengqing.hu@gmail.com>
To:     Alexander Monakov <amonakov@ispras.ru>
Cc:     Guenter Roeck <linux@roeck-us.net>, linux-hwmon@vger.kernel.org,
        linux-edac@vger.kernel.org, linux-kernel@vger.kernel.org,
        tony.luck@intel.com, x86@kernel.org, yazen.ghannam@amd.com,
        bp@alien8.de, clemens@ladisch.de
Subject: Re: [PATCH] hwmon: (k10temp) Add AMD family 17h model 60h probe
Message-ID: <20200617150735.GA405893@i716>
References: <20200616180940.GN13515@zn.tnic>
 <20200617013255.391975-1-hengqing.hu@gmail.com>
 <20200617034028.GA1614@roeck-us.net>
 <20200617071927.GA398128@i716>
 <alpine.LNX.2.20.13.2006171739010.31660@monopod.intra.ispras.ru>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
In-Reply-To: <alpine.LNX.2.20.13.2006171739010.31660@monopod.intra.ispras.ru>
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Hi,

Sorry, I apologize for didn't do much lookup that you already did the patch
submission before I submitted the patch.
I have to say we are all programmed by the programs.
Also I didn't submit to either of the lists.
A few places I did looked at are below before I did the submission.
https://pci-ids.ucw.cz/v2.2/pci.ids
https://lore.kernel.org/patchwork/project/lkml/list/

Jacky
On Wed, Jun 17, 2020 at 05:54:36PM +0300, Alexander Monakov wrote:
> Hi,
> 
> I've already said in my patch submission (which was Cc'ed to LKML,
> linux-edac and linux-hwmon so you should have been able to find it):
> 
> >> It appears SMU offsets for reading current/voltage and CCD temperature
> >> have changed for this generation (reads from currently used offsets
> >> yield zeros), so those features cannot be enabled so trivially.
> 
> In https://github.com/FlyGoat/ryzen_nb_smu/issues/3 there some
> reverse-engineered info that indicates that for Renoir, SMU region has been
> moved to 0x6F000. Its layout also changed, as far as I can tell.
> 
> (also, please ask yourself if you want to offer the maintainers an apology)
> 
> Alexander