From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Wed, 9 Sep 2020 16:16:40 +0200 Subject: [PATCH v3 9/9] usb: xhci: convert to readx_poll_sleep_timeout() In-Reply-To: References: <1599462727-2188-1-git-send-email-chunfeng.yun@mediatek.com> <1599462727-2188-9-git-send-email-chunfeng.yun@mediatek.com> <34370221-246a-02e1-5ec4-f5159dbb62ae@denx.de> <633bff84-9e2d-65dd-2af2-d34d7396e2a2@denx.de> Message-ID: <215d2f13-bc06-22d2-4988-d359ef53b6dc@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 9/9/20 4:05 PM, Bin Meng wrote: > On Wed, Sep 9, 2020 at 10:02 PM Marek Vasut wrote: >> >> On 9/9/20 3:46 PM, Bin Meng wrote: >>> On Wed, Sep 9, 2020 at 7:05 PM Marek Vasut wrote: >>>> >>>> On 9/9/20 3:55 AM, Bin Meng wrote: >>>>> Hi Marek, >>>> >>>> Hi, >>>> >>>>> On Wed, Sep 9, 2020 at 12:15 AM Marek Vasut wrote: >>>>>> >>>>>> On 9/8/20 5:45 PM, Bin Meng wrote: >>>>>>> Hi Marek, >>>>>> >>>>>> Hi, >>>>>> >>>>>>> On Tue, Sep 8, 2020 at 7:13 PM Marek Vasut wrote: >>>>>>>> >>>>>>>> On 9/8/20 3:44 AM, Bin Meng wrote: >>>>>>>>> On Mon, Sep 7, 2020 at 3:14 PM Chunfeng Yun wrote: >>>>>>>>>> >>>>>>>>>> Use readx_poll_sleep_timeout() to poll the register status >>>>>>>>>> >>>>>>>>>> Signed-off-by: Chunfeng Yun >>>>>>>>>> --- >>>>>>>>>> v3: no changes >>>>>>>>>> >>>>>>>>>> v2: fix typo of title suggested by Frank >>>>>>>>>> --- >>>>>>>>>> drivers/usb/host/xhci.c | 25 +++++++++++-------------- >>>>>>>>>> 1 file changed, 11 insertions(+), 14 deletions(-) >>>>>>>>>> >>>>>>>>> >>>>>>>>> Reviewed-by: Bin Meng >>>>>>>> >>>>>>>> Thanks. So now I have half of a RESEND patchset with RB tags, and >>>>>>>> another half of just RB tags without patchset. Can someone of you please >>>>>>>> collect the tags and resend the patchset once more with the collected >>>>>>>> tags, so it's all in one place ? >>>>>>>> >>>>>>> >>>>>>> I believe the latest version has all the RB tags. >>>>>> >>>>>> Thank you, I cannot apply a patchset which only has "Re:" in my mailbox, >>>>>> because the original patches are just not there. >>>>> >>>>> You can use patchwork to help the custodian work. I found it very >>>>> helpful to download the patches from patchwork and apply. >>>> >>>> Can you also comment on patches via patchwork ? >>>> >>>> I mean, really, can we please just stick to the usual workflow ? That's >>>> all I am asking for , nothing else, nothing new. >>> >>> I don't understand the question about "usual workflow". You asked "I >>> cannot apply a patchset which only has "Re:" in my mailbox" and I >>> recommended patchwork which will satisfy the requirement of what you >>> asked. >> >> Right, I can, but if there is more I want to say about the patch, I >> cannot do it with patchwork. Then I need the patch, which I don't have. >> Hence my remark about the usual email-based workflow. > > Yes, that's true. If we need to post comments about patches, we will > have to use email. The problem here is that for whatever reason, I didn't receive the original patches, Frank had to resend them, then I got RBs on some of each and it was a mess. I'd just like to get back to the usual. I think we agree, don't we ? Anyway, thanks for the RBs, that really helped.