Re: RE: [PATCH v2 2/3] arm/dt: tegra: add dts file for paz00

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Am Donnerstag, 27. Oktober 2011, 09:50:03 schrieb Stephen Warren:
> Marc Dietrich wrote at Wednesday, October 26, 2011 1:59 PM:
> > This adds a dts file for paz00. As a side effect, this also enables
> > the embedded controller which controls the keyboard, touchpad, power,
> > leds, and some other functions.
> 
> ...
> 
> > +++ b/arch/arm/boot/dts/tegra-paz00.dts
> > @@ -0,0 +1,70 @@
> > +/dts-v1/;
> > +
> > +/memreserve/ 0x1c000000 0x04000000;
> > +/include/ "tegra20.dtsi"
> > +
> > +/ {
> > +	model = "Toshiba AC100 / Dynabook AZ";
> > +	compatible = "compal,paz00", "nvidia,tegra20";
> > +
> > +	chosen {
> > +		bootargs = "mem=448@0 console=ttyS0,115200n8 root=/dev/mmcblk1p1";
> 
> You shouldn't need the mem= parameter here; it wasn't in your first patch set.

that's because I forgot it. Sorry, I didn't mentioned it in the changelog. I wonder 
why mem= is still needed. I've seen patches on the chromeos tree which try to reserve 
the gpu memory on demand. While we are at it, what is the vmalloc=192M used by most 
other boards for?

> > +	};
> > +
> > +	memory@0 {
> > +		reg = <0x00000000 0x20000000>;
> > +	};
> > +
> > +	i2c@7000c000 {
> > +		clock-frequency = <400000>;
> > +	};
> > +
> > +	i2c@7000c400 {
> > +		clock-frequency = <400000>;
> > +	};
> > +
> > +	i2c@7000c500 {
> > +		status = "disable";
> > +	};
> > +
> > +	nvec@7000c500 {
> > +		#address-cells = <1>;
> > +		#size-cells = <0>;
> > +		compatible = "nvidia,nvec";
> 
> I'm not convinced that's the correct compatible value, but I should
> discuss this more in the nvec patch...
> 
> > +		reg = <0x7000C500 0x100>;
> > +		interrupts = <124>;
> > +		clock-frequency = <80000>;
> > +		request-gpios = <&gpio 170 0>;
> > +		slave-addr = <138>;
> > +	};
> > 
> > 
> > +++ b/arch/arm/mach-tegra/Makefile
> > @@ -31,6 +31,7 @@ obj-${CONFIG_MACH_SEABOARD}             +=
> > board-seaboard-pinmux.o> 
> >  obj-${CONFIG_MACH_TEGRA_DT}             += board-dt.o
> >  obj-${CONFIG_MACH_TEGRA_DT}             += board-harmony-pinmux.o
> > 
> > +obj-$(CONFIG_MACH_TEGRA_DT)             += board-paz00-pinmux.o
> > 
> >  obj-${CONFIG_MACH_TEGRA_DT}             += board-seaboard-pinmux.o
> 
> What branch is your patch based on? Those lines use braces not brackets
> in the latest code (incorrect yes, but that's what's there; Olof said
> he'll fix it for the next merge window). Do you have some local patches
> in your branch before this patchset? Note: Feel free to add your lines
> using the correct syntax; I'm just wondering about the existing context
> in your patch.

branch is based on linux-next. It will create conflicts anyway during merge (given 
your trimslice update).

Marc


--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux