Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   "Matyas, Daniel" <Daniel.Matyas@analog.com>
To:     Guenter Roeck <linux@roeck-us.net>
CC:     kernel test robot <lkp@intel.com>,
        Jean Delvare <jdelvare@suse.com>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Conor Dooley <conor+dt@kernel.org>,
        Jonathan Corbet <corbet@lwn.net>,
        "linux-hwmon@vger.kernel.org" <linux-hwmon@vger.kernel.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-doc@vger.kernel.org" <linux-doc@vger.kernel.org>
Subject: RE: [PATCH v9 2/2] hwmon: max31827: add MAX31827 driver
Thread-Topic: [PATCH v9 2/2] hwmon: max31827: add MAX31827 driver
Thread-Index: AQHZh+OySu2N90N0RU+tieZpCTNw1K9mUcYAgAANiKA=
Date:   Mon, 22 May 2023 14:18:11 +0000
Message-ID: <PH0PR03MB6771D471E1AF58B0B50FE4D489439@PH0PR03MB6771.namprd03.prod.outlook.com>
References: <20230516104609.7095-1-daniel.matyas@analog.com>
 <20230516104609.7095-2-daniel.matyas@analog.com>
 <0e49e860-6f2c-48cb-9ef7-af7891b95237@roeck-us.net>
In-Reply-To: <0e49e860-6f2c-48cb-9ef7-af7891b95237@roeck-us.net>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?iso-8859-1?Q?L7ZQEUn0Kx+okD7c7FwBPHxOyKCGgGvNiN9jxPpqPhWd0Ql+mQz/gIznry?=
 =?iso-8859-1?Q?EhKzSaUU48eCSaf/YJCAZI3wQLKwfT4bttYG9kTcVaJ4jLBvrn6O6+PZiG?=
 =?iso-8859-1?Q?NagZOoo0G6160rJq8phhWawktMAfYoN+RSPjNBGKq0PGpWuKvQlIopXOzq?=
 =?iso-8859-1?Q?23PAFPzkDHWwxV1+UqzdLgmw1TYcIlmQlQriBaDi5hCebEkl6gPlDfYzhW?=
 =?iso-8859-1?Q?QfX2q2ew28TCxWSxeYyQnpAfYTRRaQDv0y9IBwSDL8c/H8XoTiecQeShRP?=
 =?iso-8859-1?Q?jIUVIf/ecowIH3xCGrNdYNuCoCyc/o4e5WeuuxCD900JsG9OBmUSUHUAaX?=
 =?iso-8859-1?Q?923/GoFQMfFG5UD7MuaUQ1mPIE30MeqJ1QVijb/4dDBx1wocQbpxkfDG74?=
 =?iso-8859-1?Q?S4WxvfeatTZ33ytOQPu0wVTSj4A5t/inPYVyyRBhrxE1RFDqDPrgXu93Ep?=
 =?iso-8859-1?Q?35WVQ09+Qw7v3jBq1OkAAV/wh5dc9SIXKbiGwBCUJ04ZVjj5V48PcbvFQR?=
 =?iso-8859-1?Q?LCa+j55AkdeAm+S6RTPYJKBiWyd2yeiABpwf02GKw9dq92QzvrbjsdFB7z?=
 =?iso-8859-1?Q?LrZf3i3/cxIujUWRz89YnOR9owIPLdiRDtxnqDp5hMsWm4CZrK9GTeJ0Zl?=
 =?iso-8859-1?Q?u+GMveL1zNJuH9uFsfdER24ZBsws4Cp6xsa+OszXrcXT7ZB/bJkMLNup9o?=
 =?iso-8859-1?Q?fvg0FJdDNjYIE+51FAS/QS9u11XbvHpRGA77y6vft5B5G22lysqmftXlzz?=
 =?iso-8859-1?Q?tnqBagJsXgK4FHQdWAlszEWIH0MxM4xd4wxOa/lEKxix/YynNn5v+F240M?=
 =?iso-8859-1?Q?Q4ZL0iYjHPCMfRvq8LO0J0gW4U8IMWcVIewQpRy45DrXj/3Kz9FK6Cl9Hi?=
 =?iso-8859-1?Q?ehqE7UwsMsWcKwpSqcUbihYJW4cK6MKdDleIk4FB8Djo8t0gubWKb4OGUu?=
 =?iso-8859-1?Q?RAmzp5LtfRJIFsB8ezo4ZIUMFCkoCW+4YogGcyTNZT3e+9My9TFjJSObwO?=
 =?iso-8859-1?Q?1nnO3ql2iDTinF1R9ENkkAZ8sYC2RZavCX3U42/tL/7JH6Ac+zgodRyMcp?=
 =?iso-8859-1?Q?NCl/HXERpbk24Zseqrxer98clbQRnvNx9MNOg4rdItDC1nz538LNIv3tPV?=
 =?iso-8859-1?Q?OCMm13L65memTmHauX/j1AXyU2W/QVbXIrbBdl8UqGrzaiuHuKC24ctnv/?=
 =?iso-8859-1?Q?sKcZVYaD0x3nEHuEH2cwtRihad+4c91vSlk3xSrGjKsGVk6RdDw1K/tW8P?=
 =?iso-8859-1?Q?YZ33qXMUuSoOah7FMRQ7LXda1ccfJitJKUqF3Z1h+aphmop5utWhKdJZSy?=
 =?iso-8859-1?Q?Z0b3yEtJXQ1TDbaUltytqTWkEgkKd7CqrtFvyfKjqZ/+MHZi3KqhI8aLCP?=
 =?iso-8859-1?Q?KLseJGKokVwb/MsPwZxncIVtmY8PRJIXMzgftgsc4nCQT1mJSlUW11goBJ?=
 =?iso-8859-1?Q?c6+L3VHUusRh3xjbQxoKWXjvbYRMwsKhXug0YlGG86NlaoIraLWWC7h/1e?=
 =?iso-8859-1?Q?c90WM7psOyNyRoDv9AyWH2Sl/cnw+EvKKSmjNQhl02zEw+VIRaUFng02uJ?=
 =?iso-8859-1?Q?1AvylDg89HraT2BJja7qDmRy3LU5JfiA/C3r8jgCdnMejIem2Q4KKCPyLw?=
 =?iso-8859-1?Q?vNBERz+HPUDdrxD6oJSmwOvnZAGWx5mQK/?=
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: analog.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: PH0PR03MB6771.namprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: bc72176b-bd60-4e2b-009f-08db5acf5fa5
X-MS-Exchange-CrossTenant-originalarrivaltime: 22 May 2023 14:18:11.2353
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: eaa689b4-8f87-40e0-9c6f-7228de4d754a
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: defB57dMPzAb77eGU4ovPUyfyjdKMU05RFClVXDzKm6/yNdAQr+VRimpVl/UGq2YPgcEGARGg4ACLUgHkuU4rjoPvXVq2qAJexieuyHMFaU=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR03MB5455
X-Proofpoint-ORIG-GUID: N458BYzrtt571VeECC5tXsZ-blBEC3Yj
X-Proofpoint-GUID: N458BYzrtt571VeECC5tXsZ-blBEC3Yj
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.254,Aquarius:18.0.957,Hydra:6.0.573,FMLib:17.11.176.26
 definitions=2023-05-22_10,2023-05-22_03,2023-05-22_02
X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 impostorscore=0 mlxscore=0
 bulkscore=0 malwarescore=0 suspectscore=0 clxscore=1015 phishscore=0
 adultscore=0 spamscore=0 mlxlogscore=717 priorityscore=1501
 lowpriorityscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2304280000 definitions=main-2305220119
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,SPF_PASS,
        T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


-----Original Message-----
From: Guenter Roeck <groeck7@gmail.com> On Behalf Of Guenter Roeck
Sent: Monday, May 22, 2023 4:25 PM
To: Matyas, Daniel <Daniel.Matyas@analog.com>
Cc: kernel test robot <lkp@intel.com>; Jean Delvare <jdelvare@suse.com>; Ro=
b Herring <robh+dt@kernel.org>; Krzysztof Kozlowski <krzysztof.kozlowski+dt=
@linaro.org>; Conor Dooley <conor+dt@kernel.org>; Jonathan Corbet <corbet@l=
wn.net>; linux-hwmon@vger.kernel.org; devicetree@vger.kernel.org; linux-ker=
nel@vger.kernel.org; linux-doc@vger.kernel.org
Subject: Re: [PATCH v9 2/2] hwmon: max31827: add MAX31827 driver

[External]

On Tue, May 16, 2023 at 01:46:07PM +0300, Daniel Matyas wrote:
> MAX31827 is a low-power temperature switch with I2C interface.
>=20
> The device is a =B11=B0C accuracy from -40=B0C to +125=B0C
> (12 bits) local temperature switch and sensor with I2C/SM- Bus=20
> interface. The combination of small 6-bump wafer-lev- el package (WLP)=20
> and high accuracy makes this temper- ature sensor/switch ideal for a=20
> wide range of applications.
>=20
> Signed-off-by: Daniel Matyas <daniel.matyas@analog.com>
> Reported-by: kernel test robot <lkp@intel.com>
> Link:=20
> https://urldefense.com/v3/__https://lore.kernel.org/oe-kbuild-all/2023
> 05112351.DBkFfs76-lkp@intel.com/__;!!A3Ni8CS0y2Y!5SeqSdRyCHcgA6Se6U-EN
> I_MJfRPgB-FEVkuwWyb944DQypaZ5XQAS6N3aZiVDd1LZDGK45xp7KJHH8RSzw$
> ---

Change log goes here. Without it, I'll have to manually re-review and compa=
re against previous versions and look up comments to ensure that all feedba=
ck has been addressed. That will take time, which is always
scarce.   don't have right now. My apologies, but that means that
review and acceptance of your driver will be delayed.

Guenter


Dear Guenter,

I didn't write a changelog, because I changed literally nothing in the code=
. I sent a new patch, because in the previous Krzysztof Kozlowski asked me =
to remove the '--base' when using 'git format-patch'. I wrote this to commi=
t 1/2, but consider it necessary to write the same thing again in patch 2/2=
.

I have the Reported-by flag, because I forgot to add it in patch v8.

I am sorry for the confusion!

Best regards,
Daniel Matyas