lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 24 Feb 2012 13:56:06 +0900 From: MyungJoo Ham <myungjoo.ham@...il.com> To: Arnd Bergmann <arnd@...db.de> Cc: Mark Brown <broonie@...nsource.wolfsonmicro.com>, linux-kernel@...r.kernel.org, NeilBrown <neilb@...e.de>, Randy Dunlap <rdunlap@...otime.net>, Mike Lockwood <lockwood@...roid.com>, Arve Hjønnevag <arve@...roid.com>, Kyungmin Park <kyungmin.park@...sung.com>, Donggeun Kim <dg77.kim@...sung.com>, gregkh@...uxfoundation.org, Linus Walleij <linus.walleij@...aro.org>, Dmitry Torokhov <dmitry.torokhov@...il.com>, Morten CHRISTIANSEN <morten.christiansen@...ricsson.com>, John Stultz <john.stultz@...aro.org>, Joerg Roedel <joerg.roedel@....com> Subject: Re: [PATCH v5 4/5] Extcon: support mutually exclusive relation between cables. On Wed, Feb 22, 2012 at 7:00 PM, Arnd Bergmann <arnd@...db.de> wrote: > On Wednesday 22 February 2012, MyungJoo Ham wrote: >> >> On Mon, Feb 20, 2012 at 11:27 AM, Mark Brown >> <broonie@...nsource.wolfsonmicro.com> wrote: >> > On Fri, Feb 10, 2012 at 03:40:37PM +0900, MyungJoo Ham wrote: >> > >> >> +What: /sys/class/extcon/.../mutually_exclusive >> >> +Date: December 2011 >> >> +Contact: MyungJoo Ham <myungjoo.ham@...sung.com> >> >> +Description: >> >> + Shows the relations of mutually exclusiveness. For example, >> >> + if the mutually_exclusive array of extcon_dev is >> >> + {0x3, 0x5, 0xC, 0x0}, the, the output is: >> >> + # cat mutually_exclusive >> >> + 0x3 >> >> + 0x5 >> >> + 0xC >> >> + # >> > >> > sysfs really isn't supposed to have more than one value per file so this >> > should be something like a directory with one array element per file. >> > It'd kind of be cheating but the values could be used as the filenames, >> > though serial numbers would be more idiomatic. >> >> Hmm.. I'll either use multiple files with a sub-directory or bin-attr >> to inhabit the values. Or I'll search for any other convinient way to >> express such an array thru sysfs. > > How about having one file for each cable and list the ones that it > cannot coexist with. In the above example, that would become > > mutually_exclusive/0: "0x6" > mutually_exclusive/1: "0x1" > mutually_exclusive/2: "0x9" > mutually_exclusive/3: "0x4" > > The information here is a bit redundant, but this would be much easier > to understand for human readers. I also wouldn't mind having a list > of exclusive cable names in each file, while would make it slightly > harder to parse but very easy for humans: > > mutually_exclusive/Video-in: "VGA" > mutually_exclusive/Video-out: "VGA" > mutually_exclusive/VGA: "Video-out Video-in" > > Arnd Ok, here goes an example how it would look like in the next revision of Extcon. I think this addresses your comments on multistate-support patch (3/5) /sys/class/extcon/max8997-muic.0/ name: "max8997-muic.0" state: 0x00000009 /* RW. 0xblahblah format for both read and write */ mutually_exclusive/ rule.0: 0x3 rule.1: 0x5 rule.2: 0xC cable.0/ name: "VGA" state: 1 /* RW: 0 or 1 */ mutually_exclusive/ cable.1 cable.2 cable.1/ ... How about this? Cheers! MyungJoo. -- MyungJoo Ham, Ph.D. System S/W Lab, S/W Center, Samsung Electronics -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists