From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Subject: Re: Add support of Wiegand card reader To: Jonathan Cameron , Matt Ranostay Cc: Jonathan Cameron , Lars-Peter Clausen , linux-iio@vger.kernel.org References: <07edbbde-ac69-82c1-948b-a14f217cd467@gmail.com> <1ab90ba2-04c2-a690-7227-5d047ff2140c@gmail.com> <20180412122458.00001e51@huawei.com> From: =?UTF-8?Q?Ga=c3=abtan_Carlier?= Message-ID: <38c0d89a-a868-e369-68d5-4b9983bca5a4@gmail.com> Date: Thu, 12 Apr 2018 14:20:17 +0200 MIME-Version: 1.0 In-Reply-To: <20180412122458.00001e51@huawei.com> Content-Type: text/plain; charset=utf-8 List-ID: Hi, On 04/12/2018 01:24 PM, Jonathan Cameron wrote: > On Wed, 11 Apr 2018 20:41:30 -0700 > Matt Ranostay wrote: > >> On Wed, Apr 11, 2018 at 1:36 AM, Gaëtan Carlier wrote: >>> On 04/11/2018 05:37 AM, Matt Ranostay wrote: >>>> On Tue, Apr 10, 2018 at 1:57 AM, Gaëtan Carlier wrote: >>>>> Hello, >>>>> Thank you for your reply. >>>>> >>>>> On 04/10/2018 01:26 AM, Matt Ranostay wrote: >>>>>> On Mon, Apr 9, 2018 at 7:05 AM, Gaëtan Carlier wrote: >>>>>>> Hello, >>>>>>> I would like to add support of Wiegand card reader [1]. Where can I place it : drivers/iio/access/wiegand.c or drivers/iio/security/wiegand.c? >>>>>> >>>>>> Question here is why is this ideal for iio versus using the input subsystem? >>>>> What can I send to userspace from a bitstream using Input subsystem ? >>>>> Datas on Wiegand depend on readers used/setup. The best is to send bitstream directly to userspace and let user split bitstream into facilty/area code, card ID, ... >>>>> My idea is to send a structure with >>>>> * number of bits read >>>>> * if two parity bits are valid >>>>> * the bitstream itself saved in a 64 bit unsigned integer (or a buffer of chars) including or not the parity bits (I still have to decide). I don't see readers with more than 60 bits. >>>>> >>>>> I already start writting driver based on humidity/dht11.c that uses similar timings. >>>> >>>> Another question is what would your struct iio_chan_spec definition >>>> look like? Namely the .type assignments >>>> >>> For now, I did not have define any channels and num_channels in struct iio_dev in probe function. >>> That is what I have to investigate today : study how buffers work for IIO subsystem. >>> The idea is to have a blocking call from the userspace and every time a card is read, the buffer is filled and pushed to userspace. Userspace application will be unblocked and the buffer will be read. >>> >> >> Yeah buffers need to work independent of userspace and can't be blocking at all. >> >>> Couldn't I create a new iio_chan_type IIO_ID (or even IIO_BIOMETRIC) ? >>> >> >> Yeah IIO_BIOMETRIC or even IIO_ID is a bit odd since it isn't an SI >> unit or similar. Maybe Jonathan has some thoughts on this > > I'll confess I'm not sure this is really a good fit for IIO. It's kind > of similar to some of the things like fingerprint readers that generate > a block of data in some cryptic format that needs a userspace library to > interpret it. > > We have talked in the past about optimal storage for binary data (GPI signals > on devices that scan them in groups). This is kind of like that I guess. > > So, Gaetan, what were you thinking the actual data layout in the buffer > would be? > The buffer will contain : a u64 variable, a counter that will tell how much bits in u64 variables a used (wiegand is generally 26 bits or 34 bits but it can rise upto 60 bits but this situation is really rare). Initially, I though that IIO subsystem was done for this kind of interface but after more reading it is only suited for ADC/DAC devices. Unfortunately, I can not use input subsystem because, I need raw data that must not be converted to Key code, ... So I am little lost. Where can I place my code ? Even is Wiegand is a very old/unsecured/... protocol, it is still used a lot. And with pulse of some microseconds, it is difficult to handle them via exported GPIO on sysfs and do all the stuff in userspace. I clearly understand (now) that Wiegand can not stay in IIO but do you have any lead for me please ? > >> >>>>> >>>>>> >>>>>> Also a few issues/points: >>>>>> >>>>>> * What is the interface to the Wiegand data? If it is inputting on a >>>>>> GPIO that is for sure not going to be real time enough to get valid >>>>>> data >>>>> Wiegand protocol uses two data lines. Both data lines are pulled-up. When a 0 must be send, D0 line is set low, when a 1 must be sent, D1 line is set low. D0 and D1 line can never be set low at the same time. Reader will convert ID of the card into binary data flow of 26, 34, .... 60 bits. >>>>> The length of a data flow is configured in the reader and all readers connected on the same Wiegand bus have generally the same length. >>>>> >>>>>> * Aren't there multiple Wiegand transport standards? >>>>> Yes they are several data length. >>>>> >>>>>> * Most keyreaders I've seen are emulated as keyboard devices so they >>>>>> use the input subsystem. >>>>> If they are emulated as keyboard, they don't use Wiegand anymore ! Wiegand is still often used for hotels, ... >>>>> >>>>>> * Also if the data being output is ASCII data, then I'm not sure that >>>>>> fits anywhere in iioData received from Wiegand bus are bits. They are not packed in 8 bits or any other size. >>>>> >>>>>> >>>>>> - Matt >>>>>> >>>>>> >>>>>>> >>>>>>> Thank you for your help? >>>>>>> Best regards, >>>>>>> Gaëtan. >>>>>>> >>>>>>> [1] https://en.wikipedia.org/wiki/Wiegand_interface >>>>>>> -- >>>>>>> To unsubscribe from this list: send the line "unsubscribe linux-iio" in >>>>>>> the body of a message to majordomo@vger.kernel.org >>>>>>> More majordomo info at http://vger.kernel.org/majordomo-info.html >>>>> >>>>> Regards, >>>>> Gaëtan. >>> >>> Thank you, >>> Regards, >>> Gaëtan. >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-iio" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html > Thank you. Regards, Gaëtan.