From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 6B55FC77B7C for ; Wed, 10 May 2023 07:47:44 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235856AbjEJHrn (ORCPT ); Wed, 10 May 2023 03:47:43 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:40516 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235609AbjEJHrm (ORCPT ); Wed, 10 May 2023 03:47:42 -0400 Received: from fgw22-7.mail.saunalahti.fi (fgw22-7.mail.saunalahti.fi [62.142.5.83]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 38C2AE5D for ; Wed, 10 May 2023 00:47:40 -0700 (PDT) Received: from localhost (88-113-26-95.elisa-laajakaista.fi [88.113.26.95]) by fgw22.mail.saunalahti.fi (Halon) with ESMTP id eeae663e-ef06-11ed-a9de-005056bdf889; Wed, 10 May 2023 10:47:38 +0300 (EEST) From: andy.shevchenko@gmail.com Date: Wed, 10 May 2023 10:47:37 +0300 To: Jiawen Wu Cc: 'Piotr Raczynski' , netdev@vger.kernel.org, jarkko.nikula@linux.intel.com, andriy.shevchenko@linux.intel.com, mika.westerberg@linux.intel.com, jsd@semihalf.com, Jose.Abreu@synopsys.com, andrew@lunn.ch, hkallweit1@gmail.com, linux@armlinux.org.uk, linux-i2c@vger.kernel.org, linux-gpio@vger.kernel.org, mengyuanlou@net-swift.com Subject: Re: [PATCH net-next v7 2/9] i2c: designware: Add driver support for Wangxun 10Gb NIC Message-ID: References: <20230509022734.148970-1-jiawenwu@trustnetic.com> <20230509022734.148970-3-jiawenwu@trustnetic.com> <009d01d9830a$c7c6dab0$57549010$@trustnetic.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <009d01d9830a$c7c6dab0$57549010$@trustnetic.com> Precedence: bulk List-ID: X-Mailing-List: linux-i2c@vger.kernel.org Wed, May 10, 2023 at 02:43:50PM +0800, Jiawen Wu kirjoitti: > On Tuesday, May 9, 2023 9:52 PM, Piotr Raczynski wrote: > > On Tue, May 09, 2023 at 10:27:27AM +0800, Jiawen Wu wrote: ... > > > /* > > > - * Initiate I2C message transfer when AMD NAVI GPU card is enabled, > > > + * Initiate I2C message transfer when polling mode is enabled, > > > * As it is polling based transfer mechanism, which does not support > > > * interrupt based functionalities of existing DesignWare driver. > > > */ > > > - if ((dev->flags & MODEL_MASK) == MODEL_AMD_NAVI_GPU) { > > > + switch (dev->flags & MODEL_MASK) { > > > + case MODEL_AMD_NAVI_GPU: > > > ret = amd_i2c_dw_xfer_quirk(adap, msgs, num); > > > goto done_nolock; > > > + case MODEL_WANGXUN_SP: > > > + ret = txgbe_i2c_dw_xfer_quirk(adap, msgs, num); > > > + goto done_nolock; > > > + default: > > > + break; > > > } > > Nit pick, when I first saw above code it looked a little weird, > > maybe it would be a little clearer with: > > > > if (i2c_dw_is_model_poll(dev)) { > > switch (dev->flags & MODEL_MASK) { > > case MODEL_AMD_NAVI_GPU: > > ret = amd_i2c_dw_xfer_quirk(adap, msgs, num); > > break; > > case MODEL_WANGXUN_SP: > > ret = txgbe_i2c_dw_xfer_quirk(adap, msgs, num); > > break; > > default: > > break; > > } > > goto done_nolock; > > } > > > > I do not insist though. > > Sure, it looks more obvious as polling mode quirk. I don't think we need a double checks. The i2c_dw_is_model_poll() will repeat the switch. Please, leave it as is in your current version. -- With Best Regards, Andy Shevchenko