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;
Date:   Sun, 26 Mar 2023 01:26:55 +0800
From:   "Ye, Xiang" <xiang.ye@intel.com>
To:     Greg Kroah-Hartman <gregkh@linuxfoundation.org>
CC:     Arnd Bergmann <arnd@arndb.de>,
        Matthias Kaehlcke <mka@chromium.org>,
        "Lee Jones" <lee@kernel.org>, Wolfram Sang <wsa@kernel.org>,
        Tyrone Ting <kfting@nuvoton.com>,
        Mark Brown <broonie@kernel.org>,
        Linus Walleij <linus.walleij@linaro.org>,
        Bartosz Golaszewski <brgl@bgdev.pl>,
        <linux-usb@vger.kernel.org>, <linux-i2c@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>, <linux-spi@vger.kernel.org>,
        <linux-gpio@vger.kernel.org>, <srinivas.pandruvada@intel.com>,
        <heikki.krogerus@linux.intel.com>,
        <andriy.shevchenko@linux.intel.com>,
        <sakari.ailus@linux.intel.com>, <zhifeng.wang@intel.com>,
        <wentong.wu@intel.com>, <lixu.zhang@intel.com>
Subject: Re: [PATCH v7 1/6] usb: Add support for Intel LJCA device
Message-ID: <ZB8u3wWTQhg4PvQe@ye-NUC7i7DNHE>
References: <20230325154711.2419569-1-xiang.ye@intel.com>
 <20230325154711.2419569-2-xiang.ye@intel.com>
 <ZB8fOTpD/uuexaV1@kroah.com>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <ZB8fOTpD/uuexaV1@kroah.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?/yIX148P+LcbC9+FpnIS5ZlBRLHrbK/XPv5+wP1+mAALNVJ3K4gaJ5uWRka2?=
 =?us-ascii?Q?A+4JDKvh4XvNVZo6ypNfsbPYHOPfJ3aT/IJXvsgr8v13zVozF7Tf6JpzwAbz?=
 =?us-ascii?Q?qaPCG2dd/LPBY2eDXItBWgZk5uIvFa/lzWR5W9pMYPgtLjvcVvMSAGKiUIoa?=
 =?us-ascii?Q?EHYMmLj/iUgcg+BAKQu2nPGLpgGKIYVxeA5gympl4mvkUxbfdnITw0uLXjCE?=
 =?us-ascii?Q?JY2jCHIVfN2T2R+7yxmftAmEPlzemHkAIu9sGw+qwRQdCbmNIVlRYpn+tJo0?=
 =?us-ascii?Q?l/dIG6kLeO+rAZ+5gLfZ+kMqi9VyIk0awtyyDluZ0QdYvGZsm7MpepsDtizl?=
 =?us-ascii?Q?bN9JIdJf+NMeUmVFtPMOwF3iSOb2fX7vEAbvTLJbuAtVkvDmZMWDo0hq7I5R?=
 =?us-ascii?Q?xCsBtNXClOMel99/bmXzXnOy535i1I0FNivNgYVlfFHl8l9eFuccm1E+Z4Cf?=
 =?us-ascii?Q?eab9A4njgHFrxd0hj3AiQE9IEAT1O2cRcl5BpwsFyU35mAFTvrSgnu2Davuh?=
 =?us-ascii?Q?iyA5JiKf8xdm86ipk5pKSYCFmvSfZhpS5m2AZD8uEyXVTm71jXCG7gopumoi?=
 =?us-ascii?Q?E+bpJvIqKDoIEqRD++vULXDNZT0JS70nSXI2tJKAk3mASoIT/atcrtW1NHEh?=
 =?us-ascii?Q?SbUvIFrXKf7Zbd+9Zv656oNHS8awO6b+Bw7kfcGWFfpoURB/q/jB87LARLqe?=
 =?us-ascii?Q?o6Py9Jxwr/yhUH6e38itRE4Qgqnis5Fhk+AL4FmxgGwuKb/XAmSWnCyunjb8?=
 =?us-ascii?Q?Y5AjTz3scxl0CXCbfWtrO7+/4c6eIel6w9UmdOrMk9June+YGZ1TB7YmfJwf?=
 =?us-ascii?Q?dK+n9IIRVkFjHshyVjBXcF51YvJbXomjBKjXht/7DP4Hs0cbHUBvcTNkgTax?=
 =?us-ascii?Q?9O3hxDIZ+JhqLqYUBeRO5iZ67D37iEI/XwDDd00l6uGmLkhGFS5QURFdIUoD?=
 =?us-ascii?Q?Imk5sy1h+6u7if2v83fnUTet6RaXXC1tqaoLQovO23DedRUBRLuG18iHMnKl?=
 =?us-ascii?Q?wj4hc/jc6fRT3SDArWkjrMix7JlQExUAPmM3QNlngF9658AllZmSDAP52sI2?=
 =?us-ascii?Q?Qrot7JfHV8yVuYEYAUlcGVL/4Ot3YH43WFi58DUI6QVINL424jQib1X3rHKU?=
 =?us-ascii?Q?BVksRQQP3liexKbbRdkxDNxaFgaS9MkKcZR3KLHjV1iY7/evjx2UxvD4A+lv?=
 =?us-ascii?Q?cBE24n4VROYiTUf0IhG1w+YLqNFVgmmLbprCXN5+MJdWMdGtBrOYtP9ZTPZE?=
 =?us-ascii?Q?6hIN4G+UNasqVdmutSI0sAaMGiE89bkf73mk9DzW4sx9y/i8i4D4j9S0crOp?=
 =?us-ascii?Q?x31yPfEfVS9fujVsYFPABxrDGmHCwc6+15iI6oW14/ffRbcG/i7EYX29Jez5?=
 =?us-ascii?Q?+eLh/qLSd+DxTME7VobRcVl26Kw5w76mUpxU2fWhXdrYk392m6WfrElavwI8?=
 =?us-ascii?Q?VCERMKXzcbMRLbkQR5bsqp3kKv3stvyUt/3RumTGTy+lrVhqsxOH4aN23CsS?=
 =?us-ascii?Q?Ae2iPSjkkDV6V1wXGLUQJzHcAtJL2jtAYISPpfrPdWPYMt4jeiujN1q+lY0G?=
 =?us-ascii?Q?Vm3XkntJhyR22rVGBCbIFKQhCPUBzd/v5AhbPsiF?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 47f7d2d4-933f-4179-8e55-08db2d5623d9
X-MS-Exchange-CrossTenant-AuthSource: DM5PR11MB1418.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 25 Mar 2023 17:26:59.7610
 (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: OqhK7NtIbMeLMs3Y3uKe7D232EFcrwOh9Kri0evvvedGzxSYtH3Opk1l8TORtCPGvA4SEOPNmuNQ0kGfpV9sdg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR11MB6354
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-2.5 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,SPF_HELO_NONE,
        SPF_NONE autolearn=unavailable autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Greg,

Thanks for the review.
On Sat, Mar 25, 2023 at 05:20:09PM +0100, Greg Kroah-Hartman wrote:
> On Sat, Mar 25, 2023 at 11:47:06PM +0800, Ye Xiang wrote:
> > This patch implements the USB part of Intel USB-I2C/GPIO/SPI adapter
> > device named "La Jolla Cove Adapter" (LJCA).
> > 
> > The communication between the various LJCA module drivers and the
> > hardware will be muxed/demuxed by this driver. Three modules (
> > I2C, GPIO, and SPI) are supported currently.
> > 
> > Each sub-module of LJCA device is identified by type field within
> > the LJCA message header.
> > 
> > The minimum code in ASL that covers this board is
> 
> As this requires ACPI, why are you not saying so in your Kconfig entry?
I would add a `depends on ACPI` item on the Kconfig entry so that the
CONFI_ACPI macro can be removed from usb-ljca, because our use case with
LJCA currently needs the ACPI binding.
> What good would this driver be without ACPI enabled?
Before, I just tried to make it compatible with other platforms that
don't support ACPI but want to use LJCA device. But, We don't have
this use case until now.

Thanks
Ye Xiang