lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 22 Aug 2012 13:33:52 +0000 From: Tony Prisk <linux@...sktech.co.nz> To: Linus Walleij <linus.walleij@...aro.org> CC: "vt8500-wm8505-linux-kernel@...glegroup.com" <vt8500-wm8505-linux-kernel@...glegroup.com>, Russell King <linux@....linux.org.uk>, Alessandro Zummo <a.zummo@...ertech.it>, Alan Cox <alan@...ux.intel.com>, Greg Kroah-Hartman <gregkh@...uxfoundation.org>, Florian Tobias Schandinat <FlorianSchandinat@....de>, Arnd Bergmann <arnd@...db.de>, Grant Likely <grant.likely@...retlab.ca>, Rob Herring <rob.herring@...xeda.com>, Rob Landley <rob@...dley.net>, Linus Walleij <linus.walleij@...ricsson.com>, Mike Turquette <mturquette@...com>, Stephen Warren <swarren@...dia.com>, "linux-arm-kernel@...ts.infradead.org" <linux-arm-kernel@...ts.infradead.org>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, "linux-fbdev@...r.kernel.org" <linux-fbdev@...r.kernel.org>, "linux-usb@...r.kernel.org" <linux-usb@...r.kernel.org>, "linux-serial@...r.kernel.org" <linux-serial@...r.kernel.org>, "devicetree-discuss@...ts.ozlabs.org" <devicetree-discuss@...ts.ozlabs.org> Subject: RE: [rtc-linux] [PATCHv3 8/9] arm: vt8500: gpio: Devicetree support for arch-vt8500 >> Converted the existing arch-vt8500 gpio to a platform_device. >> Added support for WM8505 and WM8650 GPIO controllers. >(...) >> + unsigned val; >I asked about the datatype for this "val", it sure isn't "unsigned". >I suspected the registers were only 8bit and so it should be u8. >But atleast use u32 if you must use all these bits. Sorry - missed adding this to my list of fixes. I've used u32 as the datasheet advises these are 32-bit registers and must be read/written as such. Changes included in v4. >(...) >> + val = readl(vt8500_chip->base + vt8500_chip->regs->en); >> + val |= BIT(offset); >> + writel(val, vt8500_chip->base + vt8500_chip->regs->en); >BTW: have you considered [readl|writel]_relaxed? I haven't - and to be honest I didn't know what difference it would make until I read up on it. Most of this code already existed in mainline in arch/arm/mach-vt8500/gpio.c and I simply moved it across to drivers/gpio and made some changes/additions. Having looked, I don't see any reason why it can't use the _relaxed variants. Changes included in v4. I'll post up v4 for review when I get home in about 7 hours. Work always gets in the way of being productive :) Regards Tony Prisk-- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists