Received: by 2002:a17:90a:1609:0:0:0:0 with SMTP id n9csp2146366pja; Thu, 26 Mar 2020 10:24:02 -0700 (PDT) X-Google-Smtp-Source: ADFU+vvz33QIxVNuH7lvZUc/7tU9UGEKET3U1NMLVatw9OwiZTDIcoPbwGslmK9MgC/v/P2pnkku X-Received: by 2002:a4a:6549:: with SMTP id z9mr6025219oog.37.1585243442308; Thu, 26 Mar 2020 10:24:02 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1585243442; cv=none; d=google.com; s=arc-20160816; b=xBOdMl6iGHiFTXPAk3TytpwzwYgJkJeNTwV/zTulaXyvCqJ+LvsMG7cSonri7g06CV qoT0TNQFfFtuEPmoW3BQ+9UtDXCXGKKDgzqM6WNphw7bTTWvsLEFixCllX4dtbX0DnGf TJXspji3Lyrt1JAfKlHd+cIs21Dslea9guVXy9F05+hNpNMkQYzadn2C/C1MokDfkNNi f+3yOsbNK9jrcNk2f07/61MtL+ZTDQnXrxIPFBNOCEfUxNnTXgiKeILfj3zq3upZ9NJF 5KyMRn18rhnCh/aUkNH2vTsA3sE7a8QKe42jAa5xBaT2ugoYfeHeL8F6SKUTjySRqRfi fgEQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:user-agent :in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:dkim-signature; bh=i79giKalKA4K1I349brFOAJE9ERJfJNvP/fxVy14tMI=; b=m296POeTwctAh9g6cVlqFHJ+jleYdR5OiIFEu5GlU9hdui96g/5dSevDtpGoX1Pr9f jk9COE/ucVR+V0ZlOIqn7dcf9a+K4LE+cAihBrcq9m0xZI+0I0jKd8zze6BZjIet/Ou2 DCveFxEA8iBDK/b41gBSIzokWJgMZ2UQgtoZ6L5cORClaZ/OnziwAivnQaPMwl1SPF93 9oA3R/gTJT1xx8Vx18sKlEswVVFLbzRCh+u3t4s8eG5wAX247rQEEJJyCmLPqp7UJN2v RL+C82QITzWcDmACv36RGWkiKr78O8qmVAUvb+gnIoQUMtW/cUI03iQnj2racQQQbNW6 CzYQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmx.net header.s=badeba3b8450 header.b="ionFLP/z"; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id h192si1271153oib.166.2020.03.26.10.23.48; Thu, 26 Mar 2020 10:24:02 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@gmx.net header.s=badeba3b8450 header.b="ionFLP/z"; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727906AbgCZRWk (ORCPT + 99 others); Thu, 26 Mar 2020 13:22:40 -0400 Received: from mout.gmx.net ([212.227.15.15]:50289 "EHLO mout.gmx.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727724AbgCZRWk (ORCPT ); Thu, 26 Mar 2020 13:22:40 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gmx.net; s=badeba3b8450; t=1585243347; bh=MWQpqiFq+1hK9zp12FeAP0ENS8Btp4Us1plOsjWVOA4=; h=X-UI-Sender-Class:Date:From:To:Cc:Subject:References:In-Reply-To; b=ionFLP/z5hIMPcvYYdpYjsCoEq0rpFaf+I84HduZC/LtfLDDeCNk6pc97WHph6ZVJ j8xUbf6gRkv6WPlnQHLEgCkkkDYCfGOGlyzFa35oFfRPOQshcMjcrCeFvaDfE8wNo8 GcN1PbgGofKXwYBo8iDv7T7OLRDpUPqf0fJRfF18= X-UI-Sender-Class: 01bb95c1-4bf8-414a-932a-4f6e2808ef9c Received: from ubuntu ([83.52.229.196]) by mail.gmx.com (mrgmx004 [212.227.17.184]) with ESMTPSA (Nemesis) id 1MRTRH-1iudja17wr-00NUbM; Thu, 26 Mar 2020 18:22:27 +0100 Date: Thu, 26 Mar 2020 18:22:25 +0100 From: Oscar Carter To: Dan Carpenter , Quentin Deslandes Cc: Oscar Carter , Forest Bond , Greg Kroah-Hartman , Malcolm Priestley , Colin Ian King , Gabriela Bittencourt , devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] staging: vt6656: Use ARRAY_SIZE instead of hardcoded size Message-ID: <20200326172224.GC3629@ubuntu> References: <20200318174015.7515-1-oscar.carter@gmx.com> <20200324095456.GA7693@jiffies> <20200324131830.GD4672@kadam> <20200325091924.GB15158@jiffies> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200325091924.GB15158@jiffies> User-Agent: Mutt/1.10.1 (2018-07-13) X-Provags-ID: V03:K1:HLqxRcmQo28JRdSPp73j2tg1Y8Oyog28LeDWrq1bY2PGNpeNplf w+FiAc4Vgo+vp+aHfnauFLt6+tkoivrd+7WWSQ1oYbakn8QuqIOg41G0Q7PJ53ZjBsPNA20 znqRWWPFpCp2V63I0K0ia5x4yk30Kt+pOoXGZpeP5vmUzjiv8IX5+IXZirn2nz4oyac2iXw 4+mOsuFVoCAYViHuuP3VQ== X-Spam-Flag: NO X-UI-Out-Filterresults: notjunk:1;V03:K0:0rOYc05hHsg=:YWfhe8q5SwPvx1E1cvHmPp juNhBP2AXxFVwySkTWRdQcXCV751daKlHy2ZEfcJrB+3qMG8eXEF9mG9NwjWIMIet0NahjcLI G9Aizlt49U6GwvL02tGeVgs4Ls5OR8fihViij64ldaAhF5XUAM5T7tQbl5GgH5l3TxQ7jfqoU oiTVab0A1OzWezNg4XZrcMf7+4yeTN7LsHCwyehHS3ww4zekphdRUcvCXwAisEvfa1i6OQyfz 80HC97nHG4Xh4ooh7lNvMvRYKDeoPuRbiby6vsMGGakhXiMnBcnNPd6d2ecrSSWADbMuKTUcC RRz7tQw+nZwDyyDvrt0TO/6ONDEiVmhF3v46Ehd7tTMJM/1nUBfcnOLUWyG1hq9vRC68Ne4Mo OM5Dck0eClP0kkl5JczZykLo6kxjYQGE1mRALY5v0i+2jb5SGcBo5QyS0EmfoB4mla1xMXZhP fDfQ+IZB1b/ukTKw1VOzzLlE6/pCW51iZaDCBQRbTBwsvF53dAoOYBkHBmYtuHpgdQ72W3zbr 7VgAqvLAn4r+eehlAQfv87+BqJQ1hWvvGlIcBsF2GeCUyyqAWwl8c3XnyKjhCp5Rr6ajVYLuA 81refT6rqxyyfXnZtO5xkF7icqC6l2OVyt7Wu0EUQsQGe3tZWX1My67gIv4EF0UcOGauCb7E9 w3Rv6Jm1UPC219K9EXaznesBBowSaa1IyvMY9k4odEBgPT7FbLuLRzd3CYXH/YCb+BCPWpdOv N6cMcHGyPVtXxwLyIJ6bwoAKbCdBhs/Klkkj81SpyKxxrMVIuVe72iTANLIle0OXuguOAatj/ MX8GjUTdI4xZu6mIZ9luJRzayjVDcDFnVfIdSyvAkmcRx8CasJO4AfMf+7JbqylQc729ePdbD 63jsvnRPtlk1iVPaJogGLFS44XlQeX82ZvhfnpUYi5MoUNubqylru9SzEi0OUqzVNvddlw2YL cax3k+Fl/ImsqrwsaxE1KaCAxP2af8jfCq6k12UlJjR/WIzWDhsu9z7wRniGMl3Dh+tEyBD0w m//r+0NW+CB6i01QVwkS/cvki13aVbsDtKiwYSwlLZOlw2uHEK1Qk3jQQ6aDjcBjw9qmzGhbA 9dI3HWamH6zaZf69xxaRT+PDNSiL7SzfcS+ACW7A176WC5DA+IEseUNlj2zcP41ucHfoVNss3 +lOJENqsqbV7OS8G2WcNhr+LK+5V4FQh1uvZiMzgfW3dZd3X2YmP7fq9FgZNJL934OFmz0/vn XDcOshFa+Fa2Y8raE Content-Transfer-Encoding: quoted-printable Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Mar 25, 2020 at 09:19:24AM +0000, Quentin Deslandes wrote: > On 03/24/20 16:18:30, Dan Carpenter wrote: > > That's a bit over engineering something which is pretty trivial. > > Normally, we would just make the size a define instead of a magic numb= er > > 14. > > My bad, I meant "define", not "macro". > > > If people change the size in the future (unlikely) and it causes a bug > > then they kind of deserve it because they need to ensure all the new > > stuff is initialized, right? If they change it and it results in a > > buffer overflow then static checkers would complain. If they changed = it > > and it resulted in uninitialized data being used then it would be zero > > so that's okay. > > I wasn't sure where I should stand on this, that's clearer now. > > Thanks, > Quentin Dan and Quentin, thanks for your time to review my work, and make comments= . oscar carter