I think that is an issue with your client? It seems to come up fine if I apply it locally and look at it with gvim

On Tue, 2018-07-17 at 12:25 +0200, Karol Herbst wrote:

isn't there like 1 space missing for each change? Or maybe my client is messed up, but please align it with the first letter of the parameters not the "(".

With that fixed: Reviewed-by: Karol Herbst karolherbst@gmail.com

On Tue, Jul 17, 2018 at 2:07 AM, Lyude Paul lyude@redhat.com wrote:

...

Signed-off-by: Lyude Paul lyude@redhat.com

drivers/gpu/drm/nouveau/nouveau_hwmon.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/nouveau/nouveau_hwmon.c b/drivers/gpu/drm/nouveau/nouveau_hwmon.c index 44178b4c3599..d556f24c6c48 100644 --- a/drivers/gpu/drm/nouveau/nouveau_hwmon.c +++ b/drivers/gpu/drm/nouveau/nouveau_hwmon.c @@ -655,7 +655,7 @@ nouveau_read_string(struct device *dev, enum hwmon_sensor_types type, u32 attr,

static int nouveau_read(struct device *dev, enum hwmon_sensor_types type, u32 attr,

•                                                   int channel, long
  

*val)

•        int channel, long *val)
  

{ switch (type) { case hwmon_chip: @@ -677,7 +677,7 @@ nouveau_read(struct device *dev, enum hwmon_sensor_types type, u32 attr,

static int nouveau_write(struct device *dev, enum hwmon_sensor_types type, u32 attr,

•                                                   int channel, long
  

val)

•         int channel, long val)
  

{ switch (type) { case hwmon_temp: -- 2.17.1

---

Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau