All of lore.kernel.org
 help / color / mirror / Atom feed
From: Heiko Stuebner <heiko-4mtYJXux2i+zQB+pC5nmwQ@public.gmane.org>
To: Randy Li <ayaka-xPW3/0Ywev/iB9QmIjCX8w@public.gmane.org>
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	linux-I+IVW8TIWO2tmTQ+vhA3Yw@public.gmane.org,
	mark.rutland-5wv7dgnIgG8@public.gmane.org,
	alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw@public.gmane.org
Subject: Re: [PATCH] Try to support analog audio codec at firefly reload
Date: Fri, 19 Aug 2016 16:16:00 +0200	[thread overview]
Message-ID: <6373888.a6VB9iqPkj@phil> (raw)
In-Reply-To: <1471293727-26258-1-git-send-email-ayaka-xPW3/0Ywev/iB9QmIjCX8w@public.gmane.org>

Am Dienstag, 16. August 2016, 04:42:06 CEST schrieb Randy Li:
> Hello:
>   This patch could enabled the es8328 codec at firefly reload board,
> but it doesn't not really work. I disable the mute of channel
> Left mix and Right mix in alsamixer, I could hear the noise from
> my headphone, but it never be a music.
>   I just send this patch here to see whether somebody could figure
> out what is wrong, I am really not familiar with sound.

I can't really help here as I have neither the board nor that much sound 
experience, but it's definitly customary to label patches as rfc when they're 
expected to need more work, so make that

[PATCH RFC] ...

for stuff that requires more work before it can actually be merged.


Heiko
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

WARNING: multiple messages have this Message-ID (diff)
From: heiko@sntech.de (Heiko Stuebner)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] Try to support analog audio codec at firefly reload
Date: Fri, 19 Aug 2016 16:16:00 +0200	[thread overview]
Message-ID: <6373888.a6VB9iqPkj@phil> (raw)
In-Reply-To: <1471293727-26258-1-git-send-email-ayaka@soulik.info>

Am Dienstag, 16. August 2016, 04:42:06 CEST schrieb Randy Li:
> Hello:
>   This patch could enabled the es8328 codec at firefly reload board,
> but it doesn't not really work. I disable the mute of channel
> Left mix and Right mix in alsamixer, I could hear the noise from
> my headphone, but it never be a music.
>   I just send this patch here to see whether somebody could figure
> out what is wrong, I am really not familiar with sound.

I can't really help here as I have neither the board nor that much sound 
experience, but it's definitly customary to label patches as rfc when they're 
expected to need more work, so make that

[PATCH RFC] ...

for stuff that requires more work before it can actually be merged.


Heiko

WARNING: multiple messages have this Message-ID (diff)
From: Heiko Stuebner <heiko@sntech.de>
To: Randy Li <ayaka@soulik.info>
Cc: devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-rockchip@lists.infradead.org,
	linux-arm-kernel@lists.infradead.org, linux@armlinux.org.uk,
	mark.rutland@arm.com, alsa-devel@alsa-project.org
Subject: Re: [PATCH] Try to support analog audio codec at firefly reload
Date: Fri, 19 Aug 2016 16:16:00 +0200	[thread overview]
Message-ID: <6373888.a6VB9iqPkj@phil> (raw)
In-Reply-To: <1471293727-26258-1-git-send-email-ayaka@soulik.info>

Am Dienstag, 16. August 2016, 04:42:06 CEST schrieb Randy Li:
> Hello:
>   This patch could enabled the es8328 codec at firefly reload board,
> but it doesn't not really work. I disable the mute of channel
> Left mix and Right mix in alsamixer, I could hear the noise from
> my headphone, but it never be a music.
>   I just send this patch here to see whether somebody could figure
> out what is wrong, I am really not familiar with sound.

I can't really help here as I have neither the board nor that much sound 
experience, but it's definitly customary to label patches as rfc when they're 
expected to need more work, so make that

[PATCH RFC] ...

for stuff that requires more work before it can actually be merged.


Heiko

  parent reply	other threads:[~2016-08-19 14:16 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-15 20:42 [PATCH] Try to support analog audio codec at firefly reload Randy Li
2016-08-15 20:42 ` Randy Li
2016-08-15 20:42 ` [PATCH] ARM: dts: rockchip: add analog audio support for " Randy Li
2016-08-15 20:42   ` Randy Li
     [not found] ` <1471293727-26258-1-git-send-email-ayaka-xPW3/0Ywev/iB9QmIjCX8w@public.gmane.org>
2016-08-19 14:16   ` Heiko Stuebner [this message]
2016-08-19 14:16     ` [PATCH] Try to support analog audio codec at " Heiko Stuebner
2016-08-19 14:16     ` Heiko Stuebner
2016-08-20 18:33   ` John Keeping
2016-08-20 18:33     ` John Keeping
2016-08-20 18:33     ` John Keeping

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=6373888.a6VB9iqPkj@phil \
    --to=heiko-4mtyjxux2i+zqb+pc5nmwq@public.gmane.org \
    --cc=alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw@public.gmane.org \
    --cc=ayaka-xPW3/0Ywev/iB9QmIjCX8w@public.gmane.org \
    --cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-I+IVW8TIWO2tmTQ+vhA3Yw@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=mark.rutland-5wv7dgnIgG8@public.gmane.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.