Re: [PATCH 4/7] misc: eeprom: 93xx46: Add DT binding for Atmel AT93C46D devices.

[Cory Tusar]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 11/17/2015 03:56 PM, Rob Herring wrote:
> On Mon, Nov 16, 2015 at 11:16:33PM -0500, Cory Tusar wrote:
>> This commit adds a compatible string to the eeprom_93xx46 devicetree
>> bindings in support of Atmel AT93C46D devices.
>>
>> Signed-off-by: Cory Tusar <cory.tusar@xxxxxxxxxxxxxxxxx>
>> ---
>>  Documentation/devicetree/bindings/misc/eeprom-93xx46.txt | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/Documentation/devicetree/bindings/misc/eeprom-93xx46.txt b/Documentation/devicetree/bindings/misc/eeprom-93xx46.txt
>> index ba493a8..34a0cc2 100644
>> --- a/Documentation/devicetree/bindings/misc/eeprom-93xx46.txt
>> +++ b/Documentation/devicetree/bindings/misc/eeprom-93xx46.txt
>> @@ -1,7 +1,7 @@
>>  EEPROMs (SPI) compatible with Microchip Technology 93xx46 family.
>>  
>>  Required properties:
>> -- compatible : "eeprom-93xx46"
>> +- compatible : "eeprom-93xx46", "atmel,at93c46d"
> 
> This needs to be clear that it should be the opposite order. Something 
> like this:
> 
> compatible: One of the following:
> 	"atmel,at93c46d"
> 	Followed by "eeprom-93xx46"

Will fix, thanks.

>>  - data-size : number of data bits per word (either 8 or 16)
>>  
>>  Optional properties:
>> -- 
>> 2.4.10
>>


- -- 
Cory Tusar
Principal
PID 1 Solutions, Inc.


"There are two ways of constructing a software design.  One way is to
 make it so simple that there are obviously no deficiencies, and the
 other way is to make it so complicated that there are no obvious
 deficiencies."  --Sir Charles Anthony Richard Hoare

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iEYEARECAAYFAlZLmPIACgkQHT1tsfGwHJ9FrACeKfmCgXePFz/N3v9QMV7c4ASj
SxcAnREBHu6lR5iAeyAtLpz8UCOOtDbn
=EfAl
-----END PGP SIGNATURE-----
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/