Re: [PATCH] media: st-rc: Add ST remote control driver

[Srinivas KANDAGATLA <srinivas.kandagatla@st.com>]

On 15/08/13 14:30, Pawel Moll wrote:
> On Wed, 2013-08-14 at 18:27 +0100, Srinivas KANDAGATLA wrote:
>> +Device-Tree bindings for ST IR and UHF receiver
>> +
>> +Required properties:
>> +       - compatible: should be "st,rc".
>> +       - st,uhfmode: boolean property to indicate if reception is in UHF.
>> +       - reg: base physical address of the controller and length of memory
>> +       mapped  region.
>> +       - interrupts: interrupt number to the cpu. The interrupt specifier
>> +       format depends on the interrupt controller parent.
>> +
>> +Example node:
>> +
>> +       rc: rc@fe518000 {
>> +               compatible      = "st,rc";
>> +               reg             = <0xfe518000 0x234>;
>> +               interrupts      =  <0 203 0>;
>> +       };
> 
> So is "st,uhfmode" required or optional after all? If the former, the
> example is wrong (doesn't specify required property). But as far as I
> understand it's really optional...


You are right, I will move this to optional properties section.

Thanks,
srini
> 
> Paweł
> 
> 
>