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

Groups > linux.kernel > #1556688

Re: [PATCH v2 5/5] ARM: dts: Add LEGO MINDSTORMS EV3 dts

Show all headers | View raw

On 01/11/2017 04:42 AM, Sekhar Nori wrote:
> On Friday 06 January 2017 10:03 AM, David Lechner wrote:
>
>> +	beeper {
>> +		compatible = "pwm-beeper";
>> +		pinctrl-names = "default";
>> +		pinctrl-0 = <&ehrpwm0b_pins>, <&amp_pins>;
>> +		pwms = <&ehrpwm0 1 0 0>;
>> +		enable-gpios = <&gpio 111 GPIO_ACTIVE_HIGH>;
>
> Since the enable-gpios binding for pwm beeper is still not accepted, can
> you drop the property or the node itself (if that makes more sense)?

No sound will come out of the speaker without enabling this gpio. So, I 
guess I will just drop this node for now.


>
>> +&spi0 {
>> +	status = "okay";
>> +	pinctrl-names = "default";
>> +	pinctrl-0 = <&spi0_pins>, <&spi0_cs0_pin>, <&spi0_cs3_pin>;
>> +
>> +	flash@0 {
>> +		compatible = "n25q128a13", "jedec,spi-nor";
>> +		reg = <0>;
>> +		spi-max-frequency = <50000000>;
>> +		ti,spi-wdelay = <8>;
>> +
>> +		/* Partitions are based on the official firmware from LEGO */
>> +		partitions {
>> +			#address-cells = <1>;
>> +			#size-cells = <1>;
>> +			partition@0 {
>> +				label = "U-Boot";
>> +				reg = <0 0x40000>;
>> +			};
>> +
>> +			partition@40000 {
>> +				label = "U-Boot Env";
>> +				reg = <0x40000 0x10000>;
>> +			};
>> +
>> +			partition@50000 {
>> +				label = "Kernel";
>> +				reg = <0x50000 0x200000>;
>> +			};
>> +
>> +			partition@250000 {
>> +				label = "Filesystem";
>> +				reg = <0x250000 0xa50000>;
>> +			};
>> +
>> +			partition@cb0000 {
>> +				label = "Storage";
>> +				reg = <0xcb0000 0x2f0000>;
>> +			};
>> +		};
>> +	};
>> +
>> +	adc@3 {
>> +		compatible = "ti-ads7957";
>
> So looks like this works because of_register_spi_device() sets up
> modalias of spi device from compatible string. I am fine with it, just
> highlighting it here to make sure this is acceptable practice. I did not
> really find any precedence for using SPI device name as compatible
> property in existing DTS files.

Indeed. It looks like this sort of "trivial" device binding is just used 
for i2c devices. I will submit some patches to add proper device tree 
bindings and change this to "ti,ads7957".

>
>> +		reg = <3>;
>> +		spi-max-frequency = <10000000>;
>> +		refin-supply = <&adc_ref>;
>> +	};
>
> Rest of the patch looks good to me.
>
> Thanks,
> Sekhar
>

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

Thread

[PATCH v2 0/5] Support for LEGO MINDSTORMS EV3 David Lechner <david@lechnology.com> - 2017-01-06 05:40 +0100
  [PATCH v2 5/5] ARM: dts: Add LEGO MINDSTORMS EV3 dts David Lechner <david@lechnology.com> - 2017-01-06 05:40 +0100
    Re: [PATCH v2 5/5] ARM: dts: Add LEGO MINDSTORMS EV3 dts kbuild test robot <lkp@intel.com> - 2017-01-06 19:10 +0100
    Re: [PATCH v2 5/5] ARM: dts: Add LEGO MINDSTORMS EV3 dts Sekhar Nori <nsekhar@ti.com> - 2017-01-11 11:50 +0100
      Re: [PATCH v2 5/5] ARM: dts: Add LEGO MINDSTORMS EV3 dts David Lechner <david@lechnology.com> - 2017-01-11 17:30 +0100
        Re: [PATCH v2 5/5] ARM: dts: Add LEGO MINDSTORMS EV3 dts Sekhar Nori <nsekhar@ti.com> - 2017-01-12 08:50 +0100
  [PATCH v2 1/5] ARM: davinci_all_defconfig: enable DA8xx pinconf David Lechner <david@lechnology.com> - 2017-01-06 05:40 +0100
    Re: [PATCH v2 1/5] ARM: davinci_all_defconfig: enable DA8xx pinconf Sekhar Nori <nsekhar@ti.com> - 2017-01-09 13:30 +0100
  [PATCH v2 4/5] ARM: davinci_all_defconfig: enable pwm-beeper David Lechner <david@lechnology.com> - 2017-01-06 05:40 +0100
  [PATCH v2 2/5] ARM: davinci_all_defconfig: Enable PWM modules David Lechner <david@lechnology.com> - 2017-01-06 05:40 +0100
    Re: [PATCH v2 2/5] ARM: davinci_all_defconfig: Enable PWM modules Sekhar Nori <nsekhar@ti.com> - 2017-01-09 13:30 +0100

csiph-web