Re: (subset) [PATCH v12 0/4] ASoC: codecs: add rk817 support

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

 



On Wed, 19 May 2021 15:37:50 -0500, Chris Morgan wrote:
> This series is to add support for the Rockchip rk817 audio codec which
> is present in all Rockchip rk817 MFD chips.
> 
> Changes in v12:
>  - Updated codec cell in MFD driver to a single line.
> Changes in v11:
>  - Stopped checking for presence of parent node in codec driver, as
>    driver should never be called if parent device doesn't exist to call
>    it.
>  - Made codec subnode optional and stopped returning errors if it is
>    missing.
>  - Downgraded missing codec clock message from dev_err to dev_dbg.
>    Users who choose to use the rk817 PMIC but do not choose to use the
>    codec will not get an error message unless they have debugging
>    enabled, whereas users who wish to use the codec but don't have the
>    clock defined will not get an error message unless they have
>    debug messages enabled. This seems like a fair compromise between
>    mututally exclusive users. Note that errors with the clock or codec
>    will still result in a dev_err.
>  - Updated DT documentation to note that codec subnode is optional if
>    no properties are to exist beneath it.
> Changes in v10:
>  - Correct order of test/ack/signed-off to chronological order.
>  - Removed ifdef from mfd driver keep cell from being added when the
>    codec was not to be built.
>  - Changed codec DT parsing messages from error to debug to prevent
>    errors in dmesg log in the rare but valid case a user has the rk817
>    PMIC but does not use the codec.
> Changes in v9:
>  - Add cover letter.
>  - Remove documentation for interrupt parent per Rob Herring's request.
>  - Remove unused MODULE_DEVICE_TABLE to fix a bug identified by kernel test
>    robot.
> Changes in v8:
>  - Added additional documentation for missing properties of
>    #sound-dai-cells, interrupt-parent, and wakeup-source for mfd
>    documentation.
>  - Corrected order of elements descriptions in device tree documentation.
>  - Changed name of "mic-in-differential" to
>    "rockchip,mic-in-differential".
>  - Changed name of sound card from "rockchip,rk817-codec" to "Analog".
>  - Removed unused resets and reset-names from the i2s1_2ch node.
> Changes in v7:
>  - Removed ifdef around register definitions for MFD.
>  - Replaced codec documentation with updates to MFD documentation.
>  - Reordered elements in example to comply with upstream rules.
>  - Added binding update back for Odroid Go Advance as requested.
>  - Submitting patches from gmail now.
> Changes in v6:
>  - Included additional project maintainers for correct subsystems.
>  - Removed unneeded compatible from DT documentation.
>  - Removed binding update for Odroid Go Advance (will do in separate
>    series).
> Changes in v5:
>  - Move register definitions from rk817_codec.h to main rk808.h register
>    definitions.
>  - Add volatile register for codec bits.
>  - Add default values for codec bits.
>  - Removed of_compatible from mtd driver (not necessary).
>  - Switched to using parent regmap instead of private regmap for codec.
> Changes in v4:
>  - Created set_pll() call.
>  - Created user visible gain control in mic.
>  - Check for return value of clk_prepare_enable().
>  - Removed duplicate clk_prepare_enable().
>  - Split DT documentation to separate commit.
> Changes in v3:
>  - Use DAPM macros to set audio path.
>  - Updated devicetree binding (as every rk817 has this codec chip).
>  - Changed documentation to yaml format.
>  - Split MFD changes to separate commit.
> Changes in v2:
>  - Fixed audio path registers to solve some bugs.
> 
> [...]

Applied, thanks!

[4/4] arm64: dts: rockchip: add rk817 codec to Odroid Go
      commit: 18d5c7bf50c6d820c366c2a23d71d468b14c87d6

Best regards,
-- 
Heiko Stuebner <heiko@xxxxxxxxx>



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux