Hi, On Sat, Apr 30, 2022 at 1:18 AM Joseph S. Barrera III <joebar@xxxxxxxxxxxx> wrote: > > Wormdingler is a trogdor-based board, shipping to customers as the > Lenovo IdeaPad Chromebook Duet 3. These dts files are copies from > the downstream Chrome OS 5.4 kernel, but with downstream bits removed. > > Signed-off-by: Joseph S. Barrera III <joebar@xxxxxxxxxxxx> > --- I was expecting that this patch series would be labeled "v3" and would also have version history. What happened? I provided the tags you needed for stuff like this in v2 and even wrote the history bits for you... [1] [1] https://lore.kernel.org/r/CAD=FV=XbQ7LhnxGAavLL3XDpPigwtCz0CF3YcZ=ywrXwu=uiMQ@xxxxxxxxxxxxxx/ > @@ -0,0 +1,33 @@ > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > +/* > + * Google Wormdingler board device tree source > + * > + * Copyright 2021 Google LLC. > + * > + * SKU: 0x0001 => 1 > + * - bits 11..8: Panel ID: 0x0 (INX) > + */ > + > +/dts-v1/; > + > +#include "sc7180-trogdor-wormdingler.dtsi" In my reply to your v2 I was suggesting that the rt5682s files could be done more cleanly. I guess it's not 100% needed and could always be a later cleanup, but any reason why you decided not to do it? > +&ap_tp_i2c { > + status = "disabled"; > +}; > + > +&backlight { > + pwms = <&cros_ec_pwm 0>; > +}; Downstream we have: &camcc { status = "okay"; }; Why did you remove it here? You didn't in the previous version. Coach Z and Homestar still define it upstream and they work fine. Even if we haven't finished defining the MIPI camera bits, enabling the camera clock controller should still be fine, right? Also in my reply to v2 I suggested that you delete the keypad num-rows / num-columns in wormdingler.dtsi. Hmmm, but I guess maybe we should put the num-rows/num-columns change on hold while we settle on an approach for it [1]. So I guess overall summary: * Pretty sure you want "camcc", so please spin with that. * Please figure out how to tag your series as v4 and have version history. * I would still prefer the rt5682s stuff be cleaned up but if it's really too hard we can do it in a follow-up. [1] https://lore.kernel.org/r/CAD=FV=VX8EEgkeLgKwyKvjztcjbA8UhKOUpTr-sS1_Ec=QcWbA@xxxxxxxxxxxxxx
