Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752521AbdLEWI4 (ORCPT ); Tue, 5 Dec 2017 17:08:56 -0500 Received: from mail-lf0-f66.google.com ([209.85.215.66]:36820 "EHLO mail-lf0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751633AbdLEWIw (ORCPT ); Tue, 5 Dec 2017 17:08:52 -0500 X-Google-Smtp-Source: AGs4zMY4sOj+KnPVXmLUjhskilVp/x/LFcnxEEthPvMHOLLbvycK7Kf/6q39w1jrY3qtHuRE/JECPw== From: =?UTF-8?q?Simon=20Sandstr=C3=B6m?= To: gregkh@linuxfoundation.org Cc: linux@Wolf-Entwicklungen.de, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: [PATCH v2 00/11] Fix indentation and CamelCase issue in staging/pi433 Date: Tue, 5 Dec 2017 23:08:38 +0100 Message-Id: <20171205220849.5486-1-simon@nikanor.nu> X-Mailer: git-send-email 2.11.0 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1726 Lines: 45 These patches fixes a bunch of code style issues in staging/pi433. The first patch fixes indentation in rf69_enum.h and the rest of the patches fixes CamelCase issues and other minor issues in all of staging/pi433. Changes in v2: - Instead of fixing CamelCase issue in enum data_mode, remove it completely. - Split multiple functions of type "rf69_set_x_enabled(dev, enum option_on_off)" to two functions: "rf69_enable_x(dev)" and "rf69_disable_x(dev)". - Move enum option_on_off to pi433_if.h as it's now only used for ioctl. - Fix issue where GPIOs and device not being deallocated in pi433_probe() if SET_CHECKED fails. - Simon --- Simon Sandström (11): staging: pi433: Fix indentation in rf69_enum.h staging: pi433: Capitalize constant definitions staging: pi433: Rename variable in struct pi433_rx_cfg staging: pi433: Rename enum optionOnOff in rf69_enum.h staging: pi433: Rename enum modShaping in rf69_enum.h staging: pi433: Split rf69_set_crc_enabled into two functions staging: pi433: Split rf69_set_sync_enabled into two functions staging: pi433: Remove enum data_mode staging: pi433: Combine all rf69_set_amplifier_x() staging: pi433: Move enum option_on_off to pi433_if.h staging: pi433: Remove SET_CHECKED usage from pi433_probe drivers/staging/pi433/pi433_if.c | 135 ++++++++++++++------- drivers/staging/pi433/pi433_if.h | 25 ++-- drivers/staging/pi433/rf69.c | 113 +++++------------- drivers/staging/pi433/rf69.h | 15 +-- drivers/staging/pi433/rf69_enum.h | 210 ++++++++++++++++----------------- drivers/staging/pi433/rf69_registers.h | 44 +++---- 6 files changed, 265 insertions(+), 277 deletions(-) -- 2.11.0