Re: [PATCH v6 4/5] arm64: dts: sdm845: Add serial console support

2018-05-23 Thread Doug Anderson
Rajendra, On Tue, May 22, 2018 at 11:30 PM, Rajendra Nayak wrote: > > > On 03/30/2018 10:38 PM, Karthikeyan Ramasubramanian wrote: >> From: Rajendra Nayak >> >> Add the qup uart node and geni se instance needed to >> support the serial console on

Re: [PATCH v5 3/5] i2c: i2c-qcom-geni: Add bus driver for the Qualcomm GENI I2C controller

2018-05-21 Thread Doug Anderson
Wolfram, On Fri, Mar 23, 2018 at 4:34 PM, Doug Anderson <diand...@chromium.org> wrote: > Hi, > > On Fri, Mar 23, 2018 at 1:20 PM, Karthikeyan Ramasubramanian > <krama...@codeaurora.org> wrote: >> This bus driver supports the GENI based i2c hardware controller in the

Re: [PATCH v5 5/5] arm64: dts: sdm845: Add support for an instance of I2C controller

2018-03-23 Thread Doug Anderson
Hi, On Fri, Mar 23, 2018 at 1:21 PM, Karthikeyan Ramasubramanian wrote: > + i2c10: i2c@a88000 { > + compatible = "qcom,geni-i2c"; > + reg = <0xa88000 0x4000>; > +

Re: [PATCH v5 3/5] i2c: i2c-qcom-geni: Add bus driver for the Qualcomm GENI I2C controller

2018-03-23 Thread Doug Anderson
Hi, On Fri, Mar 23, 2018 at 1:20 PM, Karthikeyan Ramasubramanian wrote: > This bus driver supports the GENI based i2c hardware controller in the > Qualcomm SOCs. The Qualcomm Generic Interface (GENI) is a programmable > module supporting a wide range of serial interfaces

Re: [PATCH v4 6/6] arm64: dts: sdm845: Add I2C controller support

2018-03-20 Thread Doug Anderson
Hi, On Tue, Mar 20, 2018 at 3:16 PM, Sagar Dharia wrote: > + pinconf { > + pins = "gpio55", "gpio56"; > + drive-strength = <2>; > +

Re: [PATCH v4 6/6] arm64: dts: sdm845: Add I2C controller support

2018-03-19 Thread Doug Anderson
Hi, On Mon, Mar 19, 2018 at 3:15 PM, Sagar Dharia wrote: >>> + pinconf { >>> + pins = "gpio55", "gpio56"; >>> + drive-strength = <2>; >>> + bias-disable; >>> +

Re: [PATCH v4 3/6] i2c: i2c-qcom-geni: Add bus driver for the Qualcomm GENI I2C controller

2018-03-19 Thread Doug Anderson
Hi, On Wed, Mar 14, 2018 at 4:58 PM, Karthikeyan Ramasubramanian wrote: > This bus driver supports the GENI based i2c hardware controller in the > Qualcomm SOCs. The Qualcomm Generic Interface (GENI) is a programmable > module supporting a wide range of serial interfaces

Re: [PATCH v4 6/6] arm64: dts: sdm845: Add I2C controller support

2018-03-16 Thread Doug Anderson
Hi, On Wed, Mar 14, 2018 at 4:58 PM, Karthikeyan Ramasubramanian wrote: > Add I2C master controller support for a built-in test I2C slave. > > Signed-off-by: Karthikeyan Ramasubramanian > --- > arch/arm64/boot/dts/qcom/sdm845-mtp.dts | 19

Re: [v3, 3/4] i2c: i2c-qcom-geni: Add bus driver for the Qualcomm GENI I2C controller

2018-03-08 Thread Doug Anderson
Hi, On Thu, Mar 8, 2018 at 5:06 PM, Sagar Dharia <sdha...@codeaurora.org> wrote: > Hi Doug > > > On 3/8/2018 2:12 PM, Doug Anderson wrote: >> >> Hi, >> >> On Wed, Mar 7, 2018 at 9:19 PM, Doug Anderson <diand...@chromium.org> >> wrote: >>

Re: [v3, 3/4] i2c: i2c-qcom-geni: Add bus driver for the Qualcomm GENI I2C controller

2018-03-08 Thread Doug Anderson
Hi, On Wed, Mar 7, 2018 at 9:19 PM, Doug Anderson <diand...@chromium.org> wrote: >>> DMA is hard and i2c transfers > 32 bytes are rare. Do we really gain >>> a lot by transferring i2c commands over DMA compared to a FIFO? >>> Enough to justify the

Re: [v3, 3/4] i2c: i2c-qcom-geni: Add bus driver for the Qualcomm GENI I2C controller

2018-03-07 Thread Doug Anderson
Hi, On Wed, Mar 7, 2018 at 6:42 PM, Sagar Dharia wrote: > Hi Doug, > Thank you for reviewing the patch. I will take a stab at a few comments > below. We will address most of the other comments in next version of I2C > patch. >> >> >> >>> + >>> +#define

Re: [v3, 3/4] i2c: i2c-qcom-geni: Add bus driver for the Qualcomm GENI I2C controller

2018-03-07 Thread Doug Anderson
Hi, On Tue, Feb 27, 2018 at 5:38 PM, Karthikeyan Ramasubramanian wrote: > This bus driver supports the GENI based i2c hardware controller in the > Qualcomm SOCs. The Qualcomm Generic Interface (GENI) is a programmable > module supporting a wide range of serial interfaces

Re: [v2, 7/7] tty: serial: msm_geni_serial: Add serial driver support for GENI based QUP

2018-02-23 Thread Doug Anderson
Hi, On Fri, Jan 12, 2018 at 5:05 PM, Karthikeyan Ramasubramanian wrote: > This driver supports GENI based UART Controller in the Qualcomm SOCs. The > Qualcomm Generic Interface (GENI) is a programmable module supporting a > wide range of serial interfaces including UART.

Re: [RFC PATCH] kbuild: Allow specifying some base host CFLAGS

2017-10-19 Thread Doug Anderson
Hi, On Wed, Oct 18, 2017 at 9:45 AM, Masahiro Yamada wrote: > 2017-10-14 3:02 GMT+09:00 Douglas Anderson : >> Right now there is a way to add some CFLAGS that affect target builds, >> but no way to add CFLAGS that affect host builds. Let's

Re: [PATCH v2 1/2] kbuild: Add a cache for generated variables

2017-10-11 Thread Doug Anderson
Hi, On Tue, Oct 10, 2017 at 8:59 PM, Masahiro Yamada wrote: > Hi Douglas, > > > 2017-10-05 7:37 GMT+09:00 Douglas Anderson : >> While timing a "no-op" build of the kernel (incrementally building the >> kernel even though nothing changed) in

Re: [RFC PATCH 1/2] kbuild: Add a cache for generated variables

2017-10-05 Thread Doug Anderson
Hi, On Thu, Oct 5, 2017 at 12:26 AM, Masahiro Yamada wrote: > As far as I tested, I always see only one space after ":=" in v2. > > I did not consider this deeply, > but something is working nicely behind the scene. Try adding this to the end of the main Makefile:

Re: [RFC PATCH 1/2] kbuild: Add a cache for generated variables

2017-10-04 Thread Doug Anderson
Hi, On Tue, Oct 3, 2017 at 9:05 PM, Masahiro Yamada wrote: > Thanks for the patches. The idea is interesting. > > I am not a Chrome developer, but cc-option could be improved somehow. > > > I examined two approaches to mitigate the pain. > > [1] Skip cc-option