All of lore.kernel.org
 help / color / mirror / Atom feed
From: Delan Azabani <delan-Boohn7ZbvbpBDgjK7y7TUQ@public.gmane.org>
To: nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
Subject: Re: hwmon directory structure bug
Date: Sat, 06 Nov 2010 11:47:29 +0800	[thread overview]
Message-ID: <4CD4CFD1.3020302@azabani.com> (raw)
In-Reply-To: <4CD4BE30.4040409-GANU6spQydw@public.gmane.org>

On 06/11/10 10:32, Martin Peres wrote:
> Le 06/11/2010 01:34, Delan Azabani a écrit :
>> The sensors implementation of nouveau has a bug where the name file is
>> not in hwmonX/device, but in the root of the hwmonX directory. Normally,
>> the name file (contains driver name) of a sensor is placed:
>>
>> /sys/class/hwmon/hwmon<int>/device/name
>>
>> This holds true for coretemp, it8718 and all other sensors I've
>> encountered. Currently, with the latest nouveau from git, the name file
>> is placed as
>>
>> /sys/class/hwmon/hwmon<int>/name
>>
>> This is incorrect and goes against the convention for hwmon directory
>> structure. Not a major bug, but a fix would be very helpful.
> oh, thanks for spotting this, I'll correct my mistake tomorrow.
> 
> Regards,
> 
> Martin
> _______________________________________________
> Nouveau mailing list
> Nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
> http://lists.freedesktop.org/mailman/listinfo/nouveau

No problem, happy to help.

Thanks and best regards,
Delan Azabani

      parent reply	other threads:[~2010-11-06  3:47 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-06  0:34 hwmon directory structure bug Delan Azabani
     [not found] ` <4CD4A293.5030502-Boohn7ZbvbpBDgjK7y7TUQ@public.gmane.org>
2010-11-06  2:32   ` Martin Peres
     [not found]     ` <4CD4BE30.4040409-GANU6spQydw@public.gmane.org>
2010-11-06  3:47       ` Delan Azabani [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4CD4CFD1.3020302@azabani.com \
    --to=delan-boohn7zbvbpbdgjk7y7tuq@public.gmane.org \
    --cc=nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.