[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