On 07/25/2012 09:06 AM, Thierry Reding wrote:
> The Medcom is a 16:9 15" terminal that is used for patient infotainment
> in hospitals.

> diff --git a/arch/arm/boot/dts/tegra20-medcom.dts 
> b/arch/arm/boot/dts/tegra20-medcom.dts

> +     model = "Avionic Design Medcom-Wide board";
> +     compatible = "ad,medcom-wide", "nvidia,tegra20";

The filename implies just "medcom" but the compatible value says
"medcom-wide". I assume the two should match. Are there multiple medcom
versions, with SW-visible differences?

> +     sdhci@c8000600 {
> +             cd-gpios = <&gpio 58 0>; /* gpio PH2 */
> +             wp-gpios = <&gpio 59 0>; /* gpio PH3 */
> +             status = "okay";
> +     };

You want a bus-width property in that node. Same for Plutux.

> +     backlight {
> +             compatible = "pwm-backlight";
> +             pwms = <&pwm 0 5000000>;

The tegra20.dtsi doesn't assign label "pwm" to the PWM controller yet...

The Plutux comments all apply here as well.
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to