| Message ID | 1539849103-11726-1-git-send-email-pmorel@linux.ibm.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | vfio: ccw: VFIO CCW cleanup part1 | expand |
On 10/18/2018 03:51 AM, Pierre Morel wrote: > The first patch of this serie is essentially cosmetic > to clarify the code. > > The second patch is a bug correction. > > > > Pierre Morel (2): > vfio: ccw: Merge BUSY and BOXED states > vfio: ccw: Register mediated device once all structures are > initialized > > drivers/s390/cio/vfio_ccw_drv.c | 8 ++++---- > drivers/s390/cio/vfio_ccw_fsm.c | 7 +------ > drivers/s390/cio/vfio_ccw_private.h | 1 - > 3 files changed, 5 insertions(+), 11 deletions(-) > You said in the changelog (which gets omitted in reply) that you added my r-b, but I don't see it. For the record, I'm still good with these. :-) - Eric
Le 10/18/18 à 21:48, Eric Farman a écrit : > > > On 10/18/2018 03:51 AM, Pierre Morel wrote: >> The first patch of this serie is essentially cosmetic >> to clarify the code. >> >> The second patch is a bug correction. >> >> >> >> Pierre Morel (2): >> vfio: ccw: Merge BUSY and BOXED states >> vfio: ccw: Register mediated device once all structures are >> initialized >> >> drivers/s390/cio/vfio_ccw_drv.c | 8 ++++---- >> drivers/s390/cio/vfio_ccw_fsm.c | 7 +------ >> drivers/s390/cio/vfio_ccw_private.h | 1 - >> 3 files changed, 5 insertions(+), 11 deletions(-) >> > > You said in the changelog (which gets omitted in reply) that you added > my r-b, but I don't see it. For the record, I'm still good with these. > :-) > > - Eric > > Oh, indeed, sorry. I wont forget next time. Pierre
On Fri, 19 Oct 2018 11:36:37 +0200 Pierre Morel <pmorel@ns360863.ovh.net> wrote: > Le 10/18/18 à 21:48, Eric Farman a écrit : > > > > > > On 10/18/2018 03:51 AM, Pierre Morel wrote: > >> The first patch of this serie is essentially cosmetic > >> to clarify the code. > >> > >> The second patch is a bug correction. > >> > >> > >> > >> Pierre Morel (2): > >> vfio: ccw: Merge BUSY and BOXED states > >> vfio: ccw: Register mediated device once all structures are > >> initialized > >> > >> drivers/s390/cio/vfio_ccw_drv.c | 8 ++++---- > >> drivers/s390/cio/vfio_ccw_fsm.c | 7 +------ > >> drivers/s390/cio/vfio_ccw_private.h | 1 - > >> 3 files changed, 5 insertions(+), 11 deletions(-) > >> > > > > You said in the changelog (which gets omitted in reply) that you added > > my r-b, but I don't see it. For the record, I'm still good with these. > > :-) > > > > - Eric > > > > > Oh, indeed, sorry. > I wont forget next time. > > Pierre In this case, I can add the R-b if I apply patch 2 (which I think I'll do).
The first patch of this serie is essentially cosmetic to clarify the code. The second patch is a bug correction. Pierre Morel (2): vfio: ccw: Merge BUSY and BOXED states vfio: ccw: Register mediated device once all structures are initialized drivers/s390/cio/vfio_ccw_drv.c | 8 ++++---- drivers/s390/cio/vfio_ccw_fsm.c | 7 +------ drivers/s390/cio/vfio_ccw_private.h | 1 - 3 files changed, 5 insertions(+), 11 deletions(-)