Message ID | 20201025005916.64747-1-luka.kovacic@sartura.hr |
---|---|
Headers | show |
Series | Add support for the IEI WT61P803 PUZZLE MCU | expand |
Hello Pavel, On Thu, Oct 29, 2020 at 7:01 PM Pavel Machek <pavel@ucw.cz> wrote: > > Hi! > > > +What: /sys/bus/serial/devices/.../iei_wt61p803_puzzle_core/power_status > > +Date: September 2020 > > +Contact: Luka Kovacic <luka.kovacic@sartura.hr> > > +Description: (RO) Power status indicates the host platform power on method. > > + Value mapping (bitwise list): > > + 0x80 - Null > > + 0x40 - Firmware flag > > + 0x20 - Power loss detection flag (powered off) > > + 0x10 - Power loss detection flag (AC mode) > > + 0x08 - Button power on > > + 0x04 - WOL power on > > + 0x02 - RTC alarm power on > > + 0x01 - AC recover power on > > It would be nice to put this into standard place somewhere. Many > machines will want to expose this information. As this is specific to this microcontroller and to how it encodes these values, I don't see a need to change this. This isn't used anywhere else. > > If not, at least spell out WoL, as it is not that common of acronym. Okay. > > > +What: /sys/bus/serial/devices/.../iei_wt61p803_puzzle_core/ac_recovery_status > > +Date: September 2020 > > +Contact: Luka Kovacic <luka.kovacic@sartura.hr> > > +Description: (RO) Host platform AC recovery status value > > I can not tell what this is from documentation... I'll expand the description. > > Best regards, > Pavel > > -- > http://www.livejournal.com/~pavelmachek Kind regards, Luka
Hello On 11/1/20 3:56 AM, Luka Kovacic wrote: > Hello Pavel, > > On Thu, Oct 29, 2020 at 7:01 PM Pavel Machek <pavel@ucw.cz> wrote: >> Hi! >> >>> +What: /sys/bus/serial/devices/.../iei_wt61p803_puzzle_core/power_status >>> +Date: September 2020 >>> +Contact: Luka Kovacic <luka.kovacic@sartura.hr> >>> +Description: (RO) Power status indicates the host platform power on method. >>> + Value mapping (bitwise list): >>> + 0x80 - Null >>> + 0x40 - Firmware flag >>> + 0x20 - Power loss detection flag (powered off) >>> + 0x10 - Power loss detection flag (AC mode) >>> + 0x08 - Button power on >>> + 0x04 - WOL power on >>> + 0x02 - RTC alarm power on >>> + 0x01 - AC recover power on >> It would be nice to put this into standard place somewhere. Many >> machines will want to expose this information. > As this is specific to this microcontroller and to how it encodes > these values, I don't see a need to change this. > This isn't used anywhere else. > >> If not, at least spell out WoL, as it is not that common of acronym. > Okay. WoL is a very common acronym especially in the networking space But the overall this section does not make sense The description says that it indicates platform power on method but what is NULL power on? There are flags for power loss detection. Does the RTC mean that the processor real time clock woke up the uC? Or that the internal RTC woke up the controller? And for the /sys/bus/serial/devices/.../iei_wt61p803_puzzle_core/ac_recovery_status what are those values? It seems like some ABI's are documented well with formats and others are just described without a format. For instance /sys/bus/serial/devices/.../iei_wt61p803_puzzle_core/version the format of this version is not described but /sys/bus/serial/devices/.../iei_wt61p803_puzzle_core/build_info is. Dan
On Mon 2020-11-02 12:29:59, Dan Murphy wrote: > Hello > > On 11/1/20 3:56 AM, Luka Kovacic wrote: > > Hello Pavel, > > > > On Thu, Oct 29, 2020 at 7:01 PM Pavel Machek <pavel@ucw.cz> wrote: > > > Hi! > > > > > > > +What: /sys/bus/serial/devices/.../iei_wt61p803_puzzle_core/power_status > > > > +Date: September 2020 > > > > +Contact: Luka Kovacic <luka.kovacic@sartura.hr> > > > > +Description: (RO) Power status indicates the host platform power on method. > > > > + Value mapping (bitwise list): > > > > + 0x80 - Null > > > > + 0x40 - Firmware flag > > > > + 0x20 - Power loss detection flag (powered off) > > > > + 0x10 - Power loss detection flag (AC mode) > > > > + 0x08 - Button power on > > > > + 0x04 - WOL power on > > > > + 0x02 - RTC alarm power on > > > > + 0x01 - AC recover power on > > > It would be nice to put this into standard place somewhere. Many > > > machines will want to expose this information. > > As this is specific to this microcontroller and to how it encodes > > these values, I don't see a need to change this. > > This isn't used anywhere else. > > > > > If not, at least spell out WoL, as it is not that common of acronym. > > Okay. > > WoL is a very common acronym especially in the networking space WoL is common. WOL is not. Better spell it out. Best regards, Pavel -- http://www.livejournal.com/~pavelmachek
Hello On 11/2/20 1:03 PM, Pavel Machek wrote: > On Mon 2020-11-02 12:29:59, Dan Murphy wrote: >> Hello >> >> On 11/1/20 3:56 AM, Luka Kovacic wrote: >>> Hello Pavel, >>> >>> On Thu, Oct 29, 2020 at 7:01 PM Pavel Machek <pavel@ucw.cz> wrote: >>>> Hi! >>>> >>>>> +What: /sys/bus/serial/devices/.../iei_wt61p803_puzzle_core/power_status >>>>> +Date: September 2020 >>>>> +Contact: Luka Kovacic <luka.kovacic@sartura.hr> >>>>> +Description: (RO) Power status indicates the host platform power on method. >>>>> + Value mapping (bitwise list): >>>>> + 0x80 - Null >>>>> + 0x40 - Firmware flag >>>>> + 0x20 - Power loss detection flag (powered off) >>>>> + 0x10 - Power loss detection flag (AC mode) >>>>> + 0x08 - Button power on >>>>> + 0x04 - WOL power on >>>>> + 0x02 - RTC alarm power on >>>>> + 0x01 - AC recover power on >>>> It would be nice to put this into standard place somewhere. Many >>>> machines will want to expose this information. >>> As this is specific to this microcontroller and to how it encodes >>> these values, I don't see a need to change this. >>> This isn't used anywhere else. >>> >>>> If not, at least spell out WoL, as it is not that common of acronym. >>> Okay. >> WoL is a very common acronym especially in the networking space > WoL is common. WOL is not. Better spell it out. Agreed. Especially if WOL does not mean Wake On Lan Dan
Hello, On Mon, Nov 2, 2020 at 7:30 PM Dan Murphy <dmurphy@ti.com> wrote: > > Hello > > On 11/1/20 3:56 AM, Luka Kovacic wrote: > > Hello Pavel, > > > > On Thu, Oct 29, 2020 at 7:01 PM Pavel Machek <pavel@ucw.cz> wrote: > >> Hi! > >> > >>> +What: /sys/bus/serial/devices/.../iei_wt61p803_puzzle_core/power_status > >>> +Date: September 2020 > >>> +Contact: Luka Kovacic <luka.kovacic@sartura.hr> > >>> +Description: (RO) Power status indicates the host platform power on method. > >>> + Value mapping (bitwise list): > >>> + 0x80 - Null > >>> + 0x40 - Firmware flag > >>> + 0x20 - Power loss detection flag (powered off) > >>> + 0x10 - Power loss detection flag (AC mode) > >>> + 0x08 - Button power on > >>> + 0x04 - WOL power on > >>> + 0x02 - RTC alarm power on > >>> + 0x01 - AC recover power on > >> It would be nice to put this into standard place somewhere. Many > >> machines will want to expose this information. > > As this is specific to this microcontroller and to how it encodes > > these values, I don't see a need to change this. > > This isn't used anywhere else. > > > >> If not, at least spell out WoL, as it is not that common of acronym. > > Okay. > > WoL is a very common acronym especially in the networking space By WOL I meant Wake-on-LAN, I will spell out the whole acronym. > > But the overall this section does not make sense > > The description says that it indicates platform power on method but what > is NULL power on? There are flags for power loss detection. I will clarify the value mapping and try to replicate some of these states so I can write a better description. > > Does the RTC mean that the processor real time clock woke up the uC? Or > that the internal RTC woke up the controller? These are all related to the platform as a whole. So the Marvell SoC and all of the required peripherals are turned on. > > And for the > /sys/bus/serial/devices/.../iei_wt61p803_puzzle_core/ac_recovery_status > what are those values? These values indicate whether the board has been shut down gracefully and whether it has been powered on automatically (when power came back) or by pressing the power button. I will also extend the documentation with the value mapping for this. > > It seems like some ABI's are documented well with formats and others are > just described without a format. > > For instance > > /sys/bus/serial/devices/.../iei_wt61p803_puzzle_core/version the format > of this version is not described but > /sys/bus/serial/devices/.../iei_wt61p803_puzzle_core/build_info is. I left out the version format descriptions as they are in the recognizable format and all of them are quite arbitrary (e.g. v1.000). > > > Dan > Kind regards, Luka