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: Tue, 10 Apr 2012 11:49:28 +0800 From: Dong Aisheng <aisheng.dong@...escale.com> To: Stephen Warren <swarren@...dotorg.org> CC: Dong Aisheng-B29396 <B29396@...escale.com>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, "linux-arm-kernel@...ts.infradead.org" <linux-arm-kernel@...ts.infradead.org>, "linus.walleij@...ricsson.com" <linus.walleij@...ricsson.com> Subject: Re: [PATCH 1/1] pinctrl: add some error checking for user interfaces On Mon, Apr 09, 2012 at 11:03:06PM +0800, Stephen Warren wrote: > On 04/09/2012 04:30 AM, Dong Aisheng wrote: > > From: Dong Aisheng <dong.aisheng@...aro.org> > > > > This patch can avoid kernel oops in case the mux or config > > function is not supported by driver. > > > > Signed-off-by: Dong Aisheng <dong.aisheng@...aro.org> > > --- > > drivers/pinctrl/pinconf.c | 4 ++++ > > drivers/pinctrl/pinmux.c | 15 +++++++++++++-- > > 2 files changed, 17 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/pinctrl/pinconf.c b/drivers/pinctrl/pinconf.c > > index eb3a14f..384dcc1 100644 > > --- a/drivers/pinctrl/pinconf.c > > +++ b/drivers/pinctrl/pinconf.c > > @@ -448,8 +448,12 @@ static void pinconf_dump_pin(struct pinctrl_dev *pctldev, > > static int pinconf_pins_show(struct seq_file *s, void *what) > > { > > struct pinctrl_dev *pctldev = s->private; > > + const struct pinconf_ops *ops = pctldev->desc->confops; > > unsigned i, pin; > > > > + if (!ops || !ops->pin_config_get) > > + return 0; > > I don't think this is necessary; it looks like this function (and those > it calls) always checks ops where they're used. > Yes, it's true. But it will still dump all pins configs although there's no config value which is almost the same as another debug entry 'pins'. It looks it does not make too much sense to dump this info if no config support. The existing pinconf_group_show did like this way(no output). So i changed pinconf_pins_show in the same behavior as pinconf_group_show. Do you think it's reasonable? Regards Dong Aisheng -- 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