On Mon, Sep 13, 2021 at 03:46:48PM +0200, Florian Eckert wrote: > Hello Jason, > > > > The device works than as expected. > > > > > > Signed-off-by: Florian Eckert <fe@xxxxxxxxxx> > > > v2: > > > * use variable type instead of le32_to_cpus function call > > > drivers/char/tpm/tpm_i2c_infineon.c | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > And if you do this it need to be made sparse clean/etc > > Sorry for the stupid question, but what exactly do you mean? There is a tool called sparse that checks the endia notations and verfies correctness It will complain if you do __le32 x x = le32tocpu(x) you neeed another variable to store the cpu version Jason