Re: [STLinux Kernel] [PATCH 4/7] ARM: stih416-b2020e: Enable PWM on the B2020 Rev-E

From: Maxime Coquelin <maxime.coquelin@st.com>
To: Peter Griffin <peter.griffin@linaro.org>,
	Lee Jones <lee.jones@linaro.org>
Cc: linux-pwm@vger.kernel.org, kernel@stlinux.com,
	devicetree@vger.kernel.org, srinivas.kandagatla@gmail.com,
	linux-kernel@vger.kernel.org, thierry.reding@gmail.com,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [STLinux Kernel] [PATCH 4/7] ARM: stih416-b2020e: Enable PWM on the B2020 Rev-E
Date: Thu, 19 Jun 2014 12:59:27 +0200	[thread overview]
Message-ID: <53A2C28F.1080007@st.com> (raw)
In-Reply-To: <20140619101236.GA7390@griffinp-ThinkPad-X1-Carbon-2nd>

[-- Attachment #1: Type: text/plain, Size: 1175 bytes --]

Hi Peter,

On 06/19/2014 12:12 PM, Peter Griffin wrote:
> Hi Lee,
> 
>>> Why are you only enabling this for rev-E board?
>>
>> Purely for testing purposes, as it's the only board I have.  If
>> anyone has the other revisions and would be happy to test the driver
>> for me then great.
> 
> Ok that's what I suspected.
> 
> If you tell me how, I can test on a Rev C board. But as there are several other 
> revisions I think checking the board schematics would be the quickest way forward.

Only Rev C & E are in the wild now, so checking these two boards will
be enough.

Thanks,
Maxime

> 
>>> If you make the stih415 DT PWM nodes part of this series as well, then 
>>> assuming there is no good reason to the contary, the PWM could 
>>> be enabled for both stih415/6 in the common stih41x-b2020 DTS file which
>>> is also included by rev-E (assuming that PWM wiring is not some rev-E specific
>>> thing).
> 
> Can you add the stih415 bindings in V2 as well?
> 
> regards,
> 
> Peter
> 
> _______________________________________________
> Kernel mailing list
> Kernel@stlinux.com
> http://www.stlinux.com/mailman/listinfo/kernel
> 

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]