Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:8 as permitted sender) client-ip=2620:137:e000::3:8;
Message-ID: <e84824057954fa8593783c4433f5b535f0da6490.camel@linux.intel.com>
Subject: Re: [PATCH v2 0/5] TPMI update for permissions
From:   srinivas pandruvada <srinivas.pandruvada@linux.intel.com>
To:     hdegoede@redhat.com, markgross@kernel.org,
        ilpo.jarvinen@linux.intel.com, andriy.shevchenko@linux.intel.com
Cc:     platform-driver-x86@vger.kernel.org, linux-kernel@vger.kernel.org
Date:   Thu, 30 Nov 2023 18:24:38 -0500
In-Reply-To: <20231130214751.3100418-1-srinivas.pandruvada@linux.intel.com>
References: <20231130214751.3100418-1-srinivas.pandruvada@linux.intel.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
User-Agent: Evolution 3.48.4 (3.48.4-1.fc38) 
MIME-Version: 1.0
Precedence: bulk

On Thu, 2023-11-30 at 13:47 -0800, Srinivas Pandruvada wrote:
> Process read/write and enabled state for feature drivers. When a
> feature
> is disabled, don't create a device to load a feature driver. When a
> read
> is blocked then don't load feature drivers. When write is blocked
> continue
> to function in read only mode.
>=20
> v2:
> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0Dropped patch platform/x8=
6/intel/tpmi: Add additional TPMI
> header fields
> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0Addressed other review co=
mments, details are in each patch
>=20
lkp@intel.com is complaining that these patches can't be applied. I
will fix any issue after review and post v3 if required.

Thanks,
Srinivas


> Srinivas Pandruvada (5):
> =C2=A0 platform/x86/intel/tpmi: Don't create devices for disabled feature=
s
> =C2=A0 platform/x86/intel/tpmi: Modify external interface to get
> read/write
> =C2=A0=C2=A0=C2=A0 state
> =C2=A0 platform/x86/intel/tpmi: Move TPMI ID definition
> =C2=A0 platform/x86: ISST: Process read/write blocked feature status
> =C2=A0 platform/x86/intel-uncore-freq: Process read/write blocked feature
> =C2=A0=C2=A0=C2=A0 status
>=20
> =C2=A0.../intel/speed_select_if/isst_tpmi_core.c=C2=A0=C2=A0=C2=A0 | 25 +=
++++++++++++
> =C2=A0drivers/platform/x86/intel/tpmi.c=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 | 35 +++++++++--------
> --
> =C2=A0.../uncore-frequency/uncore-frequency-tpmi.c=C2=A0 | 15 ++++++++
> =C2=A0include/linux/intel_tpmi.h=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
 | 18 ++++++++--
> =C2=A04 files changed, 72 insertions(+), 21 deletions(-)
>=20