linux-kernel - Re: [PATCH v4] scsi: ufs: add support of generic PHY and ICE in Qualcomm chips

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20141204155451.GA29255@infradead.org>
Date:	Thu, 4 Dec 2014 07:54:51 -0800
From:	Christoph Hellwig <hch@...radead.org>
To:	Yaniv Gardi <ygardi@...eaurora.org>
Cc:	James.Bottomley@...senPartnership.com, hch@...radead.org,
	linux-kernel@...r.kernel.org, linux-scsi@...r.kernel.org,
	linux-arm-msm@...r.kernel.org, santoshsy@...il.com,
	linux-scsi-owner@...r.kernel.org, subhashj@...eaurora.org,
	noag@...eaurora.org, draviv@...eaurora.org,
	Rob Herring <robh+dt@...nel.org>,
	Pawel Moll <pawel.moll@....com>,
	Mark Rutland <mark.rutland@....com>,
	Ian Campbell <ijc+devicetree@...lion.org.uk>,
	Kumar Gala <galak@...eaurora.org>,
	Kishon Vijay Abraham I <kishon@...com>,
	Vinayak Holikatti <vinholikatti@...il.com>,
	"James E.J. Bottomley" <JBottomley@...allels.com>,
	Grant Likely <grant.likely@...aro.org>,
	Christoph Hellwig <hch@....de>,
	Sujit Reddy Thumma <sthumma@...eaurora.org>,
	Raviv Shvili <rshvili@...eaurora.org>,
	Sahitya Tummala <stummala@...eaurora.org>,
	"open list:OPEN FIRMWARE AND..." <devicetree@...r.kernel.org>
Subject: Re: [PATCH v4] scsi: ufs: add support of generic PHY and ICE in
 Qualcomm chips

On Thu, Nov 27, 2014 at 05:59:58PM +0200, Yaniv Gardi wrote:
> In this change we add support to the generic PHY framework.
> Two UFS phys are implemented:
> qmp-20nm and qmp-28nm.
> 
> Also, the files in this change implement the UFS HW (controller & PHY)
> specific behavior in Qualcomm chips.
> Relocation of a few header files is needed in order to expose routines
> and data structures between PHY driver and UFS driver.
> 
> Also, this change include the implementation of Inline Crypto Engine (ICE)
> in Qualcomm chips.

This whole patch is a mess.  It does way to many things in one patch,
and it doesn't explain enough of it.

Please explain why you need it.  Especially as the PHY API is a generic
phy abstraction, so having to share defintions between the provider and
consumer seems wrong.  Even if you need some shared bits keep them to an
absolute minium insted of moving so much out of the driver directory.
Also if at all possible keep the shared data in a single header under
include/linux instead of having lots of global headers in a deep
directory structure.

Second split this into patches that do a single things, and explain why
you're doing each:

 1) header move if/as needed
 2) add 20nm phy driver
 3) add 28nm phy driver
 4) add ufs-qcom driver
 5) add ufs-qcom-ice support

and so on.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/