From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755044AbdLFK0L (ORCPT ); Wed, 6 Dec 2017 05:26:11 -0500 Received: from userp2120.oracle.com ([156.151.31.85]:59042 "EHLO userp2120.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755028AbdLFK0H (ORCPT ); Wed, 6 Dec 2017 05:26:07 -0500 Date: Wed, 6 Dec 2017 13:23:56 +0300 From: Dan Carpenter To: Marcus Wolf Cc: Simon =?iso-8859-1?Q?Sandstr=F6m?= , gregkh@linuxfoundation.org, devel@driverdev.osuosl.org, linux@Wolf-Entwicklungen.de, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 04/11] staging: pi433: Rename enum optionOnOff in rf69_enum.h Message-ID: <20171206100833.3ficksukt6xaazl4@mwanda> References: <20171205220849.5486-1-simon@nikanor.nu> <20171205220849.5486-5-simon@nikanor.nu> <4c89dc10-ccce-7760-f806-3a19c3edf743@smarthome-wolf.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4c89dc10-ccce-7760-f806-3a19c3edf743@smarthome-wolf.de> User-Agent: NeoMutt/20170609 (1.8.3) X-Proofpoint-Virus-Version: vendor=nai engine=5900 definitions=8736 signatures=668641 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 spamscore=0 suspectscore=0 malwarescore=0 phishscore=0 adultscore=0 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1709140000 definitions=main-1712060152 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Dec 06, 2017 at 11:46:41AM +0200, Marcus Wolf wrote: > > diff --git a/drivers/staging/pi433/rf69_enum.h b/drivers/staging/pi433/rf69_enum.h > > index babe597e2ec6..5247e9269de9 100644 > > --- a/drivers/staging/pi433/rf69_enum.h > > +++ b/drivers/staging/pi433/rf69_enum.h > > @@ -18,9 +18,9 @@ > > #ifndef RF69_ENUM_H > > #define RF69_ENUM_H > > -enum optionOnOff { > > - optionOff, > > - optionOn > > +enum option_on_off { > > + OPTION_OFF, > > + OPTION_ON > > }; > > enum mode { > > > > Hi Simon, > > nice work. > > Thank you very much for all the style fixes :-) > Wow... This was the one patch I thought was going to sink this patchset... Isn't enum optionOnOff part of the userspace headers? I thought we weren't allowed to change that. regards, dan carpenter