RE: [PATCH v3 0/8] i2c: at91: cleanup and dt support
From: Voss, Nikolaus <hidden>
Date: 2012-09-12 10:39:10
Also in:
linux-arm-kernel, linux-i2c
Possibly related (same subject, not in this thread)
- 2012-09-12 · Re: [PATCH v3 0/8] i2c: at91: cleanup and dt support · Wolfram Sang <hidden>
- 2012-09-12 · Re: [PATCH v3 0/8] i2c: at91: cleanup and dt support · ludovic.desroches <hidden>
- 2012-09-12 · Re: [PATCH v3 0/8] i2c: at91: cleanup and dt support · Wolfram Sang <hidden>
- 2012-09-12 · Re: [PATCH v3 0/8] i2c: at91: cleanup and dt support · Nicolas Ferre <hidden>
- 2012-09-12 · [PATCH v3 0/8] i2c: at91: cleanup and dt support · <hidden>
Wolfram Sang wrote on Wednesday, September 12, 2012 12:16 PM:
One thing I'd like to make, though. I'd like to squash the following patches into one: drivers/i2c/busses/i2c-at91.c: add new driver i2c: at91: use managed resources i2c: at91: add warning about transmission issues for some devices i2c: at91: use an id table for SoC dependent parameters It is especially the last patch I am mostly interested in. The id_table approach is what I like, while the original id_entry mechanism looks fishy. I'd was good for reviewing to have the patches split like this; yet for hitting mainline, I'd prefer to have the driver proper on first occasion. I already did the squashing in a test-branch and the result looks good to me. Nikolaus, Ludovic: Are you fine with this?
Absolutely. Niko