Re: [PATCH 2/3] ARM: tegra: Add Avionic Design Medcom-Wide support

[Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>]

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.