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
| ||
|
Message-Id: <201212060110.32227.PeterHuewe@gmx.de> Date: Thu, 6 Dec 2012 01:10:32 +0100 From: Peter Hüwe <PeterHuewe@....de> To: Kent Yoder <key@...ux.vnet.ibm.com> Cc: Mathias LEBLANC <Mathias.LEBLANC@...com>, "Jean-Luc BLANC" <jean-luc.blanc@...com>, "Sirrix@...per.es" <Sirrix@...per.es>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, Rajiv Andrade <mail@...jiv.net>, "tpmdd-devel@...ts.sourceforge.net" <tpmdd-devel@...ts.sourceforge.net>, Kent Yoder <shpedoikal@...il.com> Subject: Re: [tpmdd-devel] [PATCH 1/1] TPM: STMicroelectronics ST33 I2C KERNEL 3.x.x Am Donnerstag, 6. Dezember 2012, 00:09:41 schrieb Kent Yoder: > > > > > static int tpm_st33_i2c_pm_suspend(struct i2c_client *client, > > > pm_message_t mesg) > > > > > >... > > > > > > static int tpm_st33_i2c_pm_resume(struct i2c_client *client) > > > > > >,,, > > >static struct i2c_driver tpm_st33_i2c_driver = { > > > > > > .driver = { > > > > > > .owner = THIS_MODULE, > > > .name = TPM_ST33_I2C, > > > }, > > > > > > .probe = tpm_st33_i2c_probe, > > > .remove = tpm_st33_i2c_remove, > > > .resume = tpm_st33_i2c_pm_resume, > > > .suspend = tpm_st33_i2c_pm_suspend, > > > .id_table = tpm_st33_i2c_id > > > > > >}; > > > > Please convert resume/suspend to .driver.pm > > > > It's pretty easy. > > See this post for details > > http://sourceforge.net/mailarchive/message.php?msg_id=29516784 > > Rafael did spent quite a lot of effort to convert almost every driver > > back then, so we should 'fix' new ones. > > Not sure how easy this will be considering these routines are > i2c-specific -- they don't just call the tpm_tpm_* functions like the > other drivers. Why are they really i2c-specific? The only i2c specific part is the i2c_get_clientdata - which calls get_dev_drvdata internally. And I since we already get the correct struct device we don't need to use TPM_VPRIV in order to access the platform data. I'll send a patch - see next email. @Matthias: can you please test this? As I cannot do this. Thanks, Peter -- 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