lkml.org 
[lkml]   [2022]   [May]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH 1/5] arm64: dts: qcom: sc7180: Add wormdingler dts files
Hi,

On Sat, Apr 30, 2022 at 1:18 AM Joseph S. Barrera III
<joebar@chromium.org> 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@chromium.org>
> ---

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@mail.gmail.com/


> @@ -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@mail.gmail.com

\
 
 \ /
  Last update: 2022-05-02 19:19    [W:0.140 / U:0.032 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site