From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from unicorn.mansr.com ([81.2.72.234]:52138 "EHLO unicorn.mansr.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751754AbdISLyU (ORCPT ); Tue, 19 Sep 2017 07:54:20 -0400 From: =?iso-8859-1?Q?M=E5ns_Rullg=E5rd?= To: Marc Gonzalez Cc: Sean Young , Mauro Carvalho Chehab , linux-media , Thibaud Cornic , Mason Subject: Re: [PATCH v1] media: rc: Add driver for tango IR decoder References: Date: Tue, 19 Sep 2017 12:54:18 +0100 In-Reply-To: (Marc Gonzalez's message of "Tue, 19 Sep 2017 13:34:38 +0200") Message-ID: MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: 8BIT Sender: linux-media-owner@vger.kernel.org List-ID: Marc Gonzalez writes: > On 19/09/2017 11:48, Måns Rullgård wrote: > >> Did you test the NEC32 variant? I don't have anything that produces >> such codes. > > I don't have a NEC32 IR remote control either. > > IIUC, NEC32 means 16-bit address and 16-bit command. > > I checked the RTL with a HW engineer. The HW block translates the IR > pulses into logical 1s and 0s according to the protocol parameters, > stuffs the logical bits into a register, and fires an IRQ when there > are 32 bits available. The block doesn't care if the bits are significant > or just checksums (that is left up to software). In that case I suppose it ought to just work. -- Måns Rullgård