All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] tpm: remove unneeded include of actbl2.h
@ 2016-01-04 19:19 ` Jerry Snitselaar
  0 siblings, 0 replies; 5+ messages in thread
From: Jerry Snitselaar @ 2016-01-04 19:19 UTC (permalink / raw)
  To: tpmdd-devel; +Cc: linux-kernel, jarkko.sakkinen

tpm_tis.c already gets actbl2.h via linux/acpi.h -> acpi/acpi.h ->
acpi/actbl.h -> acpi/actbl2.h, so the direct include in tpm_tis.c
is not needed.

Signed-off-by: Jerry Snitselaar <jsnitsel@redhat.com>
---

Jarrko, this is a trivial thing I noticed while doing some work.
Not sure if you guys care, but thought I'd send it in anyways.

 drivers/char/tpm/tpm_tis.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/char/tpm/tpm_tis.c b/drivers/char/tpm/tpm_tis.c
index 65f7eec..461d486 100644
--- a/drivers/char/tpm/tpm_tis.c
+++ b/drivers/char/tpm/tpm_tis.c
@@ -28,7 +28,6 @@
 #include <linux/wait.h>
 #include <linux/acpi.h>
 #include <linux/freezer.h>
-#include <acpi/actbl2.h>
 #include "tpm.h"
 
 enum tis_access {
-- 
2.6.1


^ permalink raw reply related	[flat|nested] 5+ messages in thread
* [PATCH] tpm: remove unneeded include of actbl2.h
@ 2016-01-04 19:19 ` Jerry Snitselaar
  0 siblings, 0 replies; 5+ messages in thread
From: Jerry Snitselaar @ 2016-01-04 19:19 UTC (permalink / raw)
  To: tpmdd-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f
  Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA

tpm_tis.c already gets actbl2.h via linux/acpi.h -> acpi/acpi.h ->
acpi/actbl.h -> acpi/actbl2.h, so the direct include in tpm_tis.c
is not needed.

Signed-off-by: Jerry Snitselaar <jsnitsel-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
---

Jarrko, this is a trivial thing I noticed while doing some work.
Not sure if you guys care, but thought I'd send it in anyways.

 drivers/char/tpm/tpm_tis.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/char/tpm/tpm_tis.c b/drivers/char/tpm/tpm_tis.c
index 65f7eec..461d486 100644
--- a/drivers/char/tpm/tpm_tis.c
+++ b/drivers/char/tpm/tpm_tis.c
@@ -28,7 +28,6 @@
 #include <linux/wait.h>
 #include <linux/acpi.h>
 #include <linux/freezer.h>
-#include <acpi/actbl2.h>
 #include "tpm.h"
 
 enum tis_access {
-- 
2.6.1


------------------------------------------------------------------------------

^ permalink raw reply related	[flat|nested] 5+ messages in thread
* Re: [tpmdd-devel] [PATCH] tpm: remove unneeded include of actbl2.h
  2016-01-04 19:19 ` Jerry Snitselaar
  (?)
@ 2016-01-04 20:11 ` Jerry Snitselaar
  -1 siblings, 0 replies; 5+ messages in thread
From: Jerry Snitselaar @ 2016-01-04 20:11 UTC (permalink / raw)
  To: tpmdd-devel; +Cc: linux-kernel, peterhuewe, tpmdd, jarkko.sakkinen

On Mon Jan 04 16, Jerry Snitselaar wrote:
>tpm_tis.c already gets actbl2.h via linux/acpi.h -> acpi/acpi.h ->
>acpi/actbl.h -> acpi/actbl2.h, so the direct include in tpm_tis.c
>is not needed.
>
>Signed-off-by: Jerry Snitselaar <jsnitsel@redhat.com>
>---
>
>Jarrko, this is a trivial thing I noticed while doing some work.
>Not sure if you guys care, but thought I'd send it in anyways.
>
> drivers/char/tpm/tpm_tis.c | 1 -
> 1 file changed, 1 deletion(-)
>
>diff --git a/drivers/char/tpm/tpm_tis.c b/drivers/char/tpm/tpm_tis.c
>index 65f7eec..461d486 100644
>--- a/drivers/char/tpm/tpm_tis.c
>+++ b/drivers/char/tpm/tpm_tis.c
>@@ -28,7 +28,6 @@
> #include <linux/wait.h>
> #include <linux/acpi.h>
> #include <linux/freezer.h>
>-#include <acpi/actbl2.h>
> #include "tpm.h"
>
> enum tis_access {
>-- 
>2.6.1
>

Not sure what happened, but it doesn't look like it CC'd Jarkko even
though it shows it in the git send-email output. Adding the other
maintainers as well.

	0001-tpm-remove-unneeded-include-of-actbl2.h.patch

	From: Jerry Snitselaar <jsnitsel@redhat.com>
	To: tpmdd-devel@lists.sourceforge.net
	Cc: linux-kernel@vger.kernel.org,
		jarkko.sakkinen@linux.intel.com
	Subject: [PATCH] tpm: remove unneeded include of actbl2.h
	Date: Mon,  4 Jan 2016 12:19:43 -0700
	Message-Id: <1451935183-2514-1-git-send-email-jsnitsel@redhat.com>
	X-Mailer: git-send-email 2.6.1

	Send this email? ([y]es|[n]o|[q]uit|[a]ll): a
	OK. Log says:
	Server: smtp.corp.redhat.com
	MAIL FROM:<jsnitsel@redhat.com>
	RCPT TO:<tpmdd-devel@lists.sourceforge.net>
	RCPT TO:<linux-kernel@vger.kernel.org>
	RCPT TO:<jarkko.sakkinen@linux.intel.com>
	From: Jerry Snitselaar <jsnitsel@redhat.com>
	To: tpmdd-devel@lists.sourceforge.net
	Cc: linux-kernel@vger.kernel.org,
		jarkko.sakkinen@linux.intel.com
	Subject: [PATCH] tpm: remove unneeded include of actbl2.h
	Date: Mon,  4 Jan 2016 12:19:43 -0700
	Message-Id: <1451935183-2514-1-git-send-email-jsnitsel@redhat.com>
	X-Mailer: git-send-email 2.6.1

	Result: 250 2.0.0 u04JJkA6027832 Message accepted for delivery


^ permalink raw reply	[flat|nested] 5+ messages in thread
* Re: [PATCH] tpm: remove unneeded include of actbl2.h
  2016-01-04 19:19 ` Jerry Snitselaar
  (?)
  (?)
@ 2016-01-05  6:49 ` Jarkko Sakkinen
       [not found]   ` <20160105064933.GA5835-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
  -1 siblings, 1 reply; 5+ messages in thread
From: Jarkko Sakkinen @ 2016-01-05  6:49 UTC (permalink / raw)
  To: Jerry Snitselaar; +Cc: tpmdd-devel, linux-kernel

On Mon, Jan 04, 2016 at 12:19:43PM -0700, Jerry Snitselaar wrote:
> tpm_tis.c already gets actbl2.h via linux/acpi.h -> acpi/acpi.h ->
> acpi/actbl.h -> acpi/actbl2.h, so the direct include in tpm_tis.c
> is not needed.
> 
> Signed-off-by: Jerry Snitselaar <jsnitsel@redhat.com>

Acked-by: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>

/Jarkko

> ---
> 
> Jarrko, this is a trivial thing I noticed while doing some work.
> Not sure if you guys care, but thought I'd send it in anyways.
> 
>  drivers/char/tpm/tpm_tis.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/char/tpm/tpm_tis.c b/drivers/char/tpm/tpm_tis.c
> index 65f7eec..461d486 100644
> --- a/drivers/char/tpm/tpm_tis.c
> +++ b/drivers/char/tpm/tpm_tis.c
> @@ -28,7 +28,6 @@
>  #include <linux/wait.h>
>  #include <linux/acpi.h>
>  #include <linux/freezer.h>
> -#include <acpi/actbl2.h>
>  #include "tpm.h"
>  
>  enum tis_access {
> -- 
> 2.6.1
> 

^ permalink raw reply	[flat|nested] 5+ messages in thread
* Re: [PATCH] tpm: remove unneeded include of actbl2.h
       [not found]   ` <20160105064933.GA5835-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
@ 2016-01-06 14:38     ` Jarkko Sakkinen
  0 siblings, 0 replies; 5+ messages in thread
From: Jarkko Sakkinen @ 2016-01-06 14:38 UTC (permalink / raw)
  To: Jerry Snitselaar; +Cc: tpmdd-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f

On Tue, Jan 05, 2016 at 08:49:33AM +0200, Jarkko Sakkinen wrote:
> On Mon, Jan 04, 2016 at 12:19:43PM -0700, Jerry Snitselaar wrote:
> > tpm_tis.c already gets actbl2.h via linux/acpi.h -> acpi/acpi.h ->
> > acpi/actbl.h -> acpi/actbl2.h, so the direct include in tpm_tis.c
> > is not needed.
> > 
> > Signed-off-by: Jerry Snitselaar <jsnitsel-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
> 
> Acked-by: Jarkko Sakkinen <jarkko.sakkinen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>

Applied to https://github.com/jsakkine/linux-tpmdd/commits/master

/Jarkko

------------------------------------------------------------------------------

^ permalink raw reply	[flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-01-06 14:38 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-01-04 19:19 [PATCH] tpm: remove unneeded include of actbl2.h Jerry Snitselaar
2016-01-04 19:19 ` Jerry Snitselaar
2016-01-04 20:11 ` [tpmdd-devel] " Jerry Snitselaar
2016-01-05  6:49 ` Jarkko Sakkinen
     [not found]   ` <20160105064933.GA5835-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2016-01-06 14:38     ` Jarkko Sakkinen

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.