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 A6E9FEB64DC for ; Mon, 17 Jul 2023 07:56:42 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230150AbjGQH4l (ORCPT ); Mon, 17 Jul 2023 03:56:41 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55882 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229696AbjGQH4j (ORCPT ); Mon, 17 Jul 2023 03:56:39 -0400 Received: from mail-lf1-x132.google.com (mail-lf1-x132.google.com [IPv6:2a00:1450:4864:20::132]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 9EC5893; Mon, 17 Jul 2023 00:56:37 -0700 (PDT) Received: by mail-lf1-x132.google.com with SMTP id 2adb3069b0e04-4fdb3f66fd6so2361589e87.3; Mon, 17 Jul 2023 00:56:37 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1689580596; x=1692172596; h=in-reply-to:content-disposition:mime-version:references :mail-followup-to:message-id:subject:cc:to:from:date:from:to:cc :subject:date:message-id:reply-to; bh=uzcBHck4XeBHsv947H53e4eFghSZzaJsuiWxGP5iqxw=; b=aK1icLPXTDR0xU45DBZUT5kFzZp+x7tjRpCAIZi/4/7gItfaY6xk8awYtW6SuS3AgK eOS8lt8Fl2y0lBeQOTLb2fWcCuqajJ4KEhWIQ0FO1dntZF1RN7GG8cOZO4U4u4zTTxby bN9Jda/tET2f4M9RnnbkTRhimJCssAMDJCbAFm1HbrNenuU5AsCIoEuZ4zyYr2fX0mST CdiABZdN94edkvKOSvNNVaDToYSlyeNLpO2NoIZQXzUyEIp753USAPsSHx9K7J9QDYBf S+Ml/Xq4wZ0Av9V6Mz36BwQSje86L6CMEZtAhl85YDC7bTYRBsyUVcQEfLvMt8qJQJ1N QgtA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1689580596; x=1692172596; h=in-reply-to:content-disposition:mime-version:references :mail-followup-to:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=uzcBHck4XeBHsv947H53e4eFghSZzaJsuiWxGP5iqxw=; b=NJ/CvB4XS3SSkfKo7W2HIrYqeF6AShxAT2UyGYWAJh1LQuiVr+yVtzt1xlQ2R9ATYE zit/FQIk3gm0bau9LOjH1dDAC/E+E7vGJ841lyQOAJgHLss7DHnMW2UjkOgw8oK+3Qww AbPp4HK1v8xdmk8mQpvXyLPKXrCL5+/13g2B6hLPssyLP/Q3eQ8UD9FDflVE4vG2DVEh GRjbUc3cUWOX+6OCS0y798XbfaxUUko+lUPpSmkLHvWpo2WSDHGjLwwuJwiwHCAFcmmI 2pboeW4bMCl2pryhPVYVpGxygYc7eVffJlxSOdxzVBKKCvnVkiG4T6pLjIOjg30VUljZ jU8w== X-Gm-Message-State: ABy/qLZDz7QbCJBpH1HIyhuRgt0noRk3eeRW/jhfyjCkR9/6TRnZp5S0 Ca4C4Vr+bsysTwg/Zc9uEAcwowvh0/w= X-Google-Smtp-Source: APBJJlF/d2LcLXm4aCAC3voDYWq2PYKTB8s9SeU39Eo8j6FYpcSeV/LHlzvm/BdU54Xs8ntQPKpVcg== X-Received: by 2002:a2e:9c93:0:b0:2b6:e618:b593 with SMTP id x19-20020a2e9c93000000b002b6e618b593mr7671821lji.31.1689580595285; Mon, 17 Jul 2023 00:56:35 -0700 (PDT) Received: from gmail.com ([81.168.73.77]) by smtp.gmail.com with ESMTPSA id u1-20020a05600c00c100b003fbb1a9586esm7384812wmm.15.2023.07.17.00.56.34 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 17 Jul 2023 00:56:34 -0700 (PDT) Date: Mon, 17 Jul 2023 08:56:32 +0100 From: Martin Habets To: hanyu001@208suo.com Cc: davem@davemloft.net, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com, netdev@vger.kernel.org, linux-net-drivers@amd.com, linux-kernel@vger.kernel.org Subject: Re: sfc: falcon: Prefer unsigned int to bare use of unsigned Message-ID: Mail-Followup-To: hanyu001@208suo.com, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com, netdev@vger.kernel.org, linux-net-drivers@amd.com, linux-kernel@vger.kernel.org References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The subject should start with "[PATCH net-next]". Please use ./scripts/get_maintainer.pl to find the correct people to send this to. On Mon, Jul 17, 2023 at 11:43:00AM +0800, hanyu001@208suo.com wrote: > Fix checkpatch warnings: > > ./drivers/net/ethernet/sfc/falcon/net_driver.h:1167: WARNING: Prefer > 'unsigned int' to bare use of 'unsigned' > ./drivers/net/ethernet/sfc/falcon/net_driver.h:1188: WARNING: Prefer > 'unsigned int' to bare use of 'unsigned' > ./drivers/net/ethernet/sfc/falcon/net_driver.h:1188: WARNING: Prefer > 'unsigned int' to bare use of 'unsigned' > ./drivers/net/ethernet/sfc/falcon/net_driver.h:1202: WARNING: Prefer > 'unsigned int' to bare use of 'unsigned' > > Signed-off-by: maqimei <2433033762@qq.com> This does not match with the From line in your email. > --- > drivers/net/ethernet/sfc/falcon/net_driver.h | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/net/ethernet/sfc/falcon/net_driver.h > b/drivers/net/ethernet/sfc/falcon/net_driver.h > index a2c7139..d2d8f9b 100644 > --- a/drivers/net/ethernet/sfc/falcon/net_driver.h > +++ b/drivers/net/ethernet/sfc/falcon/net_driver.h > @@ -1164,7 +1164,7 @@ struct ef4_nic_type { > *************************************************************************/ > > static inline struct ef4_channel * > -ef4_get_channel(struct ef4_nic *efx, unsigned index) > +ef4_get_channel(struct ef4_nic *efx, unsigned int index) You've added an extra space here after int. Don't do that. Martin > { > EF4_BUG_ON_PARANOID(index >= efx->n_channels); > return efx->channel[index]; > @@ -1185,7 +1185,7 @@ struct ef4_nic_type { > (_efx)->channel[_channel->channel - 1] : NULL) > > static inline struct ef4_tx_queue * > -ef4_get_tx_queue(struct ef4_nic *efx, unsigned index, unsigned type) > +ef4_get_tx_queue(struct ef4_nic *efx, unsigned int index, unsigned int > type) > { > EF4_BUG_ON_PARANOID(index >= efx->n_tx_channels || > type >= EF4_TXQ_TYPES); > @@ -1199,7 +1199,7 @@ static inline bool ef4_channel_has_tx_queues(struct > ef4_channel *channel) > } > > static inline struct ef4_tx_queue * > -ef4_channel_get_tx_queue(struct ef4_channel *channel, unsigned type) > +ef4_channel_get_tx_queue(struct ef4_channel *channel, unsigned int type) > { > EF4_BUG_ON_PARANOID(!ef4_channel_has_tx_queues(channel) || > type >= EF4_TXQ_TYPES);