On (20/01/20 16:14), Andy Shevchenko wrote: > On Mon, Jan 20, 2020 at 10:36:52PM +0900, Sergey Senozhatsky wrote: > > On (20/01/20 19:38), lijiazi wrote: > > > Documentation/core-api/printk-formats.rst | 2 ++ > > > 1 file changed, 2 insertions(+) > > > > > > diff --git a/Documentation/core-api/printk-formats.rst b/Documentation/core-api/printk-formats.rst > > > index 8ebe46b1..9271f20 100644 > > > --- a/Documentation/core-api/printk-formats.rst > > > +++ b/Documentation/core-api/printk-formats.rst > > > @@ -441,6 +441,8 @@ Examples:: > > > d - detached > > > P - Populated > > > B - Populated bus > > > + O - Overlay > > > + F - Overlay free cset > > > > I think these 3 can be just one patch. Patching Documentation and > > vsprintf in separate steps might be a bit inconvenient. > > The new flags addition may be a subject to discuss. That's why I suggested > to split it out. Though the above misses the functionality for these flags. > > Adding a flag counter to the OF header is orthogonal to printf() changes, > it may be there independently. So, it should be a separate patch. Oh, yes! I meant 2 patches, not 3 patches - Documentation and vsprintf(). OF header patch is completely independent. Agreed. -ss
