Re: [PATCH 23/23] uuid: remove uuid_be

From: Andy Shevchenko
Date: Mon May 22 2017 - 14:59:48 EST


On Thu, 2017-05-18 at 10:56 +0200, Christoph Hellwig wrote:
> On Thu, May 18, 2017 at 10:57:24AM +0300, Amir Goldstein wrote:
> > I reviewed the entire series. You may add
> > Reviewed-by: Amir Goldstein <amir73il@xxxxxxxxx>
> >
> > to any of the patches as you see fit.
>
> Thanks, done!

Similar from my side, FWIW:
Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>

>
> > Now let's talk about how this is going to be merged.
> > Do you intend to send Linus a pull request?
>
> Yes, that might be the best idea.ÂÂI'm also contemplating listing
> me plus anyone volunteering (you?, Andy) as maintaines for the uuid
> code.

I would agree to be a designated reviewer for now (too many stuff to
follow as a (co-)maintainer).

> > The reason I am asking is because this last removal patch should
> > probably
> > be applied near the end of the merge window (?).
> > Because maintainers cannot apply patches with code that uses the new
> > uuid_t to linux-next branches and we don't want linux-next build to
> > fail
> > with new code that uses uuid_be...
>
> Yeah, we can probably defer that one for now.

I would also like to append some of patches from my side, though they
are WIP.

--
Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
Intel Finland Oy