Received: by 2002:ac0:a5a7:0:0:0:0:0 with SMTP id m36-v6csp1903861imm; Thu, 2 Aug 2018 03:04:13 -0700 (PDT) X-Google-Smtp-Source: AAOMgpeQC9qlZ8zwvuFFpJwVHxBLwVZLDJPePFkYqJTcefrklcKKfmb0YOTfz15zL2Zw/LGeLudW X-Received: by 2002:a17:902:9a83:: with SMTP id w3-v6mr1772281plp.75.1533204253324; Thu, 02 Aug 2018 03:04:13 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1533204253; cv=none; d=google.com; s=arc-20160816; b=BLGpfCPF87C8QgOddtl3dPeYIlPQ4+V+r7fXGuVqFPDlbn6oIB8Zp8xd5jzZkR6Mrq q4pKKw8GVPA9nfitfoeuQLxaDZk4Ww0rKeXuHz45w9LGiX2hEaY6jU398KodwFmQicCg 4QZxBAHER7s97nvN/9kFE3goqQqaKGME8iRsnoEbtuU6YMFuPynm3MZhuYDcHTjYlyOt knK2oBaNAmzFkb+x5mWSLeKdAAbJSEgvwKsD7TMljKynQw9Rb2pOYaSTJx2/xoFT8CnE Se3GhIuqBFfrV3R/YBObK4CBvDAPQ2leDRqiFE4YuZ/DjBTjEn0Ui61TwaYFvPq22vIE a4bg== 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:mime-version :organization:references:in-reply-to:message-id:subject:cc:to:from :date:arc-authentication-results; bh=3RAXf65avT6vyJZn4eoK62xypVilZIrYch4dtVPRswM=; b=FUKWXayiVhhmqxnyGgDyrBVdwhVA5A45JQP9LFzk2thzzgvWrPSh8eEqJDQeGHAuA/ N4SjLmtQy/ZS5nakWVqYFHK92Axiedl6LZVaRcjupdx1TcflS/WbHClVu2Bf3Y5428jZ Xbbr++cTXNsPzOMm06tPrhdBZu71rM7KgT4hzGMRyORqj+OXO4G2mL/p2cNl34Rungbt fXlP274Zl3LI4LLizeHcvnewRAYqLoZ//G3SQVMjnaJZ72va00pODqPE9F6lJsyfTeVz j34lUHCURtB5PEth3iEaDrvGNOmoutec7EP6XZajNQilHm7li5CCdGkNOI4u3KUAUssS yveA== ARC-Authentication-Results: i=1; mx.google.com; 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 o1-v6si1696149pfe.259.2018.08.02.03.03.59; Thu, 02 Aug 2018 03:04:13 -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; 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 S1732295AbeHBLwm (ORCPT + 99 others); Thu, 2 Aug 2018 07:52:42 -0400 Received: from www.llwyncelyn.cymru ([82.70.14.225]:45694 "EHLO fuzix.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1732126AbeHBLwk (ORCPT ); Thu, 2 Aug 2018 07:52:40 -0400 Received: from alans-desktop (82-70-14-226.dsl.in-addr.zen.co.uk [82.70.14.226]) by fuzix.org (8.15.2/8.15.2) with ESMTP id w72A2AQD011879; Thu, 2 Aug 2018 11:02:10 +0100 Date: Thu, 2 Aug 2018 11:02:10 +0100 From: Alan Cox To: rkir@google.com Cc: gregkh@linuxfoundation.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 3/3] tty: Replace goldfish_tty_line_count with a #define Message-ID: <20180802110210.65392224@alans-desktop> In-Reply-To: <20180725005133.107823-3-rkir@google.com> References: <20180725005133.107823-1-rkir@google.com> <20180725005133.107823-3-rkir@google.com> Organization: Intel Corporation X-Mailer: Claws Mail 3.16.0 (GTK+ 2.24.32; x86_64-redhat-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 24 Jul 2018 17:51:33 -0700 rkir@google.com wrote: > From: Roman Kiryanov > > The driver never mutates this variable - no benefits of > keeping it mutable. > > Signed-off-by: Roman Kiryanov > --- > Changes in v2: > - Replaced "const u32" with "#define". v1 was IMHO definitely better. And you are correct that gcc will not allocate memory for a static const object unless you do things like take its address. It will (unlike a define) however properly typecheck everything. Alan