Re: [Linuxptp-devel] [PATCH] Fix power profile config option range to UINT32_MAX

2023-05-11 Thread Miroslav Lichvar
On Wed, May 10, 2023 at 06:57:26PM -0700, Richard Cochran wrote: > On Sun, Apr 30, 2023 at 01:50:34PM -0700, Richard Cochran wrote: > > Thanks for finding this. I'd like to fix it in a simpler way. > > FWIW I am going ahead with my patch. > > @Miroslav it changes the default value for one field,

Re: [Linuxptp-devel] [PATCH] Fix power profile config option range to UINT32_MAX

2023-05-10 Thread Richard Cochran
On Sun, Apr 30, 2023 at 01:50:34PM -0700, Richard Cochran wrote: > Thanks for finding this. I'd like to fix it in a simpler way. FWIW I am going ahead with my patch. @Miroslav it changes the default value for one field, and so the test suite would need the following change. Thanks, Richard ---

Re: [Linuxptp-devel] [PATCH] Fix power profile config option range to UINT32_MAX

2023-04-30 Thread Richard Cochran
On Mon, Apr 24, 2023 at 12:21:57PM -0700, Jacob Keller wrote: > The power profile configuration options added in commit 7059a05a3fb2 > ("Introduce the power profile.") specify their maximum range as INT_MAX. > The values are stored as UInteger32 values and the default value is > 0x. On most

Re: [Linuxptp-devel] [PATCH] Fix power profile config option range to UINT32_MAX

2023-04-13 Thread Jacob Keller
On 3/29/2023 1:42 PM, Erez wrote: > On Wed, 29 Mar 2023 at 20:36, Jacob Keller wrote: > >> The power profile configuration options added in commit 7059a05a3fb2 >> ("Introduce the power profile.") specify their maximum range as INT_MAX. >> The values are stored in UInteger32 values, and the def

Re: [Linuxptp-devel] [PATCH] Fix power profile config option range to UINT32_MAX

2023-03-29 Thread Erez
On Wed, 29 Mar 2023 at 20:36, Jacob Keller wrote: > The power profile configuration options added in commit 7059a05a3fb2 > ("Introduce the power profile.") specify their maximum range as INT_MAX. > The values are stored in UInteger32 values, and the default 0x is > outside the range of a