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: Sat, 25 Aug 2012 17:53:50 +0200 From: Sebastian Hesselbarth <sebastian.hesselbarth@...il.com> To: Stephen Warren <swarren@...dotorg.org> CC: Thomas Petazzoni <thomas.petazzoni@...e-electrons.com>, Grant Likely <grant.likely@...retlab.ca>, Rob Herring <rob.herring@...xeda.com>, Rob Landley <rob@...dley.net>, Russell King <linux@....linux.org.uk>, Lior Amsalem <alior@...vell.com>, Andrew Lunn <andrew@...n.ch>, Gregory CLEMENT <gregory.clement@...e-electrons.com>, Ben Dooks <ben.dooks@...ethink.co.uk>, Linus Walleij <linus.walleij@...aro.org>, devicetree-discuss@...ts.ozlabs.org, linux-doc@...r.kernel.org, linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org Subject: Re: [PATCH v2 1/9] pinctrl: mvebu: pinctrl driver core On 08/24/2012 05:34 AM, Stephen Warren wrote: > On 08/23/2012 05:01 PM, Sebastian Hesselbarth wrote: >> So possible, valid combinations for uart1 would be: >> (a) mpp_uart1; >> (b) mpp_uart1, mpp2, mpp3; >> (c) mpp_uart1, mpp21, mpp22; >> (d) mpp_uart1, mpp2, mpp22; >> (e) mpp_uart1, mpp21, mpp3; >> [...] > In the example above, there is a single function named "uart1". If this > was all the HW supported, I'd expect the driver's > pinmux_ops.get_functions_count() to return 1, > pinmux_ops.get_function_name(0) to return "uart1", and > pinmux_ops.get_function_name(n>0) to return an error. > > In practice, I assume there are many other options that can be muxed > onto mpp2/3/21/22/uart1, so they'd be included in the list as well. > > I don't expect any scanning, no. I'd expect that tables provided by the > SoC-specific drivers to be: > > * A table of pins > * A table of groups > * A table of functions > > No scanning involved. Stephen, now I do understand but in the current driver we pass pingroups associated with the available functions, i.e. "mpp2" with "uart1", "uart2", "sdio0", aso. IMHO for the above three functions it would be better to have functions associated with the corresponding groups, i.e. "uart1" with "mpp_uart1", "mpp2", "mpp3", aso. That would require some larger rework of the driver and therefore I just wanted to make sure, that I hit your expectations/explanations. Sebastian -- 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