Re: ncpXXxh103 compensation values?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Maxime,

This data was calculated for a Murata NCP15XH103XXXXX.

You have illustrated something to me with your question though. When I
uploaded this patch, I was following the format of the compensation tables
that are already present in the file. So i used "ncpXXxh103" instead
of "ncp15xh103".
Also, it was my first patch submitted to the Linux kernel. So I may have
been over thinking some things! With some hindsight, I see that the name is
misleading. I'm going to have to revisit this patch to make it more
'generic' for other thermistors, or make it more specific for this
particular thermistor.

I hope that answers your questions. Thanks for pointing this out to me.

Regards,
Joseph

On Wed, Mar 30, 2016 at 3:03 PM, Maxime Jayat <jayatmaxime@xxxxxxxxx> wrote:

> Hello Joseph,
>
> You recently added support for the ncpXXxh103 in
> drivers/hwmon/ntc_thermistor.c with the following array of values:
>
> +static const struct ntc_compensation ncpXXxh103[] = {
> +       { .temp_c       = -40, .ohm     = 247565 },
> +       { .temp_c       = -35, .ohm     = 181742 },
> +       { .temp_c       = -30, .ohm     = 135128 },
> +       { .temp_c       = -25, .ohm     = 101678 },
> +       { .temp_c       = -20, .ohm     = 77373 },
> +       { .temp_c       = -15, .ohm     = 59504 },
> +       { .temp_c       = -10, .ohm     = 46222 },
> +       { .temp_c       = -5, .ohm      = 36244 },
> +       { .temp_c       = 0, .ohm       = 28674 },
> +       { .temp_c       = 5, .ohm       = 22878 },
> +       { .temp_c       = 10, .ohm      = 18399 },
> +       { .temp_c       = 15, .ohm      = 14910 },
> +       { .temp_c       = 20, .ohm      = 12169 },
> +       { .temp_c       = 25, .ohm      = 10000 },
> +       { .temp_c       = 30, .ohm      = 8271 },
> +       { .temp_c       = 35, .ohm      = 6883 },
> +       { .temp_c       = 40, .ohm      = 5762 },
> +       { .temp_c       = 45, .ohm      = 4851 },
> +       { .temp_c       = 50, .ohm      = 4105 },
> +       { .temp_c       = 55, .ohm      = 3492 },
> +       { .temp_c       = 60, .ohm      = 2985 },
> +       { .temp_c       = 65, .ohm      = 2563 },
> +       { .temp_c       = 70, .ohm      = 2211 },
> +       { .temp_c       = 75, .ohm      = 1915 },
> +       { .temp_c       = 80, .ohm      = 1666 },
> +       { .temp_c       = 85, .ohm      = 1454 },
> +       { .temp_c       = 90, .ohm      = 1275 },
> +       { .temp_c       = 95, .ohm      = 1121 },
> +       { .temp_c       = 100, .ohm     = 990 },
> +       { .temp_c       = 105, .ohm     = 876 },
> +       { .temp_c       = 110, .ohm     = 779 },
> +       { .temp_c       = 115, .ohm     = 694 },
> +       { .temp_c       = 120, .ohm     = 620 },
> +       { .temp_c       = 125, .ohm     = 556 },
> +};
> +
>
> Where are these taken from?
> Unlike the other thermistors in the file, these values don't seem to
> match anything in the Murata NTC Thermistor Datasheet.
> See:
>
> http://www.murata.com/~/media/webrenewal/support/library/catalog/products/thermistor/ntc/r44e.ashx
>
> --
> Maxime Jayat
>
_______________________________________________
lm-sensors mailing list
lm-sensors@xxxxxxxxxxxxxx
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors



[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux