diff options
author | Gabe Black <gabeblack@google.com> | 2013-03-28 04:32:20 +0000 |
---|---|---|
committer | Minkyu Kang <mk7.kang@samsung.com> | 2013-04-01 14:02:08 +0900 |
commit | 92809eeed22e264a770fcfc34721d986fedcbfa2 (patch) | |
tree | f546e39dc2aad5f7e606a47b782ce1c22f3878a9 /disk/part_amiga.h | |
parent | 34b5ee1f6a7d01560f31be37dca3b2eb62877c50 (diff) | |
download | u-boot-imx-92809eeed22e264a770fcfc34721d986fedcbfa2.zip u-boot-imx-92809eeed22e264a770fcfc34721d986fedcbfa2.tar.gz u-boot-imx-92809eeed22e264a770fcfc34721d986fedcbfa2.tar.bz2 |
Exynos: Tidy up the pwm_config function in the exynos pwm driver
Some small fixes in the exynos pwm driver:
1. NS_IN_HZ is non-sensical since these are not compatible units. This
constant actually describes the number of nanoseconds in a second. Renamed it
to NS_IN_SEC. Also dropped the unnecessary parenthesis.
2. The variable "period" is not used to hold a period, it's used to hold a
frequency. Renamed it to "frequency".
3. tcmp is an unsigned value, so (tcmp < 0) will never be true and the if
which checks that condition will never execute. Also, there should be no
problem if the pwm never switches, so there's no reason to subtract one from
tcmp and therefore no reason to compare it against zero. Removed both ifs. If
they weren't removed, tcmp should be a signed value.
4. Add a check for a 0 period.
Test with command "sf probe 1:0; time sf read 40008000 0 1000".
Try with different numbers of bytes and see that sane values are obtained
Build and boot U-boot with this patch, backlight works properly.
Signed-off-by: Gabe Black <gabeblack@google.com>
Signed-off-by: Akshay Saraswat <akshay.s@samsung.com>
Acked-by: Simon Glass <sjg@chromium.org>
Signed-off-by: Minkyu Kang <mk7.kang@samsung.com>
Diffstat (limited to 'disk/part_amiga.h')
0 files changed, 0 insertions, 0 deletions