Re: [PATCH 2/2] ARM: MXS: Add Crystalfontz CFA-10036 DTS

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

Hi Marek,

Le 21/06/2012 11:49, Marek Vasut a écrit :
> Dear Maxime Ripard,
> 
>> Signed-off-by: Maxime Ripard <maxime.ripard@xxxxxxxxxxxxxxxxxx>
>>
>> Cc: Brian Lily <brian@xxxxxxxxxxxxxxxx>
> 
> But then, wait ... why did you add the mach-cfa10036.c entry?

As I stated in my introduction mail, I first did the job on 3.4 before
forward porting it to 3.5. And since the MMC part worked on 3.4 and no
longer on 3.5, I wanted to keep an easily trackable file between these
two versions, which is not the case with DT.

So partly historical, partly debugging reasons :)

Anyway, I totally understand your point, I'll remove the board file for
the next postings.

>> ---
>>  arch/arm/boot/dts/cfa10036.dts |   66
>> ++++++++++++++++++++++++++++++++++++++++ 1 file changed, 66 insertions(+)
>>  create mode 100644 arch/arm/boot/dts/cfa10036.dts
>>
>> diff --git a/arch/arm/boot/dts/cfa10036.dts
>> b/arch/arm/boot/dts/cfa10036.dts new file mode 100644
>> index 0000000..ceae960
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/cfa10036.dts
>> @@ -0,0 +1,66 @@
>> +/*
>> + * Copyright 2012 Free Electrons
>> + *
>> + * The code contained herein is licensed under the GNU General Public
>> + * License. You may obtain a copy of the GNU General Public License
>> + * Version 2 or later at the following locations:
>> + *
>> + * http://www.opensource.org/licenses/gpl-license.html
>> + * http://www.gnu.org/copyleft/gpl.html
>> + */
>> +
>> +/dts-v1/;
>> +/include/ "imx28.dtsi"
>> +
>> +/ {
>> +	model = "Crystalfontz CFA-10036 Board";
>> +	compatible = "crystalfontz,cfa10036", "fsl,imx28-evk", "fsl,imx28";
>> +
>> +	memory {
>> +		reg = <0x40000000 0x08000000>;
>> +	};
>> +
>> +	apb@80000000 {
>> +		apbh@80000000 {
>> +			ssp0: ssp@80010000 {
>> +				compatible = "fsl,imx28-mmc";
>> +				pinctrl-names = "default";
>> +				pinctrl-0 = <&mmc0_4bit_pins_a
>> +					&mmc0_cd_cfg &mmc0_sck_cfg>;
>> +				bus-width = <4>;
>> +				status = "okay";
>> +			};
>> +			pinctrl@80018000 {
>> +				#address-cells = <1>;
>> +				#size-cells = <0>;
>> +				compatible = "fsl,imx28-pinctrl", "simple-bus";
>> +				reg = <0x80018000 2000>;
>> +
>> +				mmc0_4bit_pins_a: mmc0-4bit@0 {
>> +					reg = <0>;
>> +					fsl,pinmux-ids = <0x2000 0x2010 0x2020
>> +						0x2030 0x2080 0x2090 0x20a0>;
>> +					fsl,drive-strength = <1>;
>> +					fsl,voltage = <1>;
>> +					fsl,pull-up = <1>;
> 
> Do you need this copy here?

This is not a direct copy, as there is only 4 pins for the MMC bus on
this board, and one of the pins used for the additional bus pins on the
imx28evk is used to power the OLED screen on this one, so I don't want
all of the imx28 pins allocated for the mmc0 to be muxed as such.

Anyway, I'm not very familiar with pinctrl yet, so I might be missing
something.

>> +				};
>> +
>> +				duart_pins_a: duart@0 {
>> +					reg = <0>;
>> +					fsl,pinmux-ids = <0x3022 0x3032>;
>> +					fsl,drive-strength = <0>;
>> +					fsl,voltage = <1>;
>> +					fsl,pull-up = <0>;
> 
> And this?

The pins to be muxed for the DUART differs from the imx28 here as well.
On the iMX28EVK, the pins used for the DUART are 3-16 and 3-17, while we
use the 3-2 and 3-3 on this board.

> 
>> +				};
>> +			};
>> +		};
>> +
>> +		apbx@80040000 {
>> +			duart: serial@80074000 {
>> +				pinctrl-names = "default";
>> +				pinctrl-0 = <&duart_pins_a>;
>> +				status = "okay";
>> +			};
>> +		};
>> +	};
>> +};

Thanks!

Maxime


-- 
Maxime Ripard, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel



[Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [eCos]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [PDAs]     [Linux]     [Linux MIPS]     [Yosemite Campsites]     [Photos]

Add to Google Follow linuxarm on Twitter