RE: [PATCH v6 02/12] spi: intel-dg: implement region enumeration

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 
> On Mon, Sep 16, 2024 at 04:49:18PM +0300, Alexander Usyskin wrote:
> 
> > In intel-dg spi, there is no access to the spi controller, the
> > information is extracted from the descriptor region.
> 
> Which information?  I can't tell what this patch is supposed to do; as with the
> first patch this shouldn't be standalone.

The patch series was built for review, it adds functionality on top of each other. 

> 
> > +static inline u32 spi_error(struct intel_dg_spi *spi) {
> 
> > +static inline u32 spi_read32(struct intel_dg_spi *spi, u32 address) {
> 
> At least these names are too generic, please use driver specific prefixes.

Can be done but those functions are local to the driver not part of the API.






[Index of Archives]     [Linux Kernel]     [Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [Linux for Samsung SOC]     [eCos]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [Linux MIPS]     [Yosemite Campsites]

  Powered by Linux