[PATCH] sandbox: cros-ec: Add tests for the cros-ec-pwm driver

Alper Nebi Yasak alpernebiyasak at gmail.com
Wed May 19 17:28:17 CEST 2021


On 16/05/2021 23:05, Heinrich Schuchardt wrote:
> Am 16. Mai 2021 17:41:44 MESZ schrieb Alper Nebi Yasak <alpernebiyasak at gmail.com>:
>> This patch lets sandbox-cros-ec emulate a limited pwm device which has
>> multiple channels but can only set a duty cycle for each, as the actual
>> EC doesn't expose any functionality or information other than that.
>> Mapping non-generic EC_PWM_TYPE_* values to these emulated pwm channels
>> is not implemented as nothing in U-Boot uses these types.
> 
> This commit messages is full of of abbreviations which makes it unclear. Please, use human readable terms.
> 
> I guess you might be talking about
> 
> Chromium OS
> Embedded Controller
> Pulse Width Modulation
> 
> But I am not sure.

Those are pretty much correct. I'm sending a v2 with hopefully a clearer
commit message.

> Best regards
> 
> Heinrich
> 
>>
>> This emulated pwm is then used to test the cros-ec-pwm driver in
>> sandbox. Adding the cros-ec-pwm node to the sandbox test device-tree
>> unfortunately makes it the first pwm device, so this also touches some
>> other tests to make sure they still use the sandbox pwm.
>>
>> Signed-off-by: Alper Nebi Yasak <alpernebiyasak at gmail.com>


More information about the U-Boot mailing list