Message ID | 1358363143-7154-1-git-send-email-linux@prisktech.co.nz |
---|---|
State | New |
Headers | show |
On Thu, Jan 17, 2013 at 08:05:42AM +1300, Tony Prisk wrote: > UART modules on Wondermedia SoCs are connected via a gated clock > source, rather than directly to the 24Mhz reference clock. While > uboot enables UART0 for debugging, other UART ports are unavailable > until the clock is enabled. > > This patch checks that a valid clock is actually passed from devicetree, > enables the clock in probe. This change removes the fallback when a > clock was not specified as it doesn't apply any longer (and would only > work if the UART clock was already enabled). > > DTSI files are updated for VT8500, WM8505 and WM8650. > > Signed-off-by: Tony Prisk <linux@prisktech.co.nz> > --- > arch/arm/boot/dts/vt8500.dtsi | 40 +++++++++++++++++++++--- > arch/arm/boot/dts/wm8505.dtsi | 60 ++++++++++++++++++++++++++++++++---- > arch/arm/boot/dts/wm8650.dtsi | 20 ++++++++++-- > drivers/tty/serial/vt8500_serial.c | 34 +++++++++++--------- > 4 files changed, 127 insertions(+), 27 deletions(-) PLEASE always use checkpatch.pl on your patches before sending them, so people don't have to write emails saying "why didn't you fix the checkpatch.pl errors in your patch" much like this one. Sorry, not applied. greg k-h
On Thu, Jan 17, 2013 at 08:05:43AM +1300, Tony Prisk wrote: > Convert the last memory allocation (vt8500_port) to use devm_kzalloc > and remove the fail path cleanup code from vt8500_serial_probe. > > Reorder iomem mapping above clk_enable to simplify fail code. The > clock is only enabled if all other resources are available. > > Signed-off-by: Tony Prisk <linux@prisktech.co.nz> > --- > drivers/tty/serial/vt8500_serial.c | 13 ++++--------- > 1 file changed, 4 insertions(+), 9 deletions(-) As I dropped patch 3, I can't take this one either. greg k-h
On Thu, Jan 17, 2013 at 08:05:39AM +1300, Tony Prisk wrote: > Hi Greg, > > This should be the final pull request for this series, unless there are > other review comments. Changelog included. > > v2: > Restore the setting of vt8500_port->uart.uartclk which was dropped in v1. > Corrected the return-on-fail of devm_request_and_ioremap to -EADDRNOTAVAIL. > > v3: > Corrected the commit message for patch 3 > > > The following changes since commit 9931faca02c604c22335f5a935a501bb2ace6e20: > > Linux 3.8-rc3 (2013-01-09 18:59:55 -0800) > > are available in the git repository at: > > git://server.prisktech.co.nz/git/linuxwmt.git tags/vt8500/serial-fixes I don't take git pull requests for tiny stuff like this, sorry, only patches through email. But 2 of these patches had checkpatch issues, so I didn't take patches 3 and 4, please redo them. thanks, greg k-h