linux-kernel - Re: [PATCH v3 10/10] iio: pressure: mprls0025pa add SPI driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20240107163215.427b563d@jic23-huawei>
Date: Sun, 7 Jan 2024 16:32:15 +0000
From: Jonathan Cameron <jic23@...nel.org>
To: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
Cc: Petre Rodan <petre.rodan@...dimension.ro>, linux-iio@...r.kernel.org,
 linux-kernel@...r.kernel.org, Andreas Klinger <ak@...klinger.de>,
 Lars-Peter Clausen <lars@...afoo.de>, Angel Iglesias
 <ang.iglesiasg@...il.com>, Matti Vaittinen <mazziesaccount@...il.com>
Subject: Re: [PATCH v3 10/10] iio: pressure: mprls0025pa add SPI driver

On Sat, 6 Jan 2024 18:07:55 +0200
Andy Shevchenko <andriy.shevchenko@...ux.intel.com> wrote:

> On Sat, Jan 06, 2024 at 06:02:52PM +0200, Petre Rodan wrote:
> > On Sat, Jan 06, 2024 at 05:30:21PM +0200, Andy Shevchenko wrote:  
> > > On Fri, Dec 29, 2023 at 11:24:38AM +0200, Petre Rodan wrote:  
> > > > Add SPI component of the driver.  
> 
> > > > +#include <linux/mod_devicetable.h>
> > > > +#include <linux/module.h>
> > > > +#include <linux/spi/spi.h>
> > > > +#include <linux/stddef.h>  
> > > 
> > > Basically here we need additionally these ones:
> > > 
> > > device.h
> > > errno.h
> > > types.h  
> > 
> > ok, I'll add errno.h. the other two are in the shared .h file.  
> 
> Yes, but it's better to follow IWYI principle. Generally speaking
> there is no guarantee that above will be _always_ the case.
> 
> Anyway, I leave this to Jonathan.
> 
When it's a local include to the driver like this I don't mind
as much if the includes aren't there in the c file, but do generally
'slightly prefer' them to be so.  As such I've added them here and
in the i2c file (I also renamed that patch as 'refactor' isn't
exactly descriptive in the patch title!)  I suspect none of us
feel that strongly about this so rather than slowing things down
I took a fairly random decision that let me apply the series today.

Anyhow, along with dropping the text on what this was tested on in
the final patch - series applied to the togreg branch of iio.git.

However, I'll be rebasing that tree on rc1 once available hence
for now this will only be pushed out as testing for 0-day to get
a head start on poking it.

Thanks,

Jonathan