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 35FBFC04A6A for ; Tue, 15 Aug 2023 19:25:26 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S239422AbjHOTYy (ORCPT ); Tue, 15 Aug 2023 15:24:54 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59452 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S239881AbjHOTYc (ORCPT ); Tue, 15 Aug 2023 15:24:32 -0400 Received: from mail-wm1-x332.google.com (mail-wm1-x332.google.com [IPv6:2a00:1450:4864:20::332]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E831A10D1 for ; Tue, 15 Aug 2023 12:24:06 -0700 (PDT) Received: by mail-wm1-x332.google.com with SMTP id 5b1f17b1804b1-3fe8a158fcbso38944485e9.2 for ; Tue, 15 Aug 2023 12:24:06 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1692127443; x=1692732243; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=eB2YdeaffFvKapTeu97tKDXHKtW8rfmRdezXiy0N6zs=; b=SAY2gmIGNEAiGgInuM1nDGvc/w31ckngzwPdU5Nqy5hC0Nh2Va91zmvCGoJguwvcGY 20kRSVSGJXKP6DVme+avP+mwcYdlzcSUDrzWwPvB5dQFqt0dp/w9V4ASPFPW4ogmj4vh x3pE5ECuJREptlRY/PQ5DolWgEM1gYN8/r5QPaIgZwBnj4bC4q+KfKIrLK8qNd9mcvZ7 ZHWQkA/8tCoe9HfDv6JqLPFLWGUJrKapW+0lccWL7ASniljFJpcv72pKauVvxCdq9bue afDATz3M4QhchabaBBQ5x0iaP7I28TZfZHV1IuvfYkJmJDCW0E5Xl2crAyvz1ETAjo0I B1eg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1692127443; x=1692732243; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=eB2YdeaffFvKapTeu97tKDXHKtW8rfmRdezXiy0N6zs=; b=Ss/IMSd3A7AeEze/9XKHIg4toWoI6OPUv5blNxtnTjmV2Ahym/sEPNRH7+Bv0e6U2h gnksPvnOc7tJIhrXfd8mBWtEP5/QeCq4+At5OSL38dwWP/Ob5bWntXJbUMg9xjDR0ynx 67f+uZW9en+BQlrvxiSrFtaAvyK7Ba4WbCBfPb/8kov68E/8e0YXLSVQOF5QE7wLbnZm jGshaJ5VRe/VPIZE15eq7/swFnA/1bSekLrrt5R2CLWsMVudNafkJs2H7XE0H6nReUyX dR+k5pv2jGPVSH7VNuyF6KFxJsnO8CPYyt2BWdnOa4MW1SV2K8/wkDniPVJXb0Un19TK /3sQ== X-Gm-Message-State: AOJu0Yz4bOd+eOFdQ95UFAp0K8TAFWfP7p3rSDFMDV7pB4ew14Mnhba/ HvmbQ7tlebiBhTTcmKK+pThgEw== X-Google-Smtp-Source: AGHT+IHnCElvhjrz7ZJrHjNtJ31GOUgLq29EzBpHyKL8Ir+AdcbNQiYstFhuJeVysB7QpeLgwYuM6Q== X-Received: by 2002:adf:ed41:0:b0:317:cdc4:762e with SMTP id u1-20020adfed41000000b00317cdc4762emr11427959wro.63.1692127443408; Tue, 15 Aug 2023 12:24:03 -0700 (PDT) Received: from [192.168.1.20] ([178.197.214.188]) by smtp.gmail.com with ESMTPSA id c13-20020a5d528d000000b003142c85fbcdsm18681305wrv.11.2023.08.15.12.24.00 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 15 Aug 2023 12:24:02 -0700 (PDT) Message-ID: Date: Tue, 15 Aug 2023 21:23:59 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.14.0 Subject: Re: [PATCH v3 2/2] Input: cs40l50 - Initial support for Cirrus Logic CS40L50 Content-Language: en-US To: James Ogletree Cc: Dmitry Torokhov , Fred Treven , Ben Bright , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Lee Jones , Jeff LaBundy , Joel Stanley , Arnd Bergmann , Jacky Bai , Jean Delvare , Eddie James , Markus Schneider-Pargmann , ChiYuan Huang , Randy Dunlap , Wolfram Sang , "patches@cirrus.com" , "linux-input@vger.kernel.org" , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" References: <20230809191032.820271-1-james.ogletree@cirrus.com> <20230809191032.820271-3-james.ogletree@cirrus.com> From: Krzysztof Kozlowski In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-input@vger.kernel.org On 15/08/2023 17:56, James Ogletree wrote: > > >> On Aug 10, 2023, at 1:17 AM, Krzysztof Kozlowski wrote: >> >> On 09/08/2023 21:10, James Ogletree wrote: >> >>> + >>> +static int cs40l50_cs_dsp_init(struct cs40l50_private *cs40l50) >>> +{ >>> + cs40l50->dsp.num = 1; >>> + cs40l50->dsp.type = WMFW_HALO; >>> + cs40l50->dsp.dev = cs40l50->dev; >>> + cs40l50->dsp.regmap = cs40l50->regmap; >>> + cs40l50->dsp.base = CS40L50_DSP1_CORE_BASE; >>> + cs40l50->dsp.base_sysinfo = CS40L50_DSP1_SYS_INFO_ID; >>> + cs40l50->dsp.mem = cs40l50_dsp_regions; >>> + cs40l50->dsp.num_mems = ARRAY_SIZE(cs40l50_dsp_regions); >>> + cs40l50->dsp.lock_regions = 0xFFFFFFFF; >>> + cs40l50->dsp.no_core_startstop = true; >>> + cs40l50->dsp.client_ops = &cs40l50_cs_dsp_client_ops; >>> + >>> + return cs_dsp_halo_init(&cs40l50->dsp); >>> +} >>> + >>> +int cs40l50_probe(struct cs40l50_private *cs40l50) >>> +{ >>> + int error, i, irq; >>> + u32 val; >>> + >>> + mutex_init(&cs40l50->lock); >>> + >>> + error = devm_regulator_bulk_get(cs40l50->dev, ARRAY_SIZE(cs40l50_supplies), >>> + cs40l50_supplies); >>> + if (error) >>> + return dev_err_probe(cs40l50->dev, error, "Failed to request supplies\n"); >>> + >>> + error = regulator_bulk_enable(ARRAY_SIZE(cs40l50_supplies), cs40l50_supplies); >>> + if (error) >>> + return dev_err_probe(cs40l50->dev, error, "Failed to enable supplies\n"); >>> + >>> + cs40l50->reset_gpio = devm_gpiod_get_optional(cs40l50->dev, "reset", GPIOD_OUT_HIGH); >> >> None of the lines above or below seem to be wrapped according to Linux >> coding style (80). > > This patch abides by the 100-column line length limit which checkpatch.pl enforces. Checkpatch does not enforce 100, because it accepts also 80. It complains if something is longer than 100, but coding style is still the same. Format everything according to coding style and use 100 only if it increases readability. > However, I can see how some of the lines might be less jarring to the eyes if wrapped. > That will be addressed in V4. > Best regards, Krzysztof