Messages in this thread | | | Date | Tue, 1 Dec 2015 23:33:51 +0200 | From | Jarkko Sakkinen <> | Subject | Re: [PATCH v2 0/3] tpm_tis: Clean up force module parameter |
| |
On Tue, Dec 01, 2015 at 11:58:26AM -0700, Jason Gunthorpe wrote: > Drive the force=1 flow through the driver core. There are two main reasons to do this: > 1) To enable tpm_tis for OF environments requires a platform_device anyhow, so > the probe/release code needs to be re-used for that. > 2) Recent changes in the core code break the assumption that a driver will be > 'attached' to things created through platform_device_register_simple, > which causes the tpm core to blow up. > > v2: > - Make sure we request the mem resource in tpm_tis to avoid double-loading > the driver > - Re-order the init sequence so that a forced platform device gets first crack at > loading, and excludes the other mechanisms via the above > - Checkpatch clean > - Gotos renamed > > Martin, this should fix the double loading you noticed, please confirm. There > is a possibility the force path needs a bit more code to be compatible with > devm_ioremap_resource, I'm not sure, hoping not. > > Jason Gunthorpe (3): > tpm_tis: Disable interrupt auto probing on a per-device basis > tpm_tis: Use devm_ioremap_resource > tpm_tis: Clean up the force=1 module parameter
I went through the patches and didn't see anything that would shock me enough not to apply the patches in the current if they also work when tested *but* are these release critical for Linux v4.4?
I got a bit confused about the discussion that was going on about "where to fix the probe" crash whether or not both it should be fixed in both places.
Could you possibly make these apply on top of security/next and re-submit if needed?
/Jarkko
> drivers/char/tpm/tpm_tis.c | 203 +++++++++++++++++++++++++++------------------ > 1 file changed, 122 insertions(+), 81 deletions(-) > > -- > 2.1.4 >
| |