Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Thu, 31 Mar 2022 17:23:23 +0300
From:   Dan Carpenter <dan.carpenter@oracle.com>
To:     Bryan Brattlof <bb@ti.com>
Cc:     Alaa Mohamed <eng.alaamohamedsoliman.am@gmail.com>,
        outreachy@lists.linux.dev, gregkh@linuxfoundation.org,
        linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] staging: rtl8723bs: Fix No space before tabs
Message-ID: <20220331142323.GQ3293@kadam>
References: <20220331130948.74835-1-eng.alaamohamedsoliman.am@gmail.com>
 <20220331133744.lypehhmz2wxngvzl@bryanbrattlof.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20220331133744.lypehhmz2wxngvzl@bryanbrattlof.com>
User-Agent: Mutt/1.9.4 (2018-02-28)
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?gbJdadM1wY1dWFgrlN9Doo1YKkt5Q0gVBITW8JEsg+I3H+tu2ao1WonOOXTJ?=
 =?us-ascii?Q?arBBL6JQ94qyq0ko4To1gh4FdwBrKKh79iALd/uk7Ca+OycShtSeZmZ5fxfJ?=
 =?us-ascii?Q?IsU4Eag3gzdXSDIbvfM6SrT0BOnzl5zHFQfxu7qNZeJY566i3LemKtutAir2?=
 =?us-ascii?Q?nykTvB2sgmvakpC/OaRYS16AbV7zXcmFbm6Al2gS8AY8bBSfzDOirU88hZKH?=
 =?us-ascii?Q?ec5q2p06E98lWF2KnCClnExKqFwtb4K/hrAoqgaFZhjZAANnpFY0aKZIHpfh?=
 =?us-ascii?Q?51wZp5D5rY6PJFEHg4hviGLTvRHFy0Z2EKMgSeo4b5XcJY2qZfmKOZ7qobQ5?=
 =?us-ascii?Q?mqXfy3acjw6157D3OJ8weUcSxpLwV9c/alO6sHeo2EJ3B9HMNtyJFilPzDgI?=
 =?us-ascii?Q?12LA2982cDSLdQXwPkVrwt6KIaQCtm63IigRRX9VDYEGaGepW1gKYJ2LRpVR?=
 =?us-ascii?Q?UCc2Dy5REwguY8cId7hBxywrZA4R6CNhFOf9+0gnr8gr0/cw2YxbkU5g3ga3?=
 =?us-ascii?Q?p818s8S5QgQw3PfkVzDxcaTUqQMJWBRajecbbLN21eTYBExMibls1kTL6yET?=
 =?us-ascii?Q?8sHabiGXk2vy0KmVD6jrwIG2HCdOZuAe/o/yaT4dftgi99N/O4fgIVNr8fOX?=
 =?us-ascii?Q?638c3tPph7LI2FiO2wLgchOKBL5I5a/MkGbnbM9YHCjajfiaTAw/0kkef0SJ?=
 =?us-ascii?Q?TMTEukHPTsr1eZwwh/6ybeerfqwBiqJ/AokiXcDN7fP8dCFvxZ0X2cKg0549?=
 =?us-ascii?Q?HeDebzn6a95QINm9yPoPUDMR6C4u3IGlU61NFY/tBaU5EBuIm5hL3Bz/yWMl?=
 =?us-ascii?Q?NbEcgqinuAr/qgnruq0s4B2g7Yy+2UlVhcPrNiKiwcLJ8EVjDargzZq72kRj?=
 =?us-ascii?Q?7znJ3g8nHXXx8fe/oJ8K4rSmU26f93stCU2JOmM+Cm88dg7VU1fryG+nm/PE?=
 =?us-ascii?Q?yugMVdTPgOZcP2Aun1g1DYW1HoaKHEw86xhvC4ncpdTY7/TKnxcU3rstfWRu?=
 =?us-ascii?Q?fd6lh/Ssob/vxCVzYY+5LgZg1L1GZpxHD7JKefXU4ZDBiyvgRol9HZAztmDA?=
 =?us-ascii?Q?kPs6ddppekl0aJ0Ftv00FccMoWHrCiWFYZejLsdtO1QsEVUTAS9Kjn3+pM7A?=
 =?us-ascii?Q?hJPX+u/pv9UBKI2EFeJZw8KsMXL9BB8Cn07jGORFwNW+3o7P+kEiWNUG8zCT?=
 =?us-ascii?Q?D6Y9imEiwbX80TNqQkp7Rr6FuccV4DQ13IEkEfofmOiB7EutBEGO4P7pKB8C?=
 =?us-ascii?Q?rigV7fctJ98jIYRpkRWC38jwhq4UFLvIfwQ5h1HAA8KoKTMXFqjJrokGTkhc?=
 =?us-ascii?Q?KFi1TzDSaAW6qkl/s8fn1r8x4Rb63TBcnhpTKQ0cXyFlWb/ppUOz8J3wfmzx?=
 =?us-ascii?Q?j2Fwd3Nrqy3MCeBjSe7qk65dfeutKuzwdm/AWIXiNY9/HeA8SHRKAiPE5sHh?=
 =?us-ascii?Q?av1d7Zr7qrn6zrqD9JyP6WLJz9KtoY7wsuVMhhvGA7kL+h52nyZJMSHzNble?=
 =?us-ascii?Q?bJDTTMzPEh5Dd2g+AjzYoCxTqHrW4sjWrp1jj54aTbc4F8I/M+uc6kyXcyW9?=
 =?us-ascii?Q?kMZdNz8jx1p8xca3AWwE6Y6kuW7DeSEVQ2/L1MhRKgl67Gx6AjtwhdXMDhL5?=
 =?us-ascii?Q?v1XhgbcdxmODmU9ZENZrHNl/3in38sdzj2A6mOFqM6/Ew46hCVK3wXpwZYPa?=
 =?us-ascii?Q?ukjpuzBzkV5AvFvV7qXqFPeg1wv14/Nr/5aBKVUZojiuUjrY3OjUFNmfg4l6?=
 =?us-ascii?Q?hXEAPQZfcnIb2IhXqP72NNINcdLhczw=3D?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b704d7e4-c39b-47ba-4251-08da13220f0b
X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2365.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 31 Mar 2022 14:23:43.4339
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: PYc0EFmY6Q0bram9ScrgtyfGaqaoSfVnCRqBtnTrvj5qUtYJTisFGsNJC0ME8W7yTRiZrkIGkQgKkX4uPLkU0vNXCyVn++KlCJmoVB2ZD58=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR10MB1632
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.425,18.0.850
 definitions=2022-03-31_05:2022-03-30,2022-03-31 signatures=0
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 malwarescore=0 adultscore=0 mlxscore=0
 phishscore=0 spamscore=0 suspectscore=0 bulkscore=0 mlxlogscore=934
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2202240000
 definitions=main-2203310080
X-Proofpoint-ORIG-GUID: b-eNULgfwr2VOYXGCMgv_7htkxGhZkdQ
X-Proofpoint-GUID: b-eNULgfwr2VOYXGCMgv_7htkxGhZkdQ
X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_LOW,
        RCVD_IN_MSPIKE_H5,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,SPF_NONE,
        T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Thu, Mar 31, 2022 at 08:37:44AM -0500, Bryan Brattlof wrote:
> > -/* 	Created by Roger, 2008.10.21. */
> > +/* Created by Roger, 2008.10.21. */
> >  /*  */
> > -/* 	2008/12/12 MH	1. Reorganize code flow and reserve bytes. and add description. */
> > -/* 					2. Add efuse utilization collect. */
> > -/* 	2008/12/22 MH	Read Efuse must check if we write section 1 data again!!! Sec1 */
> > -/* 					write addr must be after sec5. */
> > +/* 2008/12/12 MH	1. Reorganize code flow and reserve bytes. and add description. */
> > +/*					2. Add efuse utilization collect. */
> > +/* 2008/12/22 MH	Read Efuse must check if we write section 1 data again!!! Sec1 */
> > +/*					write addr must be after sec5. */
> >  /*  */
> >
> 
> I'm wondering if it would be OK if we removed the change history and
> tried to align the indentation a little. (This may need to be multiple
> patches depending on how everyone else feels)

Yep.  Feel free to delete the historical info.

regards,
dan carpenter