April 2019 - Alsa-devel - mailman.alsa-project.org

[alsa-devel] [PATCH V6 0/3] Support more sample rate in asrc
by S.j. Wang 22 Apr '19

22 Apr '19

Support more sample rate in asrc Shengjiu Wang (3): ASoC: fsl_asrc: Fix the issue about unsupported rate ASoC: fsl_asrc: replace the process_option table with function ASoC: fsl_asrc: Unify the supported input and output rate Changes in v6 - add acked-by - fixed minor issue according to comments in v5 Changes in v5 - fix the [1/24, 8] - move fsl_asrc_sel_proc before setting Changes in v4 - add patch to Fix the [8kHz, 30kHz] open set issue. Changes in v3 - remove FSL_ASRC_RATES - refine fsl_asrc_sel_proc according to comments Changes in v2 - add more comments in code - add commit "Unify the supported input and output rate" sound/soc/fsl/fsl_asrc.c | 105 ++++++++++++++++++++++++++++++++--------------- 1 file changed, 71 insertions(+), 34 deletions(-) -- 1.9.1

1 3

Re: [alsa-devel] [PATCH V5 2/3] ASoC: fsl_asrc: replace the process_option table with function
by S.j. Wang 22 Apr '19

22 Apr '19

Hi > > > On Mon, Apr 22, 2019 at 03:15:34AM +0000, S.j. Wang wrote: > > Hi > > > > > > > > > > > On Mon, Apr 22, 2019 at 02:32:35AM +0000, S.j. Wang wrote: > > > > When we want to support more sample rate, for example > 12kHz/24kHz > > > we > > > > need update the process_option table, if we want to support more > > > > sample rate next time, the table need to be updated again. which > > > > is not flexible. > > > > > > > > We got a function fsl_asrc_sel_proc to replace the table, which > > > > can give the pre-processing and post-processing options according > > > > to the sample rate. > > > > > > > > Signed-off-by: Shengjiu Wang <shengjiu.wang(a)nxp.com> > > > > > > A couple of more small comments. > > > > > > And please add this when you resend: > > > Acked-by: Nicolin Chen <nicoleotsuka(a)gmail.com> > > > > > > > + * Unsupport cases: Tsout > 8.125 * Tsin, Tsout > 16.125 * Tsin > > > > > > Since we have a ratio validation somewhere else, it's okay to drop > > > this line - > > > - it may confuse people since the function no longer checks these > > > unsupported cases. > > > > I add this for may be in the future we forget the limitation. Just for a > reminder. > > Okay. Let's use something more practical like: > > +* Make sure to exclude following unsupported cases before calling the > function: > +* 1) outrate > 8.125 * inrate > +* 2) outrate > 16.125 * inrate Ok.

1 0

Re: [alsa-devel] [PATCH V5 2/3] ASoC: fsl_asrc: replace the process_option table with function
by S.j. Wang 22 Apr '19

22 Apr '19

Hi > > > On Mon, Apr 22, 2019 at 02:32:35AM +0000, S.j. Wang wrote: > > When we want to support more sample rate, for example 12kHz/24kHz > we > > need update the process_option table, if we want to support more > > sample rate next time, the table need to be updated again. which is > > not flexible. > > > > We got a function fsl_asrc_sel_proc to replace the table, which can > > give the pre-processing and post-processing options according to the > > sample rate. > > > > Signed-off-by: Shengjiu Wang <shengjiu.wang(a)nxp.com> > > A couple of more small comments. > > And please add this when you resend: > Acked-by: Nicolin Chen <nicoleotsuka(a)gmail.com> > > > + * Unsupport cases: Tsout > 8.125 * Tsin, Tsout > 16.125 * Tsin > > Since we have a ratio validation somewhere else, it's okay to drop this line - > - it may confuse people since the function no longer checks these > unsupported cases. I add this for may be in the future we forget the limitation. Just for a reminder. Best regards Wang shengjiu > > > +static int fsl_asrc_sel_proc(int inrate, int outrate, > > I think "void" type should be just fine as we made sure there is no > unsupported cases running in this function.

2 1

[alsa-devel] [PATCH V5 0/3] Support more sample rate in asrc
by S.j. Wang 22 Apr '19

22 Apr '19

Support more sample rate in asrc Shengjiu Wang (3): ASoC: fsl_asrc: Fix the issue about unsupported rate ASoC: fsl_asrc: replace the process_option table with function ASoC: fsl_asrc: Unify the supported input and output rate Changes in v5 - fix the [1/24, 8] - move fsl_asrc_sel_proc before setting Changes in v4 - add patch to Fix the [8kHz, 30kHz] open set issue. Changes in v3 - remove FSL_ASRC_RATES - refine fsl_asrc_sel_proc according to comments Changes in v2 - add more comments in code - add commit "Unify the supported input and output rate" sound/soc/fsl/fsl_asrc.c | 104 +++++++++++++++++++++++++++++++---------------- 1 file changed, 70 insertions(+), 34 deletions(-) -- 1.9.1

2 6

[alsa-devel] axfer: Fix creation of v1.2 headers on big-endian systems
by GitHub pull_request - opened 21 Apr '19

21 Apr '19

alsa-project/alsa-utils pull request #15 was opened from smaeul: struct block_v120_format defines these members as uint8_t. On little-endian systems, no swapping is done, and the generated block header is fine. However, on big-endian machines, the value is swapped to the high byte and then truncated by the assignment, causing both bits_per_sample and samples_per_frame to be zero. This fixes an assertion failure in container-test when later parsing the header ["assert(*samples_per_frame > 0);" in container_context_pre_process()]. Signed-off-by: Samuel Holland <samuel(a)sholland.org> Request URL : https://github.com/alsa-project/alsa-utils/pull/15 Patch URL : https://github.com/alsa-project/alsa-utils/pull/15.patch Repository URL: https://github.com/alsa-project/alsa-utils

1 0

[alsa-devel] axfer: Declare global variables as 'extern' in header
by GitHub pull_request - opened 21 Apr '19

21 Apr '19

alsa-project/alsa-utils pull request #14 was opened from smaeul: This avoids multiple definitions of the same global variable (one in each file that includes this header), and fixes a linking error when compiled with -fno-common. Signed-off-by: Samuel Holland <samuel(a)sholland.org> Request URL : https://github.com/alsa-project/alsa-utils/pull/14 Patch URL : https://github.com/alsa-project/alsa-utils/pull/14.patch Repository URL: https://github.com/alsa-project/alsa-utils

1 0

[alsa-devel] [PATCH v5 0/6] drm/bridge: sii902x: HDMI-audio support and some fixes
by Jyri Sarha 21 Apr '19

21 Apr '19

The sil,i2s-data-lanes property is still under negotiation. The question really is if there could be a generic audio i2s property to handle that. alsa_devel added to the list of recipients. Changes since v3: - Change i2s-data-lanes to sil,i2s-data-lanes The already reviewed patches are in front. Not reviewed still: - "dt-bindings: display: sii902x: Add HDMI audio bindings" - "drm/bridge: sii902x: Implement HDMI audio support" The previous round: https://patchwork.kernel.org/cover/10852643/ Jyri Sarha (5): drm/bridge: sii902x: Set output mode to HDMI or DVI according to EDID drm/bridge: sii902x: pixel clock unit is 10kHz instead of 1kHz dt-bindings: display: sii902x: Remove trailing white space dt-bindings: display: sii902x: Add HDMI audio bindings drm/bridge: sii902x: Implement HDMI audio support Tomi Valkeinen (1): drm/bridge: sii902x: add input_bus_flags .../bindings/display/bridge/sii902x.txt | 35 +- drivers/gpu/drm/bridge/sii902x.c | 478 +++++++++++++++++- 2 files changed, 505 insertions(+), 8 deletions(-) -- Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

4 13

Re: [alsa-devel] [PATCH V4 3/3] ASoC: fsl_asrc: Unify the supported input and output rate
by S.j. Wang 20 Apr '19

20 Apr '19

Hi. > > On Fri, Apr 19, 2019 at 10:23:56AM +0000, S.j. Wang wrote: > > Unify the supported input and output rate, add the 12kHz/24kHz/128kHz > > to the support list > > > > Signed-off-by: Shengjiu Wang <shengjiu.wang(a)nxp.com> > > --- > > sound/soc/fsl/fsl_asrc.c | 32 +++++++++++++++++++------------- > > 1 file changed, 19 insertions(+), 13 deletions(-) > > > > diff --git a/sound/soc/fsl/fsl_asrc.c b/sound/soc/fsl/fsl_asrc.c index > > 2c4bbc3499db..0d06e738264a 100644 > > --- a/sound/soc/fsl/fsl_asrc.c > > +++ b/sound/soc/fsl/fsl_asrc.c > > @@ -27,13 +27,14 @@ > > dev_dbg(&asrc_priv->pdev->dev, "Pair %c: " fmt, 'A' + index, > > ##__VA_ARGS__) > > > > /* Corresponding to process_option */ -static int > > supported_input_rate[] = { > > - 5512, 8000, 11025, 16000, 22050, 32000, 44100, 48000, 64000, 88200, > > - 96000, 176400, 192000, > > +static unsigned int supported_asrc_rate[] = { > > + 5512, 8000, 11025, 12000, 16000, 22050, 24000, 32000, 44100, 48000, > > + 64000, 88200, 96000, 128000, 176400, 192000, > > }; > > > > -static int supported_asrc_rate[] = { > > - 8000, 11025, 16000, 22050, 32000, 44100, 48000, 64000, 88200, 96000, > 176400, 192000, > > +static struct snd_pcm_hw_constraint_list fsl_asrc_rate_constraints = { > > + .count = ARRAY_SIZE(supported_asrc_rate), > > + .list = supported_asrc_rate, > > }; > > > > /** > > @@ -293,11 +294,11 @@ static int fsl_asrc_config_pair(struct > fsl_asrc_pair *pair) > > ideal = config->inclk == INCLK_NONE; > > > > /* Validate input and output sample rates */ > > - for (in = 0; in < ARRAY_SIZE(supported_input_rate); in++) > > - if (inrate == supported_input_rate[in]) > > + for (in = 0; in < ARRAY_SIZE(supported_asrc_rate); in++) > > + if (inrate == supported_asrc_rate[in]) > > break; > > Not sure if we still need it upon having hw_constraint. Maybe m2m needs it? Yes. Best regards Wang shengjiu

2 1

Re: [alsa-devel] [PATCH V4 2/3] ASoC: fsl_asrc: replace the process_option table with function
by S.j. Wang 20 Apr '19

20 Apr '19

Hi > > > On Fri, Apr 19, 2019 at 10:23:53AM +0000, S.j. Wang wrote: > > > @@ -289,6 +318,12 @@ static int fsl_asrc_config_pair(struct fsl_asrc_pair > *pair) > > return -EINVAL; > > } > > > > + ret = fsl_asrc_sel_proc(inrate, outrate, &pre_proc, &post_proc); > > Since the function always return 0, I am thinking of treating this function as > a lookup function, and then moving this call right before the register > settings -- as we have already made sure that both inrate and outrate are > supported. Ok. > > > + if (ret) { > > + pair_err("No supported pre-processing options\n"); > > + return ret; > > + } > > And probably no longer need this error-out. If there's a new limitation > related to this function, I believe we can add it to the rate validation > section as we are doing now -- better to have rate validation code at one > place. > Ok. > > @@ -380,8 +415,8 @@ static int fsl_asrc_config_pair(struct fsl_asrc_pair > *pair) > > /* Apply configurations for pre- and post-processing */ > > Here: > - /* Apply configurations for pre- and post-processing */ > + /* Select and apply configurations for pre- and post-processing */ > + fsl_asrc_sel_proc(inrate, outrate, &pre_proc, &post_proc); > > regmap_update_bits(asrc_priv->regmap, REG_ASRCFG, > > ASRCFG_PREMODi_MASK(index) | > ASRCFG_POSTMODi_MASK(index), > > - ASRCFG_PREMOD(index, process_option[in][out][0]) | > > - ASRCFG_POSTMOD(index, process_option[in][out][1])); > > + ASRCFG_PREMOD(index, pre_proc) | > > + ASRCFG_POSTMOD(index, post_proc));

1 0

[alsa-devel] [PATCH V4 0/3] Support more sample rate in asrc
by S.j. Wang 20 Apr '19

20 Apr '19

Support more sample rate in asrc Shengjiu Wang (3): ASoC: fsl_asrc: Fix the issue about unsupported rate ASoC: fsl_asrc: replace the process_option table with function ASoC: fsl_asrc: Unify the supported input and output rate Changes in v4 - add patch to Fix the [8kHz, 30kHz] open set issue. Changes in v3 - remove FSL_ASRC_RATES - refine fsl_asrc_sel_proc according to comments Changes in v2 - add more comments in code - add commit "Unify the supported input and output rate" sound/soc/fsl/fsl_asrc.c | 113 ++++++++++++++++++++++++++++++++--------------- 1 file changed, 77 insertions(+), 36 deletions(-) -- 1.9.1

2 7