Hi Ricardo,
On Mon, Feb 2, 2015 at 3:05 PM, Ricardo Ribalda Delgado
wrote:
> On Mon, Feb 2, 2015 at 2:04 PM, Geert Uytterhoeven
> wrote:
>>> void * pvar=varB;
>>
>> ... = &varB;
>>
>>> *pvar = ioread8(valid_memory);
>>>
>>> Depending if ioread8 returns a u8 or a unsigned int, aren't we also
>>>
Hello Geert
On Mon, Feb 2, 2015 at 2:04 PM, Geert Uytterhoeven wrote:
>> void * pvar=varB;
>
> ... = &varB;
>
>> *pvar = ioread8(valid_memory);
>>
>> Depending if ioread8 returns a u8 or a unsigned int, aren't we also
>> accessing varC?
>>
>> Could not this be a problem?
>
> Please try to compil
Hi Ricardo,
On Mon, Feb 2, 2015 at 1:49 PM, Ricardo Ribalda Delgado
wrote:
> Regarding ioread8 et al.
>
> On include/asm-generic/io.h is defined as:
> extern unsigned int ioread8(void __iomem *);
>
> On include/asm-generic/io.h:
> static inline u8 ioread8(const volatile void __iomem *addr)
>
> Pl
Hello
Regarding ioread8 et al.
On include/asm-generic/io.h is defined as:
extern unsigned int ioread8(void __iomem *);
On include/asm-generic/io.h:
static inline u8 ioread8(const volatile void __iomem *addr)
Please ignore the qualifiers right now. The first function returns an
unsigned integer,
On Thursday 29 January 2015 23:14:46 Ricardo Ribalda Delgado wrote:
> What about the different return type? u8 vs int
Too many drivers use all sorts of different types, I've given up
hope of changing drivers to agree on the same type here. Basically
you can think of 'void __iomem *' as the magic k
Hello Arnd
On Thu, Jan 29, 2015 at 11:11 PM, Arnd Bergmann wrote:
>> > I think the definitions in include/asm-generic/iomap.h are actually wrong,
>> > as they lack a const:
>> >
>> > extern unsigned int ioread8(void __iomem *);
>> > extern unsigned int ioread16(void __iomem *);
>> > e
On Thursday 29 January 2015 17:39:08 Ricardo Ribalda Delgado wrote:
> > I think the definitions in include/asm-generic/iomap.h are actually wrong,
> > as they lack a const:
> >
> > extern unsigned int ioread8(void __iomem *);
> > extern unsigned int ioread16(void __iomem *);
> > extern
Hello Arnd
On Thu, Jan 29, 2015 at 5:04 PM, Arnd Bergmann wrote:
> Casting the type of a function you call seems rather dangerous. Why not
> add an inline function in this driver as a wrapper?
>
> Arnd
Agree, please ignore this patch. Sorry for the noise
--
Ricardo Ribalda
--
To unsub
Hello Geert
On Thu, Jan 29, 2015 at 4:56 PM, Geert Uytterhoeven
wrote:
> On Thu, Jan 29, 2015 at 3:51 PM, Ricardo Ribalda Delgado
> wrote:
>> IO functions prototypes may have different argument qualifiers
>> on different architectures.
>>
>> This patch cast the assignment of the function, to mat
On Thursday 29 January 2015 15:51:13 Ricardo Ribalda Delgado wrote:
> * Setup little endian helper functions first and try to use them
> * and check if bit was correctly setup or not.
> */
> - xspi->read_fn = ioread32;
> - xspi->write_fn = iowrite32;
> +
On Thu, Jan 29, 2015 at 3:51 PM, Ricardo Ribalda Delgado
wrote:
> IO functions prototypes may have different argument qualifiers
> on different architectures.
>
> This patch cast the assignment of the function, to match the one defined
> at iomap.h.
Adding casts is (usually) not the solution to t
IO functions prototypes may have different argument qualifiers
on different architectures.
This patch cast the assignment of the function, to match the one defined
at iomap.h.
Fixes: 99082eab63449f9d spi/xilinx: Remove iowrite/ioread wrappers
Reported-by: kbuild test robot
Signed-off-by: Ricardo
12 matches
Mail list logo
