Re: [PATCH] Add support for Newflow NanoBone board

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, Aug 11, 2013 at 10:27 AM, Mark Jackson <mpfj-list@xxxxxxxxxxxxx> wrote:
> On 08/07/13 19:57, Mark Jackson wrote:
>> NanoBone Specification:
>> -----------------------
>> CPU:
>>   TI AM335x @ 720MHz
>>
>> Memory:
>>   256MB DDR3
>>   128MB NOR flash
>>   128KB FRAM
>>
>> Ethernet:
>>   2 x 10/100 connected to SMSC LAN8710 PHY
>>
>> USB:
>>   1 x USB2.0 Type A
>>
>> I2C:
>>   2Kbit EEPROM (Microchip 24AA02)
>>   RTC (Maxim DS1338)
>>   GPIO Expander (Microchip MCP23017)
>>
>> Expansion connector:
>>   6 x UART
>>   1 x MMC/SD
>>   1 x USB2.0
>>
>> Signed-off-by: Mark Jackson <mpfj@xxxxxxxxxxxxx>
>> ---
>>  MAINTAINERS                       |    6 +
>>  arch/arm/boot/dts/Makefile        |    1 +
>>  arch/arm/boot/dts/am335x-nano.dts |  388 +++++++++++++++++++++++++++++++++++++
>>  3 files changed, 395 insertions(+)
>>  create mode 100644 arch/arm/boot/dts/am335x-nano.dts
>
> Any comments on this new board file ?

Hi Mark,

Overall it looks very good to me, I just have one small comment.
Please don't redefine device nodes that are already defined in the
included am33xx.dtsi file (pinmux, ocp, gpmc, etc). You should just
use what is defined there instead of defining it again in your board
DT file.

Also these are IP modules that are internal details of the SoC so is
better to keep them outside of board DT files definition.

Please take a look to my am33xx cleanups series [1] to see how it
should be defined. Once you have made that changes feel free to add
my:

Reviewed-by: Javier Martinez Canillas <javier.martinez@xxxxxxxxxxxxxxx>

Benoit seems to be on holidays so please be patient with him taking your patch.

[1]: http://www.mail-archive.com/linux-omap@xxxxxxxxxxxxxxx/msg93738.html

Thanks a lot and best regards,
Javier
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux