RE: [PATCH v4 0/4] ocxl: OpenCAPI Cleanup

2019-03-25 Thread Alastair D'Silva
> -Original Message-
> From: Frederic Barrat 
> Sent: Tuesday, 26 March 2019 4:34 AM
> To: Greg Kurz ; Alastair D'Silva 
> Cc: alast...@d-silva.org; Arnd Bergmann ; Greg Kroah-
> Hartman ; linux-ker...@vger.kernel.org;
> Andrew Donnellan ; linuxppc-
> d...@lists.ozlabs.org
> Subject: Re: [PATCH v4 0/4] ocxl: OpenCAPI Cleanup
> 
> 
> 
> Le 25/03/2019 à 17:49, Greg Kurz a écrit :
> > Hi Alastair,
> >
> > I forgot to mention it during v3 but please don't link new version of
> > a patchset to the previous one with --in-reply-to. This is to ensure I
> > can see them in my email client without having to scroll back many
> > days in the past (which likely means a fair number of e-mails on
> > linuxppc-dev).
> 
> 
> I'm also seeing the other series (ocxl refactoring) somehow under the same
> thread. I haven't checked why, and there may be some mail client bug in the
> way, I just mention it in case you see a reason for it in the way you submit
> the patch set.
> 

I probably grabbed the wrong message-id.

-- 
Alastair D'Silva   mob: 0423 762 819
skype: alastair_dsilva msn: alast...@d-silva.org
blog: http://alastair.d-silva.orgTwitter: @EvilDeece



Re: [PATCH v4 0/4] ocxl: OpenCAPI Cleanup

2019-03-25 Thread Frederic Barrat




Le 25/03/2019 à 17:49, Greg Kurz a écrit :

Hi Alastair,

I forgot to mention it during v3 but please don't link new version
of a patchset to the previous one with --in-reply-to. This is to
ensure I can see them in my email client without having to scroll
back many days in the past (which likely means a fair number of
e-mails on linuxppc-dev).



I'm also seeing the other series (ocxl refactoring) somehow under the 
same thread. I haven't checked why, and there may be some mail client 
bug in the way, I just mention it in case you see a reason for it in the 
way you submit the patch set.


  Fred




Cheers,

--
Greg

On Mon, 25 Mar 2019 16:34:51 +1100
"Alastair D'Silva"  wrote:


From: Alastair D'Silva 

Some minor cleanups for the OpenCAPI driver as a prerequisite
for an ocxl driver refactoring to allow the driver core to
be utilised by external drivers.

Changelog:
V4:
   - Drop printf format changes as they omit the format indicator for '0'
V3:
   - Add missed header in 'ocxl: Remove some unused exported symbols'.
 This addresses the introduced sparse warnings
V2:
   - remove intermediate assignment of 'link' var in
 'Rename struct link to ocxl_link'
   - Don't shift definition of ocxl_context_attach in
 'Remove some unused exported symbols'


Alastair D'Silva (4):
   ocxl: Rename struct link to ocxl_link
   ocxl: read_pasid never returns an error, so make it void
   ocxl: Remove superfluous 'extern' from headers
   ocxl: Remove some unused exported symbols

  drivers/misc/ocxl/config.c| 13 ++---
  drivers/misc/ocxl/file.c  |  5 +-
  drivers/misc/ocxl/link.c  | 36 ++---
  drivers/misc/ocxl/ocxl_internal.h | 85 +++
  include/misc/ocxl.h   | 53 ++-
  5 files changed, 91 insertions(+), 101 deletions(-)







Re: [PATCH v4 0/4] ocxl: OpenCAPI Cleanup

2019-03-25 Thread Greg Kurz
Hi Alastair,

I forgot to mention it during v3 but please don't link new version
of a patchset to the previous one with --in-reply-to. This is to
ensure I can see them in my email client without having to scroll
back many days in the past (which likely means a fair number of
e-mails on linuxppc-dev).

Cheers,

--
Greg

On Mon, 25 Mar 2019 16:34:51 +1100
"Alastair D'Silva"  wrote:

> From: Alastair D'Silva 
> 
> Some minor cleanups for the OpenCAPI driver as a prerequisite
> for an ocxl driver refactoring to allow the driver core to
> be utilised by external drivers.
> 
> Changelog:
> V4:
>   - Drop printf format changes as they omit the format indicator for '0'
> V3:
>   - Add missed header in 'ocxl: Remove some unused exported symbols'.
> This addresses the introduced sparse warnings
> V2:
>   - remove intermediate assignment of 'link' var in
> 'Rename struct link to ocxl_link'
>   - Don't shift definition of ocxl_context_attach in
> 'Remove some unused exported symbols'
> 
> 
> Alastair D'Silva (4):
>   ocxl: Rename struct link to ocxl_link
>   ocxl: read_pasid never returns an error, so make it void
>   ocxl: Remove superfluous 'extern' from headers
>   ocxl: Remove some unused exported symbols
> 
>  drivers/misc/ocxl/config.c| 13 ++---
>  drivers/misc/ocxl/file.c  |  5 +-
>  drivers/misc/ocxl/link.c  | 36 ++---
>  drivers/misc/ocxl/ocxl_internal.h | 85 +++
>  include/misc/ocxl.h   | 53 ++-
>  5 files changed, 91 insertions(+), 101 deletions(-)
>