Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   David Laight <David.Laight@ACULAB.COM>
To:     'Jonathan Cameron' <jic23@kernel.org>,
        William Breathitt Gray <vilhelm.gray@gmail.com>
CC:     "kamel.bouhara@bootlin.com" <kamel.bouhara@bootlin.com>,
        "gwendal@chromium.org" <gwendal@chromium.org>,
        "alexandre.belloni@bootlin.com" <alexandre.belloni@bootlin.com>,
        "david@lechnology.com" <david@lechnology.com>,
        "felipe.balbi@linux.intel.com" <felipe.balbi@linux.intel.com>,
        "fabien.lahoudere@collabora.com" <fabien.lahoudere@collabora.com>,
        "linux-iio@vger.kernel.org" <linux-iio@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-stm32@st-md-mailman.stormreply.com" 
        <linux-stm32@st-md-mailman.stormreply.com>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "syednwaris@gmail.com" <syednwaris@gmail.com>,
        "patrick.havelange@essensium.com" <patrick.havelange@essensium.com>,
        "fabrice.gasnier@st.com" <fabrice.gasnier@st.com>,
        "mcoquelin.stm32@gmail.com" <mcoquelin.stm32@gmail.com>,
        "alexandre.torgue@st.com" <alexandre.torgue@st.com>
Subject: RE: [PATCH 0/4] Introduce the Counter character device interface
Thread-Topic: [PATCH 0/4] Introduce the Counter character device interface
Thread-Index: AQHWIVUDUyU0PU/R2k6dJAMe78u4OaiWaHyQ
Date:   Sun, 3 May 2020 14:21:11 +0000
Message-ID: <b2d51e3f9dfb4dd78156b2e945607e8d@AcuMS.aculab.com>
References: <cover.1588176662.git.vilhelm.gray@gmail.com>
 <20200503151314.2ac1fc2e@archlinux>
In-Reply-To: <20200503151314.2ac1fc2e@archlinux>
Accept-Language: en-GB, en-US
Content-Language: en-US
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8BIT
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

From: Jonathan Cameron
> Sent: 03 May 2020 15:13
...
> > The following are some questions I have about this patchset:
> >
> > 1. Should enums be used to represent standard counter component states
> >    (e.g. COUNTER_SIGNAL_LOW), or would these be better defined as int?
> >
> >    These standard counter component states are defined in the
> >    counter-types.h file and serve as constants used by counter device
> >    drivers and Counter subsystem components in order to ensure a
> >    consistent interface.
> >
> >    My concern is whether enum constants will cause problems when passed
> >    to userspace via the Counter character device ioctl calls. Along the
> >    same lines is whether the C bool datatype is safe to pass as well,
> >    given that it is a more modern C datatype.
> 
> For enums, I'd pass them as integers.
> 
> Bool is probably fine either way.

Always use fixed size types in any API structures.
Ensure that fields are always on their natural boundaries.

So no enums and no bools.
It may even be worth using uint64_t for any userspace pointers.

At some point you'll live to regret anything else.

	David

-
Registered Address Lakeside, Bramley Road, Mount Farm, Milton Keynes, MK1 1PT, UK
Registration No: 1397386 (Wales)