Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1449019 > unrolled thread

hwmon: xgene: Add support for X-Gene hwmon driver

Started byItaru Kitayama <itaru.kitayama@riken.jp>
First post2016-07-24 02:00 +0200
Last post2016-07-24 08:40 +0200
Articles 5 — 3 participants

Back to article view | Back to linux.kernel


Contents

  hwmon: xgene: Add support for X-Gene hwmon driver Itaru Kitayama <itaru.kitayama@riken.jp> - 2016-07-24 02:00 +0200
    Re: hwmon: xgene: Add support for X-Gene hwmon driver Hoan Tran <hotran@apm.com> - 2016-07-24 05:10 +0200
      Re: hwmon: xgene: Add support for X-Gene hwmon driver Itaru Kitayama <itaru.kitayama@riken.jp> - 2016-07-24 07:10 +0200
        Re: hwmon: xgene: Add support for X-Gene hwmon driver Hoan Tran <hotran@apm.com> - 2016-07-24 08:00 +0200
        Re: hwmon: xgene: Add support for X-Gene hwmon driver Loc Ho <lho@apm.com> - 2016-07-24 08:40 +0200

#1449019 — hwmon: xgene: Add support for X-Gene hwmon driver

FromItaru Kitayama <itaru.kitayama@riken.jp>
Date2016-07-24 02:00 +0200
Subjecthwmon: xgene: Add support for X-Gene hwmon driver
Message-ID<rYeZP-y9-9@gated-at.bofh.it>
Hi Hoan,

I've been testing your patch set on a Rev B0 system with ACPI, in dmesg
I see:

[    1.546444] xgene-slimpro-i2c APMC0D40:00: i2c mailbox channel 
request failed
[    1.570062] xgene-slimpro-hwmon APMC0D29:00: no pcc-channel property

what other packages or subsystems do we need to make this work on 
ACPI-based systems?

Itaru

[toc] | [next] | [standalone]


#1449038

FromHoan Tran <hotran@apm.com>
Date2016-07-24 05:10 +0200
Message-ID<rYhXH-2zD-7@gated-at.bofh.it>
In reply to#1449019
Hi Itaru,


> On Jul 23, 2016, at 16:51, Itaru Kitayama <itaru.kitayama@riken.jp> wrote:
> 
> Hi Hoan,
> 
> I've been testing your patch set on a Rev B0 system with ACPI, in dmesg
> I see:
> 
> [    1.546444] xgene-slimpro-i2c APMC0D40:00: i2c mailbox channel request failed
> [    1.570062] xgene-slimpro-hwmon APMC0D29:00: no pcc-channel property
> 
> what other packages or subsystems do we need to make this work on ACPI-based systems?

You need the new firmware and BIOS which support ACPI PCC mailbox.

Thanks
Hoan

> 
> Itaru

[toc] | [prev] | [next] | [standalone]


#1449051

FromItaru Kitayama <itaru.kitayama@riken.jp>
Date2016-07-24 07:10 +0200
Message-ID<rYjPP-3JA-3@gated-at.bofh.it>
In reply to#1449038
Hi Hoan,

I didn't mention in my previous post, though I think I'm using fairly 
recent version of Tianocore F/W, and I can confirm there's the PCCT table,

[    0.000000] ACPI: PCCT 0x00000047FA826000 000300 (v01 APM    XGENE 
00000003      01000013)

is this still not enough? If that's the case, I'll talk to APM for
their latest development Tianocore version.
Also, if you or your folks can fold your patch set into APM's xgene-next
GitHub repo, that would help me to keep up with the development.

Itaru

On 7/24/16 12:00 PM, Hoan Tran wrote:
> You need the new firmware and BIOS which support ACPI PCC mailbox.
>
> Thanks
> Hoan

[toc] | [prev] | [next] | [standalone]


#1449068

FromHoan Tran <hotran@apm.com>
Date2016-07-24 08:00 +0200
Message-ID<rYkCe-402-15@gated-at.bofh.it>
In reply to#1449051
Hi Itaru,

On Sat, Jul 23, 2016 at 10:04 PM, Itaru Kitayama
<itaru.kitayama@riken.jp> wrote:
> Hi Hoan,
>
> I didn't mention in my previous post, though I think I'm using fairly recent
> version of Tianocore F/W, and I can confirm there's the PCCT table,
>
> [    0.000000] ACPI: PCCT 0x00000047FA826000 000300 (v01 APM    XGENE
> 00000003      01000013)
>
> is this still not enough? If that's the case, I'll talk to APM for
> their latest development Tianocore version.

Your version is missing the "pcc-channel" property of hwmon inside
ACPI Table. You need a newer version of Tianocore.

> Also, if you or your folks can fold your patch set into APM's xgene-next
> GitHub repo, that would help me to keep up with the development.

xgene-next is currently used for device tree patch only.

Thanks
Hoan

>
> Itaru
>
>
> On 7/24/16 12:00 PM, Hoan Tran wrote:
>>
>> You need the new firmware and BIOS which support ACPI PCC mailbox.
>>
>> Thanks
>> Hoan

[toc] | [prev] | [next] | [standalone]


#1449076

FromLoc Ho <lho@apm.com>
Date2016-07-24 08:40 +0200
Message-ID<rYleV-4z8-1@gated-at.bofh.it>
In reply to#1449051
Hi Itaru,

On Sat, Jul 23, 2016 at 10:04 PM, Itaru Kitayama
<itaru.kitayama@riken.jp> wrote:
>
> Hi Hoan,
>
> I didn't mention in my previous post, though I think I'm using fairly recent version of Tianocore F/W, and I can confirm there's the PCCT table,
>
> [    0.000000] ACPI: PCCT 0x00000047FA826000 000300 (v01 APM    XGENE 00000003      01000013)
>
> is this still not enough? If that's the case, I'll talk to APM for
> their latest development Tianocore version.

You will need the to be released FW version 3.06.xx or above. This new
FW won't be official release until at least a few months. If you want
to test this new firmware, we can get you an earlier image but there
will be very limited support. Though, Hoan will support you when
possible.

> Also, if you or your folks can fold your patch set into APM's xgene-next
> GitHub repo, that would help me to keep up with the development.

This can only get pull into xgene-next if it had accepted upstream.

-Loc

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web