Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753952AbbFVDrx (ORCPT ); Sun, 21 Jun 2015 23:47:53 -0400 Received: from mailout2.samsung.com ([203.254.224.25]:50540 "EHLO mailout2.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752231AbbFVDrn (ORCPT ); Sun, 21 Jun 2015 23:47:43 -0400 X-AuditID: cbfee68f-f793b6d000005f66-44-5587855caa46 Date: Mon, 22 Jun 2015 03:47:40 +0000 (GMT) From: Maninder Singh Subject: Re: [PATCH 1/1] ssb: remove unncessary out label To: =?windows-1252?Q?Michael_B=3Fch?= Cc: "netdev@vger.kernel.org" , "linux-kernel@vger.kernel.org" , PANKAJ MISHRA Reply-to: maninder1.s@samsung.com MIME-version: 1.0 X-MTR: 20150622034340939@maninder1.s Msgkey: 20150622034340939@maninder1.s X-EPLocale: en_US.windows-1252 X-Priority: 3 X-EPWebmail-Msg-Type: personal X-EPWebmail-Reply-Demand: 0 X-EPApproval-Locale: X-EPHeader: ML X-MLAttribute: X-RootMTR: 20150622034340939@maninder1.s X-ParentMTR: X-ArchiveUser: X-CPGSPASS: N X-ConfirmMail: N,general Content-type: text/plain; charset=windows-1252 MIME-version: 1.0 Message-id: <764435082.28361434944858940.JavaMail.weblogic@ep2mlwas07a> X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFvrCIsWRmVeSWpSXmKPExsWyRsSkRjemtT3U4PUhbovLu+awOTB6fN4k F8AYxWWTkpqTWZZapG+XwJVx/t8e5oJdrBXXvro1MC5h7WLk5BASUJNYtPcxG4gtIWAiMXfe AUYIW0ziwr31QHEuoJqljBJ/7y9jhSl6cfgZI0TzHEaJffdsQWwWAVWJeY8nM4HYbAL6Emf3 rmMGsYUFLCU+bzgFVi8iYCyx4tdMJpChzAILGCWe/exkghikKLH+xhOwIl4BQYmTM5+wQCxT kZj/6hArRFxV4u/N81CXykksmXqZCcLmlZjR/pQFJj7t6xpmCFta4vysDXDfLP7+GCrOL3Hs 9g6oXgGJqWcOQtVoSvw+cxtqPp/EmoVvWWDqd51azgyz6/6WuVC9EhJbW56A3cYMdP+U7ofs ELaBxJFFc1hR/cIBZLtLvNsWD/K7hEAnh8TmnQ9YJzAqzUJSNgvJqFlIRiGrWcDIsopRNLUg uaA4Kb3IWK84Mbe4NC9dLzk/dxMjMDGc/vesfwfj3QPWhxgFOBiVeHgnNLeHCrEmlhVX5h5i NAXG00RmKdHkfGD6ySuJNzQ2M7IwNTE1NjK3NFMS510o9TNYSCA9sSQ1OzW1ILUovqg0J7X4 ECMTB6dUA2PJ96UtiuJOWWu8fvyNV/TasvjcFZVdgk0CK17tnB914rzZ27CDiS/PJW1hMX5d LcF4tHrVd+6lckUcE1PMVPvi5AJ/r94kGWDU1Fz7t7D+Plvu5lMvVO+2zUvJ/B0mrRwTFCpv sO2RJZuPvOXjrQeuPJHX4p25IX1ZzI3mqVurGNrXyB16lqvEUpyRaKjFXFScCACQFzIJBwMA AA== X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFjrKKsWRmVeSWpSXmKPExsVy+t/tXt2Y1vZQgzO/mSwu75rD5sDo8XmT XABjVJpNRmpiSmqRQmpecn5KZl66rZJ3cLxzvKmZgaGuoaWFuZJCXmJuqq2Si0+ArltmDtBQ JYWyxJxSoFBAYnGxkr6dTVF+aUmqQkZ+cYmtUrShuZGekYGeqZGeoWmslaGBgZEpUE1CWsb5 f3uYC3axVlz76tbAuIS1i5GTQ0hATWLR3sdsILaEgInEi8PPGCFsMYkL99azQdTMYZTYd88W xGYRUJWY93gyE4jNJqAvcXbvOmYQW1jAUuLzhlNgvSICxhIrfs0EquHiYBZYwCjx7GcnE8Qg RYn1N56AFfEKCEqcnPmEBWKZisT8V4dYIeKqEn9vnoc6SE5iydTLTBA2r8SM9qcsMPFpX9cw Q9jSEudnbYA7evH3x1Bxfoljt3dA9QpITD1zEKpGU+L3mdtQ8/kk1ix8ywJTv+vUcmaYXfe3 zIXqlZDY2vIE7DZmoPundD9kh7ANJI4smsOK6hcOINtd4t22+AmMsrOQZGYh6Z6FpBtZzQJG llWMoqkFyQXFSekVhnrFibnFpXnpesn5uZsYwUno2cIdjF/OWx9iFOBgVOLhfVDXHirEmlhW XJl7iFGCg1lJhHdzDVCINyWxsiq1KD++qDQntfgQoykw1iYyS4km5wMTZF5JvKGxibmpsamF gaG5uZmSOO//c7khQgLpiSWp2ampBalFMH1MHJxSDYxx7zMdRD7UJZm0na1esaf/zood2rdO H4/WCFsm8Yhlz+HN9ld1wm0tLuboHTpTK/VSx7LutPKls832ncIT5jov0r6wrvYu4/flgkdL vxz99Zr52G3PjOqaA2ImrVa36j7yxn9IK+h8zbVP/RbPCfHXVvMMb8yyuvyOqyPraoXvapG/ gQVxB1iVWIozEg21mIuKEwGu7R1lWAMAAA== DLP-Filter: Pass X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by nfs id t5M3m0A1002700 Content-Length: 565 Lines: 23 Hi Michael, >> pdev = bus->host_pci; >> mutex_init(&bus->sprom_mutex); >> - err = device_create_file(&pdev->dev, &dev_attr_ssb_sprom); >> - if (err) >> - goto out; >> - >> -out: >> - return err; >> + return device_create_file(&pdev->dev, &dev_attr_ssb_sprom); >> } > > >I don't really think this change adds any value, but if you insist on >it you get my acked-by. Yes You are right, But By this change code looks simpler. Thats why i suggested the changes. Thanks Maninder ????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????s?y??杶????i??????????i