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


Groups > linux.kernel > #1283966

Re: [PATCH v7 2/6] mfd: syscon: add a DT property to set value width

From Arnd Bergmann <arnd@arndb.de>
Newsgroups linux.kernel
Subject Re: [PATCH v7 2/6] mfd: syscon: add a DT property to set value width
Date 2015-12-04 17:10 +0100
Message-ID <qC15L-4Xp-3@gated-at.bofh.it> (permalink)
References <qAzbz-5R5-1@gated-at.bofh.it> <qABZM-7C9-17@gated-at.bofh.it> <qBp0v-5qO-43@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Wednesday 02 December 2015 18:21:02 Damien Riegel wrote:
> Lee, Arnd,
> 
> I had to make changes in this patch to address Rob's comments.
> reg-io-width is now in bytes while bus-width was a value in bits. Could
> you please review this patch ?


Ok, that's fine.

> 
> On Mon, Nov 30, 2015 at 01:00:15PM -0600, Rob Herring wrote:
> > On Mon, Nov 30, 2015 at 10:59:47AM -0500, Damien Riegel wrote:
> > > Currently syscon has a fixed configuration of 32 bits for register and
> > > values widths. In some cases, it would be desirable to be able to
> > > customize the value width.
> > > 
> > > For example, certain boards (like the ones manufactured by Technologic
> > > Systems) have a FPGA that is memory-mapped, but its registers are only
> > > 16-bit wide.
> > > 
> > > This patch adds an optional "reg-io-width" DT binding for syscon that
> > > allows to change the width for the data bus (i.e. val_bits). If this
> > > property is provided, it will also set the register stride to
> > > reg-io-width's value. If not provided, the default configuration is
> > > used.
> > > 
> > > Signed-off-by: Damien Riegel <damien.riegel@savoirfairelinux.com>
> > 
> > Acked-by: Rob Herring <robh@kernel.org>
> > 
> 

Acked-by: Arnd Bergmann <arnd@arndb.de>
--
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 | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH v7 0/6] Add board support for TS-4800 Damien Riegel <damien.riegel@savoirfairelinux.com> - 2015-11-30 17:10 +0100
  [PATCH v7 2/6] mfd: syscon: add a DT property to set value width Damien Riegel <damien.riegel@savoirfairelinux.com> - 2015-11-30 17:10 +0100
    Re: [PATCH v7 2/6] mfd: syscon: add a DT property to set value width Rob Herring <robh@kernel.org> - 2015-11-30 20:10 +0100
      Re: [PATCH v7 2/6] mfd: syscon: add a DT property to set value width Damien Riegel <damien.riegel@savoirfairelinux.com> - 2015-12-03 00:30 +0100
        Re: [PATCH v7 2/6] mfd: syscon: add a DT property to set value width Arnd Bergmann <arnd@arndb.de> - 2015-12-04 17:10 +0100
    Re: [PATCH v7 2/6] mfd: syscon: add a DT property to set value width Lee Jones <lee.jones@linaro.org> - 2015-12-07 10:50 +0100
      Re: [PATCH v7 2/6] mfd: syscon: add a DT property to set value width Damien Riegel <damien.riegel@savoirfairelinux.com> - 2015-12-07 21:00 +0100
        Re: [PATCH v7 2/6] mfd: syscon: add a DT property to set value width Arnd Bergmann <arnd@arndb.de> - 2015-12-08 11:10 +0100
        Re: [PATCH v7 2/6] mfd: syscon: add a DT property to set value width Lee Jones <lee.jones@linaro.org> - 2015-12-08 15:00 +0100
          Re: [PATCH v7 2/6] mfd: syscon: add a DT property to set value width Lee Jones <lee.jones@linaro.org> - 2015-12-08 15:20 +0100
  [PATCH v7 3/6] watchdog: ts4800: add driver for TS-4800 watchdog Damien Riegel <damien.riegel@savoirfairelinux.com> - 2015-11-30 17:10 +0100
    Re: [PATCH v7 3/6] watchdog: ts4800: add driver for TS-4800 watchdog Guenter Roeck <linux@roeck-us.net> - 2015-11-30 17:20 +0100
      Re: [PATCH v7 3/6] watchdog: ts4800: add driver for TS-4800 watchdog Damien Riegel <damien.riegel@savoirfairelinux.com> - 2015-12-04 19:50 +0100
    Re: [PATCH v7 3/6] watchdog: ts4800: add driver for TS-4800 watchdog Rob Herring <robh@kernel.org> - 2015-11-30 20:10 +0100
  [PATCH v7 5/6] of: documentation: add bindings documentation for TS-4800 Damien Riegel <damien.riegel@savoirfairelinux.com> - 2015-11-30 17:10 +0100
  [PATCH v7 6/6] ARM: dts: TS-4800: add basic device tree Damien Riegel <damien.riegel@savoirfairelinux.com> - 2015-11-30 17:10 +0100
  [PATCH v7 1/6] of: add vendor prefix for Technologic Systems Damien Riegel <damien.riegel@savoirfairelinux.com> - 2015-11-30 17:10 +0100
  [PATCH v7 4/6] ARM: imx_v6_v7_defconfig: add TS-4800 watchdog Damien Riegel <damien.riegel@savoirfairelinux.com> - 2015-11-30 17:10 +0100

csiph-web