Message ID | 5751D849.4080506@canonical.com |
---|---|
State | New |
Headers | show |
On 03.06.2016 21:19, Tim Gardner wrote: > The fix commit 2bc79ffcbb817873cc43d63118008ab75181b73d ('cxl: Poll for > outstanding IRQs when detaching a context') requires a bunch of > scaffolding. Since we already have a mix of interdependent cxl and > cxlflash patches I decided to just all cxl/cxlflash patches up to and > including the fix commit. Regression testing is straightforward since > this driver only exists on one platform and architecture. > > See attached pull request. Dunno, using the pull request yields more that just cxl changes for me (as does the diffstat of the pull request). I might be doing something wrong... -Stefan > > rtg > > >
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA512 On 06/08/2016 08:35 AM, Stefan Bader wrote: > On 03.06.2016 21:19, Tim Gardner wrote: >> The fix commit 2bc79ffcbb817873cc43d63118008ab75181b73d ('cxl: >> Poll for outstanding IRQs when detaching a context') requires a >> bunch of scaffolding. Since we already have a mix of >> interdependent cxl and cxlflash patches I decided to just all >> cxl/cxlflash patches up to and including the fix commit. >> Regression testing is straightforward since this driver only >> exists on one platform and architecture. >> >> See attached pull request. > > Dunno, using the pull request yields more that just cxl changes for > me (as does the diffstat of the pull request). I might be doing > something wrong... > Details ? - -- Tim Gardner tim.gardner@canonical.com -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.22 (GNU/Linux) iQIcBAEBCgAGBQJXWDEgAAoJED12yEX6FEfKmacP/314g3DNrF9/4/QymLLgrmzu vASp8rlPCglB2C6RA6xvtZTkCuCLZgSmWfygdYfGrEoRnIvYp8PWDYLTkDTv+5Zl ERRe2yNpuWYO8jvKWJvg5QBf++4uFCqagZ2o2e6zSuUy8jq/4eas72n6qyOtFeNF qLdUznhZJa8Y6QEGJJ8uAqZ5I2D1WK518LEgysENtODrbl0cDEFeq0L1kHEh4Kpd oi0GVl5R93sJf0LNhW8RBfQe1+dKa/rcX6MsnZDpV1w4X/Y5JDk/5/Dv+P6DRMJO YihGUMb/qgDI42KKPfww2SuS0ayNwJPz42UwrU43Bjwvb2FnTuVDDBOwEM6rv2fY B6WAiKzCL1g7RY9zXDRCJR13X/fdPB3mSb9oS8MuFnNsbME1PfjmjtpKW4AgTkvv c/RyELecR3f0C24bYLonfux237kUWm6SGqi7PdCS48ZhRwARx5sTk1vbxIFaXNni PhEZLTrhkif9mDNjE1LP8OPlKqy4Um6nrY+fJJ+8GSLcil1ehir3yT3IjMw8qD0K MOAvpYWNPBJbx0fR1DMHDKnYBQkrgv8Abu1M8Uy931jjZzMVxfrbeF+rFQvRJTDU DcaWNpeVcjRtfUrQh9JI7HD2N5/NLy9MsQtdR7R96yrkQR8Z2GJFFNcYuaH+NprN iJkpsiah6Z6dgM/XDWgY =pc6M -----END PGP SIGNATURE-----
On 08.06.2016 16:52, Tim Gardner wrote: > On 06/08/2016 08:35 AM, Stefan Bader wrote: >> On 03.06.2016 21:19, Tim Gardner wrote: >>> The fix commit 2bc79ffcbb817873cc43d63118008ab75181b73d ('cxl: >>> Poll for outstanding IRQs when detaching a context') requires a >>> bunch of scaffolding. Since we already have a mix of >>> interdependent cxl and cxlflash patches I decided to just all >>> cxl/cxlflash patches up to and including the fix commit. >>> Regression testing is straightforward since this driver only >>> exists on one platform and architecture. >>> >>> See attached pull request. > >> Dunno, using the pull request yields more that just cxl changes for >> me (as does the diffstat of the pull request). I might be doing >> something wrong... > > > Details ? drivers/scsi/mesh.c | 2 +- drivers/usb/core/hcd-pci.c | 1 - drivers/video/fbdev/aty/aty128fb.c | 1 - drivers/video/fbdev/aty/radeon_base.c | 1 - drivers/video/fbdev/imsttfb.c | 1 - drivers/video/fbdev/matrox/matroxfb_base.h | 1 - drivers/video/fbdev/offb.c | 4 - include/misc/cxl.h | 8 +- include/uapi/misc/cxl.h | 24 + sound/ppc/pmac.c | 1 - > >
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA512 On 06/08/2016 09:03 AM, Stefan Bader wrote: > On 08.06.2016 16:52, Tim Gardner wrote: >> On 06/08/2016 08:35 AM, Stefan Bader wrote: >>> On 03.06.2016 21:19, Tim Gardner wrote: >>>> The fix commit 2bc79ffcbb817873cc43d63118008ab75181b73d >>>> ('cxl: Poll for outstanding IRQs when detaching a context') >>>> requires a bunch of scaffolding. Since we already have a mix >>>> of interdependent cxl and cxlflash patches I decided to just >>>> all cxl/cxlflash patches up to and including the fix commit. >>>> Regression testing is straightforward since this driver only >>>> exists on one platform and architecture. >>>> >>>> See attached pull request. >> >>> Dunno, using the pull request yields more that just cxl changes >>> for me (as does the diffstat of the pull request). I might be >>> doing something wrong... >> >> >> Details ? > > drivers/scsi/mesh.c | 2 +- > drivers/usb/core/hcd-pci.c | 1 - > drivers/video/fbdev/aty/aty128fb.c | 1 - > drivers/video/fbdev/aty/radeon_base.c | 1 - > drivers/video/fbdev/imsttfb.c | 1 - > drivers/video/fbdev/matrox/matroxfb_base.h | 1 - > drivers/video/fbdev/offb.c | 4 - > include/misc/cxl.h | 8 +- > include/uapi/misc/cxl.h | 24 + > sound/ppc/pmac.c | 1 - >> >> > > This isn't really a merge request. The patches up for review are from 41b48dd5672ebaf077cc122af6b921ae81945753..7afdb0a5ea5a068eb9c91b110d276 115d049761b as indicated in the pull request. The master or master-next branch that you have has clearly moved since this pull request was origi nally emitted. rtg - -- Tim Gardner tim.gardner@canonical.com -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.22 (GNU/Linux) iQIcBAEBCgAGBQJXWDcRAAoJED12yEX6FEfK7nUP/0D9t5q07qi3+0GrgGLOAOGl USW2GB7aJNBBtwe+dJPp6KQ+GSdriMDNreM2JKmz4ZcDOBU26OSe2yV6WT99vTRN egISh91TBHX6fKp+cFsK6M5HUS8vClHOUi9F/ZpPkNloLLWGM4g8jTg9PsMphqAk SEgjV2f+XoxWdhYpQiHc8FvDHqH28bSZaxw6N/GQz0vZNlDwgxvn5LGVmrmET6F7 kPXvX4wgrCL8ikLkuYjTFXubFgnFp8prB8ry1oI61Xm9C/J847qg1uRfbEd//yGp Os85vFPYsdQiMfwOwGZso5Ppj0QJQiaoh4ITmy5xAmlytc84oK6Ne+AeOS8xYqNI YfqvffwjSN6RCBmdPsvSIpqlUDNnwyphOEH1FFvCnHWQi565Rx510Dke0As5LlGR CAew9CR35cflhcPlnS/do/pwwlDjRKvyxVOuc3wBJcSqyyx5wHHHtwQtGDX6OhUc G5F85LDgtPybXKPUpbX3V+LxpxhsQYqbV8Tka6GyiOPmFfAbjH3PfQZqGqb8iNQO Xek4LjBBWA0fOQ2X1B2eF6b8Pd+2vbgQZp5WUFqr8HZ0C2X+KP3tbWPycBxmQZN2 cMgs3uIiWWEYwbodbezRiZbbwI2QtAwVl/Hq5DP7DVA5lBYVVo61/JDoR/h0bulL wFDwtYvZLJ8Z0N4iHtmd =0Idl -----END PGP SIGNATURE-----