greybus-dev April 2022

greybus-dev@lists.linaro.org

13 participants
14 discussions

[PATCH 0/2] staging: greybus: fix warnings reported by checkpatch

by Jaehee Park

The first patch corrects a typo in a comment. The second patch fixes 'void function return statements are not generally useful' warning. Jaehee Park (2): staging: greybus: correct typo in comment 'Atleast' to 'At least' staging: greybus: remove return in an empty void function drivers/staging/greybus/arche-apb-ctrl.c | 2 +- drivers/staging/greybus/audio_codec.c | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) -- 2.25.1

3 years, 1 month

[PATCH 1/2] staging: greybus: correct typo in comment 'Atleast' to 'At least'

by Jaehee Park

Correct spelling typo. Signed-off-by: Jaehee Park <jhpark1013(a)gmail.com> --- drivers/staging/greybus/arche-apb-ctrl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/greybus/arche-apb-ctrl.c b/drivers/staging/greybus/arche-apb-ctrl.c index bbf3ba744fc4..45afa208d004 100644 --- a/drivers/staging/greybus/arche-apb-ctrl.c +++ b/drivers/staging/greybus/arche-apb-ctrl.c @@ -445,7 +445,7 @@ static int __maybe_unused arche_apb_ctrl_suspend(struct device *dev) static int __maybe_unused arche_apb_ctrl_resume(struct device *dev) { /* - * Atleast for ES2 we have to meet the delay requirement between + * At least for ES2 we have to meet the delay requirement between * unipro switch and AP bridge init, depending on whether bridge is in * OFF state or standby state. * -- 2.25.1

3 years, 1 month

[PATCH v2] staging: greybus: pwm: Drop assignment to struct pwm_chip::base

by Uwe Kleine-König

Since commit f9a8ee8c8bcd ("pwm: Always allocate PWM chip base ID dynamically") the value held in base isn't used any more in the PWM framework. All PMWs get assigned a dynamic ID, so the assignment is redundant and can be dropped. Reviewed-by: Johan Hovold <johan(a)kernel.org> Signed-off-by: Uwe Kleine-König <u.kleine-koenig(a)pengutronix.de> --- Hello, changes since (implicit) v1: - Add "pwm: " to Subject - Add Reviewed-by:-tag for Johan Best regards Uwe drivers/staging/greybus/pwm.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/greybus/pwm.c b/drivers/staging/greybus/pwm.c index ad20ec24031e..3fda172239d2 100644 --- a/drivers/staging/greybus/pwm.c +++ b/drivers/staging/greybus/pwm.c @@ -297,7 +297,6 @@ static int gb_pwm_probe(struct gbphy_device *gbphy_dev, pwm->dev = &gbphy_dev->dev; pwm->ops = &gb_pwm_ops; - pwm->base = -1; /* Allocate base dynamically */ pwm->npwm = pwmc->pwm_max + 1; ret = pwmchip_add(pwm); base-commit: 3123109284176b1532874591f7c81f3837bbdc17 -- 2.35.1

3 years, 1 month

[PATCH] staging: greybus: Drop assignment to struct pwm_chip::base

by Uwe Kleine-König

Since commit f9a8ee8c8bcd ("pwm: Always allocate PWM chip base ID dynamically") the value held in base isn't used any more in the PWM framework. All PMWs get assigned a dynamic ID, so the assignment is redundant and can be dropped. Signed-off-by: Uwe Kleine-König <u.kleine-koenig(a)pengutronix.de> --- drivers/staging/greybus/pwm.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/greybus/pwm.c b/drivers/staging/greybus/pwm.c index ad20ec24031e..3fda172239d2 100644 --- a/drivers/staging/greybus/pwm.c +++ b/drivers/staging/greybus/pwm.c @@ -297,7 +297,6 @@ static int gb_pwm_probe(struct gbphy_device *gbphy_dev, pwm->dev = &gbphy_dev->dev; pwm->ops = &gb_pwm_ops; - pwm->base = -1; /* Allocate base dynamically */ pwm->npwm = pwmc->pwm_max + 1; ret = pwmchip_add(pwm); base-commit: 3123109284176b1532874591f7c81f3837bbdc17 -- 2.35.1

3 years, 1 month

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

greybus-dev April 2022