Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S975990AbdDXRSN (ORCPT ); Mon, 24 Apr 2017 13:18:13 -0400 Received: from mail-qk0-f174.google.com ([209.85.220.174]:36023 "EHLO mail-qk0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S975932AbdDXRSC (ORCPT ); Mon, 24 Apr 2017 13:18:02 -0400 MIME-Version: 1.0 In-Reply-To: <402abd5e-9357-84e8-56fe-8758d119a0b6@gmail.com> References: <402abd5e-9357-84e8-56fe-8758d119a0b6@gmail.com> From: Guenter Roeck Date: Mon, 24 Apr 2017 10:18:00 -0700 Message-ID: Subject: Re: [PATCH] usb: typec: Don't prevent using constant typec_mode_desc initializers To: Mats Karrman Cc: Heikki Krogerus , Felipe Balbi , Guenter Roeck , linux-usb@vger.kernel.org, linux-kernel Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3407 Lines: 97 Hi Mats, On Mon, Apr 24, 2017 at 7:24 AM, Mats Karrman wrote: > From 98ff2be4de83f4d4b5293bfa424c274b25400e1e Mon Sep 17 00:00:00 2001 > From: Mats Karrman > Date: Mon, 24 Apr 2017 15:21:39 +0200 > Subject: [PATCH] usb: typec: Don't prevent using constant typec_mode_desc > initializers > > --- > drivers/usb/typec/typec.c | 11 ++++++----- > include/linux/usb/typec.h | 6 +++--- > 2 files changed, 9 insertions(+), 8 deletions(-) > > diff --git a/drivers/usb/typec/typec.c b/drivers/usb/typec/typec.c > index 89e540b..db5ee73 100644 > --- a/drivers/usb/typec/typec.c > +++ b/drivers/usb/typec/typec.c > @@ -291,7 +291,7 @@ typec_altmode_roles_show(struct device *dev, struct > device_attribute *attr, > } > > static void typec_init_modes(struct typec_altmode *alt, > - struct typec_mode_desc *desc, bool is_port) > + const struct typec_mode_desc *desc, bool is_port) > { > int i; > > @@ -378,7 +378,8 @@ static const struct device_type typec_altmode_dev_type = > { > }; > > static struct typec_altmode * > -typec_register_altmode(struct device *parent, struct typec_altmode_desc > *desc) > +typec_register_altmode(struct device *parent, > + const struct typec_altmode_desc *desc) Unfortunately, this patch is whitespace damaged. Thanks, Guenter > { > struct typec_altmode *alt; > int ret; > @@ -495,7 +496,7 @@ EXPORT_SYMBOL_GPL(typec_partner_set_identity); > */ > struct typec_altmode * > typec_partner_register_altmode(struct typec_partner *partner, > - struct typec_altmode_desc *desc) > + const struct typec_altmode_desc *desc) > { > return typec_register_altmode(&partner->dev, desc); > } > @@ -590,7 +591,7 @@ static const struct device_type typec_plug_dev_type = { > */ > struct typec_altmode * > typec_plug_register_altmode(struct typec_plug *plug, > - struct typec_altmode_desc *desc) > + const struct typec_altmode_desc *desc) > { > return typec_register_altmode(&plug->dev, desc); > } > @@ -1159,7 +1160,7 @@ EXPORT_SYMBOL_GPL(typec_set_pwr_opmode); > */ > struct typec_altmode * > typec_port_register_altmode(struct typec_port *port, > - struct typec_altmode_desc *desc) > + const struct typec_altmode_desc *desc) > { > return typec_register_altmode(&port->dev, desc); > } > diff --git a/include/linux/usb/typec.h b/include/linux/usb/typec.h > index ec78204..d1d2ebc 100644 > --- a/include/linux/usb/typec.h > +++ b/include/linux/usb/typec.h > @@ -117,13 +117,13 @@ struct typec_altmode_desc { > > struct typec_altmode > *typec_partner_register_altmode(struct typec_partner *partner, > - struct typec_altmode_desc *desc); > + const struct typec_altmode_desc *desc); > struct typec_altmode > *typec_plug_register_altmode(struct typec_plug *plug, > - struct typec_altmode_desc *desc); > + const struct typec_altmode_desc *desc); > struct typec_altmode > *typec_port_register_altmode(struct typec_port *port, > - struct typec_altmode_desc *desc); > + const struct typec_altmode_desc *desc); > void typec_unregister_altmode(struct typec_altmode *altmode); > > struct typec_port *typec_altmode2port(struct typec_altmode *alt); > -- > 2.1.4 >