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

[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" <alastair@xxxxxxxxxxx> wrote:
>
> > From: Alastair D'Silva <alastair@xxxxxxxxxxx>
> >
> > 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(-)