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

Groups > linux.kernel > #1536143

Re: [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file

Show all headers | View raw

On Tue, Nov 29, 2016 at 01:38:21AM +0300, Serge Semin wrote:
> See cover-letter for changelog
> 
> Signed-off-by: Serge Semin <fancer.lancer@gmail.com>
> 
> ---
>  .../devicetree/bindings/misc/idt_89hpesx.txt       | 41 ++++++++++++++++++++++

There's not a better location for this? I can't tell because you don't 
describe what the device is.

>  1 file changed, 41 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/misc/idt_89hpesx.txt
> 
> diff --git a/Documentation/devicetree/bindings/misc/idt_89hpesx.txt b/Documentation/devicetree/bindings/misc/idt_89hpesx.txt
> index 0000000..469cc93
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/misc/idt_89hpesx.txt
> @@ -0,0 +1,41 @@
> +EEPROM / CSR SMBus-slave interface of IDT 89HPESx devices
> +
> +Required properties:
> +  - compatible : should be "<manufacturer>,<type>"
> +		 Basically there is only one manufacturer: idt, but some
> +		 compatible devices may be produced in future. Following devices
> +		 are supported: 89hpes8nt2, 89hpes12nt3, 89hpes24nt6ag2,
> +		 89hpes32nt8ag2, 89hpes32nt8bg2, 89hpes12nt12g2, 89hpes16nt16g2,
> +		 89hpes24nt24g2, 89hpes32nt24ag2, 89hpes32nt24bg2;
> +		 89hpes12n3, 89hpes12n3a, 89hpes24n3, 89hpes24n3a;
> +		 89hpes32h8, 89hpes32h8g2, 89hpes48h12, 89hpes48h12g2,
> +		 89hpes48h12ag2, 89hpes16h16, 89hpes22h16, 89hpes22h16g2,
> +		 89hpes34h16, 89hpes34h16g2, 89hpes64h16, 89hpes64h16g2,
> +		 89hpes64h16ag2;
> +		 89hpes12t3g2, 89hpes24t3g2, 89hpes16t4, 89hpes4t4g2,
> +		 89hpes10t4g2, 89hpes16t4g2, 89hpes16t4ag2, 89hpes5t5,
> +		 89hpes6t5, 89hpes8t5, 89hpes8t5a, 89hpes24t6, 89hpes6t6g2,
> +		 89hpes24t6g2, 89hpes16t7, 89hpes32t8, 89hpes32t8g2,
> +		 89hpes48t12, 89hpes48t12g2.
> +		 Current implementation of the driver doesn't have any device-

Driver capabilties are irrelevant to bindings.

> +		 specific functionalities. But since each of them differs
> +		 by registers mapping, CSRs read/write restrictions can be
> +		 added in future.
> +  - reg :	 I2C address of the IDT 89HPES device.
> +
> +Optional properties:
> +  - read-only :	 Parameterless property disables writes to the EEPROM
> +  - idt,eesize : Size of EEPROM device connected to IDT 89HPES i2c-master bus
> +		 (default value is 4096 bytes if option isn't specified)
> +  - idt,eeaddr : Custom address of EEPROM device
> +		 (If not specified IDT 89HPESx device will try to communicate
> +		  with EEPROM sited by default address - 0x50)

Don't we already have standard EEPROM properties that could be used 
here?

> +
> +Example:
> +	idt_pcie_sw@60 {

Don't use '_'.

> +		compatible = "idt,89hpes12nt3";
> +		reg = <0x60>;
> +		read-only;
> +		idt,eesize = <65536>;
> +		idt,eeaddr = <0x50>;
> +	};
> -- 
> 2.6.6
> 

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

Thread

[PATCH v2 0/2] eeprom: Add IDT 89HPESx EEPROM/CSR driver Serge Semin <fancer.lancer@gmail.com> - 2016-11-28 23:40 +0100
  [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file Serge Semin <fancer.lancer@gmail.com> - 2016-11-28 23:40 +0100
    Re: [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file Greg KH <gregkh@linuxfoundation.org> - 2016-11-29 20:40 +0100
      Re: [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file Serge Semin <fancer.lancer@gmail.com> - 2016-11-29 22:20 +0100
    Re: [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file Rob Herring <robh@kernel.org> - 2016-12-05 15:50 +0100
      Re: [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file Serge Semin <fancer.lancer@gmail.com> - 2016-12-05 16:30 +0100
        Re: [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file Rob Herring <robh@kernel.org> - 2016-12-05 18:30 +0100
          Re: [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file Serge Semin <fancer.lancer@gmail.com> - 2016-12-05 20:10 +0100
            Re: [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file Serge Semin <fancer.lancer@gmail.com> - 2016-12-09 03:00 +0100
  Re: [PATCH v2 1/2] eeprom: Add IDT 89HPESx EEPROM/CSR driver Greg KH <gregkh@linuxfoundation.org> - 2016-11-29 20:40 +0100
    Re: [PATCH v2 1/2] eeprom: Add IDT 89HPESx EEPROM/CSR driver Serge Semin <fancer.lancer@gmail.com> - 2016-11-29 22:20 +0100
      Re: [PATCH v2 1/2] eeprom: Add IDT 89HPESx EEPROM/CSR driver Greg KH <gregkh@linuxfoundation.org> - 2016-11-29 22:30 +0100
        Re: [PATCH v2 1/2] eeprom: Add IDT 89HPESx EEPROM/CSR driver Serge Semin <fancer.lancer@gmail.com> - 2016-11-29 22:50 +0100
  Re: [PATCH v2 1/2] eeprom: Add IDT 89HPESx EEPROM/CSR driver Greg KH <gregkh@linuxfoundation.org> - 2016-11-29 20:40 +0100
  [PATCH v3 2/2] eeprom: Add IDT 89HPESx driver dts-binding file Serge Semin <fancer.lancer@gmail.com> - 2016-11-29 23:30 +0100
  [PATCH v3 0/2] eeprom: Add IDT 89HPESx EEPROM/CSR driver Serge Semin <fancer.lancer@gmail.com> - 2016-11-29 23:30 +0100

csiph-web