[PATCH v2] sandbox: cros-ec: Add tests for the Chrome OS PWM driver

Alper Nebi Yasak alpernebiyasak at gmail.com
Wed May 19 18:26:54 CEST 2021


On 19/05/2021 18:34, Simon Glass wrote:
> Hi Alpher,
> 
> On Wed, 19 May 2021 at 09:30, Alper Nebi Yasak <alpernebiyasak at gmail.com> wrote:
>>
>> This patch adds a limited pulse-width modulator to sandbox's Chrome OS
>> Embedded Controller emulation. The emulated PWM device supports 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. Though
>> the EC supports specifying the PWM channel by its type (e.g. display
>> backlight, keyboard backlight), this is not implemented in the emulation
>> as nothing in U-Boot uses this type specification.
>>
>> This emulated PWM device is then used to test the Chrome OS PWM driver
> 
> Typically we use "Chromium OS" for open-source software.

OK. I'll switch to that instead and send a v3 shortly.

(I also noticed that I forgot "EC" in the subject line...)

>> in sandbox. Adding the required device 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>
>> ---
>> This depends on a small fix [1] for cros-ec-pwm which otherwise fails to
>> build.
>>
>> [1] https://patchwork.ozlabs.org/project/uboot/patch/20210514134840.19380-1-alpernebiyasak@gmail.com/
>>
>> Changes in v2:
>> - Try to clarify commit message
>>
>> v1: https://patchwork.ozlabs.org/project/uboot/patch/20210516154144.141945-1-alpernebiyasak@gmail.com/
>>
>>  arch/sandbox/dts/test.dts          |  6 +++
>>  arch/sandbox/include/asm/test.h    | 10 +++++
>>  configs/sandbox64_defconfig        |  1 +
>>  configs/sandbox_defconfig          |  1 +
>>  configs/sandbox_flattree_defconfig |  1 +
>>  configs/sandbox_noinst_defconfig   |  1 +
>>  configs/sandbox_spl_defconfig      |  1 +
>>  drivers/misc/cros_ec_sandbox.c     | 47 +++++++++++++++++++++++
>>  test/cmd/pwm.c                     | 32 +++++++++++++++-
>>  test/dm/Makefile                   |  1 +
>>  test/dm/cros_ec_pwm.c              | 60 ++++++++++++++++++++++++++++++
>>  test/dm/panel.c                    |  2 +-
>>  test/dm/pwm.c                      |  6 ++-
>>  13 files changed, 164 insertions(+), 5 deletions(-)
>>  create mode 100644 test/dm/cros_ec_pwm.c
> 
> Reviewed-by: Simon Glass <sjg at chromium.org>
> 


More information about the U-Boot mailing list