RE: [PATCH 1/2] ARM: i.MX6: dts: change iomuxc pinctrl config to match Rev. 1 IMX6DQRM

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

 




I wrote:
>> @@ -759,23 +759,23 @@
>>                               gpmi-nand {
>>                                       pinctrl_gpmi_nand_1: gpmi-nand-1 {
>>                                               fsl,pins = <
>> -                                                     MX6QDL_PAD_NANDF_CLE__NAND_CLE     0xb0b1
>> -                                                     MX6QDL_PAD_NANDF_ALE__NAND_ALE     0xb0b1
[ . . . ]
>>                                       };
>>                               };

Huang Shijie [b32955@xxxxxxxxxxxxx] responds:
>I tested this patch with the imx6q-ard board and imx6dl-ard board.
>For the gpmi nand test result:
>[1] imx6q-ard: We can pass the bonie++ stress test with the imx6q-ard board.
>[2] imx6dl-ard: But we fails to read the nand ID with the imx6dl-ard board.
>In other word, this gpmi can _not_ works with this patch.

Huang, I appreciate your speedy testing and sharing of results.   While I carefully and lengthily RTFM to create this patch, we can agree that making the hardware work well is the final goal!   We have tested the patch I sent on our hardware with our tests, but certainly any patch that renders some features of some boards inoperable is unacceptable.

The patch we are discussing is about 670 lines, and the NAND part is about 60.   What is your advice regarding the remaining ~600 lines?  

Best wishes,
Alison Chaiken
Mentor Embedded Software Division
Fremont, CA, USA


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




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux