Hi Thomas,
On 08/01/2015 15:54, Thomas Petazzoni wrote:
> Dear Gregory CLEMENT,
>
> On Thu, 8 Jan 2015 11:51:44 +0100, Gregory CLEMENT wrote:
>> This SoC belongs to the Armada 38x family. The main difference with
>> the Armada 385 is that the 388 can handle two more SATA
>> ports. Currently the only consequence is the use of a different
>> compatible string for the pinctrl node, in order to be able to use the
>> pins associated to this 2 new SATA ports.
>
> I am not sure the pinmux is the only difference. Read on below.
>
>> +#include "armada-385.dtsi"
>> +
>> +/ {
>> + model = "Marvell Armada 388 family SoC";
>> + compatible = "marvell,armada388", "marvell,armada380";
>> + soc {
>
> Please, one empty new line between the last property (compatible) and
> the "soc" node.
>
>> + internal-regs {
>> + pinctrl@18000 {
>> + compatible = "marvell,mv88f6828-pinctrl";
>> + };
>> + };
>> + };
>> +};
>
> So, the 380 and 385 have two SATA ports, presumably ports 0 and 1.
> According to the Functional Specifications, section 3.1 (Internal
> Address map), those two ports are located at 0xA8000. Ports 2 and 3,
> which are only available on Armada 388, are located at 0xE0100 and
> 0xE0180 according to the same datasheet. So I believe that the node:
>
> sata@a8000 {
> compatible = "marvell,armada-380-ahci";
> reg = <0xa8000 0x2000>;
> interrupts = <GIC_SPI 26 IRQ_TYPE_LEVEL_HIGH>;
> clocks = <&gateclk 15>;
> status = "disabled";
> };
>
> should stay where it is in armada-38x.dtsi.
>
> However, the node:
>
> sata@e0000 {
> compatible = "marvell,armada-380-ahci";
> reg = <0xe0000 0x2000>;
> interrupts = <GIC_SPI 28 IRQ_TYPE_LEVEL_HIGH>;
> clocks = <&gateclk 30>;
> status = "disabled";
> };
>
> should probably move into armada-388.dtsi, since it covers ports 2 and
> 3.
I will do this.
>
> In addition, I believe that the existing armada-385-db.dts and
> armada-385-rd.dts should be renamed to armada-388-db.dts and
> armada-388-rd.dts, and they should include armada-388.dtsi instead. At
> least it's sure for the DB (which has both SATA controllers enabled).
> For the RD, it remains to be confirmed, but in general most Marvell
> development platforms use the superset SoC, which has all the
> functionalities.
The RD also uses 4 SATA ports and also the pin which are only associated
to the Armada 388, so it seems also be this Soc. I will convert them.
Thanks,
Gregory
--
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
--
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
