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:   "Sa, Nuno" <Nuno.Sa@analog.com>
To:     Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
        Alexandru Ardelean <aardelean@deviqon.com>,
        "linux-iio@vger.kernel.org" <linux-iio@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
CC:     Jonathan Cameron <jic23@kernel.org>,
        Lars-Peter Clausen <lars@metafoo.de>
Subject: RE: [PATCH v1 1/1] iio: adc: intel_mrfld_adc: explicitly add proper
 header files
Thread-Topic: [PATCH v1 1/1] iio: adc: intel_mrfld_adc: explicitly add proper
 header files
Thread-Index: AQHYgLCX5FbabFkmA0KbjkZY9UaAq61QYNmQ
Date:   Wed, 15 Jun 2022 12:15:48 +0000
Message-ID: <PH0PR03MB6786A019FB9A1BA810DDE2BC99AD9@PH0PR03MB6786.namprd03.prod.outlook.com>
References: <20220615120803.41350-1-andriy.shevchenko@linux.intel.com>
In-Reply-To: <20220615120803.41350-1-andriy.shevchenko@linux.intel.com>
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?1qfxP7a+w0YHQkuCSxePGoDlwg2KxxTk2Ttqa6uPZfYBFywHAugBzAq094?=
 =?iso-8859-1?Q?LtfkeeG7/uT+Rr16s/Vp28Gj59WhQgNGABt34JmUi4XQShZUmdxgmCaCjF?=
 =?iso-8859-1?Q?25hqc6Qr0hjxUlv/2VtPAFuFq1p3e0+niacRiMkEHgqc3OoBdEpq/qzfsn?=
 =?iso-8859-1?Q?mKa7RGULXrki1FkbqoqZl5jHECR5XLw+hgygfeeo2Pgm405uohnSV22xQg?=
 =?iso-8859-1?Q?a/PNfCoOxOfrsnRHrojCry4AKW04uWCPcYY4sL9rYXc6AzGrq4y4qWrxIe?=
 =?iso-8859-1?Q?LPG0QE+8WQU464BHSXcDdKQksfpo2phu+JqykGWvymLRqw4V+L5Fa2ec4P?=
 =?iso-8859-1?Q?5zjzPt0Ku1uA4xKxfci/6HT/ZgH0xW5VyqI3/y9CqvnhirBpgY4malbABF?=
 =?iso-8859-1?Q?fHAP0dI6IS1rRBoOIwYHXX9WUtPSVAdbHTSiidRhiePI4/xdjj6jfIW4n8?=
 =?iso-8859-1?Q?6sNFbx/7uMqSF7l9PUzkay95PzG6RbiKzdlf0v1DG8pUf/dXwlg7G4NM0T?=
 =?iso-8859-1?Q?0nkJx7ZxdXnhCr6+Qkj3o/J0tBUGWmv3N8WlRJviFKPoRW913/+YZ2FYfg?=
 =?iso-8859-1?Q?+L9BnYZxmRmEGJs7Xu/zT/l8sRVJWVHEZx5NNqybjJJ4sh1Z7co3TdwsLx?=
 =?iso-8859-1?Q?NJ6udyCiQ5jZ00SVms770Snr5cuFVoY4gufuJypaC20v72bF78wReSUFDP?=
 =?iso-8859-1?Q?ZyXD7Ts6EEiCec6oR4KfOSz3BpBcz3gZjnD6KeGnSw7OlBOJ0ldiAGztEp?=
 =?iso-8859-1?Q?RXE1fe41cMDP9z2Guhb3bqdv0zF2o+BO0DYyuFQY7EEVigUk7vixSa/0VK?=
 =?iso-8859-1?Q?qXVns9QtLtatDG45GSnvuWUeWJ1v7h8N04+6eV2+sj7wK/V0jn+YpCKleu?=
 =?iso-8859-1?Q?jvuB/QlKYjfmrr2ZgC5oSmvkePDFujMG6Ms/QLZ/j6c9B7BW/OfB9rmNMr?=
 =?iso-8859-1?Q?I1ujEEHZA8zn7Iq7iG21nxdJnS3iuCk1IYv7yhXn4MAwDciQtUMss/lDUy?=
 =?iso-8859-1?Q?Gwr7MDb7kFWQNd+htx3xFGkNXZ9/xvWac4UvqLwHX9OHMyXosdRIoncRdH?=
 =?iso-8859-1?Q?1brYrQhQ94XWEOTLCJFRlVNBND8LUjS8uXhra+m3U4oyGaAQcdzLn45Qwn?=
 =?iso-8859-1?Q?lQk6UUvdj1Qxoqt78thdWcKc/hu33+4bnVlRakHXP9MKoFgnxpCUkv9wc6?=
 =?iso-8859-1?Q?aTng4hGNpIoLyVK/AAJd85Ov6EBzE6lA02yE8+V35IgmLliRHga8O59jzT?=
 =?iso-8859-1?Q?P57zUHlUfDpXcceC1fJDWdJEkLJaWcyfeJ+3FvAnGCf23+vtWJbazXiyMH?=
 =?iso-8859-1?Q?rFFa5g7spczyjihdU17fI3B0HkFjAkOAg3SZPqDL/2an2GCMJDsvbYpl1z?=
 =?iso-8859-1?Q?CrWenakeS8D2Rfx6pUGDNKVuuRxNCPwBQVrATUejSboHcToxTJA99nUlif?=
 =?iso-8859-1?Q?UUclIiKjZ8Srdt4Eg6h9LfGMhgkI5wn9ffIoPfg4ALAaDsm1b0pT3eISNs?=
 =?iso-8859-1?Q?dSm1KsgaDk84fuU+l+qghzhEWOmJIbb00SOn+D8TMalv2Xb4v1nBXMAf5L?=
 =?iso-8859-1?Q?ouQEE/upbUs6Q93Fbo3nvLTYn2ACvuqtejZ5dwH9lVWeuUhbX4S1trd8Hj?=
 =?iso-8859-1?Q?E+cmi8hjoJLl0aGpsY1md8lBQSVlx1UAhBUE744Ub81spLzHFfxTcXSIqe?=
 =?iso-8859-1?Q?WwMIbZ9grvR/UFPKDPsggIvChNN1fCJ/qljmm5EKiIFOzxSpu8fryjeEki?=
 =?iso-8859-1?Q?i8Fde7Me3H8Y5PRBSeWyd1gpbvPtcsjssCBboKieESd4cKHtumNX3NvY/H?=
 =?iso-8859-1?Q?kR9UrDiC0A=3D=3D?=
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: PH0PR03MB6786.namprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: bcf0284c-7e78-4184-c4c5-08da4ec8c85d
X-MS-Exchange-CrossTenant-originalarrivaltime: 15 Jun 2022 12:15:48.7891
 (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: U8KxmBbEcSrVSkcBpfE4UE5d0o96goeZwFQKkGYxk7bO5y/t1WtcbfYi6z1ZD/c+vIkiBufh3H+s3kq6B1ZbhA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM8PR03MB6261
X-Proofpoint-GUID: 2UHMaLirvIpnBgig3fqoiuSqy5pf7IJ9
X-Proofpoint-ORIG-GUID: 2UHMaLirvIpnBgig3fqoiuSqy5pf7IJ9
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.205,Aquarius:18.0.874,Hydra:6.0.517,FMLib:17.11.64.514
 definitions=2022-06-15_13,2022-06-13_01,2022-02-23_01
X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 spamscore=0 clxscore=1011
 phishscore=0 mlxlogscore=862 malwarescore=0 adultscore=0
 priorityscore=1501 bulkscore=0 mlxscore=0 suspectscore=0
 lowpriorityscore=0 impostorscore=0 classifier=spam adjust=0 reason=mlx
 scancount=1 engine=8.12.0-2204290000 definitions=main-2206150049
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,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

Hi Andy,

> -----Original Message-----
> From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> Sent: Wednesday, June 15, 2022 2:08 PM
> To: Andy Shevchenko <andriy.shevchenko@linux.intel.com>;
> Alexandru Ardelean <aardelean@deviqon.com>; linux-
> iio@vger.kernel.org; linux-kernel@vger.kernel.org
> Cc: Jonathan Cameron <jic23@kernel.org>; Lars-Peter Clausen
> <lars@metafoo.de>
> Subject: [PATCH v1 1/1] iio: adc: intel_mrfld_adc: explicitly add proper
> header files
>=20
> [External]
>=20
> Do not trust the fact that iio.h includes of.h which in turn includes
> all the headers we are relying on.
>=20
> The ultimate goal is to actually drop of.h from iio.h.
>=20
> Signed-off-by: Andy Shevchenko
> <andriy.shevchenko@linux.intel.com>
> ---
>=20
> Jonathan, this should be squeezed before we remove of.h in iio.h
> I think Nuno somehow missed this

Yes, I did (compile) tested this on arm and arm64 as I said in the cover.
I guess this one won't be enabled on those archs :).

I was planning to send this on v2 (I did noticed the kernel test bot
emails) but yeah, this should actually be included with the other
patches before removing of.h from iio. Thanks!

Reviewed-by: Nuno S=E1 <nuno.sa@analog.com>