Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.36 as permitted sender) client-ip=23.128.96.36;
From:   "Wu, Wentong" <wentong.wu@intel.com>
To:     Sakari Ailus <sakari.ailus@linux.intel.com>
CC:     "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
        "Winkler, Tomas" <tomas.winkler@intel.com>,
        "hdegoede@redhat.com" <hdegoede@redhat.com>,
        "andriy.shevchenko@linux.intel.com" 
        <andriy.shevchenko@linux.intel.com>,
        "Usyskin, Alexander" <alexander.usyskin@intel.com>,
        "Wang, Zhifeng" <zhifeng.wang@intel.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH 2/2] mei: Add MEI hardware support for IVSC device
Thread-Topic: [PATCH 2/2] mei: Add MEI hardware support for IVSC device
Thread-Index: AQHaIfdexoS2VwuEC0a0G/ZumI2RtrCRCGqAgABLeyA=
Date:   Wed, 29 Nov 2023 14:11:21 +0000
Message-ID: <DM6PR11MB43169C037DC770A0941BF7E78D83A@DM6PR11MB4316.namprd11.prod.outlook.com>
References: <1701174846-16316-1-git-send-email-wentong.wu@intel.com>
 <1701174846-16316-3-git-send-email-wentong.wu@intel.com>
 <ZWcD18B916Xs1JiU@kekkonen.localdomain>
In-Reply-To: <ZWcD18B916Xs1JiU@kekkonen.localdomain>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?lV0qQTM3wsYl6an8yZ4N/JYlPV1PQciIiWh7vgMPOyMHrxVbNsXkfYyKBiFm?=
 =?us-ascii?Q?OpCLY1sbiLcVib+1zW465SoxF9FgIHDDnXffg3e5bWFzmzNiA5M9m5kQ7WSz?=
 =?us-ascii?Q?WMG3KVS3AGsHook0vVJe4y+w/+n7cLP6jymKZM/dpF81tS9QylbkSPUM8O/y?=
 =?us-ascii?Q?VAbsQjcr8cOapqySAZvPQ6d76XokJ1RwBQQ/NwdH/wFDkUXZ/dDcc/HTVhnd?=
 =?us-ascii?Q?NDydyxKi/+2lfQkvIJnERT0ihsDK+Ip2xHDd9UdOTpuxv+aE3ZmocwjT2wje?=
 =?us-ascii?Q?hoxL91oyV1aQuOUgDvxDrRYX7mJtXNP0JsKC9QYJORccNxYHTBZ/2Ts4K9G4?=
 =?us-ascii?Q?iHsdmwum16Pyy0DbD8zU/24ZbkoobHHeGXfZBDgSvMTCMH7OCmbf3f2/uHEy?=
 =?us-ascii?Q?DVZfl0twU3iS290cVHworEAZC3qdpMWuVhm8/7vVKvaFMo2IfuT/aDw9QTXD?=
 =?us-ascii?Q?oPVwqfPc9OSKCtSx5+NwIEAd/OjCZATxsyULAbkTDk88EpwoGVoV/bt2a7zU?=
 =?us-ascii?Q?v4PIkUkEJ1Yp2h4LvpG6AUThrOHFjRyg1Bu8zU9w3eoeVJiXrObfYk9JOtrR?=
 =?us-ascii?Q?69/b1SR4p5V2xg8x08rTW9gKAHK5DW2bnYPXwEFFO2QlwodlwVmwIofeutCh?=
 =?us-ascii?Q?eK90xPym+/FzCCEn4rUhCjjJMWI8pYW0NiT+Cm6Tl0biQ+KPH6Qw4QluUJBy?=
 =?us-ascii?Q?S0mMtncY97ldjU3NwcDvmvViRGZh9IrGGReukrF8E+Z6DvvVprb2mHQM5dQw?=
 =?us-ascii?Q?0dkeOz1CD2ovuJlNoG5KOWbIkvyGhypJqNbCndl+i7I6WkbtMFCVHwLd97EQ?=
 =?us-ascii?Q?+127AuocNWebXPh5r3nfE8gBB0gZ6hhkUE1rx5clez0ofPOh+XMAvYQH4N/N?=
 =?us-ascii?Q?JDHcrcT5zO7XUKeUPDGkpcSvfzV2e8Hf2LZ8Z6qxY2BybJN6v8/QS3bdGPXQ?=
 =?us-ascii?Q?rXA63YrUKZwf36mEiePGP2d5bXaMlNEx5bE0WlQqWZdxClXheFOWykxYNlsQ?=
 =?us-ascii?Q?pjYDYVmrSOajlVwdJyeUdJnEWszGE4aqnjbhqtqiwMsv2PkUJEXJYETD8320?=
 =?us-ascii?Q?CsTDFRDPFyvQSxsXFf7VZRLyJmdjoclH92NrR6OpL5takJ1skv9kcK711LYU?=
 =?us-ascii?Q?sMFD4C3ExF2uF5mFBo1zZ5uYmo8/3BNFbKMS/cRD5/c5s+K3ZT1EC7bOIgD9?=
 =?us-ascii?Q?oIY3AQi+LNnn80NEkd/b9ZUjpL/Lr6n/VYql9s4wrukHjG3Tly2BDhNrV801?=
 =?us-ascii?Q?sCtxe3yjR9L8DMRMKFTpw6fFG/Lu7SHBCQqabd4SEDkqu9Yn4aF/JDAZywAe?=
 =?us-ascii?Q?RjmYbCFRSoMbwvtJx6akGkrxLSRp7WYMeoUSEsHDOabL84BGl0NpcefDExd1?=
 =?us-ascii?Q?WFhrvv8+qBdFfHoLR25kgpAhxzwcdlWmvNkyGKarJLm7uonBvR6NQfYkQ332?=
 =?us-ascii?Q?4WPKyvW6+qfHcdj5XR4mmarfi3HFkcYOID6ItM9ir73XHoMf7pk/J5K4vDLn?=
 =?us-ascii?Q?gadH8VXzoafLIwRrrHBOB9bEm8CoeB9lSTegtNC+4t+/PtMNxzZt4GayckKS?=
 =?us-ascii?Q?DyAEJlAWeGE7sH7Va2df6t0OKW8EtxrAf62Sghh2?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DM6PR11MB4316.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: c6f275e5-293a-4c60-cc6b-08dbf0e51022
X-MS-Exchange-CrossTenant-originalarrivaltime: 29 Nov 2023 14:11:21.1656
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: F3stY+MnCpjAm/2ZQa0pdjfWQPpghHroar/3xJiIDJPoWxWOns2NUkoD7eI053uQN5Z5EkA3M1WmhtGBWQaBIg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA2PR11MB4857
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE
	autolearn=unavailable autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on pete.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (pete.vger.email [0.0.0.0]); Wed, 29 Nov 2023 06:11:48 -0800 (PST)

> From: Sakari Ailus <sakari.ailus@linux.intel.com>
> On Tue, Nov 28, 2023 at 08:34:06PM +0800, Wentong Wu wrote:
> > The protocol used for the IVSC device to communicate with HOST is MEI.
> > The MEI hardware interfaces for the IVSC device are implemented.
> >
> > The APIs are exposed by MEI framework to mei clients, e.g. mei_csi and
> > mei_ace.
> >
> > Signed-off-by: Wentong Wu <wentong.wu@intel.com>
> > Reviewed-by: Alexander Usyskin <alexander.usyskin@intel.com>
> > Reviewed-by: Sakari Ailus <sakari.ailus@linux.intel.com>
> > ---
> >  drivers/misc/mei/Kconfig        |  13 ++
> >  drivers/misc/mei/Makefile       |   3 +
> >  drivers/misc/mei/platform-vsc.c | 442
> > ++++++++++++++++++++++++++++++++++++++++
> >  3 files changed, 458 insertions(+)
> >  create mode 100644 drivers/misc/mei/platform-vsc.c
> >
> > diff --git a/drivers/misc/mei/Kconfig b/drivers/misc/mei/Kconfig index
> > 470957a..2c5312b 100644
> > --- a/drivers/misc/mei/Kconfig
> > +++ b/drivers/misc/mei/Kconfig
> > @@ -71,6 +71,19 @@ config INTEL_MEI_VSC_HW
> >  	  This driver can also be built as a module. If so, the module
> >  	  will be called mei-vsc-hw.
> >
> > +config INTEL_MEI_VSC
> > +	tristate "Intel visual sensing controller device with ME interface"
> > +	select INTEL_MEI_VSC_HW
>=20
> Changing the select here to depends on addresses the Kconfig option
> dependency issue=20

Yes, this is the plan to address the warning. Thanks

> (as select just blindly selects the options while ignoring their
> dependencies).
>=20
> I wouldn't mind having a single Kconfig option for the two drivers either=
.
> They're always used together, aren't they?

Thanks and yes, currently they're used together, but we may have more
flexibility if keep them separate to cover future hardware which may has
different transport layer.

BR,
Wentong
>=20
> > +	depends on INTEL_MEI
> > +	help
> > +	  Intel MEI over SPI driver for Intel visual sensing controller
> > +	  (IVSC) device embedded in IA platform. It supports camera sharing
> > +	  between IVSC for context sensing and IPU for typical media usage.
> > +	  Select this config will enable transport layer for IVSC device.
> > +
> > +	  This driver can also be built as a module. If so, the module
> > +	  will be called mei-vsc.
> > +
> >  source "drivers/misc/mei/hdcp/Kconfig"
> >  source "drivers/misc/mei/pxp/Kconfig"
> >  source "drivers/misc/mei/gsc_proxy/Kconfig"
>=20
> --
> Regards,
>=20
> Sakari Ailus