Re: [PATCH 3/3] nvme: fix eui_show() print format

2017-11-07 Thread Javier González
> On 7 Nov 2017, at 17.28, Christoph Hellwig wrote: > > On Sat, Nov 04, 2017 at 12:22:20PM +0100, Javier González wrote: >>> Perhaps this should just become >>> >>>%8ph >>> >>> without D >> >> That would be ok with me. > > Can you resend a patch for that? > Sure. I’ll

Re: [PATCH 3/3] nvme: fix eui_show() print format

2017-11-07 Thread Javier González
> On 7 Nov 2017, at 17.28, Christoph Hellwig wrote: > > On Sat, Nov 04, 2017 at 12:22:20PM +0100, Javier González wrote: >>> Perhaps this should just become >>> >>>%8ph >>> >>> without D >> >> That would be ok with me. > > Can you resend a patch for that? > Sure. I’ll send it tomorrow

Re: [PATCH 3/3] nvme: fix eui_show() print format

2017-11-07 Thread Christoph Hellwig
On Sat, Nov 04, 2017 at 12:22:20PM +0100, Javier González wrote: > > Perhaps this should just become > > > > %8ph > > > > without D > > That would be ok with me. Can you resend a patch for that?

Re: [PATCH 3/3] nvme: fix eui_show() print format

2017-11-07 Thread Christoph Hellwig
On Sat, Nov 04, 2017 at 12:22:20PM +0100, Javier González wrote: > > Perhaps this should just become > > > > %8ph > > > > without D > > That would be ok with me. Can you resend a patch for that?

Re: [PATCH 3/3] nvme: fix eui_show() print format

2017-11-04 Thread Javier González
> On 3 Nov 2017, at 16.16, Joe Perches wrote: > > On Fri, 2017-11-03 at 13:55 +0100, Christoph Hellwig wrote: >> On Fri, Nov 03, 2017 at 11:02:50AM +0100, Javier González wrote: >>> Signed-off-by: Javier González > [] >>> diff --git

Re: [PATCH 3/3] nvme: fix eui_show() print format

2017-11-04 Thread Javier González
> On 3 Nov 2017, at 16.16, Joe Perches wrote: > > On Fri, 2017-11-03 at 13:55 +0100, Christoph Hellwig wrote: >> On Fri, Nov 03, 2017 at 11:02:50AM +0100, Javier González wrote: >>> Signed-off-by: Javier González > [] >>> diff --git a/drivers/nvme/host/core.c b/drivers/nvme/host/core.c > [] >>>

Re: [PATCH 3/3] nvme: fix eui_show() print format

2017-11-03 Thread Joe Perches
On Fri, 2017-11-03 at 13:55 +0100, Christoph Hellwig wrote: > On Fri, Nov 03, 2017 at 11:02:50AM +0100, Javier González wrote: > > Signed-off-by: Javier González [] > > diff --git a/drivers/nvme/host/core.c b/drivers/nvme/host/core.c [] > > @@ -2108,7 +2108,7 @@ static

Re: [PATCH 3/3] nvme: fix eui_show() print format

2017-11-03 Thread Joe Perches
On Fri, 2017-11-03 at 13:55 +0100, Christoph Hellwig wrote: > On Fri, Nov 03, 2017 at 11:02:50AM +0100, Javier González wrote: > > Signed-off-by: Javier González [] > > diff --git a/drivers/nvme/host/core.c b/drivers/nvme/host/core.c [] > > @@ -2108,7 +2108,7 @@ static ssize_t eui_show(struct

Re: [PATCH 3/3] nvme: fix eui_show() print format

2017-11-03 Thread Keith Busch
On Fri, Nov 03, 2017 at 01:55:16PM +0100, Christoph Hellwig wrote: > On Fri, Nov 03, 2017 at 11:02:50AM +0100, Javier González wrote: > > Signed-off-by: Javier González > > --- > > drivers/nvme/host/core.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > >

Re: [PATCH 3/3] nvme: fix eui_show() print format

2017-11-03 Thread Keith Busch
On Fri, Nov 03, 2017 at 01:55:16PM +0100, Christoph Hellwig wrote: > On Fri, Nov 03, 2017 at 11:02:50AM +0100, Javier González wrote: > > Signed-off-by: Javier González > > --- > > drivers/nvme/host/core.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git

Re: [PATCH 3/3] nvme: fix eui_show() print format

2017-11-03 Thread Christoph Hellwig
On Fri, Nov 03, 2017 at 11:02:50AM +0100, Javier González wrote: > Signed-off-by: Javier González > --- > drivers/nvme/host/core.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/nvme/host/core.c b/drivers/nvme/host/core.c > index

Re: [PATCH 3/3] nvme: fix eui_show() print format

2017-11-03 Thread Christoph Hellwig
On Fri, Nov 03, 2017 at 11:02:50AM +0100, Javier González wrote: > Signed-off-by: Javier González > --- > drivers/nvme/host/core.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/nvme/host/core.c b/drivers/nvme/host/core.c > index ae8ab0a1ef0d..f05c81774abf

[PATCH 3/3] nvme: fix eui_show() print format

2017-11-03 Thread Javier González
Signed-off-by: Javier González --- drivers/nvme/host/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/nvme/host/core.c b/drivers/nvme/host/core.c index ae8ab0a1ef0d..f05c81774abf 100644 --- a/drivers/nvme/host/core.c +++

[PATCH 3/3] nvme: fix eui_show() print format

2017-11-03 Thread Javier González
Signed-off-by: Javier González --- drivers/nvme/host/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/nvme/host/core.c b/drivers/nvme/host/core.c index ae8ab0a1ef0d..f05c81774abf 100644 --- a/drivers/nvme/host/core.c +++ b/drivers/nvme/host/core.c @@ -2108,7