From mboxrd@z Thu Jan 1 00:00:00 1970 From: khilman@baylibre.com (Kevin Hilman) Date: Mon, 29 Aug 2016 14:28:10 -0500 Subject: [PATCH v5 0/6] Add Meson 8b / GXBB support to the IR driver In-Reply-To: <20160820095424.636-1-martin.blumenstingl@googlemail.com> (Martin Blumenstingl's message of "Sat, 20 Aug 2016 11:54:18 +0200") References: <20160819215547.20063-1-martin.blumenstingl@googlemail.com> <20160820095424.636-1-martin.blumenstingl@googlemail.com> Message-ID: To: linus-amlogic@lists.infradead.org List-Id: linus-amlogic.lists.infradead.org Martin Blumenstingl writes: > Newer Amlogic platforms (Meson 8b and GXBB) use a slightly different > register layout for their Infrared Remoete Controller. The decoder mode > is now configured in another register. Without the changes to the > meson-ir driver we are simply getting incorrect "durations" reported > from the hardware (because the hardware is not in time measurement aka > software decode mode). > > This problem was also noticed by some people trying to use this on an > ODROID-C1 and ODROID-C2 - the workaround there (probably because the > datasheets were not publicy available yet at that time) was to switch > to ir_raw_event_store_edge (which leaves it up to the kernel to measure > the duration of a pulse). See [0] and [1] for the corresponding > patches. > > Changes in v5: > - changed pin function and group names to remote_input_ao so they match > with the datasheet > > Tested-by: Neil Armstrong Thanks for the respin. I'll take the driver and DT/bindings through the arm-soc tree and Linus has taken the pinctrl patch. Kevin From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Hilman Subject: Re: [PATCH v5 0/6] Add Meson 8b / GXBB support to the IR driver Date: Mon, 29 Aug 2016 14:28:10 -0500 Message-ID: References: <20160819215547.20063-1-martin.blumenstingl@googlemail.com> <20160820095424.636-1-martin.blumenstingl@googlemail.com> Mime-Version: 1.0 Content-Type: text/plain Return-path: In-Reply-To: <20160820095424.636-1-martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org> (Martin Blumenstingl's message of "Sat, 20 Aug 2016 11:54:18 +0200") Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Martin Blumenstingl Cc: linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-gpio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-amlogic-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, narmstrong-rdvid1DuHRBWk0Htik3J/w@public.gmane.org, linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org, carlo-KA+7E9HrN00dnm+yROfE0A@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, mchehab-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, will.deacon-5wv7dgnIgG8@public.gmane.org, catalin.marinas-5wv7dgnIgG8@public.gmane.org, mark.rutland-5wv7dgnIgG8@public.gmane.org, robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, b.galvani-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org List-Id: linux-gpio@vger.kernel.org Martin Blumenstingl writes: > Newer Amlogic platforms (Meson 8b and GXBB) use a slightly different > register layout for their Infrared Remoete Controller. The decoder mode > is now configured in another register. Without the changes to the > meson-ir driver we are simply getting incorrect "durations" reported > from the hardware (because the hardware is not in time measurement aka > software decode mode). > > This problem was also noticed by some people trying to use this on an > ODROID-C1 and ODROID-C2 - the workaround there (probably because the > datasheets were not publicy available yet at that time) was to switch > to ir_raw_event_store_edge (which leaves it up to the kernel to measure > the duration of a pulse). See [0] and [1] for the corresponding > patches. > > Changes in v5: > - changed pin function and group names to remote_input_ao so they match > with the datasheet > > Tested-by: Neil Armstrong Thanks for the respin. I'll take the driver and DT/bindings through the arm-soc tree and Linus has taken the pinctrl patch. Kevin -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: khilman@baylibre.com (Kevin Hilman) Date: Mon, 29 Aug 2016 14:28:10 -0500 Subject: [PATCH v5 0/6] Add Meson 8b / GXBB support to the IR driver In-Reply-To: <20160820095424.636-1-martin.blumenstingl@googlemail.com> (Martin Blumenstingl's message of "Sat, 20 Aug 2016 11:54:18 +0200") References: <20160819215547.20063-1-martin.blumenstingl@googlemail.com> <20160820095424.636-1-martin.blumenstingl@googlemail.com> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Martin Blumenstingl writes: > Newer Amlogic platforms (Meson 8b and GXBB) use a slightly different > register layout for their Infrared Remoete Controller. The decoder mode > is now configured in another register. Without the changes to the > meson-ir driver we are simply getting incorrect "durations" reported > from the hardware (because the hardware is not in time measurement aka > software decode mode). > > This problem was also noticed by some people trying to use this on an > ODROID-C1 and ODROID-C2 - the workaround there (probably because the > datasheets were not publicy available yet at that time) was to switch > to ir_raw_event_store_edge (which leaves it up to the kernel to measure > the duration of a pulse). See [0] and [1] for the corresponding > patches. > > Changes in v5: > - changed pin function and group names to remote_input_ao so they match > with the datasheet > > Tested-by: Neil Armstrong Thanks for the respin. I'll take the driver and DT/bindings through the arm-soc tree and Linus has taken the pinctrl patch. Kevin From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-it0-f46.google.com ([209.85.214.46]:38848 "EHLO mail-it0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753500AbcH2T2X (ORCPT ); Mon, 29 Aug 2016 15:28:23 -0400 Received: by mail-it0-f46.google.com with SMTP id g62so1519238ith.1 for ; Mon, 29 Aug 2016 12:28:23 -0700 (PDT) From: Kevin Hilman To: Martin Blumenstingl Cc: linux-media@vger.kernel.org, linux-gpio@vger.kernel.org, linux-amlogic@lists.infradead.org, devicetree@vger.kernel.org, narmstrong@baylibre.com, linus.walleij@linaro.org, carlo@caione.org, linux-arm-kernel@lists.infradead.org, mchehab@kernel.org, will.deacon@arm.com, catalin.marinas@arm.com, mark.rutland@arm.com, robh+dt@kernel.org, b.galvani@gmail.com Subject: Re: [PATCH v5 0/6] Add Meson 8b / GXBB support to the IR driver References: <20160819215547.20063-1-martin.blumenstingl@googlemail.com> <20160820095424.636-1-martin.blumenstingl@googlemail.com> Date: Mon, 29 Aug 2016 14:28:10 -0500 In-Reply-To: <20160820095424.636-1-martin.blumenstingl@googlemail.com> (Martin Blumenstingl's message of "Sat, 20 Aug 2016 11:54:18 +0200") Message-ID: MIME-Version: 1.0 Content-Type: text/plain Sender: linux-media-owner@vger.kernel.org List-ID: Martin Blumenstingl writes: > Newer Amlogic platforms (Meson 8b and GXBB) use a slightly different > register layout for their Infrared Remoete Controller. The decoder mode > is now configured in another register. Without the changes to the > meson-ir driver we are simply getting incorrect "durations" reported > from the hardware (because the hardware is not in time measurement aka > software decode mode). > > This problem was also noticed by some people trying to use this on an > ODROID-C1 and ODROID-C2 - the workaround there (probably because the > datasheets were not publicy available yet at that time) was to switch > to ir_raw_event_store_edge (which leaves it up to the kernel to measure > the duration of a pulse). See [0] and [1] for the corresponding > patches. > > Changes in v5: > - changed pin function and group names to remote_input_ao so they match > with the datasheet > > Tested-by: Neil Armstrong Thanks for the respin. I'll take the driver and DT/bindings through the arm-soc tree and Linus has taken the pinctrl patch. Kevin