From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-wm0-f43.google.com ([74.125.82.43]:35968 "EHLO mail-wm0-f43.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758100AbcCCVjQ (ORCPT ); Thu, 3 Mar 2016 16:39:16 -0500 Received: by mail-wm0-f43.google.com with SMTP id n186so8918691wmn.1 for ; Thu, 03 Mar 2016 13:39:15 -0800 (PST) Subject: Re: [git:media_tree/master] [media] media: rc: nuvoton: support reading / writing wakeup sequence via sysfs To: Mauro Carvalho Chehab References: <56D87FE9.4000408@gmail.com> <20160303155200.43d4c5e7@recife.lan> <56D88EFB.1090105@gmail.com> <20160303175650.33edd31c@recife.lan> Cc: linux-media@vger.kernel.org From: Heiner Kallweit Message-ID: <56D8AEF4.2060708@gmail.com> Date: Thu, 3 Mar 2016 22:39:00 +0100 MIME-Version: 1.0 In-Reply-To: <20160303175650.33edd31c@recife.lan> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: Am 03.03.2016 um 21:56 schrieb Mauro Carvalho Chehab: > Em Thu, 03 Mar 2016 20:22:35 +0100 > Heiner Kallweit escreveu: > >> Am 03.03.2016 um 19:52 schrieb Mauro Carvalho Chehab: >>> Em Thu, 03 Mar 2016 19:18:17 +0100 >>> Heiner Kallweit escreveu: >>> >>>> Am 03.03.2016 um 12:28 schrieb Mauro Carvalho Chehab: >>>>> This is an automatic generated email to let you know that the following patch were queued at the >>>>> http://git.linuxtv.org/cgit.cgi/media_tree.git tree: >>>>> >>>>> Subject: [media] media: rc: nuvoton: support reading / writing wakeup sequence via sysfs >>>>> Author: Heiner Kallweit >>>>> Date: Mon Feb 8 17:25:59 2016 -0200 >>>>> >>>>> This patch adds a binary attribute /sys/class/rc/rc?/wakeup_data which >>>>> allows to read / write the wakeup sequence. >>>>> >>>> When working on another module I was reminded not to forget updating Documentation/ABI. >>>> I think the same applies here. This patch introduces a new sysfs attribute that should >>>> be documented. I'll submit a patch for adding Documentation/ABI/testing/sysfs-class-rc-nuvoton >>> >>> Good point. >>> >>> Another thing: wouldn't be better to use a text format? This would make >>> esier to import from LIRC's irrecord format: >>> >>> begin raw_codes >>> >>> name power >>> 850 900 1750 1800 850 900 >>> 850 900 1750 900 850 1800 >>> 850 900 850 900 850 900 >>> 1750 1800 800 >>> >>> end raw_codes >>> >>> Regards, >>> Mauro >>> >> Most likely this is possible, but it would mean that we need a parser / generator >> for this text format in the driver / kernel code. And I have my doubts that parsing >> an application-specific file format (that could change anytime) in kernel code is >> a good thing. Converting this format to raw binary data is better off in userspace >> I think. What's your opinion? > > I'm not telling that it should parse irrecord format, but, instead, to > accept an ascii sequence of timings. The problem with raw binary data > is that the format varies on big endian and long endian architectures, > with doesn't seem to be nice to an API, as a data sequence recorded > on one machine may not work on some other one. > The Nuvoton wakeup sequence is a number of bytes (bit 7 = pulse/space indicator, bit 6..0 = length / 50us). It doesn't include any multi-byte numbers. Therefore endianness doesn't affect us. However I also like the idea of having a text format with pulses / spaces in us. (as that's what utilities like mode2 display) It's easier for the user to deal with. I'll prepare an alternative patch using such a text format. >> >>>> >>>> Rgds, Heiner >>>> >>>>> In combination with the core extension for exposing the most recent raw >>>>> packet this allows to easily define and set a wakeup sequence. >>>>> >>>>> At least on my Zotac CI321 the BIOS resets the wakeup sequence at each boot >>>>> to a factory default. Therefore I use a udev rule >>>>> SUBSYSTEM=="rc", DRIVERS=="nuvoton-cir", ACTION=="add", RUN+="