On Mon, Apr 05, 2021 at 09:44:01AM -0300, Luiz Sampaio wrote: > On Mon, Apr 05, 2021 at 01:04:59PM +0200, Greg KH wrote: > > On Mon, Apr 05, 2021 at 07:50:09AM -0300, Luiz Sampaio wrote: > > > Added a sysfs entry to support writing to the offset register on page1. > > > This register is used to calibrate the chip canceling offset errors in the > > > current ADC. This means that, over time, reading the IAD register will not > > > return the correct current measurement, it will have an offset. Writing to > > > the offset register if the two's complement of the current register while > > > passing zero current to the load will calibrate the measurements. This > > > change was tested on real hardware and it was able to calibrate the chip > > > correctly. > > > > > > Signed-off-by: Luiz Sampaio <sampaio....@gmail.com> > > > --- > > > Documentation/w1/slaves/w1_ds2438.rst | 11 +++++- > > > drivers/w1/slaves/w1_ds2438.c | 49 +++++++++++++++++++++++++++ > > > 2 files changed, 59 insertions(+), 1 deletion(-) > > > > In this, and the previous patch, you added new sysfs files, but no > > update to Documentation/ABI/ for them. Please fix that up. > > > > thanks, > > > > greg k-h > > Hello! I'm sorry about some errors, this is my first patch and I'm not sure > about some things in the documentation. I really appreciate the responses and > guidance.
No problem, again you should try wrapping your email lines :) > The file I need to add to Documentation/ABI/ is going to be in testing or > stable? And the file I need to create can be called, for instance, > sysfs-driver-w1_ds2438? stable I guess as you know this is what you want to do. And the file name seems right, thanks. greg k-h
