Date: Tue, 29 Mar 2016 07:54:48 -0700
From: Bjorn Andersson <bjorn.andersson@linaro.org>
To: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
Cc: Andy Gross <andy.gross@linaro.org>, linux-arm-msm@vger.kernel.org,
        devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-kernel@vger.kernel.org, linux-soc@vger.kernel.org
Subject: Re: [PATCH 00/12] ARM: dts: Add dragonboard-600c support.
Message-ID: <20160329145448.GP8929@tuxbot>
References: <1458762366-9233-1-git-send-email-srinivas.kandagatla@linaro.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1458762366-9233-1-git-send-email-srinivas.kandagatla@linaro.org>
User-Agent: Mutt/1.5.23 (2014-03-12)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 2016
Lines: 55

On Wed 23 Mar 12:46 PDT 2016, Srinivas Kandagatla wrote:

> Hi Andy,
> 
> This patchset adds support to Dragonboard 600c which is based on APQ8064.
> 
> With this patchset, except spi I was able to test all the below features
> on this board on top of linus master branch.
> 1> i2c
> 2> spi
> 3> sd/mmc with card detect
> 4> eMMC
> 5> USB
> 6> SATA
> 7> on board Ethernet based on PCIE.
> 8> user and activity leds.
> 
> AHCI SATA is broken on APQ8064 on mainline, due to changes in libahci, fix is
> being discussed which should not effect this dt patches anyway.
> This patchset also has a fix to i2c/spi pinctrls which was not set correctly
> in my previous apq8064 patches, I have verified this patchset with eeprom and
> sensors on the board, you might want to take that patch in next rc.
> 
> Thanks,
> srini
> 
> Srinivas Kandagatla (12):
>   ARM: dts: apq8064: fix the pinctrls for i2c and spi
>   ARM: dts: apq8064: add support to gsbi1 uart
>   ARM: dts: apq8064: add gsbi7 i2c support
>   ARM: dts: dragonboard-600c: add board support with serial
>   ARM: dts: dragonboard-600c: add pmic regulator supplies
>   ARM: dts: dragonboard-600c: Add eMMC and SD card support
>   ARM: dts: dragonboard-600c: add usb support
>   ARM: dts: dragonboard-600c: add pcie support
>   ARM: dts: dragonboard-600c: add on board sata support.
>   ARM: dts: dragonboard-600c: Add on board leds support
>   ARM: dts: dragonboard-600c: add i2c support
>   ARM: dts: dragonboard-600c: add spi support

Please abbreviate db600c (and drop those upper case As).

> 
>  arch/arm/boot/dts/Makefile                         |   1 +
>  .../arm/boot/dts/qcom-apq8064-dragonboard-600c.dts | 377 +++++++++++++++++++++

Please make this qcom-apq8064-arrow-db600c.dts

>  arch/arm/boot/dts/qcom-apq8064-pins.dtsi           |  39 +++
>  arch/arm/boot/dts/qcom-apq8064.dtsi                |  41 ++-
>  4 files changed, 452 insertions(+), 6 deletions(-)
>  create mode 100644 arch/arm/boot/dts/qcom-apq8064-dragonboard-600c.dts

Regards,
Bjorn