Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751610AbdFIKUw (ORCPT ); Fri, 9 Jun 2017 06:20:52 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:50548 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751519AbdFIKUv (ORCPT ); Fri, 9 Jun 2017 06:20:51 -0400 Date: Fri, 9 Jun 2017 12:20:43 +0200 From: Greg Kroah-Hartman To: Heiko Carstens Cc: linux-kernel@vger.kernel.org, Martin Schwidefsky , Julian Wiedmann , Ursula Braun , Peter Oberparleiter , Gerald Schaefer , linux-s390@vger.kernel.org Subject: Re: [PATCH 10/11] s390: drivers: convert to use DRIVER_ATTR_RO/WO Message-ID: <20170609102043.GA11061@kroah.com> References: <20170609090314.13991-1-gregkh@linuxfoundation.org> <20170609090314.13991-10-gregkh@linuxfoundation.org> <20170609101141.GB3617@osiris> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170609101141.GB3617@osiris> User-Agent: Mutt/1.8.3 (2017-05-23) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1260 Lines: 32 On Fri, Jun 09, 2017 at 12:11:41PM +0200, Heiko Carstens wrote: > On Fri, Jun 09, 2017 at 11:03:13AM +0200, Greg Kroah-Hartman wrote: > > We are trying to get rid of DRIVER_ATTR(), and the s390 drivers' > > attributes can be trivially changed to use DRIVER_ATTR_RO() and > > DRIVER_ATTR_WO(). > > > > Cc: Martin Schwidefsky > > Cc: Heiko Carstens > > Cc: Julian Wiedmann > > Cc: Ursula Braun > > Cc: Peter Oberparleiter > > Cc: Gerald Schaefer > > Cc: > > Signed-off-by: Greg Kroah-Hartman > > --- > > drivers/s390/char/sclp.c | 12 ++++++------ > > drivers/s390/char/vmlogrdr.c | 7 ++----- > > drivers/s390/net/ctcm_main.c | 6 +++--- > > drivers/s390/net/lcs.c | 6 +++--- > > drivers/s390/net/netiucv.c | 14 ++++++-------- > > drivers/s390/net/qeth_core_main.c | 6 +++--- > > 6 files changed, 23 insertions(+), 28 deletions(-) > > I assume this will go upstream via your tree. Yup, I can take it. > Acked-by: Heiko Carstens Thanks for the review! greg k-h