<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Nov 8, 2017 at 11:18 AM, Deucher, Alexander <span dir="ltr"><<a href="mailto:Alexander.Deucher@amd.com" target="_blank">Alexander.Deucher@amd.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">> -----Original Message-----<br>
> From: Mark Brown [mailto:<a href="mailto:broonie@kernel.org">broonie@kernel.org</a>]<br>
> Sent: Wednesday, November 08, 2017 1:48 PM<br>
> To: Alex Deucher<br>
> Cc: amd-gfx list; <a href="mailto:alsa-devel@alsa-project.org">alsa-devel@alsa-project.org</a>; Maling list - DRI developers;<br>
> Mukunda, Vijendar; Liam Girdwood; Takashi Iwai; Guenter Roeck; Deucher,<br>
> Alexander<br>
> Subject: Re: [PATCH] ASoC: amd: fix report accurate hw_ptr during dma<br>
><br>
> On Wed, Nov 08, 2017 at 01:40:32PM -0500, Alex Deucher wrote:<br>
> > On Wed, Nov 8, 2017 at 1:22 PM, Mark Brown <<a href="mailto:broonie@kernel.org">broonie@kernel.org</a>><br>
> wrote:<br>
><br>
> > > Like I said in reply to your other mail please don't resubmit already<br>
> > > applied patches.  The current tip of my topic/amd branch appears to be<br>
> > > this very patch, if there's anything needs changing please send an<br>
> > > incremental patch.<br>
><br>
> > I'm not seeing this one in your tree either.  This is just a resend of<br>
> > Guenter's patch from an hour ago with the chromium stuff removed.<br>
> > Maybe you already applied it in the interim?<br>
><br>
> Is this different to "ASoC: amd: Report accurate hw_ptr during dma"<br>
> which was applied at 16:07?<br>
<br>
</div></div>Yes, this is a fix for that patch.  It fixes a 64 bit division that wasn't properly handled.<br>
<br></blockquote><div><br></div><div>In that case, the subject should reflect the problem fixed, the description should describe the problem, and there should be a Fixes: tag pointing to the problematic patch.</div><div><br></div><div>Sorry, I was not aware that the problematic patch is already pending upstream, or I would have submitted a proper patch upstream myself.</div><div><br></div><div>Guenter</div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Alex<br>
<br>
</blockquote></div><br></div></div>