* [GIT PULL] Char/Misc driver patches for 4.12-rc1
@ 2017-05-05 0:18 Greg KH
2017-05-05 2:28 ` Linus Torvalds
0 siblings, 1 reply; 12+ messages in thread
From: Greg KH @ 2017-05-05 0:18 UTC (permalink / raw)
To: Linus Torvalds, Andrew Morton, Arnd Bergmann; +Cc: linux-kernel
The following changes since commit c02ed2e75ef4c74e41e421acb4ef1494671585e8:
Linux 4.11-rc4 (2017-03-26 14:15:16 -0700)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git/ tags/char-misc-4.12-rc1
for you to fetch changes up to 2a76f89fa58c769241cfc21f2614705591519ae3:
firmware: google memconsole: Fix return value check in platform_memconsole_init() (2017-04-26 22:54:54 +0200)
----------------------------------------------------------------
char/misc patches for 4.12-rc1
Here is the big set of new char/misc driver drivers and features for
4.12-rc1.
There's lots of new drivers added this time around, new firmware drivers
from Google, more auxdisplay drivers, extcon drivers, fpga drivers, and
a bunch of other driver updates. Nothing major, except if you happen to
have the hardware for these drivers, and then you will be happy :)
All of these have been in linux-next for a while with no reported
issues.
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
----------------------------------------------------------------
Aban Bedel (1):
nvmem: core: Allow allocating several anonymous nvmem devices
Alan Tull (2):
fpga: add config complete timeout
MAINTAINERS: fpga: update email and directory paths
Alexander Usyskin (2):
mei: drop amthif internal client
mei: implement fsync
Anatolij Gustschin (2):
dt: bindings: fpga: add xilinx slave-serial binding description
fpga manager: Add Xilinx slave serial SPI driver
Andrew F. Davis (2):
w1: Use kernel common min() implementation
w1: Remove unneeded use of assert() and remove w1_log.h
Andy Shevchenko (3):
Revert "extcon: usb-gpio: add support for ACPI gpio interface"
auxdisplay: Move panel.c to drivers/auxdisplay folder
auxdisplay: Move arm-charlcd.c to drivers/auxdisplay folder
Arnd Bergmann (4):
drivers/misc: aspeed-lpc-ctrl: fix printk format warning again
aspeed-lpc-ctrl: include linux/types.h for uapi header
auxdisplay: ht16k33: don't access uninitialized data
lkdtm: turn off kcov for lkdtm_rodata_do_nothing:
Chanwoo Choi (4):
extcon: Add new extcon_register_notifier_all() to monitor all external connectors
Merge branch 'ib-extcon-4.12' into HEAD
extcon: Remove porting compatibility of swich class
extcon: Use BIT() macro for the left-shift operation
Charles Keepax (1):
extcon: arizona: Wait for any running HPDETs to complete on jack removal
Cyril Bur (2):
drivers/misc: Add Aspeed LPC control driver
drivers/misc: Aspeed LPC control fix compile error and warning
Dan Williams (1):
device-dax: fix cdev leak
Dinh Nguyen (1):
fpga: fix sparse warnings in fpga-mgr and fpga-bridge
Dmitry Torokhov (3):
rapidio: use is_visible() to hide switch-specific attributes
zorro: stop creating attributes by hand
auxdisplay: ht16k33: use le16_to_cpup() to fetch LE16 data
Elena Reshetova (2):
drivers, char: convert vma_data.refcnt from atomic_t to refcount_t
drivers, firewire: convert fw_node.ref_count from atomic_t to refcount_t
Florian Fainelli (2):
FPGA: Add TS-7300 FPGA manager
ARM: ep93xx: Register ts73xx-fpga manager driver for TS-7300
Geert Uytterhoeven (7):
auxdisplay: charlcd: Extract character LCD core from misc/panel
auxdisplay: charlcd: Add support for 4-bit interfaces
auxdisplay: charlcd: Add support for displays with more than two lines
dt-bindings: auxdisplay: Add bindings for Hitachi HD44780
auxdisplay: Add HD44780 Character LCD support
MAINTAINERS: Add file patterns for fpga device tree bindings
auxdisplay: hd44780: Fix DT properties to include units of measurement
Greg Kroah-Hartman (3):
Merge 4.11-rc4 into char-misc-next
Merge tag 'extcon-next-for-4.12' of git://git.kernel.org/.../chanwoo/extcon into char-misc-next
goldfish_pipe: fix build warning about using too much stack.
Hans de Goede (3):
extcon: intel-cht-wc: Add Intel Cherry Trail Whiskey Cove PMIC extcon driver
extcon: intel-cht-wc: Disable external 5v boost converter on probe
extcon: intel-cht-wc: Ignore failure to detect charger-type on host mode exit
Icenowy Zheng (2):
nvmem: sunxi-sid: read NVMEM size from device compatible
nvmem: sunxi-sid: add support for H3's SID controller
Jason Gunthorpe (1):
IB/ucm: utilize new cdev_device_add helper function
Javier Martinez Canillas (4):
auxdisplay: img-ascii-lcd: Fix module autoload
misc: tsl2550: Add OF device ID table
misc: ds1682: Add OF device ID table
eeprom: idt_89hpesx: Add OF device ID table
Jin Qian (1):
goldfish_pipe: An implementation of more parallel pipe
Joe Perches (1):
drivers/char: Convert remaining use of pr_warning to pr_warn
Joel Holdsworth (2):
Documentation: Add binding document for Lattice iCE40 FPGA manager
fpga: Add support for Lattice iCE40 FPGAs
Julius Werner (1):
firmware: Add coreboot device tree binding documentation
K. Y. Srinivasan (3):
Drivers: hv: Fix a typo
Drivers: hv: Base autoeoi enablement based on hypervisor hints
Drivers: hv: Issue explicit EOI when autoeoi is not enabled
Kees Cook (1):
lkdtm: add bad USER_DS test
Logan Gunthorpe (13):
chardev: add helper function to register char devs with a struct device
device-dax: utilize new cdev_device_add helper function
input: utilize new cdev_device_add helper function
gpiolib: utilize new cdev_device_add helper function
tpm-chip: utilize new cdev_device_add helper function
platform/chrome: cros_ec_dev - utilize new cdev_device_add helper function
infiniband: utilize the new cdev_set_parent function
iio:core: utilize new cdev_device_add helper function
media: utilize new cdev_device_add helper function
mtd: utilize new cdev_device_add helper function
rapidio: utilize new cdev_device_add helper function
rtc: utilize new cdev_device_add helper function
scsi: utilize new cdev_device_add helper function
Mariusz Bialonczyk (4):
w1: add missing DS2413 documentation
w1: add support for DS2438 Smart Battery Monitor
w1: add documentation for w1_ds2438
w1: w1_ds2760.h: fix defines indentation
Martyn Welch (2):
docs: Add kernel-doc comments to VME driver API
docs: Update VME documentation to include kerneldoc comments
Matt Ranostay (2):
pps: add ioctl_compat function to correct ioctl definitions
pps: fix padding issue with PPS_FETCH for ioctl_compat
Matthew Gerlach (4):
fpga pr ip: Core driver support for Altera Partial Reconfiguration IP.
fpga dt: bindings for Altera Partial Reconfiguration IP.
fpga pr ip: Platform driver for Altera Partial Reconfiguration IP.
fpga fr br: update supported version numbers
Matthias Kaehlcke (1):
hpet: Make cmd parameter of hpet_ioctl_common() unsigned
Maxime Ripard (1):
MAINTAINERS: nvmem: Remove myself from maintainers
Michael Davidson (1):
misc: lkdtm: Add volatile to intentional NULL pointer reference
Michael Grzeschik (1):
nvmem: Add driver for the i.MX IIM
Michal Sojka (3):
uio: Allow handling of non page-aligned memory regions
uio_mf624: Refactor memory info initialization
uio_mf624: Align memory regions to page size and set correct offsets
Mike Leach (1):
coresight: Fixes coresight DT parse to get correct output port ID.
Ming Lei (1):
MAINTAINERS: update firmware loader entry
Moritz Fischer (6):
fpga: Add flag to indicate bitstream needs decrypting
fpga: zynq: Add support for encrypted bitstreams
fpga: region: Add fpga-region property 'encrypted-fpga-config'
fpga: bridge: Replace open-coded list_for_each + list_entry
fpga: altera_freeze_bridge: Constify ops
fpga: Add support for Xilinx LogiCORE PR Decoupler
Peng Fan (2):
dt-bindings: imx-ocotp: add compatible string for i.MX7D/S
nvmem: octop: Add i.MX7D support
Peter Chen (1):
extcon: usb-gpio: Do not enable USB as wakeup source by default
Richard Leitner (3):
nvmem: imx-ocotp: fix usage of "dev" pointers
nvmem: imx-ocotp: clear error bit after reading locked values
nvmem: imx-ocotp: add write support
Rob Herring (1):
binder: Add 'hwbinder' to the default devices
Roger Quadros (2):
extcon: usb-gpio: Don't miss event during suspend/resume
extcon: palmas: Don't miss GPIO events during suspend/resume
Sascha Hauer (1):
dt-bindings: nvmem: Add i.MX IIM binding doc
Saul Wold (1):
ver_linux: Use /usr/bin/awk instead of /bin/awk
Shile Zhang (1):
hangcheck-timer: Fix typo in comment
Stephen Hemminger (9):
vmbus: only reschedule tasklet if time limit exceeded
hyperv: fix warning about missing prototype
vmbus: remove useless return's
vmbus: remove unnecessary initialization
vmbus: fix spelling errors
hyperv: remove unnecessary return variable
vmbus: make channel_message table constant
vmbus: cleanup header file style
vmbus: expose debug info for drivers
Thierry Escande (4):
firmware: google memconsole: Remove useless submenu in Kconfig
firmware: google memconsole: Move specific EBDA parts
firmware: google memconsole: Add coreboot support
firmware: google memconsole: Add ARM/ARM64 support
Tobias Klauser (2):
fpga altera-hps2fpga: disable/unprepare clock on error in alt_fpga_bridge_probe()
fpga: region: release FPGA region reference in error path
Tomas Winkler (4):
mei: add pci driver ops shutdown handler.
mei: hbm: remove variable length arrays
mei: bus: elminate variable length arrays
mei: drop the TODO from samples
Varsha Rao (4):
drivers: char: misc: Replace "foo * bar" with "foo *bar".
drivers: char: misc: Add space after ','.
drivers: char: misc: Add blank line after declaration.
drivers: char: misc: Replace printk with pr_err.
Wei Yongjun (2):
firmware: Google VPD: Fix return value check in vpd_platform_init()
firmware: google memconsole: Fix return value check in platform_memconsole_init()
Wei-Ning Huang (2):
firmware: Google VPD: import lib_vpd source files
firmware: Google VPD sysfs driver
Documentation/DocBook/rapidio.tmpl | 3 -
.../devicetree/bindings/auxdisplay/hit,hd44780.txt | 45 +
.../devicetree/bindings/firmware/coreboot.txt | 33 +
.../devicetree/bindings/fpga/altera-pr-ip.txt | 12 +
.../devicetree/bindings/fpga/fpga-region.txt | 1 +
.../bindings/fpga/lattice-ice40-fpga-mgr.txt | 21 +
.../bindings/fpga/xilinx-slave-serial.txt | 44 +
.../bindings/nvmem/allwinner,sunxi-sid.txt | 6 +-
.../devicetree/bindings/nvmem/imx-iim.txt | 22 +
.../devicetree/bindings/nvmem/imx-ocotp.txt | 5 +
Documentation/driver-api/vme.rst | 363 ++------
Documentation/extcon/porting-android-switch-class | 123 ---
Documentation/w1/slaves/00-INDEX | 4 +
Documentation/w1/slaves/w1_ds2413 | 50 ++
Documentation/w1/slaves/w1_ds2438 | 63 ++
MAINTAINERS | 8 +-
arch/arm/mach-ep93xx/ts72xx.c | 26 +
arch/x86/hyperv/hv_init.c | 2 +-
arch/x86/include/uapi/asm/hyperv.h | 7 +-
arch/x86/kernel/cpu/mshyperv.c | 3 +
drivers/android/Kconfig | 2 +-
drivers/auxdisplay/Kconfig | 304 +++++++
drivers/auxdisplay/Makefile | 4 +
drivers/{misc => auxdisplay}/arm-charlcd.c | 0
drivers/auxdisplay/charlcd.c | 818 +++++++++++++++++
drivers/auxdisplay/hd44780.c | 326 +++++++
drivers/auxdisplay/ht16k33.c | 20 +-
drivers/auxdisplay/img-ascii-lcd.c | 1 +
drivers/{misc => auxdisplay}/panel.c | 827 ++----------------
drivers/char/hangcheck-timer.c | 2 +-
drivers/char/hpet.c | 2 +-
drivers/char/misc.c | 11 +-
drivers/char/mspec.c | 9 +-
drivers/char/tpm/tpm-chip.c | 19 +-
drivers/char/virtio_console.c | 2 +-
drivers/dax/dax.c | 33 +-
drivers/extcon/Kconfig | 7 +
drivers/extcon/Makefile | 1 +
drivers/extcon/devres.c | 61 ++
drivers/extcon/extcon-arizona.c | 46 +
drivers/extcon/extcon-intel-cht-wc.c | 395 +++++++++
drivers/extcon/extcon-palmas.c | 6 +
drivers/extcon/extcon-usb-gpio.c | 10 +-
drivers/extcon/extcon.c | 88 +-
drivers/extcon/extcon.h | 3 +
drivers/firewire/core-topology.c | 2 +-
drivers/firewire/core.h | 8 +-
drivers/firmware/google/Kconfig | 59 +-
drivers/firmware/google/Makefile | 10 +-
drivers/firmware/google/coreboot_table-acpi.c | 88 ++
drivers/firmware/google/coreboot_table-of.c | 82 ++
drivers/firmware/google/coreboot_table.c | 94 ++
drivers/firmware/google/coreboot_table.h | 50 ++
drivers/firmware/google/memconsole-coreboot.c | 109 +++
drivers/firmware/google/memconsole-x86-legacy.c | 154 ++++
drivers/firmware/google/memconsole.c | 155 +---
drivers/firmware/google/memconsole.h | 43 +
drivers/firmware/google/vpd.c | 332 +++++++
drivers/firmware/google/vpd_decode.c | 99 +++
drivers/firmware/google/vpd_decode.h | 58 ++
drivers/fpga/Kconfig | 42 +
drivers/fpga/Makefile | 6 +
drivers/fpga/altera-freeze-bridge.c | 32 +-
drivers/fpga/altera-hps2fpga.c | 15 +-
drivers/fpga/altera-pr-ip-core-plat.c | 68 ++
drivers/fpga/altera-pr-ip-core.c | 220 +++++
drivers/fpga/fpga-bridge.c | 17 +-
drivers/fpga/fpga-mgr.c | 2 +-
drivers/fpga/fpga-region.c | 15 +-
drivers/fpga/ice40-spi.c | 207 +++++
drivers/fpga/ts73xx-fpga.c | 156 ++++
drivers/fpga/xilinx-pr-decoupler.c | 161 ++++
drivers/fpga/xilinx-spi.c | 198 +++++
drivers/fpga/zynq-fpga.c | 28 +-
drivers/gpio/gpiolib.c | 23 +-
drivers/hv/channel.c | 10 +-
drivers/hv/channel_mgmt.c | 48 +-
drivers/hv/connection.c | 65 +-
drivers/hv/hv.c | 5 +-
drivers/hv/hv_balloon.c | 2 -
drivers/hv/hv_fcopy.c | 2 -
drivers/hv/hv_kvp.c | 12 +-
drivers/hv/hv_snapshot.c | 2 -
drivers/hv/hyperv_vmbus.h | 29 +-
drivers/hv/ring_buffer.c | 22 +-
drivers/hv/vmbus_drv.c | 4 +-
drivers/hwtracing/coresight/of_coresight.c | 2 +-
drivers/iio/industrialio-core.c | 15 +-
drivers/infiniband/core/ucm.c | 35 +-
drivers/infiniband/core/user_mad.c | 4 +-
drivers/infiniband/core/uverbs_main.c | 2 +-
drivers/infiniband/hw/hfi1/device.c | 2 +-
drivers/input/evdev.c | 11 +-
drivers/input/joydev.c | 11 +-
drivers/input/mousedev.c | 11 +-
drivers/media/cec/cec-core.c | 16 +-
drivers/media/media-devnode.c | 20 +-
drivers/misc/Kconfig | 293 +------
drivers/misc/Makefile | 5 +-
drivers/misc/aspeed-lpc-ctrl.c | 266 ++++++
drivers/misc/ds1682.c | 7 +
drivers/misc/eeprom/idt_89hpesx.c | 57 ++
drivers/misc/lkdtm.h | 1 +
drivers/misc/lkdtm_bugs.c | 13 +-
drivers/misc/lkdtm_core.c | 1 +
drivers/misc/mei/Makefile | 1 -
drivers/misc/mei/amthif.c | 340 --------
drivers/misc/mei/bus-fixup.c | 9 +-
drivers/misc/mei/bus.c | 6 -
drivers/misc/mei/client.c | 14 +-
drivers/misc/mei/client.h | 5 -
drivers/misc/mei/hbm.c | 29 +-
drivers/misc/mei/init.c | 14 +-
drivers/misc/mei/interrupt.c | 38 +-
drivers/misc/mei/main.c | 125 +--
drivers/misc/mei/mei_dev.h | 51 --
drivers/misc/mei/pci-me.c | 31 +-
drivers/misc/mei/pci-txe.c | 29 +-
drivers/misc/tsl2550.c | 7 +
drivers/mtd/ubi/build.c | 91 +-
drivers/mtd/ubi/vmt.c | 49 +-
drivers/nvmem/Kconfig | 11 +
drivers/nvmem/Makefile | 2 +
drivers/nvmem/core.c | 3 +-
drivers/nvmem/imx-iim.c | 173 ++++
drivers/nvmem/imx-ocotp.c | 254 +++++-
drivers/nvmem/sunxi_sid.c | 89 +-
drivers/platform/chrome/cros_ec_dev.c | 31 +-
drivers/platform/goldfish/goldfish_pipe.c | 970 ++++++++++++++-------
drivers/pps/pps.c | 123 ++-
drivers/rapidio/devices/rio_mport_cdev.c | 24 +-
drivers/rapidio/rio-sysfs.c | 76 +-
drivers/rapidio/rio.c | 3 -
drivers/rapidio/rio.h | 2 -
drivers/rtc/class.c | 14 +-
drivers/rtc/rtc-core.h | 10 -
drivers/rtc/rtc-dev.c | 17 -
drivers/scsi/osd/osd_uld.c | 56 +-
drivers/uio/uio.c | 2 +-
drivers/uio/uio_mf624.c | 48 +-
drivers/vme/vme.c | 469 +++++++++-
drivers/w1/masters/matrox_w1.c | 6 -
drivers/w1/slaves/Kconfig | 6 +
drivers/w1/slaves/Makefile | 1 +
drivers/w1/slaves/w1_ds2438.c | 390 +++++++++
drivers/w1/slaves/w1_ds2760.h | 10 +-
drivers/w1/w1.c | 1 -
drivers/w1/w1_family.h | 1 +
drivers/w1/w1_int.c | 1 -
drivers/w1/w1_io.c | 1 -
drivers/w1/w1_log.h | 31 -
drivers/w1/w1_netlink.c | 5 +-
drivers/zorro/zorro-driver.c | 15 +-
drivers/zorro/zorro-sysfs.c | 76 +-
drivers/zorro/zorro.c | 3 -
drivers/zorro/zorro.h | 3 +-
fs/char_dev.c | 86 ++
include/linux/cdev.h | 5 +
include/linux/extcon.h | 21 +-
include/linux/fpga/altera-pr-ip-core.h | 29 +
include/linux/fpga/fpga-mgr.h | 4 +
include/linux/hyperv.h | 31 +-
include/linux/uio_driver.h | 13 +-
include/linux/vme.h | 12 +-
include/misc/charlcd.h | 42 +
include/uapi/linux/aspeed-lpc-ctrl.h | 61 ++
include/uapi/linux/pps.h | 19 +
samples/mei/TODO | 2 -
scripts/ver_linux | 2 +-
169 files changed, 8199 insertions(+), 3238 deletions(-)
create mode 100644 Documentation/devicetree/bindings/auxdisplay/hit,hd44780.txt
create mode 100644 Documentation/devicetree/bindings/firmware/coreboot.txt
create mode 100644 Documentation/devicetree/bindings/fpga/altera-pr-ip.txt
create mode 100644 Documentation/devicetree/bindings/fpga/lattice-ice40-fpga-mgr.txt
create mode 100644 Documentation/devicetree/bindings/fpga/xilinx-slave-serial.txt
create mode 100644 Documentation/devicetree/bindings/nvmem/imx-iim.txt
delete mode 100644 Documentation/extcon/porting-android-switch-class
create mode 100644 Documentation/w1/slaves/w1_ds2413
create mode 100644 Documentation/w1/slaves/w1_ds2438
rename drivers/{misc => auxdisplay}/arm-charlcd.c (100%)
create mode 100644 drivers/auxdisplay/charlcd.c
create mode 100644 drivers/auxdisplay/hd44780.c
rename drivers/{misc => auxdisplay}/panel.c (71%)
create mode 100644 drivers/extcon/extcon-intel-cht-wc.c
create mode 100644 drivers/firmware/google/coreboot_table-acpi.c
create mode 100644 drivers/firmware/google/coreboot_table-of.c
create mode 100644 drivers/firmware/google/coreboot_table.c
create mode 100644 drivers/firmware/google/coreboot_table.h
create mode 100644 drivers/firmware/google/memconsole-coreboot.c
create mode 100644 drivers/firmware/google/memconsole-x86-legacy.c
create mode 100644 drivers/firmware/google/memconsole.h
create mode 100644 drivers/firmware/google/vpd.c
create mode 100644 drivers/firmware/google/vpd_decode.c
create mode 100644 drivers/firmware/google/vpd_decode.h
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/ice40-spi.c
create mode 100644 drivers/fpga/ts73xx-fpga.c
create mode 100644 drivers/fpga/xilinx-pr-decoupler.c
create mode 100644 drivers/fpga/xilinx-spi.c
create mode 100644 drivers/misc/aspeed-lpc-ctrl.c
delete mode 100644 drivers/misc/mei/amthif.c
create mode 100644 drivers/nvmem/imx-iim.c
create mode 100644 drivers/w1/slaves/w1_ds2438.c
delete mode 100644 drivers/w1/w1_log.h
create mode 100644 include/linux/fpga/altera-pr-ip-core.h
create mode 100644 include/misc/charlcd.h
create mode 100644 include/uapi/linux/aspeed-lpc-ctrl.h
delete mode 100644 samples/mei/TODO
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [GIT PULL] Char/Misc driver patches for 4.12-rc1
2017-05-05 0:18 [GIT PULL] Char/Misc driver patches for 4.12-rc1 Greg KH
@ 2017-05-05 2:28 ` Linus Torvalds
2017-05-05 16:00 ` James Bottomley
0 siblings, 1 reply; 12+ messages in thread
From: Linus Torvalds @ 2017-05-05 2:28 UTC (permalink / raw)
To: Greg KH, James Bottomley, Jarkko Sakkinen
Cc: Andrew Morton, Arnd Bergmann, Linux Kernel Mailing List
On Thu, May 4, 2017 at 5:18 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
>
> Here is the big set of new char/misc driver drivers and features for
> 4.12-rc1.
Ugh. I'm not particularly happy with the conflicts I got and my
resolutions there-of.
I did resolve them - in the case of drivers/scsi/osd/osd_uld.c as per
James' suggestion from his SCSI pull. Thanks.
But even that resolution I'm not entirely happy with: somebody should
check that it cleans up oud properly
But the one I'm really unhappy with is my tpm-chip.c resolution.
In particular, commit 8dbbf5825181 ("tpm-chip: utilize new
cdev_device_add helper function") made the tpm-chip code use that
cdev_device_add/del pattern for chip->[c]dev. Fine.
But then commit fdc915f7f719 ("tpm: expose spaces via a device link
/dev/tpmrm<n>") added the *exact* same old pattern to a new
"chip->[c]devs" (note the extra 's') and did so in a particularly ugly
way too.
James, why did you do that nasty
if (chip->flags & TPM_CHIP_FLAG_TPM2)
*twice*, instead of just doing things properly inside *one* test?
Anyway, my merge resolution tries to just apply the same
cdev_device_add/del pattern to the new chip->[c]devs entries, because
not doing so seemed criminally ugly.
It compiles. It looks better than the mess it was. But it may not work.
James, Jarkko, you need to look at that tpm merge of mine. And James,
double-check my osd_uld thing too.
Linus
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [GIT PULL] Char/Misc driver patches for 4.12-rc1
2017-05-05 2:28 ` Linus Torvalds
@ 2017-05-05 16:00 ` James Bottomley
2017-05-05 16:33 ` Linus Torvalds
2017-05-05 16:38 ` Greg KH
0 siblings, 2 replies; 12+ messages in thread
From: James Bottomley @ 2017-05-05 16:00 UTC (permalink / raw)
To: Linus Torvalds, Greg KH, Jarkko Sakkinen
Cc: Andrew Morton, Arnd Bergmann, Linux Kernel Mailing List
On Thu, 2017-05-04 at 19:28 -0700, Linus Torvalds wrote:
> On Thu, May 4, 2017 at 5:18 PM, Greg KH <gregkh@linuxfoundation.org>
> wrote:
> >
> > Here is the big set of new char/misc driver drivers and features
> > for 4.12-rc1.
>
> Ugh. I'm not particularly happy with the conflicts I got and my
> resolutions there-of.
Yes, we really should have done this via a postmerge tree. We've had
so little cause to use them recently, I suspect everyone's forgotten
how.
> I did resolve them - in the case of drivers/scsi/osd/osd_uld.c as per
> James' suggestion from his SCSI pull. Thanks.
>
> But even that resolution I'm not entirely happy with: somebody should
> check that it cleans up oud properly
It looks fine to me, but I'll ask the osd people (well person, since
it's very little used).
> But the one I'm really unhappy with is my tpm-chip.c resolution.
>
> In particular, commit 8dbbf5825181 ("tpm-chip: utilize new
> cdev_device_add helper function") made the tpm-chip code use that
> cdev_device_add/del pattern for chip->[c]dev. Fine.
>
> But then commit fdc915f7f719 ("tpm: expose spaces via a device link
> /dev/tpmrm<n>") added the *exact* same old pattern to a new
> "chip->[c]devs" (note the extra 's') and did so in a particularly
> ugly
> way too.
>
> James, why did you do that nasty
>
> if (chip->flags & TPM_CHIP_FLAG_TPM2)
>
> *twice*, instead of just doing things properly inside *one* test?
>
> Anyway, my merge resolution tries to just apply the same
> cdev_device_add/del pattern to the new chip->[c]devs entries, because
> not doing so seemed criminally ugly.
>
> It compiles. It looks better than the mess it was. But it may not
> work.
>
> James, Jarkko, you need to look at that tpm merge of mine. And James,
> double-check my osd_uld thing too.
I'm not going to defend the earlier coding, but you've lost the real
device_add() calls in the merge, meaning the tpm devices don't actually
get made visible at all. I suspect assuming device_add() is done by
cdev_device_add() because of the name is going to be our next anti
-pattern, so you're at least ahead of the game ...
I think the below is the correct fix, but we should really take it
through the TPM list (or at least give me time for some runtime
testing) because I suddenly think the error leg unwinding isn't correct
in the original, so I've redone that too.
James
---
diff --git a/drivers/char/tpm/tpm-chip.c b/drivers/char/tpm/tpm-chip.c
index 322b8a51ffc6..867cef5ca9e5 100644
--- a/drivers/char/tpm/tpm-chip.c
+++ b/drivers/char/tpm/tpm-chip.c
@@ -272,24 +272,30 @@ EXPORT_SYMBOL_GPL(tpmm_chip_alloc);
static int tpm_add_char_device(struct tpm_chip *chip)
{
int rc;
+ const char *errstr;
+ struct device *errdev = &chip->dev;
rc = cdev_device_add(&chip->cdev, &chip->dev);
if (rc) {
- dev_err(&chip->dev,
- "unable to cdev_device_add() %s, major %d, minor %d, err=%d\n",
- dev_name(&chip->dev), MAJOR(chip->dev.devt),
- MINOR(chip->dev.devt), rc);
- return rc;
+ errstr = "cdev_device_add for main device";
+ goto error1;
+ }
+ rc = device_add(&chip->dev);
+ if (rc) {
+ errstr = "device_add for main device";
+ goto error2;
}
-
if (chip->flags & TPM_CHIP_FLAG_TPM2) {
+ errdev = &chip->devs;
rc = cdev_device_add(&chip->cdevs, &chip->devs);
if (rc) {
- dev_err(&chip->devs,
- "unable to cdev_device_add() %s, major %d, minor %d, err=%d\n",
- dev_name(&chip->devs), MAJOR(chip->devs.devt),
- MINOR(chip->devs.devt), rc);
- return rc;
+ errstr = "cdev_device_add for rm device";
+ goto error3;
+ }
+ rc = device_add(&chip->devs);
+ if (rc) {
+ errstr = "device_add for rm device";
+ goto error4;
}
}
@@ -299,6 +305,19 @@ static int tpm_add_char_device(struct tpm_chip *chip)
mutex_unlock(&idr_lock);
return rc;
+
+ error4:
+ cdev_del(&chip->cdevs);
+ error3:
+ device_del(&chip->dev);
+ error2:
+ cdev_del(&chip->cdev);
+ error1:
+ dev_err(&chip->dev,
+ "Failed %s %s, major %d, minor %d, err=%d\n",
+ errstr, dev_name(errdev), MAJOR(errdev->devt),
+ MINOR(errdev->devt), rc);
+ return rc;
}
static void tpm_del_char_device(struct tpm_chip *chip)
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [GIT PULL] Char/Misc driver patches for 4.12-rc1
2017-05-05 16:00 ` James Bottomley
@ 2017-05-05 16:33 ` Linus Torvalds
2017-05-05 16:49 ` James Bottomley
2017-05-05 16:38 ` Greg KH
1 sibling, 1 reply; 12+ messages in thread
From: Linus Torvalds @ 2017-05-05 16:33 UTC (permalink / raw)
To: James Bottomley
Cc: Greg KH, Jarkko Sakkinen, Andrew Morton, Arnd Bergmann,
Linux Kernel Mailing List
On Fri, May 5, 2017 at 9:00 AM, James Bottomley
<James.Bottomley@hansenpartnership.com> wrote:
>
> I'm not going to defend the earlier coding, but you've lost the real
> device_add() calls in the merge, meaning the tpm devices don't actually
> get made visible at all. I suspect assuming device_add() is done by
> cdev_device_add() because of the name is going to be our next anti
> -pattern, so you're at least ahead of the game ...
Don't be silly. That's *exactly* what cdev_device_add() does.
That's the whole and only point of the whole function: it does *both*
the cdev_add() _and_ the device_add(), and it handles errors sanely
and unwinds things.
So removing the device_add() is what the code should do. It's also
exactly what the commit I pointed you at does (the one that clashed
with your addition of the new ":[c]devs" cases. Let me repeat:
8dbbf5825181 tpm-chip: utilize new cdev_device_add helper function
So your patch looks like complete garbage, and I think you're confused
about what cdev_device_add() is.
I think you're confusing it with the nasty old "cdev_add()" function
that indeed didn't do a "device_add()", but that's exactly why
"cdev_device_add()" as added, so that you don't have to have that
nasty pattern of having both.
That said, I do think that my merge is missing a "cdev_device_del()"
in the [c]dev if the second cdev_device_add() (on [c]devs) fails.
Although I also considered just saying "maybe if the second device
register fails, we still want to just succeed, and at least leave the
first one available". Because why shouldn't you let people access at
least the old interface even if the new interface couldn't be set up?
So that's part of why I'd like people to look at that resolutin. But
more importantly it should be tested.
Linus
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [GIT PULL] Char/Misc driver patches for 4.12-rc1
2017-05-05 16:00 ` James Bottomley
2017-05-05 16:33 ` Linus Torvalds
@ 2017-05-05 16:38 ` Greg KH
2017-05-05 19:46 ` James Bottomley
2017-05-05 20:01 ` Linus Torvalds
1 sibling, 2 replies; 12+ messages in thread
From: Greg KH @ 2017-05-05 16:38 UTC (permalink / raw)
To: James Bottomley
Cc: Linus Torvalds, Jarkko Sakkinen, Andrew Morton, Arnd Bergmann,
Linux Kernel Mailing List
On Fri, May 05, 2017 at 09:00:06AM -0700, James Bottomley wrote:
> On Thu, 2017-05-04 at 19:28 -0700, Linus Torvalds wrote:
> > On Thu, May 4, 2017 at 5:18 PM, Greg KH <gregkh@linuxfoundation.org>
> > wrote:
> > >
> > > Here is the big set of new char/misc driver drivers and features
> > > for 4.12-rc1.
> >
> > Ugh. I'm not particularly happy with the conflicts I got and my
> > resolutions there-of.
>
> Yes, we really should have done this via a postmerge tree. We've had
> so little cause to use them recently, I suspect everyone's forgotten
> how.
Huh? You could have pulled in my tree into this one, or I could have
done that for you, my trees are not rebased at all, and they get used
this way every other release or so for this very reason.
> > James, Jarkko, you need to look at that tpm merge of mine. And James,
> > double-check my osd_uld thing too.
>
> I'm not going to defend the earlier coding, but you've lost the real
> device_add() calls in the merge, meaning the tpm devices don't actually
> get made visible at all. I suspect assuming device_add() is done by
> cdev_device_add() because of the name is going to be our next anti
> -pattern, so you're at least ahead of the game ...
It's not an anti-pattern at all, it is ment to fix the bugs you, and
others, keep making :)
> @@ -272,24 +272,30 @@ EXPORT_SYMBOL_GPL(tpmm_chip_alloc);
> static int tpm_add_char_device(struct tpm_chip *chip)
> {
> int rc;
> + const char *errstr;
> + struct device *errdev = &chip->dev;
>
> rc = cdev_device_add(&chip->cdev, &chip->dev);
> if (rc) {
> - dev_err(&chip->dev,
> - "unable to cdev_device_add() %s, major %d, minor %d, err=%d\n",
> - dev_name(&chip->dev), MAJOR(chip->dev.devt),
> - MINOR(chip->dev.devt), rc);
> - return rc;
> + errstr = "cdev_device_add for main device";
> + goto error1;
> + }
> + rc = device_add(&chip->dev);
Not to pile on, but as Linus said, this is totally wrong. Did you test
it? chip->dev is already registered at this point in time...
thanks,
greg k-h
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [GIT PULL] Char/Misc driver patches for 4.12-rc1
2017-05-05 16:33 ` Linus Torvalds
@ 2017-05-05 16:49 ` James Bottomley
0 siblings, 0 replies; 12+ messages in thread
From: James Bottomley @ 2017-05-05 16:49 UTC (permalink / raw)
To: Linus Torvalds
Cc: Greg KH, Jarkko Sakkinen, Andrew Morton, Arnd Bergmann,
Linux Kernel Mailing List
On Fri, 2017-05-05 at 09:33 -0700, Linus Torvalds wrote:
> On Fri, May 5, 2017 at 9:00 AM, James Bottomley
> <James.Bottomley@hansenpartnership.com> wrote:
> >
> > I'm not going to defend the earlier coding, but you've lost the
> > real device_add() calls in the merge, meaning the tpm devices don't
> > actually get made visible at all. I suspect assuming device_add()
> > is done by cdev_device_add() because of the name is going to be our
> > next anti-pattern, so you're at least ahead of the game ...
>
> Don't be silly. That's *exactly* what cdev_device_add() does.
Oh, you're right, sorry, I was going by the patches that were cc'd to
the TPM list, which only had the two line addition for
cdev_device_add() ... apparently no-one sent any updates. Yes, the new
function looks much better and the resolution works. Sorry for the
noise.
James
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [GIT PULL] Char/Misc driver patches for 4.12-rc1
2017-05-05 16:38 ` Greg KH
@ 2017-05-05 19:46 ` James Bottomley
2017-05-05 20:01 ` Linus Torvalds
1 sibling, 0 replies; 12+ messages in thread
From: James Bottomley @ 2017-05-05 19:46 UTC (permalink / raw)
To: Greg KH
Cc: Linus Torvalds, Jarkko Sakkinen, Andrew Morton, Arnd Bergmann,
Linux Kernel Mailing List
On Fri, 2017-05-05 at 09:38 -0700, Greg KH wrote:
> On Fri, May 05, 2017 at 09:00:06AM -0700, James Bottomley wrote:
> > On Thu, 2017-05-04 at 19:28 -0700, Linus Torvalds wrote:
> > > On Thu, May 4, 2017 at 5:18 PM, Greg KH <
> > > gregkh@linuxfoundation.org>
> > > wrote:
> > > >
> > > > Here is the big set of new char/misc driver drivers and
> > > > features
> > > > for 4.12-rc1.
> > >
> > > Ugh. I'm not particularly happy with the conflicts I got and my
> > > resolutions there-of.
> >
> > Yes, we really should have done this via a postmerge tree. We've
> > had so little cause to use them recently, I suspect everyone's
> > forgotten how.
>
> Huh? You could have pulled in my tree into this one, or I could have
> done that for you, my trees are not rebased at all, and they get used
> this way every other release or so for this very reason.
Well, I think this is the process issue, isn't it? You're telling me I
could have run the tree or you could (actually, you don't mean me: this
is a security subsystem where Jarkko sends a pull to James Morris who
sends it on to Linus so it would be JamesM running it). I'm asking the
question why we didn't. I think we didn't because once Stephen worked
out the merge commit everyone forgot about the problem.
So the first process question is: "is what we did what should happen".
We can say yes and stop here: linux-next did the merges with conflict
resolution. The osd initial merge was wrong, so we went back on linux
-scsi and had it corrected and I remembered to send in the resolution
from linux-next with the SCSI pull request so as not to have the
obvious but wrong resolution problem happen again. The same thing
could be done with the security tree. I think it's easy to conclude
that this is probably good enough.
If we have more of a desire to avoid tree conflicts, then postmerge
trees could be the answer, but in in this case, since changing the
existing API users to the new API (which is where all the nasty
conflicts were) weren't really time critical, the code could have been
staged as a new API plus patches that use it in one cycle and
conversions of existing users in a second. The later could have gone
through the proper Maintainer trees and we'd never have seen a conflict
in any of the subsystems.
James
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [GIT PULL] Char/Misc driver patches for 4.12-rc1
2017-05-05 16:38 ` Greg KH
2017-05-05 19:46 ` James Bottomley
@ 2017-05-05 20:01 ` Linus Torvalds
2017-05-06 5:09 ` Stephen Rothwell
1 sibling, 1 reply; 12+ messages in thread
From: Linus Torvalds @ 2017-05-05 20:01 UTC (permalink / raw)
To: Greg KH
Cc: James Bottomley, Jarkko Sakkinen, Andrew Morton, Arnd Bergmann,
Linux Kernel Mailing List
On Fri, May 5, 2017 at 9:38 AM, Greg KH <gregkh@linuxfoundation.org> wrote:
> On Fri, May 05, 2017 at 09:00:06AM -0700, James Bottomley wrote:
>> On Thu, 2017-05-04 at 19:28 -0700, Linus Torvalds wrote:
>> >
>> > Ugh. I'm not particularly happy with the conflicts I got and my
>> > resolutions there-of.
>>
>> Yes, we really should have done this via a postmerge tree. We've had
>> so little cause to use them recently, I suspect everyone's forgotten
>> how.
>
> Huh? You could have pulled in my tree into this one, or I could have
> done that for you, my trees are not rebased at all, and they get used
> this way every other release or so for this very reason.
I actually would have preferred to not get any early merges, but what
I was unhappy about is that I also didn't really get any heads-up
about the cdev_device_add() conflict.
I did get notified about the other conflict (thanks, James), but
somehow the cdev_device_add() changes didn't cause the same kind of
notification.
So my unhappiness is not about me having to resolve things (I'm happy
to do that) but about how apparently -next failed to notice that part
of my merge resolution. Or maybe it was noticed in -next, but then the
information about it got lost.
I prefer doing merge resolutions myself, but I *also* really really
prefer the two sides of the conflict having been more aware of the
clash.
Linus
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [GIT PULL] Char/Misc driver patches for 4.12-rc1
2017-05-05 20:01 ` Linus Torvalds
@ 2017-05-06 5:09 ` Stephen Rothwell
2017-05-06 18:00 ` Linus Torvalds
0 siblings, 1 reply; 12+ messages in thread
From: Stephen Rothwell @ 2017-05-06 5:09 UTC (permalink / raw)
To: Linus Torvalds
Cc: Greg KH, James Bottomley, Jarkko Sakkinen, Andrew Morton,
Arnd Bergmann, Linux Kernel Mailing List
Hi Linus,
On Fri, 5 May 2017 13:01:34 -0700 Linus Torvalds <torvalds@linux-foundation.org> wrote:
>
> I actually would have preferred to not get any early merges, but what
> I was unhappy about is that I also didn't really get any heads-up
> about the cdev_device_add() conflict.
>
> I did get notified about the other conflict (thanks, James), but
> somehow the cdev_device_add() changes didn't cause the same kind of
> notification.
>
> So my unhappiness is not about me having to resolve things (I'm happy
> to do that) but about how apparently -next failed to notice that part
> of my merge resolution. Or maybe it was noticed in -next, but then the
> information about it got lost.
>
> I prefer doing merge resolutions myself, but I *also* really really
> prefer the two sides of the conflict having been more aware of the
> clash.
Would that be this?
From: Stephen Rothwell <sfr@canb.auug.org.au>
To: Greg KH <greg@kroah.com>, Arnd Bergmann <arnd@arndb.de>, Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
Cc: linux-next@vger.kernel.org, linux-kernel@vger.kernel.org, Logan Gunthorpe <logang@deltatee.com>, James Bottomley <James.Bottomley@HansenPartnership.com>
Subject: linux-next: manual merge of the char-misc tree with the tpmdd tree
Date: Fri, 24 Mar 2017 14:33:02 +1100
Hi all,
Today's linux-next merge of the char-misc tree got a conflict in:
drivers/char/tpm/tpm-chip.c
between commits:
67b67480db8b ("tpm: infrastructure for TPM spaces")
b8e3586e8536 ("tpm: expose spaces via a device link /dev/tpmrm<n>")
from the tpmdd tree and commit:
8dbbf5825181 ("tpm-chip: utilize new cdev_device_add helper function")
from the char-misc tree.
I fixed it up (see below) and can carry the fix as necessary. This
is now fixed as far as linux-next is concerned, but any non trivial
conflicts should be mentioned to your upstream maintainer when your tree
is submitted for merging. You may also want to consider cooperating
with the maintainer of the conflicting tree to minimise any particularly
complex conflicts.
--
Cheers,
Stephen Rothwell
diff --cc drivers/char/tpm/tpm-chip.c
index aade6995f310,935f0e92ad61..000000000000
--- a/drivers/char/tpm/tpm-chip.c
+++ b/drivers/char/tpm/tpm-chip.c
@@@ -214,22 -186,7 +214,20 @@@ struct tpm_chip *tpm_chip_alloc(struct
chip->flags |= TPM_CHIP_FLAG_VIRTUAL;
cdev_init(&chip->cdev, &tpm_fops);
+ cdev_init(&chip->cdevs, &tpmrm_fops);
chip->cdev.owner = THIS_MODULE;
+ chip->cdevs.owner = THIS_MODULE;
- chip->cdev.kobj.parent = &chip->dev.kobj;
- chip->cdevs.kobj.parent = &chip->devs.kobj;
+
+ chip->work_space.context_buf = kzalloc(PAGE_SIZE, GFP_KERNEL);
+ if (!chip->work_space.context_buf) {
+ rc = -ENOMEM;
+ goto out;
+ }
+ chip->work_space.session_buf = kzalloc(PAGE_SIZE, GFP_KERNEL);
+ if (!chip->work_space.session_buf) {
+ rc = -ENOMEM;
+ goto out;
+ }
return chip;
@@@ -273,44 -229,13 +271,22 @@@ static int tpm_add_char_device(struct t
{
int rc;
- rc = cdev_add(&chip->cdev, chip->dev.devt, 1);
+ rc = cdev_device_add(&chip->cdev, &chip->dev);
if (rc) {
dev_err(&chip->dev,
- "unable to cdev_add() %s, major %d, minor %d, err=%d\n",
+ "unable to cdev_device_add() %s, major %d, minor %d, err=%d\n",
dev_name(&chip->dev), MAJOR(chip->dev.devt),
MINOR(chip->dev.devt), rc);
+ return rc;
+ }
- rc = device_add(&chip->dev);
- if (rc) {
- dev_err(&chip->dev,
- "unable to device_register() %s, major %d, minor %d, err=%d\n",
- dev_name(&chip->dev), MAJOR(chip->dev.devt),
- MINOR(chip->dev.devt), rc);
-
- cdev_del(&chip->cdev);
- return rc;
- }
-
- if (chip->flags & TPM_CHIP_FLAG_TPM2)
- rc = cdev_add(&chip->cdevs, chip->devs.devt, 1);
- if (rc) {
- dev_err(&chip->dev,
- "unable to cdev_add() %s, major %d, minor %d, err=%d\n",
- dev_name(&chip->devs), MAJOR(chip->devs.devt),
- MINOR(chip->devs.devt), rc);
- return rc;
- }
-
+ if (chip->flags & TPM_CHIP_FLAG_TPM2)
- rc = device_add(&chip->devs);
++ rc = cdev_device_add(&chip->cdevs, &chip->devs);
+ if (rc) {
+ dev_err(&chip->dev,
- "unable to device_register() %s, major %d, minor %d, err=%d\n",
++ "unable to cdev_device_add() %s, major %d, minor %d, err=%d\n",
+ dev_name(&chip->devs), MAJOR(chip->devs.devt),
+ MINOR(chip->devs.devt), rc);
- cdev_del(&chip->cdevs);
return rc;
}
@@@ -447,10 -371,6 +422,8 @@@ void tpm_chip_unregister(struct tpm_chi
{
tpm_del_legacy_sysfs(chip);
tpm_bios_log_teardown(chip);
- if (chip->flags & TPM_CHIP_FLAG_TPM2) {
- cdev_del(&chip->cdevs);
- device_del(&chip->devs);
- }
++ if (chip->flags & TPM_CHIP_FLAG_TPM2)
++ cdev_device_del(&chip->cdevs, &chip->devs);
tpm_del_char_device(chip);
}
EXPORT_SYMBOL_GPL(tpm_chip_unregister);
--
Cheers,
Stephen Rothwell
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [GIT PULL] Char/Misc driver patches for 4.12-rc1
2017-05-06 5:09 ` Stephen Rothwell
@ 2017-05-06 18:00 ` Linus Torvalds
2017-05-06 18:12 ` James Bottomley
0 siblings, 1 reply; 12+ messages in thread
From: Linus Torvalds @ 2017-05-06 18:00 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Greg KH, James Bottomley, Jarkko Sakkinen, Andrew Morton,
Arnd Bergmann, Linux Kernel Mailing List
On Fri, May 5, 2017 at 10:09 PM, Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>
> On Fri, 5 May 2017 13:01:34 -0700 Linus Torvalds <torvalds@linux-foundation.org> wrote:
>>
>>
>> I prefer doing merge resolutions myself, but I *also* really really
>> prefer the two sides of the conflict having been more aware of the
>> clash.
>
> Would that be this?
Yup. Apparently neither Greg nor James ended up reacting to that
email, though, so by the time I got the pull requests there was no
mention of it anywhere.
Linus
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [GIT PULL] Char/Misc driver patches for 4.12-rc1
2017-05-06 18:00 ` Linus Torvalds
@ 2017-05-06 18:12 ` James Bottomley
2017-07-12 4:50 ` Stephen Rothwell
0 siblings, 1 reply; 12+ messages in thread
From: James Bottomley @ 2017-05-06 18:12 UTC (permalink / raw)
To: Linus Torvalds, Stephen Rothwell
Cc: Greg KH, Jarkko Sakkinen, Andrew Morton, Arnd Bergmann,
Linux Kernel Mailing List
On Sat, 2017-05-06 at 11:00 -0700, Linus Torvalds wrote:
> On Fri, May 5, 2017 at 10:09 PM, Stephen Rothwell <
> sfr@canb.auug.org.au> wrote:
> >
> > On Fri, 5 May 2017 13:01:34 -0700 Linus Torvalds <
> > torvalds@linux-foundation.org> wrote:
> > >
> > >
> > > I prefer doing merge resolutions myself, but I *also* really
> > > really prefer the two sides of the conflict having been more
> > > aware of the clash.
> >
> > Would that be this?
>
> Yup. Apparently neither Greg nor James ended up reacting to that
> email, though,
Yes, we did, but for the one in SCSI ... as I said the original
conflict resolution with our tree was eventually found to be slightly
wrong so there was an email thread over it.
There's not much I can do about the one in tpmdd-devel because it's not
my tree. Even Jarkko can't do much more than tell James Morris for the
Security tree, and I think this came up after it had already been
pulled into that tree.
> so by the time I got the pull requests there was no
> mention of it anywhere.
Well, there was in the SCSI pull request, but the only reason I
remembered is because I'd made a special note of the potential resolve
problem when this came up on the SCSI mailing list. The original merge
conflict email came 6 weeks before the merge window, which is why
everyone had had time to forget.
What about resending the conflict reminders at -rc7 ... that way we
only have a week or two to forget again?
The other issue is that one of the potential trees only got notified
directly (the char-misc one) because the tpmdd tree takes an indirect
pull route. I'm not sure what we can do about this one.
James
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [GIT PULL] Char/Misc driver patches for 4.12-rc1
2017-05-06 18:12 ` James Bottomley
@ 2017-07-12 4:50 ` Stephen Rothwell
0 siblings, 0 replies; 12+ messages in thread
From: Stephen Rothwell @ 2017-07-12 4:50 UTC (permalink / raw)
To: James Bottomley
Cc: Linus Torvalds, Greg KH, Jarkko Sakkinen, Andrew Morton,
Arnd Bergmann, Linux Kernel Mailing List
Hi all,
On Sat, 06 May 2017 11:12:36 -0700 James Bottomley <James.Bottomley@HansenPartnership.com> wrote:
>
> What about resending the conflict reminders at -rc7 ... that way we
> only have a week or two to forget again?
OK, I attempted to do that this time ... I wonder if it made a
difference :-)
> The other issue is that one of the potential trees only got notified
> directly (the char-misc one) because the tpmdd tree takes an indirect
> pull route. I'm not sure what we can do about this one.
I have also tried (in most cases) to notify the effected maintainers
that ask Linus to pull.
--
Cheers,
Stephen Rothwell
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2017-07-12 4:50 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-05-05 0:18 [GIT PULL] Char/Misc driver patches for 4.12-rc1 Greg KH
2017-05-05 2:28 ` Linus Torvalds
2017-05-05 16:00 ` James Bottomley
2017-05-05 16:33 ` Linus Torvalds
2017-05-05 16:49 ` James Bottomley
2017-05-05 16:38 ` Greg KH
2017-05-05 19:46 ` James Bottomley
2017-05-05 20:01 ` Linus Torvalds
2017-05-06 5:09 ` Stephen Rothwell
2017-05-06 18:00 ` Linus Torvalds
2017-05-06 18:12 ` James Bottomley
2017-07-12 4:50 ` Stephen Rothwell
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.