Re: [PATCH 0/5] Remove 0 MHz as a valid current frequency (v4)

2021-10-19 Thread Paul Menzel
Dear Luben, Thank you for your quick reply. Am 19.10.21 um 09:43 schrieb Luben Tuikov: On 2021-10-19 03:23, Paul Menzel wrote: Sorry, two more style nits. 1. Could you please use 75 characters per line for the text width of the commit messages. Currently, especially 4/5, are hard to read

Re: [PATCH 0/5] Remove 0 MHz as a valid current frequency (v4)

2021-10-19 Thread Luben Tuikov
On 2021-10-19 03:23, Paul Menzel wrote: > Dear Luben, > > > Sorry, two more style nits. > > 1. Could you please use 75 characters per line for the text width of > the commit messages. Currently, especially 4/5, are hard to read being > so short. This is the default we use--I've not made any

Re: [PATCH 0/5] Remove 0 MHz as a valid current frequency (v4)

2021-10-19 Thread Paul Menzel
Dear Luben, Sorry, two more style nits. 1. Could you please use 75 characters per line for the text width of the commit messages. Currently, especially 4/5, are hard to read being so short. 2. No idea, what is done in amd-gfx, but for me it is more common to put the iteration number

[PATCH 0/5] Remove 0 MHz as a valid current frequency (v4)

2021-10-18 Thread Luben Tuikov
Some ASICs support low-power functionality for the whole ASIC or just an IP block. When in such low-power mode, some sysfs interfaces would report a frequency of 0, e.g., $cat /sys/class/drm/card0/device/pp_dpm_sclk 0: 500Mhz 1: 0Mhz * 2: 2200Mhz $_ An operating frequency of 0 MHz doesn't make