dri-devel.lists.freedesktop.org archive mirror
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: dri-devel@lists.freedesktop.org
Cc: Jernej Skrabec <jernej.skrabec@siol.net>,
	alsa-devel@alsa-project.org, Jonas Karlman <jonas@kwiboo.se>,
	Neil Armstrong <narmstrong@baylibre.com>,
	Russell King <linux@armlinux.org.uk>,
	Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
Subject: Re: [PATCH for-5.6 0/2] drm/bridge: dw-hdmi: PCM API updates
Date: Thu, 09 Jan 2020 10:10:09 +0100	[thread overview]
Message-ID: <s5h8smhm1vy.wl-tiwai@suse.de> (raw)
In-Reply-To: <20191210154536.29819-1-tiwai@suse.de>

On Tue, 10 Dec 2019 16:45:34 +0100,
Takashi Iwai wrote:
> 
> Hi,
> 
> this is a patch set for updating ALSA PCM API usages in dw-hdmi
> driver.  I already tried to "fix" the driver some time ago but it was
> utterly wrong.  So this is a combination of the revised patch and
> another cleanup patch.
> 
> The first one is to change the buffer allocation mechanism in the
> driver to the manual allocation of the h/w buffer and the automatic
> allocation of PCM stream buffers via the new standard API.  The
> significant change is that size of the h/w buffer isn't no longer
> controlled via ALSA preallocation proc file but rather via the new
> module option (if any).
> 
> The second one is a oneliner patch just to remove the superfluous PCM
> ops.
> 
> Both need the ALSA PCM core changes in 5.5-rc1, so please apply them
> on top of 5.5-rc1 or later.  Or, just let me know if I should apply
> them through sound git tree.
> 
> 
> thanks,
> 
> Takashi
> 
> ===
> 
> Takashi Iwai (2):
>   drm/bridge: dw-hdmi: Follow the standard ALSA memalloc way
>   drm/bridge: dw-hdmi: Drop superfluous ioctl PCM ops

Any chance for reviewing these patches?

Since this driver is the only one who is still using the old ALSA
vmalloc API, I'd like to change it and drop the old API in 5.6.


thanks,

Takashi
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

  parent reply	other threads:[~2020-01-09  9:10 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-12-10 15:45 [PATCH for-5.6 0/2] drm/bridge: dw-hdmi: PCM API updates Takashi Iwai
2019-12-10 15:45 ` [PATCH for-5.6 1/2] drm/bridge: dw-hdmi: Follow the standard ALSA memalloc way Takashi Iwai
2019-12-10 15:45 ` [PATCH for-5.6 2/2] drm/bridge: dw-hdmi: Drop superfluous ioctl PCM ops Takashi Iwai
2020-01-09  9:10 ` Takashi Iwai [this message]
2020-01-09 10:18   ` [PATCH for-5.6 0/2] drm/bridge: dw-hdmi: PCM API updates Neil Armstrong
2020-01-09 10:25   ` Russell King - ARM Linux admin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=s5h8smhm1vy.wl-tiwai@suse.de \
    --to=tiwai@suse.de \
    --cc=Laurent.pinchart@ideasonboard.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=jernej.skrabec@siol.net \
    --cc=jonas@kwiboo.se \
    --cc=linux@armlinux.org.uk \
    --cc=narmstrong@baylibre.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).