[OpenWrt-Devel] [PATCH 1/4] [WEIO] Adding WeIO profile
drasko.draskovic at gmail.com
Mon Jun 30 08:14:32 EDT 2014
On Mon, Jun 30, 2014 at 2:09 PM, John Crispin <john at phrozen.org> wrote:
> please resend with the patches inline, it is not possible to comment
> when they are attached.
No problem, I will be re-sending the patches inline.
> also, my first impression is that the patches are really messy and the
> commit text only partially matches what is inside the patches.
> 1/4 seems to have a pile of unrelated white space fixes.
> 1/4 adds a file only for 2/4 and 3/4 to "fix it"
Yes, this might be true, as this was my development process. Should
these be squashed then?
> 4/4 adds code with incorrect indenting
Yes, this should be fixed
> this is all not bad and we can fix it, however for that to be possible
> you need to send the patches inline.
> using git send-email is the best way to do so.
Will do this.
> last but not least, this is a carambola2 carrier board, with a lpc uC on
> it ? is this the uC that had a usb device stack in firmware ?
WeIO is based on Carambola2 module, but has some changes - like
different GPIOs for buttons and leds, some other peripherals, etc...
LPC uC has USB stack in the FW, and the all the FW can be found here:
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
More information about the openwrt-devel