Re: [PATCH v2 4/5] thermal: tsens: Add support for SDM845

From: Amit Kucheria
Date: Thu Jun 14 2018 - 06:24:37 EST


On Thu, Jun 14, 2018 at 9:48 AM, Vivek Gautam
<vivek.gautam@xxxxxxxxxxxxxx> wrote:
> Hi Amit,
>
> On Tue, Jun 12, 2018 at 4:24 PM, Amit Kucheria <amit.kucheria@xxxxxxxxxx> wrote:
>> SDM845 uses the TSENS v2 IP block
>>
>> Signed-off-by: Amit Kucheria <amit.kucheria@xxxxxxxxxx>
>> ---

<snip>

>> +
>> +static const struct tsens_ops ops_sdm845 = {
>> + .init = init_common,
>> + .get_temp = get_temp_tsens_v2,
>> +};
>> +
>> +const struct tsens_data data_sdm845 = {
>
> Just a minor nit. 'static' here?

Thanks for the review.

This file just went away in favour of a common tsens-v2.c that will
support all v2 SoCs. I'll be sending out an updated patchset soon.

>
>> + .ops = &ops_sdm845,
>> +};
>
> [snip]
>
> Thanks & Regards
> Vivek