Sorry for that. I'll split the patch and I'll come again ;)
El dia 11/04/2017 22:12, "Karol Herbst" va escriure:
thanks for the work, but could you please split that patch?
It looks like you are doing several things at once and it isn't really
easy to review like this.
https://bugs.freedesktop.org/show_bug.cgi?id=100676
--- Comment #1 from Ben Skeggs ---
Can you boot your kernel with "log_buf_len=8M drm.debug=0x14
nouveau.debug=disp=trace,i2c=trace,bios=trace" and post an updated log?
--
You are receiving this mail because:
You are the
2017-04-13 11:08 GMT+02:00 Oscar Salvador :
> This patch introduces the structure "struct hwmon_ops" and sets up the
> ".visible" operation.
> Is also a preparation for the next patch where all work is being done.
>
> ---
Also your series got a little messed up, there are two 1/4 patches,
and both are different.
2017-04-13 11:07 GMT+02:00 Oscar Salvador :
> Hi again,
>
> I've split the patches as Karol Herbst suggested.
> I hope now it's fine.
>
> This series of patches introduce the
Hi Oscar,
this is a cover letter, there should be no patch attached to it, but a
stat of the following patches and an explenation what the series is
doing.
2017-04-13 11:07 GMT+02:00 Oscar Salvador :
> Hi again,
>
> I've split the patches as Karol Herbst suggested.
Just a non-technical (but important!) point: if you send four patches,
make sure their title (the first line of your git commit messages)
describes the content. Given you re-used the same title four times, I
dare to say it's too generic.
Also, I noticed that this is not the standard git patch
https://bugs.freedesktop.org/show_bug.cgi?id=100676
Bug ID: 100676
Summary: gp104 - messy(random green and/or missing pixel lines)
then no output at and after boot
Product: xorg
Version: git
Hardware: x86-64 (AMD64)
https://bugs.freedesktop.org/show_bug.cgi?id=100567
--- Comment #4 from kevin@potatofrom.space ---
Created attachment 130834
--> https://bugs.freedesktop.org/attachment.cgi?id=130834=edit
dmesg with SCHED_ERROR (starting at 73576 sec)
It happened again! I believe the monitors were turned off
https://bugs.freedesktop.org/show_bug.cgi?id=100423
Arthur Borsboom changed:
What|Removed |Added
OS|All |Linux (All)
This patch introduces the structure "struct hwmon_ops" and sets up the
".visible" operation.
Is also a preparation for the next patch.
--- linux/drivers/gpu/drm/nouveau/nouveau_hwmon.c.orig 2017-04-12
19:18:09.638073562 +0200
+++ linux/drivers/gpu/drm/nouveau/nouveau_hwmon.c 2017-04-12
This patch replaces symbolic permissions with the numeric ones and adds me to
the authors too.
--- linux/drivers/gpu/drm/nouveau/nouveau_hwmon.c.orig 2017-04-13
10:18:37.471129756 +0200
+++ linux/drivers/gpu/drm/nouveau/nouveau_hwmon.c 2017-04-13
10:19:58.182025638 +0200
@@ -1,5 +1,6 @@
This patch creates special group attributes for special attrs like
"*auto_point*".
We check if we need them, and if we do, we set them up in special_groups
structure, that then
we pass to hwmon_device_register_with_info.
--- linux/drivers/gpu/drm/nouveau/nouveau_hwmon.c.orig 2017-04-13
Here is where most of the work is being done.
We are replacing the old API with the new one, that means changing the
functions layout and remove unnecessary code.
We also set up the other operations:
.read
.write
.read_string
--- linux/drivers/gpu/drm/nouveau/nouveau_hwmon.c.orig 2017-04-13
This patch introduces the structure "struct hwmon_ops" and sets up the
".visible" operation.
Is also a preparation for the next patch where all work is being done.
--- linux/drivers/gpu/drm/nouveau/nouveau_hwmon.c.orig 2017-04-12
19:22:29.070573187 +0200
+++
Hi again,
I've split the patches as Karol Herbst suggested.
I hope now it's fine.
This series of patches introduce the new hwmon_device_register_with_info
and gets rid of the old hwmon_device_register.
This patch adds the default sensors with their possible config values.
Just to prepare for
On 04/13/2017 01:22 AM, Alastair Bridgewater wrote:
On Tue, Apr 11, 2017 at 1:32 PM, Ilia Mirkin > wrote:
On Tue, Apr 11, 2017 at 1:11 PM, Alastair Bridgewater
>
16 matches
Mail list logo