On 09/27/2012 06:44 PM, Simon Glass wrote:
> Add LCD definitions and also a proposed binding for LCD displays.
> 
> The PWM is as per what will likely be committed to linux-next soon.
> 
> The displaymode binding comes from a proposal here:
> 
> http://lists.freedesktop.org/archives/dri-devel/2012-July/024875.html
> 
> The panel binding is new, and fills a need to specify the panel
> timings and other tegra-specific information. Should a binding appear
> that allows the pwm to handle this automatically, we can revisit
> this.

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

> +     host1x {
> +             compatible = "nvidia,tegra20-host1x", "simple-bus";
> +             reg = <0x50000000 0x00024000>;
> +             interrupts = <0 65 0x04   /* mpcore syncpt */
> +                           0 67 0x04>; /* mpcore general */
> +
> +             #address-cells = <1>;
> +             #size-cells = <1>;
> +
> +             ranges = <0x54000000 0x54000000 0x04000000>;
> +
> +             /* video-encoding/decoding */
> +             mpe {
> +                     reg = <0x54040000 0x00040000>;
> +                     interrupts = <0 68 0x04>;
> +             };

Shouldn't all of these nodes have status="disabled", since in general
boards will want to opt-in to these modules. In fact, many of these
nodes won't end up (ever?) being used in U-Boot; perhaps we should only
add the nodes we care about.

> +             /* display controllers */
> +             dc@54200000 {
> +                     compatible = "nvidia,tegra20-dc";
> +                     reg = <0x54200000 0x00040000>;
> +                     interrupts = <0 73 0x04>;
> +
> +                     rgb {
> +                             status = "disabled";
> +                     };
> +             };

I think we definitely want status="disabled" in the dc nodes themselves,
since we definitely want boards in U-Boot to control whether to enable
the dc.

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to