Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752138AbdLEGsH (ORCPT ); Tue, 5 Dec 2017 01:48:07 -0500 Received: from mail-wm0-f46.google.com ([74.125.82.46]:34576 "EHLO mail-wm0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751325AbdLEGsE (ORCPT ); Tue, 5 Dec 2017 01:48:04 -0500 X-Google-Smtp-Source: AGs4zMZaJ6lZhQn+4CdhwoXyPgZpKHNgi4DmwQOL2kaD1QcjI5Y5JIFaLU6SX9SxLbwyBk8HFkA01A== Subject: Re: [PATCH] thermal/drivers/hisi: Remove confusing error message To: Eduardo Valentin Cc: rui.zhang@intel.com, "open list:THERMAL" , open list References: <1499439833-32531-1-git-send-email-daniel.lezcano@linaro.org> <20171205015231.GC3536@localhost.localdomain> From: Daniel Lezcano Message-ID: Date: Tue, 5 Dec 2017 07:48:01 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 In-Reply-To: <20171205015231.GC3536@localhost.localdomain> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1293 Lines: 32 On 05/12/2017 02:52, Eduardo Valentin wrote: > Hello, > > Catching up on old patches. > On Fri, Jul 07, 2017 at 05:03:52PM +0200, Daniel Lezcano wrote: >> The sensor id is unknown at init time and we use all id in the authorized >> MAX_SENSORS interval to register the sensor. On this SoC there is one >> thermal-zone with one sensor on it. No need to spit on the console everytime we >> failed to register thermal sensors, information which is deliberaly known as it >> is part of the discovery process. >> >> hisi_thermal f7030700.tsensor: failed to register sensor id 0: -19 >> hisi_thermal f7030700.tsensor: failed to register thermal sensor: -19 >> hisi_thermal f7030700.tsensor: failed to register sensor id 1: -19 >> hisi_thermal f7030700.tsensor: failed to register thermal sensor: -19 >> hisi_thermal f7030700.tsensor: failed to register sensor id 3: -19 >> hisi_thermal f7030700.tsensor: failed to register thermal sensor: -19 >> >> Remove the error messages > > Is this still needed? I am assuming no. Right, no longer needed. -- Daniel -- Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog