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 mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id D1F93C433EF for ; Mon, 1 Nov 2021 14:32:29 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id A3CED60EE9 for ; Mon, 1 Nov 2021 14:32:29 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232111AbhKAOfB (ORCPT ); Mon, 1 Nov 2021 10:35:01 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57358 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231811AbhKAOfA (ORCPT ); Mon, 1 Nov 2021 10:35:00 -0400 Received: from mail-wm1-x32f.google.com (mail-wm1-x32f.google.com [IPv6:2a00:1450:4864:20::32f]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 25B45C061714 for ; Mon, 1 Nov 2021 07:32:19 -0700 (PDT) Received: by mail-wm1-x32f.google.com with SMTP id c71-20020a1c9a4a000000b0032cdcc8cbafso76246wme.3 for ; Mon, 01 Nov 2021 07:32:19 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=Nk+zvXAt/yARZGxNV8fi4WGClbaS5+zS5+rohIvR53Y=; b=GUcjWsJLUg2Lf2WmjGU/RQeJcsua7eeLBxuChpYxP84Ylg2tZelUYkJxQv6ZXlVeO7 8mWWgVkxsk4XcBhKxmMdwwd7kC26Qzw/3/WLIqBwgFt6nMyOezDBclDAi/HkCCj9iK6Y uZMweyotbPljtq08Efsdx4usDZu+iDk5BD/tfgZwAvNREUEcCu7TzLlMpX+ud5K0o+UQ LxTioXUX0OAobNBRFNMQkQ2bEpgYthF+RKOdSGIw9V+6WoqJmvZjRo7MnNw1M0/Yx5C4 Ov65aJlQLKr6DrHSmrgecYBxlK6gGri5rS2rn8kKEB036vmgIC0dLcy3vtmz46+rI4CU Gbwg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=Nk+zvXAt/yARZGxNV8fi4WGClbaS5+zS5+rohIvR53Y=; b=qBHpJp/y1kvgovEJJj61X7TO1KmmcKD8QAIOeSMQbsU3jlxoRxsRVk2e854rcYGO42 UjqD6YMoM3tNB7WAysXqxr36nrdf0njLDkOC2yDAEav+/imWY3YN40/K/YJ0rD4YRdYk 7aZuAVF/iAiXvkjF24fE1xn6rn3cfb1oXGOkmsXMCP1y81ocGZGqNKkvW+HclgkHyBDp nYtwVPkNdCEUkdfykJ9Z6/5RzFeWJR86JoahFPKwcbxPzF/T8nEekQaOo5SMKqaB0Lcr Chowx29Ybe70js8AzyVNdqmcbTOaodTb9ppMhvTQsdMCOtehcUTXkt8/odeSjztV0lLH wI6w== X-Gm-Message-State: AOAM530AuRjIzyTZlsqvB3gxJ9LO5F2jwXK3Tx1Th78dbG+2sAGJnN86 OUJMhc8MPow1i5yq6MteaFfJ0QhxHaoz2w== X-Google-Smtp-Source: ABdhPJzh/TukZxPlocC0fnpJzW8NxmJVcRR7+AL1erPqINggPG3XDXjtuK7yb/AS3p2jYC9Js1HIew== X-Received: by 2002:a1c:f405:: with SMTP id z5mr31418913wma.72.1635777137721; Mon, 01 Nov 2021 07:32:17 -0700 (PDT) Received: from localhost.localdomain ([2a01:4b00:f411:e700:e085:8cb7:7bf6:5d62]) by smtp.gmail.com with ESMTPSA id l8sm17157721wry.43.2021.11.01.07.32.17 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 01 Nov 2021 07:32:17 -0700 (PDT) From: Karolina Drobnik To: outreachy-kernel@googlegroups.com Cc: gregkh@linuxfoundation.org, forest@alittletooquiet.net, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, Karolina Drobnik Subject: [PATCH v2 0/8] subject: staging: vt6655: Fix line wrapping in `RFvWriteWakeProgSyn` Date: Mon, 1 Nov 2021 14:31:58 +0000 Message-Id: X-Mailer: git-send-email 2.30.2 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This patch set is a series of small refactorings of the function `RFvWriteWakeProgSyn`, now renamed to `rf_write_wake_prog_syn`. The work here allowed to shorten lines marked by checkpatch.pl as being too long by: * introducing two new temporary variables (`idx` and `init_table`) * rewriting a conditional to switch between two modes of AL7320 initialization. In addition to this, the patch set renames the function to align it with the kernel coding style, updates the function description and removes two unnecessary conditionals checking if the value of `init_count` is lower than a predefined value. Changes in v2: * drop "staging: vt6655: Introduce `data` temporary variable" patch, suggested by Julia Lawall * amend "staging: vt6655: Rewrite conditional in AL7320 initialization" patch to use indexing and `init_table` temporary variable * add two new patches to remove unnecessary checks for init count, suggested by Mike Rapoport : - "staging: vt6655: Delete bogus check for `init_count` in AL2230" - "staging: vt6655: Delete bogus check for `init_count` in AL7230" Karolina Drobnik (8): staging: vt6655: Introduce `idx` temporary variable staging: vt6655: Use incrementation in `idx` staging: vt6655: Remove unused `i` increments staging: vt6655: Rewrite conditional in AL7320 initialization staging: vt6655: Rename `RFvWriteWakeProgSyn` function staging: vt6655: Update comment for `rf_write_wake_prog_syn` staging: vt6655: Delete bogus check for `init_count` in AL2230 staging: vt6655: Delete bogus check for `init_count` in AL7230 drivers/staging/vt6655/channel.c | 2 +- drivers/staging/vt6655/rf.c | 44 +++++++++++++------------------- drivers/staging/vt6655/rf.h | 2 +- 3 files changed, 20 insertions(+), 28 deletions(-) -- 2.30.2