On Fri, Mar 09, 2018 at 06:51:36PM +0100, Paul Cercueil wrote:
> Le 2018-03-09 16:12, Ezequiel Garcia a écrit :
> > @@ -164,8 +171,46 @@ struct jz4740_mmc_host {
> > * trigger is when data words in MSC_TXFIFO is < 8.
> > */
> > #define JZ4740_MMC_FIFO_HALF_SIZE 8
> > +
> > + void (*write_irq_mask)(struct jz4740_mmc_host *host, uint32_t val);
> > + void (*write_irq_reg)(struct jz4740_mmc_host *host, uint32_t val);
> > + uint32_t (*read_irq_reg)(struct jz4740_mmc_host *host);
> > };
>
> I'm not 100% fan about the callback idea, the original commit
> (https://github.com/gcwnow/linux/commit/62472091) doesn't use them and
> is
> 30 lines shorter.
>
> I'm not terribly against either, so if nobody else bug on that, feel
> free
> to ignore my comment.
I was thinking the same as Paul too to be honest. Unless there is a
measurable benefit to having callbacks, I think its cleaner and more
readable to stick to conditionals and normal abstractions where
appropriate.
Cheers
James
Attachment:
signature.asc
Description: Digital signature
