linux-kernel - Re: [PATCH 0/2] add rockchip i2s driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <11713484.S15eBXkq6o@diego>
Date:	Fri, 11 Jul 2014 12:03:50 +0200
From:	Heiko Stübner <heiko@...ech.de>
To:	jianqun <xjq@...k-chips.com>
Cc:	lgirdwood@...il.com, broonie@...nel.org, perex@...ex.cz,
	tiwai@...e.de, grant.likely@...aro.org, robh+dt@...nel.org,
	zhangqing@...k-chips.com, linux-kernel@...r.kernel.org,
	alsa-devel@...a-project.org, devicetree@...r.kernel.org,
	hj@...k-chips.com, kever.yang@...k-chips.com,
	huangtao@...k-chips.com, zyw@...k-chips.com, yzq@...k-chips.com,
	zhenfu.fang@...k-chips.com, cf@...k-chips.com, kfx@...k-chips.com
Subject: Re: [PATCH 0/2] add rockchip i2s driver

Hi Jianqun,

Am Freitag, 11. Juli 2014, 17:54:47 schrieb jianqun:
> From: Jianqun Xu <xjq@...k-chips.com>
> 
> This patch is to add driver for I2S controller in RK3xxx SoCs.
> 
> The only one change in this patch against which has been reviewed before is
> to fix randconfig build error with next-20140710 by add "#include
> <linux/module.h>".

as you know, the i2s patches were already included in the sound tree ... so 
please create a follow-up patch fixing the randconfig issue, without 
resubmitting the whole series again.


Heiko


> 
> Jianqun Xu (2):
>   ASoC: dt-bindings: add rockchip i2s bindings
> 
> changes since v2:
>  - change the description of dma-names, adviced by Mark
>  - change the description of interrupts, adviced by Mark
>  - list clock-names to be easy to read, adviced by Mark
>  - modify compatible for rk3288 example, adviced by Mark and Heiko
> 
>  changes since v1:
>  - modify the description of clock-names property, adviced by Mark Rutland.
>  - modify the example to fit for rk3288.
> 
> 
>   ASoC: add driver for Rockchip RK3xxx I2S controller
> 
> changes since v3:
>  - fix randconfig build error with next-20140710 by add "#include
> <linux/module.h>" tested with config given by robot test.
> 
> changes since v2:
>  - replace 4 with DMA_SLAVE_BUSWIDTH_4_BYTES
>  - modify supported rate to SNDRV_PCM_RATE_8000_192000
>  - rename head file to "rockchi_i2s.h"
> 
> changes since v1:
>  - proper indentation for function arguments, adviced by Varka Bhadram
>  - replace sizeof(struct rk_i2s_dev) with sizeof(*i2s), adviced by Mark
> Rutland - not to include head files which are unused actually in driver,
> adviced by Mark Brown - modify i2s_tx_status/i2s_rx_status to more
> meaningful with tx_start/rx_start, adviced by Mark Brown - use
> regmap_update_bits to reduce the amount of time spent locked, adviced by
> Mark Brown - add warning for possible error while waiting for the hardware
> to be ready, adviced by Mark Brown - error checking for clk_set_rate,
> adviced by Mark Brown
>  - error checking for i2s_runtime_resume, adviced by Mark Brown
>  - remove set_clkdiv operation, clock setting will do it, adviced by Mark
> Brown - no need SND_SOC for SND_SOC_ROCKCHIP, also allow the driver to be
> select when COMPILE_TEST is selected, adviced by Lars-Peter Clausen
>  - not to include head files which are unused actually in driver, adviced by
> Lars-Peter Clausen - I2S core just use one snd_soc_dai_driver struct,
> adviced by Lars-Peter Clausen - delete rockchip_pcm.c and related codes,
> use generic dmaengine API, adviced by Lars-Peter and Mark Brown - a driver
> should never attempt to change its own device name, just use the name that
> the kernel set, adviced by Lars-Peter and Mark Brown
> 
> 
>  .../devicetree/bindings/sound/rockchip-i2s.txt     |   37 ++
>  sound/soc/Kconfig                                  |    1 +
>  sound/soc/Makefile                                 |    1 +
>  sound/soc/rockchip/Kconfig                         |   12 +
>  sound/soc/rockchip/Makefile                        |    4 +
>  sound/soc/rockchip/rockchip_i2s.c                  |  531
> ++++++++++++++++++++ sound/soc/rockchip/rockchip_i2s.h                  | 
> 223 ++++++++ 7 files changed, 809 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/sound/rockchip-i2s.txt
> create mode 100644 sound/soc/rockchip/Kconfig
>  create mode 100644 sound/soc/rockchip/Makefile
>  create mode 100644 sound/soc/rockchip/rockchip_i2s.c
>  create mode 100644 sound/soc/rockchip/rockchip_i2s.h

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/