diff --git a/hosts/modules/hal/pine64-pinephone-pro/rk3399-pinephone-pro-sound.dtso b/hosts/modules/hal/pine64-pinephone-pro/rk3399-pinephone-pro-sound.dtso index 7b4b9bc41..3fc2dd694 100644 --- a/hosts/modules/hal/pine64-pinephone-pro/rk3399-pinephone-pro-sound.dtso +++ b/hosts/modules/hal/pine64-pinephone-pro/rk3399-pinephone-pro-sound.dtso @@ -188,7 +188,7 @@ // declared in rk3399-base.dtsi, as `i2c1: i2c@ff110000 { ... }` &i2c1 { status = "okay"; - clock-frequency = <400000>; + // clock-frequency = <400000>; // from rk3399-firefly.dts // note that i2c3 in pinephone-pro is defined as 450/15 rise/fall i2c-scl-rising-time-ns = <300>; @@ -210,10 +210,10 @@ clock-names = "mclk"; // XXX: this differs from megi's kernel (which declares in3 as differential). // the schematics show `IN2P` / `IN2N` as being differential inputs though, with no mention of "in3" - // realtek,in2-differential; - realtek,in3-differential; + realtek,in2-differential; + // realtek,in3-differential; // megi has lout-differential, but schematic shows LOUT{R,L} => 4G_IN_{R,L} i.e. stereo - realtek,lout-differential; + // realtek,lout-differential; // codec has ldo1-en connected to 1.8V via pull-up, i.e. permanently enabled, no GPIO? // realtek,ldo1-en-gpios = VCCA1V8_CODEC; // PDM_SDI1_MIC -> IN1P/DMIC1_DAT @@ -231,8 +231,8 @@ #sound-dai-cells = <0>; // TODO: i think all of these can be removed? - assigned-clocks = <&cru SCLK_I2S0_8CH>; - assigned-clock-rates = <11289599>; // 44100 * 256 + // assigned-clocks = <&cru SCLK_I2S0_8CH>; + // assigned-clock-rates = <11289599>; // 44100 * 256 // // rt5645 describes avdd-supply, cpvdd-supply, but rt5640 does not. // // // avdd-supply = <&vcca3v0_codec>;