Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1212939

Re: [PATCH linux-next v4 5/5] mtd: atmel-quadspi: add driver for Atmel QSPI controller

Show all headers | View raw

On Tuesday, August 25, 2015 at 12:17:37 PM, Cyrille Pitchen wrote:
> Hi Marek,

Hi!

> Le 24/08/2015 13:03, Marek Vasut a écrit :
> > On Monday, August 24, 2015 at 12:14:00 PM, Cyrille Pitchen wrote:
> >> This driver add support to the new Atmel QSPI controller embedded into
> >> sama5d2x SoCs. It expects a NOR memory to be connected to the QSPI
> >> controller.
> >> 
> >> Signed-off-by: Cyrille Pitchen <cyrille.pitchen@atmel.com>
> >> Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>
> > 
> > Hi,
> > 
> > [...]
> > 
> >> +/* Register access macros */
> > 
> > These are functions, not macros :)
> > 
> > btw is there any reason for these ? I'd say, just put the read*() and
> > write*() functions directly into the code and be done with it, it is
> > much less confusing.
> 
> If you don't mind, I'd rather keep some of these inline functions. I have
> no strong justification, it's more a personal taste: it makes lines
> shorter as it avoids the need to add "->regs + ".
> Also it makes the code consistent with other Atmel drivers which already
> use such wrappers.
> 
> However I'll fix the comment and remove the byte and word versions, which
> are not used. So only qspi_readl() and qspi_writel() are left.
> 
> Does it sound good to you?

In my mind, seeing explicit readl_relaxed() somewhere is much easier to
digest than seeing some wrapper, which I have to look up. But please do
wait for others to voice their concern too, I might not be the best person
to tell you what to do when it comes to wrapping IO accessors ;-)

Best regards,
Marek Vasut
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH linux-next v4 5/5] mtd: atmel-quadspi: add driver for Atmel QSPI controller Cyrille Pitchen <cyrille.pitchen@atmel.com> - 2015-08-24 12:20 +0200
  Re: [PATCH linux-next v4 5/5] mtd: atmel-quadspi: add driver for Atmel QSPI controller Marek Vasut <marex@denx.de> - 2015-08-24 13:10 +0200
    Re: [PATCH linux-next v4 5/5] mtd: atmel-quadspi: add driver for  Atmel QSPI controller Russell King - ARM Linux <linux@arm.linux.org.uk> - 2015-08-24 15:00 +0200
      Re: [PATCH linux-next v4 5/5] mtd: atmel-quadspi: add driver for Atmel QSPI controller Marek Vasut <marex@denx.de> - 2015-08-24 15:20 +0200
    Re: [PATCH linux-next v4 5/5] mtd: atmel-quadspi: add driver for  Atmel QSPI controller Cyrille Pitchen <cyrille.pitchen@atmel.com> - 2015-08-24 19:10 +0200
      Re: [PATCH linux-next v4 5/5] mtd: atmel-quadspi: add driver for Atmel QSPI controller Marek Vasut <marex@denx.de> - 2015-08-24 19:50 +0200
        Re: [PATCH linux-next v4 5/5] mtd: atmel-quadspi: add driver for  Atmel QSPI controller Jonas Gorski <jogo@openwrt.org> - 2015-08-25 11:50 +0200
          Re: [PATCH linux-next v4 5/5] mtd: atmel-quadspi: add driver for  Atmel QSPI controller Cyrille Pitchen <cyrille.pitchen@atmel.com> - 2015-08-25 12:30 +0200
    Re: [PATCH linux-next v4 5/5] mtd: atmel-quadspi: add driver for  Atmel QSPI controller Cyrille Pitchen <cyrille.pitchen@atmel.com> - 2015-08-25 12:20 +0200
      Re: [PATCH linux-next v4 5/5] mtd: atmel-quadspi: add driver for Atmel QSPI controller Marek Vasut <marex@denx.de> - 2015-08-25 12:30 +0200
        Re: [PATCH linux-next v4 5/5] mtd: atmel-quadspi: add driver for  Atmel QSPI controller Brian Norris <computersforpeace@gmail.com> - 2015-08-25 18:00 +0200
  RE: [PATCH linux-next v4 5/5] mtd: atmel-quadspi: add driver for  Atmel QSPI controller Bean Huo 霍斌斌 (beanhuo)   <beanhuo@micron.com> - 2015-08-25 03:50 +0200
    Re: [PATCH linux-next v4 5/5] mtd: atmel-quadspi: add driver for  Atmel QSPI controller Cyrille Pitchen <cyrille.pitchen@atmel.com> - 2015-08-25 13:30 +0200

csiph-web