Received: by 2002:a05:6358:9144:b0:117:f937:c515 with SMTP id r4csp9353358rwr; Thu, 11 May 2023 13:43:49 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ4GrhOhj6SyHOsnLIO6WZLwDfMU+2cn4vasUIAEqLTxqnTjvz+Tl4VJs0t+y8mZSZLRC0VH X-Received: by 2002:a17:90a:318c:b0:247:53b4:ec52 with SMTP id j12-20020a17090a318c00b0024753b4ec52mr22472424pjb.8.1683837829114; Thu, 11 May 2023 13:43:49 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1683837829; cv=none; d=google.com; s=arc-20160816; b=sYi5xMZ9m69aKtCf+n7HCAGU1j7o3jDsnyoyyJKQEYc9ofuUz1LcdHWIHWN0s7h1xv WU4tpdJtVTKqSf4et/b1A4t8N7Ul39XArQiZ53MQwwaccJTFzniQ9SVv3MaoLhEplA4U yDVraiNOYZF1r4BXUOxD2vfeAo2U/uLwvQOiB1POHGtXKuygkpMHH+LlfDCpcHjvJUW+ maWQI/kTOxV+TCXaw8EQq6E8EqXljbTRq1r8AZdgHA6uUh1cz8Kse/VGMqsRCb8pOjCg jY3bqRnCOAg2tETORPkAs8C/XquXtDdAz6TQy+fxCBgTYHpQ35axYnLmBzXiNCT6myim e3JA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:cc:to:subject :message-id:date:from:in-reply-to:references:mime-version :dkim-signature; bh=cIq+csdZEem8CM7fcgH+5/GZeWFKK87200SdyT17Gx0=; b=QQ2smlNnoPcZLXDZwCoGeajfleD1RjX6VDoTwZGP5sunpUodw7Kv/XDqr7ySVuDNJg q1CSdi5ImQEPP0WokSzLWg3NNUxY1Uyp9X71L+1UX4t0GKb2Ito03wOStrsSdH58A6+I YTuxzQEugRY1KRrNw8xzg1QZAXkcEITUCGQgU2td6U2MvALbi37I1/0a+xG7Q/dMIoEI t6yvZ9UKZr68MBM3rH7AL1CM/zHNmO+mkkaaOWrLN672pDsJO/8YAcnXgQqGhtopcEHg PcUMsQZ9sDxC5SFDlPxGeb2VKEXagvxCGUVacAVSjIfpQczeNs1f3i3umwE6aScxgC48 M39w== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20221208 header.b="a1rVpvl/"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id v71-20020a63894a000000b00521274d891esi7266712pgd.183.2023.05.11.13.43.34; Thu, 11 May 2023 13:43:49 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20221208 header.b="a1rVpvl/"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S239392AbjEKU3h (ORCPT + 99 others); Thu, 11 May 2023 16:29:37 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:38196 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S238918AbjEKU3g (ORCPT ); Thu, 11 May 2023 16:29:36 -0400 Received: from mail-pl1-x62f.google.com (mail-pl1-x62f.google.com [IPv6:2607:f8b0:4864:20::62f]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id CFA3B49DB; Thu, 11 May 2023 13:29:35 -0700 (PDT) Received: by mail-pl1-x62f.google.com with SMTP id d9443c01a7336-1a69e101070so17786595ad.1; Thu, 11 May 2023 13:29:35 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1683836975; x=1686428975; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=cIq+csdZEem8CM7fcgH+5/GZeWFKK87200SdyT17Gx0=; b=a1rVpvl/Pn1WWSGMfSMkv2Hxawdf9hhhC1VBT4W0Wmfssa+RjlcSH40UzxVVWmsrT1 lBn6e6k4dacM/AxVcp4zm19aar1NjFJdGo7i4u6YmlUswm9opOVShy8A83p+VkxOuI82 vqhbTgm83HDxusvCAP4BztOBQfU7+MX0SeoqshAFXkiIiFlqPl3XjFe7QFNhqR6ihnr8 Z3NGiRtvIajZHyikPyWgB95PTJLWoGhiwPDCL5K1ncbP8bfu1/ry29cvK6suibd5MvN3 RqeK/TAHJkzTdBZX5p3YBhktKyX3NA7ArHzAg2EQBKMZIPyniISKryUjeC42k11Ryz3I 4yFw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1683836975; x=1686428975; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=cIq+csdZEem8CM7fcgH+5/GZeWFKK87200SdyT17Gx0=; b=Q+uLN7kel4MNJBd6PZYQAayE3JaavQIHXMIu7C6CBk5EbkiUe37WNu3tw7lE4YVgto Cl8qkvgpf+x/6xup1ifIx4qpP+B02+va8Q1IbXE3VD5Gt1FJP84sbYN9bZAdNijJuk7Q XXp6pGCFZL46wuXT3VUPrekKWeiNV7q+bTZq6utF/Z47gnY0ulPWvTvVFKiH5e5UCqCD Y1l8W/XfiPx0ZgJgYhD7UsLtsNLbz6BraiL4GXaBe+siGMjY9BEoot4O4Sck9V0E8gOM hMdMDLVkWspRowTo3miJLYNMpX8s30BXvb/Xr4FpBeMY2+aM+k1gsIggpYjL1rLrqdeb DZdA== X-Gm-Message-State: AC+VfDxilFUnwTe2f35oIw2ZeL3BZQbqEFMMit+VNScUWQ24SnXKRNEn Pvm7uZV5rpqHHRhGWBzMgZRBf8LonbJtG8vdRh0= X-Received: by 2002:a17:902:d4c4:b0:1ac:40f7:8b5a with SMTP id o4-20020a170902d4c400b001ac40f78b5amr26750525plg.3.1683836975198; Thu, 11 May 2023 13:29:35 -0700 (PDT) MIME-Version: 1.0 References: <20230511135632.78344-1-bstruempfel@ultratronik.de> In-Reply-To: <20230511135632.78344-1-bstruempfel@ultratronik.de> From: Fabio Estevam Date: Thu, 11 May 2023 17:29:23 -0300 Message-ID: Subject: Re: [PATCH] spi: Add option to keep the MOSI line low, when it is idle. To: Boerge Struempfel Cc: bstruempfel@ultratronik.de, Mark Brown , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , NXP Linux Team , linux-spi@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Schrempf Frieder Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FROM, RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Boerge, On Thu, May 11, 2023 at 10:58=E2=80=AFAM Boerge Struempfel wrote: > > By default, the imx spi controller uses a high mosi line, whenever it is > idle. This may not be desired in all use cases. For example neopixel > leds can get confused and flicker due to misinterpreting the idle state. > Therefore, we introduce a new spi-mode bit, with which the idle behaviour > can be overwritten on a per device basis. ... > + if (of_property_read_bool(nc, "spi-mosi-idle-low")) > + spi->mode |=3D SPI_MOSI_IDLE_LOW; Yes, this is useful. As this is a new property, please send a patch that adds it to: Documentation/devicetree/bindings/spi/spi-peripheral-props.yaml Thanks