[PATCH] ASoC: amd: fix report accurate hw_ptr during dma

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Nov 8, 2017 at 11:18 AM, Deucher, Alexander <
Alexander.Deucher at amd.com> wrote:

> > -----Original Message-----
> > From: Mark Brown [mailto:broonie at kernel.org]
> > Sent: Wednesday, November 08, 2017 1:48 PM
> > To: Alex Deucher
> > Cc: amd-gfx list; alsa-devel at alsa-project.org; Maling list - DRI
> developers;
> > Mukunda, Vijendar; Liam Girdwood; Takashi Iwai; Guenter Roeck; Deucher,
> > Alexander
> > Subject: Re: [PATCH] ASoC: amd: fix report accurate hw_ptr during dma
> >
> > On Wed, Nov 08, 2017 at 01:40:32PM -0500, Alex Deucher wrote:
> > > On Wed, Nov 8, 2017 at 1:22 PM, Mark Brown <broonie at kernel.org>
> > wrote:
> >
> > > > Like I said in reply to your other mail please don't resubmit already
> > > > applied patches.  The current tip of my topic/amd branch appears to
> be
> > > > this very patch, if there's anything needs changing please send an
> > > > incremental patch.
> >
> > > I'm not seeing this one in your tree either.  This is just a resend of
> > > Guenter's patch from an hour ago with the chromium stuff removed.
> > > Maybe you already applied it in the interim?
> >
> > Is this different to "ASoC: amd: Report accurate hw_ptr during dma"
> > which was applied at 16:07?
>
> Yes, this is a fix for that patch.  It fixes a 64 bit division that wasn't
> properly handled.
>
>
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.

Sorry, I was not aware that the problematic patch is already pending
upstream, or I would have submitted a proper patch upstream myself.

Guenter


> Alex
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20171108/f4521876/attachment-0001.html>


[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux