Received: by 2002:a05:6358:3188:b0:123:57c1:9b43 with SMTP id q8csp541595rwd; Wed, 31 May 2023 01:50:28 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ4OY7nAGcmAmnGn31zSyIw7Wk51VI6iSc9/kosAgC2yfexa8pUVC5b5jj0aBglKyXRJvAEj X-Received: by 2002:a05:6808:bc6:b0:38b:37dc:88d3 with SMTP id o6-20020a0568080bc600b0038b37dc88d3mr4266885oik.45.1685523028254; Wed, 31 May 2023 01:50:28 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1685523028; cv=none; d=google.com; s=arc-20160816; b=OM+URO8MusUn92RQp7kxq4g7u2IXiPuisbfy5oIHrtB5tEzXNuj95avYDgLDQqltmL 6tjNEPRRsHTKtmbh+MSkOvOai6zRy/NAXawoSVjSPHldn/oz6UMO6/BFs/hDkbbxzNgd 1vQ7Q6NB1qNVwDF2jZ2jjVFIjumyTeNLuXu3YtpugGngF9ECeFmHfbTWXE4EqRyxFN+w mGb/WbMqHWlkDERbJp9Ti4iPo+RGgxaECheL3CiRam943yGSFc6P2/Edgn0eiynd4uM0 YlF9NNOgqV614E+nFt82bgQoBo0/vfBh3H9YPBqxeXPd5qHbRj0jNUAK9C2eQAvqJMxm 3RKw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:user-agent:in-reply-to:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :dkim-signature; bh=211+wVlWeP/3HAR+B+VJMtiN6UM45J+l2Sz2XbJV7oQ=; b=K5JillmTUG6h4ZxNfSItujaWjRd44nI3BNPpMizxXdisfQMLygYMB2SjALBU8KVBZu PRYkUU3DQcb/yQBbJ4HQkk3dsMgCEDmDjtizt+b0M/KUWCp1siWRI+o79IcMn47VN8gY zWmOkeAqCbxNFZj59IzOl8Y8kkzmljKS7XRFpcgSUm/1enoTwEEwaEVFf7B+xibi7cvC GaQfaPZyDxBOGx7hGJQdfA+mBRIYH8GxBZs8/EfMM4b2plj5Hhh5cy1JTar1mq++Q5te T83NvdZX3q6fMMwmQe07HKr7uF8zDfYthy/L/3/8B9nVMNA/DIteCuIVbhK/GrNK8b3x 3fRw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@cirrus.com header.s=PODMain02222019 header.b="pX/EZtav"; 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=REJECT sp=REJECT dis=NONE) header.from=cirrus.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id f23-20020a633817000000b0053f0cdab81esi607708pga.357.2023.05.31.01.50.14; Wed, 31 May 2023 01:50:28 -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=@cirrus.com header.s=PODMain02222019 header.b="pX/EZtav"; 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=REJECT sp=REJECT dis=NONE) header.from=cirrus.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235035AbjEaIex (ORCPT + 99 others); Wed, 31 May 2023 04:34:53 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53786 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232570AbjEaIeh (ORCPT ); Wed, 31 May 2023 04:34:37 -0400 Received: from mx0b-001ae601.pphosted.com (mx0b-001ae601.pphosted.com [67.231.152.168]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 21BD0BE; Wed, 31 May 2023 01:34:35 -0700 (PDT) Received: from pps.filterd (m0077474.ppops.net [127.0.0.1]) by mx0b-001ae601.pphosted.com (8.17.1.19/8.17.1.19) with ESMTP id 34V5d1D5002650; Wed, 31 May 2023 03:34:04 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cirrus.com; h=date : from : to : cc : subject : message-id : references : mime-version : content-type : in-reply-to; s=PODMain02222019; bh=211+wVlWeP/3HAR+B+VJMtiN6UM45J+l2Sz2XbJV7oQ=; b=pX/EZtavXMLquh00TLBNnO/V4Ci3vEqHDeL4eB0RPzYD5FUcgLYo/sloYfULIcAQ1tBA F3PluRqU8ZeQ85A7OuA97pQ0ltStLpRuPgPqrk53V8mDcpv2eLIn12tinnRxNdVzX+iJ W4kC/59vnSL5xAm0mrWNUYa8MJLne+T061kHKP5tacYDyjqj0NAmr+UutZYM1u8RRZ6m 4wvwVEqO/y3hSWojX0KZlMliFDotTIgDFh6XsvYj0UXmLzZ2qF531amR597Hoouf2p3l tlVeOQRd5oJhLef9X/QCvgJbaL9WrlvVlsXpU1M2XEUAlxh60puusvaPSg1FJcVDDovv UA== Received: from ediex02.ad.cirrus.com ([84.19.233.68]) by mx0b-001ae601.pphosted.com (PPS) with ESMTPS id 3que9mvft2-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 31 May 2023 03:34:04 -0500 Received: from ediex02.ad.cirrus.com (198.61.84.81) by ediex02.ad.cirrus.com (198.61.84.81) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1118.26; Wed, 31 May 2023 09:34:02 +0100 Received: from ediswmail.ad.cirrus.com (198.61.86.93) by anon-ediex02.ad.cirrus.com (198.61.84.81) with Microsoft SMTP Server id 15.2.1118.26 via Frontend Transport; Wed, 31 May 2023 09:34:02 +0100 Received: from ediswmail.ad.cirrus.com (ediswmail.ad.cirrus.com [198.61.86.93]) by ediswmail.ad.cirrus.com (Postfix) with ESMTP id AE23A11C6; Wed, 31 May 2023 08:34:02 +0000 (UTC) Date: Wed, 31 May 2023 08:34:02 +0000 From: Charles Keepax To: Mark Brown CC: , , , , , , , , , , , , , , , Subject: Re: [PATCH v2 5/6] spi: cs42l43: Add SPI controller support Message-ID: <20230531083402.GF68926@ediswmail.ad.cirrus.com> References: <20230530122112.1314458-1-ckeepax@opensource.cirrus.com> <20230530122112.1314458-6-ckeepax@opensource.cirrus.com> <171e2054-e41c-46a5-b478-f699909c5bd7@sirena.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <171e2054-e41c-46a5-b478-f699909c5bd7@sirena.org.uk> User-Agent: Mutt/1.5.21 (2010-09-15) X-Proofpoint-GUID: 5Yhc_ulsONjFQYDneXH2QsBaUlWrHPop X-Proofpoint-ORIG-GUID: 5Yhc_ulsONjFQYDneXH2QsBaUlWrHPop X-Proofpoint-Spam-Reason: safe X-Spam-Status: No, score=-2.7 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_EF,RCVD_IN_DNSWL_LOW,SPF_HELO_NONE,SPF_PASS, T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED 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 On Tue, May 30, 2023 at 10:30:46PM +0100, Mark Brown wrote: > On Tue, May 30, 2023 at 01:21:11PM +0100, Charles Keepax wrote: > > A couple of small things: > > > +static unsigned int cs42l43_clock_divs[16] = { > > + 2, 2, 4, 6, 8, 10, 12, 14, 16, 18, 20, 22, 24, 26, 28, 30 > > +}; > > Do we need to specify the size of the array? I just had to count the > number of initialisers :( Should probably also be const. > > > + for (; buf < block - (sizeof(u32) - 1); buf += sizeof(u32)) > > + regmap_write(regmap, CS42L43_TX_DATA, *(const u32 *)buf); > > We're passing a byte stream through a u32 here - are you sure this is > endian safe? Ah shoot, yeah Andy made some comments on this that seem to have got lost in my mass of fixups. I will fixup for a v3. Thanks, Charles