On Fri, Jul 21, 2023 at 11:25:17AM -0400, Hugo Villeneuve wrote: > On Thu, 20 Jul 2023 21:38:21 +0200 > Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> wrote: > > > On Wed, Jul 19, 2023 at 09:14:23PM +0200, Greg KH wrote: > > > On Wed, Jul 19, 2023 at 02:40:48PM -0400, Hugo Villeneuve wrote: > > > > On Tue, 20 Jun 2023 12:16:45 -0400 > > > > Hugo Villeneuve <hugo@xxxxxxxxxxx> wrote: > > > > > > > > > On Tue, 20 Jun 2023 18:45:51 +0300 > > > > > Andy Shevchenko <andy.shevchenko@xxxxxxxxx> wrote: > > > > > > > > > > > On Tue, Jun 20, 2023 at 6:42 PM Hugo Villeneuve <hugo@xxxxxxxxxxx> wrote: > > > > > > > On Tue, 20 Jun 2023 18:35:48 +0300 > > > > > > > Andy Shevchenko <andy.shevchenko@xxxxxxxxx> wrote: > > > > > > > > On Tue, Jun 20, 2023 at 6:33 PM Hugo Villeneuve <hugo@xxxxxxxxxxx> wrote: > > > > > > > > > On Tue, 20 Jun 2023 18:18:12 +0300 > > > > > > > > > Andy Shevchenko <andy.shevchenko@xxxxxxxxx> wrote: > > > > > > > > > > On Tue, Jun 20, 2023 at 5:08 PM Hugo Villeneuve <hugo@xxxxxxxxxxx> wrote: > > > > > > > > > > > On Sun, 4 Jun 2023 22:31:04 +0300 > > > > > > > > > > > Andy Shevchenko <andy.shevchenko@xxxxxxxxx> wrote: > > > > > > > > > > > > ... > > > > > > > > > > > > > > > > > did you have a chance to look at V8 (sent two weks ago) which fixed all > > > > > > > > > > > of what we discussed? > > > > > > > > > > > > > > > > > > > > The patch 6 already has my tag, anything specific you want me to do? > > > > > > > > > > > > > > > > > > Hi Andy, > > > > > > > > > I forgot to remove your "Reviewed-by: Andy..." tag before sending V8 > > > > > > > > > since there were some changes involved in patch 6 and I wanted you to > > > > > > > > > review them. Can you confirm if the changes are correct? > > > > > > > > > > > > > > > > > > I also added a new patch "remove obsolete out_thread label". It has no > > > > > > > > > real impact on the code generation itself, but maybe you can review and > > > > > > > > > confirm if tags are ok or not, based on commit message and also > > > > > > > > > additional commit message. > > > > > > > > > > > > > > > > Both are fine to me. > > > > > > > > > > > > > > Hi, > > > > > > > Ok, thank you for reviewing this. > > > > > > > > > > > > > > I guess now we are good to go with this series if the stable tags and > > > > > > > patches order are good after Greg's review? > > > > > > > > > > > > Taking into account that we are at rc7, and even with Fixes tags in > > > > > > your series I think Greg might take this after v6.5-0rc1 is out. It's > > > > > > up to him how to proceed with that. Note, he usually has thousands of > > > > > > patches in backlog, you might need to respin it after the above > > > > > > mentioned rc1. > > > > > > > > > > Ok, understood. > > > > > > > > > > Let's wait then. > > > > > > > > Hi Andy/Greg, > > > > we are now at v6.5-rc2 and I still do not see any of our patches in > > > > linus or gregkh_tty repos. > > > > > > > > Is there something missing from my part (or someone else) to go forward > > > > with integrating these patches (v8) for v6.5? > > > > > > My queue is huge right now, please be patient, I want to have them all > > > handled by the end of next week... > > > > > > You can always help out by reviewing other patches on the mailing list > > > to reduce my review load. > > > > Wait, no, this series was superseeded by v8, and in there you said you > > were going to send a new series. So please, fix it up and send the > > updated version of the series, this one isn't going to be applied for > > obvious reasons. > > Hi Greg, > I never said that I would resend another update for this current > serie (unless of course if it was to address a new comment). Re-reading > that email made me realise that it was maybe not perfectly clear the > way I wrote it. > > What I said was that, once V8 was finally applied and > incorporated in the kernel, then I would send a completely new and > different serie to address issues/concerns/improvements/suggestions > noted during the review of this serie (example: conversion of bindings > to YAML and improve DTS node names, etc). We already agreed with some > maintainers (ex: Conor Dooley) that it was reasonnable to do so. > > That is why I asked Andy if we were good to go with V8 and he > confirmed that, and that it was now up to you to integrate it if your > review was satisfactory. > > Hope this clears things and we can integrate it soon. I don't have any of your series in my review queue at all, so please resend them. thanks, greg k-h
