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 X-Spam-Level: X-Spam-Status: No, score=-2.3 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_PASS,USER_AGENT_MUTT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 435D0C6778F for ; Sat, 28 Jul 2018 14:51:19 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 0073B20878 for ; Sat, 28 Jul 2018 14:51:18 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 0073B20878 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=linuxfoundation.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729057AbeG1QSC (ORCPT ); Sat, 28 Jul 2018 12:18:02 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:33884 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728996AbeG1QSC (ORCPT ); Sat, 28 Jul 2018 12:18:02 -0400 Received: from localhost (D57D388D.static.ziggozakelijk.nl [213.125.56.141]) by mail.linuxfoundation.org (Postfix) with ESMTPSA id B8B7AC77; Sat, 28 Jul 2018 14:51:15 +0000 (UTC) Date: Sat, 28 Jul 2018 16:51:12 +0200 From: Greg KH To: rkir@google.com Cc: linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 2/3] tty: Make constants to be enums instead of #define in goldfish.c Message-ID: <20180728145112.GA12252@kroah.com> References: <20180725005133.107823-1-rkir@google.com> <20180725005133.107823-2-rkir@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180725005133.107823-2-rkir@google.com> User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jul 24, 2018 at 05:51:32PM -0700, rkir@google.com wrote: > From: Roman Kiryanov > > enums produce better compilation errors than defines. Yes, they can, if you use them correctly. Which you are not doing here, so this patch really does nothing. Why should we take it? confused, greg k-h