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

Groups > linux.kernel > #1298694

Re: [PATCH v1 0/6] misc: add reboot mode driver

Show all headers | View raw

On Monday 28 December 2015 16:56:12 Heiko Stübner wrote:
> Am Montag, 28. Dezember 2015, 16:28:55 schrieb Arnd Bergmann:
> > On Wednesday 23 December 2015 17:31:45 Andy Yan wrote:
> > > +        { .compatible = "rockchip,reboot-mode-nvram",
> > > +                .data = (void *)&reboot-mode-nvram },
> > > +        {},
> > > +};
> > 
> > nvram is a complex topic by itself, because there are so many ways to do it.
> > I think what you are referring to here is a battery-backed memory that uses
> > one or more bytes at a fixed offset to store a particular piece of
> > information, as the drivers/char/nvram.c driver does. Maybe we should put
> > the reboot mode into that driver then?
> > 
> > There are other nvram drivers at various places in the kernel, and each may
> > be slightly different, or completely different, like the EFIVARs driver on
> > UEFI firmware or the key/value store on Open Firmware, these probably need
> > their own methods and not share the generic driver.
> 
> actually we now have drivers/nvmem/* that does that byte-wise addressing for 
> eeproms, efuses, etc in a generic way.

Good point, so some of the reboot-reason users will be able to use that
framework, but some will not:

- drivers/nvmem only works for fixed-length data in a fixed location, but
  not for key/value pairs, TLVs etc.

- Coming back to an earlier problem I pointed out, a lot of the things
  stored in an nvram need a consistent user interface. This includes stuff
  like kernel command line, boot image name, boot device, console
  configuration, network mac address. We probably want the reboot reason
  to fit into the wider framework for these, and drivers/nvmem doesn't
  (currently) look like a good place for that.

	Arnd
--
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 v1 0/6] misc: add reboot mode driver Andy Yan <andy.yan@rock-chips.com> - 2015-12-22 10:10 +0100
  [PATCH v1 1/6] dt-bindings: misc: add document for reboot-mode driver Andy Yan <andy.yan@rock-chips.com> - 2015-12-22 10:10 +0100
    Re: [PATCH v1 1/6] dt-bindings: misc: add document for reboot-mode  driver Rob Herring <robh@kernel.org> - 2015-12-23 01:40 +0100
      Re: [PATCH v1 1/6] dt-bindings: misc: add document for reboot-mode driver Moritz Fischer <moritz.fischer@ettus.com> - 2015-12-23 09:40 +0100
      Re: [PATCH v1 1/6] dt-bindings: misc: add document for reboot-mode  driver Andy Yan <andy.yan@rock-chips.com> - 2015-12-23 10:10 +0100
  [PATCH v1 4/6] soc: rockchip: add reboot mode driver Andy Yan <andy.yan@rock-chips.com> - 2015-12-22 10:20 +0100
  [PATCH v1 3/6] misc: add reboot mode driver Andy Yan <andy.yan@rock-chips.com> - 2015-12-22 10:20 +0100
  [PATCH v1 5/6] ARM: dts: rockchip: add reboot-mode node Andy Yan <andy.yan@rock-chips.com> - 2015-12-22 10:20 +0100
    Re: [PATCH v1 5/6] ARM: dts: rockchip: add reboot-mode node Heiko Stübner <heiko@sntech.de> - 2015-12-22 12:30 +0100
  [PATCH v1 2/6] dt-bindings: soc: add document for rockchip reboot-mode driver Andy Yan <andy.yan@rock-chips.com> - 2015-12-22 10:20 +0100
  [PATCH v1 6/6] ARM64: dts: rockchip: add reboot-mode node Andy Yan <andy.yan@rock-chips.com> - 2015-12-22 10:30 +0100
  Re: [PATCH v1 0/6] misc: add reboot mode driver Alexandre Belloni <alexandre.belloni@free-electrons.com> - 2015-12-22 17:50 +0100
    Re: [PATCH v1 0/6] misc: add reboot mode driver Andy Yan <andy.yan@rock-chips.com> - 2015-12-23 10:40 +0100
      Re: [PATCH v1 0/6] misc: add reboot mode driver Arnd Bergmann <arnd@arndb.de> - 2015-12-28 16:40 +0100
        Re: [PATCH v1 0/6] misc: add reboot mode driver Heiko Stübner <heiko@sntech.de> - 2015-12-28 17:00 +0100
          Re: [PATCH v1 0/6] misc: add reboot mode driver Arnd Bergmann <arnd@arndb.de> - 2015-12-28 17:20 +0100
        Re: [PATCH v1 0/6] misc: add reboot mode driver Andy Yan <andy.yan@rock-chips.com> - 2015-12-29 09:00 +0100

csiph-web