Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Oleksandr Natalenko <oleksandr@natalenko.name>
To:     Zev Weiss <zev@bewilderbeest.net>
Cc:     linux-hwmon@vger.kernel.org, Guenter Roeck <linux@roeck-us.net>,
        Jean Delvare <jdelvare@suse.com>, openbmc@lists.ozlabs.org,
        linux-kernel@vger.kernel.org, Renze Nicolai <renze@rnplus.nl>,
        Rob Herring <robh+dt@kernel.org>, devicetree@vger.kernel.org
Subject: Re: [PATCH 0/5] hwmon: (nct6775) Add i2c support
Date:   Sun, 27 Feb 2022 22:38:31 +0100
Message-ID: <4719747.31r3eYUQgx@natalenko.name>
In-Reply-To: <YhrFizhgOpZbi3dE@hatter.bewilderbeest.net>
References: <20220226133047.6226-1-zev@bewilderbeest.net> <2620147.mvXUDI8C0e@natalenko.name> <YhrFizhgOpZbi3dE@hatter.bewilderbeest.net>
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

Hello.

On ned=C4=9Ble 27. =C3=BAnora 2022 1:27:55 CET Zev Weiss wrote:
> On Sat, Feb 26, 2022 at 04:14:12PM PST, Oleksandr Natalenko wrote:
> >Hello.
> >
> >On sobota 26. =C3=BAnora 2022 14:30:42 CET Zev Weiss wrote:
> >> Hello,
> >>
> >> This patch series augments the existing nct6775 driver with support
> >> for the hardware's i2c interface.
> >
> >Is it something I can test on my ASUS Pro WS X570-ACE board as an ordina=
ry user, and if so, how?
> >
>=20
> You could certainly test that the nct6775-platform driver still works as=
=20
> it did previously, which would be good to confirm -- you'll need to=20
> enable CONFIG_SENSORS_NCT6775_PLATFORM now to build it.

Ack.

>  From what I've been able to find about that board though it looks like=20
> it doesn't have a BMC, so testing the i2c driver on it probably isn't=20
> going to be possible.  (Even if it does in fact have a BMC, it would=20
> require at least a partial port of OpenBMC or similar, and re-flashing=20
> your BMC firmware with that, and is hence a non-trivial undertaking.)

It should have, the BMC is based on RTL8117, although I have no idea if it =
is something that can be called true IPMI as I've never enabled/used it.

Thanks.

>=20
>=20
> Zev
>=20
>=20


=2D-=20
Oleksandr Natalenko (post-factum)