Re: [PATCH 1/2] acpi/dptf: document sysfs atttributes
On Fri, 2018-02-09 at 11:31 +0100, Rafael J. Wysocki wrote: > On Wed, Feb 7, 2018 at 2:18 PM, Aishwarya Pant> wrote: > > The descriptions have been collected from git commit logs and > > reading > > through code. > > > > Signed-off-by: Aishwarya Pant > > --- > > Documentation/ABI/testing/sysfs-platform-dptf | 30 > > +++ > > 1 file changed, 30 insertions(+) > > create mode 100644 Documentation/ABI/testing/sysfs-platform-dptf > > > > diff --git a/Documentation/ABI/testing/sysfs-platform-dptf > > b/Documentation/ABI/testing/sysfs-platform-dptf > > new file mode 100644 > > index ..e8168e5198d7 > > --- /dev/null > > +++ b/Documentation/ABI/testing/sysfs-platform-dptf > > @@ -0,0 +1,30 @@ > > +sysfs interface for ACPI DPTF platform power driver > > +--- > > The header isn't necessary. > > > + > > +What: /sys/bus/platform/devices/INT3407:00/dptf_power/cha > > rger_type > > +What: /sys/bus/platform/devices/INT3407:00/dptf_power/ada > > pter_rating_mw > > +What: /sys/bus/platform/devices/INT3407:00/dptf_power/max > > _platform_power_mw > > +What: /sys/bus/platform/devices/INT3407:00/dptf_power/pla > > tform_power_source > > +What: /sys/bus/platform/devices/INT3407:00/dptf_power/bat > > tery_steady_power > > There should be one entry per attribute AFAICS. > > Also I need Srinivas to tell me whether or not the descriptions are > accurate enough. Descriptions are correct. Thanks, Srinivas > > > +Date: Jul, 2016 > > +KernelVersion: v4.10 > > +Contact: linux-a...@vger.kernel.org > > +Description: > > + charger_type: (RO) The charger type- > > Traditional, > > + Hybrid or NVDC. > > + > > + adapter_rating_mw: (RO) Adapter rating in > > milliwatts (the > > + maximum Adapter power). > > Must be 0 if no > > + AC Adaptor is plugged in. > > + > > + max_platform_power_mw: (RO) Maximum platform power > > that can be > > + supported by the battery in > > milliwatts. > > + > > + platform_power_source: (RO) Display the platform > > power source > > + 0x00 = DC > > + 0x01 = AC > > + 0x02 = USB > > + 0x03 = Wireless Charger > > + > > + battery_steady_power_mw:(RO) The maximum sustained > > power for > > + battery in milliwatts. > > --
Re: [PATCH 1/2] acpi/dptf: document sysfs atttributes
On Fri, 2018-02-09 at 11:31 +0100, Rafael J. Wysocki wrote: > On Wed, Feb 7, 2018 at 2:18 PM, Aishwarya Pant > wrote: > > The descriptions have been collected from git commit logs and > > reading > > through code. > > > > Signed-off-by: Aishwarya Pant > > --- > > Documentation/ABI/testing/sysfs-platform-dptf | 30 > > +++ > > 1 file changed, 30 insertions(+) > > create mode 100644 Documentation/ABI/testing/sysfs-platform-dptf > > > > diff --git a/Documentation/ABI/testing/sysfs-platform-dptf > > b/Documentation/ABI/testing/sysfs-platform-dptf > > new file mode 100644 > > index ..e8168e5198d7 > > --- /dev/null > > +++ b/Documentation/ABI/testing/sysfs-platform-dptf > > @@ -0,0 +1,30 @@ > > +sysfs interface for ACPI DPTF platform power driver > > +--- > > The header isn't necessary. > > > + > > +What: /sys/bus/platform/devices/INT3407:00/dptf_power/cha > > rger_type > > +What: /sys/bus/platform/devices/INT3407:00/dptf_power/ada > > pter_rating_mw > > +What: /sys/bus/platform/devices/INT3407:00/dptf_power/max > > _platform_power_mw > > +What: /sys/bus/platform/devices/INT3407:00/dptf_power/pla > > tform_power_source > > +What: /sys/bus/platform/devices/INT3407:00/dptf_power/bat > > tery_steady_power > > There should be one entry per attribute AFAICS. > > Also I need Srinivas to tell me whether or not the descriptions are > accurate enough. Descriptions are correct. Thanks, Srinivas > > > +Date: Jul, 2016 > > +KernelVersion: v4.10 > > +Contact: linux-a...@vger.kernel.org > > +Description: > > + charger_type: (RO) The charger type- > > Traditional, > > + Hybrid or NVDC. > > + > > + adapter_rating_mw: (RO) Adapter rating in > > milliwatts (the > > + maximum Adapter power). > > Must be 0 if no > > + AC Adaptor is plugged in. > > + > > + max_platform_power_mw: (RO) Maximum platform power > > that can be > > + supported by the battery in > > milliwatts. > > + > > + platform_power_source: (RO) Display the platform > > power source > > + 0x00 = DC > > + 0x01 = AC > > + 0x02 = USB > > + 0x03 = Wireless Charger > > + > > + battery_steady_power_mw:(RO) The maximum sustained > > power for > > + battery in milliwatts. > > --
Re: [PATCH 1/2] acpi/dptf: document sysfs atttributes
On Fri, Feb 9, 2018 at 11:54 AM, Aishwarya Pantwrote: > On Fri, Feb 09, 2018 at 11:31:35AM +0100, Rafael J. Wysocki wrote: >> On Wed, Feb 7, 2018 at 2:18 PM, Aishwarya Pant wrote: >> > The descriptions have been collected from git commit logs and reading >> > through code. >> > >> > Signed-off-by: Aishwarya Pant >> > --- >> > Documentation/ABI/testing/sysfs-platform-dptf | 30 >> > +++ >> > 1 file changed, 30 insertions(+) >> > create mode 100644 Documentation/ABI/testing/sysfs-platform-dptf >> > >> > diff --git a/Documentation/ABI/testing/sysfs-platform-dptf >> > b/Documentation/ABI/testing/sysfs-platform-dptf >> > new file mode 100644 >> > index ..e8168e5198d7 >> > --- /dev/null >> > +++ b/Documentation/ABI/testing/sysfs-platform-dptf >> > @@ -0,0 +1,30 @@ >> > +sysfs interface for ACPI DPTF platform power driver >> > +--- >> >> The header isn't necessary. >> >> > + >> > +What: >> > /sys/bus/platform/devices/INT3407:00/dptf_power/charger_type >> > +What: >> > /sys/bus/platform/devices/INT3407:00/dptf_power/adapter_rating_mw >> > +What: >> > /sys/bus/platform/devices/INT3407:00/dptf_power/max_platform_power_mw >> > +What: >> > /sys/bus/platform/devices/INT3407:00/dptf_power/platform_power_source >> > +What: >> > /sys/bus/platform/devices/INT3407:00/dptf_power/battery_steady_power >> >> There should be one entry per attribute AFAICS. > > I grouped attributes by the date in which they were added for brevity, because > otherwise fields like Date added, KernelVersion and Contact information are > the > same. > > I can split them up if it makes more sense... That's what the format is from looking at the existing files.
Re: [PATCH 1/2] acpi/dptf: document sysfs atttributes
On Fri, Feb 9, 2018 at 11:54 AM, Aishwarya Pant wrote: > On Fri, Feb 09, 2018 at 11:31:35AM +0100, Rafael J. Wysocki wrote: >> On Wed, Feb 7, 2018 at 2:18 PM, Aishwarya Pant wrote: >> > The descriptions have been collected from git commit logs and reading >> > through code. >> > >> > Signed-off-by: Aishwarya Pant >> > --- >> > Documentation/ABI/testing/sysfs-platform-dptf | 30 >> > +++ >> > 1 file changed, 30 insertions(+) >> > create mode 100644 Documentation/ABI/testing/sysfs-platform-dptf >> > >> > diff --git a/Documentation/ABI/testing/sysfs-platform-dptf >> > b/Documentation/ABI/testing/sysfs-platform-dptf >> > new file mode 100644 >> > index ..e8168e5198d7 >> > --- /dev/null >> > +++ b/Documentation/ABI/testing/sysfs-platform-dptf >> > @@ -0,0 +1,30 @@ >> > +sysfs interface for ACPI DPTF platform power driver >> > +--- >> >> The header isn't necessary. >> >> > + >> > +What: >> > /sys/bus/platform/devices/INT3407:00/dptf_power/charger_type >> > +What: >> > /sys/bus/platform/devices/INT3407:00/dptf_power/adapter_rating_mw >> > +What: >> > /sys/bus/platform/devices/INT3407:00/dptf_power/max_platform_power_mw >> > +What: >> > /sys/bus/platform/devices/INT3407:00/dptf_power/platform_power_source >> > +What: >> > /sys/bus/platform/devices/INT3407:00/dptf_power/battery_steady_power >> >> There should be one entry per attribute AFAICS. > > I grouped attributes by the date in which they were added for brevity, because > otherwise fields like Date added, KernelVersion and Contact information are > the > same. > > I can split them up if it makes more sense... That's what the format is from looking at the existing files.
Re: [PATCH 1/2] acpi/dptf: document sysfs atttributes
On Fri, Feb 09, 2018 at 11:31:35AM +0100, Rafael J. Wysocki wrote: > On Wed, Feb 7, 2018 at 2:18 PM, Aishwarya Pantwrote: > > The descriptions have been collected from git commit logs and reading > > through code. > > > > Signed-off-by: Aishwarya Pant > > --- > > Documentation/ABI/testing/sysfs-platform-dptf | 30 > > +++ > > 1 file changed, 30 insertions(+) > > create mode 100644 Documentation/ABI/testing/sysfs-platform-dptf > > > > diff --git a/Documentation/ABI/testing/sysfs-platform-dptf > > b/Documentation/ABI/testing/sysfs-platform-dptf > > new file mode 100644 > > index ..e8168e5198d7 > > --- /dev/null > > +++ b/Documentation/ABI/testing/sysfs-platform-dptf > > @@ -0,0 +1,30 @@ > > +sysfs interface for ACPI DPTF platform power driver > > +--- > > The header isn't necessary. > > > + > > +What: /sys/bus/platform/devices/INT3407:00/dptf_power/charger_type > > +What: > > /sys/bus/platform/devices/INT3407:00/dptf_power/adapter_rating_mw > > +What: > > /sys/bus/platform/devices/INT3407:00/dptf_power/max_platform_power_mw > > +What: > > /sys/bus/platform/devices/INT3407:00/dptf_power/platform_power_source > > +What: > > /sys/bus/platform/devices/INT3407:00/dptf_power/battery_steady_power > > There should be one entry per attribute AFAICS. I grouped attributes by the date in which they were added for brevity, because otherwise fields like Date added, KernelVersion and Contact information are the same. I can split them up if it makes more sense... Aishwarya > > Also I need Srinivas to tell me whether or not the descriptions are > accurate enough. > > > +Date: Jul, 2016 > > +KernelVersion: v4.10 > > +Contact: linux-a...@vger.kernel.org > > +Description: > > + charger_type: (RO) The charger type- Traditional, > > + Hybrid or NVDC. > > + > > + adapter_rating_mw: (RO) Adapter rating in milliwatts > > (the > > + maximum Adapter power). Must be 0 > > if no > > + AC Adaptor is plugged in. > > + > > + max_platform_power_mw: (RO) Maximum platform power that > > can be > > + supported by the battery in > > milliwatts. > > + > > + platform_power_source: (RO) Display the platform power > > source > > + 0x00 = DC > > + 0x01 = AC > > + 0x02 = USB > > + 0x03 = Wireless Charger > > + > > + battery_steady_power_mw:(RO) The maximum sustained power for > > + battery in milliwatts. > > --
Re: [PATCH 1/2] acpi/dptf: document sysfs atttributes
On Fri, Feb 09, 2018 at 11:31:35AM +0100, Rafael J. Wysocki wrote: > On Wed, Feb 7, 2018 at 2:18 PM, Aishwarya Pant wrote: > > The descriptions have been collected from git commit logs and reading > > through code. > > > > Signed-off-by: Aishwarya Pant > > --- > > Documentation/ABI/testing/sysfs-platform-dptf | 30 > > +++ > > 1 file changed, 30 insertions(+) > > create mode 100644 Documentation/ABI/testing/sysfs-platform-dptf > > > > diff --git a/Documentation/ABI/testing/sysfs-platform-dptf > > b/Documentation/ABI/testing/sysfs-platform-dptf > > new file mode 100644 > > index ..e8168e5198d7 > > --- /dev/null > > +++ b/Documentation/ABI/testing/sysfs-platform-dptf > > @@ -0,0 +1,30 @@ > > +sysfs interface for ACPI DPTF platform power driver > > +--- > > The header isn't necessary. > > > + > > +What: /sys/bus/platform/devices/INT3407:00/dptf_power/charger_type > > +What: > > /sys/bus/platform/devices/INT3407:00/dptf_power/adapter_rating_mw > > +What: > > /sys/bus/platform/devices/INT3407:00/dptf_power/max_platform_power_mw > > +What: > > /sys/bus/platform/devices/INT3407:00/dptf_power/platform_power_source > > +What: > > /sys/bus/platform/devices/INT3407:00/dptf_power/battery_steady_power > > There should be one entry per attribute AFAICS. I grouped attributes by the date in which they were added for brevity, because otherwise fields like Date added, KernelVersion and Contact information are the same. I can split them up if it makes more sense... Aishwarya > > Also I need Srinivas to tell me whether or not the descriptions are > accurate enough. > > > +Date: Jul, 2016 > > +KernelVersion: v4.10 > > +Contact: linux-a...@vger.kernel.org > > +Description: > > + charger_type: (RO) The charger type- Traditional, > > + Hybrid or NVDC. > > + > > + adapter_rating_mw: (RO) Adapter rating in milliwatts > > (the > > + maximum Adapter power). Must be 0 > > if no > > + AC Adaptor is plugged in. > > + > > + max_platform_power_mw: (RO) Maximum platform power that > > can be > > + supported by the battery in > > milliwatts. > > + > > + platform_power_source: (RO) Display the platform power > > source > > + 0x00 = DC > > + 0x01 = AC > > + 0x02 = USB > > + 0x03 = Wireless Charger > > + > > + battery_steady_power_mw:(RO) The maximum sustained power for > > + battery in milliwatts. > > --
Re: [PATCH 1/2] acpi/dptf: document sysfs atttributes
On Wed, Feb 7, 2018 at 2:18 PM, Aishwarya Pantwrote: > The descriptions have been collected from git commit logs and reading > through code. > > Signed-off-by: Aishwarya Pant > --- > Documentation/ABI/testing/sysfs-platform-dptf | 30 > +++ > 1 file changed, 30 insertions(+) > create mode 100644 Documentation/ABI/testing/sysfs-platform-dptf > > diff --git a/Documentation/ABI/testing/sysfs-platform-dptf > b/Documentation/ABI/testing/sysfs-platform-dptf > new file mode 100644 > index ..e8168e5198d7 > --- /dev/null > +++ b/Documentation/ABI/testing/sysfs-platform-dptf > @@ -0,0 +1,30 @@ > +sysfs interface for ACPI DPTF platform power driver > +--- The header isn't necessary. > + > +What: /sys/bus/platform/devices/INT3407:00/dptf_power/charger_type > +What: > /sys/bus/platform/devices/INT3407:00/dptf_power/adapter_rating_mw > +What: > /sys/bus/platform/devices/INT3407:00/dptf_power/max_platform_power_mw > +What: > /sys/bus/platform/devices/INT3407:00/dptf_power/platform_power_source > +What: > /sys/bus/platform/devices/INT3407:00/dptf_power/battery_steady_power There should be one entry per attribute AFAICS. Also I need Srinivas to tell me whether or not the descriptions are accurate enough. > +Date: Jul, 2016 > +KernelVersion: v4.10 > +Contact: linux-a...@vger.kernel.org > +Description: > + charger_type: (RO) The charger type- Traditional, > + Hybrid or NVDC. > + > + adapter_rating_mw: (RO) Adapter rating in milliwatts (the > + maximum Adapter power). Must be 0 if > no > + AC Adaptor is plugged in. > + > + max_platform_power_mw: (RO) Maximum platform power that can > be > + supported by the battery in > milliwatts. > + > + platform_power_source: (RO) Display the platform power source > + 0x00 = DC > + 0x01 = AC > + 0x02 = USB > + 0x03 = Wireless Charger > + > + battery_steady_power_mw:(RO) The maximum sustained power for > + battery in milliwatts. > --
Re: [PATCH 1/2] acpi/dptf: document sysfs atttributes
On Wed, Feb 7, 2018 at 2:18 PM, Aishwarya Pant wrote: > The descriptions have been collected from git commit logs and reading > through code. > > Signed-off-by: Aishwarya Pant > --- > Documentation/ABI/testing/sysfs-platform-dptf | 30 > +++ > 1 file changed, 30 insertions(+) > create mode 100644 Documentation/ABI/testing/sysfs-platform-dptf > > diff --git a/Documentation/ABI/testing/sysfs-platform-dptf > b/Documentation/ABI/testing/sysfs-platform-dptf > new file mode 100644 > index ..e8168e5198d7 > --- /dev/null > +++ b/Documentation/ABI/testing/sysfs-platform-dptf > @@ -0,0 +1,30 @@ > +sysfs interface for ACPI DPTF platform power driver > +--- The header isn't necessary. > + > +What: /sys/bus/platform/devices/INT3407:00/dptf_power/charger_type > +What: > /sys/bus/platform/devices/INT3407:00/dptf_power/adapter_rating_mw > +What: > /sys/bus/platform/devices/INT3407:00/dptf_power/max_platform_power_mw > +What: > /sys/bus/platform/devices/INT3407:00/dptf_power/platform_power_source > +What: > /sys/bus/platform/devices/INT3407:00/dptf_power/battery_steady_power There should be one entry per attribute AFAICS. Also I need Srinivas to tell me whether or not the descriptions are accurate enough. > +Date: Jul, 2016 > +KernelVersion: v4.10 > +Contact: linux-a...@vger.kernel.org > +Description: > + charger_type: (RO) The charger type- Traditional, > + Hybrid or NVDC. > + > + adapter_rating_mw: (RO) Adapter rating in milliwatts (the > + maximum Adapter power). Must be 0 if > no > + AC Adaptor is plugged in. > + > + max_platform_power_mw: (RO) Maximum platform power that can > be > + supported by the battery in > milliwatts. > + > + platform_power_source: (RO) Display the platform power source > + 0x00 = DC > + 0x01 = AC > + 0x02 = USB > + 0x03 = Wireless Charger > + > + battery_steady_power_mw:(RO) The maximum sustained power for > + battery in milliwatts. > --