Re: linux-next: build failure after merge of the tpmdd tree

[Jarkko Sakkinen]

On Wed, Jun 05, 2019 at 12:09:46PM +1000, Stephen Rothwell wrote:
> Hi all,
> 
> After merging the tpmdd tree, today's linux-next build (arm
> multi_v7_defconfig) failed like this:

I'm sorry about the incident and thank you for reporting this.

/Jarkko

linux-next: build failure after merge of the tpmdd tree

[Stephen Rothwell]

Hi all,

After merging the tpmdd tree, today's linux-next build (arm
multi_v7_defconfig) failed like this:

include/linux/tpm_eventlog.h: In function '__calc_tpm2_event_size':
drivers/firmware/efi/tpm.c:7:35: error: implicit declaration of function 
'early_memremap'; did you mean 'early_memtest'? 
[-Werror=implicit-function-declaration]
 #define TPM_MEMREMAP(start, size) early_memremap(start, size)
   ^~
include/linux/tpm_eventlog.h:182:13: note: in expansion of macro 'TPM_MEMREMAP'
   mapping = TPM_MEMREMAP((unsigned long)marker_start,
 ^~~~
In file included from drivers/firmware/efi/tpm.c:13:
include/linux/tpm_eventlog.h:182:11: warning: assignment to 'void *' from 'int' 
makes pointer from integer without a cast [-Wint-conversion]
   mapping = TPM_MEMREMAP((unsigned long)marker_start,
   ^
drivers/firmware/efi/tpm.c:8:35: error: implicit declaration of function 
'early_memunmap'; did you mean 'early_memtest'? 
[-Werror=implicit-function-declaration]
 #define TPM_MEMUNMAP(start, size) early_memunmap(start, size)
   ^~
include/linux/tpm_eventlog.h:207:4: note: in expansion of macro 'TPM_MEMUNMAP'
TPM_MEMUNMAP(mapping, mapping_size);
^~~~
In file included from drivers/firmware/efi/tpm.c:13:
include/linux/tpm_eventlog.h:209:12: warning: assignment to 'void *' from 'int' 
makes pointer from integer without a cast [-Wint-conversion]
mapping = TPM_MEMREMAP((unsigned long)marker,
^
include/linux/tpm_eventlog.h:243:11: warning: assignment to 'void *' from 'int' 
makes pointer from integer without a cast [-Wint-conversion]
   mapping = TPM_MEMREMAP((unsigned long)marker,
   ^
In file included from ./arch/arm/include/generated/asm/early_ioremap.h:1,
 from drivers/firmware/efi/tpm.c:15:
include/asm-generic/early_ioremap.h: At top level:
include/asm-generic/early_ioremap.h:13:14: error: conflicting types for 
'early_memremap'
 extern void *early_memremap(resource_size_t phys_addr,
  ^~
drivers/firmware/efi/tpm.c:7:35: note: previous implicit declaration of 
'early_memremap' was here
 #define TPM_MEMREMAP(start, size) early_memremap(start, size)
   ^~
include/linux/tpm_eventlog.h:182:13: note: in expansion of macro 'TPM_MEMREMAP'
   mapping = TPM_MEMREMAP((unsigned long)marker_start,
 ^~~~
In file included from ./arch/arm/include/generated/asm/early_ioremap.h:1,
 from drivers/firmware/efi/tpm.c:15:
include/asm-generic/early_ioremap.h:20:13: warning: conflicting types for 
'early_memunmap'
 extern void early_memunmap(void *addr, unsigned long size);
 ^~
drivers/firmware/efi/tpm.c:8:35: note: previous implicit declaration of 
'early_memunmap' was here
 #define TPM_MEMUNMAP(start, size) early_memunmap(start, size)
   ^~
include/linux/tpm_eventlog.h:207:4: note: in expansion of macro 'TPM_MEMUNMAP'
TPM_MEMUNMAP(mapping, mapping_size);
^~~~
drivers/firmware/efi/tpm.c: In function 'efi_tpm_eventlog_init':
drivers/firmware/efi/tpm.c:81:10: warning: passing argument 1 of 
'tpm2_calc_event_log_size' makes pointer from integer without a cast 
[-Wint-conversion]
  tbl_size = tpm2_calc_event_log_size(efi.tpm_final_log
  ~
  + sizeof(final_tbl->version)
  
  + sizeof(final_tbl->nr_events),
  ^~
drivers/firmware/efi/tpm.c:20:43: note: expected 'void *' but argument is of 
type 'long unsigned int'
 static int tpm2_calc_event_log_size(void *data, int count, void *size_info)
 ~~^~~~
cc1: some warnings being treated as errors

Caused by commit

  b25b956d13d5 ("tpm: Reserve the TPM final events table")

I have used the tpmdd tree from next-20190604 for today.

-- 
Cheers,
Stephen Rothwell


pgpVmsRswq6ex.pgp
Description: OpenPGP digital signature

Re: linux-next: build failure after merge of the tpmdd tree

[Jarkko Sakkinen]

On Thu, Apr 18, 2019 at 12:40:48PM -0700, Matthew Garrett wrote:
> On Tue, Apr 16, 2019 at 8:36 AM Jarkko Sakkinen
>  wrote:
> > Matthew, looking at the code I guess the includes are in wrong order
> > i.e. early_ioremap.h should be included before tpm_eventlog.h. Do you
> > agree that this is the correct conclusion? I can do the update.
> 
> Yes, I believe that that's the correct fix.

I'll hold  up until the previous flush of patches is in
security/next-general (bug fixes for v5.1 changes).

/Jarkko

Re: linux-next: build failure after merge of the tpmdd tree

[Matthew Garrett]

On Tue, Apr 16, 2019 at 8:36 AM Jarkko Sakkinen
 wrote:
> Matthew, looking at the code I guess the includes are in wrong order
> i.e. early_ioremap.h should be included before tpm_eventlog.h. Do you
> agree that this is the correct conclusion? I can do the update.

Yes, I believe that that's the correct fix.

Re: linux-next: build failure after merge of the tpmdd tree

[Jarkko Sakkinen]

On Tue, Apr 16, 2019 at 01:07:40PM +1000, Stephen Rothwell wrote:
> Hi Jarkko,
> 
> After merging the tpmdd tree, today's linux-next build (arm
> multi_v7_defconfig) failed like this:

Matthew, looking at the code I guess the includes are in wrong order
i.e. early_ioremap.h should be included before tpm_eventlog.h. Do you
agree that this is the correct conclusion? I can do the update.

Just as a temporary measure I removed the commits from my next branch.

/Jarkko

linux-next: build failure after merge of the tpmdd tree

[Stephen Rothwell]

Hi Jarkko,

After merging the tpmdd tree, today's linux-next build (arm
multi_v7_defconfig) failed like this:

include/linux/tpm_eventlog.h: In function '__calc_tpm2_event_size':
drivers/firmware/efi/tpm.c:7:35: error: implicit declaration of function 
'early_memremap'; did you mean 'early_memtest'? 
[-Werror=implicit-function-declaration]
 #define TPM_MEMREMAP(start, size) early_memremap(start, size)
   ^~
include/linux/tpm_eventlog.h:182:13: note: in expansion of macro 'TPM_MEMREMAP'
   mapping = TPM_MEMREMAP((unsigned long)marker_start,
 ^~~~
In file included from drivers/firmware/efi/tpm.c:13:
include/linux/tpm_eventlog.h:182:11: warning: assignment to 'void *' from 'int' 
makes pointer from integer without a cast [-Wint-conversion]
   mapping = TPM_MEMREMAP((unsigned long)marker_start,
   ^
drivers/firmware/efi/tpm.c:8:35: error: implicit declaration of function 
'early_memunmap'; did you mean 'early_memtest'? 
[-Werror=implicit-function-declaration]
 #define TPM_MEMUNMAP(start, size) early_memunmap(start, size)
   ^~
include/linux/tpm_eventlog.h:203:4: note: in expansion of macro 'TPM_MEMUNMAP'
TPM_MEMUNMAP(mapping, mapping_size);
^~~~
In file included from drivers/firmware/efi/tpm.c:13:
include/linux/tpm_eventlog.h:205:12: warning: assignment to 'void *' from 'int' 
makes pointer from integer without a cast [-Wint-conversion]
mapping = TPM_MEMREMAP((unsigned long)marker_start,
^
include/linux/tpm_eventlog.h:225:14: warning: assignment to 'void *' from 'int' 
makes pointer from integer without a cast [-Wint-conversion]
  mapping = TPM_MEMREMAP((unsigned long)marker_start,
  ^
include/linux/tpm_eventlog.h:251:11: warning: assignment to 'void *' from 'int' 
makes pointer from integer without a cast [-Wint-conversion]
   mapping = TPM_MEMREMAP((unsigned long)marker_start,
   ^
In file included from ./arch/arm/include/generated/asm/early_ioremap.h:1,
 from drivers/firmware/efi/tpm.c:15:
include/asm-generic/early_ioremap.h: At top level:
include/asm-generic/early_ioremap.h:13:14: error: conflicting types for 
'early_memremap'
 extern void *early_memremap(resource_size_t phys_addr,
  ^~
drivers/firmware/efi/tpm.c:7:35: note: previous implicit declaration of 
'early_memremap' was here
 #define TPM_MEMREMAP(start, size) early_memremap(start, size)
   ^~
include/linux/tpm_eventlog.h:182:13: note: in expansion of macro 'TPM_MEMREMAP'
   mapping = TPM_MEMREMAP((unsigned long)marker_start,
 ^~~~
In file included from ./arch/arm/include/generated/asm/early_ioremap.h:1,
 from drivers/firmware/efi/tpm.c:15:
include/asm-generic/early_ioremap.h:20:13: warning: conflicting types for 
'early_memunmap'
 extern void early_memunmap(void *addr, unsigned long size);
 ^~
drivers/firmware/efi/tpm.c:8:35: note: previous implicit declaration of 
'early_memunmap' was here
 #define TPM_MEMUNMAP(start, size) early_memunmap(start, size)
   ^~
include/linux/tpm_eventlog.h:203:4: note: in expansion of macro 'TPM_MEMUNMAP'
TPM_MEMUNMAP(mapping, mapping_size);
^~~~

Caused by commit

  05165bf3d231 ("tpm: Abstract crypto agile event size calculations")

and maybe

  cdb75b359079 ("tpm: Reserve the TPM final events table")

I have used the tmpdd tree from next-20190415 for today.

-- 
Cheers,
Stephen Rothwell


pgpYHTiuNFdVt.pgp
Description: OpenPGP digital signature

Re: linux-next: build failure after merge of the tpmdd tree

[Jarkko Sakkinen]

Hi

On Wed, May 10, 2017 at 01:23:27PM +1000, Stephen Rothwell wrote:
> Hi Jarkko,
> 
> After merging the tpmdd tree, today's linux-next build (powerpc
> allyesconfig) failed like this:
> 
> drivers/char/tpm/tpm_tis.c: In function 'tpm_tis_init':
> drivers/char/tpm/tpm_tis.c:193:23: error: passing argument 1 of 
> 'check_acpi_tpm2' from incompatible pointer type 
> [-Werror=incompatible-pointer-types]
>   rc = check_acpi_tpm2(dev);
>^
> drivers/char/tpm/tpm_tis.c:129:12: note: expected 'struct acpi_device *' but 
> argument is of type 'struct device *'
>  static int check_acpi_tpm2(struct acpi_device *dev)
> ^
> 
> Caused by commit
> 
>   e3b975d27b46 ("tpm_tis: Consolidate the platform and acpi probe flow")
> 
> CONFIG_ACPI is not set for this build.
> 
> I have added the following patch for today.
> 
> From: Stephen Rothwell 
> Date: Wed, 10 May 2017 13:15:35 +1000
> Subject: [PATCH] tpm_tis: fixup for CONFIG_ACPI not set
> 
> Signed-off-by: Stephen Rothwell 
> ---
>  drivers/char/tpm/tpm_tis.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/char/tpm/tpm_tis.c b/drivers/char/tpm/tpm_tis.c
> index 54d90e31a7b6..b14d4aa97af8 100644
> --- a/drivers/char/tpm/tpm_tis.c
> +++ b/drivers/char/tpm/tpm_tis.c
> @@ -126,7 +126,7 @@ static int check_acpi_tpm2(struct device *dev)
>   return 0;
>  }
>  #else
> -static int check_acpi_tpm2(struct acpi_device *dev)
> +static int check_acpi_tpm2(struct device *dev)
>  {
>   return 0;
>  }
> -- 
> 2.11.0
> 
> -- 
> Cheers,
> Stephen Rothwell

Thank you! I'll squash this.

/Jarkko

Re: linux-next: build failure after merge of the tpmdd tree

[Jarkko Sakkinen]

Hi

On Wed, May 10, 2017 at 01:23:27PM +1000, Stephen Rothwell wrote:
> Hi Jarkko,
> 
> After merging the tpmdd tree, today's linux-next build (powerpc
> allyesconfig) failed like this:
> 
> drivers/char/tpm/tpm_tis.c: In function 'tpm_tis_init':
> drivers/char/tpm/tpm_tis.c:193:23: error: passing argument 1 of 
> 'check_acpi_tpm2' from incompatible pointer type 
> [-Werror=incompatible-pointer-types]
>   rc = check_acpi_tpm2(dev);
>^
> drivers/char/tpm/tpm_tis.c:129:12: note: expected 'struct acpi_device *' but 
> argument is of type 'struct device *'
>  static int check_acpi_tpm2(struct acpi_device *dev)
> ^
> 
> Caused by commit
> 
>   e3b975d27b46 ("tpm_tis: Consolidate the platform and acpi probe flow")
> 
> CONFIG_ACPI is not set for this build.
> 
> I have added the following patch for today.
> 
> From: Stephen Rothwell 
> Date: Wed, 10 May 2017 13:15:35 +1000
> Subject: [PATCH] tpm_tis: fixup for CONFIG_ACPI not set
> 
> Signed-off-by: Stephen Rothwell 
> ---
>  drivers/char/tpm/tpm_tis.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/char/tpm/tpm_tis.c b/drivers/char/tpm/tpm_tis.c
> index 54d90e31a7b6..b14d4aa97af8 100644
> --- a/drivers/char/tpm/tpm_tis.c
> +++ b/drivers/char/tpm/tpm_tis.c
> @@ -126,7 +126,7 @@ static int check_acpi_tpm2(struct device *dev)
>   return 0;
>  }
>  #else
> -static int check_acpi_tpm2(struct acpi_device *dev)
> +static int check_acpi_tpm2(struct device *dev)
>  {
>   return 0;
>  }
> -- 
> 2.11.0
> 
> -- 
> Cheers,
> Stephen Rothwell

Thank you! I'll squash this.

/Jarkko

linux-next: build failure after merge of the tpmdd tree

[Stephen Rothwell]

Hi Jarkko,

After merging the tpmdd tree, today's linux-next build (powerpc
allyesconfig) failed like this:

drivers/char/tpm/tpm_tis.c: In function 'tpm_tis_init':
drivers/char/tpm/tpm_tis.c:193:23: error: passing argument 1 of 
'check_acpi_tpm2' from incompatible pointer type 
[-Werror=incompatible-pointer-types]
  rc = check_acpi_tpm2(dev);
   ^
drivers/char/tpm/tpm_tis.c:129:12: note: expected 'struct acpi_device *' but 
argument is of type 'struct device *'
 static int check_acpi_tpm2(struct acpi_device *dev)
^

Caused by commit

  e3b975d27b46 ("tpm_tis: Consolidate the platform and acpi probe flow")

CONFIG_ACPI is not set for this build.

I have added the following patch for today.

From: Stephen Rothwell 
Date: Wed, 10 May 2017 13:15:35 +1000
Subject: [PATCH] tpm_tis: fixup for CONFIG_ACPI not set

Signed-off-by: Stephen Rothwell 
---
 drivers/char/tpm/tpm_tis.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/char/tpm/tpm_tis.c b/drivers/char/tpm/tpm_tis.c
index 54d90e31a7b6..b14d4aa97af8 100644
--- a/drivers/char/tpm/tpm_tis.c
+++ b/drivers/char/tpm/tpm_tis.c
@@ -126,7 +126,7 @@ static int check_acpi_tpm2(struct device *dev)
return 0;
 }
 #else
-static int check_acpi_tpm2(struct acpi_device *dev)
+static int check_acpi_tpm2(struct device *dev)
 {
return 0;
 }
-- 
2.11.0

-- 
Cheers,
Stephen Rothwell

linux-next: build failure after merge of the tpmdd tree

[Stephen Rothwell]

Hi Jarkko,

After merging the tpmdd tree, today's linux-next build (powerpc
allyesconfig) failed like this:

drivers/char/tpm/tpm_tis.c: In function 'tpm_tis_init':
drivers/char/tpm/tpm_tis.c:193:23: error: passing argument 1 of 
'check_acpi_tpm2' from incompatible pointer type 
[-Werror=incompatible-pointer-types]
  rc = check_acpi_tpm2(dev);
   ^
drivers/char/tpm/tpm_tis.c:129:12: note: expected 'struct acpi_device *' but 
argument is of type 'struct device *'
 static int check_acpi_tpm2(struct acpi_device *dev)
^

Caused by commit

  e3b975d27b46 ("tpm_tis: Consolidate the platform and acpi probe flow")

CONFIG_ACPI is not set for this build.

I have added the following patch for today.

From: Stephen Rothwell 
Date: Wed, 10 May 2017 13:15:35 +1000
Subject: [PATCH] tpm_tis: fixup for CONFIG_ACPI not set

Signed-off-by: Stephen Rothwell 
---
 drivers/char/tpm/tpm_tis.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/char/tpm/tpm_tis.c b/drivers/char/tpm/tpm_tis.c
index 54d90e31a7b6..b14d4aa97af8 100644
--- a/drivers/char/tpm/tpm_tis.c
+++ b/drivers/char/tpm/tpm_tis.c
@@ -126,7 +126,7 @@ static int check_acpi_tpm2(struct device *dev)
return 0;
 }
 #else
-static int check_acpi_tpm2(struct acpi_device *dev)
+static int check_acpi_tpm2(struct device *dev)
 {
return 0;
 }
-- 
2.11.0

-- 
Cheers,
Stephen Rothwell

Re: linux-next: build failure after merge of the tpmdd tree

[Jarkko Sakkinen]

On Mon, May 02, 2016 at 10:19:27PM +1000, Stephen Rothwell wrote:
> Hi Jarkko,
> 
> On Mon, 2 May 2016 14:10:36 +0300 Jarkko Sakkinen 
>  wrote:
> >
> > I applied the fix and merged it.
> 
> Unfortunately, you seem to have turned the leading whitespace from a
> TAB to spaces :-(

Fixed. Sorry. I also did cross-compilation for PPC to make sure that
this works. I'll take PPC compilation as part of my testing procedure
from now on in order to prevent this problem from happening again
in the first place.

> -- 
> Cheers,
> Stephen Rothwell

/Jarkko

Re: linux-next: build failure after merge of the tpmdd tree

[Jarkko Sakkinen]

On Mon, May 02, 2016 at 10:19:27PM +1000, Stephen Rothwell wrote:
> Hi Jarkko,
> 
> On Mon, 2 May 2016 14:10:36 +0300 Jarkko Sakkinen 
>  wrote:
> >
> > I applied the fix and merged it.
> 
> Unfortunately, you seem to have turned the leading whitespace from a
> TAB to spaces :-(

Fixed. Sorry. I also did cross-compilation for PPC to make sure that
this works. I'll take PPC compilation as part of my testing procedure
from now on in order to prevent this problem from happening again
in the first place.

> -- 
> Cheers,
> Stephen Rothwell

/Jarkko

Re: linux-next: build failure after merge of the tpmdd tree

[Stephen Rothwell]

Hi Jarkko,

On Mon, 2 May 2016 14:10:36 +0300 Jarkko Sakkinen 
 wrote:
>
> I applied the fix and merged it.

Unfortunately, you seem to have turned the leading whitespace from a
TAB to spaces :-(

-- 
Cheers,
Stephen Rothwell

Re: linux-next: build failure after merge of the tpmdd tree

[Stephen Rothwell]

Hi Jarkko,

On Mon, 2 May 2016 14:10:36 +0300 Jarkko Sakkinen 
 wrote:
>
> I applied the fix and merged it.

Unfortunately, you seem to have turned the leading whitespace from a
TAB to spaces :-(

-- 
Cheers,
Stephen Rothwell

Re: linux-next: build failure after merge of the tpmdd tree

[Jarkko Sakkinen]

On Sun, May 01, 2016 at 11:12:22PM +1000, Stephen Rothwell wrote:
> Hi Jarkko,
> 
> On Thu, 28 Apr 2016 12:02:24 +0300 Jarkko Sakkinen 
>  wrote:
> >
> > On Thu, 2016-04-28 at 15:32 +1000, Stephen Rothwell wrote:
> > > Hi Jarkko,
> > > 
> > > After merging the tpmdd tree, today's linux-next build (powerpc
> > > allyesconfig) failed like this:
> > > 
> > > In file included from /home/sfr/next/next/include/linux/rcupdate.h:38:0,
> > >  from /home/sfr/next/next/include/linux/idr.h:18,
> > >  from /home/sfr/next/next/include/linux/kernfs.h:14,
> > >  from /home/sfr/next/next/include/linux/sysfs.h:15,
> > >  from /home/sfr/next/next/include/linux/kobject.h:21,
> > >  from /home/sfr/next/next/include/linux/device.h:17,
> > >  from /home/sfr/next/next/include/linux/dma-mapping.h:6,
> > >  from 
> > > /home/sfr/next/next/drivers/char/tpm/tpm_ibmvtpm.c:18:
> > > /home/sfr/next/next/drivers/char/tpm/tpm_ibmvtpm.c: In function 
> > > 'tpm_ibmvtpm_probe':
> > > /home/sfr/next/next/include/linux/spinlock.h:295:1: error: expected ';' 
> > > before 'do'
> > >  do {   \
> > >  ^
> > > /home/sfr/next/next/drivers/char/tpm/tpm_ibmvtpm.c:632:2: note: in 
> > > expansion of macro
> > > 'spin_lock_init'
> > >   spin_lock_init(>rtce_lock);
> > >   ^
> > > 
> > > Caused by commit
> > > 
> > >   28157164b056 ("tpm: Remove useless priv field in struct 
> > > tpm_vendor_specific")
> > > 
> > > A ';' was missed.
> > > 
> > > I added the following patch for today.  
> > 
> > Sorry about this and thank you.
> > 
> > /Jarkko
> > 
> > > From: Stephen Rothwell 
> > > Date: Thu, 28 Apr 2016 15:27:17 +1000
> > > Subject: [PATCH] tpm: fix for typo in tpm/tpm_ibmvtpm.c
> > > 
> > > Fixes: 28157164b056 ("tpm: Remove useless priv field in struct 
> > > tpm_vendor_specific")
> > > Signed-off-by: Stephen Rothwell 
> > > ---
> > >  drivers/char/tpm/tpm_ibmvtpm.c | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > 
> > > diff --git a/drivers/char/tpm/tpm_ibmvtpm.c 
> > > b/drivers/char/tpm/tpm_ibmvtpm.c
> > > index 6b22826f0e11..946025a7413b 100644
> > > --- a/drivers/char/tpm/tpm_ibmvtpm.c
> > > +++ b/drivers/char/tpm/tpm_ibmvtpm.c
> > > @@ -627,7 +627,7 @@ static int tpm_ibmvtpm_probe(struct vio_dev *vio_dev,
> > >  
> > >   crq_q->index = 0;
> > >  
> > > - dev_set_drvdata(>dev, ibmvtpm)
> > > + dev_set_drvdata(>dev, ibmvtpm);
> > >  
> > >   spin_lock_init(>rtce_lock);
> > >  
> > > -- 
> > > 2.7.0
> > >   
> 
> Ping?

Acked-by: Jarkko Sakkinen 

I applied the fix and merged it.

> -- 
> Cheers,
> Stephen Rothwell

/Jarkko

Re: linux-next: build failure after merge of the tpmdd tree

[Jarkko Sakkinen]

On Sun, May 01, 2016 at 11:12:22PM +1000, Stephen Rothwell wrote:
> Hi Jarkko,
> 
> On Thu, 28 Apr 2016 12:02:24 +0300 Jarkko Sakkinen 
>  wrote:
> >
> > On Thu, 2016-04-28 at 15:32 +1000, Stephen Rothwell wrote:
> > > Hi Jarkko,
> > > 
> > > After merging the tpmdd tree, today's linux-next build (powerpc
> > > allyesconfig) failed like this:
> > > 
> > > In file included from /home/sfr/next/next/include/linux/rcupdate.h:38:0,
> > >  from /home/sfr/next/next/include/linux/idr.h:18,
> > >  from /home/sfr/next/next/include/linux/kernfs.h:14,
> > >  from /home/sfr/next/next/include/linux/sysfs.h:15,
> > >  from /home/sfr/next/next/include/linux/kobject.h:21,
> > >  from /home/sfr/next/next/include/linux/device.h:17,
> > >  from /home/sfr/next/next/include/linux/dma-mapping.h:6,
> > >  from 
> > > /home/sfr/next/next/drivers/char/tpm/tpm_ibmvtpm.c:18:
> > > /home/sfr/next/next/drivers/char/tpm/tpm_ibmvtpm.c: In function 
> > > 'tpm_ibmvtpm_probe':
> > > /home/sfr/next/next/include/linux/spinlock.h:295:1: error: expected ';' 
> > > before 'do'
> > >  do {   \
> > >  ^
> > > /home/sfr/next/next/drivers/char/tpm/tpm_ibmvtpm.c:632:2: note: in 
> > > expansion of macro
> > > 'spin_lock_init'
> > >   spin_lock_init(>rtce_lock);
> > >   ^
> > > 
> > > Caused by commit
> > > 
> > >   28157164b056 ("tpm: Remove useless priv field in struct 
> > > tpm_vendor_specific")
> > > 
> > > A ';' was missed.
> > > 
> > > I added the following patch for today.  
> > 
> > Sorry about this and thank you.
> > 
> > /Jarkko
> > 
> > > From: Stephen Rothwell 
> > > Date: Thu, 28 Apr 2016 15:27:17 +1000
> > > Subject: [PATCH] tpm: fix for typo in tpm/tpm_ibmvtpm.c
> > > 
> > > Fixes: 28157164b056 ("tpm: Remove useless priv field in struct 
> > > tpm_vendor_specific")
> > > Signed-off-by: Stephen Rothwell 
> > > ---
> > >  drivers/char/tpm/tpm_ibmvtpm.c | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > 
> > > diff --git a/drivers/char/tpm/tpm_ibmvtpm.c 
> > > b/drivers/char/tpm/tpm_ibmvtpm.c
> > > index 6b22826f0e11..946025a7413b 100644
> > > --- a/drivers/char/tpm/tpm_ibmvtpm.c
> > > +++ b/drivers/char/tpm/tpm_ibmvtpm.c
> > > @@ -627,7 +627,7 @@ static int tpm_ibmvtpm_probe(struct vio_dev *vio_dev,
> > >  
> > >   crq_q->index = 0;
> > >  
> > > - dev_set_drvdata(>dev, ibmvtpm)
> > > + dev_set_drvdata(>dev, ibmvtpm);
> > >  
> > >   spin_lock_init(>rtce_lock);
> > >  
> > > -- 
> > > 2.7.0
> > >   
> 
> Ping?

Acked-by: Jarkko Sakkinen 

I applied the fix and merged it.

> -- 
> Cheers,
> Stephen Rothwell

/Jarkko

Re: linux-next: build failure after merge of the tpmdd tree

[Stephen Rothwell]

Hi Jarkko,

On Thu, 28 Apr 2016 12:02:24 +0300 Jarkko Sakkinen 
 wrote:
>
> On Thu, 2016-04-28 at 15:32 +1000, Stephen Rothwell wrote:
> > Hi Jarkko,
> > 
> > After merging the tpmdd tree, today's linux-next build (powerpc
> > allyesconfig) failed like this:
> > 
> > In file included from /home/sfr/next/next/include/linux/rcupdate.h:38:0,
> >  from /home/sfr/next/next/include/linux/idr.h:18,
> >  from /home/sfr/next/next/include/linux/kernfs.h:14,
> >  from /home/sfr/next/next/include/linux/sysfs.h:15,
> >  from /home/sfr/next/next/include/linux/kobject.h:21,
> >  from /home/sfr/next/next/include/linux/device.h:17,
> >  from /home/sfr/next/next/include/linux/dma-mapping.h:6,
> >  from /home/sfr/next/next/drivers/char/tpm/tpm_ibmvtpm.c:18:
> > /home/sfr/next/next/drivers/char/tpm/tpm_ibmvtpm.c: In function 
> > 'tpm_ibmvtpm_probe':
> > /home/sfr/next/next/include/linux/spinlock.h:295:1: error: expected ';' 
> > before 'do'
> >  do {   \
> >  ^
> > /home/sfr/next/next/drivers/char/tpm/tpm_ibmvtpm.c:632:2: note: in 
> > expansion of macro
> > 'spin_lock_init'
> >   spin_lock_init(>rtce_lock);
> >   ^
> > 
> > Caused by commit
> > 
> >   28157164b056 ("tpm: Remove useless priv field in struct 
> > tpm_vendor_specific")
> > 
> > A ';' was missed.
> > 
> > I added the following patch for today.  
> 
> Sorry about this and thank you.
> 
> /Jarkko
> 
> > From: Stephen Rothwell 
> > Date: Thu, 28 Apr 2016 15:27:17 +1000
> > Subject: [PATCH] tpm: fix for typo in tpm/tpm_ibmvtpm.c
> > 
> > Fixes: 28157164b056 ("tpm: Remove useless priv field in struct 
> > tpm_vendor_specific")
> > Signed-off-by: Stephen Rothwell 
> > ---
> >  drivers/char/tpm/tpm_ibmvtpm.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/drivers/char/tpm/tpm_ibmvtpm.c b/drivers/char/tpm/tpm_ibmvtpm.c
> > index 6b22826f0e11..946025a7413b 100644
> > --- a/drivers/char/tpm/tpm_ibmvtpm.c
> > +++ b/drivers/char/tpm/tpm_ibmvtpm.c
> > @@ -627,7 +627,7 @@ static int tpm_ibmvtpm_probe(struct vio_dev *vio_dev,
> >  
> >     crq_q->index = 0;
> >  
> > -   dev_set_drvdata(>dev, ibmvtpm)
> > +   dev_set_drvdata(>dev, ibmvtpm);
> >  
> >     spin_lock_init(>rtce_lock);
> >  
> > -- 
> > 2.7.0
> >   

Ping?

-- 
Cheers,
Stephen Rothwell

Re: linux-next: build failure after merge of the tpmdd tree

[Stephen Rothwell]

Hi Jarkko,

On Thu, 28 Apr 2016 12:02:24 +0300 Jarkko Sakkinen 
 wrote:
>
> On Thu, 2016-04-28 at 15:32 +1000, Stephen Rothwell wrote:
> > Hi Jarkko,
> > 
> > After merging the tpmdd tree, today's linux-next build (powerpc
> > allyesconfig) failed like this:
> > 
> > In file included from /home/sfr/next/next/include/linux/rcupdate.h:38:0,
> >  from /home/sfr/next/next/include/linux/idr.h:18,
> >  from /home/sfr/next/next/include/linux/kernfs.h:14,
> >  from /home/sfr/next/next/include/linux/sysfs.h:15,
> >  from /home/sfr/next/next/include/linux/kobject.h:21,
> >  from /home/sfr/next/next/include/linux/device.h:17,
> >  from /home/sfr/next/next/include/linux/dma-mapping.h:6,
> >  from /home/sfr/next/next/drivers/char/tpm/tpm_ibmvtpm.c:18:
> > /home/sfr/next/next/drivers/char/tpm/tpm_ibmvtpm.c: In function 
> > 'tpm_ibmvtpm_probe':
> > /home/sfr/next/next/include/linux/spinlock.h:295:1: error: expected ';' 
> > before 'do'
> >  do {   \
> >  ^
> > /home/sfr/next/next/drivers/char/tpm/tpm_ibmvtpm.c:632:2: note: in 
> > expansion of macro
> > 'spin_lock_init'
> >   spin_lock_init(>rtce_lock);
> >   ^
> > 
> > Caused by commit
> > 
> >   28157164b056 ("tpm: Remove useless priv field in struct 
> > tpm_vendor_specific")
> > 
> > A ';' was missed.
> > 
> > I added the following patch for today.  
> 
> Sorry about this and thank you.
> 
> /Jarkko
> 
> > From: Stephen Rothwell 
> > Date: Thu, 28 Apr 2016 15:27:17 +1000
> > Subject: [PATCH] tpm: fix for typo in tpm/tpm_ibmvtpm.c
> > 
> > Fixes: 28157164b056 ("tpm: Remove useless priv field in struct 
> > tpm_vendor_specific")
> > Signed-off-by: Stephen Rothwell 
> > ---
> >  drivers/char/tpm/tpm_ibmvtpm.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/drivers/char/tpm/tpm_ibmvtpm.c b/drivers/char/tpm/tpm_ibmvtpm.c
> > index 6b22826f0e11..946025a7413b 100644
> > --- a/drivers/char/tpm/tpm_ibmvtpm.c
> > +++ b/drivers/char/tpm/tpm_ibmvtpm.c
> > @@ -627,7 +627,7 @@ static int tpm_ibmvtpm_probe(struct vio_dev *vio_dev,
> >  
> >     crq_q->index = 0;
> >  
> > -   dev_set_drvdata(>dev, ibmvtpm)
> > +   dev_set_drvdata(>dev, ibmvtpm);
> >  
> >     spin_lock_init(>rtce_lock);
> >  
> > -- 
> > 2.7.0
> >   

Ping?

-- 
Cheers,
Stephen Rothwell

Re: linux-next: build failure after merge of the tpmdd tree

[Jarkko Sakkinen]

On Thu, 2016-04-28 at 15:32 +1000, Stephen Rothwell wrote:
> Hi Jarkko,
> 
> After merging the tpmdd tree, today's linux-next build (powerpc
> allyesconfig) failed like this:
> 
> In file included from /home/sfr/next/next/include/linux/rcupdate.h:38:0,
>  from /home/sfr/next/next/include/linux/idr.h:18,
>  from /home/sfr/next/next/include/linux/kernfs.h:14,
>  from /home/sfr/next/next/include/linux/sysfs.h:15,
>  from /home/sfr/next/next/include/linux/kobject.h:21,
>  from /home/sfr/next/next/include/linux/device.h:17,
>  from /home/sfr/next/next/include/linux/dma-mapping.h:6,
>  from /home/sfr/next/next/drivers/char/tpm/tpm_ibmvtpm.c:18:
> /home/sfr/next/next/drivers/char/tpm/tpm_ibmvtpm.c: In function 
> 'tpm_ibmvtpm_probe':
> /home/sfr/next/next/include/linux/spinlock.h:295:1: error: expected ';' 
> before 'do'
>  do {   \
>  ^
> /home/sfr/next/next/drivers/char/tpm/tpm_ibmvtpm.c:632:2: note: in expansion 
> of macro
> 'spin_lock_init'
>   spin_lock_init(>rtce_lock);
>   ^
> 
> Caused by commit
> 
>   28157164b056 ("tpm: Remove useless priv field in struct 
> tpm_vendor_specific")
> 
> A ';' was missed.
> 
> I added the following patch for today.

Sorry about this and thank you.

/Jarkko

> From: Stephen Rothwell 
> Date: Thu, 28 Apr 2016 15:27:17 +1000
> Subject: [PATCH] tpm: fix for typo in tpm/tpm_ibmvtpm.c
> 
> Fixes: 28157164b056 ("tpm: Remove useless priv field in struct 
> tpm_vendor_specific")
> Signed-off-by: Stephen Rothwell 
> ---
>  drivers/char/tpm/tpm_ibmvtpm.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/char/tpm/tpm_ibmvtpm.c b/drivers/char/tpm/tpm_ibmvtpm.c
> index 6b22826f0e11..946025a7413b 100644
> --- a/drivers/char/tpm/tpm_ibmvtpm.c
> +++ b/drivers/char/tpm/tpm_ibmvtpm.c
> @@ -627,7 +627,7 @@ static int tpm_ibmvtpm_probe(struct vio_dev *vio_dev,
>  
>   crq_q->index = 0;
>  
> - dev_set_drvdata(>dev, ibmvtpm)
> + dev_set_drvdata(>dev, ibmvtpm);
>  
>   spin_lock_init(>rtce_lock);
>  
> -- 
> 2.7.0
> 

Re: linux-next: build failure after merge of the tpmdd tree

[Jarkko Sakkinen]

On Thu, 2016-04-28 at 15:32 +1000, Stephen Rothwell wrote:
> Hi Jarkko,
> 
> After merging the tpmdd tree, today's linux-next build (powerpc
> allyesconfig) failed like this:
> 
> In file included from /home/sfr/next/next/include/linux/rcupdate.h:38:0,
>  from /home/sfr/next/next/include/linux/idr.h:18,
>  from /home/sfr/next/next/include/linux/kernfs.h:14,
>  from /home/sfr/next/next/include/linux/sysfs.h:15,
>  from /home/sfr/next/next/include/linux/kobject.h:21,
>  from /home/sfr/next/next/include/linux/device.h:17,
>  from /home/sfr/next/next/include/linux/dma-mapping.h:6,
>  from /home/sfr/next/next/drivers/char/tpm/tpm_ibmvtpm.c:18:
> /home/sfr/next/next/drivers/char/tpm/tpm_ibmvtpm.c: In function 
> 'tpm_ibmvtpm_probe':
> /home/sfr/next/next/include/linux/spinlock.h:295:1: error: expected ';' 
> before 'do'
>  do {   \
>  ^
> /home/sfr/next/next/drivers/char/tpm/tpm_ibmvtpm.c:632:2: note: in expansion 
> of macro
> 'spin_lock_init'
>   spin_lock_init(>rtce_lock);
>   ^
> 
> Caused by commit
> 
>   28157164b056 ("tpm: Remove useless priv field in struct 
> tpm_vendor_specific")
> 
> A ';' was missed.
> 
> I added the following patch for today.

Sorry about this and thank you.

/Jarkko

> From: Stephen Rothwell 
> Date: Thu, 28 Apr 2016 15:27:17 +1000
> Subject: [PATCH] tpm: fix for typo in tpm/tpm_ibmvtpm.c
> 
> Fixes: 28157164b056 ("tpm: Remove useless priv field in struct 
> tpm_vendor_specific")
> Signed-off-by: Stephen Rothwell 
> ---
>  drivers/char/tpm/tpm_ibmvtpm.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/char/tpm/tpm_ibmvtpm.c b/drivers/char/tpm/tpm_ibmvtpm.c
> index 6b22826f0e11..946025a7413b 100644
> --- a/drivers/char/tpm/tpm_ibmvtpm.c
> +++ b/drivers/char/tpm/tpm_ibmvtpm.c
> @@ -627,7 +627,7 @@ static int tpm_ibmvtpm_probe(struct vio_dev *vio_dev,
>  
>   crq_q->index = 0;
>  
> - dev_set_drvdata(>dev, ibmvtpm)
> + dev_set_drvdata(>dev, ibmvtpm);
>  
>   spin_lock_init(>rtce_lock);
>  
> -- 
> 2.7.0
> 

linux-next: build failure after merge of the tpmdd tree

[Stephen Rothwell]

Hi Jarkko,

After merging the tpmdd tree, today's linux-next build (powerpc
allyesconfig) failed like this:

In file included from /home/sfr/next/next/include/linux/rcupdate.h:38:0,
 from /home/sfr/next/next/include/linux/idr.h:18,
 from /home/sfr/next/next/include/linux/kernfs.h:14,
 from /home/sfr/next/next/include/linux/sysfs.h:15,
 from /home/sfr/next/next/include/linux/kobject.h:21,
 from /home/sfr/next/next/include/linux/device.h:17,
 from /home/sfr/next/next/include/linux/dma-mapping.h:6,
 from /home/sfr/next/next/drivers/char/tpm/tpm_ibmvtpm.c:18:
/home/sfr/next/next/drivers/char/tpm/tpm_ibmvtpm.c: In function 
'tpm_ibmvtpm_probe':
/home/sfr/next/next/include/linux/spinlock.h:295:1: error: expected ';' before 
'do'
 do {   \
 ^
/home/sfr/next/next/drivers/char/tpm/tpm_ibmvtpm.c:632:2: note: in expansion of 
macro 'spin_lock_init'
  spin_lock_init(>rtce_lock);
  ^

Caused by commit

  28157164b056 ("tpm: Remove useless priv field in struct tpm_vendor_specific")

A ';' was missed.

I added the following patch for today.

From: Stephen Rothwell 
Date: Thu, 28 Apr 2016 15:27:17 +1000
Subject: [PATCH] tpm: fix for typo in tpm/tpm_ibmvtpm.c

Fixes: 28157164b056 ("tpm: Remove useless priv field in struct 
tpm_vendor_specific")
Signed-off-by: Stephen Rothwell 
---
 drivers/char/tpm/tpm_ibmvtpm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/char/tpm/tpm_ibmvtpm.c b/drivers/char/tpm/tpm_ibmvtpm.c
index 6b22826f0e11..946025a7413b 100644
--- a/drivers/char/tpm/tpm_ibmvtpm.c
+++ b/drivers/char/tpm/tpm_ibmvtpm.c
@@ -627,7 +627,7 @@ static int tpm_ibmvtpm_probe(struct vio_dev *vio_dev,
 
crq_q->index = 0;
 
-   dev_set_drvdata(>dev, ibmvtpm)
+   dev_set_drvdata(>dev, ibmvtpm);
 
spin_lock_init(>rtce_lock);
 
-- 
2.7.0

-- 
Cheers,
Stephen Rothwell

linux-next: build failure after merge of the tpmdd tree

[Stephen Rothwell]

Hi Jarkko,

After merging the tpmdd tree, today's linux-next build (powerpc
allyesconfig) failed like this:

In file included from /home/sfr/next/next/include/linux/rcupdate.h:38:0,
 from /home/sfr/next/next/include/linux/idr.h:18,
 from /home/sfr/next/next/include/linux/kernfs.h:14,
 from /home/sfr/next/next/include/linux/sysfs.h:15,
 from /home/sfr/next/next/include/linux/kobject.h:21,
 from /home/sfr/next/next/include/linux/device.h:17,
 from /home/sfr/next/next/include/linux/dma-mapping.h:6,
 from /home/sfr/next/next/drivers/char/tpm/tpm_ibmvtpm.c:18:
/home/sfr/next/next/drivers/char/tpm/tpm_ibmvtpm.c: In function 
'tpm_ibmvtpm_probe':
/home/sfr/next/next/include/linux/spinlock.h:295:1: error: expected ';' before 
'do'
 do {   \
 ^
/home/sfr/next/next/drivers/char/tpm/tpm_ibmvtpm.c:632:2: note: in expansion of 
macro 'spin_lock_init'
  spin_lock_init(>rtce_lock);
  ^

Caused by commit

  28157164b056 ("tpm: Remove useless priv field in struct tpm_vendor_specific")

A ';' was missed.

I added the following patch for today.

From: Stephen Rothwell 
Date: Thu, 28 Apr 2016 15:27:17 +1000
Subject: [PATCH] tpm: fix for typo in tpm/tpm_ibmvtpm.c

Fixes: 28157164b056 ("tpm: Remove useless priv field in struct 
tpm_vendor_specific")
Signed-off-by: Stephen Rothwell 
---
 drivers/char/tpm/tpm_ibmvtpm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/char/tpm/tpm_ibmvtpm.c b/drivers/char/tpm/tpm_ibmvtpm.c
index 6b22826f0e11..946025a7413b 100644
--- a/drivers/char/tpm/tpm_ibmvtpm.c
+++ b/drivers/char/tpm/tpm_ibmvtpm.c
@@ -627,7 +627,7 @@ static int tpm_ibmvtpm_probe(struct vio_dev *vio_dev,
 
crq_q->index = 0;
 
-   dev_set_drvdata(>dev, ibmvtpm)
+   dev_set_drvdata(>dev, ibmvtpm);
 
spin_lock_init(>rtce_lock);
 
-- 
2.7.0

-- 
Cheers,
Stephen Rothwell