On 03/03/2025 09:20, Svyatoslav Ryhel wrote: > пн, 3 бер. 2025 р. о 10:18 Krzysztof Kozlowski <krzk@xxxxxxxxxx> пише: >> >> On 03/03/2025 09:11, Svyatoslav Ryhel wrote: >>> пн, 3 бер. 2025 р. о 09:54 Krzysztof Kozlowski <krzk@xxxxxxxxxx> пише: >>>> >>>> On 27/02/2025 11:55, Svyatoslav Ryhel wrote: >>>>>>> + >>>> >>>> Please kindly trim the replies from unnecessary context. It makes it >>>> much easier to find new content. >>>> >>>>>>> + maxim,usb-in-current-limit-microamp: >>>>>>> + description: >>>>>>> + USB Input current limit >>>>>>> + minimum: 100000 >>>>>>> + default: 500000 >>>>>>> + maximum: 1500000 >>>>>>> + >>>>>>> + maxim,ac-in-current-limit-microamp: >>>>>>> + description: >>>>>>> + AC Input current limit >>>>>>> + minimum: 100000 >>>>>>> + default: 500000 >>>>>>> + maximum: 1500000 >>>>>> >>>>>> Half of these properties as well are not suitable and duplicate existing >>>>>> sysfs interface. >>>>>> >>>>> >>>>> All these properties allow configure the charger to suit the device on >>>>> which it is used. None of them are required but are a nice addition. >>>>> Why you are denying me an ability to fully utilize hardware I have and >>>>> tune it to the device? All those values represent hardware registers >>>>> which can be customized for the device, not for the end user to mess >>>>> with. >>>> >>>> Because you put user-space choice or OS policy into the DT and DT is not >>>> for that. >>>> >>> >>> Those are NOT user-space choice or OS policy those are vendor >>> configuration for a specific device and are NOT and NEVER were exposed >> >> Then look at existing devices. We had these discussions in the past and >> these are usually exposed to user-space. >> > > Provide an example, where there is same or similar configuration. If you tried even a bit, you would easily find them by grep for timer/hours/minutes. You do not even try but put this work on maintainer. Do the homework and try a bit harder instead of pushing this on me. Best regards, Krzysztof
