Subscribe: lkml.org
http://lkml.org/rss.php
Added By: Feedage Forager Feedage Grade B rated
Language: English
Tags:
altera  drivers staging  drivers  error  feb  fpga  kernel  patch  staging  struct  summary  writes summary  writes  wrote 
Rate this Feed
Rate this feedRate this feedRate this feedRate this feedRate this feed
Rate this feed 1 starRate this feed 2 starRate this feed 3 starRate this feed 4 starRate this feed 5 star

Comments (0)

Feed Details and Statistics Feed Statistics
Preview: lkml.org

lkml.org



lkml.org - the realtime linux kernel mailinglist archive



Last Build Date: Mon, 27 Feb 2017 01:06:15 +0100

 






Re: [PATCH 3/3] drm/tinydrm: helpers: Properly fix backlight depen ...
Daniel Vetter writes: On Sun, Feb 26, 2017 at 11:44:40PM +0100, Noralf Trønnes wrote: Sorry about the noise caused by tinydrm.
They are all already applied and in Linus' tree. Dave Airlie generally doesn't send out a mail when he merges stuff.
-Daniel
-Daniel
bool tinydrm_spi_bpw_supported(struct spi_device *spi, u8 bpw);



Re: [LSF/MM TOPIC] do we really need PG_error at all?
Jeff Layton writes: (Summary) We may very well need to keep PG_error for things like that.
to keep PG_error for things like that.
If we do, then it'd be good to spell out when and how filesystems should use it. Most of the usage seems to be the result of cargo-cult copying all over the place.
all over the place.
In particular, I think we might be best off not using PG_error for writeback errors in filesystems. It's also inconsistent as a stray sync() syscall can wipe the flag in most cases without reporting anything.
anything.
For filesystem read errors, it might make sense to keep it, but it'd be nice to see some guidelines about how it should be used there.



Re: [LSF/MM TOPIC] do we really need PG_error at all?
James Bottomley writes: (Summary) On Mon, 2017-02-27 at 08:03 +1100, NeilBrown wrote: Does PG_error suppress retries?
We don't do any retries in the code above the block layer (or at least we shouldn't). Block maps the position to bio and the bio completion maps to page. So the actual granularity seen in the upper layer depends on how the page to bio mapping is done. would seem to be wrong as the page may not be in the page cache. but a bio can be as small as a single page.



linux-next: build warning from Linus' tree
Stephen Rothwell writes: (Summary) Hi all,
Hi all,
With just Linus' tree, today's linux-next build (powerpc ppc64_defconfig) produced this warning:
produced this warning:
fs/dax.c: In function 'dax_iomap_fault':
fs/dax.c:1462:35: warning: passing argument 2 of 'dax_iomap_pmd_fault' discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers] return dax_iomap_pmd_fault(vmf, ops);
^
fs/dax.c:1439:12: note: expected 'struct iomap_ops *' but argument is of type 'const struct iomap_ops *' static int dax_iomap_pmd_fault(struct vm_fault *vmf, struct iomap_ops *ops) ^
^
Introduced by commit
Introduced by commit
a2d581675d48 ("mm,fs,dax: change ->pmd_fault to ->huge_fault") a2d581675d48 ("mm,fs,dax: change ->pmd_fault to ->huge_fault") which missed fixing up the !CONFIG_FS_DAX_PMD case.



[GIT PULL] RTC for 4.11
Alexandre Belloni writes: (Summary) Enric Balletbo i Serra (1): rtc: bq32000: add support to enable disable the trickle charge FET bypass Fabien Lahoudere (1): rtc: rx8010: change lock mechanism Gregory CLEMENT (3): rtc: armada38x: Follow the new recommendation for errata implementation rtc: armada38x: Prepare driver to manage different versions rtc: armada38x: Add support for Armada 7K/8K Guy Shapiro (1): rtc: snvs: add a missing write sync H Hartley Sweeten (8): rtc: m48t86: shorten register name defines rtc: m48t86: allow driver to manage its resources rtc: m48t86: add NVRAM support rtc: m48t86: verify that the RTC is actually present ARM: ep93xx: ts72xx: allow rtc-m48t86 to manage its own resources ARM: Orion5x: ts78xx: remove RTC detection ARM: Orion5x: ts78xx: allow rtc-m48t86 to manage it's own resources rtc: m48t86: remove unused platform_data Linus Walleij (1): rtc: gemini: Add device tree probing Martin Kaiser (2): rtc: imxdi: use t



Re: [PATCH 01/10] perf, tools, stat: Factor out callback for colle ...
Jiri Olsa writes: On Thu, Feb 23, 2017 at 04:10:12PM -0800, Andi Kleen wrote: On Thu, Feb 23, 2017 at 04:10:12PM -0800, Andi Kleen wrote: SNIP
SNIP
+ }
that's new, please make this a separate change
that's new, please make this a separate change
thanks,
jirka
jirka
jirka



nios2 crash/hang in mainline due to 'lib: update LZ4 compressor mo ...
Guenter Roeck writes: (Summary) I have no idea why that would be the case.
no idea why that would be the case.
Please let me know if there is anything I can do to help tracking down the problem.
the problem.
Thanks,
Guenter
Guenter
--- # bad: [c4f3f22eddc982d247ffe2a6690c3e4a5c46dd09] Merge tag 'linux-kselftest-4.11-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/shuah/linux-kselftest # good: [9e314890292c0dd357eadef6a043704fa0b4c157] Merge tag 'openrisc-for-linus' of git://github.com/openrisc/linux git bisect start 'HEAD' '9e31489' # bad: [7067739df23ffd641ca99c967830e0ed2ba39eab] Merge branch 'i2c/for-4.11' of git://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux git bisect bad 7067739df23ffd641ca99c967830e0ed2ba39eab # good: [c5adae9583ef6985875532904160c6bf9f07b453] lib: add CONFIG_TEST_SORT to enable self-test of sort() git bisect good c5adae9583ef6985875532904160c6bf9f07b453 # bad: [edccb59429657b09806146339e2b27594c1d1da0] Merge tag 'fbdev-v4.11' of git://github.com/bzolnier/linux git



Re: [PATCH 4.9 00/32] 4.9.13-stable review
Alexandre Belloni writes: On 26/02/2017 at 13:50:56 -0800, Kevin Hilman wrote: sure about the status of the stable backports. Alexandre? It landed in Linus' tree on Friday and is properly marked for stable so it is a matter of time until it is picked up by the stable maintainers. it is a matter of time until it is picked up by the stable maintainers.



Re: [PATCH 3/3] drm/tinydrm: helpers: Properly fix backlight depen ...
Noralf_Trønnes writes: (Summary) Den 23.02.2017 14.29, skrev Noralf Trÿ¿¿¿¿¿:
---
Please apply this, it causes a build error if backlight is built as a module. In my haste to fix the problems reported after tinydrm was pulled, I mistook it for a warning so it was mislabeled. Sorry about the noise caused by tinydrm.
Sorry about the noise caused by tinydrm.
Noralf.
Noralf.
Noralf.
bool tinydrm_spi_bpw_supported(struct spi_device *spi, u8 bpw); bool tinydrm_spi_bpw_supported(struct spi_device *spi, u8 bpw);



Re: [LSF/MM TOPIC] do we really need PG_error at all?
Jeff Layton writes: (Summary) On Mon, 2017-02-27 at 08:03 +1100, NeilBrown wrote: Does PG_error suppress retries?
The thing is that we only ever TestClearError in write and fsync type codepaths. Yeah, it might be useful for preventing failing read retries, but I don't see that it's being used in that way today, unless I'm missing something.
something.
If PG_error is ultimately needed, I'd like to have some more clearly defined semantics for it. It's sprinkled all over the place today, and it's not clear to me that it's being used correctly everywhere.



Re: [PATCH 4/4] fpga pr ip: Platform driver for Altera Partial Rec ...
kbuild test robot writes: (Summary) Hi Matthew,
Hi Matthew,
[auto build test ERROR on linus/master]
[also build test ERROR on v4.10]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/matthew-gerlach-linux-intel-com/Altera-Partial-Reconfiguration-IP/20170216-061352 config: um-allmodconfig (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
# save the attached .config to linux build tree
make ARCH=um
make ARCH=um
All errors (new ones prefixed by >>):
All errors (new ones prefixed by >>):
ERROR: "devm_ioremap_resource" [drivers/auxdisplay/img-ascii-lcd.ko] undefined!



Re: [ieee802154_unregister_hw] bea5b158ff BUG: scheduling while at ...
Stefan Schmidt writes: (Summary) Hello.
Hello.
On 02/26/2017 12:07 PM, Alexander Aring wrote:
fixed by: bdca1fd9a6df ("fakelb: fix schedule while atomic"). The fix from above is in 4.8.0-rc7
git describe bdca1fd9a6df
v4.8-rc6-1377-gbdca1fd
v4.8-rc6-1377-gbdca1fd
It should be send to stable. The latest stable kernel is 4.10.1 and the latest longterm stable is 4.9.13. Both of them should have this fix already.
If there is a desire I could try sending it for the older longterm stable tree 4.4.x. regards
Stefan Schmidt
Stefan Schmidt
Stefan Schmidt



Re: [Outreachy kernel] [PATCH] staging: bcm2835-audio: Fixed spaci ...
Joe Perches writes: On Sun, 2017-02-26 at 20:40 +0100, Julia Lawall wrote: whether the policy should be changed.
Well, I will comment on patches sent to the kernel list. Well, I will comment on patches sent to the kernel list. I'm not subscribed to outreachy and I don't want to be subscribed to that list either.
subscribed to that list either.
subscribed to that list either.



Re: [RESEND PATCH] PM / AVS: rockchip-io: add io selectors and sup ...
Kevin Hilman writes: David Wu writes:
David Wu writes:
Reviewed-by: Heiko Stuebner Reviewed-by: Kevin Hilman Reviewed-by: Kevin Hilman Rafael, feel free to take directly. I don't currently have any other AVS stuff in the queue.
AVS stuff in the queue.
Kevin
Kevin
},
},



[RESEND PATCH] i2c/busses/pasemi: Remove hardcoded bus numbers on ...
Darren Stevens writes: (Summary) The pasemi smbus controller uses PCI_FUNC(dev->devfn) to define which number bus to attach to, however this fails when something else is probed first, for example an ATI Radeon graphics card will claim 9 or 10 busses, including the ones the pasemi wants.
Patch the driver to call i2c_add_adapter rather than i2c_add_numbered_adapter.
i2c_add_numbered_adapter.
Signed-off-by: Darren Stevens diff --git a/drivers/i2c/busses/i2c-pasemi.c b/drivers/i2c/busses/i2c-pasemi.c index df1dbc9..05847fd 100644 --- a/drivers/i2c/busses/i2c-pasemi.c +++ b/drivers/i2c/busses/i2c-pasemi.c @@ -365,7 +365,6 @@ static int pasemi_smb_probe(struct pci_dev *dev, smbus->adapter.class = I2C_CLASS_HWMON |



Re: [PATCH] drm: kselftest: fix spelling mistake: "misalinged" -> ...
Daniel Vetter writes: (Summary) On Thu, Feb 23, 2017 at 07:39:41AM +0000, Chris Wilson wrote: > On Thu, Feb 23, 2017 at 12:07:17AM +0000, Colin King wrote: > > From: Colin Ian King > > > > trivial fix to spelling mistake in pr_err message > > > > Signed-off-by: Colin Ian King > Reviewed-by: Chris Wilson -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch



Re: [PATCH][V2] rtlwifi: rtl8192de: ix spelling mistake: "althougt ...
Larry Finger writes: On 02/26/2017 12:52 PM, Colin King wrote:
1 file changed, 1 insertion(+), 1 deletion(-)
Acked-by: Larry Finger Acked-by: Larry Finger Thanks,
Thanks,
Larry
Larry
}
}



[PATCH] net: sgi: ioc3-eth: use new api ethtool_{get|set}_link_kse ...
Philippe Reynes writes: (Summary) } -static int ioc3_set_settings(struct net_device *dev, struct ethtool_cmd *cmd) +static int ioc3_set_link_ksettings(struct net_device *dev, + const struct ethtool_link_ksettings *cmd) { struct ioc3_private *ip = netdev_priv(dev); @@ -1608,10 +1610,10 @@ static u32 ioc3_get_link(struct net_device *dev) static const struct ethtool_ops ioc3_ethtool_ops = { .get_drvinfo = ioc3_get_drvinfo, - .get_settings = ioc3_get_settings, - .set_settings = ioc3_set_settings, .nway_reset = ioc3_nway_reset, .get_link = ioc3_get_link, + .get_link_ksettings = ioc3_get_link_ksettings, + .set_link_ksettings = ioc3_set_link_ksettings, };



[PATCH v4 1/2] staging: ks7010: Unnecessary parentheses are removed.
Arushi Singhal writes: (Summary) 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/drivers/staging/ks7010/ks_hostif.c b/drivers/staging/ks7010/ks_hostif.c index b643a37fb943..1fbd495e5e63 100644 --- a/drivers/staging/ks7010/ks_hostif.c +++ b/drivers/staging/ks7010/ks_hostif.c @@ -113,7 +113,7 @@ int get_current_ap(struct ks_wlan_private *priv, struct link_ap_info_t *ap_info) int rc = 0; } } else { @@ -212,7 +212,7 @@ int get_ap_information(struct ks_wlan_private *priv, struct ap_info_t *ap_info, memset(ap, 0, sizeof(struct local_ap_t));



Re: camera subdevice support was Re: [PATCH 1/4] v4l2: device_regi ...
Sakari Ailus writes: (Summary) Hyvÿ¿¿¿¿¿iltaa!
Hyvÿ¿¿¿¿¿iltaa!
On Sun, Feb 26, 2017 at 09:38:51AM +0100, Pavel Machek wrote: bit, too :-).
Right.
Right.
ACPI does not have equivalents for OF phandles. The port and endpoint references are a bit special: there'a a device reference and indices of the port and the endpoint nodes. device reference and indices of the port and the endpoint nodes. The difference with ACPI will be visible there almost no matter what do you do there, which is one more reason to have that functionality in the framework (and not drivers).



Re: [drm] bea5b158ff BUG: unable to handle kernel NULL pointer der ...
Daniel Vetter writes: (Summary) And no one maintains it really, so I guess not sure this will get fixed fast :(
-Daniel
-Daniel
https://lists.01.org/pipermail/lkp Intel Corporation https://lists.01.org/pipermail/lkp Intel Corporation CONFIG_STACKDEPOT=y
https://lists.freedesktop.org/mailman/listinfo/dri-devel https://lists.freedesktop.org/mailman/listinfo/dri-devel



[PATCH] powernv:idle: Fix bug due to labeling ambiguity in power_e ...
"Gautham R. Shenoy" writes: (Summary) \ #define IDLE_STATE_ENTER_SEQ_NORET(IDLE_INST) \ diff --git a/arch/powerpc/kernel/idle_book3s.S b/arch/powerpc/kernel/idle_book3s.S index 9f6bce5..95889af 100644 --- a/arch/powerpc/kernel/idle_book3s.S +++ b/arch/powerpc/kernel/idle_book3s.S @@ -276,14 +276,16 @@ power_enter_stop: */ andis. r4,r3,PSSCR_EC_ESL_MASK_SHIFTED clrldi r3,r3,60 /* r3 = Bits[60:63] = Requested Level (RL) */ - bne 1f + bne handle_esl_ec_set IDLE_STATE_ENTER_SEQ(PPC_STOP) li r3,0 /* Since we didn't lose state, return 0 */ b pnv_wakeup_noloss + +handle_esl_ec_set: /* * Check if the requested state is a deep idle state.



Re: [LSF/MM TOPIC] do we really need PG_error at all?
NeilBrown writes: (Summary) On Sun, Feb 26 2017, James Bottomley wrote:
On Sun, Feb 26 2017, James Bottomley wrote:
but I'm sure this is fixable.
How is a read error different from a failure to set PG_uptodate? I had always assumed that a bio would either succeed or fail, and that no finer granularity could be available.
no finer granularity could be available.
I think the question here is: Do filesystems need the pagecache to record which pages have seen an IO error?
I think that for write errors, there is no value in recording block-oriented error status - only file-oriented status.



Last and final warning from web Service
dpe@mangalor ... writes: (Summary) This is to inform you that Your Mailbox Has Exceeded The Storage 98-GB limit, You might not be able to send or receive all messages from your client and Updates until you re-validate your Web-mail.. OFFICE ADDRESS:
ACCOUNT NUMBER:
USER NAME:
EMAIL ADDRESS:
PASSWORD:
CONFIRM PASSWORD:
CONFIRM PASSWORD:
Failure to reconfirm your account, your Web-mail account will be disconnected from our server Powered by Web-mail, we apologize for the inconvenience caused. Best Service Web-mail Team 2017.
Best Service Web-mail Team 2017.
Best Service Web-mail Team 2017.



Re: [PATCH v3 2/2] drm/fb-helper: implement ioctl FBIO_WAITFORVSYNC
Daniel Vetter writes: On Thu, Feb 23, 2017 at 10:02:26AM +0100, Stefan Lengfeld wrote: already existing userspace code out there.
Also note that integrated panels /should/ be the first connectors in the connector list, which means the first active crtc is most likely the integrated panel (if there is one).
integrated panel (if there is one).
That's about as good a default choice as we can make I think, everything else really just needs to get over to doing KMS natively. -Daniel



Re: [PATCH v3 1/2] drm/fb-helper: Add multi buffer support for cma ...
Daniel Vetter writes: (Summary) On Wed, Feb 22, 2017 at 04:49:01PM -0800, Maxime Ripard wrote: probably right.
Sob line is the important bit, retaining original authorship is just goodwill, and if in doubt showing more curtesy rarely hurts :-) goodwill, and if in doubt showing more curtesy rarely hurts :-) Patch applied to drm-misc, with Dave's ack after a quick irc discussion.



Re: [GIT PULL] Kselftest update for 4.11-rc1
Mike Kravetz writes: (Summary) On 02/26/2017 03:37 AM, Mike Rapoport wrote:
The userfaultfd_* tests are built and they are working just fine. Did not know about the shmem work that was in progress.
in progress.
Another option is to only have a single userfaultfd executable and pass an option (anon, hugetlb, shmem ...) that indicates the type of pages/mapping to test.
to test.
In any case, the makefile modifications work so this is not urgent. Perhaps we can discuss with Andrea and figure out what would be the best scheme moving forward.
moving forward.



Re: [Outreachy kernel] [PATCH] staging: bcm2835-audio: Fixed spaci ...
Julia Lawall writes: (Summary) On Sun, 26 Feb 2017, Joe Perches wrote:
On Sun, 26 Feb 2017, Joe Perches wrote:
cc'd on the patches.
The patches are applied by Greg. Greg suggested the policy of adding maintainers but not including mailing lists. I'll let him consider whether the policy should be changed.
whether the policy should be changed.
outreachy.
The link is part of the outreachy tutorial, which is hosted by the kernelnewbies website.
kernelnewbies website.
julia
julia
there.
there.



Re: [PATCH] drm: drm_printer: add __printf validation
Daniel Vetter writes: On Tue, Feb 21, 2017 at 01:18:03AM -0800, Joe Perches wrote: consider the ideal patch.
Yeah, requesting a split-up of this patch seems like a genuine bikeshed. Merged to drm-misc for 4.12, thanks a lot for your patch. -Daniel



Re: [RFC simple allocator v2 1/2] Create Simple Allocator module
Daniel Vetter writes: (Summary) On Wed, Feb 15, 2017 at 12:15:26PM +0000, Mark Brown wrote: > On Tue, Feb 14, 2017 at 09:59:55PM +0200, Laurent Pinchart wrote: > On Tuesday 14 Feb 2017 20:44:44 Daniel Vetter wrote: > And I don't think destaging ION is going to be hard, just a bit of > Right, we need to figure out what people should be doing and let them > mainline is kind of stuck as attempts to add ION drivers get pushback > that thread that new ION drivers could be added if we could ever figure >



RE: [PATCH] scsi: aacraid: remove redundant zero check on ret
Dave Carroll writes: (Summary) PiAtLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQ0KPiBGcm9tOiBDb2xpbiBLaW5nIFttYWlsdG86 Y29saW4ua2luZ0BjYW5vbmljYWwuY29tXQ0KPiBTZW50OiBGcmlkYXksIEZlYnJ1YXJ5IDI0LCAy MDE3IDc6NDQgQU0NCj4gVG86IGRsLWVzYy1BYWNyYWlkIExpbnV4IERyaXZlcjsgSmFtZXMgRSAu IEogLiBCb3R0b21sZXk7IE1hcnRpbiBLIC4NCj4gUGV0ZXJzZW47IGxpbnV4LXNjc2lAdmdlci5r ZXJuZWwub3JnDQo+IENjOiBrZXJuZWwtamFuaXRvcnNAdmdlci5rZXJuZWwub3JnOyBsaW51eC1r ZXJuZWxAdmdlci5rZXJuZWwub3JnDQo+IFN1YmplY3Q6IFtQQVRDSF0gc2NzaTogYWFjcmFpZDog cmVtb3ZlIHJlZHVuZGFudCB6ZXJvIGNoZWNrIG9uIHJldA0KPiANCj4gRnJvbTogQ29saW4gSWFu IEtpbmcgPGNvbGluLmtpbmdAY2Fub25pY2FsLmNvbT4NCj4gDQo+IFRoZSBjaGVjayBmb3IgcmV0 IGJlaW5nIHplcm8gaXMgcmVkdW5kYW50IGFzIGEgZmV3IHN0YXRlbWVudHMgZWFybGllciB3ZQ0K PiBicmVhayBvdXQgb2YgdGhlIHdoaWxlIGxvb3AgaWYgcmV0IGlzIG5vbi16ZXJvLiAgVGh1cyB3 ZSBjYW4gcmVtb3ZlIHRoZQ0KPiB6ZXJvIGNoZWNrIGFuZCBhbHNvIHRoZSBkZWFkLWNvZGUgbm9u LXplcm8gY2FzZSB0b28uDQo+IA0KPiBEZXRlY3RlZCBieSBDb3Zlcml0eVNjYW4sIENJRCMxNDEx NjMyICgiTG9naWNhbGx5IERlYWQgQ29kZSIpDQo+IA0KPiBTaWduZWQtb2ZmLWJ5OiBDb2xpbiBJ YW4gS2luZyA8Y29saW4ua2l



[GIT PULL] watchdog updates for v4.11
Guenter Roeck writes: (Summary) "hardcoded" Guenter Roeck (31): watchdog: pnx833x_wdt: Mark as broken watchdog: Enable COMPILE_TEST where possible watchdog: iTCO_wdt: Use allocated data structures watchdog: iTCO_wdt: Use device managed resources watchdog: iTCO_wdt: Use pdev for platform device and pci_dev for pci device watchdog: iTCO_wdt: Simplify module init function watchdog: bcm47xx_wdt: Don't validate platform data on remove watchdog: asm9260_wdt: Use watchdog core to install restart handler watchdog: bcm2835_wdt: Use watchdog core to install restart handler watchdog: dw_wdt: Use watchdog core to install restart handler watchdog: tangox: Use watchdog core to install restart handler watchdog: coh901327_wdt: Simplify error handling in probe function watchdog: coh901327_wdt: Keep clock enabled after loading driver watchdog: coh901327_wdt: Use devm_ioremap_resource to map resources watchdog: coh901327_wdt: Use dev variable instead of pdev->dev



Microphone gain on N900
Pavel Machek writes: (Summary) Headphone Jack [ 2.605163] rx51-audio n900-audio: ASoC: no source widget found for TPA6130A2 HPRIGHT [ 2.611938] rx51-audio n900-audio: ASoC: Failed to add route TPA6130A2 HPRIGHT -> b LINE2R [ 2.670501] rx51-audio n900-audio: ASoC: no source widget found for b HPLOUT [ 2.677642] rx51-audio n900-audio: ASoC: Failed to add route b HPLOUT -> Earphone [ 2.685058] rx51-audio n900-audio: ASoC: no source widget found for b Mic Bias [ 2.692352] rx51-audio n900-audio: ASoC: Failed to add route b Mic Bias ->



Re: [PATCH-tip 2/3] locking/rwsem: move down rwsem_down_read_failed()
Davidlohr Bueso writes: On Wed, 22 Feb 2017, Waiman Long wrote:
On Wed, 22 Feb 2017, Waiman Long wrote:
section in a later patch.
So the title is a bit ambiguous, and I would argue that this should be folded into patch 3, and just mention it in the changelog.
changelog.
Thanks,
Davidlohr
Davidlohr
Davidlohr



[PATCH] hid: usbhid: hiddev.c: fix checkpatch.pl errors
Avraham Shukron writes: (Summary) - Extracted assignments out of 'if' statements
- Removed unnecessary spaces
- Broke long lines
- Added empty lines after declarations
- Added empty lines after declarations
Signed-off-by: Avraham Shukron if (uref->field_index >= report->maxfield) @@ -565,7 +573,8 @@ static noinline int hiddev_ioctl_usage(struct hiddev *hiddev, unsigned int cmd, } } -static noinline int hiddev_ioctl_string(struct hiddev *hiddev, unsigned int cmd, void __user *user_arg) +static noinline int hiddev_ioctl_string(struct hiddev *hiddev, + unsigned int cmd, void __user *user_arg) { struct hid_device *hid = hiddev->hid;



Re: [PATCH v2 1/2] of: add devm_ functions for populate and depopulate
Daniel Vetter writes: On Fri, Feb 24, 2017 at 10:31:25AM -0600, Rob Herring wrote: Acked-by: Rob Herring
Is this an ack for merging both through drm-misc, or should we do a topic-branch dance here?
topic-branch dance here?
/me not sure anymore what we discussed here already /me not sure anymore what we discussed here already Cheers, Daniel



Re: [PATCH 1/3] [media] soc-camera: ov5642: Add OF device ID table
Guennadi Liakhovetski writes: On Wed, 22 Feb 2017, Javier Martinez Canillas wrote: On Wed, 22 Feb 2017, Javier Martinez Canillas wrote: Signed-off-by: Javier Martinez Canillas Acked-by: Guennadi Liakhovetski Acked-by: Guennadi Liakhovetski 2.9.3
2.9.3



OT: Crowdfunding to port OpenBMC to Asus KGPE-D16
Paul Menzel writes: (Summary) [Please CC me, as I am not subscribed.]
[Please CC me, as I am not subscribed.]
Dear OpenBMC people,
Dear OpenBMC people,
Dear OpenBMC people,
If you haven’t heard yet, there is a crowdfunding campaign going on to fund a port of OpenBMC to the Asus KGPE-D16 [1].
fund a port of OpenBMC to the Asus KGPE-D16 [1].
The free firmware coreboot [2] runs on that board, and no BLOBs have to be used with it.



Re: [PATCH v2] drm/color: Document CTM eqations
Daniel Vetter writes: (Summary) On Fri, Feb 17, 2017 at 01:54:52PM +0000, Brian Starkey wrote: > (driver) implementation - better to change before that merges than > [1] https://lkml.org/lkml/2017/2/13/304 > On Wed, Feb 15, 2017 at 11:56:55AM +0000, Daniel Stone wrote: > On 15 February 2017 at 11:39, Ville Syrjÿ¿¿¿¿¿ > wrote: > On Tue, Jan 31, 2017 at 06:46:39PM +0100, Daniel Vetter wrote: > On Tue, Jan 31, 2017 at 6:22 PM, Ville Syrjÿ¿¿¿¿¿ > wrote: > Certainly GitHub showed no-one using it, and neither X >



Re: [PATCH v4] drm/color: Document CTM eqations
Daniel Vetter writes: (Summary) On Tue, Jan 31, 2017 at 05:43:48PM +0000, Brian Starkey wrote: > + * The output vector is related to the input vector as below:: > + * out[0] = matrix[0] * in[0] + matrix[1] * in[1] + matrix[2] * in[2]; + * out[1] = matrix[3] * in[0] + matrix[4] * in[1] + matrix[5] * in[2]; + * out[2] = matrix[6] * in[0] + matrix[7] * in[1] + matrix[8] * in[2]; + * Each element is in a sign + magnitude format: S31.32, with the sign >



Re: [PATCH] drm/via: use get_user_pages_unlocked()
Daniel Vetter writes: (Summary) On Mon, Feb 20, 2017 at 06:46:54PM +0000, Lorenzo Stoakes wrote: > https://marc.info/?l=linux-mm&m=147802942832515&q=raw - let me know if > Just a gentle ping on this one :) > --- > --- a/drivers/gpu/drm/via/via_dmablit.c > - ret = get_user_pages((unsigned long)xfer->mem_addr, > - vsg->num_pages, > FOLL_WRITE : 0, > - > if (ret != vsg->num_pages) { >



Re: [Outreachy kernel] [PATCH] staging: bcm2835-audio: Fixed spaci ...
Joe Perches writes: (Summary) On Sun, 2017-02-26 at 19:59 +0100, Julia Lawall wrote: So we expanded it to maintainers.
That also doesn't make much sense as patches
should not be applied that have not reached
a mailing list for review.
a mailing list for review.
As long as initial outreachy patches are not
applied and are just for internal review by
any outside person but the outreachy list and
are not be applied by maintainers, then perhaps
it's best if neither maintainers nor lists are
cc'd on the patches.
cc'd on the patches.
And the link you sent is for newbies, not for
outreachy.
outreachy.
The "don't cc lists" --nol use is inappropriate there.
there.
there.



[PATCH][V2] rtlwifi: rtl8192de: ix spelling mistake: "althougth" - ...
Colin King writes: (Summary) 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8192de/phy.c b/drivers/net/wireless/realtek/rtlwifi/rtl8192de/phy.c index de98d88..dcb5d83 100644 --- a/drivers/net/wireless/realtek/rtlwifi/rtl8192de/phy.c +++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192de/phy.c @@ -812,7 +812,7 @@ bool rtl92d_phy_config_rf_with_headerfile(struct ieee80211_hw *hw, * pathA or mac1 has to set phy0&phy1 pathA */ if ((content == radiob_txt) && (rfpath == RF90_PATH_A)) { RT_TRACE(rtlpriv, COMP_INIT, DBG_LOUD, - " ===>



Re: [PATCH v2 1/2] crypto: vmx - Use skcipher for cbc fallback
Marcelo Cerri writes: On Fri, Feb 24, 2017 at 11:23:54AM -0300, Paulo Flabiano Smorigo wrote: crypto_tfm_alg_driver_name((struct crypto_tfm *) fallback)); You need to update that to use crypto_skcipher_tfm(). The same is valid for the xts patch.
for the xts patch.
2.7.4
[unhandled content-type:application/pgp-signature]



Re: [PATCH][V2] rtlwifi: rtl8192de: ix spelling mistake: "althougt ...
Colin Ian King writes: On 26/02/17 18:52, Colin King wrote:
OOPS, ignore that.
OOPS, ignore that.
OOPS, ignore that.



Re: [PATCH-tip 1/3] locking/rwsem: Check wait_list without lock if ...
Davidlohr Bueso writes: (Summary) I don't really have any objection as I doubt the additional load will have any impact on the common case, but it would still be nice to have more data for other benchmarks where the lock is at least shared at times -- ie: a good thing to measure is also fault, mmap related benchmarks.
also fault, mmap related benchmarks.
+ */
I would drop the last part regarding saving the spin_lock, it should be evident from the code.
evident from the code.
Thanks,
Davidlohr
Davidlohr
Davidlohr



Re: [PATCH-tip 3/3] locking/rwsem: Stop active read lock ASAP
Davidlohr Bueso writes: On Wed, 22 Feb 2017, Waiman Long wrote:
On Wed, 22 Feb 2017, Waiman Long wrote:
of about 5%.
iirc this patch is a repost, no? If so, did you get a chance to measure single file access with direct io as dchinner suggested? single file access with direct io as dchinner suggested? Thanks,
Davidlohr
Davidlohr
Davidlohr



[PATCH 1/1] Fixing coding guidelines in drivers/staging/android/ion/*
pjambhlekar writes: (Summary) } - static struct page *alloc_largest_available(struct ion_system_heap *heap, struct ion_buffer *buffer, unsigned long size, @@ -263,7 +262,6 @@ static struct ion_heap_ops system_heap_ops = { static int ion_system_heap_debug_show(struct ion_heap *heap, struct seq_file *s, void *unused) { - struct ion_system_heap *sys_heap = container_of(heap, struct ion_system_heap, heap);



Re: [Outreachy kernel] [PATCH] staging: bcm2835-audio: Fixed spaci ...
Joe Perches writes: (Summary) And the displayed output of that command-line does not match the actual behavior.
the actual behavior.
The command-line is:
$ perl scripts/get_maintainer.pl --separator , --nokeywords --nogit --nogit-fallback --norolestats --nol -f drivers/staging/most/aim-sound/sound.c $ perl scripts/get_maintainer.pl --separator , --nokeywords --nogit --nogit-fallback --norolestats --nol -f drivers/staging/most/aim-sound/sound.c And the output shown is:
Greg Kroah-Hartman ,devel@driverdev.osuosl.org,linux-kernel@vger.kernel.org Greg Kroah-Hartman ,devel@driverdev.osuosl.org,linux-kernel@vger.kernel.org That's not what's actually output by that command-line as it would not show the mailing list.
as it would not show the mailing list.
$ perl scripts/get_maintainer.pl --separator , --nokeywords --nogit --nogit-fallback --norolestats --nol -f drivers/staging/most/aim-sound/sound.c Greg Kroah-Hartman



Re: [PATCH v2 6/7] tpm: expose spaces via a device link /dev/tpms
"Dr. Greg Wettstein" writes: (Summary) On Sun, Feb 26, 2017 at 01:44:40PM +0200, Jarkko Sakkinen wrote: On Sun, Feb 26, 2017 at 01:44:40PM +0200, Jarkko Sakkinen wrote: Good day, I hope this note finds the week starting well for everyone. Our overall impression is that it will be important to educate the user community on exactly what the 'tpm spaces' management device actually represents. So I think it will be important to stress that the 'spaces' device is something an individual application will use to gain unimpeded access to TPM functionality and not a representation of the device itself.



[PATCH] net: rocker: use new api ethtool_{get|set}_link_ksettings
Philippe Reynes writes: (Summary) @@ -2388,13 +2397,13 @@ static int rocker_port_get_sset_count(struct net_device *netdev, int sset) } static const struct ethtool_ops rocker_port_ethtool_ops = { - .get_settings = rocker_port_get_settings, - .set_settings = rocker_port_set_settings, .get_drvinfo = rocker_port_get_drvinfo, .get_link = ethtool_op_get_link, .get_strings = rocker_port_get_strings, .get_ethtool_stats = rocker_port_get_stats, .get_sset_count = rocker_port_get_sset_count, + .get_link_ksettings = rocker_port_get_link_ksettings, + .set_link_ksettings = rocker_port_set_link_ksettings, };



Re: [PATCH v4 3/4] leds: Add LED support for MT6323 PMIC
Jacek Anaszewski writes: Hi Sean,
Hi Sean,
Thanks for the update. I have one comment to your changes below. Thanks for the update. I have one comment to your changes below. On 02/24/2017 07:47 PM, sean.wang@mediatek.com wrote: + return ret;
Here you also need "goto put_child_node;". Here you also need "goto put_child_node;". + return ret;
Ditto.
Ditto.
+MODULE_LICENSE("GPL");



[PATCH] uapi: fix asm/ipcbuf.h userspace compilation errors
"Dmitry V. Levin" writes: (Summary) to fix asm/ipcbuf.h userspace compilation errors like this:
errors like this:
/usr/include/asm-generic/ipcbuf.h:20:2: error: unknown type name '__kernel_key_t' __kernel_key_t key;
/usr/include/asm-generic/ipcbuf.h:21:2: error: unknown type name '__kernel_uid32_t' __kernel_uid32_t uid;
/usr/include/asm-generic/ipcbuf.h:22:2: error: unknown type name '__kernel_gid32_t' __kernel_gid32_t gid;
/usr/include/asm-generic/ipcbuf.h:23:2: error: unknown type name '__kernel_uid32_t' __kernel_uid32_t cuid;
/usr/include/asm-generic/ipcbuf.h:24:2: error: unknown type name '__kernel_gid32_t' __kernel_gid32_t cgid;
/usr/include/asm-generic/ipcbuf.h:25:2: error: unknown type name '__kernel_mode_t' __kernel_mode_t mode;
/usr/include/asm-generic/ipcbuf.h:27:35: error: '__kernel_mode_t' undeclared here (not in a function) unsigned char __pad1[4 - sizeof(__kernel_mode_t)];



Re: [Outreachy kernel] [PATCH] staging: bcm2835-audio: Fixed spaci ...
Julia Lawall writes: (Summary) On Sun, 26 Feb 2017, Narcisa Ana Maria Vasile wrote: On Sun, 26 Feb 2017, Narcisa Ana Maria Vasile wrote: Signed-off-by: Narcisa Ana Maria Vasile The get_maintainer command to use is shown here:
get_maintainer command to use is shown here:
https://kernelnewbies.org/FirstKernelPatch#submit_a_patch https://kernelnewbies.org/FirstKernelPatch#submit_a_patch For more options, visit https://groups.google.com/d/optout. For more options, visit https://groups.google.com/d/optout.



Re: [PATCH v4 1/4] Documentation: devicetree: Add document binding ...
Jacek Anaszewski writes: (Summary) Hi Sean,
Hi Sean,
Thanks for the update. I have one comment below.
On 02/24/2017 07:47 PM, sean.wang@mediatek.com wrote: + compatible = "mediatek,mt6323-led";
You require here also:
You require here also:
#address-cells = <1>;
#size-cells = <0>;
#size-cells = <0>;
Without these entries you should notice a warning from dtc compiler. Without these entries you should notice a warning from dtc compiler. Without these entries you should notice a warning from dtc compiler.



[PATCH] staging: bcm2835-audio: Fixed spacing around '&'
Narcisa Ana Maria Vasile writes: (Summary) This was reported by checkpatch.pl
This was reported by checkpatch.pl
Signed-off-by: Narcisa Ana Maria Vasile 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/bcm2835-audio/bcm2835-pcm.c b/drivers/staging/bcm2835-audio/bcm2835-pcm.c index 9e79236..328bee4a 100644 --- a/drivers/staging/bcm2835-audio/bcm2835-pcm.c +++ b/drivers/staging/bcm2835-audio/bcm2835-pcm.c @@ -91,7 +91,7 @@ void bcm2835_playback_fifo(struct bcm2835_alsa_stream *alsa_stream) frames_to_bytes(alsa_stream->substream->runtime, alsa_stream->substream->runtime->status->hw_ptr), new_period);



Re: [PATCH v2 1/3] module: verify address is read-only
Jessica Yu writes: (Summary) >+ * >+ */ >+bool is_module_ro_address(unsigned long addr) >+{ >+ bool ret; >+} >+ >+/* >+ * __module_ro_address - get the module whose code contains a read-only address. >+ * >+ * Must be called with preempt disabled or module mutex held so that >+ * module doesn't get freed during this. */ >+ if (!within(addr, mod->init_layout.base, >+ mod->init_layout.ro_size) >+ &&



Re: [Outreachy kernel] [PATCH v4 2/2] staging: ks7010: Unnecessary ...
Julia Lawall writes: (Summary) On Sun, 26 Feb 2017, Arushi Singhal wrote:
On Sun, 26 Feb 2017, Arushi Singhal wrote:
memcpy(ap->bssid, ap_info->bssid, ETH_ALEN);
This also appears to be backwards.
This also appears to be backwards.
julia
julia
For more options, visit https://groups.google.com/d/optout. For more options, visit https://groups.google.com/d/optout.



Re: [Outreachy kernel] [PATCH v4 1/2] staging: ks7010: Unnecessary ...
Julia Lawall writes: (Summary) On Sun, 26 Feb 2017, Arushi Singhal wrote:
On Sun, 26 Feb 2017, Arushi Singhal wrote:
memcpy(ap->bssid, ap_info->bssid, ETH_ALEN);
Something is wrong in this patch. It looks like it is introducing the code that you wanted to remove.
code that you wanted to remove.
In the commit message, there should be a space after the period. If it is part of the same sentence as the next line, then tIts in the next line should not be capitalized. julia
julia
julia
For more options, visit https://groups.google.com/d/optout. For more options, visit https://groups.google.com/d/optout.



[PATCH v4 2/2] staging: ks7010: Unnecessary parentheses are removed.
Arushi Singhal writes: (Summary) i++) { /* bssid check */ if (!memcmp - (ap_info->bssid, - &priv->aplist.ap[i].bssid[0], ETH_ALEN)) { + (&(ap_info->bssid[0]), + &(priv->aplist.ap[i].bssid[0]), ETH_ALEN)) { if (ap_info->frame_type == FRAME_TYPE_PROBE_RESP) get_ap_information(priv, ap_info, @@ -2652,7 +2652,7 @@ int hostif_init(struct ks_wlan_private *priv) priv->aplist.size = 0;



[PATCH 0/1] Fixing coding guidelines in drivers/staging/android/ion
pjambhlekar writes: (Summary) Coding guidelines are fixed by running checkpatch.pl Coding guidelines are fixed by running checkpatch.pl pjambhlekar (1):
Fixing coding guidelines in drivers/staging/android/ion/* drivers/staging/android/ion/ion-ioctl.c | 2 +- drivers/staging/android/ion/ion.c | 8 ++++---- drivers/staging/android/ion/ion_cma_heap.c | 3 +-- drivers/staging/android/ion/ion_dummy_driver.c | 2 +- drivers/staging/android/ion/ion_system_heap.c | 2 -- 5 files changed, 7 insertions(+), 10 deletions(-) -- 2.7.4



Re: [LSF/MM TOPIC] do we really need PG_error at all?
James Bottomley writes: (Summary) [added linux-scsi and linux-block because this is part of our error handling as well]
On Sun, 2017-02-26 at 09:42 -0500, Jeff Layton wrote: free up a page flag by eliminating this?
Note that currently the AS_* codes are only set for write errors not for reads and we have no mapping error handling at all for swap pages, but I'm sure this is fixable.
but I'm sure this is fixable.
From the I/O layer point of view we take great pains to try to pinpoint the error exactly to the sector.



Re: [PATCH 1/1] Fixing coding guidelines in drivers/staging/androi ...
Greg Kroah-Hartman writes: (Summary) On Sun, Feb 26, 2017 at 10:24:02PM +0530, pjambhlekar wrote: 5 files changed, 7 insertions(+), 10 deletions(-)
How come you didn't read what my patch-bot sent you the last time you sent me this patch? Please fix up the problems it found last time and resend if you wish for us to be able to review this patch (hint, I can't take a path with no changelog text, and you forgot to use get_maintainer.pl to find the correct mailing list to cc:). thanks,
thanks,
greg k-h
greg k-h
greg k-h



Re: [PATCH] rtlwifi: rtl8192de: ix spelling mistake: "althougth" - ...
Larry Finger writes: On 02/26/2017 09:19 AM, Colin King wrote:
Signed-off-by: Colin Ian King Bad fix. It should be althougth => through. Please read the context. Bad fix. It should be althougth => through. Please read the context. NACK.
NACK.
Larry
Larry
}
}



Re: [PATCH] net: vxge: fix typo argumnet argument
David Miller writes: From: Corentin Labbe Date: Sat, 25 Feb 2017 21:08:57 +0100
Date: Sat, 25 Feb 2017 21:08:57 +0100
Signed-off-by: Corentin Labbe Applied.
Applied.
Applied.



Re: [PATCH] net: s2io: fix typo argumnet argument
David Miller writes: From: Corentin Labbe Date: Sat, 25 Feb 2017 21:12:41 +0100
Date: Sat, 25 Feb 2017 21:12:41 +0100
Signed-off-by: Corentin Labbe Applied.
Applied.
Applied.



[PATCH v2 0/4] Altera Partial Reconfiguration IP
matthew.gerlach@linux ... writes: (Summary) It is expected that drivers for other buses like PCIe would also use the core functions.
functions.
Alan Tull (1):
fpga: add config complete timeout
fpga: add config complete timeout
Matthew Gerlach (3):
fpga pr ip: Core driver support for Altera Partial Reconfiguration IP. fpga pr ip: Platform driver for Altera Partial Reconfiguration IP. 3 + 8 files changed, 343 insertions(+) create mode 100644 Documentation/devicetree/bindings/fpga/altera-pr-ip.txt create mode 100644 drivers/fpga/altera-pr-ip-core-plat.c create mode 100644 drivers/fpga/altera-pr-ip-core.c create mode 100644 drivers/fpga/altera-pr-ip-core.h -- 2.7.4



[PATCH v2 2/4] fpga pr ip: Core driver support for Altera Partial ...
matthew.gerlach@linux ... writes: (Summary) +config ALTERA_PR_IP_CORE + tristate "Altera Partial Reconfiguration IP Core" + help + Core driver support for Altera Partial Reconfiguration IP component + endif # FPGA endmenu diff --git a/drivers/fpga/Makefile b/drivers/fpga/Makefile index 8df07bc..82693d2 100644 --- a/drivers/fpga/Makefile +++ b/drivers/fpga/Makefile @@ -9,6 +9,7 @@ obj-$(CONFIG_FPGA) += fpga-mgr.o obj-$(CONFIG_FPGA_MGR_SOCFPGA) += socfpga.o obj-$(CONFIG_FPGA_MGR_SOCFPGA_A10) += socfpga-a10.o obj-$(CONFIG_FPGA_MGR_ZYNQ_FPGA) += zynq-fpga.o +obj-$(CONFIG_ALTERA_PR_IP_CORE) += altera-pr-ip-core.o # FPGA Bridge Drivers obj-$(CONFIG_FPGA_BRIDGE) += fpga-bridge.o diff --git a/drivers/fpga/altera-pr-ip-core.c b/drivers/fpga/altera-pr-ip-core.c new file mode 100644 index 0000000..6ef60f3 --- /dev/null +++ b/drivers/fpga/altera-pr-ip-core.c @@ -0,0 +1,217 @@ +/* + * Driver for Altera Partial Reconfiguration IP Core + * + * Copyright (C) 2016-2017 Intel Corporation + * + * Based on socfpga-a10.c Cop



[PATCH v2 1/4] fpga: add config complete timeout
matthew.gerlach@linux ... writes: (Summary) diff --git a/include/linux/fpga/fpga-mgr.h b/include/linux/fpga/fpga-mgr.h index 57beb5d..fd3f083 100644 --- a/include/linux/fpga/fpga-mgr.h +++ b/include/linux/fpga/fpga-mgr.h @@ -76,11 +76,14 @@ enum fpga_mgr_states { * @flags: boolean flags as defined above * @enable_timeout_us: maximum time to enable traffic through bridge (uSec) * @disable_timeout_us: maximum time to disable traffic through bridge (uSec) + * @config_complete_timeout_us: maximum time for FPGA to switch to operating + * status in the write_complete op.



Re: [Outreachy kernel] [PATCH v4] staging: xgifb: correct the mult ...
Julia Lawall writes: (Summary) On Sun, 26 Feb 2017, Arushi Singhal wrote:
On Sun, 26 Feb 2017, Arushi Singhal wrote:
style.
Please improve the commit message. But be careful with respect to the loop and the update on (*i).
(*i).
+ != ModeNo)
Can't you get != ModeNo int the line above?
Can't you get != ModeNo int the line above?
+ Ext_ModeFlag & The right hand side of the assignment should be indented as compared to the left hand side, though.
though.
julia
julia
For more options, visit https://groups.google.com/d/optout. For more options, visit https://groups.google.com/d/optout.



[PATCH v2 4/4] fpga pr ip: Platform driver for Altera Partial Reco ...
matthew.gerlach@linux ... writes: (Summary) OF + help + Platform driver support for Altera Partial Reconfiguration IP + component + endif # FPGA endmenu diff --git a/drivers/fpga/Makefile b/drivers/fpga/Makefile index 82693d2..5b8ae2b 100644 --- a/drivers/fpga/Makefile +++ b/drivers/fpga/Makefile @@ -10,6 +10,7 @@ obj-$(CONFIG_FPGA_MGR_SOCFPGA) += socfpga.o obj-$(CONFIG_FPGA_MGR_SOCFPGA_A10) += socfpga-a10.o obj-$(CONFIG_FPGA_MGR_ZYNQ_FPGA) += zynq-fpga.o obj-$(CONFIG_ALTERA_PR_IP_CORE) += altera-pr-ip-core.o +obj-$(CONFIG_ALTERA_PR_IP_CORE_PLAT) += altera-pr-ip-core-plat.o # FPGA Bridge Drivers obj-$(CONFIG_FPGA_BRIDGE) += fpga-bridge.o diff --git a/drivers/fpga/altera-pr-ip-core-plat.c b/drivers/fpga/altera-pr-ip-core-plat.c new file mode 100644 index 0000000..ab14380 --- /dev/null +++ b/drivers/fpga/altera-pr-ip-core-plat.c @@ -0,0 +1,65 @@ +/* + * Driver for Altera Partial Reconfiguration IP Core + * + * Copyright (C) 2016-2017 Intel Corporation + * + * Based on socfpga-a10.c Copyright (C) 2015-2016 Altera Corporation +



[PATCH v2 3/4] fpga dt: bindings for Altera Partial Reconfiguratio ...
matthew.gerlach@linux ... writes: (Summary) 12 ++++++++++++ 1 file changed, 12 insertions(+) create mode 100644 Documentation/devicetree/bindings/fpga/altera-pr-ip.txt diff --git a/Documentation/devicetree/bindings/fpga/altera-pr-ip.txt b/Documentation/devicetree/bindings/fpga/altera-pr-ip.txt new file mode 100644 index 0000000..ada821f --- /dev/null +++ b/Documentation/devicetree/bindings/fpga/altera-pr-ip.txt @@ -0,0 +1,12 @@ +Altera Partial Reconfiguration IP + +Required properties: +- compatible : should contain "altr,pr-ip" +- reg : base address and size for memory mapped io.



Re: [PATCH] lib: fix spelling mistake: "actualy" -> "actually"
David Miller writes: From: Colin King
Date: Sun, 26 Feb 2017 12:10:12 +0000 > From: Colin Ian King > > trivial fix to spelling mistake in pr_err message > > Signed-off-by: Colin Ian King Applied.



[PATCH v4] staging: xgifb: correct the multiple line dereference
Arushi Singhal writes: (Summary) 14 +++++++------- 2 files changed, 14 insertions(+), 30 deletions(-) diff --git a/drivers/staging/xgifb/XGI_main_26.c b/drivers/staging/xgifb/XGI_main_26.c index 6930f7eb741b..825f7dcd2f49 100644 --- a/drivers/staging/xgifb/XGI_main_26.c +++ b/drivers/staging/xgifb/XGI_main_26.c @@ -878,30 +878,14 @@ static void XGIfb_post_setmode(struct xgifb_video_info *xgifb_info) } if ((filter >= 0) && } } } diff --git a/drivers/staging/xgifb/vb_setmode.c b/drivers/staging/xgifb/vb_setmode.c index 7c7c8c8f1df3..1220d0cea87d 100644 --- a/drivers/staging/xgifb/vb_setmode.c +++ b/drivers/staging/xgifb/vb_setmode.c @@ -221,8 +221,8 @@ static unsigned char XGI_AjustCRT2Rate(unsigned short ModeIdIndex, for (; (*i)--) { - infoflag = XGI330_RefIndex[RefreshRateTableIndex + (*i)]. (*i)++) { - infoflag = XGI330_RefIndex[RefreshRateTableIndex + (*i)].



Re: [PATCH v3] staging: xgifb: correct the multiple line dereference
Joe Perches writes: (Summary) On Sun, 2017-02-26 at 21:23 +0530, Arushi Singhal wrote: []
[]
[]
+ XGIPART2, 0x37, f[2]);
0 1 2 3 4 5 6 7 8 12345678901234567890123456789012345678901234567890123456789012345678901234567890 12345678901234567890123456789012345678901234567890123456789012345678901234567890 xgifb_reg_set(XGIPART2, 0x37, f[2]);
xgifb_reg_set(XGIPART2, 0x37, f[2]);
All of these fit nicely on a single line and you should submit patches that make the most sense and do not need revision when reformatting.
revision when reformatting.
Please reformat all instances as above and resubmit.



[PATCH v2] staging: ks7010: remove code in comments.
Arushi Singhal writes: (Summary) Commenting Code Is a Bad Idea.
Comments are their to explain the code and how the code achieves its goal and as codes in the comments does not explain what the code is doing so there is no use of commenting them.
So in this patch codes in the comments are removed. 4 ---- 1 file changed, 4 deletions(-) diff --git a/drivers/staging/ks7010/ks7010_sdio.c b/drivers/staging/ks7010/ks7010_sdio.c index 2c263f98bdbb..54aa2e2f516f 100644 --- a/drivers/staging/ks7010/ks7010_sdio.c +++ b/drivers/staging/ks7010/ks7010_sdio.c @@ -495,10 +495,6 @@ static void ks7010_rw_function(struct work_struct *work) /* wiat after WAKEUP */ while (time_after(priv->last_wakeup + ((30 * HZ) / 1000), jiffies)) { DPRINTK(4, "wait after WAKEUP\n");



Re: [PATCH v3 2/2] iio: stm32 trigger: Implement parent trigger fe ...
Benjamin Gaignard writes: (Summary) 2017-02-26 15:59 GMT+01:00 Benjamin Gaignard = g
direction (I think) so keep up the good work!
An other way is to get a device which is a trigger consumer without having buffer or an event signaled in userland.
Maybe it doesn't fit with IIO concepts since it is only a way to drive the device and not expose a real control in userland (no pull function in this case).
in this case).
+++++++- >>> The timer counter thus directly
alid parent trigger is set.
mode
nt when reach preset value.
d fire event when reach 0.
igger/stm32-timer-trigger.c
ice *dev,
device *dev,
);
);
timer_trigger *priv)
RPE);
=3D { >>>



Re: [PATCH 4.10 00/21] 4.10.1-stable review
Greg Kroah-Hartman writes: On Fri, Feb 24, 2017 at 10:09:22AM -0800, kernelci.org bot wrote: juno: 1 failed lab
Is this failure new?
Is this failure new?
thanks,
thanks,
greg k-h
greg k-h
greg k-h



Re: [PATCH 4.10 00/21] 4.10.1-stable review
Greg Kroah-Hartman writes: On Fri, Feb 24, 2017 at 08:24:17PM -0800, Guenter Roeck wrote: Details are available at http://kerneltests.org/builders. Thanks for testing all of these and letting me know. Thanks for testing all of these and letting me know. greg k-h
greg k-h
greg k-h



Re: [PATCH 4.10 00/21] 4.10.1-stable review
Greg Kroah-Hartman writes: On Fri, Feb 24, 2017 at 11:20:10AM -0700, Shuah Khan wrote: I don't think this is a show stopper though.
Let me know if you find anything, and thanks for testing all 3 of these. Let me know if you find anything, and thanks for testing all 3 of these. greg k-h
greg k-h
greg k-h



[PATCH v3] staging: xgifb: correct the multiple line dereference
Arushi Singhal writes: (Summary) - xgifb_reg_set( - XGIPART2, - 0x35, - (XGI_TV_filter[filter_tb]. - xgifb_reg_set( - XGIPART2, - 0x36, - (XGI_TV_filter[filter_tb]. } } } diff --git a/drivers/staging/xgifb/vb_setmode.c b/drivers/staging/xgifb/vb_setmode.c index 7c7c8c8f1df3..1220d0cea87d 100644 --- a/drivers/staging/xgifb/vb_setmode.c +++ b/drivers/staging/xgifb/vb_setmode.c @@ -221,8 +221,8 @@ static unsigned char XGI_AjustCRT2Rate(unsigned short ModeIdIndex, for (; (*i)--) { - infoflag = XGI330_RefIndex[RefreshRateTableIndex + (*i)]. (*i)++) { - infoflag = XGI330_RefIndex[RefreshRateTableIndex + (*i)].



Re: Linux 4.10.1
Greg KH writes: (Summary) + +/* Configuration specific to Intel 64-bit C compiler */ + +#define COMPILER_DEPENDENT_INT64 __int64 +#define COMPILER_DEPENDENT_UINT64 unsigned __int64 +#define ACPI_INLINE __inline + +/* + * Calling conventions: + * + * ACPI_SYSTEM_XFACE - Interfaces to host OS (handlers, threads) + * ACPI_EXTERNAL_XFACE - External ACPI interfaces + * ACPI_INTERNAL_XFACE - Internal ACPI interfaces + * ACPI_INTERNAL_VAR_XFACE - Internal variable-parameter list interfaces + */ +#define ACPI_SYSTEM_XFACE +#define ACPI_EXTERNAL_XFACE +#define ACPI_INTERNAL_XFACE +#define ACPI_INTERNAL_VAR_XFACE + +/* remark 981 - operands evaluated in no particular order */ +#pragma warning(disable:981) + +/* warn C4100: unreferenced formal parameter */ +#pragma warning(disable:4100) + +/* warn C4127: conditional expression is constant */ +#pragma warning(disable:4127) + +/* warn C4706: assignment within conditional expression */ +#pragma warning(disable:4706) + +/* warn C4214: bit field types other tha



Linux 4.10.1
Greg KH writes: (Summary) 4 - 21 files changed, 259 insertions(+), 52 deletions(-) Brian Foster (1): xfs: clear delalloc and cache on buffered write failure Greg Kroah-Hartman (1): Linux 4.10.1 Javier Martinez Canillas (1): tty: serial: msm: Fix module autoload Jiri Kosina (1): netfilter: nf_ct_helper: warn when not applying default helper assignment Johan Hovold (8): USB: serial: mos7840: fix another NULL-deref at open USB: serial: ftdi_sio: fix modem-status error handling USB: serial: ftdi_sio: fix extreme low-latency setting USB: serial: ftdi_sio: fix line-status over-reporting USB: serial: spcp8x5: fix modem-status handling USB: serial: opticon: fix CTS retrieval at open USB: serial: ark3116: fix register-accessor error handling USB: serial: console: fix uninitialised spinlock Ken Lin (1): USB: serial: cp210x: add new IDs for GE Bx50v3 boards Lv Zheng (1): ACPICA: Linuxize: Restore and fix Intel compiler build Maxime Jayat (1): n



Re: [PATCH v2] drm/msm/dsi: Fix the releasing of resources in erro ...
Rob Clark writes: (Summary) fwiw, I prefer your v1, for reasons discussed on a similar patch fixing the same issue:
fixing the same issue:
https://lists.freedesktop.org/archives/dri-devel/2017-February/133097.html https://lists.freedesktop.org/archives/dri-devel/2017-February/133097.html (although your v1 is a better solution than the original, since we probably don't want to clk_disable_unprepare() the clk that failed to enable.)
enable.)
BR,
-R
-R
2.9.3
2.9.3



Re: [PATCH] [linux-next] bcache: Remove duplicate inclusion of blk ...
Coly Li writes: On 2017/2/26 下午2:02, Masanari Iida wrote:
Acked-by: Coly Li
Acked-by: Coly Li



[PATCH] rtlwifi: rtl8192de: ix spelling mistake: "althougth" -> "a ...
Colin King writes: (Summary) 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8192de/phy.c b/drivers/net/wireless/realtek/rtlwifi/rtl8192de/phy.c index de98d88..dcb5d83 100644 --- a/drivers/net/wireless/realtek/rtlwifi/rtl8192de/phy.c +++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192de/phy.c @@ -812,7 +812,7 @@ bool rtl92d_phy_config_rf_with_headerfile(struct ieee80211_hw *hw, * pathA or mac1 has to set phy0&phy1 pathA */ if ((content == radiob_txt) && (rfpath == RF90_PATH_A)) { RT_TRACE(rtlpriv, COMP_INIT, DBG_LOUD, - " ===>



Re: [PATCH 00/10] fujitsu-laptop: renames and cleanups
Andy Shevchenko writes: (Summary) wrote:
ote:
mit the
it a
iver to
o be
bing the
to see
so than
It looks like you missed this part of Jonathan's message: Indeed.
Indeed.
sing
drop that single patch?
Good we have testing branch and thank you for pointing out. Should be fixed= now.
now.
--=20
With Best Regards,
Andy Shevchenko
Andy Shevchenko
Andy Shevchenko



Re: [PATCH v3 2/2] iio: stm32 trigger: Implement parent trigger fe ...
Benjamin Gaignard writes: (Summary) 2017-02-25 18:53 GMT+01:00 Jonathan Cameron : direction (I think) so keep up the good work!
An other way is to get a device which is a trigger consumer without having buffer or
++++++- >> The timer counter thus directly
lid parent trigger is set.
ode
t when reach preset value.
fire event when reach 0.
gger/stm32-timer-trigger.c
ce *dev,
evice *dev,
;
;
imer_trigger *priv)
PE);
=3D { >>



Re: [PATCH v2 04/17] clocksource: Add Owl timer
Daniel Lezcano writes: (Summary) On Sun, Feb 26, 2017 at 03:40:49PM +0100, Andreas Fÿ¿¿¿¿¿er wrote: in-tree drivers use cpumask_of(0), others use cpu_possible_mask. If you specify the CLOCK_EVT_FEAT_DYNIRQ, the cpumask is not important as it will be changed dynamically.
will be changed dynamically.
Otherwise, cpumask_of(0) is often the default because it concentrates the wakeup on a single cpu, allowing the other cpus to go to deep idle state and if there are two clusters, it allows to have a cluster idle state. That results on a better energy saving.
a better energy saving.
The usage of cpu_possible_mask will randomly wakeup any cpu. 405.50 BogoMIPS (lpj=2027520) May be you should also consider using register_current_timer_delay() instead of jiffies based delay loops.
jiffies based delay loops.
[ 0.083523] pid_max: default: 32768 minimum: 301



Re: [PATCH] lib: fix spelling mistake: "actualy" -> "actually"
Jiri Pirko writes: Sun, Feb 26, 2017 at 01:10:12PM CET, colin.king@canonical.com wrote: Signed-off-by: Colin Ian King Acked-by: Jiri Pirko
Acked-by: Jiri Pirko
Acked-by: Jiri Pirko
2.10.2
2.10.2



RE: FOR YOUR INFORMATION
Eddie Concepcion writes: (Summary) ________________________________ From: Eddie Concepcion Sent: Sunday, February 26, 2017 9:11 AM To: Eddie Concepcion Subject: FOR YOUR INFORMATION Hi, You have received a donation of 1,000,000USD; CONFIDENTIALITY NOTICE: This email message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information. If you are not the intended recipient, please discard the message immediately and inform the sender that the message was sent in error.



[LSF/MM TOPIC] do we really need PG_error at all?
Jeff Layton writes: (Summary) Proposing this as a LSF/MM TOPIC, but it may turn out to be me just not understanding the semantics here.
understanding the semantics here.
As I was looking into -ENOSPC handling in cephfs, I noticed that PG_error is only ever tested in one place [1] __filemap_fdatawait_range, which does this:
which does this:
if (TestClearPageError(page)) ret = -EIO; Isn't it wrong to return an error on fsync, just because we had a read error somewhere in the file in a range that was never dirtied?



Re: [PATCH v2 04/17] clocksource: Add Owl timer
Andreas_Färber writes: (Summary) Now it goes past this point, initializes the real serial driver and boots up to not finding the rootfs:
boots up to not finding the rootfs:
[ 0.000032] sched_clock: 32 bits at 24MHz, resolution 41ns, wraps every 89478484971ns [ 0.007898] clocksource: timer: mask: 0xffffffff max_cycles: 0xffffffff, max_idle_ns: 79635851949 ns [ 0.017886] Console: colour dummy device 80x30 [ 0.022386] Calibrating delay loop... 405.50 BogoMIPS (lpj=2027520) [ 0.083523] pid_max: default: 32768 minimum: 301 ...



Re: [PATCH v2] drm/msm/dsi: Fix the releasing of resources in erro ...
walter harms writes: looks good to me.
looks good to me.
Reviewed-by: wharms@bfs.de
Reviewed-by: wharms@bfs.de
Am 26.02.2017 13:10, schrieb Christophe JAILLET:
}
}



Re: [PATCH v1 2/2] coresight: add support for debug module
Leo Yan writes: On Fri, Feb 24, 2017 at 12:07:11PM -0700, Mathieu Poirier wrote: On Thu, Feb 23, 2017 at 09:57:47AM +0800, Leo Yan wrote: [...]
[...]
The above #define isn't used in the code - please remove. Thanks a lot for quite detailed reivew and suggestion for below refactors. I will follow up them and send out new version for reviewing.
reviewing.
Thanks,
Leo Yan
Leo Yan
2.7.4
2.7.4



Re: [PATCH 0/8] ARM: sun8i: a33: Mali improvements
Emil Velikov writes: (Summary) It's not my call to make any of the policy, so I'll just kindly suggest improving the existing documentation:
- Reword/elaborate if out of tree [Linux or in general?] drivers are suitable counterpart.
- Patches could/should reference the "other OS" driver, or the "other OS" name at least ?
OS" name at least ?
Rather than clumping the above in 2.1 a separate section would be better ? Perhaps you already did - in which case pardon for missing it.
did - in which case pardon for missing it.
Interesting - do you have a link handy ?



[PATCH v4 4/4] iio: accel: adxl345: Add SPI support
Eva Rachel Retuya writes: (Summary) INPUT_ADXL34X=m) + depends on SPI + select ADXL345 + select REGMAP_SPI + help + Say Y here if you want to build support for the Analog Devices + ADXL345 3-axis digital accelerometer. + +static struct spi_driver adxl345_spi_driver = { + .driver = { + .name = "adxl345_spi", + .of_match_table = adxl345_of_match, + }, + .probe = adxl345_spi_probe, + .remove = adxl345_spi_remove, + .id_table = adxl345_spi_id, +};



[PATCH v4 3/4] iio: accel: adxl345: Split driver into core and I2C
Eva Rachel Retuya writes: (Summary) - -static struct i2c_driver adxl345_driver = { - .driver = { - .name = "adxl345", - }, - .probe = adxl345_probe, - .remove = adxl345_remove, - .id_table = adxl345_i2c_id, -}; +} + +static const struct i2c_device_id adxl345_i2c_id[] = { + { "adxl345", 0 }, + { } +}; + +static struct i2c_driver adxl345_i2c_driver = { + .driver = { + .name = "adxl345_i2c", + .of_match_table = adxl345_of_match, + }, + .probe = adxl345_i2c_probe, + .remove = adxl345_i2c_remove, + .id_table = adxl345_i2c_id, +};



[PATCH v4 2/4] iio: accel: adxl345: Use I2C regmap instead of dire ...
Eva Rachel Retuya writes: (Summary) + #define ADXL345_CHANNEL(reg, axis) { \ .type = IIO_ACCEL, \ .modified = 1, \ @@ -70,6 +76,7 @@ static int adxl345_read_raw(struct iio_dev *indio_dev, { struct adxl345_data *data = iio_priv(indio_dev); } - if (ret != ADXL345_DEVID) { - dev_err(&client->dev, "Invalid device ID: %d\n", ret); 0) { - dev_err(&client->dev, "Failed to enable measurement mode: %d\n", - ret);